Repository navigation
fix(android): deliver original caller instructions to native Codex - #976
Merged
Merged
Conversation
Signed-off-by: egerev <egor@gosync.ai>
egerev
marked this pull request as ready for review
November 13, 2026 10:51
egerev
enabled auto-merge
November 13, 2026 10:51
Signed-off-by: egerev <egor@gosync.ai>
egerev
force-pushed
the
bb/android-thr_x5spfefi64
branch
from
November 13, 2026 11:15
5113bc4 to
fee49a5
Compare
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.
Bare/native Android turns omitted owner_instruction and stringified task parameters, losing original words and restrictions. Deliver the five existing task-input fields losslessly, preserve natural replies, and reject changed typed input on the same request ID without another native turn. Provenance and enforced limits remain in transport/storage.
The first exact-head CI run also exposed test-process contamination: two in-process backup_runner.main call sites left umask0077 behind. Later root-owned namespace result files became0600 and unprivileged firewall tests could not read them. Restore the exact prior mask in finally at those two test boundaries; explicitly use022 only for synthetic namespace results inside private test directories. Production backup/firewall code, modes, assertions, timeout/error behavior and test identities remain unchanged.
Spec: ANYM-993, ANYM-971, ANYM-693, ANYM-727
Change path: AUTO (existing input-contract and narrowly allocated verification-fixture defects; no permissions, policy, CI or production change)
Review record: none (owner 11-11: no reviews)
Owner note: Original instructions and typed task values reach the Android agent alongside the goal.
Model: gpt-6.1-sol medium
Settle text: none
Validation:
Baseline: source d72 and private retrospective confirm omitted owner_instruction. This slice proves protocol input delivery and test isolation only; no speed or live-order completion claim. Root owns native dialogue/tier/context followups and integrated same-goal live acceptance.
Undo: ordinary source revert. No deployment/runtime change occurred.
Spec impact: