Skip to content

fix(host): send an inbox answer's files to the service - #3475

Merged
luke-agent[bot] merged 1 commit into
mainfrom
luke/fix-inbox-answer-attachments
Oct 17, 2026
Merged

luke-agent[bot] merged 1 commit into
mainfrom
luke/fix-inbox-answer-attachments

Conversation

@dastratakos

@dastratakos dastratakos commented Oct 17, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Files attached to an Inbox answer never reached the service. codingAgentInboxAnswer in packages/host/src/host-operator.ts sent the answer's reply, choices, values, note, and key to the gateway, but left out its attachments, so the composer dropped them before they reached HostedCodingAgentClient.answerInbox. It now sends them whenever the answer has any.
  • New test in host-operator.test.ts: an answer with a file reaches the gateway call with the file included. It fails without the fix.

Found while building the desktop half of Inbox thread messages (stacked on #3469). Split out off main so it lands right away.

Evidence

  • Platform-independent checks: pnpm check:affected passes except src/main/app-menu.test.ts, which fails only because the sandbox has no Electron binary.

Invariants

  • ./scripts/check.sh green. (CI)
  • JSON Schema goldens unchanged (LUKE_UPDATE_FIXTURES not run, or diff explained line by line). (CI)
  • Every behavior change has, in this PR, the test that fails without it; no change-detector test added. (manual; root AGENTS.md "Testing")
  • Gateway envelope goldens unchanged. (CI)
  • No new as type assertion outside the allowlisted files. (CI via anti-slop)
  • No Effect.runPromise/runSync/runFork outside the runtime edges and the shims tools/anti-slop/effect-edges.json records. (CI via anti-slop/no-run-promise-outside-edges)
  • No new setTimeout/setInterval/new Promise/AbortController/fs.watch outside the files tools/anti-slop/effect-edges.json records. (CI via anti-slop/no-raw-async-primitives)
  • No Effect, Stream, or Layer written as a statement and dropped; a description a caller walks past runs nothing. (CI via pnpm discarded-effect)
  • Test count equals the pre-PR count or the delta is listed. (+1 in host-operator.test.ts)
  • Each hand-rolled file the PR replaces is deleted, or the part that remains is a permanent adaptor named in root AGENTS.md's "The permanent adaptors" with the reason it stays.
  • Every CLAUDE.md/AGENTS.md sentence naming a changed part is edited; root pair stays byte-identical. (CI for pair existence; manual for wording)
  • PRIVACY.md unchanged, or the change is a product decision called out in the PR body.
  • Package package.json deps match what the sources import by bare specifier; effect via catalog:. (CI)
  • Barrel/door rule: no Node-reaching Effect module (@effect/platform-node, effect/sql) behind a barrel the renderer or a web function opens. (CI: existing renderer node: grep plus a new grep for those specifiers in renderer bundles)

Notes

  • Blockers or follow-up verification: None

Made with Luke


Devin Review


Summary by cubic

Fixes inbox answer attachments never reaching the service. The host operator now forwards an answer's attachments to the gateway, so files attached to an answer arrive with its reply, note, and choices.

  • Adds a test verifying an answer with a file includes the file in the gateway call.

Written for commit 18084f1. Summary will update on new commits.

View guided diff Turn on auto-fix

The host operator forwarded an answer's reply, choices, values, note, and
key to the gateway but left its attachments out, so files attached to an
answer never reached the service.

Co-authored-by: luke-agent[bot] <339870089+luke-agent[bot]@users.noreply.github.com>
@vercel

vercel Bot commented Oct 17, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
luke-web Ready Ready Preview Oct 17, 2026 1:26pm UTC

Request Review

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Devin Review

@luke-agent
luke-agent Bot added this pull request to the merge queue Oct 17, 2026
Merged via the queue into main with commit e13a3f7 Oct 17, 2026
11 checks passed
@luke-agent
luke-agent Bot deleted the luke/fix-inbox-answer-attachments branch October 17, 2026 13:34

This branch was successfully deployed

1 active deployment
Preview — 18084f11 Deployed Oct 17, 2026 by vercel[bot]
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.

1 participant