Route Local Test-Runner Confusion to Verification Discipline

[ptr727]

A generic uv run pytest guessed without reading OPERATIONS.md fails to spawn in this repo's lint-only Python Scripts profile, and reads as a missing dependency rather than the profile working as intended.

GOVERNANCE.md "Verification Discipline" already states the underlying rule: read the check a change owes from OPERATIONS.md, not from a generic guessed command. This adds an explicit bullet naming the test-runner case, and an AGENTS.md routing-table row so the section is reachable without depending on the python-codestyle Skill firing, the same shape as #763's fix for the analogous lint-tool case.

Fixes#779

🤖 Generated with Claude Code

Summary by CodeRabbit

• Documentation
  • Clarified the recommended verification procedure for Python scripts.
  • Documented how to interpret unavailable or failing test runners.
  • Added guidance to use the repository’s documented verification command rather than generic test commands.

[qodo-code-review]

Code Review by Qodo

🐞 Bugs (0)📘 Rule violations (0)📎 Requirement gaps (0)

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Tip of the day

💡 Did you know, you can ask Qodo to dismiss a finding you disagree with, with your reason on record

More tips ↗ | Customize Qodo ↗ | Qodo docs ↗

[qodo-code-review]

PR Summary by Qodo

Route Local Test Failures to Repository Verification Guidance

📝 Documentation🕐 Less than 10 minutes

AI Description

• Route local test-runner problems directly to Verification Discipline.
• Clarify that lint-only profiles intentionally omit generic test runners.
• Require repository-declared verification commands instead of guessed pytest invocations.

Diagram

graph TD
Task(["Local test"]) --> Agents["AGENTS routing"] --> Governance["Verification discipline"] --> Operations["OPERATIONS command"] --> Check(["Repository check"])

Loading

High-Level Assessment

The direct documentation route is appropriate because it extends the existing Verification Discipline contract and makes it discoverable without relying on the python-codestyle Skill. Duplicating test commands in AGENTS.md or adding a dependency to lint-only profiles would weaken the repository-specific verification model.

Files changed (2) +2 / -0

Documentation (2) +2 / -0

AGENTS.mdRoute local test issues to Verification Discipline+1/-0

Route local test issues to Verification Discipline

• Adds a routing-table entry for local test execution and missing or unspawnable test runners. This makes the governing verification section discoverable without relying on a language-specific Skill trigger.

AGENTS.md

GOVERNANCE.mdClarify test-runner behavior in lint-only profiles+1/-0

Clarify test-runner behavior in lint-only profiles

• Explains that a generic pytest spawn failure is expected in lint-only Python Scripts profiles and does not prove tests are inapplicable. Directs contributors to use the verification invocation declared by OPERATIONS.md.

GOVERNANCE.md

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 90b997fa-23ad-459c-a50a-24164df871c6

📥 Commits

Reviewing files that changed from the base of the PR and between 12aa242 and d872eb5.

📒 Files selected for processing (2)

• AGENTS.md
• GOVERNANCE.md

Included review availability: Your plan provides up to 10 included reviews per hour; 8 remain after this review.

---

📝 Walkthrough

Walkthrough

The change adds verification guidance for lint-only Python Scripts profiles. It references the documented OPERATIONS.md command and python-codestyle Skill instead of requiring a generic uv run pytest command.

Changes

Verification guidance

Layer / File(s)	Summary
Document verification rules AGENTS.md, GOVERNANCE.md	Adds the Verification Discipline reference and clarifies that a failed uv run pytest spawn is expected for lint-only Python Scripts profiles. Verification must use the command documented in OPERATIONS.md and the python-codestyle Skill.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk:⚪ Minimal · up to d872e

This localized documentation change clarifies how to select the repository’s test command and adds a direct routing entry; no actionable merge-blocking risk remains beyond normal checks and review.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly describes the main change: routing local test-runner failures to the repository's verification guidance.
Linked Issues check	✅ Passed	The documentation changes address issue #779 by explaining that uv run pytest may fail to spawn in the lint-only Python Scripts profile and by directing agents to the documented verification command…
Out of Scope Changes check	✅ Passed	The changes are limited to related guidance in AGENTS.md and GOVERNANCE.md. No unrelated code or configuration changes are present.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…

Full details: Linked Issues check

Explanation

The documentation changes address issue #779 by explaining that uv run pytest may fail to spawn in the lint-only Python Scripts profile and by directing agents to the documented verification command.

Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (2 skipped: 2 unsupported.)

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix-779-test-invocation-routing

---

_{Comment @coderabbitai help to get the list of available commands.}