Skip to content

feat: expose final task files to external MCP clients - #792

Merged
BytePioneer-AI merged 1 commit into
BytePioneer-AI:mainfrom
ump45nose:codex/upstream-task-result-files
Nov 12, 2026
Merged

BytePioneer-AI merged 1 commit into
BytePioneer-AI:mainfrom
ump45nose:codex/upstream-task-result-files

Conversation

@ump45nose

@ump45nose ump45nose commented Nov 10, 2026 •

Copy link
Copy Markdown
Contributor

External MCP clients can read a task's text but cannot retrieve the files linked in its final report. Add thread_file_read so clients can download those files through the authenticated connection and deliver them with their own attachment tools.

Files must be explicitly linked in the latest final report and resolve to regular files inside the Thread's execution workspace. The snapshot exposes cwd when known. Reads are bounded to 1 MiB per chunk (256 KiB by default), and continuation checks a file revision to avoid combining different versions. No public download URL or directory browsing is added.

Validation:

  • Host Runtime and its local dependencies typechecked; 18 focused result-file, official MCP SDK, and Thread snapshot tests passed; affected-file ESLint and formatting passed.
  • CI passed Linux x64, Linux ARM64, and Windows. The macOS lane failed waiting for relay capacity/reconnection in packages/web-server/test/relay-client.test.ts:96 (65-second timeout); this PR does not modify that code: https://github.com/BytePioneer-AI/codex-host/actions/runs/42812630647/job/129817638097.

Linux headless external-client availability is tracked separately in #788.

@coderabbitai

coderabbitai Bot commented Nov 10, 2026

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

Share

@BytePioneer-AI
BytePioneer-AI merged commit 76c1587 into BytePioneer-AI:main Nov 12, 2026
4 of 5 checks passed
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