fix(ci): merge rebased #643 head lineage#646
Merged
KooshaPari merged 2970 commits intocodex/pr611-base-fix-replayedfrom Feb 27, 2026
Merged
fix(ci): merge rebased #643 head lineage#646KooshaPari merged 2970 commits intocodex/pr611-base-fix-replayedfrom
KooshaPari merged 2970 commits intocodex/pr611-base-fix-replayedfrom
Conversation
Build Fixes: - Fix duplicate type definitions in kiro_websearch_handler.go (McpRequest, McpResponse, WebSearchResults) - Fix undefined authID and wsURL variables in codex_websockets_executor.go by naming parameters - Remove unused imports (crypto/sha256, encoding/hex) from codex_websockets_executor.go - Add missing syscall import to cmd/cliproxyctl/main.go for error handling - Remove incomplete showConfigPaths block from cmd/server/main.go (undefined functions) - Remove unused strings import from copilot/token_test.go Track 1.2 - ACP Adapter: - Implement ACP adapter to translate Claude/OpenAI protocol messages to ACP protocol - Add acp_request.go: Request translation and validation - Add acp_response.go: Response translation and formatting - Add acp_adapter.go: Main adapter logic with registry integration - Add unit tests in acp_adapter_registry_test.go Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
chore: merge ci-compile-fix-clean-single lane closeout and fixes
# Conflicts: # pkg/llmproxy/api/handlers/management/auth_files.go # pkg/llmproxy/api/handlers/management/management_extra_test.go
chore: merge full-sdk/test-cleanup stack into main
…type - Fixed missing closing brace in for loop - Changed return type from string to bool for proper if statement usage - Updated caller to use boolean check
- Fix unused sync/atomic import in kiro_websearch_handler.go - Fix handlers_metadata_test.go to use correct gin context key - Remove broken test files with undefined symbols Testing: Build PASS, Vet PASS, Tests PASS
This reverts commit 2464a28.
…2-20260223-034902' into cliproxy-clean-merge
…2-20260223-035004' into cliproxy-clean-merge
Align Codex SDK auth package types and sync required check names with current workflows. Co-authored-by: Codex <noreply@openai.com>
|
Warning You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again! |
KooshaPari
merged commit 5e2aceb
into
codex/pr611-base-fix-replayed
3 of 9 checks passed
|
Important Review skippedToo many files! This PR contains 297 files, which is 147 over the limit of 150. ⛔ Files ignored due to path filters (3)
📒 Files selected for processing (297)
You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
8 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Brings rebased lineage into #643 head so #643 can merge cleanly.\n\nCo-authored-by: Codex noreply@openai.com