Skip to content

Support agy 1.5.7 trajectories and record full qualification - #10

Merged
mjacobs merged 6 commits into
mainfrom
review/agy-1.5.7-support
Nov 12, 2026
Merged

mjacobs merged 6 commits into
mainfrom
review/agy-1.5.7-support

Conversation

@mjacobs

@mjacobs mjacobs commented Nov 12, 2026 •

Copy link
Copy Markdown
Owner

agy 1.5.7 permission and background-task conversations could fail completeness verification even when their native SQLite data was intact. This change qualifies the captured forms, preserves execution metadata in transcripts, and reduces repeated watcher work when a daemon serves an older in-memory trajectory.

The decoder adds only the authenticated prompt-template map to the historical schema. It retains full-source comparison and the existing explicit preservation of retired subagent model choices. Task input, approvals, denials, canceled reads, running/canceled commands, cron results and swarm model enumeration have bounded projection rules and regression fixtures.

Markdown uses linear fence selection and displays saved tool Markdown literally, so embedded backticks, unfinished fences and HTML cannot consume later steps. Transcripts include hook results, permission interactions and task details. Unchanged content mismatches get a 30-second retry delay on Linux and macOS, with immediate retry after source or endpoint changes. macOS file identities support these holds while successful verification remains uncached without a boot identifier. IPv6 discovery keeps credentials tied to the listener's address family and ownership.

Validation:

  • The 1.5.7 protobuf snapshot verifies 401 binary byte ranges and a 34-file closure/round trip.
  • All 506 copied conversations qualify: 496 ordinary verifications and ten with the existing declared source-only fields.
  • Six fresh runtime conversations match offline export, RPC and verified sync exactly on steps and generator/executor metadata; all 506 captured trajectories render successfully.
  • Local Go checks, race tests, lint and Python maintenance tests passed. Completed automatic reviews have no unresolved findings.

The final compatibility record replaces the earlier intermediate records. Stale daemon caching, tokenless-daemon authentication and unexercised organization-policy branches remain documented limits. No deployment or live-service restart is included.

Qualify retained cron schedules, board reads, artifact operations and daemon
commands. Preserve retired subagent model choices in indexed source-only records
when current RPC omits them, while comparing every remaining field in full.
Preserve 401 embedded records, the 34-file trajectory closure and its verified
protoc round trip. The snapshot provides schema provenance without changing
production compatibility.
Decode authenticated prompt-template overrides and qualify captured task input,
approvals, denials, canceled reads, background commands and cron/swarm results
without weakening full-source comparison or adding a source-only exemption.

Show execution records and task output in Markdown. Select fences in one linear
scan and contain saved Markdown so unfinished blocks cannot absorb later steps.
Back off unchanged daemon mismatches for 30 seconds, explain sync reasons and
discover IPv6 loopback listeners without crossing credential families.

The isolated audit verifies all 506 sources, including ten with existing
source-only records. Six fresh runtime conversations match offline export, RPC
and verified sync. Go checks, race tests and Python maintenance checks pass.
Keep the final qualified record in both the root and doctor embed: 506 native
sources, ten with the existing source-only declaration, and the version-scoped
trajectory fingerprint. Preserve the original audit provenance unchanged.
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Nov 12, 2026 •

Copy link
Copy Markdown
Codex Review Summary: ✅ Completed

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-11-12T22:44:11.008555Z f91e7e1 New commits
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

Use macOS file identity and change timestamps to retain in-process refusal
holds. Keep successful verification uncached without a boot identifier.

Cover same-size atomic replacement with restored modification time and
continued content verification. Existing watcher tests cover retry suppression,
deadline recovery, and invalidation on source or daemon changes.
Use the lifecycle suite startup allowance instead of a 0.4-second deadline
that can expire before macOS starts the fixture. Delay the rejecting listener
to cover startup slower than the old deadline while retaining authentication,
timeout, and child-cleanup assertions.
@mjacobs
mjacobs merged commit f91e7e1 into main Nov 12, 2026
3 checks passed
@mjacobs
mjacobs deleted the review/agy-1.5.7-support branch November 12, 2026 23:03
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