Skip to content

drive: cloud run 88aaba35 - #53

Merged
kjgbot merged 1 commit into
mainfrom
cloud/run-88aaba35
Aug 30, 2026
Merged

drive: cloud run 88aaba35#53
kjgbot merged 1 commit into
mainfrom
cloud/run-88aaba35

Conversation

@kjgbot

@kjgbot kjgbot commented Aug 30, 2026

Copy link
Copy Markdown
Contributor

Automated drive work from cloud run 88aaba35-06a2-40db-afc4-b55fb75e6e54.

The sandbox cannot open PRs (no remote, no GitHub token), so this was delivered
from a host that can. Verification and adversarial review ran in-run — see
ops/reviews/ in the diff. A human merges.

Work produced by cloud run 88aaba35-06a2-40db-afc4-b55fb75e6e54 in a workflow sandbox and delivered from
this host, because a sandbox has no remote and no GitHub token.

Verification and adversarial review ran in-run; see ops/reviews/ in the diff.
@coderabbitai

coderabbitai Bot commented Aug 30, 2026

Copy link
Copy Markdown

Review Change Stack

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Free

Run ID: a0ff8d96-3c80-41b8-bc99-535fde87537c

📥 Commits

Reviewing files that changed from the base of the PR and between 018c281 and e62ca87.

⛔ Files ignored due to path filters (1)
  • kernel/package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing (4)
  • ops/NEXT.md
  • sdk/src/index.ts
  • sdk/src/worker.ts
  • sdk/tests/live-kernel.test.ts

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

The SDK adds and exports AgentWorker. The worker attaches to the journal client, executes declared agent CLIs as subprocesses, captures output, and reports step results. A live-kernel test validates execution through the done state.

Changes

Agent worker

Layer / File(s) Summary
Worker contract and lifecycle
ops/NEXT.md, sdk/src/worker.ts, sdk/src/index.ts
The SDK defines AgentWorker and AgentWorkerOptions, exports them publicly, and implements attach and close lifecycle handling.
Dispatch and CLI execution
sdk/src/worker.ts
The worker handles agent dispatches, runs declared CLIs with instructions, captures process output, and reports success or worker errors.
Live-kernel worker validation
sdk/tests/live-kernel.test.ts
The test creates an executable CLI, attaches the worker with pins, starts an agent run, and polls until the step reaches done.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Merge Risk: ⚪ Minimal · up to e62ca

The PR adds an SDK worker and updates its export, tests, dependency metadata, and work-plan documentation; no actionable merge-blocking risk remains after normal checks and review.

Sequence Diagram(s)

sequenceDiagram
  participant JournalClient
  participant AgentWorker
  participant AgentCLI
  JournalClient->>AgentWorker: Dispatch agent step
  AgentWorker->>AgentCLI: Spawn CLI with instruction
  AgentCLI-->>AgentWorker: Return output and exit status
  AgentWorker->>JournalClient: Report stepComplete result
Loading

Poem

A rabbit starts the worker bright

It sends each step into the night
The CLI hops, then sends text back
The journal marks the finished track
Tests wait until the state is done


Note

🎁 Summarized by CodeRabbit Free

Your organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Pro by visiting https://app.coderabbit.ai/settings/billing.

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

@kjgbot
kjgbot merged commit 9681f11 into main Aug 30, 2026
1 check passed
kjgbot pushed a commit that referenced this pull request Aug 30, 2026
…cloud review-swarm

The worker landed in PR #53 (sdk/src/worker.ts, 91 lines). Three subsequent
PRs (#54, #55, #56) tried to rewrite it and were closed as duplicates. The
review-swarm has flagged every one of them.

Retarget the loop to build .github/workflows/review-swarm.yml — fires
workflows/review-swarm.yaml on PR open via agent-relay cloud run and posts
the transcripts back to the PR. Makes the swarm actually govern merges
instead of a laptop-side shell loop.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
kjgbot pushed a commit that referenced this pull request Aug 31, 2026
Gate 2's primitives all landed (event triggers PR #14, worker PR #53,
poller, spec fixture). What's missing is a continuous runner that
composes them. This sub-PR adds sdk/src/hn-monitor-runner.ts.

Part of a coordinated gate-2-GREEN push via agent-relay:
  A: SDK runner (this brief)
  B: end-to-end integration test
  C: CLI wrapper (`flows hn-monitor start`)
  D: gate-2 GREEN declaration in STATE.md + RFC

The gate2-lead agent retargets this file between sub-PRs.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
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