Skip to content
This repository was archived by the owner on Oct 9, 2026. It is now read-only.
This repository was archived by the owner on Oct 9, 2026. It is now read-only.

Validate Codex Desktop and app-server compatibility #34

Closed

Description

@shiny-code-bot

Objective

Validate Codex Desktop and app-server compatibility earlier in the MVP sequence so later overlays target the right session, thread, and API boundaries.

Finish Line

Codex Lab has a concrete app-server/Desktop compatibility gate before bridge, review, or automation overlays depend on it.

Current Status

State: Waiting on #42 before merge. PR #41 remains open and conflict-mergeable, but it is not green: codex-lab-app / Build macOS ARM64 Codex Lab app is failing on the macOS self-hosted runner.

What is complete:

  • PR Document app-server Desktop compatibility gate #41 documents the Desktop/app-server compatibility gate in codex-rs/app-server/README.md.
  • The branch is clean at 4dd9ed06358b9012513c65b362818b94b7f446e8; the net PR diff is README-only.
  • Review agents checked the gate; fixes were applied for schema command location, untracked schema files, protocol-test wording, TOC discoverability, initialization originator coverage, successful resume coverage, and copy-paste error propagation.
  • Local validation passed:
    • just write-app-server-schema plus tracked and untracked schema zero-diff checks.
    • just test -p codex-app-server-protocol (226 passed).
    • Targeted codex-app-server compatibility tests (11 passed, 816 skipped).
    • just test -p codex-cli strict_config_rejects_unknown_config_fields_for_app_server (1 passed, 269 skipped).
    • just fmt completed with the known existing Ruff/TOML exclude-newer = "7 days" parser warnings.
    • git diff --check passed.
  • GitHub CI passed on the latest SHA: Codex exec harness, spelling, build-test, and cargo-deny.

Blocker:

Cleanup and experiments already tried:

  • Used the PR watcher retry budget: 3 failed-check retries.
  • Quarantined only the runner's sqlx_macros release artifacts and reran.
  • Quarantined the self-hosted runner's whole codex-rs/target/release directory and reran.
  • Tried a short CARGO_TARGET_DIR workflow workaround; it did not fix the app build and was reverted from the PR.
  • Fresh local /tmp release builds reproduce the malformed proc-macro dylib.
  • Cargo profile override experiments for sqlx-macros/build override did not resolve the failure.
  • Local toolchain evidence points at macOS 27.0 / Xcode 27.0 / SDK 27.0 fresh release proc-macro output; an older cached loadable dylib was built with SDK 26.4.

Next action:

Scope

In:

  • App-server v2 compatibility checks.
  • Desktop startup/probe/version assumptions.
  • Thread lifecycle and resume behavior.
  • codex exec --json / resume event stream compatibility where Desktop or tooling depends on it.
  • Schema fixture drift checks.

Out:

Acceptance Criteria

  • just write-app-server-schema produces zero unexpected diff when schema is unchanged.
  • Relevant app-server protocol tests pass.
  • Desktop-facing startup/version/thread assumptions are documented with evidence.
  • Any compatibility gap becomes a focused implementation issue before bridge/browser/control work begins.

Relationships

Child of #28. Related to #4 for identity/channel naming and to #1 for overlay capability mapping.

Validation

Run app-server protocol tests and schema generation checks scoped to changed surfaces.

Decisions

App-server v2 is the active API surface for new work; compatibility should be validated before Code Bridge and automation overlays.

Open Questions

Which Desktop checks can be automated locally versus requiring a manual app-server/Desktop smoke test?

Activity

  1. added
    plan:donePlan completed or superseded
    and removed
    plan:activePlan is actionable now
    on Jun 12, 2026
  2. shiny-code-bot commented on Jun 12, 2026

    @shiny-code-bot
    CollaboratorAuthor

    Completed via PR #41: #41

    Evidence:

    • PR Document app-server Desktop compatibility gate #41 is merged at head 38c2be4adcc14730b448cc5ce760f9eb9b020f6f.
    • codex-rs/app-server/README.md now documents the Desktop/app-server compatibility gate and exact validation commands.
    • Local validation previously passed for schema zero-diff, app-server protocol tests, targeted app-server compatibility tests, CLI strict config test, formatting, and git diff --check.
    • Review agents were clean after the README gate fixes.
    • GitHub CI is green on the merged PR: cargo-deny, Codex exec harness, spelling, build-test, and Build macOS ARM64 Codex Lab app.
    • The temporary macOS app-build blocker was tracked in Fix macOS app-build sqlx_macros LINKEDIT failure #42 and is now closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    planDurable planning issueplan:donePlan completed or superseded

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions