Skip to content

[codex-action] write temp output under RUNNER_TEMP#81

Draft
viyatb-oai wants to merge 1 commit intomainfrom
codex/viyatb/use-runner-temp-for-output
Draft

[codex-action] write temp output under RUNNER_TEMP#81
viyatb-oai wants to merge 1 commit intomainfrom
codex/viyatb/use-runner-temp-for-output

Conversation

@viyatb-oai
Copy link
Copy Markdown
Collaborator

@viyatb-oai viyatb-oai commented Mar 25, 2026

Summary

  • Move implicit codex exec temp output files from /tmp to $RUNNER_TEMP
  • Export the temp-root helper and guard main() so the behavior is testable
  • Add a regression test covering RUNNER_TEMP precedence

Root Cause

The action created --output-last-message files under shared /tmp, which is writable from the Codex sandbox in the default workspace-write configuration. A symlink planted in that location could redirect the host-side final-message write to arbitrary files such as $GITHUB_ENV or $GITHUB_OUTPUT.

Validation

  • pnpm test
  • pnpm run check

@viyatb-oai viyatb-oai requested a review from bolinfest March 25, 2026 06:55
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