molecule-ai/molecule-ai-workspace-template-codex
51
0
Fork 0
Code Issues 12 Pull Requests 11 Actions Packages Projects Releases Wiki Activity

fix(codex): forward image attachments to app-server #41

Merged
hongming merged 2 commits from fix/l4-vision-attachments into main 2026-05-23 23:39:05 +00:00
Conversation 2 Commits 2 Files Changed 3
+54 -4
hongming commented 2026-05-23 23:19:04 +00:00
Owner
Copy Link
Copy Source

Live tenant image-upload smoke showed attachment metadata reaching runtimes but image bytes not being delivered to the model path.\n\nThis PR adds targeted L4 forwarding and regression tests.\n\nValidation:\n- runtime: PYTHONPATH=. pytest -q tests/test_a2a_nonblocking_inbox.py tests/test_executor_helpers.py::test_build_user_content_with_files_image_is_multimodal tests/test_executor_helpers.py::test_extract_attached_files_fetches_platform_pending_attachment\n- codex: PYTHONPATH=. pytest -q tests/test_executor.py

Live tenant image-upload smoke showed attachment metadata reaching runtimes but image bytes not being delivered to the model path.\n\nThis PR adds targeted L4 forwarding and regression tests.\n\nValidation:\n- runtime: PYTHONPATH=. pytest -q tests/test_a2a_nonblocking_inbox.py tests/test_executor_helpers.py::test_build_user_content_with_files_image_is_multimodal tests/test_executor_helpers.py::test_extract_attached_files_fetches_platform_pending_attachment\n- codex: PYTHONPATH=. pytest -q tests/test_executor.py
hongming added 1 commit 2026-05-23 23:19:05 +00:00
fix(codex): forward image attachments to app-server
CI / validate (push) Blocked by required conditions
Details
CI / Adapter unit tests (push) Successful in 8s
Details
CI / Template validation (static) (push) Successful in 32s
Details
CI / Adapter unit tests (pull_request) Successful in 10s
Details
CI / Template validation (static) (pull_request) Successful in 42s
Details
CI / Template validation (runtime) (pull_request) Successful in 1m42s
Details
CI / Template validation (runtime) (push) Successful in 2m7s
Details
CI / T4 tier-4 conformance (live) (push) Successful in 1m56s
Details
CI / T4 tier-4 conformance (live) (pull_request) Successful in 1m26s
Details
CI / validate (pull_request) Successful in 4s
Details
4a5469f63f
app-fe added 1 commit 2026-05-23 23:22:36 +00:00
chore(codex): pin runtime 0.2.2
CI / Adapter unit tests (push) Successful in 14s
Details
CI / Template validation (static) (push) Successful in 41s
Details
CI / Template validation (static) (pull_request) Successful in 34s
Details
CI / Adapter unit tests (pull_request) Successful in 16s
Details
CI / Template validation (runtime) (push) Successful in 2m53s
Details
CI / T4 tier-4 conformance (live) (push) Successful in 1m32s
Details
CI / Template validation (runtime) (pull_request) Successful in 2m32s
Details
CI / T4 tier-4 conformance (live) (pull_request) Successful in 2m41s
Details
CI / validate (push) Successful in 1s
Details
CI / validate (pull_request) Successful in 1s
Details
97188be667
infra-runtime-be approved these changes 2026-05-23 23:26:20 +00:00
infra-runtime-be left a comment
Member
Copy Link
Copy Source

Approved after static review and local tests. Codex app-server protocol supports localImage and this patch forwards image parts into turn/start and turn/steer.

Approved after static review and local tests. Codex app-server protocol supports localImage and this patch forwards image parts into turn/start and turn/steer.
dev-lead approved these changes 2026-05-23 23:26:20 +00:00
dev-lead left a comment
Member
Copy Link
Copy Source

Approved. Scoped template adapter fix with regression tests and runtime 0.2.2 pin.

Approved. Scoped template adapter fix with regression tests and runtime 0.2.2 pin.
hongming merged commit 1c798d8f25 into main 2026-05-23 23:39:05 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
infra-runtime-be
dev-lead
Labels
Clear labels
tier:low
per dev-SOP tier system
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
agent-dev-a agent-dev-b agent-pm agent-researcher agent-reviewer agent-reviewer-1 agent-reviewer-cr2 app-fe (Molecule AI · app-fe) app-lead (Molecule AI · app-lead) app-qa (Molecule AI · app-qa) claude-ceo-assistant claude-ci-reader core-be (Molecule AI · core-be) core-devops (Molecule AI · core-devops) core-fe (Molecule AI · core-fe) core-lead (Molecule AI · core-lead) core-offsec (Molecule AI · core-offsec) core-qa (Molecule AI · core-qa) core-security (Molecule AI · core-security) core-uiux (Molecule AI · core-uiux) cp-be (Molecule AI · cp-be) cp-lead (Molecule AI · cp-lead) cp-qa (Molecule AI · cp-qa) cp-security (Molecule AI · cp-security) cui (Zhanlin Cui) dev-lead (Molecule AI · dev-lead) devops-engineer documentation-specialist (Molecule AI · documentation-specialist) fullstack-engineer (Molecule AI · fullstack-engineer) godwin hongming hongming-ceo-delegated hongming-codex-laptop hongming-kimi-laptop hongming-pc2 hongming-personal infra-lead (Molecule AI · infra-lead) infra-runtime-be (Molecule AI · infra-runtime-be) infra-sre (Molecule AI · infra-sre) integration-tester (Molecule AI · integration-tester) molecule-code-reviewer molecule-runtime-release-bot (Molecule Runtime Release Bot) plugin-dev (Molecule AI · plugin-dev) pm publish-runtime-bot pypi-publisher (Molecule AI PyPI Publisher (RFC#596)) release-manager (Molecule AI · release-manager) sdk-dev (Molecule AI · sdk-dev) sdk-lead (Molecule AI · sdk-lead) sop-tier-bot (SOP Tier-Check Bot) technical-writer (Molecule AI · technical-writer) triage-operator (Molecule AI · triage-operator)
No Assignees
4 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: molecule-ai/molecule-ai-workspace-template-codex#41
Delete Branch "fix/l4-vision-attachments"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?