Skip to content

Isolate tmp-dependent tests from ambient git#22199

Closed
starr-openai wants to merge 9 commits into
mainfrom
starr/remote-ci-test-cwd-no-tmp-20260511
Closed

Isolate tmp-dependent tests from ambient git#22199
starr-openai wants to merge 9 commits into
mainfrom
starr/remote-ci-test-cwd-no-tmp-20260511

Conversation

@starr-openai
Copy link
Copy Markdown
Contributor

@starr-openai starr-openai commented May 11, 2026

Why

The remote full-ci path has been picking up host-dependent git state from temp directories, which makes unrelated tests fail when /tmp or another ancestor looks like a repository. This PR removes that ambient dependency from the first layer of the CI reliability stack.

What changed

  • isolate project-root-sensitive test fixtures from ambient git metadata
  • make the TUI chatwidget test project path unique per test run, while normalizing snapshots back to /tmp/project
  • keep repo-root-sensitive core-skills and secrets tests from depending on whatever git markers happen to exist above their temp paths

Stack

  • PR #22199 Isolate tmp-dependent tests from ambient git
  • PR #22201 Use remote fs for turn diff repo root
  • PR #22200 Emit unified exec end on startup failure
  • PR #22389 Stabilize remote routing e2e tests

Validation

  • Current-head CI is green.
  • No local Codex test run; relying on CI for this stack.

@starr-openai starr-openai requested a review from a team as a code owner May 11, 2026 19:49
@starr-openai starr-openai force-pushed the starr/remote-ci-test-cwd-no-tmp-20260511 branch from 44df901 to 29912b9 Compare May 11, 2026 20:05
@starr-openai starr-openai changed the title Keep remote test cwd out of tmp Keep remote test temp paths out of tmp May 11, 2026
@starr-openai starr-openai force-pushed the starr/remote-ci-test-cwd-no-tmp-20260511 branch from 29912b9 to 2ae4060 Compare May 11, 2026 20:21
@starr-openai starr-openai changed the title Keep remote test temp paths out of tmp Keep remote tests off ambient tmp git May 11, 2026
@starr-openai starr-openai force-pushed the starr/remote-ci-test-cwd-no-tmp-20260511 branch from 2ae4060 to 25467e0 Compare May 11, 2026 20:35
@starr-openai starr-openai changed the title Keep remote tests off ambient tmp git Ignore ambient tmp git markers May 11, 2026
@starr-openai starr-openai force-pushed the starr/remote-ci-test-cwd-no-tmp-20260511 branch from 25467e0 to 93ea883 Compare May 11, 2026 21:03
@starr-openai starr-openai changed the title Ignore ambient tmp git markers Isolate tmp-dependent tests from ambient git May 11, 2026
@starr-openai starr-openai force-pushed the starr/remote-ci-test-cwd-no-tmp-20260511 branch 6 times, most recently from bfb9a7b to b1b01f5 Compare May 12, 2026 23:02
starr-openai added a commit that referenced this pull request May 13, 2026
Co-authored-by: Codex <noreply@openai.com>
starr-openai added a commit that referenced this pull request May 13, 2026
Co-authored-by: Codex <noreply@openai.com>
starr-openai added a commit that referenced this pull request May 13, 2026
Co-authored-by: Codex <noreply@openai.com>
starr-openai added a commit that referenced this pull request May 13, 2026
Co-authored-by: Codex <noreply@openai.com>
starr-openai added a commit that referenced this pull request May 13, 2026
Co-authored-by: Codex <noreply@openai.com>
starr-openai added a commit that referenced this pull request May 13, 2026
Co-authored-by: Codex <noreply@openai.com>
starr-openai added a commit that referenced this pull request May 13, 2026
Co-authored-by: Codex <noreply@openai.com>
starr-openai added a commit that referenced this pull request May 13, 2026
Co-authored-by: Codex <noreply@openai.com>
starr-openai added a commit that referenced this pull request May 13, 2026
Co-authored-by: Codex <noreply@openai.com>
starr-openai added a commit that referenced this pull request May 13, 2026
Co-authored-by: Codex <noreply@openai.com>
starr-openai added a commit that referenced this pull request May 13, 2026
Co-authored-by: Codex <noreply@openai.com>
@starr-openai starr-openai force-pushed the starr/remote-ci-test-cwd-no-tmp-20260511 branch from 7930961 to d9ba8e7 Compare May 13, 2026 01:27
starr-openai added a commit that referenced this pull request May 13, 2026
Co-authored-by: Codex <noreply@openai.com>
starr-openai added a commit that referenced this pull request May 13, 2026
Co-authored-by: Codex <noreply@openai.com>
starr-openai added a commit that referenced this pull request May 13, 2026
Co-authored-by: Codex <noreply@openai.com>
starr-openai added a commit that referenced this pull request May 13, 2026
Co-authored-by: Codex <noreply@openai.com>
starr-openai added a commit that referenced this pull request May 13, 2026
Co-authored-by: Codex <noreply@openai.com>
starr-openai added a commit that referenced this pull request May 13, 2026
Co-authored-by: Codex <noreply@openai.com>
starr-openai and others added 9 commits May 13, 2026 11:46
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
@starr-openai starr-openai force-pushed the starr/remote-ci-test-cwd-no-tmp-20260511 branch from d9ba8e7 to 0134886 Compare May 13, 2026 18:48
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