Uh oh!
There was an error while loading. Please reload this page.
feat(headless): report active prune evidence - #323
Merged
Conversation
23 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Add active tool-result prune diagnostics to Harbor cell output, A/B summaries, and markdown reports, including a B-active paired comparison slice.
Why
This covers part of the #293 phase-1 reporting gap for active tool-result pruning. Runtime already emits active prune diagnostics, but headless A/B reports previously only recognized stale
prunedToolResults. That made active-prune runs hard to inspect and could mix untriggered or unpaired attempts into safety conclusions.Scope
Changed:
activePrunedToolResults,activeEstimatedTokensSaved, andactiveArchiveFailuresin Harbor cell output.activeToolResultPrunein Harbor context budget policy snapshots.activePrunedToolResults > 0.(taskId, rep)attempts.Not included:
Verification
npm --workspace @maka/headless test -- --test-name-pattern "renders active prune subset pair coverage"failed because the active subset line omittedattempts/observed/missing/coverageand full token/cost fields.npm --workspace @maka/headless test -- --test-name-pattern "active prune subset|context budget activation": 490 pass, 0 fail. The package test script still ran the full@maka/headlesssuite.results.jsonl:/Users/yuhan/.local/maka-eval/runs/issue293-prune-ab/issue293-active-smoke-001/runtime-policy-ab-report.md/Users/yuhan/.local/maka-eval/runs/issue293-prune-ab/issue293-active-smoke-001count-dataset-tokens,extract-elfactivePrunedToolResults=1091activeEstimatedTokensSaved=1400611activeArchiveFailures=0User-facing impact
None. This is headless benchmark reporting only.
Reviewer notes
The smoke run showed active pruning activates and reports correctly, but both B attempts ended with
incomplete_tool_callsafter hitting the single-turn 50-step cap. That is a harness limitation for longer tasks, not a non-inferiority result. Formal #293 evidence should wait for either shorter stable tasks or a benchmark-safe multi-turn continuation harness.