YAO-135: C-3 new-chapter detection by file path (unblocks YAO-106/C04)#29
Merged
Conversation
…th set Resolves the CI self-conflict between C-3 (check-code-ratio.ts) and C-5 (check-no-frontmatter-in-chapters.ts) that blocked YAO-106 (C04). - check-code-ratio.ts: drop frontmatter judgment + NEW_CHAPTER_NN prefix set. New chapters are now identified by an explicit NEW_CHAPTER_FILES path set, sourced from spec §9.3.1 'new-chapter landing-file table'. - V2-REVISION-SPEC.md §0.1.1 / §0.5.4 / §9.3 line 569: point at §9.3.1 as the single authoritative source for new-chapter identity (drops residual '§6.2 全新增比例 column' wording that was never machine-readable). - V2-REVISION-SPEC.md §9.3.1: new subsection with landing-file table. C04 -> docs/26-配置迁移即代码.md locked; remaining 7 new chapters left blank, to be filled by OC-Dev as each issue is scheduled. New chapters append from NN=26 (v1 occupies 00-25), so no v1 file rename is bundled with a new-chapter PR. source_commit: 290fdc9481a70612bc5823aa4ed225c52c52aad3 Co-authored-by: multica-agent <github@multica.ai>
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.
Resolves YAO-135 (C04 blocker arbitration).
Conflict
check-code-ratio.ts) required新增章节: yesin chapter frontmatter to identify new chapters.check-no-frontmatter-in-chapters.ts, YAO-99) bans ALL frontmatter indocs/*.md.docs/04-System-Prompt-工程.md, so C04 can't reuse the v1 NN without bundling a v1 rename into the C04 PR (which would pollute C-1 / C-2 gates).Arbitration
Route B (manifest / file-path). Stays consistent with already-merged §0.1.1 (no chapter frontmatter, ever).
Changes
scripts/check-code-ratio.ts: drop frontmatter judgment + NN-prefix set; identify new chapters by an explicitNEW_CHAPTER_FILESpath set, with C04 =docs/26-配置迁移即代码.md. Other 7 new chapters added as each issue is scheduled (by OC-Dev).docs/V2-REVISION-SPEC.md:docs/26-配置迁移即代码.mdlocked; remaining 7 left blank. Rule: new chapters append from NN=26 (v1 occupies 00–25), so no v1 file rename is bundled with a new-chapter PR.Out of scope
docs/04-System-Prompt-工程.mdstays at NN=04 — its rename todocs/06-System-Prompt-与-Output-Style-注入.mdbelongs to C06's own PR.multica issue update.Verification
bun scripts/check-code-ratio.ts --files docs/04-System-Prompt-工程.md docs/26-配置迁移即代码.md→ skips v1 file with correct message; would gate the C04 file when present.bun scripts/check-no-frontmatter-in-chapters.ts→ still passes on all 25 v1 files (unchanged).source_commit: 290fdc9481a70612bc5823aa4ed225c52c52aad3