Skip to content

cli: add provio log --json - #40

Merged
bhaskargurram-ai merged 2 commits into
mainfrom
fix-log-json
Oct 13, 2026
Merged

bhaskargurram-ai merged 2 commits into
mainfrom
fix-log-json

Conversation

@bhaskargurram-ai

Copy link
Copy Markdown
Member

Fixes #14.

provio log --json prints one JSON object per session (JSON Lines), with the same fields as the table: session_id, records, denied and approved_by_human. JSON Lines streams well into jq -c and needs no wrapping array. The default table output stays the same, and a missing ledger fails the same way with or without --json.

Adds integration tests in crates/provio-cli/tests/log_json.rs that drive the real binary through provio check --format codex.

Docs: README command table, docs/INTERFACES.md and the CHANGELOG.

🤖 Generated with Claude Code

https://claude.ai/code/session_017bQP5nwJVPMXqMhewzhQv5


Generated by Claude Code

Scripts and CI get one SessionSummary object per line (session_id,
records, denied, approved_by_human) instead of parsing the table. The
default output is unchanged; a missing ledger fails the same way.

Fixes #14

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_017bQP5nwJVPMXqMhewzhQv5
Signed-off-by: Bhaskar Gurram <gurrambhaskar.ai@gmail.com>
@bhaskargurram-ai
bhaskargurram-ai requested a review from a team as a code owner October 13, 2026 23:02
@github-actions

github-actions Bot commented Oct 13, 2026 •

Copy link
Copy Markdown

proofcheck: ✅ The PR's claims are backed by evidence

2/2 claims verified · 2 fail→pass, 1 pass→pass, 0 failing on head · 4/4 changed hunks are required by the evidence · 6 files (+115/−5)

Claims

Claim Evidence
✅ verified fixes #‍14
“Fixes #‍14.”
2 tests fail on base and pass on head
✅ verified Adds integration tests
“Adds integration tests in crates/provio-cli/tests/log_json.rs that drive the real binary through provio check --format codex.”
3 tests added/changed in 1 file

Tests: do the changed tests fail before the change and pass after it?

✅ At least one changed test fails on the base commit and passes on the PR head: it exercises the change.

Projects: rust (cargo): evidence, 2 fail→pass · python (pytest in adapters/python): no tests changed

Result Test Base Head
✅ fail → pass rust:crates/provio-cli/tests/log_json.rs::log_json_prints_one_object_per_session failed passed
✅ fail → pass rust:crates/provio-cli/tests/log_json.rs::log_json_without_a_ledger_fails_like_the_table failed passed
⚠️ pass → pass rust:crates/provio-cli/tests/log_json.rs::log_without_json_keeps_the_table passed passed

Issue reproduction

  • #‍14: No runnable ```python code block or >>> session found in the issue.

Diff evidence map

Each changed hunk was reverted on its own and the fail→pass tests / changed reproducers were re-run. ✅ required: reverting it breaks the evidence. ⚠️ not exercised: nothing that was measured depends on it.

Hunk Status Evidence that broke
crates/provio-cli/src/cmds.rs:341-344 ✅ required rust:crates/provio-cli/tests/log_json.rs::log_json_prints_one_object_per_session, rust:crates/provio-cli/tests/log_json.rs::log_json_without_a_ledger_fails_like_…
crates/provio-cli/src/cmds.rs:353-358 ✅ required rust:crates/provio-cli/tests/log_json.rs::log_json_prints_one_object_per_session
crates/provio-cli/src/main.rs:164-168 ✅ required rust:crates/provio-cli/tests/log_json.rs::log_json_prints_one_object_per_session, rust:crates/provio-cli/tests/log_json.rs::log_json_without_a_ledger_fails_like_…
crates/provio-cli/src/main.rs:314 ✅ required rust:crates/provio-cli/tests/log_json.rs::log_json_prints_one_object_per_session, rust:crates/provio-cli/tests/log_json.rs::log_json_without_a_ledger_fails_like_…

Scope

  • 6 files changed (+115 / −5 lines)

proofcheck judges evidence, not authorship: it runs the PR's tests on the base and head commits, replays issue reproducers and checks claims against what it measured. It does not guess whether a human or an AI wrote the code. ⚠️ means no evidence, not wrong. Base 2bc309e → head 99851a9 · proofcheck 1.0.0

@github-actions github-actions Bot added the proofcheck: verified proofcheck: the PR's claims are backed by executed evidence label Oct 13, 2026
Signed-off-by: Bhaskar Gurram <gurrambhaskar.ai@gmail.com>
@bhaskargurram-ai
bhaskargurram-ai merged commit cb4b645 into main Oct 13, 2026
34 of 35 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

proofcheck: verified proofcheck: the PR's claims are backed by executed evidence

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CLI: writ log --json for scripting

1 participant