docs: drop a dead link and fix a duplicated character - #618
Open
MincongZhou wants to merge 1 commit into
Open
MincongZhou wants to merge 1 commit into
MincongZhou wants to merge 1 commit into
Conversation
`CLAUDE_CODE_WORKFLOW_DESIGN_2026-08-23.md` was never committed to this repository, so the scope note in the fan-out policy linked to a 404. The sentence keeps its text and drops the unusable link; the same-dated `..._RUNTIME_CONTRACT_...` sibling is called a contract interview in both line 437 and docs/research/README.md, so redirecting to it would be a guess. The V2 acceptance row read `无无解释 running`. This document already describes the condition as `无解释的 running` at line 72, so the row now matches that wording.
Author
|
The only failing check is Background terminals (Windows), and it is unrelated to this change It failed in The rest of that job passed (1778 pass / 1 fail), and Node 22.19.0, Node 24, Node 26 and This account cannot rerun the workflow: Could a maintainer rerun the failed job? Thank you. |
This branch has not been deployed
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.
Problem
Two defects in the research/design notes.
docs/research/CLAUDE_CODE_WORKFLOW_FANOUT_POLICY_2026-08-23.md:5links to./CLAUDE_CODE_WORKFLOW_DESIGN_2026-08-23.md. That file does not exist, and never has:The only same-dated siblings in
docs/research/are..._FANOUT_POLICY_2026-08-23.md(the linking file itself) and
..._RUNTIME_CONTRACT_2026-08-23.md.docs/design/OPENPI_WORKFLOW_V2_DESIGN_2026-08-23.md:536, in the deterministic-gateacceptance table, reads
| Terminal write failure | 无无解释 \running` |`.Value
The fan-out policy's scope note stops sending readers to a 404, and the acceptance table reads
consistently with the rest of the document that contains it.
Approach
For (1), the unusable link is dropped and the sentence keeps its text.
I deliberately did not redirect it to
CLAUDE_CODE_WORKFLOW_RUNTIME_CONTRACT_2026-08-23.md.That document is called a "运行时合同访谈" in line 437 of the same file and
"version-scoped Workflow contract interview" in
docs/research/README.md, while the sentencecalls its target a "生命周期研究". The names do not match, so redirecting would be a guess, and a
wrong redirect is worse than no link. If you know the intended target, one line restores the
navigation.
For (2), the row is aligned with wording this document already uses. Line 72 says:
So the acceptance row becomes
无解释的 \running`— the doubled 无 was a typo and the 的 was missing, giving无无解释→无解释的`.Validation
*.mdnow reports 0 broken links. Before thischange it reported exactly the one above.
bun run check— pass (config/docs/discipline contracts, Web build, biome format, biomelint with
--error-on-warnings,tsc --noEmit).git diff --check— pass.No code, test, or config changes; documentation only.
Impact
unknown target instead of redirecting it.