Skip to content

Deepen Claude and Grok Harness stream interpretation - #460

Merged
JacobStephens2 merged 1 commit into
mainfrom
issue-459
Oct 6, 2026
Merged

JacobStephens2 merged 1 commit into
mainfrom
issue-459

Conversation

@JacobStephens2

@JacobStephens2 JacobStephens2 commented Oct 6, 2026 •

Copy link
Copy Markdown
Owner

Summary

Preserve Claude and Grok progress, final messages, spend, failures and Resume behavior behind one Harness stream interpretation module.

 Claude adapter
-  ClaudeProgress
+  Stream::claude()
 Grok adapter
-  rewrite → serialize → ClaudeProgress → replace completed facts
+  Stream::grok()

Both dialects share one parse per line and init/tool/result handling; their outcome, diagnostics, summary selection and killed-work rules stay private. Delete the superseded decoders and move their behavioral coverage to the shared module.

Closes #459

Evidence

  • Before: All 14 new compatibility characterization tests passed against the original decoders.
    After: The same 14 tests pass against the shared decoder; all 127 Harness tests and 18 Grok session, progress and background-work scenarios pass.
  • Final validation: cargo check --all-targets, cargo fmt --check, cargo clippy --all-targets -- -D warnings, and cargo test (1238 passed, 0 failed, 0 ignored).

Test seams

  • Each real Claude/Grok Harness adapter's Interpretation: raw frames through condense, then consuming finish, observing progress, final text, summaries, outcome and Resume facts.

Unaddressed findings

Standards

None. The Standards review against main reported no actionable findings.

Spec

None. The Spec review against main reported no actionable findings.

Merge Danger

Door: two-way

An internal refactor with no persistent-data or execution-policy changes; revert restores the previous decoders.

Blast Radius: Sessions

Claude and Grok progress interpretation, reporting and Resume evidence.

Built with codex · gpt-6.1-sol · xhigh

@JacobStephens2
JacobStephens2 merged commit f0a48c4 into main Oct 6, 2026
13 checks passed
@JacobStephens2
JacobStephens2 deleted the issue-459 branch October 6, 2026 22:44
This was referenced Oct 7, 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.

Deepen Claude and Grok Harness stream interpretation

1 participant