Uh oh!
There was an error while loading. Please reload this page.
docs: resolve the four suspected-dead repo paths in content/docs prose (#3867) - #4393
Merged
Merged
Conversation
#3867) One coordinate was wrong and is fixed; three are real categories the #3735 gate-expansion card will need rules for, and are now labelled as such in the prose that states them. - troubleshooting.md: `packages/react/src/context/ThemeContext` -> `.tsx`. The referent was right (ThemeProvider is defined there, at line 128, and reaches @object-ui/react via context/index.ts) — only the extension was missing, so the coordinate did not resolve. - plugin-development.md: `packages/plugin-board/` is scaffolder output, not a repo package (create-plugin/src/index.ts:74,97 writes cwd/packages/plugin-NAME). Kept, with one sentence saying so. - rfcs/0001-clipboard-paste.md: the user guide is a planned deliverable of a still-Draft RFC whose capability never landed. Marked planned. - utilities/runner.mdx: `packages/runner/src/app-data/` is git-ignored and reader-supplied; the same page already says so eight lines later. Left unchanged. Fixes#3867
The latest updates on your projects. Learn more about Vercel for GitHub. |
yinlianghui
commented
Aug 12, 2026
CollaboratorAuthor
ACCEPT — PM 复核 (session
Flipping ready + arming auto-merge. Generated by Claude Code |
yinlianghui
marked this pull request as ready for review
August 12, 2026 01:27
Uh oh!
There was an error while loading. Please reload this page.
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 freeto 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.
Fixes#3867
Scope is the triage split, unchanged: the 4 suspected-dead coordinates only. The
content/docs/**gate expansion stays a separate card — this PR adds inputs for it (three of the four turned out to be categories, not defects) but does not touch any gate.Per the triage caveat, the per-item deliverable was fix-or-reclassify-with-evidence, not blind path surgery. Measured on
origin/main@bb58d1d61: all four tokens were still dead there before the first edit (instrument reading below).Outcome
content/docs/guide/plugin-development.md:53packages/plugin-board/packages/create-plugin/src/index.ts:74buildsplugin-+ the sanitized name and:97joins it aspath.join(process.cwd(), 'packages', fullPackageName)— so the documented commandnpx @object-ui/create-plugin boarddoes emit exactlypackages/plugin-board/, in the reader's workspace. The prose was right; it just never said the directory is generated. One sentence added to say so.content/docs/guide/troubleshooting.md:167(card said:168)packages/react/src/context/ThemeContextThemeProvideris defined atpackages/react/src/context/ThemeContext.tsx:128and reaches@object-ui/reactthroughpackages/react/src/context/index.ts(export * from './ThemeContext'), which is what the page's own code sample imports from. Only the extension was missing, so the coordinate did not resolve. Added.tsx. (Not the otherThemeProvider,packages/providers/src/ThemeProvider.tsx— that one is not what the sample imports.)content/docs/utilities/runner.mdx:146packages/runner/src/app-data/packages/runner/.gitignoreline 1 is literallysrc/app-data. The same page already spells this out eight lines below the flagged one, atrunner.mdx:154-156: "src/app-data/is git-ignored (seepackages/runner/.gitignore) and is not part of a fresh checkout — you supply it".packages/runner/README.md:110says "createpackages/runner/src/app-data/", andpackages/runner/src/App.navigation.test.tsx:31stubs the loader for the same reason. Editing here would only duplicate correct prose, so the honest deliverable is the reclassification.content/docs/rfcs/0001-clipboard-paste.md:477content/docs/guide/clipboard-paste.mdstatus: Draft, and none of its section-10 deliverables exists: noclipboardsubmodule underpackages/core/src, and zero repo-wide hits forusePasteToGrid,PastePreviewDialogorclipboardPaste. The only clipboard code shipping today isuseCellClipboardand theImportWizardparsers in@object-ui/plugin-grid. No guide page created (explicitly out of scope); instead a status note now marks section 10 as the plan rather than as openable pages.No item was already fixed by an intervening PR — all four were live at
bb58d1d61.Why three of four are categories, not defects
That is the reusable output for the gate-expansion card. Items 1 and 3 are the same shape the card already identified for
packages/plugin-yourpluginandapps/console/dist/: a path the reader creates or the build produces, stated correctly by a doc that is doing its job. Item 4 is a fourth shape: a planned coordinate inside a deliverables list of a Draft RFC. None of them is fixable by path surgery, and none should be silenced with a per-line exemption entry — they want rules.Verification
All commands run at the repo root of the worktree, on the pushed commit.
Instrument re-run — the card's own extractor (
extractPathTokensimported fromscripts/check-skills-paths.mjs, not a re-implementation), walked overcontent/docs/**for.mdand.mdx(the gate's own walker is.md-only, while #3867 measured 183.md/.mdxfiles). Scratch script, deliberately not checked in.Before (
bb58d1d61, clean tree):After:
Reading: one dead token removed (item 2 now resolves), zero new dead tokens. Assertions rise by 1 and resolutions by 2 because the item-1 sentence cites
packages/create-plugin/src/index.ts— a token that resolves; thecwd/packages/plugin-NAMEshape in the same sentence carries angle brackets, soPATTERN_REclassifies it as a pattern rather than an assertion, which is the same treatment the extractor already gives the other placeholder paths. The three remaining flagged lines are the reclassified ones, unchanged by design, and the other six were already listed by the card as deliberate / template / build-artifact tokens (out of scope here).Repo gates:
check-skills-pathsis unchanged from before this PR (84/85, 1 baselined) — no regression; this PR does not touchskills/**or any gate script. The changeset gate self-determines for a docs-only change, so no changeset and no label are needed.Not run, deliberately: package builds and the vitest suites. Nothing outside
content/docs/**prose changed — no source, no test, no script, no config.Generated by Claude Code