feat(inbox): file a draft once you reviewed the PR on GitHub yourself

[jtomaszewski]

The problem

You can review a PR the ordinary way — GitHub's own review button, cerber not
involved. That clears the review request, and nothing here hears about it.

The queue lists what you have not settled locally, so the draft cerber wrote
for that PR goes on sitting in the inbox as work still waiting on you, days
after the PR went back to its author. The daemon can't retire it either: it
only deletes rows that fall out of the awaiting search and hold no work
(isPureStub), and a finished draft is work. Nothing but a merge, a send, or a
click of yours ever moves it.

It is not a corner case. A dry run of the rule below over the author's own
queue found three of five open drafts stuck exactly this way.

The fix

The poll now asks the other question too. For an artifact GitHub no longer
requests you on, it reads the PR's reviews and — when one of yours stands
there — files the draft under settled, recording which review settled it.

Nothing is sent and nothing is deleted. The status goes ready → reviewed, the
new filed block keeps the review it acted on, and the draft is still yours to
open, re-review or send.

One guard keeps this from undoing a deliberate act. Runs now record who asked
for them (run.trigger): a draft you requested after that review — pasted the
URL in, pressed re-review — is a second opinion you went and got, so it is left
alone. Only a draft the poll wrote on its own is filed.

Details

• fetchOwnReview / latestOwnReview (src/core/gh.ts) — the read, and the
  pure part of it. A PENDING review is one nobody has seen (no submitted_at,
  and GitHub still asks you); a DISMISSED one is a review a maintainer struck
  off. Neither counts.
• filedByOwnReview (src/server/daemon.ts) — the whole rule, exported and
  unit-tested: a finished unsent draft, a review of yours, and no user-requested
  run since that review.
• Cost: it rides the existing state-check leash in syncQueue — at most 10
  artifacts per poll, once each per 30 minutes — so it is one extra gh api
  call on rows that qualify and none on the rest. Rows GitHub still requests you
  on never reach it.
• The cockpit says who did what. A filed row is tagged reviewed on GitHub,
  never reviewed — that would claim a click you never made. The strip names
  the date, and opening the review shows a banner linking your review on the PR.
• requestTag learned the same fact. When the author later re-requests you,
  a filed row reappears under open requests, where the conversation read would
  have tagged it "you haven't replied" at someone who demonstrably did — the
  same class of bug sentTag was written for, reachable again through this
  path.
• A re-review rebuilds the artifact with filed: null, so re-drafting a filed
  review un-files it with no extra machinery.

Verification

• pnpm typecheck && pnpm test && pnpm build — green; 379 tests, 14 new.
• New tests cover the filing itself (files when GitHub stops asking and holds
  your review; leaves it while the request stands; leaves it when you never
  reviewed; keeps a second opinion you asked for; never touches a sent review),
  the review parse (PENDING, DISMISSED, latest-of-yours), and the three cockpit
  readouts.
• Dry-run against live GitHub over a real ~/.cerber/reviews: three drafts
  matched with a review of the user's own on each, two rows GitHub still
  requests were left untouched.

Worth knowing

Not addressed here: the daemon can re-draft a PR minutes after you reviewed it
on GitHub, because gh search lags the API. Closing that would mean a reviews
read per awaiting PR on every poll — double the per-poll traffic to save a
few-minute window — so it is deliberately left out.

[Copilot]

Pull request overview

This PR teaches cerber to automatically file a local draft under “settled” when GitHub is no longer requesting your review and GitHub already shows a submitted review from you—covering the case where you reviewed directly on GitHub rather than via cerber’s Send path.

Changes:

• Add a filed metadata block to artifacts / inbox rows to record when/why a draft was filed (and which GitHub review triggered it).
• Extend the daemon poll to fetch your latest standing PR review from GitHub and file eligible drafts, guarded by a new run.trigger field to avoid filing drafts the user explicitly requested after that review.
• Update the web UI to surface “reviewed on GitHub” tags/banners, and add unit tests + documentation for the new behavior.

Reviewed changes

Copilot reviewed 20 out of 20 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
web/src/types.ts	Adds Filed and wires filed onto inbox rows and artifacts for UI consumption.
web/src/inbox.ts	Displays new “reviewed on GitHub” row/tag behavior and explanatory strip text when filed is present.
web/src/inbox.test.ts	Adds coverage for row tagging / strip messaging / open-request tag behavior for filed drafts.
web/src/Detail.tsx	Shows a banner on the detail view when a draft was filed due to a GitHub review.
src/server/index.ts	Ensures newly created/rerun artifacts record run.trigger: "user" and initializes filed: null; exposes filed in list items.
src/server/daemon.ts	Implements the “file if you reviewed on GitHub” rule and adds filedByOwnReview guard logic.
src/server/daemon.test.ts	Adds tests for filing behavior, trigger guard, and avoids GitHub calls when still requested.
src/server/chat.test.ts	Updates test fixtures for the new filed field.
src/runner/review.ts	Adds trigger option and persists it into artifact.run for daemon/user attribution.
src/runner/chat.test.ts	Updates test fixtures for the new filed field and run.trigger.
src/core/state.test.ts	Updates test fixtures for the new filed field and run.trigger.
src/core/send.test.ts	Updates test fixtures for the new filed field.
src/core/revise.test.ts	Updates test fixtures for the new filed field.
src/core/refresh.test.ts	Updates test fixtures for the new filed field.
src/core/gh.ts	Adds fetchOwnReview and latestOwnReview to detect your latest standing GitHub review.
src/core/gh.test.ts	Adds tests for latestOwnReview behavior (ignores PENDING/DISMISSED, picks latest standing review).
src/core/autosend.test.ts	Updates test fixtures for the new filed field.
src/core/artifact.ts	Extends the artifact schema with FiledInfo and run.trigger fields.
README.md	Documents the new “reviewed on GitHub → auto-filed” behavior.
CLAUDE.md	Updates architecture documentation to reflect the new daemon filing path and trigger guard.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[Copilot]

Pull request overview

Copilot reviewed 20 out of 20 changed files in this pull request and generated no new comments.

[github-actions]

🎉 This PR is included in version 0.21.0 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀