wmc/CodexModelSwitcher
Watch
1
0
Fork
You've already forked CodexModelSwitcher
 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

refactor: split proxy and menu views #38

Merged
wmc merged 1 commit from codex/split-proxy-ui-files into main 2026-06-23 09:44:03 +00:00
Conversation 0 Commits 1 Files changed 10 +1969 -1956
wmc commented 2026-06-23 09:43:58 +00:00
Owner
Copy link

Summary

  • Split SwiftUI menu panel code so ContentView keeps orchestration while ServiceSectionView and ServiceEditorView live in focused source files.
  • Split compatibility proxy support into HTTP, stream, request-conversion, and response-conversion files while preserving existing test-facing APIs.
  • Updated README development notes and TODO.md; removed transient plan docs to keep root docs as the source of truth.

Verification

  • DEVELOPER_DIR=/Applications/Xcode-beta.app/Contents/Developer xcodebuild test -project CodexModelSwitcher.xcodeproj -scheme CodexModelSwitcher -configuration Debug -destination platform=macOS -derivedDataPath /tmp/CodexModelSwitcher-DerivedData-split-proxy-first CODE_SIGNING_ALLOWED=NO -only-testing:CodexModelSwitcherTests/CompatibilityProxyServerTests
  • DEVELOPER_DIR=/Applications/Xcode-beta.app/Contents/Developer xcodebuild test -project CodexModelSwitcher.xcodeproj -scheme CodexModelSwitcher -configuration Debug -destination platform=macOS -derivedDataPath /tmp/CodexModelSwitcher-DerivedData-split-appstore-first CODE_SIGNING_ALLOWED=NO -only-testing:CodexModelSwitcherTests/AppStoreTests
  • swiftc -parse CodexModelSwitcher/*.swift
  • git diff --check
  • DEVELOPER_DIR=/Applications/Xcode-beta.app/Contents/Developer scripts/verify.sh -> ==> verification complete

Review

  • Subagent review found no code blockers; documentation-policy feedback was addressed by removing transient docs/superpowers files.
## Summary - Split SwiftUI menu panel code so ContentView keeps orchestration while ServiceSectionView and ServiceEditorView live in focused source files. - Split compatibility proxy support into HTTP, stream, request-conversion, and response-conversion files while preserving existing test-facing APIs. - Updated README development notes and TODO.md; removed transient plan docs to keep root docs as the source of truth. ## Verification - `DEVELOPER_DIR=/Applications/Xcode-beta.app/Contents/Developer xcodebuild test -project CodexModelSwitcher.xcodeproj -scheme CodexModelSwitcher -configuration Debug -destination platform=macOS -derivedDataPath /tmp/CodexModelSwitcher-DerivedData-split-proxy-first CODE_SIGNING_ALLOWED=NO -only-testing:CodexModelSwitcherTests/CompatibilityProxyServerTests` - `DEVELOPER_DIR=/Applications/Xcode-beta.app/Contents/Developer xcodebuild test -project CodexModelSwitcher.xcodeproj -scheme CodexModelSwitcher -configuration Debug -destination platform=macOS -derivedDataPath /tmp/CodexModelSwitcher-DerivedData-split-appstore-first CODE_SIGNING_ALLOWED=NO -only-testing:CodexModelSwitcherTests/AppStoreTests` - `swiftc -parse CodexModelSwitcher/*.swift` - `git diff --check` - `DEVELOPER_DIR=/Applications/Xcode-beta.app/Contents/Developer scripts/verify.sh` -> `==> verification complete` ## Review - Subagent review found no code blockers; documentation-policy feedback was addressed by removing transient docs/superpowers files.
wmc merged commit 1c1be0efdd into main 2026-06-23 09:44:03 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
wmc/CodexModelSwitcher!38
No description provided.