Skip to content

Address image harness review findings#75

Merged
cbusillo merged 1 commit into
local/cbusillo-overlayfrom
fix/image-harness-review-followup
May 13, 2026
Merged

Address image harness review findings#75
cbusillo merged 1 commit into
local/cbusillo-overlayfrom
fix/image-harness-review-followup

Conversation

@cbusillo
Copy link
Copy Markdown
Owner

Summary

  • address Claude/Gemini post-merge review findings from the no-token image replay harness
  • make fake Responses request recording and response assignment atomic
  • keep parsed command metadata for summaries while preserving readable scenario command strings
  • scope image replay assertions to request input and require the 65536 bytes placeholder detail

Validation

  • python3 -m py_compile tools/code-exec-harness/harness.py
  • python3 -m json.tool tools/code-exec-harness/scenarios/image-history-replay.json >/dev/null
  • python3 tools/code-exec-harness/harness.py tools/code-exec-harness/scenarios/image-history-replay.json --code-bin /Users/cbusillo/.local/bin/code --sandbox danger-full-access
  • ./build-fast.sh

Refs #68
Refs #63

@cbusillo cbusillo merged commit 65bcfda into local/cbusillo-overlay May 13, 2026
1 check passed
@cbusillo cbusillo deleted the fix/image-harness-review-followup branch May 13, 2026 21:25
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