Skip to content

Support bundled Computer Use and richer MCP event mapping#62

Merged
gabrielMalonso merged 2 commits into
mainfrom
codex/investigar-computer-use-no-t3
May 23, 2026
Merged

Support bundled Computer Use and richer MCP event mapping#62
gabrielMalonso merged 2 commits into
mainfrom
codex/investigar-computer-use-no-t3

Conversation

@gabrielMalonso
Copy link
Copy Markdown
Owner

@gabrielMalonso gabrielMalonso commented May 23, 2026

Summary

  • Centralized Codex process environment building so CODEX_HOME is normalized consistently across provider startup, session runtime, and text generation.
  • Added discovery for bundled Computer Use app paths, preserved valid inherited SKY_CUA_SERVICE_PATH, and aligned default discovery with an inherited HOME when CODEX_HOME is unset.
  • Improved Codex adapter event mapping for MCP tool calls and startup status notifications, including stable data payloads for tool metadata.
  • Projected MCP startup status updates into orchestration activities so the web UI can receive the new status events.
  • Added coverage for environment resolution, MCP lifecycle mapping, and MCP startup status projection behavior.

Testing

  • bun run test src/provider/CodexEnvironment.test.ts src/orchestration/Layers/ProviderRuntimeIngestion.test.ts src/provider/Layers/CodexAdapter.test.ts from apps/server
  • bun fmt
  • bun lint (passes with existing warnings outside this PR's changed files)
  • bun typecheck

@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:L 100-499 changed lines (additions + deletions). labels May 23, 2026
@gabrielMalonso
Copy link
Copy Markdown
Owner Author

@codex review for security regressions, missing tests, risky behavior changes, logic bugs, i18n regressions, and violations of applicable AGENTS.md review guidelines.

@chatgpt-codex-connector
Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@gabrielMalonso
Copy link
Copy Markdown
Owner Author

Nao faz sentido: o comentario do Codex foi apenas um aviso operacional de limite de uso, sem achado de review testavel no diff.

- project `mcp.status.updated` events into thread activities
- resolve bundled Computer Use from inherited `HOME` when `CODEX_HOME` is unset
@gabrielMalonso
Copy link
Copy Markdown
Owner Author

@codex review for security regressions, missing tests, risky behavior changes, logic bugs, i18n regressions, and violations of applicable AGENTS.md review guidelines.

@chatgpt-codex-connector
Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@gabrielMalonso
Copy link
Copy Markdown
Owner Author

Nao faz sentido: o novo retorno do Codex tambem foi apenas aviso operacional de limite de uso, sem achado de review testavel no diff do head 9befe3f.

@gabrielMalonso gabrielMalonso merged commit 9f41d97 into main May 23, 2026
5 of 7 checks passed
@gabrielMalonso gabrielMalonso deleted the codex/investigar-computer-use-no-t3 branch May 23, 2026 15:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L 100-499 changed lines (additions + deletions). vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant