[TheGreatAxios]

Closes CL-11935 (Slice 5 of 5 for CL-11929; stacked on cl-11934-share-a-package-and-record-reviewers-decisions).

Summary

• Extract structured triage bullet items (summary, targetKind, slideNumber, status: "pending" | "incorporated" | "dismissed" | "replied") when reviewer feedback is recorded (apps/hub/src/artifacts/approval-rounds.ts), without automatically messaging the Coordinator.
• Add person-only POST /:projectId/approval-rounds/:roundId/triage/:itemId endpoint (Incorporate, Dismiss, Reply) so the project owner reads and gates all reviewer feedback first.
• When the project owner chooses Incorporate, forward the feedback item to the active Coordinator as the owner in a clearly delimited external block (--- reviewer feedback (external, <name>, <persona>, package v<N>) ---) naming the target document kind (apps/hub/src/config/approval-rounds.ts).
• Render the owner feedback triage list in apps/web/src/pages/workspace/approval-rounds.tsx with Incorporate, Dismiss, and Reply actions.

[linear-code]

CL-11935

[corbits-triage]

Triage requests changes:

• CI green: required checks are failing

Nothing triage asked to change still fails, so this request no longer stands.

[TheGreatAxios]

The gate holds. Only an owner Incorporate mails the coordinator, and it mails that one stored bullet. Recording a decision, dismiss, and reply do not.

Two gaps:

1. The mail does not name the package version or the reviewer's decision. incorporatedFeedbackMessage in apps/hub/src/config/approval-rounds.ts sends persona, reviewer, and slide. The coordinator prompt expects package version, audience, reviewer, decision, and note. A note on an older package can be applied to the wrong generation.

2. Incorporate is not once-only. triageItem checks pending, mails, then writes incorporated as separate steps. A double-click, or a mail that succeeds before the status write, sends the same bullet twice. The 409 only applies after a successful write.

This pull request is no longer ready to approve, so this approval no longer stands.

[corbits-triage]

Triage requests changes:

• Errors kept: added code matches a code pattern
  • apps/hub/src/artifacts/approval-rounds.ts:1127-1129 bare-catch: A catch without a binding drops the error; check for the expected failure, or catch it and rethrow the rest.

This pull request is no longer ready to approve, so this approval no longer stands.

[corbits-triage]

Triage requests changes:

• Conventions: added code matches a code pattern
  • apps/web/src/pages/workspace/approval-rounds.tsx:2042 empty-list-fallback: Invented fallback `?? []`; handle the missing value instead.
  • apps/web/src/pages/workspace/approval-rounds.tsx:2050-2062 promise-chain: Promise chain; use async/await with try/catch.
  • apps/web/src/pages/workspace/approval-rounds.tsx:2050-2061 promise-chain: Promise chain; use async/await with try/catch.

[corbits-triage]

Triage requests changes:

• Conventions: added code matches a code pattern
  • apps/web/src/pages/workspace/approval-rounds.tsx:2061 empty-list-fallback: Invented fallback `?? []`; handle the missing value instead.
  • apps/web/src/pages/workspace/approval-rounds.tsx:2069-2081 promise-chain: Promise chain; use async/await with try/catch.
  • apps/web/src/pages/workspace/approval-rounds.tsx:2069-2080 promise-chain: Promise chain; use async/await with try/catch.
  • apps/web/src/pages/workspace/approval-rounds.tsx:2175-2177 promise-chain: Promise chain; use async/await with try/catch.
  • apps/web/src/pages/workspace/approval-rounds.tsx:2178-2183 promise-chain: Promise chain; use async/await with try/catch.
  • apps/web/src/pages/workspace/approval-rounds.tsx:2184-2194 promise-chain: Promise chain; use async/await with try/catch.
  • apps/web/src/pages/workspace/approval-rounds.tsx:2184-2191 promise-chain: Promise chain; use async/await with try/catch.

Nothing triage asked to change still fails, so this request no longer stands.

This pull request is no longer ready to approve, so this approval no longer stands.

[corbits-triage]

Triage requests changes:

• Merge conflicts: merge conflicts with base

[corbits-triage]

Triage requests changes:

• CI green: required checks are failing
  • typecheck

[corbits-triage]

Triage: Pass

No changes needed from you. A maintainer will review this pull request.

Stacked on #60: only this pull request's own changes were checked.

Notes:

• mixes unrelated changes: apps/web/src/workspace-approvals-setting.ts

Triage will approve, request review.