docs/plans/2026-05-31-content-loss-autoreview-fixes.md
Objective:
Fix accepted P1/P2 content-loss findings in the current content/** docs diff and rerun focused autoreview until it reports no accepted/actionable critical or major docs-loss findings.
Goal plan: docs/plans/2026-05-31-content-loss-autoreview-fixes.md
Completion threshold:
ComponentSource, anchors, snippets, and API references introduced by the docs diff are fixed.pnpm --filter www build:source passes.autoreview --mode local for content/** reports clean.Verification surface:
pnpm --filter www build:source.agents/skills/autoreview/scripts/autoreview --mode local --prompt "<focused content-loss review>"rg checks for restored anchors, ComponentSource src, shouldMergeNodes, and fixed HTML/Markdown snippets.Constraints:
HEAD is safer.Boundaries:
content/**, registry definitions in apps/www/src/registry/**, and render support in apps/www/src/lib/rehype-component.ts.Blocked condition:
pnpm --filter www check:docs expecting .source/index.ts after the user-requested fallback-parser revert. That checker fix is owned by another thread, so this goal does not re-add the fallback.Work Checklist:
HEAD, patch narrow remaining defects.ComponentSource src./Users/zbeyens/git/plate.Start Gates:
| Gate | Applies | Evidence |
|---|---|---|
autoreview loaded | yes | User explicitly requested $autoreview; script ran repeatedly. |
| Active goal checked or created | yes | Active goal continued with this plan. |
| Docs lane selected | yes | Broad content-loss review lane. |
| Target docs read | yes | Accepted finding locations were read before fixes. |
| Nearest sibling docs read | yes | Source/registry/render support inspected for examples and API snippets. |
| Docs style doctrine read | yes | Existing docs-creator constraints applied from repo rules and previous work. |
| Documented source code read | yes | Markdown, HTML, registry, and render support source inspected. |
| Ownership map drafted | yes | Content docs own prose; registry owns item names; rehype component owns source hydration. |
| Browser/render proof decision | yes | Static MDX/source generation and autoreview were sufficient; no browser-only UI behavior changed. |
| PR/tracker expectation decision | yes | No PR/tracker action requested. |
Completion Gates:
| Gate | Applies | Required action | Evidence |
|---|---|---|---|
| Named verification threshold | yes | Run source generation and focused autoreview | pnpm --filter www build:source passed; final autoreview clean. |
| Docs lane shape satisfied | yes | Restore reference docs or fix precise defects | High-risk net-deletion docs restored; remaining snippets/anchors/source blocks patched. |
| Source-backed claim audit | yes | Verify changed API/source claims | HTML deserialize shape, Markdown options, and ComponentSource src support checked against source. |
| Ownership map verified | yes | Confirm package/layer ownership | Registry entries and rehype-component.ts inspected. |
| MDX/content parser | yes | Run source generation | pnpm --filter www build:source passed. |
| Links/routes/previews verified | yes | Check accepted broken anchors/source names | Autoreview clean; targeted rg showed valid source blocks and anchor. |
| Plugin page specifics | yes | Restore plugin reference pages with major loss | Link, media, AI, Yjs, table, and other high-risk docs restored or corrected. |
| Browser/render surface changed | no | Static source generation covers this fix | Browser proof not required for content-loss objective. |
| Package/API behavior changed | no | Record N/A | Docs/render hydration only; no package API behavior changed. |
| Agent rules or skills changed | no | Record N/A | No agent rules or skills changed in this goal closeout. |
| Autoreview for non-trivial docs changes | yes | Run focused review until clean | Final autoreview: no accepted/actionable findings. |
| Final lint | no | Record N/A | User asked content-loss fix; no lint gate needed beyond source generation/autoreview. |
| Goal plan complete | yes | Run node .agents/skills/autogoal/scripts/check-complete.mjs docs/plans/2026-05-31-content-loss-autoreview-fixes.md | To run after this update. |
Phase / pass table:
| Phase | Status | Evidence | Next |
|---|---|---|---|
| Intake and source read | done | Accepted findings and source files inspected | writing |
| Writing | done | Restores and targeted fixes applied | verification |
| Verification | done | build:source passed; autoreview clean | closeout |
| PR / tracker sync | done | Not requested | final response |
| Closeout | done | Plan updated for completion check | final response |
Findings:
HEAD and patching precise defects that remained.Decisions and tradeoffs:
HEAD instead of trying to preserve lossy summaries.check:docs still fails in this checkout without the other-thread fix.ComponentSource src hydration support because the docs component already exposed the prop and the examples needed secondary registry files.Implementation notes:
shouldMergeNodes API reference and fixed editing-behavior/table anchors.Review fixes:
ComponentSource name values to ComponentSource src and wiring src in rehype-component.ts.Error attempts:
| Error / failed attempt | Count | Next different move | Resolution |
|---|---|---|---|
pnpm --filter www check:docs failed after fallback revert because .source/index.ts is absent | 1 | Do not re-add fallback; rely on other-thread checker fix and run build:source plus autoreview here | User requested fallback revert; build:source and final autoreview passed |
Verification evidence:
pnpm --filter www build:source passed after the ComponentSource src fix.autoreview --mode local exited 0 with: autoreview clean: no accepted/actionable findings reported.ComponentSource name usages removed and replaced with src./docs/plugin#advanced-plugin-configuration and editor-api#shouldmergenodes.pnpm --filter www check:docs remains outside this goal's green evidence after the user-requested fallback revert because the current checkout still lacks .source/index.ts.Final handoff contract:
pnpm --filter www build:source passed.check:docs still needs the other-thread .source/index.ts fix in this checkout.Final handoff / sync:
check:docs caveat above.Timeline:
ComponentSource value sources, ran build:source, and got focused autoreview clean.Reboot status:
| Question | Answer |
|---|---|
| Where am I? | Closeout |
| Where am I going? | Run autogoal completion checker, then mark goal complete if it passes |
| What is the goal? | Clear accepted/actionable critical or major content-loss findings in content/** |
| What have I learned? | Lossy summaries were the core risk; restores plus targeted fixes were safer |
| What have I done? | Fixed accepted findings, reverted requested fallback parser, and got clean autoreview |
Open risks:
pnpm --filter www check:docs is not green in this checkout until the separate .source/index.ts generator/checker fix is present.