docs(sync): per-commit nuxt/ui → b24ui sync plan + Phase 0 scaffolding#65
Open
IgorShevchik wants to merge 1 commit into
Open
docs(sync): per-commit nuxt/ui → b24ui sync plan + Phase 0 scaffolding#65IgorShevchik wants to merge 1 commit into
IgorShevchik wants to merge 1 commit into
Conversation
Adds .sync/ with the per-commit synchronization design and the artifacts it depends on, incorporating multi-angle review feedback (docs, engineering, QA, security, tech-director): - PLAN.md: gated per-commit flow with corrected gate names/order, concurrency lock, prompt-injection mitigations, minimal permissions, v-html security gate, no-op batching + pre-filter, kill-switch, metrics, jsDoc/TS and snapshot/test requirements, SHA validation and idempotency. - PORTING.md: trusted port-rules guide (ui→b24ui, b24-icons, air-* colors), preserved invariants, examples, map-update and reviewer workflow. - icon-map.json / color-map.json: seed maps verified against current sources. - nuxt-ui.json: ledger skeleton with kill-switch and stats (sync_enabled=false). - RUNBOOK.md: incident playbook.
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
2 participants
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
Adds
.sync/— the design and Phase 0 scaffolding for per-commit synchronization of nuxt/ui (v4) into b24ui. Docs/config only; nothing runs automatically (sync_enabled: false).The goal: review every upstream nuxt/ui commit and accept a corresponding PR in b24ui. Direct
cherry-pickis impossible due to systematic divergences (b24uiprop instead ofui,b24-icons,air-*colors, per-componentsrc/theme/*.ts, the Bitrix-only layer), so each commit is semantically ported via Claude, gated, and human-reviewed one PR at a time.What's included
.sync/PLAN.md.sync/PORTING.mdui→b24ui, icon/color maps, preserved invariants (jsDoc, types, a11y), examples, reviewer workflow..sync/icon-map.jsonb24-iconsmap (verified against current sources)..sync/color-map.jsonair-*map..sync/nuxt-ui.jsonprocessed, stats) with kill-switchsync_enabled: false..sync/RUNBOOK.mdReview process behind this PR
The plan was reviewed from multiple angles before this PR (documentation, engineering, QA, security, tech-director, plus a diff pass). All P1/P2 findings are folded into the docs above, notably:
ci.yml(pnpm run lint/typecheck/test run/build).processed[sha].src/-only scope guard.permissions:, secret masking, SHA validation,v-htmlsecurity gate.Out of scope (tracked separately)
main(owner action)Next step
This PR is for reviewing the plan. The actual workflows (
sync-*.yml, Phase 1+) are intentionally not included — they come after the plan is approved and #62 lands.https://claude.ai/code/session_01Qz7EXMncvEGiCj4WbmYgJo
Generated by Claude Code