[vchamarthi]

Summary

Three repo commands lacked options their siblings already have, so a script or CI job had to parse stdout to use them.

• repo fetch --json prints {"sha", "git_dir"} instead of a tab-separated line.
• repo validate --out FILE and repo budget --out FILE write the full report as JSON, as repo scan and repo inspect do.
• repo validate --fail-on-warning exits 1 on a warning, as repo budget --fail-on-warning does.

Changes

• dokimi/dokimi/commands/repo.py: the four options above; a write failure for --out prints cannot write <path> and exits 1, as scan does.
• dokimi/docs/repo-evaluation-contract.md: the command synopsis lists the new options.
• Tests in tests/test_repo_manifest.py and tests/test_repo_budget.py; one changelog fragment.

Risk

Low. All four options are additive. Default output and exit codes of fetch, validate and budget are unchanged. No persisted format changes. The contract doc is also edited by several open PRs (#107, #113, #114, #115, #116, #120, #137), so expect a trivial rebase on the synopsis lines.

Testing

On Linux (Python 3.12): pytest tests/test_repo_manifest.py tests/test_repo_budget.py tests/test_docs_code_refs.py passes (235 tests); ruff check . and mypy dokimi are clean. The full pytest run is left to CI.

Checklist

• Tests added/updated and pytest passes (touched files and docs refs locally; full run in CI)
• ruff, mypy, and schema checks pass (no schema change)
• Docs updated (or N/A): contract synopsis updated
• No ad-hoc run artifacts committed
• New files carry the SPDX license header (or are exempt in .github/.licenserc.yaml): the only new file is a changelog fragment, which is exempt

[Copilot]

🟡 Changes recommended

Both new persisted report formats need registered versioned schemas and schema-aware artifact writes.

1 open issue

• Register schemas and version --out report artifacts · New

What changed in this PR

Adds output and warning-gate options to repository evaluation commands while preserving defaults.

Changes:

• Adds repo fetch --json.
• Adds --out to validation and budget commands, plus validation’s --fail-on-warning.
• Updates tests, documentation, and changelog.

File	Description
dokimi/​tests/​test_repo_manifest.py	Tests fetch JSON and validation options.
dokimi/​tests/​test_repo_budget.py	Tests budget file output and write failures.
dokimi/​dokimi/​commands/​repo.py	Implements and registers the new options.
dokimi/​docs/​repo-evaluation-contract.md	Updates command usage examples.
dokimi/​changelog.d/​20261021_130000_vchamarthi_repo-cli-output-parity.md	Documents the added options.

_{Review effort: Balanced}

[codecov-commenter]

⚠️ Please install the to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.40%. Comparing base (9ad5506) to head (6267044).
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #150      +/-   ##
==========================================
+ Coverage   91.38%   91.40%   +0.02%     
==========================================
  Files         216      216              
  Lines       26722    26738      +16     
  Branches     4373     4376       +3     
==========================================
+ Hits        24419    24441      +22     
+ Misses       1637     1633       -4     
+ Partials      666      664       -2     

Files with missing lines	Coverage Δ
dokimi/dokimi/commands/repo.py	96.34% <100.00%> (+0.90%)	⬆️

... and 2 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[vchamarthi]

@exai-bot /review

[exai-bot]

Note

AI-generated review of 9ee05ee33d58, requested by @vchamarthi, using the default review skill. It can be wrong: verify a finding before acting on it. React with a thumbs up or down on a finding to tell the bot whether it helped.
Model us.anthropic.claude-sonnet-5-5, effort default, 7 of 93 turns, 0.5 of 48 minutes, 121k in / 3k out tokens.

The change adds repo fetch --json, --out for repo validate and repo budget, and repo validate --fail-on-warning. All four are additive and the defaults are unchanged. I found no correctness bugs. The exit-code logic, error handling for --out write failures, and the tests (including the open_source monkeypatch and GitTree(git_dir, sha) signature) check out against the code. The one issue is that the new --out files are unversioned, unlike scan and inspect.

1 finding is on the changed lines, as comments below.

For a human reviewer

• Decide whether validate/budget --out files should be versioned artifacts like scan/inspect, or plain JSON.

[exai-bot]

Minor (plausible, maintainability): --out writes unstamped JSON, unlike scan/inspect which the docs say it matches

repo scan and repo inspect write --out through save_artifact(kind, ...). That call stamps the file with a schema version, and --json output is also stamped. The new validate --out and budget --out call Path.write_text(json.dumps(...)) directly, so the file has no schema/version stamp. The changelog and PR text say it works "as repo scan and repo inspect do", which is only partly true. A script that reads these files cannot tell the format version. Because the shape is not registered, it can change later without any compatibility check. The same write block is also duplicated in both commands. If an unversioned dump is intended, say so in the help text and docs. Otherwise, register artifact kinds for the validate report and the budget inventory.

Suggested fix:

Add artifact kinds (e.g. repo_validation, repo_budget) with schemas and use save_artifact, or document that these files are plain unversioned JSON.

[napetrov]

Additive, defaults unchanged, --out written before the exit so a failing validate still leaves its report. Good.

Pushed 6267044: the two new raw --out writes go through one _write_json helper instead of two copies of the try/except. (scan/inspect keep save_artifact, since they're schema-stamped.)

[Copilot]

🟢 Approval recommended

The changes are narrowly scoped, preserve defaults, and include focused regression tests with no unresolved findings.

1 open issue

• Register schemas and version --out report artifacts

_{Review effort: Balanced}