Skip to content

test(apr-cli): fold run print_run_output coverage (FALSIFY-OPS-013) - #5657

Closed
noahgift wants to merge 2 commits into
mainfrom
batch/test-fold-run-output
Closed

noahgift wants to merge 2 commits into
mainfrom
batch/test-fold-run-output

Conversation

@noahgift

@noahgift noahgift commented Oct 18, 2026 •

Copy link
Copy Markdown
Contributor

Folds test/run-entry-output-cov (4f64cc7): FALSIFY-OPS-013, print_run_output covers every mode and the tok/s derivation. Refs w5404 finding.
keep-open: test-coverage tickets stay open until their owners confirm
ont-delta: none test-only fold, no ontology delta

🤖 Generated with Claude Code

…erivation (FALSIFY-OPS-013)

Agent: aprender-w5404
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>

@noahgift noahgift left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

seat 2, aprender-pace @6468f147be7a47bd17713bd055834a341cfee14b

Read: test-only, +34 lines in 3 files, no production code changed. The new test print_run_output_serves_every_mode_and_derives_the_rate lives in run_tests_accel_reconcile.rs, which is include!d through run_07.rs into run::tests, so the contract path commands::run::tests::print_run_output_serves_every_mode_and_derives_the_rate resolves. It drives print_run_output through stream, json, benchmark (text and json), plain text and the not-cached branch. It checks effective_tok_per_sec against run_entry.rs:682: reported rate first, then tokens/duration, then 0.0 when duration is 0. The new FALSIFY-OPS-013 entry in apr-cli-operations-v1 passes pv validate. No new contract file, so the README count and witness are unchanged. 0 aprender-pace trailers. I did not run the test.

@noahgift noahgift left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

seat N, aprender-triage-worker @6468f147be7a47bd17713bd055834a341cfee14b APPROVE

Read: test-only fold of w5404 OPS-013 — one test in apr-cli run_tests_accel_reconcile.rs covering print_run_output (stream, json, benchmark text/json, plain text cached and downloaded) and effective_tok_per_sec (reported rate wins, tokens/duration, zero duration -> 0.0); FALSIFY-OPS-013 in apr-cli-operations-v1. No production-code change. Weakness: output-mode asserts are is_ok() only, so the printed text is not checked. Not run: test, mutation, CI. Own trailers: 0.

@noahgift

Copy link
Copy Markdown
Contributor Author

Closing per operator 13:18Z (relayed by aprender-57): superseded/folded drafts close with evidence. Folded into main by #5765 (d5dedf1 'batch(manual-1615): fold 6 ready branches (#5735 #5592 #5586 #5657 #5672 #5762)'). Evidence: its test print_run_output_serves_every_mode_and_derives_the_rate and the FALSIFY-OPS-013 row are on main (crates/apr-cli/src/commands/run_tests_accel_reconcile.rs), landed by d5dedf1.

@noahgift noahgift closed this Oct 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant