Skip to content

docs: refresh screens map with figma refs - #1237

Merged
ovitrif merged 2 commits into
masterfrom
docs/screens-map-1233
Sep 9, 2026
Merged

docs: refresh screens map with figma refs#1237
ovitrif merged 2 commits into
masterfrom
docs/screens-map-1233

Conversation

@ovitrif

@ovitrif ovitrif commented Sep 8, 2026

Copy link
Copy Markdown
Collaborator

Closes #1233

This PR:

  1. Regenerates docs/screens-map.md from the current *Screen.kt set, grouped by feature package, with a Flow › Frame column pointing at the matching frame on the latest Bitkit - Handoff vNN Figma page
  2. Drops the React Native column, which only served the migration
  3. Adds a unit test that fails when a *Screen.kt is added or removed without a map row. A row is the only requirement: todo marks a screen shipped ahead of its design, n/a a screen with no design by intent
  4. Points AGENTS.md at the map next to the existing Figma rule

Description

The map was last touched on 2026-03-06 and had drifted: 68 of 155 screens had no entry and 5 entries pointed at deleted files. Agent-driven reviews use this file to turn a changed Compose screen into a Figma frame, so every screen now has one row and a stable frame name (node ids change between handoff iterations, names do not). Flow names come from the SectionTitle above each frame on the handoff page. Screens implemented before a design exists carry todo; dev and debug screens carry n/a.

Preview

N/A — no user-visible changes.

QA Notes

Manual Tests

N/A

Automated Checks

  • Unit test added: ScreensMapTest.kt walks app/src/main/java for *Screen.kt files and compares the set with the file names referenced in docs/screens-map.md, reporting missing and stale entries. Verified locally that it passes on this branch and fails when a screen file is moved away.
  • Detekt: no findings in the added test; the remaining reported findings are pre-existing on master.

🤖 Generated with Claude Code

Closes #1233

Regenerate docs/screens-map.md from the current *Screen.kt set, grouped by
feature package, and map every screen to its `Flow › Frame` on the latest
`Bitkit - Handoff vNN` Figma page. Drop the React Native column, which only
served the migration.

Add ScreensMapTest so a *Screen.kt added or removed without a map row fails
the unit test run, and point AGENTS.md at the map next to the Figma rule.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@greptile-apps

This comment has been minimized.

Comment thread app/src/test/java/to/bitkit/docs/ScreensMapTest.kt
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@ovitrif ovitrif self-assigned this Sep 8, 2026
@ovitrif ovitrif changed the title docs: refresh screens map with figma frames and drift test docs: refresh screens map with figma refs Sep 9, 2026
@ovitrif
ovitrif merged commit 2f67d59 into master Sep 9, 2026
18 checks passed
@ovitrif
ovitrif deleted the docs/screens-map-1233 branch September 9, 2026 10:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

docs: refresh screens map and map figma flows

2 participants