#48 landed the root CLAUDE.md and the four shared architecture notes, and the
merge resolved every file they had in common to that version. That is the right
division — this branch keeps only what #48 did not carry — but it left four
loose ends.
- .gitignore had both PRs' .claude blocks stacked. Collapsed to one, keeping
the un-ignore for .claude/settings.json and the local-only excludes.
- CLAUDE.md ground rule 4 said #47 "removes" docs/; it has merged and the
folder is gone. Also points at monorepo.md and at the per-package CLAUDE.md
files this branch adds, which nothing referenced.
- grab-help-docs/CLAUDE.md still described docs/ as a live landing page. It is
deleted; what matters now is that Vercel's Root Directory still points at it,
so the note sends the reader to documentation.md for the dashboard fix.
- packages/grab-api/CLAUDE.md now carries the zero-runtime-dependencies claim
that #48 raised to a repo-wide ground rule.
No code touched: test/ and packages/grab-url-cli/src/ are identical to master,
and the 11 failures in test/page-archive.test.ts are unchanged and still
master's own.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01EoGK1G8bnF6riGQekWLtYr
Follow-up to #49, which merged at 13:59 while this reconciliation was still being written — so these four fixes missed it by about a minute. #49 is merged and can't carry them, hence a fresh PR.
Four files, +22/−9, documentation only.
What's wrong on
masterright now1.
.gitignorecarries both PRs'.claudeblocks, stacked#48 and #49 each added one and the merge kept both:
Collapsed to the second, which is a superset — it keeps the un-ignore that makes
.claude/settings.jsontrackable and the local-only excludes.2.
CLAUDE.mdground rule 4 says #47 "removes"docs/#47 merged; the folder is gone. Changed to past tense.
3.
grab-help-docs/CLAUDE.mdstill describesdocs/as a live landing pageIt no longer exists. Rewritten around what still matters: it's deleted, don't recreate it, and the Vercel project's Root Directory still points at the missing folder — with a pointer to
documentation.md, which has the dashboard fix.4. Nothing referenced the per-package
CLAUDE.mdfiles ormonorepo.md#49 added nine per-package files and
monorepo.md, but the rootCLAUDE.md— which came from #48 — has no pointer to either, so an agent reading it never learns they exist. Added a row and a line to the Detailed-notes section.Also
packages/grab-api/CLAUDE.mdnow carries the zero-runtime-dependencies claim that #48 raised to a repo-wide ground rule. It's the single most important constraint on that package and its own file didn't state it.Verification
masterbefore fixing, and confirmed fixed here..claude/settings.jsonstill tracked after the.gitignorecollapse; all 6.claudefiles still tracked..claude/architecture/*.mdlink inCLAUDE.mdresolves.test/andpackages/grab-url-cli/src/are untouched, so the 11 pre-existing failures intest/page-archive.test.ts(red onmastersince ✨ feat(cli): archive a page into a folder with --page #45) are unchanged, as is the Vercel check, whose Root Directory is a dashboard setting no commit can fix. Both were diagnosed in the 📝 docs(claude): add per-package CLAUDE.md files and monorepo.md #49 comments.🤖 Generated with Claude Code
https://claude.ai/code/session_01EoGK1G8bnF6riGQekWLtYr
Generated by Claude Code