Skip to content

fix(openai): inject WS Codex image bridge tool#2937

Merged
Wei-Shaw merged 1 commit into
Wei-Shaw:mainfrom
Kanshan03:fix/ws-codex-image-bridge
Jun 1, 2026
Merged

fix(openai): inject WS Codex image bridge tool#2937
Wei-Shaw merged 1 commit into
Wei-Shaw:mainfrom
Kanshan03:fix/ws-codex-image-bridge

Conversation

@Kanshan03
Copy link
Copy Markdown
Contributor

Summary

  • inject image_generation tool and bridge instructions for Codex CLI WS ingress requests when image generation is allowed
  • reuse image-generation permission result for WS image intent checks
  • add targeted ingress WS test for the Codex image bridge

Test

  • docker run --rm -v "$PWD/backend:/src:ro" -v sub2api-go-build-cache:/root/.cache/go-build -v sub2api-go-mod-cache:/go/pkg/mod -w /app -e GOPROXY=https://goproxy.cn,direct -e GOSUMDB=sum.golang.google.cn golang:1.26.3-alpine sh -c 'cp -a /src/. /app/ && go test -v -timeout=2m ./internal/service -run TestOpenAIGatewayService_ProxyResponsesWebSocketFromClient_InjectsCodexImageBridge -count=1'

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 1, 2026

All contributors have signed the CLA. ✅
Posted by the CLA Assistant Lite bot.

@Kanshan03 Kanshan03 force-pushed the fix/ws-codex-image-bridge branch from 5b49129 to 2a075a8 Compare June 1, 2026 03:25
@Kanshan03
Copy link
Copy Markdown
Contributor Author

I have read the CLA Document and I hereby sign the CLA

@Kanshan03
Copy link
Copy Markdown
Contributor Author

recheck

github-actions Bot added a commit that referenced this pull request Jun 1, 2026
@Kanshan03
Copy link
Copy Markdown
Contributor Author

recheck

@Wei-Shaw Wei-Shaw merged commit 3ba4a51 into Wei-Shaw:main Jun 1, 2026
7 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 1, 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.

2 participants