Repository navigation
docs(audits): reword the grants row without pipes in claim-table.md - #2965
Conversation
Line 193 had a `||` inside a code span. The table's awk check splits on every pipe, so the row counted 9 cells against the 7 the table expects. The sentence now says the same thing without the pipes. Co-Authored-By: Claude Haiku 5.5 <noreply@anthropic.com> Co-authored-by: quire <WyJxdWlyZSIsImRlZmF1bHQiXQ@agents.commonly.invalid> Agent-Model: claude/claude-haiku-5-5/high
|
No actionable comments were generated in the recent review.
 |
| Layer / File(s) | Summary |
|---|---|
Update call-record claim wording docs/audits/claim-table.md |
The claim now describes ToolCall.create as stamping the write time when record.at is unset. |
Priority:
Estimated code review effort: 1 (Trivial) | ~3 minutes
Change: Other
Possibly related PRs
- Team-Commonly/commonly#2964: Addresses the malformed grants row in the same claim table.
- Team-Commonly/commonly#2817: Added the grants audit section containing this call-record claim.
Merge Risk: 
e660d
-
No actionable merge-blocking risk remains for this documentation-only change.
Pre-merge checks: 
-
Check name Status Explanation Description check The description accurately explains the documentation-only change and the table validation issue caused by pipe characters. Title check The title clearly and concisely identifies the documentation change: rewording the grants row to remove pipe characters. Linked Issues check Check skipped because no linked issues were found for this pull request. Out of Scope Changes check Check skipped because no linked issues were found for this pull request.
Finishing Touches
-
🧪 Generate unit tests (beta)
-
- Commit to this branch
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Comment @coderabbitai help to get the list of available commands.
|
Parked: v3 takes fixes only while v4 is rebuilt (ledger D169). Reopen if a live user needs it. |
lilyshen0722
left a comment
lilyshen0722
left a comment
There was a problem hiding this comment.
Tier 0 operator review (Launch), part of the 10-31 parked-PR sweep under D218. This PR changes only docs, tests, or a warn-only script that nothing runs in CI, so it has no runtime path. I read the diff at this head, and it merges cleanly in sequence with the other Tier 0 PRs in this batch (#2964, #2965, #2967, #2802, #2958, #2966) on main.
GATE: PASS
Summary
docs/audits/claim-table.mdline 193 (the grants call-record row) hadrecord.at || new Date()inside a code span. The table's awk check splits on every pipe, so that row counts 9 cells where the table expects 7. This PR rewords the sentence without the pipes. No claim changes; the cited code lines are the same.Check
awk -F'|' '/^\|/ && NF != 7' docs/audits/claim-table.mdprints nothing.Docs-only, not a docs-site change, so no HN hold. Gate pending; Lily routes.
Written by Quire, a Commonly agent (TASK-020 follow-up).
🤖 Generated with Claude Code