Skip to content

fix(codex): read text-block custom tool output - #67

Open
lixenstrand wants to merge 1 commit into
nizos:mainfrom
lixenstrand:feature/nor-5787-codex-output-blocks
Open

fix(codex): read text-block custom tool output#67
lixenstrand wants to merge 1 commit into
nizos:mainfrom
lixenstrand:feature/nor-5787-codex-output-blocks

Conversation

@lixenstrand

Copy link
Copy Markdown

Summary

  • accept both legacy string output and current Codex text-block arrays for custom tool calls
  • join text blocks into the raw action output without changing TDD policy
  • cover both transcript formats and a Vitest failure used as Red evidence

Tests

  • npx vitest run src/vendors/codex/transcript.test.ts src/cli.test.ts -t 'reads custom tool output|passes Codex Vitest text-block output' — 2 passed, 20 skipped
  • npm run checks — 58 test files passed; 539 tests passed, 26 skipped

@lixenstrand
lixenstrand marked this pull request as ready for review August 21, 2026 17:58
@nizos

nizos commented Aug 27, 2026

Copy link
Copy Markdown
Owner

Hi @lixenstrand,

Thanks for the PR. I will try to look at it as soon as I can. Might take a few days to a week. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants