Repository navigation
Integrate the verified ACP baseline and reliability stack - #161
Merged
Merged
Conversation
…' into stack/acp-runtime-base
# Conflicts: # .github/workflows/acp.yml
Maximooch
marked this pull request as ready for review
October 28, 2026 22:23
This was referenced Oct 28, 2026
This branch was successfully deployed
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 free
to 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.
This integration preserves the commit history of all 25 ACP drafts and three reliability prerequisites (#133–#160), then closes baseline gaps found with a live ChatGPT-backed model. The primary checkout is untouched.
Live
gpt-6.1-solchecks through Penguin's existing OAuth login passed: MCP discovery/invocation, native file read/write with approval, a rejected write leaving the preexisting canary unchanged with no alternative tool attempt, PNG plus embedded context, close/load and fresh-process recall, cancellation, and two isolated parallel writes with separate response streams. The shipped CLI now activates only its configured provider's saved login; no Codex/Minecraft credentials are imported.Live-found fixes included here:
Validation: all 306 selected tests passed on final integration head
7f855cc85b8c071d79bdf641dbb67aeed8a828ad, including native Windows processes, storage, ACP wire contracts, permissions, startup, messaging, and default-runtime regressions. All executed pull-request CI checks passed, including the four Windows/Linux ACP jobs across Python 3.11 and 3.12. Merged asabd8a663a073fae77ec35154ef8f9fbcb1a1e14f; its tree is identical to the tested head. Post-merge CI is rerunning normally.No purchase or new subscription was made. Documented optional-feature and recovery limits remain in
context/tasks/acp.mdanddocs/ACP.md. Permission enforcement is an application policy, not an OS sandbox. Parallel root runtimes remain opt-in; live child processes are not recovered after adapter death.