docs: align public guides with current release - #849
Conversation
Claude audit (merge-authority lane)Head SHA: Claude Audit: PASS Summary: Docs-only consistency PR (README, CHANGELOG, and several docs/*.md files) plus matching test updates in tests/test_release_hygiene.py and tests/test_jira_adoption.py. Changes scrub a private repo name and other private identifiers from public docs, fix stale/incorrect CLI command examples ( Findings:
|
| | Host | Current qualification | Jira context posture | | ||
| |---|---|---| | ||
| | Codex | Qualified for session coordination, controller telemetry, and the local working-copy lease | Code Mower REST is authoritative; connected Atlassian Rovo MCP is optional read/context only | | ||
| | Claude Code | Qualified for the same session, telemetry, and lease contract | Code Mower REST is authoritative; connected Atlassian Rovo MCP is optional read/context only | | ||
| | Cursor | Qualified for the same session, telemetry, and lease contract | Code Mower REST is authoritative; Atlassian MCP is optional read/context only and noninteractive reads need approved tool access such as `--auto-review` | | ||
| | Devin, Grok Bot, Antigravity, Muse, custom hosts | Recognized host identities for briefs and telemetry; execution remains an explicit handoff or provider-specific transport until separately qualified | No implicit Jira authority; use the guarded Code Mower tracker commands for writes | | ||
|
|
||
| The lease, shared Jira brief, and explicit Cursor qualification are currently | ||
| unreleased changes on `main`; see [Participants And Sessions](sessions.md) for | ||
| the exact release boundary. |
There was a problem hiding this comment.
💡 Quality: Orchestrator Hosts table states qualification as unconditional
The new table (lines 24-29) flatly states Codex, Claude Code, and Cursor are all "Qualified for the same session, telemetry, and lease contract," and only the prose note two lines below clarifies that the lease and Cursor qualification are unreleased on main (not in the published v1.1.2 package). A reader who copies or skims just the table (a common pattern for reference tables) could believe this is current v1.1.2 behavior. Consider adding a footnote marker directly in the table cells for the unreleased items (e.g. "Qualified for session coordination (unreleased on main)") rather than relying solely on the trailing paragraph.
Mark unreleased status inline per row instead of only in a trailing paragraph:
| Codex | Qualified for session coordination and controller telemetry; the working-copy lease is unreleased on `main` | ... |
| Claude Code | Qualified for the same session and telemetry; the lease is unreleased on `main` | ... |
| Cursor | Qualification, telemetry, and lease are unreleased on `main` (not in v1.1.2) | ... |
Was this helpful? React with 👍 / 👎
| def test_public_docs_match_current_commands_and_privacy_boundary(self) -> None: | ||
| paths = sorted((ROOT / "docs").rglob("*.md")) + [ | ||
| ROOT / "README.md", | ||
| ROOT / "CHANGELOG.md", | ||
| ] | ||
| combined = "\n".join(path.read_text(encoding="utf-8") for path in paths) | ||
|
|
||
| self.assertNotIn("code-mower tracker status", combined) | ||
| self.assertNotIn("code-mower audit pr", combined) | ||
| self.assertNotIn("Until that binding lands", combined) | ||
| for private_identifier in ( | ||
| "DrinkBetter-AI/mobile-app", | ||
| "/Us" + "ers/" + "j" + "huber", | ||
| "/home/" + "j" + "huber", | ||
| "github.com/" + "jeff" + "huber/", |
There was a problem hiding this comment.
💡 Quality: New hygiene test doesn't cover examples/*/README.md or CONTRIBUTING.md
test_public_docs_match_current_commands_and_privacy_boundary only scans docs/*.md, README.md, and CHANGELOG.md for stale commands and private identifiers, but the README's docs map (this PR) links to examples/demo-calibration/README.md and examples/board-demo/README.md, which are also public-facing docs and aren't covered by this hygiene sweep. Widening the paths glob to include examples/**/*.md (and any other public root docs like CONTRIBUTING.md/SUPPORT.md) would close this gap for future edits, even though no leaks exist in those files today.
Include examples//*.md in the privacy/command hygiene scan:**
paths = (
sorted((ROOT / "docs").rglob("*.md"))
+ sorted((ROOT / "examples").rglob("*.md"))
+ [ROOT / "README.md", ROOT / "CHANGELOG.md"]
)
Was this helpful? React with 👍 / 👎
|
Note Automatic reviews are paused because your team has used its included automatic processing for this billing period (headroom scales with your seat count). You can still comment "Gitar review" to run one anytime, and automatic reviews resume on their own by October 1. Add seats for more headroom. Code Review 👍 Approved with suggestions 0 resolved / 2 findingsAligns public guides with the current v1.1.2 release while documenting unreleased main-branch behavior, fixing stale command examples, Jira transport wording, orchestrator guidance, and release chronology. Two minor suggestions: add inline footnote markers in the Orchestrator Hosts table to clarify unreleased qualifications rather than relying on trailing prose, and expand the new release-hygiene test to cover 💡 Quality: Orchestrator Hosts table states qualification as unconditional📄 docs/provider-matrix.md:24-33 The new table (lines 24-29) flatly states Codex, Claude Code, and Cursor are all "Qualified for the same session, telemetry, and lease contract," and only the prose note two lines below clarifies that the lease and Cursor qualification are unreleased on Mark unreleased status inline per row instead of only in a trailing paragraph💡 Quality: New hygiene test doesn't cover examples/*/README.md or CONTRIBUTING.md📄 tests/test_release_hygiene.py:7952-7966
Include examples/**/*.md in the privacy/command hygiene scan🤖 Prompt for agentsOptionsDisplay: compact → Counting what did not apply, without listing it. Comment with these commands to change the behavior for this request:
Was this helpful? React with 👍 / 👎 | Gitar |
Closes #848
Summary
Builder: codex
Verification
python -m unittest discover -s tests(2,676 tests)ruff check .python -m buildpython -m twine check dist/*actionlintpython scripts/privacy_scan.py --json