Skip to content

docs: fork hygiene — README banner, upstream sync policy, fork changelog - #10

Merged
axisrow merged 2 commits into
mainfrom
ao/claude-devtools-6/fork-hygiene
Sep 21, 2026
Merged

axisrow merged 2 commits into
mainfrom
ao/claude-devtools-6/fork-hygiene

Conversation

@axisrow

@axisrow axisrow commented Sep 20, 2026

Copy link
Copy Markdown
Owner

Summary

Closes #7

🤖 Generated with Claude Code

…hangelog

Closes #7

Co-Authored-By: Claude Code <noreply@anthropic.com>

@axisrow axisrow left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verdict: changes requested (one factual fix; otherwise accurate and well-scoped docs PR).

Verified claims: upstream dormancy holds (last commit on matt1398/claude-devtools is 2026-05-13); pnpm analyze:session / pnpm analyze:sessions exist in package.json and match PR #1; upstream PR matt1398#235 exists and is open; anchor #upstream-sync-policy matches the new heading and the relative link from CHANGELOG.md resolves; the Fork section above [Unreleased] is acceptable since it is explicitly explained.

One blocking finding inline: the README banner advertises a Claude Code plugin that does not exist in the repo. Fix that line and this is good to merge.

Comment thread README.md Outdated
Co-Authored-By: Claude Code <noreply@anthropic.com>

@axisrow axisrow left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verdict: approved (no inline findings).

Re-review of head 36882fc after the previous round: the blocking finding is fixed — the README banner no longer advertises a Claude Code plugin (now: session-audit CLI and fixes); no plugin mentions remain in the README at this head.

All other claims were verified previously and are unchanged: upstream dormancy holds (last upstream commit 2026-05-13); pnpm analyze:session / pnpm analyze:sessions exist in package.json and match the CHANGELOG Fork entry and PR #1; upstream PR matt1398#235 exists and is open; the #upstream-sync-policy anchor and the relative CHANGELOG link resolve; the Fork section placement above [Unreleased] is explicitly explained.

Docs-only diff, nothing to test. Good to merge.

@axisrow
axisrow merged commit fee8241 into main Sep 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fork hygiene: README banner + upstream sync policy

1 participant