Skip to content

fix(sourcehunt): bound machine progress records - #206

Merged
ropoctl merged 1 commit into
mainfrom
codex/sourcehunt-machine-record-boundaries
Sep 3, 2026
Merged

fix(sourcehunt): bound machine progress records#206
ropoctl merged 1 commit into
mainfrom
codex/sourcehunt-machine-record-boundaries

Conversation

@ropoctl

@ropoctl ropoctl commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Summary

  • enforce the machine protocol 1 MiB outgoing-record ceiling
  • project SourceHunt progress onto bounded metadata and counts instead of transporting bulk file, symbol, and finding-ID arrays
  • retain bounded terminal compaction as a final safety net
  • report staged early exits accurately instead of labeling deep runs as quick depth

Checkpoints remain workspace artifacts and do not cross the machine protocol.

Why

Machine-protocol records must remain bounded regardless of repository size. SourceHunt could complete preprocessing successfully and then fail while emitting an oversized progress or result record across FD3.

Progress is observability metadata, not artifact transport. This change makes that boundary explicit and keeps progress and terminal records within the protocol ceiling.

Validation

  • 49 passed, 1 deselected in focused machine and checkpoint tests
  • Ruff passed
  • git diff --check passed
  • live deep jq preprocess completed with a 378,808-byte filesystem checkpoint, a 1,291-byte maximum machine record, and no checkpoint in the terminal payload
  • verified depth remained deep and the staged-exit detail was Run stopped after preprocess

Open PR overlap

Reviewed the open SourceHunt PRs that touch runner.py or sourcehunt.py. They cover hunter progress limits, batch sizing, trace persistence, sandbox setup, and hunt/resume behavior; none owns the machine-record boundary or progress-event projection.

@ropoctl
ropoctl force-pushed the codex/sourcehunt-machine-record-boundaries branch from 14b02c9 to c782b1e Compare September 3, 2026 20:28
@ropoctl
ropoctl merged commit 1da3c22 into main Sep 3, 2026
1 check passed
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.

2 participants