Heads-up on a small one that your own convention already covers, so this is
a pointer rather than a proposal.
docs/benchmarking.md line 45 has a "session_id" value in its sample
output that is a real v4-shaped UUID rather than a synthetic one. Your
docs/code-reviews/README.md asks for 00000000-0000-4000-8000-<12hex> in
its place, and states the reason better than I would: public git history
cannot be scrubbed, so an identifier is burned the moment it reaches
main and can only be worked around forward.
I have not quoted the value here, for that reason. It is the only match in
the file for the v4 pattern.
This is the same class as #318,
scrubbed in e3149ae (#319) — so there is already a remedy you have
applied once, and the fix is a one-line replacement.
For what it is worth, the convention is doing its job: this surfaced from
the same tools/absence-scan.mjs check that found the #318 pair, run over
a merge of your main into a fork. I have replaced the value locally with
a non-identifier placeholder ("<session-id>") rather than a synthetic
UUID, since the surrounding text does not need the shape to stay legible —
either works for your tree.
No action needed from me; flagging it because it is your rule and I would
want the same pointer.
— Proxy fork maintainer, via Claude Code (agent-assisted)
Heads-up on a small one that your own convention already covers, so this is
a pointer rather than a proposal.
docs/benchmarking.mdline 45 has a"session_id"value in its sampleoutput that is a real v4-shaped UUID rather than a synthetic one. Your
docs/code-reviews/README.mdasks for00000000-0000-4000-8000-<12hex>inits place, and states the reason better than I would: public git history
cannot be scrubbed, so an identifier is burned the moment it reaches
mainand can only be worked around forward.I have not quoted the value here, for that reason. It is the only match in
the file for the v4 pattern.
This is the same class as #318,
scrubbed in
e3149ae(#319) — so there is already a remedy you haveapplied once, and the fix is a one-line replacement.
For what it is worth, the convention is doing its job: this surfaced from
the same
tools/absence-scan.mjscheck that found the #318 pair, run overa merge of your
maininto a fork. I have replaced the value locally witha non-identifier placeholder (
"<session-id>") rather than a syntheticUUID, since the surrounding text does not need the shape to stay legible —
either works for your tree.
No action needed from me; flagging it because it is your rule and I would
want the same pointer.
— Proxy fork maintainer, via Claude Code (agent-assisted)