feat(cli): drive scan curation from Figma devStatus + v2 manifest#86
Open
nathanacurtis wants to merge 6 commits into
Open
feat(cli): drive scan curation from Figma devStatus + v2 manifest#86nathanacurtis wants to merge 6 commits into
nathanacurtis wants to merge 6 commits into
Conversation
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- 2x page grid padding and nav–content gutter at wide viewports - Increase sidebar item spacing (+4px) and padding (+2px → 8px) - Remove distinct surface colors and borders between nav/sidebar/content - Force dark mode and hide theme toggle - Move GitHub and Figma Plugin links from header to top of sidebar Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…format
Scan now reads each component's devStatus from the document tree and uses
READY_FOR_DEV as the default inclusion signal. Rescans merge with the prior
manifest, preserving manual checkbox edits unless devStatus changed for a
row (Figma wins). New flags:
--keep-checks preserve prior checks even when devStatus flips
--reset-checks ignore prior manifest entirely
Manifest format upgraded to v2 (table with Dev Status column, scan format
version header, file last modified). Legacy v1 checkbox-list manifests are
detected and migrated automatically; the migration code lives in an
isolated module (ManifestMigrationV1ToV2) marked DELETE-WHEN for clean
removal once v1 manifests are no longer in the wild.
Docs in site/cli/{commands/scan,getting-started,workflows} updated for the
new format, default rule, merge behavior, and flags.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Usage, Format, Authoring (Defaults, Rescan), Examples, Arguments, Options, Migration. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
scannow drives curation from Figma's Ready for Dev signal: components withdevStatus: READY_FOR_DEVare checked by default; libraries without any devStatus signal fall back to the legacy heuristic.devStatuschanged for a row (Figma wins).--keep-checkslocks manual edits regardless;--reset-checksignores the prior manifest entirely.Dev Statuscolumn,**Scan format version:**header,**File last modified:**). Legacy v1 (checkbox-list) manifests are detected and migrated automatically by an isolatedManifestMigrationV1ToV2module markedDELETE-WHENfor clean future removal.cli/commands/scan.md,cli/getting-started.md,cli/workflows.md.Test plan
ManifestParserV2,ManifestMigrationV1ToV2,deriveDefaultInclusion,mergeRows(20 new tests).ComponentDiscoverytests updated for the newdevStatusfield; new test verifies extraction +getFileLastModified().specs-testingFigma file:TEST PropBinding 1as Ready for Dev:1 updated by devStatus, 186 preserved— devStatus flip flowed through, prior manual checks preserved.--reset-checks: 1 selected (only the READY_FOR_DEV row).🤖 Generated with Claude Code