Skip to content

source: split CodexAppServer mapping helpers#57

Merged
gaelic-ghost merged 1 commit into
mainfrom
source/split-codex-app-server-mappings
May 3, 2026
Merged

source: split CodexAppServer mapping helpers#57
gaelic-ghost merged 1 commit into
mainfrom
source/split-codex-app-server-mappings

Conversation

@gaelic-ghost

Copy link
Copy Markdown
Owner

Summary

  • keep CodexAppServer.swift focused on the actor/runtime implementation
  • move approval and elicitation protocol response payload helpers into CodexAppServer+ProtocolPayloads.swift
  • move generated-wire to public model mapping extensions into CodexAppServer+WireMapping.swift

Notes

  • The moved helpers are internal instead of same-file private so CodexAppServer can keep using them across files; no public API is added.

Verification

  • swift test
  • swift build
  • git diff --check
  • bash scripts/repo-maintenance/validate-all.sh

@gaelic-ghost gaelic-ghost added the enhancement New feature or request label May 3, 2026
@gaelic-ghost gaelic-ghost merged commit 6041a62 into main May 3, 2026
1 check passed
@gaelic-ghost gaelic-ghost deleted the source/split-codex-app-server-mappings branch May 3, 2026 22:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant