Skip to content

Align app-server v2 guidance with split protocol modules#150

Merged
andrei-hasna merged 6 commits into
mainfrom
openloops/open-codewith/57ae6577-44e4-4ac2-8638-5b1d48cbf83c-d34ac09e
Jul 21, 2026
Merged

Align app-server v2 guidance with split protocol modules#150
andrei-hasna merged 6 commits into
mainfrom
openloops/open-codewith/57ae6577-44e4-4ac2-8638-5b1d48cbf83c-d34ac09e

Conversation

@andrei-hasna

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

Copy link
Copy Markdown
Contributor

Summary:

  • Point app-server v2 guidance at the split app-server-protocol/src/protocol/v2/ module directory instead of the removed monolithic v2.rs file.
  • Name concrete module examples in agent guidance, MCP docs, and the app-server experimental-field checklist.
  • Refresh MODULE.bazel.lock after CI reported the Bazel module lock was out of date; the lock diff only adds generated facts for anyhow_1.0.103 and tar_0.4.45.
  • No app-server schema regeneration was required because protocol/schema shapes were not changed.

Validation:

  • git diff --check main..HEAD
  • branch diff secret-pattern scan before push: no matches
  • test ! -f codex-rs/app-server-protocol/src/protocol/v2.rs
  • find codex-rs/app-server-protocol/src/protocol/v2 -maxdepth 1 -type f | sort | sed -n '1,60p'
  • git grep -n -E "app-server-protocol/src/protocol/v2\\.rs|protocol/v2\\.rs" HEAD -- .codewith CODEWITH.md AGENTS.md docs codex-rs ':(exclude)codex-rs/target/**' ':(exclude)codex-rs/app-server-protocol/schema/**': no matches
  • just test-fast -p codex-app-server-protocol: 262 passed
  • bazel mod deps --lockfile_mode=error with redirected /tmp caches: passed

Notes:

  • In this OpenLoops sandbox, .codewith/ is bind-mounted read-only, so the local working-tree file cannot be rewritten. The committed tree was updated and validated directly with git show HEAD:.codewith/CODEWITH.md and git grep HEAD.
  • Local Bazel commands needed redirected caches because the sandbox cannot create the default Bazel cache under ~/.cache.

@andrei-hasna
andrei-hasna merged commit 667c752 into main Jul 21, 2026
25 of 27 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 21, 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