Skip to content

fix(app-server): stabilize cross-platform CI regressions - #434

Merged
andrei-hasna merged 4 commits into
mainfrom
fix/app-server-cross-platform-ci
Jul 28, 2026
Merged

fix(app-server): stabilize cross-platform CI regressions#434
andrei-hasna merged 4 commits into
mainfrom
fix/app-server-cross-platform-ci

Conversation

@andrei-hasna

@andrei-hasna andrei-hasna commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Summary

  • make the schedule test harness preserve notifications and run reliably on Windows
  • canonicalize managed-worktree paths and rebind legacy workspace-write permissions on Windows
  • persist thread metadata before active-session heartbeats so mailbox ownership remains valid
  • add focused regressions for path aliases, permission rebinding, and immediate heartbeat visibility

Verification

  • just test-fast-target /tmp/codewith-app-server-cross-platform-target -p codex-app-server message_processor_schedule_tests (27 passed)
  • just test-fast-target /tmp/codewith-app-server-cross-platform-target -p codex-app-server --test background_agent (32 passed)
  • just test-fast-target /tmp/codewith-app-server-cross-platform-target -p codex-app-server --test all thread_mailbox_dispatcher_does_not_steal_live_target_from_dispatch_disabled_process (1 passed)
  • app-server Bazel argument-comment lint targets (7 compatible targets passed)
  • staged and origin/main..HEAD gitleaks scans (clean)
  • Blacksmith Testbox exact head 5c8ae0df6da8ad96090ca46fe547e574602f9c4f: https://github.com/hasna/codewith/actions/runs/30344383449 (27 + 32 + 1 tests passed)

Scope note

The workspace-wide argument-comment lint currently stops on 11 pre-existing required_consecutive_turns literals in codex-rs/state/src/runtime/goals.rs; this PR does not modify that out-of-scope file. The app-server-only Bazel lint target set is green.


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

@andrei-hasna
andrei-hasna force-pushed the fix/app-server-cross-platform-ci branch from 5c8ae0d to 275b16c Compare July 28, 2026 09:54
@andrei-hasna
andrei-hasna merged commit 95c318a into main Jul 28, 2026
45 of 59 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 28, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant