Skip to content

test(server): isolate agent-task-recovery fixtures (#7613) - #7649

Merged
lidge-jun merged 3 commits into
devfrom
lane/l2-test-flakes
Oct 24, 2026
Merged

lidge-jun merged 3 commits into
devfrom
lane/l2-test-flakes

Conversation

@lidge-jun

@lidge-jun lidge-jun commented Oct 24, 2026 •

Copy link
Copy Markdown
Owner

Summary

  • Closes test(server): agent-task-recovery Retry-After floor case flakes under load #7613. Scope the transient retry stubs to each case's recovery endpoint and forwarded user-agent, with distinct synthetic account identities. Foreign sends cannot advance another case's fetch count or timestamps; unexpected sends are recorded even if recovery catches their rejection.
  • Track the cache resolver's actual request callback through response processing and deadline cleanup. Teardown aborts shared work, releases deferred mocks, drains callbacks and handlers, and restores fetch before releasing the spend-home lease. This preserves the existing 2,400 ms floor assertion and 10,000 ms case timeout. No production recovery behavior changes.
  • Decision: keep this fix in tests and one new fixture helper; shared layout registrations and size caps are unchanged. The historical stray request is still unattributed: an unmodified pinned trace passed 495/495, while deterministic synthetic tests exposed the isolation and teardown defects. The separate Remote Link focus issue is handled independently. Local workflow roadmap files remain outside this PR's write scope.

Verification

  • Bun 1.4.3: three synthetic fixture regressions failed against extracted legacy behavior (foreign fetch resolved, close completed early, gate stayed unreleased), then passed after the fix. Final fixture checks: 4 pass / 0 fail.
  • bun test tests/server/agent-task-recovery.test.ts with pinned Bun 1.4.3: exit 0; 103 pass / 0 fail, including the unchanged Retry-After floor.
  • bun run typecheck: exit 0. Additional strict helper typecheck: exit 0 (tsc --ignoreConfig --noEmit --strict --skipLibCheck --target esnext --module esnext --moduleResolution bundler --resolveJsonModule --esModuleInterop --types bun-types tests/helpers/agent-task-recovery-fixture.ts).
  • bun run privacy:scan and bun run structure:check: exit 0.
  • bun run test:changed: exit 0; 103 pass / 0 fail across the changed test file at head 876aa4defa4eb32ddb0fdf442ed689c42a85b4e8. OCX_TEST_RUNNER_BUN points to the locally downloaded 1.4.3 binary.
  • A natural-load full root run started before the patch. It completed with exit 1: 52,138 pass / 162 skip / 1 fail (the parallel lane was 51,400 pass / 1 fail); the failure is the unrelated connected Desktop uses hub IDs across a cold restart (stored profile=false) at tests/claude-integration/claude-desktop-remote-hub.test.ts:249. All 21 serial lanes passed. The run began on the pre-patch base and does not certify the final head. This is not final-head passing evidence. No test was skipped or weakened. A second full local run is disproportionate while concurrent lanes share this host and the first run already takes several minutes; final-head focused/import-connected tests and Ubuntu CI cover this fixture change.
  • NOT RUN: a second full root suite on the final head. Independent gpt-6.1-sol high review at 876aa4defa4eb32ddb0fdf442ed689c42a85b4e8: PASS, no findings; exact-head Ubuntu CI is queued; all required local validation and the independent review are complete; this PR remains draft while exact-head CI is queued.

Checklist

  • Scope stays focused and avoids unrelated cleanup.
  • Docs or release notes were updated when needed (test-only; runtime documentation remains accurate).
  • Security-sensitive changes were reviewed for secrets, auth, and unsafe defaults (synthetic fixture identities only; no runtime security boundary changed).

@coderabbitai

coderabbitai Bot commented Oct 24, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

Caution

Review failed

The pull request is closed.

Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions

Copy link
Copy Markdown
Contributor

✅ Deterministic PR hygiene checks passed.

@github-actions github-actions Bot added the chore Maintenance, CI, tests, refactors, or build changes (not a user-facing bug or feature). label Oct 24, 2026
@lidge-jun

lidge-jun commented Oct 24, 2026 •

Copy link
Copy Markdown
Owner Author

Coverage status: PROVISIONAL — the manager’s updated rule requires all omitted batches to execute. The earlier PASS is not a final verdict; a full-coverage addendum will report the remainder results and final verdict. Original command exits/counts below remain unchanged.

Local CI-equivalent (macOS) at f749269

Overall: PROVISIONAL — full-suite coverage pending under the updated verification rule.

Host: juns-Mac-Studio.local; macOS 26.6.2 (25G83), Darwin arm64. Scratch-installed test runner: Bun 1.4.3; package runtime pin (dependencies.bun): 1.4.4.

Git setup: fetch pull head, resolve FETCH_HEAD, confirm detached HEAD, and compute merge-base completed with exit 0; no test counts apply. Head was re-fetched after verification to check for movement.
Detached HEAD: f74926974ebaf303a14079e8a5c50590e6405543. Merge base with fetched origin/dev: 47316142fe06171a94de02d63f1727dbed4c0382. Reference base: 47316142fe06171a94de02d63f1727dbed4c0382.

Workflow source: .github/workflows/ci.yml from fetched origin/dev (retained with logs). Root/GUI frozen installs, GUI build, and bounded coreutils setup were shared by the jobs; passing setup/gates were not repeated. No PR branch changes, push, approval, or merge.

Environment: CI=true RUNNER_OS=macOS RUNNER_ARCH=ARM64; scratch Bun first on PATH, GNU coreutils gnubin next; GIT_DIR, GIT_INDEX_FILE, GIT_WORK_TREE and all inherited GIT_*, BUN_TEST_*, OCX_TEST_* overrides explicitly removed before checks. Tests ran directly outside hooks.

Suite environment: BUN_TEST_FILE_SCOPE=all BUN_TEST_BATCH_SIZE=12 BUN_TEST_PARALLEL=1 BUN_TEST_BATCH_TIMEOUT_SECONDS=300 OCX_TEST_NO_QUEUE=1 OCX_TEST_FULL_SUITE=1; TEST_SHARD = 1/3, 2/3, 3/3, then control 1/1. Commands retained the CI runner’s first-failure exit behavior, 300-second batch deadlines, 20-minute platform and 75-minute control caps. No retry changed a failure into a pass.

Command (working directory) Exit Pass / fail / skip Seconds
command -v bun; bun --version; node --version; command -v timeout; timeout --version | head -1 (root) 0 n/a 0.02
bun install --frozen-lockfile (root) 0 n/a 1.06
bun install --frozen-lockfile (gui/) 0 n/a 0.15
bun run build (gui/) 0 n/a 7.95
brew list coreutils >/dev/null 2>&1 || brew install coreutils; brew --prefix coreutils; command -v timeout (root) 0 n/a 0.55
bun run lint (gui/) 0 n/a 0.89
bun x tsc --noEmit (root) 0 n/a 2.04
bun x tsc --noEmit -p tests/tsconfig.doctor-service-memory-contract.json (root) 0 n/a 1.46
bun x tsc --ignoreConfig --noEmit --strict --target ESNext --module ESNext --moduleResolution bundler --types bun-types --skipLibCheck scripts/ci/docker-smoke.ts (root) 0 n/a 0.26
bun test --isolate tests (gui/) 0 3493 / 0 / 0 110.93
bun run privacy:scan (root) 0 n/a 1.54
bun run skill:surface:check (root) 0 n/a 0.09
bun build scripts/release.ts --target=bun --outdir=/var/folders/h7/f1j0n8vj6ggcy9wprpxkfdsc0000gn/T/ocx-macos-ci-h860y3r5/results-7649-f7492697/release-check (root) 0 n/a 0.05
bun run src/cli/index.ts help (root) 0 n/a 0.21
bun run structure:check (root) 0 n/a 0.14
bun test --isolate ./tests/storage/api-storage-policy-already-running.test.ts ./tests/storage/api-storage-policy-mutation-busy.test.ts ./tests/storage/api-storage-policy-put-race.test.ts ./tests/storage/api-storage-policy-run.test.ts ./tests/storage/api-storage-policy.test.ts ./tests/storage/api-storage.test.ts (root) 0 9 / 0 / 0 4.47
bun test --isolate ./tests/server/api-usage.test.ts (root) 0 34 / 0 / 0 1.08
bash scripts/ci/run-bun-test-batches.sh "$TEST_SHARD" (env TEST_SHARD=1/3) (root) 1 2046 / 1 / 0 103.41
bash scripts/ci/run-bun-test-batches.sh "$TEST_SHARD" (env TEST_SHARD=2/3) (root) 1 2636 / 1 / 3 103.54
bash scripts/ci/run-bun-test-batches.sh "$TEST_SHARD" (env TEST_SHARD=3/3) (root) 0 17603 / 0 / 35 607.37
bash scripts/ci/run-bun-test-batches.sh "$TEST_SHARD" (env TEST_SHARD=1/1) (root) 1 6509 / 1 / 10 279.89

Failures and base comparisons:

Base scratch setup: bun install --frozen-lockfile at root exit 0, same command in gui/ exit 0, and bun run build in gui/ exit 0 (test counts n/a). Base comparisons use the common suite environment above with TEST_SHARD=1/3.

  • BASE-IDENTICAL tests/claude-integration/claude-desktop-remote-hub.test.ts: connected Desktop uses hub IDs across a cold restart (stored profile=false); assertion at :249 (same assertion on base).
    Base 47316142fe06171a94de02d63f1727dbed4c0382: bun test --isolate --parallel=1 --timeout 60000 ./tests/claude-integration/claude-desktop-remote-hub.test.ts, exit 1, 1 pass / 1 fail / 0 skip. Same scratch Bun and suite environment; one run per distinct file, reused across PRs. This comparison does not erase the head failure.
  • BASE-IDENTICAL tests/cli/cli-codex-cli-update-boundary.test.ts: Codex CLI selection inspection boundary > plan identity binds the bytes a real npm-global install placed; assertion at :287: expected blocked_active_session, received target_not_newer (same on base).
    Base 47316142fe06171a94de02d63f1727dbed4c0382: bun test --isolate --parallel=1 --timeout 60000 ./tests/cli/cli-codex-cli-update-boundary.test.ts, exit 1, 20 pass / 1 fail / 2 skip. Same scratch Bun and suite environment; one run per distinct file, reused across PRs. This comparison does not erase the head failure.

NOT RUN:

  • 1/3: batches 12–88 (CI runner stopped on its first failure).
  • suite-1-3: job-tail CLI help smoke after failed test step (the shared gates CLI help was run successfully).
  • 2/3: batches 15–85 (CI runner stopped on its first failure).
  • suite-2-3: job-tail CLI help smoke after failed test step (the shared gates CLI help was run successfully).
  • 1/1: batches 33–253 (CI runner stopped on its first failure).
  • suite-1-1: job-tail CLI help smoke after failed test step (the shared gates CLI help was run successfully).
  • Hosted Actions and other platform jobs are outside this macOS leg. No docs-only PR was included in the six-PR queue.

Raw command logs and structured results retained on this host in /var/folders/h7/f1j0n8vj6ggcy9wprpxkfdsc0000gn/T/ocx-macos-ci-h860y3r5/results-7649-f7492697; base comparison logs in /var/folders/h7/f1j0n8vj6ggcy9wprpxkfdsc0000gn/T/ocx-macos-ci-h860y3r5/base-results. Counts are the tests actually executed before first-failure termination, not the entire repository test count.

@lidge-jun

Copy link
Copy Markdown
Owner Author

Local CI-equivalent (Linux) at f749269

Final verdict: PASS (temporary rule; raw base-identical test failures retained). This includes every primary batch selected by all four Linux shards.

Host lidge-AI-AI, Ubuntu 24.04.4 LTS x86_64, kernel 7.0.0-30-generic. Workflow authority: origin/dev at 8066099651f8ed356bb73a67d9c99978ea65447c. Bun runner/gates 1.4.3; runtime pin 1.4.4. Detached exact head f74926974ebaf303a14079e8a5c50590e6405543; brief head f74926974ebaf303a14079e8a5c50590e6405543; moved from brief False. Base 47316142fe06171a94de02d63f1727dbed4c0382; observed merge-base with origin/dev 47316142fe06171a94de02d63f1727dbed4c0382.

All new suite/base invocations use documented OPENCODEX_TEST_TRUSTED_FIXTURE_DIR=/home/lidgeai/ocx-linux-ci.p2R3NT/trusted-fixtures (new private directory, mode 0700 under trusted ownership). Original results remain unchanged; no completed head batch was retried. GIT_DIR, GIT_INDEX_FILE, and GIT_WORK_TREE are explicitly absent. CI=true; shard scope general, batch size at most 12, no parallel override, per-test timeout 60000 ms, batch process deadline 120 s, TERM/KILL grace 15 s.

Batch file lists are generated by the exact head’s original planner. Only its script-directory anchors point back to the original location; its test command/deadlines are unchanged. Every previously executed batch file list was checked against the reconstructed manifest. Remaining batches ran serially; any new failure stayed failed and did not suppress later missing batches.

Shard Original batches executed Remaining batches executed Required total Coverage
1/4 63 0 63 COMPLETE
2/4 58 0 58 COMPLETE
3/4 14 50 64 COMPLETE
4/4 10 56 66 COMPLETE

Original CI commands (no reruns):

Command Exit Pass / fail / skip
bun install --frozen-lockfile 0 N/A
cd gui && bun install --frozen-lockfile 0 N/A
cd gui && bun run build 0 N/A
bash scripts/ci/run-bun-test-batches.sh 1/4 0 13577/0/35
bash scripts/ci/run-bun-test-batches.sh 2/4 0 12304/0/50
bash scripts/ci/run-bun-test-batches.sh 3/4 1 2566/0/3
bash scripts/ci/run-bun-test-batches.sh 4/4 1 1887/1/0
cd gui && bun run lint 0 N/A
bun x tsc --noEmit 0 N/A
bun x tsc --noEmit -p tests/tsconfig.doctor-service-memory-contract.json 0 N/A
bun x tsc --ignoreConfig --noEmit --strict --target ESNext --module ESNext --moduleResolution bundler --types bun-types --skipLibCheck scripts/ci/docker-smoke.ts 0 N/A
cd gui && bun test --isolate tests 0 3493/0/0
bun run privacy:scan 0 N/A
bun run skill:surface:check 0 N/A
bun build scripts/release.ts --target=bun --outdir=/home/lidgeai/ocx-linux-ci.p2R3NT/evidence-7649/release-build 0 N/A
bun run src/cli/index.ts help 0 N/A
bun run structure:check 0 N/A
bun test --isolate ./tests/storage/api-storage-policy-already-running.test.ts ./tests/storage/api-storage-policy-mutation-busy.test.ts ./tests/storage/api-storage-policy-put-race.test.ts ./tests/storage/api-storage-policy-run.test.ts ./tests/storage/api-storage-policy.test.ts ./tests/storage/api-storage.test.ts 0 9/0/0
bun test --isolate ./tests/server/api-usage.test.ts 0 34/0/0

Remaining primary batches — each command is timeout --signal=TERM --kill-after=15s 120s bun test --isolate --timeout 60000 <exact manifest files>:

Shard / batch Exit Pass Fail Skip
3/4, 15/64 0 302 0 0
3/4, 16/64 0 183 0 30
3/4, 17/64 0 163 0 2
3/4, 18/64 0 127 0 2
3/4, 19/64 0 133 0 0
3/4, 20/64 0 151 0 1
3/4, 21/64 0 302 0 1
3/4, 22/64 0 384 0 0
3/4, 23/64 0 436 0 0
3/4, 24/64 0 81 0 0
3/4, 25/64 0 81 0 23
3/4, 26/64 0 278 0 1
3/4, 27/64 0 356 0 0
3/4, 28/64 0 300 0 1
3/4, 29/64 0 190 0 0
3/4, 30/64 0 188 0 1
3/4, 31/64 0 135 0 0
3/4, 32/64 0 158 0 0
3/4, 33/64 0 186 0 0
3/4, 34/64 0 188 0 0
3/4, 35/64 0 186 0 0
3/4, 36/64 0 49 0 0
3/4, 37/64 0 18 0 0
3/4, 38/64 0 170 0 0
3/4, 39/64 0 202 0 0
3/4, 40/64 0 157 0 0
3/4, 41/64 0 226 0 0
3/4, 42/64 0 292 0 0
3/4, 43/64 0 104 0 0
3/4, 44/64 0 311 0 0
3/4, 45/64 0 324 0 0
3/4, 46/64 0 497 0 0
3/4, 47/64 0 245 0 0
3/4, 48/64 0 266 0 0
3/4, 49/64 0 522 0 1
3/4, 50/64 0 193 0 0
3/4, 51/64 0 328 0 2
3/4, 52/64 0 240 0 0
3/4, 53/64 0 118 0 0
3/4, 54/64 0 275 0 0
3/4, 55/64 0 158 0 0
3/4, 56/64 0 7 0 0
3/4, 57/64 0 55 0 0
3/4, 58/64 0 130 0 0
3/4, 59/64 0 102 0 0
3/4, 60/64 0 46 0 0
3/4, 61/64 0 217 0 0
3/4, 62/64 0 339 0 0
3/4, 63/64 0 200 0 0
3/4, 64/64 0 209 0 1
4/4, 11/66 0 275 0 0
4/4, 12/66 0 344 0 0
4/4, 13/66 0 291 0 0
4/4, 14/66 0 484 0 0
4/4, 15/66 0 347 0 0
4/4, 16/66 0 177 0 0
4/4, 17/66 0 237 0 2
4/4, 18/66 0 223 0 0
4/4, 19/66 0 231 0 1
4/4, 20/66 0 216 0 0
4/4, 21/66 0 321 0 1
4/4, 22/66 0 180 0 2
4/4, 23/66 0 299 0 1
4/4, 24/66 0 213 0 0
4/4, 25/66 0 39 0 0
4/4, 26/66 0 17 0 0
4/4, 27/66 0 299 0 0
4/4, 28/66 0 221 0 1
4/4, 29/66 0 149 0 0
4/4, 30/66 0 113 0 0
4/4, 31/66 0 152 0 0
4/4, 32/66 0 251 0 0
4/4, 33/66 0 155 0 0
4/4, 34/66 0 141 0 0
4/4, 35/66 0 281 0 0
4/4, 36/66 0 300 0 0
4/4, 37/66 0 403 0 0
4/4, 38/66 0 214 0 0
4/4, 39/66 0 209 0 0
4/4, 40/66 0 160 0 0
4/4, 41/66 0 275 0 0
4/4, 42/66 0 187 0 0
4/4, 43/66 0 138 0 0
4/4, 44/66 0 330 0 0
4/4, 45/66 0 246 0 0
4/4, 46/66 0 172 0 0
4/4, 47/66 0 83 0 0
4/4, 48/66 0 506 0 1
4/4, 49/66 0 301 0 0
4/4, 50/66 0 10 0 0
4/4, 51/66 0 26 0 0
4/4, 52/66 0 175 0 0
4/4, 53/66 0 157 0 0
4/4, 54/66 0 294 0 0
4/4, 55/66 0 218 0 1
4/4, 56/66 0 332 0 1
4/4, 57/66 0 337 0 0
4/4, 58/66 0 10 0 0
4/4, 59/66 0 8 0 0
4/4, 60/66 0 166 0 0
4/4, 61/66 0 4 0 0
4/4, 62/66 0 86 0 0
4/4, 63/66 0 26 0 0
4/4, 64/66 0 142 0 2
4/4, 65/66 0 79 0 0
4/4, 66/66 0 9 0 0

Remainder totals: 21767 pass / 0 fail / 79 skip. Combined original + remainder command counts: 55637 pass / 1 fail / 167 skip. Failed checks remain failed; final disposition uses only the explicit BASE-IDENTICAL exception.

Failure classification across original and remainder evidence:

  • BASE-IDENTICAL — test-3-of-4 shard 3/4 batch 14/64 (12 files): process exit 1 with batch counts {'pass': 336, 'fail': 0, 'skip': 0}. Exact 12-file base batch exit 1, counts {'pass': 336, 'fail': 0, 'skip': 0, 'todo': 0}. Root cause unattributed; failed process status retained.
  • BASE-IDENTICAL — tests/claude-integration/claude-desktop-remote-hub.test.ts:249 — connected Desktop uses hub IDs across a cold restart (stored profile=false). Base exit 1, 1/1/0 pass/fail/skip; same failing label reproduced.

Base comparisons (cached once per distinct file/exact failed batch):

  • bun test --isolate --timeout 60000 tests/cli/cli-link-enrollment.test.ts tests/cli/cli-link-force.test.ts tests/cli/cli-logs-request-id.test.ts tests/cli/cli-models-display-name.test.ts tests/cli/cli-models-local-result.test.ts tests/cli/cli-models-order.test.ts tests/cli/cli-models-prompt-length-pricing.test.ts tests/cli/cli-models-reasoning.test.ts tests/cli/cli-native-profile.test.ts tests/cli/cli-observe-snapshot.test.ts tests/cli/cli-provider-sync-result.test.ts tests/cli/cli-resolve.test.ts (cwd /home/lidgeai/ocx-linux-ci.p2R3NT/base-47316142fe, Bun 1.4.3): exit 1; 336 pass / 0 fail / 0 skip. Log /home/lidgeai/ocx-linux-ci.p2R3NT/base-evidence/batch-a122bb89d34e55.log; cached once per distinct file/batch.
  • bun test --isolate --timeout 60000 ./tests/claude-integration/claude-desktop-remote-hub.test.ts (cwd /tmp/ocx-linux-ci.MlWNCd/base-47316142fe, Bun 1.4.3): exit 1; 1 pass / 1 fail / 0 skip. Log /tmp/ocx-linux-ci.MlWNCd/base-evidence/tests__claude-integration__claude-desktop-remote-hub.test.ts.log; cached once per distinct file/batch.
  • bun install --frozen-lockfile (cwd /home/lidgeai/ocx-linux-ci.p2R3NT/base-47316142fe, Bun 1.4.3): exit 0; 0 pass / 0 fail / 0 skip. Log /home/lidgeai/ocx-linux-ci.p2R3NT/base-evidence/install-root.log; cached once per distinct file/batch.
  • bun install --frozen-lockfile (cwd /home/lidgeai/ocx-linux-ci.p2R3NT/base-47316142fe/gui, Bun 1.4.3): exit 0; 0 pass / 0 fail / 0 skip. Log /home/lidgeai/ocx-linux-ci.p2R3NT/base-evidence/install-gui.log; cached once per distinct file/batch.
  • bun run build (cwd /home/lidgeai/ocx-linux-ci.p2R3NT/base-47316142fe/gui, Bun 1.4.3): exit 0; 0 pass / 0 fail / 0 skip. Log /home/lidgeai/ocx-linux-ci.p2R3NT/base-evidence/build-gui.log; cached once per distinct file/batch.

Every original non-test gate result is retained in the original-command table above. No source edits, pushes, approvals or merges were performed. Source status: clean (git status --short empty); HEAD unchanged. Final fetched head f74926974ebaf303a14079e8a5c50590e6405543.

NOT RUN: Hosted CI and platforms/jobs outside the requested Linux scope

Exact manifests and raw logs (retained on the named Linux host):

  • Shard 3: /home/lidgeai/ocx-linux-ci.p2R3NT/evidence-7649/remainder-3-manifest.tsv, SHA-256 67a06d3eaf96faede124c4129dde93eaa4abef726f1e50bc7c92bf0b5237f811; planner validation True.
  • Shard 4: /home/lidgeai/ocx-linux-ci.p2R3NT/evidence-7649/remainder-4-manifest.tsv, SHA-256 eef290af4ed4e4c2b68ec95d4d3e49ba19aa1f3d59d9421503863221e37fb1a1; planner validation True.
  • Per-batch argv/exits/counts: /home/lidgeai/ocx-linux-ci.p2R3NT/evidence-7649/remainder-results.json; all original commands: /home/lidgeai/ocx-linux-ci.p2R3NT/evidence-7649/results.json; final evidence: /home/lidgeai/ocx-linux-ci.p2R3NT/evidence-7649/coverage-analysis.json.

UTC: 2026-10-24T11:45:56.021210+00:00

@lidge-jun
lidge-jun marked this pull request as ready for review October 24, 2026 12:44
@lidge-jun
lidge-jun requested a review from Ingwannu as a code owner October 24, 2026 12:44
@lidge-jun

Copy link
Copy Markdown
Owner Author

Integration evidence for exact head f749269 (lane L2):

  • Independent review: fresh gpt-6.1-sol PASS, 0 findings.
  • Lane verification: synthetic regressions red (0/3) before the fix and green after; pinned agent-task-recovery suite 103/0; repeated natural trace 495/0; typecheck, privacy, structure exit 0.
  • Local CI-equivalent: Linux (lidge-ts) 251/251 primary batches, 55637 pass / 1 base-identical fail / 167 skip, all gates exit 0. macOS (mac-studio) sharded batches all executed (only the base-identical update-stop-first :687, Desktop :249 and CLI boundary :287), control run stopped at the owner's request at 194/253 with no new failures (remaining control batches NOT RUN).
  • Closes test(server): agent-task-recovery Retry-After floor case flakes under load #7613. The 2400 ms Retry-After floor assertion is unchanged.

Static merge simulation on current dev af8cd35 (this Mac): dev=af8cd35a0665eace70abe9cc83f4263222bcdf79 base=47316142fe06171a94de02d63f1727dbed4c0382 pr_files= 2 overlap=[]; merge clean; install exit=0; gui install exit=0; tsc exit=0; tsc doctor exit=0; structure:check exit=0; privacy:scan exit=0; layout+ratchet guards exit=0 52 pass 0 fail ;

Merged by the triage manager as maintainer integration (MAINTAINERS.md, dev only) under the owner's static-judgment rule while GitHub Actions is degraded; dev push CI will be checked when Actions recovers.

@lidge-jun
lidge-jun merged commit c5ba1e1 into dev Oct 24, 2026
27 of 29 checks passed
@lidge-jun
lidge-jun deleted the lane/l2-test-flakes branch October 24, 2026 12:44
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 24, 2026 •

Copy link
Copy Markdown

Codex Review Summary

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

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-24T12:46:42.495145Z f749269 Draft marked ready
ℹ️ 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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore Maintenance, CI, tests, refactors, or build changes (not a user-facing bug or feature).

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant