feat: integrate the 0.4.6 line (R20, F1-F5, F7, F8) - #53
Merged
Conversation
F1 reject responsive export before Figma fetch F2 discover breakpoints by width when slot names collide F3 reconcile export output against project theme F4 filter token suggestions by prop category F5 budget section preview output evenly F7 validate bare tokens in the typography prop
Embed mode resolution, responsive array order and prop conventions in devup_project_context so clients stop guessing them. Add --install-skills, --check-skills and --skill-dir. Fetches upstream SKILL.md with ETag caching, records SHA-256 provenance and writes atomically.
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
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.
Lands
owjs3901/integration-0-4-6onmain. It was the base of #48 and had never passed CI on its own.What is in it
What I fixed to make it mergeable
Ten rustfmt violations. This branch had never been green; those ten are why every
verifyjob on #48 failed in under a minute. They are fixed here rather than carried further. (They were also what made #48's CI look like the bridge's fault — it wasn't.)One conflict, in
crates/devup-mcp/Cargo.toml: a comment blockmaincarries and the incoming commit does not. Keptmain's.A changepack, for the 18 changed crate files.
The new bundle gate earned its keep immediately
This branch changes
explore.jsandsection_index.js, and those scripts are compiled into the bridge plugin at build time. Soplugin/dist/was stale the moment these commits landed on top of #50 — and the gate added in #50 caught it on the first run:Rebuilt and committed. Gate now clean. That is exactly the drift the gate exists to prevent, on its first real opportunity.
Verification
cargo fmt --all -- --checkcargo clippy --locked --workspace --all-targets --all-features -D warningscargo test --workspacemainis 1090; +22)cargo insta test --checkcargo test --locked -p devup-mcp --test stdio_smokecargo metadata --lockedHarness: no regression. All 14 measurable screens are identical to
main's thresholds — popup 3.64 / 2.06 / 0.85, keyframes 2.69, grid 2.96, notice 3.74 / 2.12 / 1.16, about 5.43 / 2.93 / 1.77, landing 4.78 / 2.47 / 1.50.One pre-existing condition, not caused by this branch
report-446-1971no longer acquires: the Figma file changed upstream, so the cached call returnsDEVUP_FIGMA_VERSION_CHANGED. I verified this against a binary built frommain— it skips identically. Unrelated to these commits.Two things worth noting about how that surfaced: the acquisition skipped the screen and carried on instead of dying with a traceback, and the run still exits non-zero so a skipped screen cannot pass unnoticed. Both are the harness-determinism work behaving as designed. The screen needs re-acquisition against the current file, which is separate work.