Skip to content

docs(harness): document Compose completion watches - #1064

Draft
mfpiccolo wants to merge 1 commit into
mainfrom
fix/compose-operation-trigger-compat
Draft

docs(harness): document Compose completion watches#1064
mfpiccolo wants to merge 1 commit into
mainfrom
fix/compose-operation-trigger-compat

Conversation

@mfpiccolo

Copy link
Copy Markdown

Summary

Document the correct Harness workflow for observing one compose::add operation.

Guidance

  • Generate an operation ID before starting the operation.
  • Register compose-operation before invoking compose::add.
  • Use the exact same operation_id in the trigger and Compose request.
  • Set terminal_only: true so progress events do not consume a one-shot completion watch.
  • Do not use operation_id: null for per-operation correlation; it is a wildcard subscription.
  • Read compose::operation as a race-recovery check.
  • Treat terminal: true as the completion marker for both success and failure.

Scope

No active Harness worker code required modification: Harness already consumes __binding correctly. This documentation accompanies the iii change, which fixes Compose-side metadata forwarding.

@vercel

vercel Bot commented Sep 2, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated
workers Ready Ready Preview Sep 2, 2026 9:24pm UTC
workers-tech-spec Ready Ready Preview Sep 2, 2026 9:24pm UTC

Request Review

@coderabbitai

coderabbitai Bot commented Sep 2, 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

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

Comment @coderabbitai help to get the list of available commands.

@github-actions

github-actions Bot commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

skill-check — worker

0 verified, 70 skipped (no docs/).

Layer Result
structure
vale
ai
render

Four for four. Nicely done.

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