[koh-sh]

Harness review of CLAUDE.md, .claude/ and the new e2e/CLAUDE.md, from /doctor, /doctor prompt-audit, /fewer-permission-prompts, /insights and claude plugin validate.

Adopted changes

#	Change	Evidence
1	kong-cli.md: the "switch on the command string" pattern is from Kong's own README	This repo's README never mentions switch ctx.Command() or Kong
2	kong-cli.md: name VersionCmd.Run as the existing exception to "no method bodies in cmd/cli.go"	func (v *VersionCmd) Run is at cmd/cli.go:63; it predates the rule
3	CLAUDE.md: add vhs to the mise-managed tools	.mise.toml:11
4	CLAUDE.md: add mise run demo / demo-validate to the command list	.mise.toml:86-95
5	CLAUDE.md: only commands with cross-field constraints have Validate()	Validate() exists on ReviewCmd, PRCmd, LocateCmd only
6	upgrade-tools skill: update every bun-version in e2e.yml	Two occurrences (e2e.yml:36, :65), one per job
7	CLAUDE.md: mark the hook overview, data flow and three design decisions as deprecated cchook, in the past tense	README marks cchook/cclocate deprecated; CLAUDE.md did so only in the CLI and package sections, and plan-mode-based proposals came up in past sessions
8	CLAUDE.md: edit files with the Edit tool, not perl/sed in-place substitution	Past sessions in this repo had perl substitutions corrupt files (a wrong delimiter broke a Go test file)
12	CLAUDE.md: drop seven generic rules (Go idioms, DRY, naming, comments, concise README)	/doctor: behavior the model follows by default (~130 tokens per session)
13	Move the E2E test notes to e2e/CLAUDE.md	/doctor: loaded only when working under e2e/ (~340 tokens off other sessions)
14	Add .claude/settings.json with five read-only Go commands; go doc -http and go list -mod…/-modfile still prompt via ask rules	/fewer-permission-prompts: go doc 95, go build ./... 51, go list -m 17, go vet ./... 16, go tool cover -func=coverage.out 4 uses. go doc -http serves HTTP and -modfile may write go.mod (go help doc, go help build)

mise run ci passes.

Not adopted

• chore(deps): Bump actions/create-github-app-token from 2 to 3 #9 State "commit directly to main" in CLAUDE.md: already covered by local memory, no recurrence since
• feat: add commd pr subcommand for reviewing GitHub PR markdown files #10 Remove the Build & Test Commands section: loses notes mise tasks does not show (e.g. e2e vs e2e-basic)
• chore(deps): Bump github.com/yuin/goldmark from 1.7.16 to 1.7.17 #11 Remove the Architecture section: loses cross-cutting knowledge (ReviewState save rules, the web session token) that is costly to rediscover

Verification in a new session

The harness loads at session start, so check these in a fresh session:

• 長い行だと横にはみ出ている。横にスクロールできないので見えない #1, ラリーが続いた時、viewedのチェックを引き継ぎたい #2: on a task under cmd/, the model does not look for a switch pattern in this README and does not flag VersionCmd.Run as a violation
• planを承認した後に直接claudeのplan modeを解除して作業をさせたい #3–chore(deps): Bump goreleaser/goreleaser-action from 6 to 7 #5: the model treats vhs/demo as project tooling and does not add Validate() to HookCmd/VersionCmd
• chore(deps): Bump github.com/mattn/go-runewidth from 0.0.19 to 0.0.20 #6: /upgrade-tools with a bun bump updates both bun-version lines in e2e.yml
• chore(deps): Bump github.com/mattn/go-runewidth from 0.0.20 to 0.0.21 #7: feature proposals do not build on the plan-mode hook flow
• chore(deps): Bump github.com/charmbracelet/glamour from 0.10.0 to 1.0.0 #8: file edits go through the Edit tool, not perl -pi / sed -i
• ci: fix ghasec security alerts across all workflows #12: the model still keeps subcommands consistent (the one remaining Code rule)
• chore(deps): Bump github.com/yuin/goldmark from 1.7.16 to 1.8.2 #13: /context at session start no longer lists the E2E notes; after reading a file under e2e/, e2e/CLAUDE.md is loaded
• chore(deps): Bump actions/setup-go from 6.3.0 to 6.4.0 #14: go doc … / go build ./... run without a permission prompt in a checkout without .claude/settings.local.json (e.g. a worktree), while go doc -http still prompts

🤖 Generated with Claude Code

https://claude.ai/code/session_01RKh8XqY1bD6M11zbzS4gVD

[github-actions]

Code Metrics Report

	main (c01578d)	#71 (a87f869)	+/-
Coverage	91.5%	91.5%	0.0%
Code to Test Ratio	1:1.5	1:1.5	0.0
Test Execution Time	6s	5s	-1s

Details

  |                     | main (c01578d) | #71 (a87f869) | +/-  |
  |---------------------|----------------|---------------|------|
  | Coverage            |          91.5% |         91.5% | 0.0% |
  |   Files             |             45 |            45 |    0 |
  |   Lines             |           3168 |          3168 |    0 |
  |   Covered           |           2899 |          2899 |    0 |
  | Code to Test Ratio  |          1:1.5 |         1:1.5 |  0.0 |
  |   Code              |           6575 |          6575 |    0 |
  |   Test              |          10261 |         10261 |    0 |
+ | Test Execution Time |             6s |            5s |  -1s |

---

Reported by octocov