Files
PaperClipAI/server/src/__tests__/issue-thread-interaction-routes.test.ts
T
scotttongandPaperclip 71f0c0c99e feat: add opt-in Coordinator workspace for chat, tasks, and decisions (#17983)
<!-- entire-trail-link-start -->
https://entire.com/gh/paperclipai/-/trails/147/changes/paperclip/147
<!-- entire-trail-link-end -->

## Thinking Path

> - Paperclip is the open source app people use to manage AI agents for
work.
> - People coordinate work through conversations, tasks, and requests
for input.
> - These surfaces must share task state and preserve the context of
each decision.
> - The existing workspace makes people switch surfaces to follow that
context.
> - This pull request adds an optional Coordinator workspace with linked
conversations, tasks, documents, and decisions.
> - Each person can enable it in Settings > Experimental and return to
the classic workspace.
> - The benefit is a coherent path from a request to an assigned task
and reviewed work.

## Linked Issues or Issue Description

**Subsystem affected**
UI, server, shared contracts, and database.

**Problem or motivation**
People need a consistent way to start tasks from agent conversations,
answer questions, review work, and return to the source context.
Decisions and task state must agree across these surfaces.

**Proposed solution**
Add a personal, default-off workspace experiment. Keep one conversation
per agent. Add task proposals, compact task and decision cards,
contextual document panels, and a Needs You queue with search, filters,
sorting, grouping, and history.

**Alternatives considered**
Replacing the classic workspace for everyone would remove the opt-out
path. Keeping separate decision state in each surface would create
conflicting answers. This change uses the existing canonical task and
interaction records instead.

**Roadmap alignment**
This supports the existing agent chat, task management, and human review
capabilities. It is an experimental presentation choice. The user
requested this isolated implementation after design review. No Discord
discussion is claimed.

## What Changed

- Add per-person workspace preferences, a default-off setting, and an
additive migration.
- Add Coordinator navigation, pinned destinations, task lists, agent
switching, and contextual panels.
- Link task suggestions and creation to the source conversation. Default
new task ownership to the current agent unless the user specifies
another owner.
- Share pending questions, decision receipts, exact review targets, and
history across tasks, chat, and Needs You.
- Preserve task access, assignment authorization, revision binding, and
required output review gates.
- Add compact document links, relative timestamps, composer controls,
and reduced-motion support for task and decision transitions.
- Add component stories, regression coverage, and capability-map
documentation.

## Verification

- Final head: `4b839aae1e314bd7e8635a766f1d205ff1c03ba6`, based on
master `8bc051245f77a47bee0f8ba45896b8bfa5922e0a`. Verified October 29,
2026 at 00:10 UTC: **MERGEABLE, 56 checks passed, four conditional
skips, no pending or failed checks**. [Current CI
run](https://github.com/paperclipai/paperclip/actions/runs/40790639688)
passed without a rerun on this head.
- Fresh Greptile **5/5**, zero new comments and zero open review
threads; Superagent passed with no suspicious changes.
[Checks](https://github.com/paperclipai/paperclip/pull/17983/checks).
- Local workspace typecheck/build passed after the latest rebase. The
final account-read patch passed UI typecheck/build and all 12 related
preference/settings tests, including a delayed real-client request
across account changes. Earlier scoped checks cover review
deduplication/replay, decision deep links, cross-surface state, access
boundaries, and migration replay.
- Local database/browser rechecks after the latest rebase were limited
by exhausted macOS shared-memory IDs: `initdb` failed before database
creation. The local run passed 31 UI/snapshot tests and skipped 24
database tests; those skips and the blocked local browser start are not
claimed as passes. The independent Linux CI database and browser checks
passed. No running dev database or host setting was changed.
- Upstream migration 0403 is preserved. Preferences and durable review
retry aliases are generated as replay-safe migrations
`0404_tense_silver_sable.sql` and `0405_harsh_avengers.sql`; five newest
snapshots are retained. Every detected conflict was rebased and pushed
before waiting for CI/review, as requested.
- The PR is unmerged. The active master ruleset still requires
code-owner approval for the changed Paperclip skill; no approval or rule
change was performed.

<details>
<summary>Earlier verification failures, integration history, and
evidence</summary>

These entries preserve prior checkpoints and failures; the current
acceptance result is above.

- Focused UI and server checks from the isolated build are recorded in
the implementation plan.
- Browser checks covered the opt-in shell, navigation, task/document
links, and decision card states. No new live provider trial is claimed
for this handoff.
- Current acceptance head: `4b839aae1e314bd7e8635a766f1d205ff1c03ba6`,
rebased onto master `8bc051245` immediately after its GitHub
review-repair migration took 0403. Preferences are generated as
idempotent `0404_tense_silver_sable.sql`, preserving upstream
SQL/journal/snapshot history and five newest snapshots. GitHub reports
MERGEABLE after the immediate lease-protected push. Scott's updated
instruction is to rebase and push every conflict resolution first,
without waiting for CI or Greptile; verification follows the push.
- The final reviewed feature tree was consolidated for repeatable
single-commit rebases. The complete previous commit history remains
reachable through local recovery ref
`codex/coordinator-recovery-before-rebase-20261028` (resolved snapshot
`0e1bbdf4e`). The actual rebase completed, and its tree hash exactly
matches that resolved snapshot. The force-with-lease required the known
remote head `0e6ef7c44`, protecting against concurrent branch changes.
No PR merge or permission/runner setting change occurred. Fresh
CI/review are in progress for the current head.
- `2001132fa` received Greptile 5/5 with one nonblocking account-switch
read finding. Current head detaches path-coalesced preference GETs and
disables HTTP caching, retaining React Query's account-scoped sharing. A
delayed real-client GET regression proves account B can load its saved
choice before account A responds, and A's late response only updates A's
cache. Preference/settings tests 12/12 and UI typecheck passed; its
review thread is resolved. Fresh current-head CI/reviews are pending.
The prior CI chat shard 3 ended with “The operation was canceled” and no
assertion failure ([job
123171595631](https://github.com/paperclipai/paperclip/actions/runs/40789688333/job/123171595631),
`/tmp/pr17983-2001132-chat3.log`); no code/timeout change is attributed
to that cancellation.
- `843e823b4` reached fresh Greptile 5/5 with zero comments/open threads
and Superagent pass. All browser tests including the repaired YAML shard
passed, and 54 checks passed. General server shard 9 ended with “The
operation was canceled” and no failed assertion; its test step was
marked cancelled (job
[123164350893](https://github.com/paperclipai/paperclip/actions/runs/40787501761/job/123164350893),
log `/tmp/pr17983-843e-server9.log`). A single unchanged-head rerun was
started after the workflow completed, but master moved while it ran. A
local nested-clone check timed out in its 30-second database-startup
hook and then failed cleanup before the 20 database cases ran; this is
retained as a local verification limit
(`/tmp/pr17983-843e-nested-clone-check.log`), not claimed as a pass or
proof of the CI cancellation cause.
- Following Scott's instruction, the new conflict was rebased and pushed
immediately without waiting for that rerun. Upstream's
`0403_natural_cerise.sql` is preserved; regenerated preferences and
retry-alias migrations are `0404_tense_silver_sable.sql` and
`0405_harsh_avengers.sql`, replay-safe with five newest snapshots
retained. The YAML helper conflict combines upstream's click and header
behavior with the editor-selection guard and injected regression.
Recovery ref `codex/coordinator-recovery-843e823b4` preserves pre-rebase
work. Current head is MERGEABLE. Full workspace typecheck/build passed;
UI and snapshot checks passed 31 tests, but 24 database tests were
skipped by the local support probe, and the local browser server could
not initialize PostgreSQL. A direct disposable `initdb` diagnostic
established the host limitation: `could not create shared memory
segment: No space left on device` (`shmget`, shared-memory IDs
exhausted, not disk space). No dev database or host setting was changed.
Diagnostics: `/tmp/pr17983-2001132-initdb-probe.log`,
`/tmp/pr17983-2001132-db-probe.log`; checks:
`/tmp/pr17983-2001132-tests.log`, `/tmp/pr17983-2001132-browser.log`,
`/tmp/pr17983-2001132-types.log`, `/tmp/pr17983-2001132-build.log`.
These local skipped/blocked checks are not claimed as passes; fresh CI
on independent runners is in progress.
- `d18f683cd` reached fresh Greptile 5/5 with zero comments/open threads
and Superagent pass, but [E2E shard
7](https://github.com/paperclipai/paperclip/actions/runs/40785966243/job/123159719925)
failed parsing an Advanced YAML clipboard read beginning with “Skip to
Main Content.” The log is retained at `/tmp/pr17983-d18f-e2e7.log`.
Current head hardens the test helper to verify editor focus and
selection, clear stale clipboard content, and require the YAML header at
the start of a fresh copy before parsing. Saved-state assertions and
timeouts are unchanged. All six Advanced YAML browser journeys passed on
a disposable local instance (41.0s); the exact lost-response journey
also passed with a one-time injected surrounding-page copy, verifying
rejection/reacquisition (17.6s). The precise browser focus/clipboard
timing in CI remains unproven; the observed invalid-read path is now
guarded and deterministically exercised. Logs:
`/tmp/pr17983-yaml-clipboard-scope-browser.log`,
`/tmp/pr17983-yaml-clipboard-injected-browser.log`. Fresh current-head
CI and reviews are pending.
- `19e6dcb0b` passed Superagent and Greptile 5/5, with one nonblocking
deep-link finding. Current head preserves attention links using
`?decisionId=`: the requested pending decision is rendered even beyond
the first 40 cards, focused and scrolled into view, then only that
parameter is consumed so later refreshes do not refocus. Other URL
parameters remain. Needs You tests 10/10 and UI typecheck passed; its
review thread is resolved. The preceding post-rebase regression run
passed 45/45 tests. Fresh current-head CI/reviews are pending.
- `6d772b7cc` reached Greptile 5/5 with zero comments/open threads and
Superagent pass. While 32 checks had passed, master moved and caused a
documentation conflict in `maps/feature-map/onboarding.md`. Rebased
immediately, preserving both onboarding paragraphs. The complete
four-commit branch was pushed as `19e6dcb0b` before waiting on
CI/review, per Scott's instruction. A transient Git index lock
interrupted the first continue; the command sequence briefly pushed
intermediate `34e298c07`, then the completed rebase was immediately
pushed. All feature and security/retry fixes are present; local recovery
ref `codex/coordinator-recovery-6d772b7cc` preserves the pre-rebase
history. The only tree differences from that ref are upstream's two
smoke-test files and the combined documentation paragraph. GitHub
reports MERGEABLE. Fresh current-head verification is in progress;
prior-head review results are not current acceptance.
- `21efa3120` passed Superagent but Greptile scored 4/5 for a valid
retry-key persistence bug: an acknowledged duplicate key could reopen a
resolved review. Current head adds an additive, replay-safe
`0404_light_swarm.sql` migration for indexed, task-scoped aliases from
acknowledged retry keys to the canonical interaction. Alias registration
and retry lookup occur under the existing task lock; payload/actor
mismatch still conflicts, and each canonical review permits at most 100
aliases. Original and acknowledged keys replay the terminal result
rather than creating a new gate. Five newest migration snapshots are
retained. Service/routes tests 165/165 passed; final quality review plus
migration replay/snapshot checks 25/25 passed; server typecheck passed.
The P1 thread is resolved in code; fresh current-head reviews and CI are
pending.
- `69c2c7543` passed 55 checks and Greptile 5/5 with no open Greptile
findings, but Superagent check
[123140675812](https://github.com/paperclipai/paperclip/pull/17983/checks?check_run_id=123140675812)
correctly flagged unbounded pending quality-card creation. Current head
fixes that under the existing task-row lock: equivalent requests with
fresh idempotency keys reuse the pending card; conflicting requests for
an unresolved deliverable return the existing interaction ID; pending
cards are capped at ten per task across all reviewers. Distinct outputs
remain independent, and already-answered items in a partially pending
batch can receive a fresh review. Added real-database concurrency,
cross-card duplicate, partial-answer, and hard-cap regressions. Quality
review 22/22 tests and final server typecheck passed; earlier
service/routes 163/163 tests passed before the final partial-answer
refinement. One added assertion initially expected a service HTTP error,
but duplicate keys are rejected earlier by the existing shared schema;
the assertion now verifies that validation. Fresh current-head
CI/reviews are pending; the action-required security result is not
treated as passed.
- `c9c575e5f` completed all 56 checks successfully with four conditional
skips, Greptile 5/5, and Superagent pass. Its two P2 review findings are
fixed in the current head: independently bound quality approvals now
each remain visible in attention; changing/clearing URL-backed
assignee/status filters updates the URL and survives task navigation.
Both review threads are resolved. Regression evidence: attention service
34/34 tests passed, CoordinatorTasks 20/20 passed, UI and server
typechecks and the full workspace build passed on `69c2c7543`. The first
new UI test run caught an assertion racing React Router's transition;
waiting for the navigation before asserting passes without weakening the
navigation/remount checks. These older-head green checks are not claimed
for the new fix commit.
- Preceding `0e6ef7c44` completed all 56 checks successfully with four
conditional skips, fresh Greptile 5/5 and no open findings, and passed
Superagent. A fifth migration-metadata collision arrived before the
final Runner job finished. Those green results are preserved as
prior-head evidence, not current acceptance.
- Previous fully verified migration head:
`0e6ef7c44696a377f35e69521b93587576374bd0`, integrating master
`f8226b051` after the fourth migration-number collision. Master added
terminal GitHub-response migration 0401 while the final native check
ran; upstream 0401 SQL/journal/snapshot is preserved unchanged, and
preferences are generated as idempotent `0402_yielding_blink.sql` on
top. Exactly the five newest snapshots remain. GitHub reports MERGEABLE
after the push. All 1,155 relevant
migration/review/interaction/native-execution/remote-assets tests,
workspace typecheck/build, migration guards and whitespace checks pass.
Fresh exact-head CI/Greptile/Superagent are pending.
- Immediately preceding head `36c67cf03` completed all 56 checks
successfully with four conditional skips, fresh Greptile 5/5 and no open
findings, plus passed Superagent security. No retry was needed for that
head's CI. Master advanced and introduced only a migration-metadata
conflict while its final native check ran, so that green result does not
establish acceptance of the current integration. Four consecutive
collisions (0398, 0399, 0400, 0401) during verification are retained as
concrete integration churn; a brief pause of competing migration merges
was requested, without assuming it happened or changing any repo
setting. The PR has not been merged.
- Prior fully verified head: `36c67cf03`, on the clean `30059c0ed`
integration of master `efd65c112`. The last Greptile P2 is fixed: Needs
attention retains the participant-agent scope through authoritative
server membership queries for each pending task, including historical
participation when another agent currently owns the task. It does not
approximate participation with current assignment or enumerate every
company task. The actual view-switch regression and all 31 focused
task/routing/collection tests, UI typecheck/build, token gates and
whitespace checks pass. Greptile completed a fresh 5/5 review on this
exact head (review 19) with zero new comments and no open review
threads. CI and Superagent remain in progress; no current-head CI
failure has been reported.
- Head `30059c0ed` received Greptile 5/5 with that one nonblocking
participant finding, now fixed. Superagent passed that head.
Current-base integration restored the ephemeral RunsOn fleet after a
superseded workflow retained queued aggregate jobs: normal cancellation
was insufficient; the single force-cancel request was interrupted but
readback confirmed that obsolete run cancelled and current CI began on
the trusted fleet. No cancellation retry or runner/security/workflow
changes were made. Its partial CI was superseded by the necessary
participant fix and is not current acceptance.
- Previous compatibility/base-sync head:
`30059c0ededc0eab4e04e8299a884f36db8b292f`, integrating master
`efd65c112` and the Inbox/workspace compatibility fixes. GitHub reports
MERGEABLE after the push. Coordinator keeps Inbox standalone under More
without rewriting the instance's Combined Inbox preference; opting out
restores that preference. Saved merged-Inbox URLs resolve to their
original owner, task status presets remain usable, and canonical
repeated `workspace` parameters retain scoped, deduplicated pagination
(including Needs attention). All 192 broader
routing/Inbox/sidebar/privacy/review tests and the final 16 task-page
tests pass, as do workspace typecheck/build, token gates and whitespace
checks. The two review threads are resolved. Fresh exact-head CI,
Greptile and Superagent results are pending.
- The preceding `d5b9be5bd` remained mergeable but its CI routing gate
intentionally fell back to GitHub-hosted runners: “stacked base merge
commit does not contain the current base branch.” All 45 downstream jobs
stayed queued. Current master was merged cleanly to refresh the pinned
base; no runner/security settings or workflow logic were changed.
Greptile scored that preceding head 4/5 with the two compatibility
findings fixed here. Its automatic Superagent scan was inconclusive at
checkout; the documented retry was still pending when superseded. Those
results do not establish acceptance of this current head.
- Previous approval/migration head:
`d5b9be5bd3b8d6b0437e42bf48138377f34fa67d`, integrating master
`ba5bba86d` and the stale-output approval correction. GitHub reports
MERGEABLE after the push. The fresh P1 finding is fixed: every submitted
item now validates and locks its bound output inside the resolution
transaction, whether or not it carries a required-quality-review
binding. Real database regressions verify that an advanced document
revision or deleted attachment refuses the entire batch without
recording approvals, and current ordinary output reviews still resolve.
All 272 combined migration/review/interaction tests pass; workspace
typecheck/build passed. Fresh exact-head Greptile and CI are pending.
The automatic Superagent scan ended inconclusive at checkout; one
documented on-demand retry is pending. These are explicit verification
limits, not passed security results. The P1 thread is resolved.
- Master consumed the next migration slot three times during this
verification pass: confirmation recovery took 0398, continuation
recovery took 0399, and GitHub acknowledgement recovery took 0400. Each
upstream SQL/journal/snapshot is preserved unchanged. Preferences are
now generated as idempotent `0401_white_lenny_balinger.sql` from
upstream 0400; exactly five snapshots remain, and replay coverage
locates this migration by table content. This is recorded concurrent
integration churn, not a claim that older green results apply to the
latest base.
- Head `ed3b68da9` fixed the two history/URL findings with 26 focused
regression tests, server/UI typechecks, workspace build and token gates
passing. Greptile then scored that head 4/5 with the stale-output P1
corrected in the current head. Its partial CI is superseded, not counted
as current acceptance.
- Previous filter-fix head: `ed3b68da9`, based on the `8458d39d9` merge
of master `063cfd438`. The two fresh Greptile findings are fixed:
history outcome filters now match recorded per-item verdicts in SQL
before pagination (mixed reviews remain in both applicable outcomes),
and same-page URL assignee changes/back/forward navigation update that
filter without clearing independent filters. All 26 focused
database/navigation/collection tests, server/UI typechecks, workspace
build, token gates and whitespace checks pass. The two review threads
are resolved. Fresh exact-head CI, Greptile and Superagent results are
pending.
- The `8458d39d9` migration integration also passed all 328 targeted
migration/interaction/confirmation tests, workspace typecheck/build and
Superagent security scan. Greptile scored that head 4/5 with the two
functional gaps fixed in the current head; those prior-head checks are
not current acceptance.
- Migration integration head:
`8458d39d9721f4cb312c2390070a0609f8cddd1a`, integrating master
`063cfd43813cabf18417605fb9eefb9aacf95c8d`. GitHub reports MERGEABLE
after the push. Master advanced twice during this conflict-resolution
pass: first its confirmation-recovery migration took 0398, then its
continuation-recovery migration took 0399. Both upstream migrations and
journal entries remain unchanged; Coordinator preferences are now
generated as idempotent `0400_sad_guardian.sql` from upstream's latest
snapshot. Exactly the five newest snapshots remain. The replay
regression now locates the table migration by content instead of
hardcoding a renumbered filename.
- All 328 targeted question/review/confirmation/migration tests passed
again after the second merge. Workspace typecheck/build and fresh
exact-head CI/reviews are in progress. The preceding `0c46f46d6` passed
workspace typecheck/build and Superagent security scan, but was
superseded because master consumed migration 0399 while those checks
ran. That prior-head result is not acceptance of the current head. No
production settings or runner/security permissions were changed, and the
PR has not been merged.
- Previous integration head: `0c46f46d6f2b23417ab872ce3cb6ff2673647287`,
integrating master `287b04d72f1b9fca1412e05435c75b8302ba0710`. GitHub
reports MERGEABLE after the push. The renewed conflicts were
`0398_snapshot.json`, `_journal.json`, and the top imports in
`issue-thread-interactions.ts`. Upstream's `0398` confirmation-recovery
index and journal history are preserved unchanged; Coordinator
preferences were regenerated as idempotent `0399_bouncy_miek.sql`, with
replay coverage updated and exactly the five newest snapshots retained.
Both terminal confirmation notifications and Coordinator task
assignment/required review behavior are preserved.
- All 328 targeted interaction/review/confirmation/migration tests pass,
as do workspace typecheck, map validation and whitespace checks. The
workspace build and fresh exact-head CI/Greptile/Superagent checks are
pending. The first migration-generation attempt hit the sandbox's local
IPC restriction before generating files; the unchanged authorized local
generation completed successfully. Prior-head green checks below do not
establish acceptance of this merge.
- Previous acceptance head: `8b7dcf739ed15afc51351fdfa0e0e5fbc75ab788`,
with master `1aa3a21ac` integrated and GitHub reporting MERGEABLE.
Greptile completed a fresh 5/5 review on this exact head with zero new
actionable findings and no open review threads. The [fresh CI
workflow](https://github.com/paperclipai/paperclip/actions/runs/40752864319)
completed SUCCESS on attempt 2 at 19:40:08 UTC, including all eight E2E
shards. Final completed check set: 55 succeeded, four conditional checks
skipped, zero pending/queued/running checks, and one
neutral/inconclusive Superagent scan. The auxiliary commitperclip review
and isolated native Runner compilation/cache-reuse check both passed.
This is not a claim that Superagent produced a passed security verdict.
GitHub still reports MERGEABLE against current master
`60b8089a44b4eaa37e627adbfdf2f130bda13335`.
- Current-head E2E: all eight shards and the aggregate passed, including
the previously failing YAML browser journey. Serialized server shard 9/9
failed: the protected YAML projection test exceeded its original
15-second timeout; its late update request then reached the next test's
reset mock (the captured payload is the prior test's model/label
update). All 88 tests in that exact unchanged file passed locally in
38.03 seconds, with no timeout or assertion changes. The original
slow-request cause remains unproven. CI failure log:
`/tmp/pr17983-8b7-serialized9.log`; local recovery:
`/tmp/pr17983-adapter-yaml-repro.log`. The single bounded unchanged
shard rerun passed, and the workflow completed SUCCESS. Rerun log:
`/tmp/pr17983-8b7-serialized9-rerun.log`. The initial failure is
retained; a passing retry does not establish its original slow-request
cause.
- Greptile scored preceding head `4873704be` 5/5 with one valid P2:
opened task decision records missed company-scoped live invalidation.
The current head uses the existing company decision prefix for
resolution and effect-completion events, and resets private record
payloads on access changes. Its thread is resolved. All 107 focused
record/live-update tests, UI typecheck/build, token checks and
whitespace checks pass.
- CI on `4873704be` failed E2E shard 7/8 in the Advanced YAML
lost-creation-response journey: its clipboard helper selected page text
starting with “Skip to Main Content” instead of YAML. The exact
unchanged journey passed in a disposable local browser instance (1 test,
17.1 seconds). No assertion, timeout or production behavior was
weakened; the original failure cause remains unproven. Failure log:
`/tmp/pr17983-487-e2e7.log`; local check:
`/tmp/pr17983-agent-yaml-repro.log`. The full unchanged Advanced YAML
browser suite also passed locally (6 tests, 37.8 seconds). Fresh
current-head CI subsequently passed this journey and all other E2E
shards.
- Both the automatic and one supported on-demand Superagent scan on
`4873704be` ended neutral/inconclusive: “Pinned PR scan merge-base
checkout failed or timed out.” These do not count as a passed security
verdict. The current head's automatic scan and one documented on-demand
retry also returned the identical neutral checkout failure, with zero
annotations or further diagnostics. This is an external scanner blocker,
not a passed security verdict. No security settings or permissions were
changed.
- Recorded decision links now render and focus the full source-task
record in both Classic and Coordinator, including recorded choice,
submitted inputs, and the existing effect-result resolver. Fresh
terminal records take precedence over older open-decision cache entries.
Context and account checks keep mismatched or inaccessible records out
of the DOM; no decision-history tab was added to tasks or chat. The
shared company live-run feed now drives live-only filtering, including
ended runs and terminal tasks with stale run entries. All 213 focused
tests, UI typecheck/build, token and whitespace checks pass; the final
focus follow-up passed all 188 task-detail tests and UI typecheck. The
first new navigation test failed because its test host omitted company
context; the host was corrected before the passing run.
- Previous integration head: `375c5e89516705050d94bab99336212841b2e6e1`
merges master `1aa3a21ac` after four additional upstream commits. The
sole conflict was competing imports in `server/src/services/issues.ts`;
both quality-review policy and inbound wake-notice notifications are
preserved. The PR-specific review import now sits beside
execution-policy imports instead of the frequently changing first line.
GitHub mergeability is rechecked after the push.
- On the combined master tree, workspace typecheck/build, all 153
affected UI tests, map validation, and whitespace checks pass. Four
server suites passed 77 tests; the fifth suite's module-import hook
timed out while broad verification was running. Its unchanged isolated
run passed all 41 tests in 7.2 seconds with the original timeout. This
records recovery without claiming the original import-timeout cause is
proven. CI on `375c5e895` passed every substantive job, including all
eight E2E shards and Superagent; Greptile found the two functional gaps
fixed in the current head. A short-lived `669d90876` commit added the
main fixes before the focused navigation follow-up.
- Immediately preceding head `9d9737b49` completed all 56 checks
successfully with four conditional skips, Greptile 5/5 and no open
findings, plus a successful Superagent scan. The signoff E2E shard
passed on its single accepted rerun; its initial cancellation/timing
failure is retained below. Master moved again while those checks ran,
requiring this final integration rather than treating the prior head's
green result as current acceptance.
- Previous head `9d9737b496911a6054d0ea22d1b42b28b115e9ab` includes
master `d30c8f42b842327ee1c18c41a2efd6ba623da5cd`; that head was
subsequently conflicted by moving master. The master merge preserved
durable account-bound writes and Coordinator UX. Workspace
typecheck/build, 416 focused UI tests, and 142 affected comment-route
tests passed. A stable full UI run on preceding head `938ad1665` passed
all 9,833 tests across 836 files; its final full workspace build also
passed.
- Greptile found nested proposal project inheritance and Classic
bound-revision links on `91be74952`. Both were fixed with 142
interaction-route tests, 113 focused UI tests, and UI/server typechecks.
Its next review identified a valid draft-preservation bug in the new
Classic loading UI. Head `9d9737b49` keeps loading/error notices outside
editors and annotations while preserving dirty drafts; 115 focused
review UI tests pass, including pending/failed revision loads and
continued draft editing. All related threads are resolved. Greptile
completed a fresh 5/5 review of
`9d9737b496911a6054d0ea22d1b42b28b115e9ab` with no open findings.
- CI on `938ad1665` exposed an integration regression with the newly
merged durable startup saves: the personal preference gate blocked the
task page before its first session lookup. The guarded title editor now
mounts while that lookup is pending, with presentation-specific tools
still gated. All 216 affected component tests pass. The unchanged
five-test `disconnect-write-recovery.spec.ts` browser suite passes
against a newly launched disposable local-trusted PostgreSQL instance on
port 33219, including the exact failed startup-save journey. No live
provider, hosted authentication or production proof is claimed. The
helper-owned test server exited and the port has no listener. UI
typecheck, built-browser assets, token and whitespace checks pass.
- Initial durable-write merge tests found duplicate imports and unscoped
draft fixtures, fixed before 416 focused tests passed. A sandbox
typecheck could not create its IPC socket; the supported rerun passed.
The first revision-link test used a wrong Return to latest button
assertion, corrected before the 113-test pass. The startup E2E
reproduction initially still failed because an outer shell gate also
blocked the page; the subsequent fix passed all five cases without
changing the test or its timeouts. A focused loading-header assertion
initially used an incorrect fixture title; correcting it preceded the
216-test pass. These failures remain recorded.
- Earlier routing gates fell back to GitHub-hosted runners while master
was moving or conflicted. Superseded queued runs were cancelled, then
the `938ad1665` routing gate explicitly selected the ephemeral RunsOn
fleet. Its CI actively executed and completed most tests; only the
diagnosed startup E2E shard and aggregate failed, alongside the valid
Greptile finding. Auxiliary GitHub-hosted workflows also completed
successfully on earlier heads, so queueing alone is not an
administrative blocker. Fresh [CI on
`9d9737b49`](https://github.com/paperclipai/paperclip/actions/runs/40744012678)
is the acceptance target; the previously failing startup E2E shard now
passes. A different signoff E2E shard failed during executor
resubmission when its fixture run was cancelled as
`issue_continuation_waiting_on_review`. The exact unchanged journey
passed locally (1 test, 46 seconds), and the preceding head with
identical server/signoff code passed that CI shard. This supports a
bounded rerun but does not establish the timing failure's root cause.
After the workflow finished, GitHub accepted one rerun of that failed
shard and its dependent aggregate; both passed, completing all checks on
that head. The failure log is retained at `/tmp/pr17983-9d9737-e2e7.log`
and the local reproduction at `/tmp/pr17983-signoff-repro.log`. No
assertion or timeout was weakened.
- On the first merge, full workspace typecheck/build, all 9,622 UI
tests, 267 tests across seven affected server suites, two migration
checks, map validation, token gates, and the PR diff whitespace check
passed. Greptile then found duplicate local task-search filtering and
lost completed item-review detail. Both are fixed, with 42 passing
focused UI tests and passing UI typecheck/build.
- A subsequent Greptile review found attachment confirmations without
working links and mobile history without a visible host. Confirmation
layouts now derive the authenticated immutable-content endpoint from the
attachment ID, and Needs You uses the existing full-screen mobile sheet.
All 119 focused tests pass, including missing/stale display URLs, both
review layouts, and both mobile history entry buttons. UI
typecheck/build and token checks pass. Full workspace typecheck/build
also pass after the second master merge.
- Greptile gave head 22f69c304 a 5/5 score with one non-blocking
desktop-history reopening finding. Head 6dd52b6d3 fixed it, passed all
nine history tests, UI typecheck/build, token gates, and the full
9,679-test UI suite, then received a fresh 5/5 review with zero open
findings. These are prior-head results; review on the master-sync commit
is pending.
- The chat and workspace CI jobs on head 22f69c304 were interrupted by
runner shutdown signals and cancellation before completing their tests.
Their logs do not report failing assertions. Those interrupted jobs are
not claimed as passes. The new head starts fresh runs.
- The first merge CI had a Slack session-sync idle assertion fail after
an unexpected database read. Its exact isolated reproduction passes, and
the chat CI shard passed on head 618c5a2a3. The original failure's cause
remains undetermined. The broader local reproduction, run concurrently
with full verification, instead encountered 21 timeout failures across
its two test chunks; it is not claimed as a passing full local server
run. A concurrent local UI run had eight failures (seven timeouts and
one delayed autosave assertion). The complete UI run on the final head
passed all 9,679 tests across 830 files with two workers; no test
timeout or assertion was weakened. This establishes recovery without
claiming that every earlier timeout's cause has been proved. An earlier
intermediate UI run also overlapped the second master merge and failed
the newly added Advanced-sidebar expectation, so it is not treated as
stable-tree evidence. The newly merged UI dependencies initially
required refreshing the local install from the upstream package
manifest; the generated local lockfile update is not committed, per
repository policy.
- Superagent returned neutral/inconclusive scans on the preceding heads
because its pinned merge-base checkout failed or timed out. The
available retry endpoint returned 404. These are not successful security
verdicts. The fresh scan on `91be74952` passed (PR scan passed; no
suspicious PR changes detected). The `938ad1665` scan again returned
neutral/inconclusive with the same pinned merge-base checkout error; the
check-suite rerun endpoint returned 404. The fresh `9d9737b49` scan
completed successfully at 18:22 UTC with no suspicious changes detected.
Earlier neutral scans are not counted as successful security verdicts.
- Prior head `57902152ac3218096fca669d33ff18400a2a86ae` passed 9,618 UI
tests across 826 files, received Greptile 5/5 with no open findings, and
completed CI with 56 passes and four conditional skips. These are
historical results, not acceptance of the latest merge commit.
- The first full build found generated capability-contract drift. The
generated outputs were refreshed. Initial sandbox runs could not open
local IPC sockets; verification was rerun with local test access.
- Current master was merged. The workspace preference migration was
regenerated after the existing migration history.
- The UI run first found two outdated expectations: early custom-answer
validation and the document access error copy. Both were corrected, and
the full UI suite then passed.
- The local server run found three failures in unchanged tests. The Muse
test passes with Python 3.13 instead of the host's Python 3.9. The two
workspace tests pass when their test process uses the expected master
default branch. These environment settings do not change application
behavior or global machine settings. The unsharded server run was
stopped after those reproductions; CI supplies the full sharded server
result. No successful full local server run is claimed.
- Skill inventory generation needs an external eval checkout, which is
absent locally. The existing generator functions refreshed only the live
skill inventory and its derived files. The checked-in eval inventory was
preserved. The inventory validation gate passes.
- Greptile found a dropped MCP review requirement, missing per-item
output links, generic item-review summaries, and a history-refresh
failure. The fixes have 32 passing database-backed server tests and
focused rendered-card tests. A follow-up partial-review finding was
corrected by forwarding the canonical mutation status before settling
the card. Attachments open through the authenticated immutable-content
route in both card layouts. The final focused UI set has 117 passing
tests, including these regressions.
- Both browser shards that first failed expected a disabled-chat
message. The copy was restored, and those shards passed in CI on the
next commit. The two journeys also passed locally. One local attempt
first timed out before feature disable while waiting for the provider's
initial message; its isolated reproduction passed. That earlier timing
failure remains recorded and is not presented as an explained
infrastructure failure.
- Serialized server shard 9 first failed during pnpm setup with
ECONNRESET, before dependencies or tests ran. Its single rerun passed
setup and all tests. The build and production runtime boot passed on the
first attempt, but that run's public-install step timed out during Pi
setup. The single build-job rerun passed compilation, runtime boot, and
clean public-install verification. Those setup scripts are unchanged by
this PR. The successful retry establishes recovery; the earlier Pi
timeout's underlying cause remains undetermined. The aggregate CI gate
is green: [workflow
run](https://github.com/paperclipai/paperclip/actions/runs/40637789695).

</details>

## Risks

- This touches shared task, interaction, and completion paths.
Regression tests must cover classic and experimental behavior,
authorization, duplicate answers, and stale review targets.
- The additive preference table must migrate before the setting is used.
The setting defaults off.
- The experiment changes navigation and message presentation. Canonical
task and document records remain the source of truth.
- The initial task brief and saved plan can repeat information. This
change fixes chronology; a broader redundancy policy remains a design
decision.

## Model Used

OpenAI GPT-6 Codex with reasoning and tool-assisted code editing, tests,
and browser verification. Earlier iterations used Codex models in this
task; their exact model IDs are not available in the current context.

## Checklist

- [x] I have included a thinking path that traces from project context
to this change
- [x] I have specified the model used
- [x] I have checked ROADMAP.md
- [x] I have searched GitHub for duplicate or related PRs
- [x] I have described the issue in-PR following the relevant issue
template
- [x] I have not referenced internal/instance-local Paperclip issues or
links
- [x] My branch name describes the change
- [x] I have run relevant tests locally and they pass
- [x] I have added or updated tests where applicable
- [x] I have updated relevant documentation
- [x] I have documented risks
- [x] All Paperclip CI gates are green
- [x] Greptile is 5/5 with no open recommendations or follow-ups
- [x] I will address review comments before requesting merge

---------

Co-authored-by: Paperclip <noreply@paperclip.ing>
2026-10-28 17:20:01 -07:00

3683 lines
141 KiB
TypeScript

import express from "express";
import request from "supertest";
import { beforeAll, beforeEach, describe, expect, it, vi } from "vitest";
const ASSIGNEE_AGENT_ID = "11111111-1111-4111-8111-111111111111";
const UNRELATED_AGENT_ID = "33333333-3333-4333-8333-333333333333";
const CREATED_AGENT_ID = "22222222-2222-4222-8222-222222222222";
const ISSUE_ID = "aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa";
const OTHER_ISSUE_ID = "bbbbbbbb-bbbb-4bbb-8bbb-bbbbbbbbbbbb";
// Authenticated run ids are real UUIDs in production, and the per-run
// cross-issue influence counter fails closed on a malformed one before it can
// reach the database, so the fixtures must look like the real thing.
const RUN_1 = "d1111111-1111-4111-8111-111111111111";
const RUN_2 = "d2222222-2222-4222-8222-222222222222";
const RUN_3 = "d3333333-3333-4333-8333-333333333333";
const RUN_9 = "d9999999-9999-4999-8999-999999999999";
const RUN_CROSS_COMPANY = "dccccccc-cccc-4ccc-8ccc-cccccccccccc";
const RUN_INVALID = "d0000000-0000-4000-8000-000000000000";
const RUN_LOW_TRUST = "d0000000-0000-4000-8000-000000000001";
const RUN_TASK_BRIDGE = "d0000000-0000-4000-8000-000000000002";
const RUN_WATCHDOG = "d0000000-0000-4000-8000-000000000003";
const mockIssueService = vi.hoisted(() => ({
getById: vi.fn(),
listReviewAttention: vi.fn(),
addComment: vi.fn(),
}));
const mockDecisionReceipts = vi.hoisted(() => ({ list: vi.fn(), listCreatedTasks: vi.fn() }));
const mockDocumentService = vi.hoisted(() => ({ getIssueDocumentRevision: vi.fn() }));
const mockProjectService = vi.hoisted(() => ({ getById: vi.fn() }));
const mockInteractionService = vi.hoisted(() => ({
listForIssue: vi.fn(),
getForIssue: vi.fn(),
create: vi.fn(),
acceptInteraction: vi.fn(),
acceptSuggestedTasks: vi.fn(),
rejectInteraction: vi.fn(),
rejectSuggestedTasks: vi.fn(),
expireRequestConfirmationsSupersededByHistoricalComments: vi.fn(),
expirePendingInteractionsForTerminalIssue: vi.fn(),
answerQuestions: vi.fn(),
submitItemVerdicts: vi.fn(),
cancelQuestions: vi.fn(),
skipInteraction: vi.fn(),
dismissInteraction: vi.fn(),
withdrawInteraction: vi.fn(),
recordSecretProposalExecutionResult: vi.fn(),
}));
const mockHeartbeatService = vi.hoisted(() => ({
wakeup: vi.fn(async () => undefined),
cancelRun: vi.fn(async () => null),
}));
const mockRequestNativeQuestionRunCancellation = vi.hoisted(() =>
vi.fn(async () => null as string | null)
);
vi.mock("../services/native-runtime/native-question-bridge.js", () => ({
deliverNativeQuestionResponse: vi.fn(async () => "not_native"),
requestNativeQuestionRunCancellation: mockRequestNativeQuestionRunCancellation,
validateNativeQuestionResponseInput: vi.fn(),
}));
const mockQuestionResponseDeliveries = vi.hoisted(() => ({
deliver: vi.fn(async () => null),
}));
const mockResolveTaskWatchdogMutationScope = vi.hoisted(() => vi.fn(async () => ({ kind: "none" })));
const mockResolveCoreTrustPreset = vi.hoisted(() => vi.fn(() => ({ kind: "standard" })));
const mockRunAttribution = vi.hoisted(() => ({
value: {
companyId: "company-1",
agentId: "22222222-2222-4222-8222-222222222222",
responsibleUserId: null,
} as Record<string, unknown> | null,
}));
const mockAccessDecide = vi.hoisted(() => vi.fn(async (input: { action?: string }) => ({
allowed: true,
action: input.action,
reason: "allow_explicit_grant",
explanation: "Allowed by test grant.",
})));
const mockLogActivity = vi.hoisted(() => vi.fn(async () => undefined));
const mockReviewTransition = vi.hoisted(() => ({
value: null as null | { actorType: string; actorId: string; details: Record<string, unknown> },
}));
const mockDbSelectWhere = vi.hoisted(() => vi.fn(() => ({
then: (onFulfilled: (rows: unknown[]) => unknown, onRejected?: (reason: unknown) => unknown) =>
Promise.resolve([{ companyId: "company-1", agentId: CREATED_AGENT_ID, contextSnapshot: null }]).then(
onFulfilled,
onRejected,
),
})));
const mockDbSelectFrom = vi.hoisted(() => vi.fn(() => ({ where: mockDbSelectWhere })));
const mockDbSelect = vi.hoisted(() => vi.fn(() => ({ from: mockDbSelectFrom })));
// The per-run cross-issue influence counter runs in its own locking
// transaction, so the route harness has to model it: the locked run row (whose
// persisted context snapshot names the source issue) and the observation count.
const mockCrossIssueInfluence = vi.hoisted(() => ({
sourceIssueId: null as string | null,
priorCount: 0,
inserted: [] as Array<Record<string, unknown>>,
}));
const mockDbTransaction = vi.hoisted(() => vi.fn(async (callback: (tx: unknown) => unknown) => callback({
select: (selection: Record<string, unknown>) => ({
from: () => ({
where: () => {
if (Object.keys(selection).includes("count")) {
return {
then: (resolve: (rows: unknown[]) => unknown) =>
resolve([{ count: mockCrossIssueInfluence.priorCount }]),
};
}
const run = mockRunAttribution.value;
return {
for: () => ({
then: (resolve: (rows: unknown[]) => unknown) => resolve(run
? [{
id: run.runId ?? null,
companyId: run.companyId ?? null,
agentId: run.agentId ?? null,
responsibleUserId: run.responsibleUserId ?? null,
contextSnapshot: { issueId: mockCrossIssueInfluence.sourceIssueId },
}]
: []),
}),
};
},
}),
}),
insert: () => ({
values: async (value: Record<string, unknown>) => {
mockCrossIssueInfluence.inserted.push(value);
if (value.action === "issue.cross_issue_influence_observed") mockCrossIssueInfluence.priorCount += 1;
},
}),
})));
const mockDb = vi.hoisted(() => ({
select: mockDbSelect,
transaction: mockDbTransaction,
}));
vi.mock("@paperclipai/shared/telemetry", () => ({
trackAgentTaskCompleted: vi.fn(),
trackErrorHandlerCrash: vi.fn(),
}));
vi.mock("../telemetry.js", () => ({
getTelemetryClient: vi.fn(() => ({ track: vi.fn() })),
}));
vi.mock("../services/task-watchdog-scope.js", () => ({
TASK_WATCHDOG_ORIGIN_KIND: "task_watchdog",
resolveTaskWatchdogMutationScope: mockResolveTaskWatchdogMutationScope,
taskWatchdogScopeAllowsIssueMutation: vi.fn(async (_db, scope) => scope),
}));
vi.mock("../services/trust-preset-resolver.js", () => ({
LOW_TRUST_ISSUE_ANCESTRY_MAX_DEPTH: 100,
resolveCoreTrustPreset: mockResolveCoreTrustPreset,
}));
function registerModuleMocks() {
vi.doMock("../services/conversation-decision-receipts.js", () => ({
conversationDecisionReceiptsService: () => mockDecisionReceipts,
}));
vi.doMock("../services/question-response-delivery.js", () => ({
questionResponseDeliveryService: () => mockQuestionResponseDeliveries,
}));
vi.doMock("../services/index.js", () => ({
companyService: () => ({
getById: vi.fn(async () => ({ id: "company-1" })),
}),
accessService: () => ({
canUser: vi.fn(async () => true),
decide: mockAccessDecide,
hasPermission: vi.fn(async () => true),
}),
agentService: () => ({
getById: vi.fn(async () => ({ id: CREATED_AGENT_ID, companyId: "company-1", permissions: null })),
resolveByReference: vi.fn(async (_companyId: string, raw: string) => ({
ambiguous: false,
agent: { id: raw },
})),
}),
clampIssueListLimit: (value: number) => value,
companySkillService: () => ({
completeTestRunForIssue: vi.fn(async () => null),
}),
ISSUE_LIST_DEFAULT_LIMIT: 500,
ISSUE_LIST_MAX_LIMIT: 1000,
documentAnnotationService: () => ({ remapOpenThreadsForDocument: async () => [] }),
documentService: () => mockDocumentService,
executionWorkspaceService: () => ({}),
feedbackService: () => ({
listIssueVotesForUser: vi.fn(async () => []),
saveIssueVote: vi.fn(async () => ({ vote: null, consentEnabledNow: false, sharingEnabled: false })),
}),
goalService: () => ({}),
heartbeatService: () => mockHeartbeatService,
instanceSettingsService: () => ({
get: vi.fn(async () => ({
id: "instance-settings-1",
general: {
censorUsernameInLogs: false,
feedbackDataSharingPreference: "prompt",
},
})),
listCompanyIds: vi.fn(async () => ["company-1"]),
}),
issueApprovalService: () => ({}),
issueReferenceService: () => ({
deleteDocumentSource: async () => undefined,
diffIssueReferenceSummary: () => ({
addedReferencedIssues: [],
removedReferencedIssues: [],
currentReferencedIssues: [],
}),
emptySummary: () => ({ outbound: [], inbound: [] }),
listIssueReferenceSummary: async () => ({ outbound: [], inbound: [] }),
syncComment: async () => undefined,
syncDocument: async () => undefined,
syncIssue: async () => undefined,
}),
issueRecoveryActionService: () => ({
getActiveForIssue: vi.fn(async () => null),
listActiveForIssues: vi.fn(async () => new Map()),
}),
issueService: () => mockIssueService,
issueThreadInteractionService: () => mockInteractionService,
taskWatchdogService: () => ({
getActiveForIssue: vi.fn(async () => null),
upsertForIssue: vi.fn(),
disableForIssue: vi.fn(async () => null),
revalidateMutationScope: vi.fn(async (scope: unknown) => ({ allowed: true, scope })),
}),
logActivity: mockLogActivity,
projectService: () => mockProjectService,
routineService: () => ({
syncRunStatusForIssue: vi.fn(async () => undefined),
}),
workProductService: () => ({}),
}));
}
function createIssue(overrides: Record<string, unknown> = {}) {
return {
id: "aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa",
companyId: "company-1",
status: "in_progress",
workMode: "standard",
priority: "medium",
projectId: null,
goalId: null,
parentId: null,
assigneeAgentId: ASSIGNEE_AGENT_ID,
assigneeUserId: null,
createdByUserId: "local-board",
identifier: "PAP-1714",
title: "Persist interactions",
executionPolicy: null,
executionState: null,
hiddenAt: null,
...overrides,
};
}
async function loadAppModules() {
const routes = await import("../routes/issues.js");
const middleware = await import("../middleware/index.js");
return [routes, middleware] as const;
}
async function createApp(actor: Record<string, unknown> = {
type: "board",
userId: "local-board",
companyIds: ["company-1"],
source: "local_implicit",
isInstanceAdmin: false,
}, routeOptions: Record<string, unknown> = {}) {
if (actor.type === "agent") {
mockRunAttribution.value = {
runId: actor.runId,
companyId: actor.companyId ?? "company-1",
agentId: actor.agentId,
responsibleUserId: actor.onBehalfOfUserId ?? null,
};
}
const [{ issueRoutes }, { errorHandler }] = await loadAppModules();
const app = express();
app.use(express.json());
app.use((req, _res, next) => {
(req as any).actor = actor;
next();
});
app.use("/api", issueRoutes(mockDb as any, {} as any, routeOptions));
app.use(errorHandler);
return app;
}
async function resolveMockInteraction(
args: unknown[],
interaction: Record<string, unknown>,
) {
const mutationOptions = args[4] as {
afterResolveInTransaction?: (
tx: Record<string, unknown>,
resolved: Record<string, unknown>,
) => Promise<void>;
} | undefined;
await mutationOptions?.afterResolveInTransaction?.({}, interaction);
return interaction;
}
describe("issue thread interaction routes", () => {
beforeAll(async () => {
vi.resetModules();
vi.doUnmock("../routes/issues.js");
vi.doUnmock("../routes/authz.js");
vi.doUnmock("../middleware/index.js");
vi.doUnmock("../services/index.js");
registerModuleMocks();
await loadAppModules();
}, 60_000);
beforeEach(() => {
// Every mock here is a vi.hoisted() singleton. All 70+ tests share it.
// A queued mockResolvedValueOnce() value can outlive its own test and
// leak into a later, unrelated test. vi.resetAllMocks() drains that
// queue for every mock in one call. It also keeps each mock's
// constructor-provided vi.fn(impl) default. So the code below only
// sets values that must differ from that default.
// vi.clearAllMocks() clears call history only. It does not drain the
// queue. That gap once let a leftover queued value deny an unrelated
// later test.
vi.resetAllMocks();
// mockRunAttribution.value is a plain object, not a vi.fn().
// resetAllMocks() does not reset it. createApp() overwrites it for an
// agent actor. A board actor leaves whatever value a prior test set here.
mockRunAttribution.value = {
companyId: "company-1",
agentId: CREATED_AGENT_ID,
responsibleUserId: null,
};
mockQuestionResponseDeliveries.deliver.mockResolvedValue(null);
mockRequestNativeQuestionRunCancellation.mockResolvedValue(null);
mockResolveTaskWatchdogMutationScope.mockResolvedValue({ kind: "none" });
mockResolveCoreTrustPreset.mockReturnValue({ kind: "standard" });
mockAccessDecide.mockImplementation(async (input: { action?: string }) => ({
allowed: true,
action: input.action,
reason: "allow_explicit_grant",
explanation: "Allowed by test grant.",
}));
mockIssueService.getById.mockResolvedValue(createIssue());
mockDecisionReceipts.list.mockResolvedValue({ items: [], nextCursor: null });
mockDecisionReceipts.listCreatedTasks.mockResolvedValue({ items: [], nextCursor: null });
mockDocumentService.getIssueDocumentRevision.mockResolvedValue(null);
mockProjectService.getById.mockImplementation(async (id: string) => ({ id, companyId: "company-1" }));
mockIssueService.listReviewAttention.mockResolvedValue(new Map());
mockInteractionService.listForIssue.mockResolvedValue([]);
mockInteractionService.expireRequestConfirmationsSupersededByHistoricalComments.mockResolvedValue([]);
mockInteractionService.expirePendingInteractionsForTerminalIssue.mockResolvedValue([]);
mockInteractionService.getForIssue.mockResolvedValue({
id: "interaction-withdraw",
kind: "ask_user_questions",
createdByAgentId: CREATED_AGENT_ID,
sourceRunId: RUN_1,
requestedResolverPolicy: "board_or_agents",
effectiveResolverPolicy: "board_or_agents",
continuationPolicy: "wake_assignee",
status: "pending",
payload: { version: 1, questions: [] },
});
mockInteractionService.withdrawInteraction.mockImplementation((...args) => resolveMockInteraction(args, {
id: "interaction-withdraw",
companyId: "company-1",
issueId: "aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa",
kind: "request_confirmation",
createdByAgentId: CREATED_AGENT_ID,
status: "cancelled",
continuationPolicy: "wake_assignee",
payload: { version: 1, prompt: "Proceed?" },
result: { version: 1, outcome: "withdrawn", reason: "Replanning" },
}));
mockInteractionService.recordSecretProposalExecutionResult.mockImplementation(
async (_issue, _interactionId, _proposalId, execution) => ({
...(await mockInteractionService.acceptInteraction.mock.results.at(-1)?.value)?.interaction,
id: _interactionId,
companyId: "company-1",
issueId: "aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa",
kind: "request_confirmation",
status: "accepted",
continuationPolicy: "wake_assignee",
payload: {
version: 1,
prompt: "Create the binding?",
secretProposal: {
version: 1,
proposalId: _proposalId,
configPath: "access.NEW_ALIAS",
},
},
result: { version: 1, outcome: "accepted", secretProposal: { version: 1, ...execution } },
}),
);
mockInteractionService.create.mockResolvedValue({
id: "interaction-1",
companyId: "company-1",
issueId: "aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa",
kind: "suggest_tasks",
status: "pending",
continuationPolicy: "wake_assignee",
idempotencyKey: null,
sourceCommentId: null,
sourceRunId: RUN_1,
payload: {
version: 1,
tasks: [{ clientKey: "task-1", title: "One" }],
},
result: null,
createdAt: "2026-04-20T12:00:00.000Z",
updatedAt: "2026-04-20T12:00:00.000Z",
});
mockInteractionService.acceptInteraction.mockResolvedValue({
interaction: {
id: "interaction-1",
companyId: "company-1",
issueId: "aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa",
kind: "suggest_tasks",
status: "accepted",
continuationPolicy: "wake_assignee",
idempotencyKey: null,
sourceCommentId: "comment-1",
sourceRunId: RUN_1,
payload: {
version: 1,
tasks: [{ clientKey: "task-1", title: "One" }],
},
result: {
version: 1,
createdTasks: [{ clientKey: "task-1", issueId: "child-1" }],
skippedClientKeys: ["task-2"],
},
createdAt: "2026-04-20T12:00:00.000Z",
updatedAt: "2026-04-20T12:05:00.000Z",
resolvedAt: "2026-04-20T12:05:00.000Z",
},
createdIssues: [
{
id: "child-1",
assigneeAgentId: CREATED_AGENT_ID,
status: "todo",
},
],
});
mockInteractionService.rejectInteraction.mockResolvedValue({
id: "interaction-1",
companyId: "company-1",
issueId: "aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa",
kind: "suggest_tasks",
status: "rejected",
continuationPolicy: "wake_assignee",
idempotencyKey: null,
sourceCommentId: "comment-1",
sourceRunId: RUN_1,
payload: {
version: 1,
tasks: [{ clientKey: "task-1", title: "One" }],
},
result: {
version: 1,
rejectionReason: "Not actionable enough",
},
createdAt: "2026-04-20T12:00:00.000Z",
updatedAt: "2026-04-20T12:05:00.000Z",
resolvedAt: "2026-04-20T12:05:00.000Z",
});
mockInteractionService.answerQuestions.mockResolvedValue({
id: "interaction-2",
companyId: "company-1",
issueId: "aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa",
kind: "ask_user_questions",
status: "answered",
continuationPolicy: "wake_assignee",
idempotencyKey: null,
sourceCommentId: "comment-2",
sourceRunId: RUN_2,
payload: {
version: 1,
questions: [{
id: "scope",
prompt: "Scope?",
selectionMode: "single",
options: [{ id: "phase-1", label: "Phase 1" }],
}],
},
result: {
version: 1,
answers: [{ questionId: "scope", optionIds: ["phase-1"] }],
},
createdAt: "2026-04-20T12:00:00.000Z",
updatedAt: "2026-04-20T12:06:00.000Z",
resolvedAt: "2026-04-20T12:06:00.000Z",
});
mockInteractionService.submitItemVerdicts.mockResolvedValue({
interaction: {
id: "interaction-verdicts",
companyId: "company-1",
issueId: "aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa",
kind: "request_item_verdicts",
status: "pending",
continuationPolicy: "wake_assignee",
idempotencyKey: null,
sourceCommentId: "comment-verdicts",
sourceRunId: "run-verdicts",
payload: {
version: 1,
prompt: "Review generated artifacts.",
items: [
{ id: "api", label: "API route" },
{ id: "docs", label: "Docs" },
],
verdicts: ["approve", "reject"],
requireReasonOn: ["reject"],
allowBulkApprove: true,
},
result: {
version: 1,
outcome: "resolved",
complete: false,
items: [
{
id: "docs",
verdict: "reject",
reason: "Missing examples",
resolvedByUserId: "local-board",
resolvedAt: "2026-04-20T12:06:00.000Z",
},
],
},
createdAt: "2026-04-20T12:00:00.000Z",
updatedAt: "2026-04-20T12:06:00.000Z",
resolvedAt: null,
},
newlyResolvedItemIds: ["docs"],
});
mockInteractionService.cancelQuestions.mockImplementation((...args) => resolveMockInteraction(args, {
id: "interaction-2",
companyId: "company-1",
issueId: "aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa",
kind: "ask_user_questions",
status: "cancelled",
continuationPolicy: "wake_assignee",
idempotencyKey: null,
sourceCommentId: "comment-2",
sourceRunId: RUN_2,
payload: {
version: 1,
questions: [{
id: "scope",
prompt: "Scope?",
selectionMode: "single",
options: [{ id: "phase-1", label: "Phase 1" }],
}],
},
result: {
version: 1,
answers: [],
cancelled: true,
cancellationReason: null,
summaryMarkdown: null,
},
createdAt: "2026-04-20T12:00:00.000Z",
updatedAt: "2026-04-20T12:05:00.000Z",
resolvedAt: "2026-04-20T12:05:00.000Z",
}));
mockDbSelect.mockImplementation(() => ({ from: mockDbSelectFrom }));
mockDbSelectFrom.mockImplementation(() => ({ where: mockDbSelectWhere }));
mockDbSelectWhere.mockImplementation(() => ({
then: (onFulfilled: (rows: unknown[]) => unknown, onRejected?: (reason: unknown) => unknown) =>
Promise.resolve(mockRunAttribution.value ? [mockRunAttribution.value] : []).then(
onFulfilled,
onRejected,
),
orderBy: () => ({
limit: () => Promise.resolve(mockReviewTransition.value ? [mockReviewTransition.value] : []),
}),
}));
mockReviewTransition.value = null;
// Default: the authenticated run is already working the issue under test, so
// resolution is a same-issue write and the cross-issue counter must ignore it.
mockCrossIssueInfluence.sourceIssueId = ISSUE_ID;
mockCrossIssueInfluence.priorCount = 0;
mockCrossIssueInfluence.inserted.length = 0;
});
describe.each([
{ route: "decision-receipts", method: "list" as const },
{ route: "chat-handoffs", method: "listCreatedTasks" as const },
])("owned conversation $route", ({ route, method }) => {
const owner = {
type: "board", userId: "paperclip-id:chat-owner", source: "cloud_tenant",
companyIds: ["company-1"], isInstanceAdmin: false,
};
const path = `/api/issues/${ISSUE_ID}/${route}`;
const list = mockDecisionReceipts[method];
beforeEach(() => {
mockIssueService.getById.mockResolvedValue(createIssue({
conversationAgentId: ASSIGNEE_AGENT_ID, conversationUserId: owner.userId,
}));
});
it("returns the owner's canonical records with bounded pagination and no writes or wake", async () => {
const page = { items: [method === "list"
? { id: "interaction:decision-1", issueId: OTHER_ISSUE_ID }
: { id: `chat_handoff:${OTHER_ISSUE_ID}`, issueId: OTHER_ISSUE_ID, identifier: "PAP-1715",
title: "Write the launch post", status: "todo", createdAt: "2026-10-26T15:00:00Z", sourceRunId: null, suggestionInteractionId: null }],
nextCursor: "next-page" };
list.mockResolvedValue(page);
const response = await request(await createApp(owner)).get(path).query({ limit: 100, cursor: "prior-page" });
expect(response.status).toBe(200);
expect(response.body).toEqual(page);
expect(mockAccessDecide).toHaveBeenCalledWith(expect.objectContaining({
actor: owner, action: "issue:read", resource: expect.objectContaining({ issueId: ISSUE_ID }),
}));
expect(list).toHaveBeenCalledWith("company-1", ISSUE_ID, {
actor: owner, userId: owner.userId, cursor: "prior-page", limit: 100,
});
expect(mockLogActivity).not.toHaveBeenCalled();
expect(mockHeartbeatService.wakeup).not.toHaveBeenCalled();
});
it("leaves omitted pagination to the history service defaults", async () => {
const response = await request(await createApp(owner)).get(path);
expect(response.status).toBe(200);
expect(list).toHaveBeenCalledWith("company-1", ISSUE_ID, {
actor: owner, userId: owner.userId,
});
});
it.each(["chat-owner", "paperclip-id:another-user"])("does not treat another identity as the owner: %s", async (userId) => {
const response = await request(await createApp({ ...owner, userId })).get(path);
expect(response.status).toBe(404);
expect(list).not.toHaveBeenCalled();
});
it.each([
{ name: "agent", actor: { type: "agent", agentId: ASSIGNEE_AGENT_ID, companyId: "company-1", runId: RUN_1 } },
{ name: "board without user", actor: { ...owner, userId: undefined } },
])("denies $name before reading conversation content", async ({ actor }) => {
const response = await request(await createApp(actor)).get(path);
expect(response.status).toBe(403);
expect(list).not.toHaveBeenCalled();
});
it("rejects a normal task even when the user can read it", async () => {
mockIssueService.getById.mockResolvedValue(createIssue({ conversationUserId: owner.userId }));
expect((await request(await createApp(owner)).get(path)).status).toBe(404);
expect(list).not.toHaveBeenCalled();
});
it("rechecks current conversation access even for the owner", async () => {
mockAccessDecide.mockResolvedValue({
allowed: false, action: "issue:read", reason: "deny_issue_private", explanation: "Access revoked",
});
expect((await request(await createApp(owner)).get(path)).status).toBe(404);
expect(list).not.toHaveBeenCalled();
});
it("does not read conversation records across the company boundary", async () => {
mockIssueService.getById.mockResolvedValue(createIssue({
companyId: "company-2", conversationAgentId: ASSIGNEE_AGENT_ID, conversationUserId: owner.userId,
}));
expect((await request(await createApp(owner)).get(path)).status).toBe(404);
expect(list).not.toHaveBeenCalled();
});
it.each([
{ limit: 0 }, { limit: 101 }, { limit: 1.5 }, { limit: "not-a-number" },
{ cursor: "x".repeat(1001) }, { agentId: ASSIGNEE_AGENT_ID },
])("rejects unsupported or unbounded conversation query %j", async (query) => {
expect((await request(await createApp(owner)).get(path).query(query)).status).toBe(400);
expect(list).not.toHaveBeenCalled();
});
});
describe("exact document revision reads", () => {
const path = (key = "launch-post", revisionId = RUN_1) =>
`/api/issues/${ISSUE_ID}/documents/${encodeURIComponent(key)}/revisions/${encodeURIComponent(revisionId)}`;
const board = { type: "board", userId: "reviewer", companyIds: ["company-1"], source: "session" };
const agent = { type: "agent", agentId: ASSIGNEE_AGENT_ID, companyId: "company-1", runId: RUN_2 };
it.each([{ name: "board", actor: board }, { name: "agent", actor: agent }])(
"returns the requested immutable revision for an authorized $name",
async ({ actor }) => {
const revision = {
id: RUN_1, documentId: OTHER_ISSUE_ID, revision: 2,
title: "Launch draft", body: "The exact version that was reviewed.",
};
mockDocumentService.getIssueDocumentRevision.mockResolvedValue(revision);
const response = await request(await createApp(actor)).get(path(" Launch-Post "));
expect(response.status).toBe(200);
expect(response.body).toEqual(revision);
expect(mockAccessDecide).toHaveBeenCalledWith(expect.objectContaining({
actor, action: "issue:read", resource: expect.objectContaining({ issueId: ISSUE_ID }),
}));
expect(mockDocumentService.getIssueDocumentRevision).toHaveBeenCalledWith(ISSUE_ID, "launch-post", RUN_1);
expect(mockLogActivity).not.toHaveBeenCalled();
expect(mockHeartbeatService.wakeup).not.toHaveBeenCalled();
},
);
it.each([{ name: "board", actor: board }, { name: "agent", actor: agent }])(
"denies a $name without current issue read access before looking up the revision",
async ({ actor }) => {
mockAccessDecide.mockResolvedValue({ allowed: false, action: "issue:read", reason: "deny_issue_private", explanation: "Revoked" });
const response = await request(await createApp(actor)).get(path());
expect(response.status).toBe(404);
expect(mockDocumentService.getIssueDocumentRevision).not.toHaveBeenCalled();
},
);
it("denies a cross-company issue before looking up the revision", async () => {
mockIssueService.getById.mockResolvedValue(createIssue({ companyId: "company-2" }));
expect((await request(await createApp(board)).get(path())).status).toBe(404);
expect(mockDocumentService.getIssueDocumentRevision).not.toHaveBeenCalled();
});
it("returns not found for a missing issue without asking for its document", async () => {
mockIssueService.getById.mockResolvedValue(null);
expect((await request(await createApp(board)).get(path())).status).toBe(404);
expect(mockDocumentService.getIssueDocumentRevision).not.toHaveBeenCalled();
});
it("returns not found when the revision does not belong to the requested issue and key", async () => {
mockDocumentService.getIssueDocumentRevision.mockResolvedValue(null);
const response = await request(await createApp(board)).get(path());
expect(response.status).toBe(404);
expect(response.body).toEqual({ error: "Document revision not found" });
expect(mockDocumentService.getIssueDocumentRevision).toHaveBeenCalledWith(ISSUE_ID, "launch-post", RUN_1);
expect(mockLogActivity).not.toHaveBeenCalled();
});
it.each([
{ name: "invalid key characters", key: "draft.html", revisionId: RUN_1 },
{ name: "empty trimmed key", key: " ", revisionId: RUN_1 },
{ name: "overlong key", key: "a".repeat(65), revisionId: RUN_1 },
{ name: "invalid revision ID", key: "launch-post", revisionId: "latest" },
])("rejects $name before document lookup", async ({ key, revisionId }) => {
const response = await request(await createApp(board)).get(path(key, revisionId));
expect(response.status).toBe(400);
expect(response.body).toMatchObject({ error: "Invalid document revision reference" });
expect(mockDocumentService.getIssueDocumentRevision).not.toHaveBeenCalled();
});
});
it("creates board-authored interactions", async () => {
const app = await createApp();
const createRes = await request(app)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions")
.send({
kind: "suggest_tasks",
payload: {
version: 1,
tasks: [{ clientKey: "task-1", title: "One" }],
},
});
expect(createRes.status).toBe(201);
expect(mockInteractionService.create).toHaveBeenCalled();
expect(mockLogActivity).toHaveBeenCalledWith(
expect.anything(),
expect.objectContaining({
action: "issue.thread_interaction_created",
details: expect.objectContaining({
interactionId: "interaction-1",
interactionKind: "suggest_tasks",
}),
}),
);
}, 10_000);
it("records a board dismissal without resolving or waking, and denies agent dismissal", async () => {
mockInteractionService.dismissInteraction.mockResolvedValue({ id: "request-1", kind: "request_confirmation", status: "pending", dismissedAt: "2026-10-22T12:00:00Z" });
const path = "/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions/request-1/dismiss";
const result = await request(await createApp()).post(path).send({});
expect(result.status).toBe(200);
expect(result.body).toMatchObject({ status: "pending", dismissedAt: "2026-10-22T12:00:00Z" });
expect(mockInteractionService.dismissInteraction).toHaveBeenCalledWith(expect.objectContaining({ companyId: "company-1" }), "request-1", { userId: "local-board" });
expect(mockLogActivity).toHaveBeenCalledWith(expect.anything(), expect.objectContaining({ action: "issue.thread_interaction_dismissed" }));
expect(mockHeartbeatService.wakeup).not.toHaveBeenCalled();
mockInteractionService.dismissInteraction.mockClear();
expect((await request(await createApp({ type: "agent", agentId: CREATED_AGENT_ID, companyId: "company-1", runId: RUN_1 })).post(path).send({})).status).toBe(403);
expect(mockInteractionService.dismissInteraction).not.toHaveBeenCalled();
});
it("hides private task requests from same-company members without task access before dismissal", async () => {
mockAccessDecide.mockResolvedValue({ allowed: false, action: "issue:read", reason: "deny_issue_private",
explanation: "Task access was revoked.",
});
const app = await createApp({ type: "board", userId: "same-company-member", companyIds: ["company-1"],
source: "session", isInstanceAdmin: false,
});
const result = await request(app)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions/request-1/dismiss").send({});
expect(result.status).toBe(404);
expect(result.body).toEqual({ error: "Issue not found" });
expect(mockInteractionService.dismissInteraction).not.toHaveBeenCalled();
expect(mockInteractionService.getForIssue).not.toHaveBeenCalled();
expect(mockLogActivity).not.toHaveBeenCalled();
expect(mockHeartbeatService.wakeup).not.toHaveBeenCalled();
});
it("normalizes a canonical-only mixed question form at the HTTP boundary", async () => {
const questionSet = { schema: "paperclip.question_set.v1", questions: [
{ id: "repo", prompt: "Repository URL?", required: true, answerMode: "text" },
{ id: "scope", prompt: "Review scope?", required: true, answerMode: "single_select", options: [{ id: "all", label: "All changes" }, { id: "selected", label: "Selected changes" }] },
] };
const res = await request(await createApp())
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions")
.send({ kind: "ask_user_questions", payload: { version: 1, questionSet } });
expect(res.status).toBe(201);
expect(mockInteractionService.create).toHaveBeenCalledWith(expect.anything(), expect.objectContaining({
payload: expect.objectContaining({ questionSet, questions: [
expect.objectContaining({ id: "repo", options: [{ id: "paperclip_text_answer", label: "Type an answer", freeText: true }] }),
expect.objectContaining({ id: "scope", options: questionSet.questions[1].options }),
] }),
}), expect.anything());
});
it.each(["in_review", "done"])("keeps interaction reads free of expiry, activity writes, and recovery while %s", async (status) => {
mockIssueService.getById.mockResolvedValue(createIssue({
status,
assigneeAgentId: ASSIGNEE_AGENT_ID,
}));
mockInteractionService.listForIssue.mockResolvedValue([]);
await request(await createApp())
.get("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions")
.expect(200, []);
expect(mockInteractionService.listForIssue).toHaveBeenCalledExactlyOnceWith(ISSUE_ID);
expect(mockInteractionService.expireRequestConfirmationsSupersededByHistoricalComments).not.toHaveBeenCalled();
expect(mockInteractionService.expirePendingInteractionsForTerminalIssue).not.toHaveBeenCalled();
expect(mockLogActivity).not.toHaveBeenCalled();
expect(mockHeartbeatService.wakeup).not.toHaveBeenCalled();
});
it("wakes the addressed agent when an interaction is created", async () => {
mockInteractionService.create.mockResolvedValueOnce({
id: "interaction-addressed",
companyId: "company-1",
issueId: "aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa",
kind: "ask_user_questions",
status: "pending",
continuationPolicy: "wake_assignee",
addresseeAgentId: ASSIGNEE_AGENT_ID,
requestedResolverPolicy: "anyone",
effectiveResolverPolicy: "anyone",
idempotencyKey: null,
sourceCommentId: null,
sourceRunId: null,
payload: { version: 1, questions: [] },
result: null,
createdAt: "2026-07-25T12:00:00.000Z",
updatedAt: "2026-07-25T12:00:00.000Z",
});
const app = await createApp();
const res = await request(app)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions")
.send({
kind: "ask_user_questions",
addresseeAgentId: ASSIGNEE_AGENT_ID,
payload: {
version: 1,
questions: [{
id: "scope",
prompt: "Which scope?",
selectionMode: "single",
options: [{ id: "phase-1", label: "Phase 1" }],
}],
},
});
expect(res.status).toBe(201);
expect(mockHeartbeatService.wakeup).toHaveBeenCalledWith(
ASSIGNEE_AGENT_ID,
expect.objectContaining({
reason: "interaction_pending",
idempotencyKey: "interaction-pending:interaction-addressed",
payload: expect.objectContaining({
issueId: "aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa",
interactionId: "interaction-addressed",
}),
contextSnapshot: expect.objectContaining({ wakeReason: "interaction_pending" }),
}),
);
});
it("does not route agent attention for a human-only interaction", async () => {
mockInteractionService.create.mockResolvedValueOnce({
id: "interaction-human-attention",
companyId: "company-1",
issueId: "aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa",
kind: "ask_user_questions",
status: "pending",
continuationPolicy: "wake_assignee",
addresseeAgentId: ASSIGNEE_AGENT_ID,
requestedResolverPolicy: "human_only",
effectiveResolverPolicy: "human_only",
resolverPolicyProvenance: "explicit",
effectiveResolverPolicySource: "requested",
idempotencyKey: null,
sourceCommentId: null,
sourceRunId: null,
createdByAgentId: null,
payload: { version: 1, questions: [] },
result: null,
createdAt: "2026-07-25T12:00:00.000Z",
updatedAt: "2026-07-25T12:00:00.000Z",
});
const app = await createApp();
const res = await request(app)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions")
.send({
kind: "ask_user_questions",
addresseeAgentId: ASSIGNEE_AGENT_ID,
resolverPolicy: "human_only",
payload: {
version: 1,
questions: [{
id: "scope",
prompt: "Which scope?",
selectionMode: "single",
options: [{ id: "phase-1", label: "Phase 1" }],
}],
},
});
expect(res.status).toBe(201);
expect(mockHeartbeatService.wakeup).not.toHaveBeenCalled();
});
it("returns 400 for agent-addressed tool-action confirmations", async () => {
const app = await createApp();
const res = await request(app)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions")
.send({
kind: "request_confirmation",
addresseeAgentId: ASSIGNEE_AGENT_ID,
payload: {
version: 1,
prompt: "Run the tool?",
toolAction: {
version: 1,
actionRequestId: "11111111-1111-4111-8111-111111111111",
invocationId: "22222222-2222-4222-8222-222222222222",
toolName: "send_email",
toolDisplayName: "Send email",
connectionId: "33333333-3333-4333-8333-333333333333",
applicationId: "44444444-4444-4444-8444-444444444444",
appDisplayName: "Gmail",
risk: "write",
previewMarkdown: "Send an email to the reviewed recipient.",
argumentsSummaryJson: '{"to":"recipient@example.com"}',
argumentsHash: "reviewed-arguments-hash",
expiresAt: "2026-07-25T16:00:00.000Z",
},
},
});
expect(res.status).toBe(400);
expect(res.body.error).toContain("cannot be addressed");
expect(mockInteractionService.create).not.toHaveBeenCalled();
});
it("accepts suggested tasks and wakes created assignees plus the current assignee", async () => {
const app = await createApp();
const res = await request(app)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions/interaction-1/accept")
.send({ selectedClientKeys: ["task-1"] });
expect(res.status).toBe(200);
expect(mockInteractionService.acceptInteraction).toHaveBeenCalledWith(
expect.objectContaining({ id: "aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa" }),
"interaction-1",
{ selectedClientKeys: ["task-1"] },
expect.objectContaining({ userId: "local-board" }),
);
expect(mockHeartbeatService.wakeup).toHaveBeenCalledTimes(2);
expect(mockHeartbeatService.wakeup).toHaveBeenNthCalledWith(
1,
CREATED_AGENT_ID,
expect.objectContaining({
source: "assignment",
reason: "issue_assigned",
payload: expect.objectContaining({
issueId: "child-1",
mutation: "interaction_accept",
}),
}),
);
expect(mockHeartbeatService.wakeup).toHaveBeenNthCalledWith(
2,
ASSIGNEE_AGENT_ID,
expect.objectContaining({
source: "automation",
reason: "issue_commented",
payload: expect.objectContaining({
issueId: "aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa",
interactionId: "interaction-1",
interactionStatus: "accepted",
sourceCommentId: "comment-1",
sourceRunId: RUN_1,
}),
}),
);
});
it("answers questions through the durable delivery service", async () => {
const app = await createApp();
const res = await request(app)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions/interaction-2/respond")
.send({
answers: [{ questionId: "scope", optionIds: ["phase-1"] }],
});
expect(res.status).toBe(200);
expect(mockInteractionService.answerQuestions).toHaveBeenCalled();
expect(mockQuestionResponseDeliveries.deliver).toHaveBeenCalledWith(
"interaction-2",
);
expect(mockHeartbeatService.wakeup).not.toHaveBeenCalled();
expect(mockLogActivity).toHaveBeenCalledWith(
expect.anything(),
expect.objectContaining({
action: "issue.thread_interaction_answered",
}),
);
});
it.each(["paperclip-id:chat-owner", "chat-owner", "paperclip-id:another-user"])("keeps chat response delivery limited to the exact owner: %s", async (userId) => {
const issue = createIssue({ conversationAgentId: ASSIGNEE_AGENT_ID, conversationUserId: "paperclip-id:chat-owner" });
mockIssueService.getById.mockResolvedValue(issue);
mockInteractionService.getForIssue.mockResolvedValue({
id: "interaction-2", kind: "ask_user_questions", status: "pending",
requestedResolverPolicy: "human_only", effectiveResolverPolicy: "human_only",
addresseeUserId: "paperclip-id:chat-owner", continuationPolicy: "wake_assignee",
payload: { version: 1, questions: [] },
});
const app = await createApp({ type: "board", userId, source: "cloud_tenant", companyIds: ["company-1"] });
const response = await request(app).post(`/api/issues/${ISSUE_ID}/interactions/interaction-2/respond`)
.send({ answers: [{ questionId: "scope", optionIds: ["phase-1"] }] });
if (userId === "paperclip-id:chat-owner") {
expect(response.status).toBe(200);
expect(mockInteractionService.answerQuestions).toHaveBeenCalledWith(expect.anything(), "interaction-2", expect.anything(), expect.objectContaining({ userId }));
expect(mockQuestionResponseDeliveries.deliver).toHaveBeenCalledWith("interaction-2");
} else {
expect(response.status).toBe(403);
expect(mockInteractionService.answerQuestions).not.toHaveBeenCalled();
expect(mockQuestionResponseDeliveries.deliver).not.toHaveBeenCalled();
expect(mockHeartbeatService.wakeup).not.toHaveBeenCalled();
}
});
it("routes wake-on-accept question answers through the same causal delivery service", async () => {
mockInteractionService.answerQuestions.mockResolvedValueOnce({
id: "interaction-2",
companyId: "company-1",
issueId: "aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa",
kind: "ask_user_questions",
status: "answered",
continuationPolicy: "wake_assignee_on_accept",
sourceCommentId: "comment-2",
sourceRunId: RUN_2,
payload: { version: 1, questions: [] },
result: { version: 1, answers: [{ questionId: "scope", optionIds: ["phase-1"] }] },
});
const app = await createApp();
const res = await request(app)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions/interaction-2/respond")
.send({ answers: [{ questionId: "scope", optionIds: ["phase-1"] }] });
expect(res.status).toBe(200);
expect(mockQuestionResponseDeliveries.deliver).toHaveBeenCalledWith("interaction-2");
expect(mockHeartbeatService.wakeup).not.toHaveBeenCalled();
});
it("submits item verdicts and emits one continuation wake with resolved item ids", async () => {
const app = await createApp();
const res = await request(app)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions/interaction-verdicts/verdicts")
.send({
verdicts: [{ id: "docs", verdict: "reject", reason: "Missing examples" }],
});
expect(res.status).toBe(200);
expect(mockInteractionService.submitItemVerdicts).toHaveBeenCalledWith(
expect.objectContaining({ id: "aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa" }),
"interaction-verdicts",
{ verdicts: [{ id: "docs", verdict: "reject", reason: "Missing examples" }] },
expect.objectContaining({ userId: "local-board" }),
);
expect(mockHeartbeatService.wakeup).toHaveBeenCalledTimes(1);
expect(mockHeartbeatService.wakeup).toHaveBeenCalledWith(
ASSIGNEE_AGENT_ID,
expect.objectContaining({
reason: "issue_commented",
idempotencyKey: expect.stringMatching(
/^request_item_verdicts:aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa:interaction-verdicts:/,
),
payload: expect.objectContaining({
interactionId: "interaction-verdicts",
interactionKind: "request_item_verdicts",
interactionStatus: "pending",
sourceCommentId: "comment-verdicts",
sourceRunId: "run-verdicts",
newlyResolvedItemIds: ["docs"],
itemVerdicts: {
newlyResolvedItemIds: ["docs"],
coalesceWindowMs: 2000,
},
}),
contextSnapshot: expect.objectContaining({
interactionId: "interaction-verdicts",
interactionKind: "request_item_verdicts",
interactionStatus: "pending",
newlyResolvedItemIds: ["docs"],
itemVerdicts: {
newlyResolvedItemIds: ["docs"],
coalesceWindowMs: 2000,
},
}),
}),
);
expect(mockLogActivity).toHaveBeenCalledWith(
expect.anything(),
expect.objectContaining({
action: "issue.thread_interaction_item_verdicts_submitted",
details: expect.objectContaining({
interactionKind: "request_item_verdicts",
newlyResolvedItemCount: 1,
newlyResolvedItemIds: ["docs"],
complete: false,
}),
}),
);
});
it("allows a board user to withdraw and wakes the assignee", async () => {
const app = await createApp();
const res = await request(app)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions/interaction-withdraw/withdraw")
.send({ reason: "Replanning" });
expect(res.status).toBe(200);
expect(mockInteractionService.withdrawInteraction).toHaveBeenCalledWith(
expect.objectContaining({ id: "aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa" }),
"interaction-withdraw",
{ reason: "Replanning" },
expect.objectContaining({ userId: "local-board" }),
expect.objectContaining({ afterResolveInTransaction: expect.any(Function) }),
);
expect(mockHeartbeatService.wakeup).toHaveBeenCalledWith(ASSIGNEE_AGENT_ID, expect.objectContaining({
payload: expect.objectContaining({ interactionStatus: "cancelled" }),
}));
expect(mockLogActivity).toHaveBeenCalledWith(expect.anything(), expect.objectContaining({
action: "issue.thread_interaction_withdrawn",
}));
});
it("cancels the bound native run when its question is withdrawn", async () => {
mockInteractionService.withdrawInteraction.mockImplementationOnce((...args) => resolveMockInteraction(args, {
id: "interaction-withdraw",
companyId: "company-1",
issueId: ISSUE_ID,
kind: "ask_user_questions",
createdByAgentId: CREATED_AGENT_ID,
sourceRunId: RUN_1,
status: "cancelled",
continuationPolicy: "none",
payload: { version: 1, questions: [] },
result: { version: 1, answers: [], cancelled: true },
}));
mockRequestNativeQuestionRunCancellation.mockResolvedValueOnce(RUN_1);
const res = await request(await createApp())
.post(`/api/issues/${ISSUE_ID}/interactions/interaction-withdraw/withdraw`)
.send({ reason: "No longer needed" });
expect(res.status).toBe(200);
expect(mockRequestNativeQuestionRunCancellation).toHaveBeenCalledWith(
expect.anything(),
expect.objectContaining({ id: "interaction-withdraw", sourceRunId: RUN_1 }),
{ kind: "interaction_withdrawn", interactionId: "interaction-withdraw" },
);
expect(mockHeartbeatService.cancelRun).toHaveBeenCalledWith(
RUN_1,
"Question withdrawn while waiting for operator input",
expect.objectContaining({
resultJson: expect.objectContaining({
withdrawnInteractionId: "interaction-withdraw",
withdrawnByActorType: "user",
}),
}),
);
});
it("keeps a durable withdrawal intent when immediate native cancellation fails", async () => {
mockInteractionService.withdrawInteraction.mockImplementationOnce((...args) => resolveMockInteraction(args, {
id: "interaction-withdraw",
companyId: "company-1",
issueId: ISSUE_ID,
kind: "ask_user_questions",
createdByAgentId: CREATED_AGENT_ID,
sourceRunId: RUN_1,
status: "cancelled",
continuationPolicy: "none",
payload: { version: 1, questions: [] },
result: { version: 1, answers: [], cancelled: true },
}));
mockRequestNativeQuestionRunCancellation.mockResolvedValueOnce(RUN_1);
mockHeartbeatService.cancelRun.mockRejectedValueOnce(new Error("process unavailable"));
const res = await request(await createApp())
.post(`/api/issues/${ISSUE_ID}/interactions/interaction-withdraw/withdraw`)
.send({ reason: "No longer needed" });
expect(res.status).toBe(200);
expect(mockRequestNativeQuestionRunCancellation).toHaveBeenCalledWith(
expect.anything(),
expect.objectContaining({ id: "interaction-withdraw" }),
{ kind: "interaction_withdrawn", interactionId: "interaction-withdraw" },
);
});
it("allows the creator agent to withdraw and wakes a different assignee", async () => {
mockIssueService.getById.mockResolvedValueOnce(createIssue({ status: "in_review", reviewPolicy: null }));
mockInteractionService.getForIssue.mockResolvedValueOnce({
id: "interaction-withdraw",
kind: "request_confirmation",
status: "pending",
createdByAgentId: CREATED_AGENT_ID,
sourceRunId: RUN_1,
requestedResolverPolicy: "anyone",
effectiveResolverPolicy: "anyone",
payload: { version: 1, prompt: "Proceed?" },
});
const app = await createApp({ type: "agent", agentId: CREATED_AGENT_ID, companyId: "company-1", runId: RUN_1 });
const res = await request(app)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions/interaction-withdraw/withdraw")
.send({});
expect(res.status).toBe(200);
expect(mockHeartbeatService.wakeup).toHaveBeenCalledWith(ASSIGNEE_AGENT_ID, expect.anything());
});
it("allows the assignee agent to withdraw without waking itself", async () => {
mockIssueService.getById.mockResolvedValueOnce(createIssue({ status: "todo" }));
const app = await createApp({ type: "agent", agentId: ASSIGNEE_AGENT_ID, companyId: "company-1", runId: RUN_2 });
const res = await request(app)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions/interaction-withdraw/withdraw")
.send({});
expect(res.status).toBe(200);
expect(mockHeartbeatService.wakeup).not.toHaveBeenCalled();
});
it("rejects withdrawal by an unrelated agent", async () => {
const app = await createApp({ type: "agent", agentId: "33333333-3333-4333-8333-333333333333", companyId: "company-1", runId: RUN_3 });
const res = await request(app)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions/interaction-withdraw/withdraw")
.send({});
expect(res.status).toBe(403);
expect(mockInteractionService.withdrawInteraction).not.toHaveBeenCalled();
});
it("lets a watchdog-scoped assignee withdraw through ordinary containment", async () => {
mockResolveTaskWatchdogMutationScope.mockResolvedValue({
kind: "watchdog",
watchdogId: "watchdog-1",
companyId: "company-1",
watchedIssueId: "aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa",
watchdogIssueId: null,
stopFingerprint: "stop-1",
});
const app = await createApp({ type: "agent", agentId: ASSIGNEE_AGENT_ID, companyId: "company-1", runId: RUN_WATCHDOG });
const res = await request(app)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions/interaction-withdraw/withdraw")
.send({});
expect(mockInteractionService.withdrawInteraction).toHaveBeenCalledWith(
expect.anything(),
"interaction-withdraw",
{},
expect.objectContaining({ agentId: ASSIGNEE_AGENT_ID, runId: RUN_WATCHDOG }),
expect.objectContaining({ afterResolveInTransaction: expect.any(Function) }),
);
expect(res.status).toBe(200);
});
it("rejects withdrawal by low-trust actors", async () => {
mockResolveCoreTrustPreset.mockReturnValueOnce({ kind: "low_trust_review" });
const app = await createApp({ type: "agent", agentId: ASSIGNEE_AGENT_ID, companyId: "company-1", runId: RUN_LOW_TRUST });
const res = await request(app)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions/interaction-withdraw/withdraw")
.send({});
expect(res.status).toBe(403);
expect(res.body).toMatchObject({ code: "interaction_scope_denied" });
expect(mockInteractionService.withdrawInteraction).not.toHaveBeenCalled();
});
it("cancels question interactions and emits a continuation wake", async () => {
const app = await createApp();
const res = await request(app)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions/interaction-2/cancel")
.send({});
expect(res.status).toBe(200);
expect(res.body.status).toBe("cancelled");
expect(mockInteractionService.cancelQuestions).toHaveBeenCalledWith(
expect.objectContaining({ id: "aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa" }),
"interaction-2",
{},
expect.objectContaining({ userId: "local-board" }),
expect.objectContaining({ afterResolveInTransaction: expect.any(Function) }),
);
expect(mockHeartbeatService.wakeup).toHaveBeenCalledWith(
ASSIGNEE_AGENT_ID,
expect.objectContaining({
reason: "issue_commented",
payload: expect.objectContaining({
interactionId: "interaction-2",
interactionKind: "ask_user_questions",
interactionStatus: "cancelled",
sourceCommentId: "comment-2",
sourceRunId: RUN_2,
}),
}),
);
expect(mockLogActivity).toHaveBeenCalledWith(
expect.anything(),
expect.objectContaining({
action: "issue.thread_interaction_cancelled",
}),
);
});
it("durably marks a board-cancelled native question before cancelling its run", async () => {
mockRequestNativeQuestionRunCancellation.mockResolvedValueOnce(RUN_2);
const res = await request(await createApp())
.post(`/api/issues/${ISSUE_ID}/interactions/interaction-2/cancel`)
.send({});
expect(res.status).toBe(200);
expect(mockRequestNativeQuestionRunCancellation).toHaveBeenCalledWith(
expect.anything(),
expect.objectContaining({ id: "interaction-2", sourceRunId: RUN_2 }),
{ kind: "interaction_cancelled", interactionId: "interaction-2" },
);
expect(mockHeartbeatService.cancelRun).toHaveBeenCalledWith(
RUN_2,
"Cancelled while waiting for operator input",
expect.objectContaining({
resultJson: expect.objectContaining({ cancelledInteractionId: "interaction-2" }),
}),
);
});
it("accepts request confirmations and wakes the current assignee when configured for accept-only wakeups", async () => {
mockInteractionService.acceptInteraction.mockResolvedValueOnce({
interaction: {
id: "interaction-3",
companyId: "company-1",
issueId: "aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa",
kind: "request_confirmation",
status: "accepted",
continuationPolicy: "wake_assignee_on_accept",
idempotencyKey: null,
sourceCommentId: null,
sourceRunId: RUN_3,
payload: {
version: 1,
prompt: "Apply this plan?",
},
result: {
version: 1,
outcome: "accepted",
},
createdAt: "2026-04-20T12:00:00.000Z",
updatedAt: "2026-04-20T12:05:00.000Z",
resolvedAt: "2026-04-20T12:05:00.000Z",
},
createdIssues: [],
});
const app = await createApp();
const res = await request(app)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions/interaction-3/accept")
.send({});
expect(res.status).toBe(200);
expect(mockHeartbeatService.wakeup).toHaveBeenCalledTimes(1);
expect(mockHeartbeatService.wakeup).toHaveBeenCalledWith(
ASSIGNEE_AGENT_ID,
expect.objectContaining({
reason: "issue_commented",
payload: expect.objectContaining({
interactionId: "interaction-3",
interactionKind: "request_confirmation",
interactionStatus: "accepted",
}),
}),
);
expect(mockHeartbeatService.wakeup.mock.calls[0]?.[1]?.payload).not.toHaveProperty("toolAction");
expect(mockHeartbeatService.wakeup.mock.calls[0]?.[1]?.contextSnapshot).not.toHaveProperty("toolAction");
});
it("executes an accepted tool-action confirmation through the gateway callback", async () => {
const approveToolActionRequest = vi.fn().mockResolvedValue({
status: "executed",
resultSummary: "Added row 42",
});
mockInteractionService.acceptInteraction.mockResolvedValueOnce({
interaction: {
id: "interaction-tool-action",
companyId: "company-1",
issueId: "aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa",
kind: "request_confirmation",
status: "accepted",
continuationPolicy: "wake_assignee",
payload: {
version: 1,
prompt: "Approve the action?",
toolAction: {
version: 1,
actionRequestId: "action-request-1",
toolName: "google_sheets_add_row",
},
},
result: { version: 1, outcome: "accepted" },
},
createdIssues: [],
});
const app = await createApp(undefined, { approveToolActionRequest });
const res = await request(app)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions/interaction-tool-action/accept")
.send({});
expect(res.status).toBe(200);
expect(approveToolActionRequest).toHaveBeenCalledWith({
companyId: "company-1",
issueId: "aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa",
interactionId: "interaction-tool-action",
actionRequestId: "action-request-1",
actor: { agentId: null, userId: "local-board" },
});
const expectedToolAction = {
toolName: "google_sheets_add_row",
actionRequestId: "action-request-1",
decision: "accepted",
executionStatus: "executed",
resultSummary: "Added row 42",
instructions: "the approved google_sheets_add_row action already ran — do not call the tool again; continue with this result.",
};
expect(mockHeartbeatService.wakeup).toHaveBeenCalledWith(
ASSIGNEE_AGENT_ID,
expect.objectContaining({
payload: expect.objectContaining({ toolAction: expectedToolAction }),
contextSnapshot: expect.objectContaining({ toolAction: expectedToolAction }),
}),
);
});
it("wakes with failure instructions after an accepted tool action fails", async () => {
const approveToolActionRequest = vi.fn().mockResolvedValue({
status: "failed",
error: "Connector timed out",
});
mockInteractionService.acceptInteraction.mockResolvedValueOnce({
interaction: {
id: "interaction-tool-action-failed",
companyId: "company-1",
issueId: "aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa",
kind: "request_confirmation",
status: "accepted",
continuationPolicy: "wake_assignee",
payload: {
version: 1,
prompt: "Approve the action?",
toolAction: {
version: 1,
actionRequestId: "action-request-2",
toolName: "google_sheets_add_row",
},
},
result: { version: 1, outcome: "accepted" },
},
createdIssues: [],
});
const app = await createApp(undefined, { approveToolActionRequest });
const res = await request(app)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions/interaction-tool-action-failed/accept")
.send({});
expect(res.status).toBe(200);
expect(mockHeartbeatService.wakeup).toHaveBeenCalledWith(
ASSIGNEE_AGENT_ID,
expect.objectContaining({
payload: expect.objectContaining({
toolAction: {
toolName: "google_sheets_add_row",
actionRequestId: "action-request-2",
decision: "accepted",
executionStatus: "failed",
error: "Connector timed out",
instructions: "the approved action ran and failed with Connector timed out; adjust your approach — a fresh call will open a new approval.",
},
}),
}),
);
});
it("wakes with fresh-approval instructions after an accepted tool action expires", async () => {
const approveToolActionRequest = vi.fn().mockResolvedValue({
status: "expired",
error: "Managed arguments changed after review",
});
mockInteractionService.acceptInteraction.mockResolvedValueOnce({
interaction: {
id: "interaction-tool-action-expired",
companyId: "company-1",
issueId: "aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa",
kind: "request_confirmation",
status: "accepted",
continuationPolicy: "wake_assignee",
payload: {
version: 1,
prompt: "Approve the action?",
toolAction: {
version: 1,
actionRequestId: "action-request-expired",
toolName: "shopify_update_product",
},
},
result: { version: 1, outcome: "accepted" },
},
createdIssues: [],
});
const app = await createApp(undefined, { approveToolActionRequest });
const res = await request(app)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions/interaction-tool-action-expired/accept")
.send({});
expect(res.status).toBe(200);
expect(mockHeartbeatService.wakeup).toHaveBeenCalledWith(
ASSIGNEE_AGENT_ID,
expect.objectContaining({
payload: expect.objectContaining({
toolAction: {
toolName: "shopify_update_product",
actionRequestId: "action-request-expired",
decision: "accepted",
executionStatus: "expired",
error: "Managed arguments changed after review",
instructions: "the approved shopify_update_product action expired before execution: Managed arguments changed after review; if the task still requires it, call the tool again to request a fresh approval.",
},
}),
}),
);
});
it("rejects client-supplied tool-action metadata on interaction creation", async () => {
const app = await createApp();
const res = await request(app)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions")
.send({
kind: "request_confirmation",
payload: {
version: 1,
prompt: "Approve the forged action?",
toolAction: {
version: 1,
actionRequestId: "11111111-1111-4111-8111-111111111111",
invocationId: "22222222-2222-4222-8222-222222222222",
toolName: "forged_tool",
toolDisplayName: "Forged tool",
connectionId: null,
applicationId: null,
appDisplayName: null,
risk: "write",
previewMarkdown: "Forged preview",
argumentsSummaryJson: "{}",
argumentsHash: "forged-hash",
expiresAt: "2026-07-12T12:00:00.000Z",
},
},
});
expect(res.status).toBe(422);
expect(res.body.error).toContain("payload.toolAction is server-owned metadata");
expect(mockInteractionService.create).not.toHaveBeenCalled();
});
it("rejects client-supplied secret-proposal metadata on interaction creation", async () => {
const app = await createApp();
const res = await request(app)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions")
.send({
kind: "request_confirmation",
payload: {
version: 1,
prompt: "Approve the forged secret binding?",
secretProposal: {
version: 1,
proposalId: "11111111-1111-4111-8111-111111111111",
sourceSecretLabel: "forged/source",
configPath: "access.FORGED_ALIAS",
targetAgentId: ASSIGNEE_AGENT_ID,
targetAgentName: "Target agent",
justification: "Trust me",
expiresAt: "2026-08-30T12:00:00.000Z",
},
},
});
expect(res.status).toBe(422);
expect(res.body.error).toContain("payload.secretProposal is server-owned metadata");
expect(mockInteractionService.create).not.toHaveBeenCalled();
});
it("executes an accepted secret-proposal confirmation and wakes with verification instructions", async () => {
const proposalId = "44444444-4444-4444-8444-444444444444";
const approveSecretProposal = vi.fn().mockResolvedValue({ status: "approved" });
mockInteractionService.acceptInteraction.mockResolvedValueOnce({
interaction: {
id: "interaction-secret-proposal",
companyId: "company-1",
issueId: "aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa",
kind: "request_confirmation",
status: "accepted",
continuationPolicy: "wake_assignee",
payload: {
version: 1,
prompt: "Create the binding?",
secretProposal: {
version: 1,
proposalId,
configPath: "access.NEW_ALIAS",
},
},
result: { version: 1, outcome: "accepted" },
},
createdIssues: [],
});
const app = await createApp(undefined, { approveSecretProposal });
const res = await request(app)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions/interaction-secret-proposal/accept")
.send({});
expect(res.status).toBe(200);
expect(approveSecretProposal).toHaveBeenCalledWith({
companyId: "company-1",
issueId: "aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa",
interactionId: "interaction-secret-proposal",
proposalId,
actor: { agentId: null, userId: "local-board" },
});
expect(mockInteractionService.recordSecretProposalExecutionResult).toHaveBeenCalledWith(
expect.objectContaining({ id: "aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa" }),
"interaction-secret-proposal",
proposalId,
{ status: "executed" },
);
expect(mockHeartbeatService.wakeup).toHaveBeenCalledWith(
ASSIGNEE_AGENT_ID,
expect.objectContaining({
payload: expect.objectContaining({
secretProposal: expect.objectContaining({
proposalId,
configPath: "access.NEW_ALIAS",
executionStatus: "executed",
instructions: expect.stringContaining("GET /api/agents/me/secrets"),
}),
}),
}),
);
});
it("records a failed secret-proposal execution and posts a thread comment", async () => {
const proposalId = "55555555-5555-4555-8555-555555555555";
const approveSecretProposal = vi.fn().mockRejectedValue(new Error("binding failed"));
mockInteractionService.acceptInteraction.mockResolvedValueOnce({
interaction: {
id: "interaction-secret-proposal-failed",
companyId: "company-1",
issueId: "aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa",
kind: "request_confirmation",
status: "accepted",
continuationPolicy: "wake_assignee",
payload: {
version: 1,
prompt: "Create the binding?",
secretProposal: {
version: 1,
proposalId,
configPath: "access.NEW_ALIAS",
},
},
result: { version: 1, outcome: "accepted" },
},
createdIssues: [],
});
const app = await createApp(undefined, { approveSecretProposal });
const res = await request(app)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions/interaction-secret-proposal-failed/accept")
.send({});
expect(res.status).toBe(200);
expect(mockInteractionService.recordSecretProposalExecutionResult).toHaveBeenCalledWith(
expect.anything(),
"interaction-secret-proposal-failed",
proposalId,
{ status: "failed", errorCode: "secret_proposal_execution_failed" },
);
expect(mockIssueService.addComment).toHaveBeenCalledWith(
"aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa",
expect.stringContaining("Binding created: **no**"),
{ userId: "local-board" },
);
expect(res.body.result.secretProposal).toMatchObject({
status: "failed",
errorCode: "secret_proposal_execution_failed",
});
});
it("forwards plan-document confirmations to the interaction service for revision validation", async () => {
const app = await createApp();
const res = await request(app)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions")
.send({
kind: "request_confirmation",
payload: {
version: 1,
prompt: "Approve the plan?",
target: {
type: "issue_document",
issueId: "aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa",
documentId: "bbbbbbbb-bbbb-4bbb-8bbb-bbbbbbbbbbbb",
key: "plan",
revisionId: "cccccccc-cccc-4ccc-8ccc-cccccccccccc",
revisionNumber: 1,
},
},
});
// The route delegates plan-target validation to the service, which rejects a
// stale/missing revision atomically inside its insert transaction
// (assertRequestConfirmationTargetIsCurrent). The route must pass the target
// through unchanged rather than pre-checking it non-atomically.
expect(res.status).toBe(201);
expect(mockInteractionService.create).toHaveBeenCalledWith(
expect.objectContaining({ id: "aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa" }),
expect.objectContaining({
kind: "request_confirmation",
payload: expect.objectContaining({
target: expect.objectContaining({
key: "plan",
revisionId: "cccccccc-cccc-4ccc-8ccc-cccccccccccc",
}),
}),
}),
expect.anything(),
);
});
it("accepts request checkbox confirmations with selected option ids and wakes the assignee", async () => {
mockInteractionService.acceptInteraction.mockResolvedValueOnce({
interaction: {
id: "interaction-checkbox",
companyId: "company-1",
issueId: "aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa",
kind: "request_checkbox_confirmation",
status: "accepted",
continuationPolicy: "wake_assignee",
idempotencyKey: null,
sourceCommentId: null,
sourceRunId: "run-checkbox",
payload: {
version: 1,
prompt: "Delete selected files?",
options: [
{ id: "file-a", label: "a.txt" },
{ id: "file-b", label: "b.txt", description: "Generated build output" },
],
},
result: {
version: 1,
outcome: "accepted",
selectedOptionIds: ["file-b"],
},
createdAt: "2026-04-20T12:00:00.000Z",
updatedAt: "2026-04-20T12:05:00.000Z",
resolvedAt: "2026-04-20T12:05:00.000Z",
},
createdIssues: [],
});
const app = await createApp();
const res = await request(app)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions/interaction-checkbox/accept")
.send({ selectedOptionIds: ["file-b"] });
expect(res.status).toBe(200);
expect(mockInteractionService.acceptInteraction).toHaveBeenCalledWith(
expect.objectContaining({ id: "aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa" }),
"interaction-checkbox",
{ selectedOptionIds: ["file-b"] },
expect.objectContaining({ userId: "local-board" }),
);
expect(mockHeartbeatService.wakeup).toHaveBeenCalledTimes(1);
expect(mockHeartbeatService.wakeup).toHaveBeenCalledWith(
ASSIGNEE_AGENT_ID,
expect.objectContaining({
reason: "issue_commented",
payload: expect.objectContaining({
interactionId: "interaction-checkbox",
interactionKind: "request_checkbox_confirmation",
interactionStatus: "accepted",
checkboxSelection: {
prompt: "Delete selected files?",
selectedOptionIds: ["file-b"],
selectedOptions: [{ id: "file-b", label: "b.txt", description: "Generated build output" }],
},
}),
contextSnapshot: expect.objectContaining({
checkboxSelection: {
prompt: "Delete selected files?",
selectedOptionIds: ["file-b"],
selectedOptions: [{ id: "file-b", label: "b.txt", description: "Generated build output" }],
},
}),
}),
);
expect(mockLogActivity).toHaveBeenCalledWith(
expect.anything(),
expect.objectContaining({
action: "issue.thread_interaction_accepted",
details: expect.objectContaining({
interactionKind: "request_checkbox_confirmation",
interactionStatus: "accepted",
}),
}),
);
});
it("preserves accepted empty checkbox selections in assignee wake context", async () => {
mockInteractionService.acceptInteraction.mockResolvedValueOnce({
interaction: {
id: "interaction-checkbox-empty",
companyId: "company-1",
issueId: "aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa",
kind: "request_checkbox_confirmation",
status: "accepted",
continuationPolicy: "wake_assignee",
idempotencyKey: null,
sourceCommentId: null,
sourceRunId: "run-checkbox",
payload: {
version: 1,
prompt: "Delete selected files?",
options: [
{ id: "file-a", label: "a.txt", description: "Temporary export" },
{ id: "file-b", label: "b.txt", description: "Generated build output" },
],
},
result: {
version: 1,
outcome: "accepted",
selectedOptionIds: [],
},
createdAt: "2026-04-20T12:00:00.000Z",
updatedAt: "2026-04-20T12:05:00.000Z",
resolvedAt: "2026-04-20T12:05:00.000Z",
},
createdIssues: [],
});
const app = await createApp();
const res = await request(app)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions/interaction-checkbox-empty/accept")
.send({ selectedOptionIds: [] });
expect(res.status).toBe(200);
expect(mockHeartbeatService.wakeup).toHaveBeenCalledTimes(1);
expect(mockHeartbeatService.wakeup).toHaveBeenCalledWith(
ASSIGNEE_AGENT_ID,
expect.objectContaining({
payload: expect.objectContaining({
checkboxSelection: {
prompt: "Delete selected files?",
selectedOptionIds: [],
selectedOptions: [],
},
}),
contextSnapshot: expect.objectContaining({
checkboxSelection: {
prompt: "Delete selected files?",
selectedOptionIds: [],
selectedOptions: [],
},
}),
}),
);
});
it.each([
{ label: "explicit", targetIssueId: ISSUE_ID },
{ label: "omitted", targetIssueId: undefined },
{ label: "null", targetIssueId: null },
])("forces a fresh workspace-aware session when accepting a planning confirmation with $label issueId", async ({ targetIssueId }) => {
mockIssueService.getById.mockResolvedValueOnce(createIssue({ workMode: "planning" }));
mockInteractionService.acceptInteraction.mockResolvedValueOnce({
interaction: {
id: "interaction-plan",
companyId: "company-1",
issueId: "aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa",
kind: "request_confirmation",
status: "accepted",
continuationPolicy: "wake_assignee_on_accept",
idempotencyKey: "confirmation:issue:plan:revision",
sourceCommentId: null,
sourceRunId: "run-plan",
payload: {
version: 1,
prompt: "Approve this plan?",
target: {
type: "issue_document",
...(targetIssueId !== undefined ? { issueId: targetIssueId } : {}),
documentId: "document-plan",
key: "plan",
revisionId: "revision-plan",
revisionNumber: 1,
},
},
result: {
version: 1,
outcome: "accepted",
},
createdAt: "2026-04-20T12:00:00.000Z",
updatedAt: "2026-04-20T12:05:00.000Z",
resolvedAt: "2026-04-20T12:05:00.000Z",
},
createdIssues: [],
continuationIssue: {
id: "aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa",
assigneeAgentId: ASSIGNEE_AGENT_ID,
assigneeUserId: null,
status: "todo",
workMode: "standard",
},
});
const app = await createApp();
const res = await request(app)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions/interaction-plan/accept")
.send({});
expect(res.status).toBe(200);
expect(mockHeartbeatService.wakeup).toHaveBeenCalledTimes(1);
expect(mockHeartbeatService.wakeup).toHaveBeenCalledWith(
ASSIGNEE_AGENT_ID,
expect.objectContaining({
reason: "issue_commented",
contextSnapshot: expect.objectContaining({
issueId: "aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa",
interactionId: "interaction-plan",
interactionKind: "request_confirmation",
interactionStatus: "accepted",
planReviewInteraction: expect.objectContaining({
id: "interaction-plan",
kind: "request_confirmation",
status: "accepted",
acceptedTargetRevision: expect.objectContaining({
issueId: "aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa",
documentId: "document-plan",
key: "plan",
revisionId: "revision-plan",
revisionNumber: 1,
}),
result: expect.objectContaining({
outcome: "accepted",
}),
}),
forceFreshSession: true,
workspaceRefreshReason: "accepted_plan_confirmation",
}),
}),
);
expect(mockLogActivity).toHaveBeenCalledWith(
expect.anything(),
expect.objectContaining({
action: "issue.updated",
details: expect.objectContaining({
source: "request_confirmation_accept",
workMode: "standard",
_previous: expect.objectContaining({ workMode: "planning" }),
}),
}),
);
});
it("does not project an explicitly different issue's approved plan into the current issue wake", async () => {
mockInteractionService.acceptInteraction.mockResolvedValueOnce({
interaction: {
id: "interaction-other-plan",
companyId: "company-1",
issueId: ISSUE_ID,
kind: "request_confirmation",
status: "accepted",
continuationPolicy: "wake_assignee_on_accept",
sourceRunId: RUN_1,
payload: {
version: 1,
prompt: "Approve the other issue's plan?",
target: {
type: "issue_document",
issueId: OTHER_ISSUE_ID,
key: "plan",
revisionId: "other-revision",
revisionNumber: 2,
},
},
result: { version: 1, outcome: "accepted" },
},
createdIssues: [],
});
const response = await request(await createApp())
.post(`/api/issues/${ISSUE_ID}/interactions/interaction-other-plan/accept`)
.send({});
expect(response.status).toBe(200);
expect(mockHeartbeatService.wakeup).toHaveBeenCalledTimes(1);
const wake = mockHeartbeatService.wakeup.mock.calls[0]?.[1] as unknown as {
contextSnapshot: Record<string, unknown>;
payload: Record<string, unknown>;
};
expect(wake.contextSnapshot).not.toHaveProperty("planReviewInteraction");
expect(wake.payload).not.toHaveProperty("planReviewInteraction");
expect(wake.contextSnapshot).not.toHaveProperty("forceFreshSession");
});
it("forces a fresh workspace-aware session when accepting a plan document confirmation on a standard-work issue", async () => {
mockIssueService.getById.mockResolvedValueOnce(createIssue({ workMode: "standard" }));
mockInteractionService.acceptInteraction.mockResolvedValueOnce({
interaction: {
id: "interaction-standard-plan",
companyId: "company-1",
issueId: "aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa",
kind: "request_confirmation",
status: "accepted",
continuationPolicy: "wake_assignee_on_accept",
idempotencyKey: "confirmation:issue:plan:revision-standard",
sourceCommentId: null,
sourceRunId: "run-standard-plan",
payload: {
version: 1,
prompt: "Approve this plan?",
target: {
type: "issue_document",
issueId: "aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa",
documentId: "document-plan",
key: "plan",
revisionId: "revision-standard",
revisionNumber: 2,
},
},
result: {
version: 1,
outcome: "accepted",
},
createdAt: "2026-04-20T12:00:00.000Z",
updatedAt: "2026-04-20T12:05:00.000Z",
resolvedAt: "2026-04-20T12:05:00.000Z",
},
createdIssues: [],
});
const app = await createApp();
const res = await request(app)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions/interaction-standard-plan/accept")
.send({});
expect(res.status).toBe(200);
expect(mockHeartbeatService.wakeup).toHaveBeenCalledTimes(1);
expect(mockHeartbeatService.wakeup).toHaveBeenCalledWith(
ASSIGNEE_AGENT_ID,
expect.objectContaining({
reason: "issue_commented",
contextSnapshot: expect.objectContaining({
issueId: "aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa",
interactionId: "interaction-standard-plan",
interactionKind: "request_confirmation",
interactionStatus: "accepted",
forceFreshSession: true,
workspaceRefreshReason: "accepted_plan_confirmation",
}),
}),
);
});
it("wakes the returned agent when accepting an agent-authored confirmation from a board review assignee", async () => {
mockIssueService.getById.mockResolvedValueOnce(createIssue({
status: "in_review",
assigneeAgentId: null,
assigneeUserId: "local-board",
}));
mockInteractionService.acceptInteraction.mockResolvedValueOnce({
interaction: {
id: "interaction-4",
companyId: "company-1",
issueId: "aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa",
kind: "request_confirmation",
status: "accepted",
continuationPolicy: "wake_assignee_on_accept",
idempotencyKey: null,
sourceCommentId: null,
sourceRunId: "run-4",
payload: {
version: 1,
prompt: "Approve this plan?",
},
result: {
version: 1,
outcome: "accepted",
},
createdAt: "2026-04-20T12:00:00.000Z",
updatedAt: "2026-04-20T12:05:00.000Z",
resolvedAt: "2026-04-20T12:05:00.000Z",
},
createdIssues: [],
continuationIssue: {
id: "aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa",
assigneeAgentId: CREATED_AGENT_ID,
assigneeUserId: null,
status: "todo",
},
});
const app = await createApp();
const res = await request(app)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions/interaction-4/accept")
.send({});
expect(res.status).toBe(200);
expect(mockHeartbeatService.wakeup).toHaveBeenCalledTimes(1);
expect(mockHeartbeatService.wakeup).toHaveBeenCalledWith(
CREATED_AGENT_ID,
expect.objectContaining({
source: "automation",
reason: "issue_commented",
payload: expect.objectContaining({
issueId: "aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa",
interactionId: "interaction-4",
interactionKind: "request_confirmation",
interactionStatus: "accepted",
}),
}),
);
expect(mockLogActivity).toHaveBeenCalledWith(
expect.anything(),
expect.objectContaining({
action: "issue.updated",
details: expect.objectContaining({
source: "request_confirmation_accept",
assigneeAgentId: CREATED_AGENT_ID,
assigneeUserId: null,
_previous: expect.objectContaining({
assigneeUserId: "local-board",
}),
}),
}),
);
});
it("does not emit a continuation wake when request confirmations are rejected", async () => {
mockInteractionService.rejectInteraction.mockResolvedValueOnce({
id: "interaction-3",
companyId: "company-1",
issueId: "aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa",
kind: "request_confirmation",
status: "rejected",
continuationPolicy: "wake_assignee_on_accept",
idempotencyKey: null,
sourceCommentId: null,
sourceRunId: RUN_3,
payload: {
version: 1,
prompt: "Apply this plan?",
},
result: {
version: 1,
outcome: "rejected",
reason: "Needs changes",
},
createdAt: "2026-04-20T12:00:00.000Z",
updatedAt: "2026-04-20T12:05:00.000Z",
resolvedAt: "2026-04-20T12:05:00.000Z",
});
const app = await createApp();
const res = await request(app)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions/interaction-3/reject")
.send({ reason: "Needs changes" });
expect(res.status).toBe(200);
expect(mockHeartbeatService.wakeup).not.toHaveBeenCalled();
});
it("wakes the assignee to revise a rejected plan even when its policy is accept-only", async () => {
mockInteractionService.rejectInteraction.mockResolvedValueOnce({
id: "interaction-rejected-plan",
companyId: "company-1",
issueId: "aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa",
kind: "request_confirmation",
status: "rejected",
continuationPolicy: "wake_assignee_on_accept",
idempotencyKey: "confirmation:issue:plan:revision-2",
sourceCommentId: null,
sourceRunId: RUN_3,
payload: {
version: 1,
prompt: "Approve this plan?",
target: {
type: "issue_document",
issueId: "aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa",
documentId: "document-plan",
key: "plan",
revisionId: "revision-2",
revisionNumber: 2,
},
},
result: {
version: 1,
outcome: "rejected",
reason: "Keep the API smaller and add a Unicode test.",
},
createdAt: "2026-04-20T12:00:00.000Z",
updatedAt: "2026-04-20T12:05:00.000Z",
resolvedAt: "2026-04-20T12:05:00.000Z",
});
const res = await request(await createApp())
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions/interaction-rejected-plan/reject")
.send({ reason: "Keep the API smaller and add a Unicode test." });
expect(res.status).toBe(200);
expect(mockHeartbeatService.wakeup).toHaveBeenCalledWith(
ASSIGNEE_AGENT_ID,
expect.objectContaining({
payload: expect.objectContaining({
planReviewInteraction: expect.objectContaining({
id: "interaction-rejected-plan",
status: "rejected",
target: expect.objectContaining({
key: "plan",
revisionId: "revision-2",
}),
result: expect.objectContaining({
outcome: "rejected",
reason: "Keep the API smaller and add a Unicode test.",
}),
}),
}),
contextSnapshot: expect.objectContaining({
planReviewInteraction: expect.objectContaining({ status: "rejected" }),
}),
}),
);
});
it("delivers generic confirmation rejection feedback as the next turn message", async () => {
mockInteractionService.rejectInteraction.mockResolvedValueOnce({
id: "interaction-warm-turn",
companyId: "company-1",
issueId: "aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa",
kind: "request_confirmation",
status: "rejected",
continuationPolicy: "wake_assignee",
idempotencyKey: "warm-turn-1",
sourceCommentId: null,
sourceRunId: RUN_3,
payload: {
version: 1,
prompt: "Continue to turn two?",
target: {
type: "custom",
key: "warm_turn_1",
revisionId: "turn-1",
},
},
result: {
version: 1,
outcome: "rejected",
reason: "Read T1, append T2, and verify both lines.",
},
createdAt: "2026-04-20T12:00:00.000Z",
updatedAt: "2026-04-20T12:05:00.000Z",
resolvedAt: "2026-04-20T12:05:00.000Z",
});
const res = await request(await createApp())
.post(
"/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions/interaction-warm-turn/reject",
)
.send({ reason: "Read T1, append T2, and verify both lines." });
expect(res.status).toBe(200);
expect(mockHeartbeatService.wakeup).toHaveBeenCalledWith(
ASSIGNEE_AGENT_ID,
expect.objectContaining({
payload: expect.objectContaining({
paperclipAgentMessage: {
text: "Read T1, append T2, and verify both lines.",
source: "interaction_rejection",
sessionId: "interaction-warm-turn",
},
}),
contextSnapshot: expect.objectContaining({
paperclipAgentMessage: expect.objectContaining({
text: "Read T1, append T2, and verify both lines.",
}),
}),
}),
);
});
it("returns a rejected native completion review with a narrow reviewer-reason continuation", async () => {
const issue = createIssue({ status: "in_review" });
mockIssueService.getById.mockResolvedValue(issue);
mockInteractionService.rejectInteraction.mockResolvedValueOnce({
id: "interaction-native-completion-review",
companyId: "company-1",
issueId: issue.id,
kind: "request_confirmation",
status: "rejected",
continuationPolicy: "wake_assignee",
idempotencyKey: null,
sourceCommentId: null,
sourceRunId: RUN_3,
payload: {
version: 1,
prompt: "Approve completion?",
target: { type: "custom", key: "native_completion_review", revisionId: "decision-29" },
},
result: {
version: 1,
outcome: "rejected",
reason: "Run the external verification and report only that result.",
},
createdAt: "2026-04-20T12:00:00.000Z",
updatedAt: "2026-04-20T12:05:00.000Z",
resolvedAt: "2026-04-20T12:05:00.000Z",
});
const res = await request(await createApp())
.post(`/api/issues/${issue.id}/interactions/interaction-native-completion-review/reject`)
.send({ reason: "Run the external verification and report only that result." });
expect(res.status).toBe(200);
expect(mockHeartbeatService.wakeup).toHaveBeenCalledWith(
ASSIGNEE_AGENT_ID,
expect.objectContaining({
payload: expect.objectContaining({
nativeCompletionReview: expect.objectContaining({
decisionId: "decision-29",
outcome: "rejected",
reviewerReason: "Run the external verification and report only that result.",
instruction: expect.stringContaining("do not redo completed implementation"),
}),
}),
contextSnapshot: expect.objectContaining({
nativeCompletionReview: expect.objectContaining({ reviewerReason: expect.any(String) }),
}),
}),
);
});
it("overrides accept-only continuation when rejection consumes the last review path", async () => {
const issue = createIssue({ status: "in_review" });
mockIssueService.getById.mockResolvedValue(issue);
mockIssueService.listReviewAttention.mockResolvedValue(new Map([[
issue.id,
{ state: "stalled", paths: [], reason: "review path consumed" },
]]));
mockInteractionService.rejectInteraction.mockResolvedValueOnce({
id: "interaction-last-review-path",
companyId: "company-1",
issueId: issue.id,
kind: "request_confirmation",
status: "rejected",
continuationPolicy: "wake_assignee_on_accept",
idempotencyKey: null,
sourceCommentId: null,
sourceRunId: "run-last-review-path",
payload: { version: 1, prompt: "Approve this?" },
result: { version: 1, outcome: "rejected", reason: "Needs changes" },
createdAt: "2026-04-20T12:00:00.000Z",
updatedAt: "2026-04-20T12:05:00.000Z",
resolvedAt: "2026-04-20T12:05:00.000Z",
});
const res = await request(await createApp())
.post(`/api/issues/${issue.id}/interactions/interaction-last-review-path/reject`)
.send({ reason: "Needs changes" });
expect(res.status).toBe(200);
expect(mockHeartbeatService.wakeup).toHaveBeenCalledWith(
ASSIGNEE_AGENT_ID,
expect.objectContaining({
payload: expect.objectContaining({
reviewPathLost: true,
reviewPathConsumedRef: "interaction-last-review-path",
}),
contextSnapshot: expect.objectContaining({
reviewPathLost: true,
reviewPathInstruction: expect.stringContaining("Restore a reviewer"),
}),
}),
);
});
it("wakes with decline instructions when a tool-action confirmation is rejected", async () => {
mockInteractionService.rejectInteraction.mockResolvedValueOnce({
id: "interaction-tool-action-rejected",
companyId: "company-1",
issueId: "aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa",
kind: "request_confirmation",
status: "rejected",
continuationPolicy: "wake_assignee",
idempotencyKey: null,
sourceCommentId: null,
sourceRunId: "run-tool-action-rejected",
payload: {
version: 1,
prompt: "Approve the action?",
toolAction: {
version: 1,
actionRequestId: "action-request-3",
toolName: "google_sheets_add_row",
},
},
result: {
version: 1,
outcome: "rejected",
reason: "Use the sandbox sheet instead",
},
createdAt: "2026-04-20T12:00:00.000Z",
updatedAt: "2026-04-20T12:05:00.000Z",
resolvedAt: "2026-04-20T12:05:00.000Z",
});
const app = await createApp();
const res = await request(app)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions/interaction-tool-action-rejected/reject")
.send({ reason: "Use the sandbox sheet instead" });
expect(res.status).toBe(200);
const expectedToolAction = {
toolName: "google_sheets_add_row",
actionRequestId: "action-request-3",
decision: "rejected",
executionStatus: "rejected",
declineReason: "Use the sandbox sheet instead",
instructions: "the action was declined: Use the sandbox sheet instead; do not retry the same call — adjust your approach or mark the task blocked/in_review with the decline reason.",
};
expect(mockHeartbeatService.wakeup).toHaveBeenCalledWith(
ASSIGNEE_AGENT_ID,
expect.objectContaining({
payload: expect.objectContaining({ toolAction: expectedToolAction }),
contextSnapshot: expect.objectContaining({ toolAction: expectedToolAction }),
}),
);
});
it("does not emit an accept-only continuation wake for rejected suggested tasks", async () => {
mockInteractionService.rejectInteraction.mockResolvedValueOnce({
id: "interaction-5",
companyId: "company-1",
issueId: "aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa",
kind: "suggest_tasks",
status: "rejected",
continuationPolicy: "wake_assignee_on_accept",
idempotencyKey: null,
sourceCommentId: null,
sourceRunId: "run-5",
payload: {
version: 1,
tasks: [{ clientKey: "task-1", title: "One" }],
},
result: {
version: 1,
rejectionReason: "Not now",
},
createdAt: "2026-04-20T12:00:00.000Z",
updatedAt: "2026-04-20T12:05:00.000Z",
resolvedAt: "2026-04-20T12:05:00.000Z",
});
const app = await createApp();
const res = await request(app)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions/interaction-5/reject")
.send({ reason: "Not now" });
expect(res.status).toBe(200);
expect(mockHeartbeatService.wakeup).not.toHaveBeenCalled();
});
it("allows agent-authored interaction creation and stamps the active run id", async () => {
const app = await createApp({
type: "agent",
agentId: CREATED_AGENT_ID,
companyId: "company-1",
runId: RUN_1,
});
const res = await request(app)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions")
.send({
kind: "suggest_tasks",
idempotencyKey: "interaction:task-1",
payload: {
version: 1,
tasks: [{ clientKey: "task-1", title: "One" }],
},
});
expect(res.status).toBe(201);
expect(mockInteractionService.create).toHaveBeenCalledWith(
expect.objectContaining({ id: "aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa" }),
expect.objectContaining({
kind: "suggest_tasks",
idempotencyKey: "interaction:task-1",
sourceRunId: RUN_1,
}),
{
agentId: CREATED_AGENT_ID,
userId: null,
},
);
});
it("allows a different in-scope agent run to respond when policy permits", async () => {
mockIssueService.getById.mockResolvedValueOnce(createIssue({ status: "todo" }));
const app = await createApp({
type: "agent",
agentId: ASSIGNEE_AGENT_ID,
companyId: "company-1",
runId: RUN_2,
});
const res = await request(app)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions/interaction-2/respond")
.send({ answers: [{ questionId: "scope", optionIds: ["phase-1"] }] });
expect(res.status).toBe(200);
expect(mockInteractionService.answerQuestions).toHaveBeenCalledWith(
expect.anything(),
"interaction-2",
expect.anything(),
expect.objectContaining({ agentId: ASSIGNEE_AGENT_ID, runId: RUN_2, userId: null }),
);
expect(mockQuestionResponseDeliveries.deliver).toHaveBeenCalledWith("interaction-2");
expect(mockHeartbeatService.wakeup).not.toHaveBeenCalled();
expect(mockLogActivity).toHaveBeenCalledWith(expect.anything(), expect.objectContaining({
actorType: "agent",
agentId: ASSIGNEE_AGENT_ID,
runId: RUN_2,
details: expect.objectContaining({ resolutionActorKind: "agent" }),
}));
});
it("allows an agent to accept another agent's pending review confirmation by default", async () => {
mockReviewTransition.value = {
actorType: "agent",
actorId: CREATED_AGENT_ID,
details: { reviewInteractionId: "interaction-agent-review" },
};
mockIssueService.getById.mockResolvedValueOnce(createIssue({
status: "in_review",
reviewPolicy: null,
createdByAgentId: CREATED_AGENT_ID,
createdByUserId: null,
}));
mockInteractionService.getForIssue.mockResolvedValueOnce({
id: "interaction-agent-review",
kind: "request_confirmation",
status: "pending",
createdByAgentId: CREATED_AGENT_ID,
sourceRunId: RUN_1,
requestedResolverPolicy: "anyone",
effectiveResolverPolicy: "anyone",
payload: { version: 1, prompt: "Approve this review?" },
});
mockInteractionService.acceptInteraction.mockResolvedValueOnce({
interaction: {
id: "interaction-agent-review",
companyId: "company-1",
issueId: "aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa",
kind: "request_confirmation",
status: "accepted",
continuationPolicy: "none",
requestedResolverPolicy: "anyone",
effectiveResolverPolicy: "anyone",
payload: { version: 1, prompt: "Approve this review?" },
result: { version: 1, outcome: "accepted" },
},
createdIssues: [],
});
const app = await createApp({
type: "agent",
agentId: ASSIGNEE_AGENT_ID,
companyId: "company-1",
runId: RUN_2,
});
const res = await request(app)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions/interaction-agent-review/accept")
.send({});
expect(res.status).toBe(200);
expect(mockInteractionService.acceptInteraction).toHaveBeenCalledWith(
expect.anything(),
"interaction-agent-review",
{},
{
agentId: ASSIGNEE_AGENT_ID,
runId: RUN_2,
userId: null,
resolverPolicyRestriction: "anyone",
suggestedTaskEffectsAuthorized: true,
},
);
expect(mockLogActivity).toHaveBeenCalledWith(expect.anything(), expect.objectContaining({
actorType: "agent",
agentId: ASSIGNEE_AGENT_ID,
details: expect.objectContaining({ resolutionActorKind: "agent" }),
}));
});
it("allows an agent to reject a pending review confirmation by default", async () => {
mockReviewTransition.value = {
actorType: "agent",
actorId: CREATED_AGENT_ID,
details: { reviewInteractionId: "interaction-agent-reject" },
};
mockIssueService.getById.mockResolvedValueOnce(createIssue({
status: "in_review",
reviewPolicy: null,
createdByAgentId: CREATED_AGENT_ID,
createdByUserId: null,
}));
mockInteractionService.getForIssue.mockResolvedValueOnce({
id: "interaction-agent-reject",
kind: "request_confirmation",
status: "pending",
createdByAgentId: CREATED_AGENT_ID,
sourceRunId: RUN_1,
requestedResolverPolicy: "anyone",
effectiveResolverPolicy: "anyone",
payload: { version: 1, prompt: "Approve this review?" },
});
const app = await createApp({
type: "agent",
agentId: ASSIGNEE_AGENT_ID,
companyId: "company-1",
runId: RUN_2,
});
const res = await request(app)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions/interaction-agent-reject/reject")
.send({ reason: "Needs changes" });
expect(res.status).toBe(200);
expect(mockInteractionService.rejectInteraction).toHaveBeenCalledWith(
expect.anything(),
"interaction-agent-reject",
{ reason: "Needs changes" },
expect.objectContaining({
agentId: ASSIGNEE_AGENT_ID,
resolverPolicyRestriction: "anyone",
}),
);
});
it("keeps an unrelated pending confirmation human-only on an in-review issue", async () => {
mockReviewTransition.value = {
actorType: "agent",
actorId: CREATED_AGENT_ID,
details: { reviewInteractionId: "interaction-agent-review" },
};
mockIssueService.getById.mockResolvedValueOnce(createIssue({
status: "in_review",
reviewPolicy: null,
createdByAgentId: CREATED_AGENT_ID,
createdByUserId: null,
}));
mockInteractionService.getForIssue.mockResolvedValueOnce({
id: "interaction-unrelated-confirmation",
kind: "request_confirmation",
status: "pending",
createdByAgentId: UNRELATED_AGENT_ID,
sourceRunId: RUN_1,
requestedResolverPolicy: "board_only",
effectiveResolverPolicy: "board_only",
payload: { version: 1, prompt: "Approve an unrelated operation?" },
});
const app = await createApp({
type: "agent",
agentId: ASSIGNEE_AGENT_ID,
companyId: "company-1",
runId: RUN_2,
});
const res = await request(app)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions/interaction-unrelated-confirmation/accept")
.send({});
expect(res.status).toBe(403);
expect(res.body).toMatchObject({
error: "This issue-thread interaction is human-only",
code: "interaction_human_only",
});
expect(mockInteractionService.acceptInteraction).not.toHaveBeenCalled();
});
it("keeps a same-requester sibling confirmation human-only", async () => {
mockReviewTransition.value = {
actorType: "agent",
actorId: CREATED_AGENT_ID,
details: { reviewInteractionId: "interaction-agent-review" },
};
mockIssueService.getById.mockResolvedValueOnce(createIssue({
status: "in_review",
reviewPolicy: null,
createdByAgentId: CREATED_AGENT_ID,
createdByUserId: null,
}));
mockInteractionService.getForIssue.mockResolvedValueOnce({
id: "interaction-requester-sibling",
kind: "request_confirmation",
status: "pending",
createdByAgentId: CREATED_AGENT_ID,
sourceRunId: RUN_1,
requestedResolverPolicy: "board_only",
effectiveResolverPolicy: "board_only",
payload: { version: 1, prompt: "Approve a different operation?" },
});
const app = await createApp({
type: "agent",
agentId: ASSIGNEE_AGENT_ID,
companyId: "company-1",
runId: RUN_2,
});
const res = await request(app)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions/interaction-requester-sibling/accept")
.send({});
expect(res.status).toBe(403);
expect(res.body).toMatchObject({
error: "This issue-thread interaction is human-only",
code: "interaction_human_only",
});
expect(mockInteractionService.acceptInteraction).not.toHaveBeenCalled();
});
it.each([
{
name: "it addresses a different agent",
requesterAgentId: CREATED_AGENT_ID,
interaction: {
addresseeAgentId: "agent-other-reviewer",
createdByAgentId: CREATED_AGENT_ID,
sourceRunId: RUN_1,
},
code: "interaction_addressee_mismatch",
},
{
name: "the agent created it",
requesterAgentId: ASSIGNEE_AGENT_ID,
interaction: { createdByAgentId: ASSIGNEE_AGENT_ID, sourceRunId: RUN_1 },
code: "interaction_creator_excluded",
},
{
name: "the same run created it",
requesterAgentId: CREATED_AGENT_ID,
interaction: { createdByAgentId: CREATED_AGENT_ID, sourceRunId: RUN_2 },
code: "interaction_creator_excluded",
},
])("rejects an agent review verdict when $name", async ({ interaction, code, requesterAgentId }) => {
mockReviewTransition.value = {
actorType: "agent",
actorId: requesterAgentId,
details: { reviewInteractionId: "interaction-agent-review-scope" },
};
mockIssueService.getById.mockResolvedValueOnce(createIssue({
status: "in_review",
reviewPolicy: "not_creator",
createdByAgentId: requesterAgentId,
createdByUserId: null,
}));
mockInteractionService.getForIssue.mockResolvedValueOnce({
id: "interaction-agent-review-scope",
kind: "request_confirmation",
status: "pending",
requestedResolverPolicy: "anyone",
effectiveResolverPolicy: "anyone",
payload: { version: 1, prompt: "Approve this review?" },
...interaction,
});
const app = await createApp({
type: "agent",
agentId: ASSIGNEE_AGENT_ID,
companyId: "company-1",
runId: RUN_2,
});
const res = await request(app)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions/interaction-agent-review-scope/accept")
.send({});
expect(res.status).toBe(403);
expect(res.body).toMatchObject({ code });
expect(mockInteractionService.acceptInteraction).not.toHaveBeenCalled();
});
it("rejects an agent review-confirmation verdict under human_only with actionable copy", async () => {
mockReviewTransition.value = {
actorType: "agent",
actorId: CREATED_AGENT_ID,
details: { reviewInteractionId: "interaction-human-only" },
};
mockIssueService.getById.mockResolvedValueOnce(createIssue({
status: "in_review",
reviewPolicy: "human_only",
createdByAgentId: CREATED_AGENT_ID,
createdByUserId: null,
}));
mockInteractionService.getForIssue.mockResolvedValueOnce({
id: "interaction-human-only",
kind: "request_confirmation",
status: "pending",
createdByAgentId: CREATED_AGENT_ID,
sourceRunId: RUN_1,
requestedResolverPolicy: "anyone",
effectiveResolverPolicy: "anyone",
payload: { version: 1, prompt: "Approve this review?" },
});
const app = await createApp({
type: "agent",
agentId: ASSIGNEE_AGENT_ID,
companyId: "company-1",
runId: RUN_2,
});
const res = await request(app)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions/interaction-human-only/accept")
.send({});
expect(res.status).toBe(403);
expect(res.body).toMatchObject({
error: "This issue-thread interaction is human-only",
code: "interaction_human_only",
details: { code: "interaction_human_only", effectiveResolverPolicy: "human_only" },
});
expect(mockInteractionService.acceptInteraction).not.toHaveBeenCalled();
});
it.each([
{ action: "accept", body: {} },
{ action: "reject", body: { reason: "Needs changes" } },
])("rejects an agent $action verdict under human_only when a user review transition omitted the interaction binding", async ({ action, body }) => {
mockReviewTransition.value = {
actorType: "user",
actorId: "local-board",
details: { status: "in_review", _previous: { status: "in_progress" } },
};
mockIssueService.getById.mockResolvedValueOnce(createIssue({
status: "in_review",
reviewPolicy: "human_only",
createdByAgentId: null,
createdByUserId: "local-board",
}));
mockInteractionService.getForIssue.mockResolvedValueOnce({
id: "interaction-user-review-unbound",
kind: "request_confirmation",
status: "pending",
createdByAgentId: null,
createdByUserId: "local-board",
sourceRunId: null,
requestedResolverPolicy: "board_or_agents",
effectiveResolverPolicy: "board_or_agents",
payload: { version: 1, prompt: "Approve this review?" },
});
const app = await createApp({
type: "agent",
agentId: ASSIGNEE_AGENT_ID,
companyId: "company-1",
runId: RUN_2,
});
const res = await request(app)
.post(`/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions/interaction-user-review-unbound/${action}`)
.send(body);
expect(res.status).toBe(403);
expect(res.body).toMatchObject({
details: {
code: "review_policy_denied",
policy: "human_only",
},
});
expect(mockInteractionService.acceptInteraction).not.toHaveBeenCalled();
expect(mockInteractionService.rejectInteraction).not.toHaveBeenCalled();
});
it("allows an unrelated agent-resolvable confirmation under a restrictive issue review policy", async () => {
mockReviewTransition.value = {
actorType: "user",
actorId: "local-board",
details: { status: "in_review", _previous: { status: "in_progress" } },
};
mockIssueService.getById.mockResolvedValueOnce(createIssue({
status: "in_review",
reviewPolicy: "human_only",
createdByAgentId: null,
createdByUserId: "local-board",
}));
mockInteractionService.getForIssue.mockResolvedValueOnce({
id: "interaction-unrelated",
kind: "request_confirmation",
status: "pending",
createdByAgentId: UNRELATED_AGENT_ID,
createdByUserId: null,
sourceRunId: "run-1",
requestedResolverPolicy: "board_or_agents",
effectiveResolverPolicy: "board_or_agents",
payload: { version: 1, prompt: "Confirm an independent action?" },
});
mockInteractionService.acceptInteraction.mockResolvedValueOnce({
interaction: {
id: "interaction-unrelated",
companyId: "company-1",
issueId: "aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa",
kind: "request_confirmation",
status: "accepted",
continuationPolicy: "none",
idempotencyKey: null,
sourceCommentId: null,
sourceRunId: "run-1",
payload: { version: 1, prompt: "Confirm an independent action?" },
result: { version: 1, outcome: "accepted" },
createdAt: "2026-04-20T12:00:00.000Z",
updatedAt: "2026-04-20T12:05:00.000Z",
resolvedAt: "2026-04-20T12:05:00.000Z",
},
createdIssues: [],
});
const app = await createApp({
type: "agent",
agentId: ASSIGNEE_AGENT_ID,
companyId: "company-1",
runId: RUN_2,
});
const res = await request(app)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions/interaction-unrelated/accept")
.send({});
expect(res.status, JSON.stringify(res.body)).toBe(200);
expect(mockInteractionService.acceptInteraction).toHaveBeenCalled();
});
it("allows only the addressed agent or board to resolve an addressed interaction", async () => {
const addressed = {
id: "interaction-addressed",
kind: "ask_user_questions",
createdByAgentId: CREATED_AGENT_ID,
addresseeAgentId: ASSIGNEE_AGENT_ID,
sourceRunId: RUN_1,
requestedResolverPolicy: "board_or_agents",
effectiveResolverPolicy: "board_or_agents",
payload: { version: 1, questions: [] },
};
mockInteractionService.answerQuestions.mockImplementationOnce(async (_issue, interactionId) => ({
...addressed,
id: interactionId,
status: "answered",
result: { version: 1, answers: [] },
}));
mockInteractionService.getForIssue
.mockResolvedValueOnce(addressed)
.mockResolvedValueOnce(addressed)
.mockResolvedValueOnce(addressed);
mockIssueService.getById
.mockResolvedValueOnce(createIssue({ status: "todo" }))
.mockResolvedValueOnce(createIssue({ status: "todo" }))
.mockResolvedValueOnce(createIssue({ status: "todo" }));
const addresseeApp = await createApp({
type: "agent",
agentId: ASSIGNEE_AGENT_ID,
companyId: "company-1",
runId: RUN_2,
});
const addressee = await request(addresseeApp)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions/interaction-addressed/respond")
.send({ answers: [] });
expect(addressee.status).toBe(200);
expect(mockQuestionResponseDeliveries.deliver).toHaveBeenCalledWith("interaction-addressed");
expect(mockHeartbeatService.wakeup).not.toHaveBeenCalled();
const unrelatedApp = await createApp({
type: "agent",
agentId: UNRELATED_AGENT_ID,
companyId: "company-1",
runId: RUN_3,
});
const unrelated = await request(unrelatedApp)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions/interaction-addressed/respond")
.send({ answers: [] });
expect(unrelated.status).toBe(403);
expect(unrelated.body.error).toContain("addressed agent");
const boardApp = await createApp();
const board = await request(boardApp)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions/interaction-addressed/respond")
.send({ answers: [] });
expect(board.status).toBe(200);
});
it("preserves creator exclusion for legacy board_or_agents rows", async () => {
mockInteractionService.getForIssue.mockResolvedValueOnce({
id: "interaction-2",
kind: "ask_user_questions",
createdByAgentId: CREATED_AGENT_ID,
sourceRunId: RUN_1,
requestedResolverPolicy: "board_or_agents",
effectiveResolverPolicy: "board_or_agents",
resolverPolicyProvenance: "legacy_inherited_restriction",
payload: { version: 1, questions: [] },
});
mockIssueService.getById.mockResolvedValueOnce(createIssue({
status: "todo",
assigneeAgentId: CREATED_AGENT_ID,
}));
const app = await createApp({
type: "agent",
agentId: CREATED_AGENT_ID,
companyId: "company-1",
runId: RUN_9,
});
const res = await request(app)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions/interaction-2/respond")
.send({ answers: [] });
expect(res.status).toBe(403);
expect(res.body).toMatchObject({ code: "interaction_creator_excluded" });
expect(mockInteractionService.answerQuestions).not.toHaveBeenCalled();
});
it.each([
{
kind: "suggest_tasks",
action: "accept",
body: { selectedClientKeys: ["task-1"] },
payload: { version: 1, tasks: [{ clientKey: "task-1", title: "Task" }] },
service: "acceptInteraction",
},
{
kind: "ask_user_questions",
action: "respond",
body: { answers: [] },
payload: { version: 1, questions: [] },
service: "answerQuestions",
},
{
kind: "request_confirmation",
action: "accept",
body: {},
payload: { version: 1, prompt: "Continue?" },
service: "acceptInteraction",
},
{
kind: "request_checkbox_confirmation",
action: "accept",
body: { selectedOptionIds: [] },
payload: { version: 1, prompt: "Select", options: [] },
service: "acceptInteraction",
},
{
kind: "request_item_verdicts",
action: "verdicts",
body: { verdicts: [{ id: "item-1", verdict: "approve" }] },
payload: {
version: 1,
prompt: "Review",
items: [{ id: "item-1", label: "Item" }],
verdicts: ["approve", "reject"],
},
service: "submitItemVerdicts",
},
])("lets the creator and same run resolve $kind through anyone", async ({
kind,
action,
body,
payload,
service,
}) => {
mockInteractionService.getForIssue.mockResolvedValueOnce({
id: `interaction-${kind}`,
kind,
status: "pending",
createdByAgentId: CREATED_AGENT_ID,
sourceRunId: RUN_9,
requestedResolverPolicy: "anyone",
effectiveResolverPolicy: "anyone",
payload,
});
mockIssueService.getById.mockResolvedValueOnce(createIssue({
status: "todo",
assigneeAgentId: CREATED_AGENT_ID,
}));
const app = await createApp({
type: "agent",
agentId: CREATED_AGENT_ID,
companyId: "company-1",
runId: RUN_9,
});
const res = await request(app)
.post(`/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions/interaction-${kind}/${action}`)
.send(body);
expect(res.status).toBe(200);
expect(mockInteractionService[service as keyof typeof mockInteractionService]).toHaveBeenCalledWith(
expect.anything(),
`interaction-${kind}`,
body,
expect.objectContaining({ agentId: CREATED_AGENT_ID, runId: RUN_9 }),
);
});
it("enforces same-run exclusion under not_creator and requires a resolver run id", async () => {
mockInteractionService.getForIssue.mockResolvedValueOnce({
id: "interaction-2",
kind: "ask_user_questions",
createdByAgentId: CREATED_AGENT_ID,
sourceRunId: RUN_2,
requestedResolverPolicy: "not_creator",
effectiveResolverPolicy: "not_creator",
payload: { version: 1, questions: [] },
});
mockIssueService.getById.mockResolvedValueOnce(createIssue({ status: "todo" }));
const sameRunApp = await createApp({
type: "agent",
agentId: ASSIGNEE_AGENT_ID,
companyId: "company-1",
runId: RUN_2,
});
const sameRun = await request(sameRunApp)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions/interaction-2/respond")
.send({ answers: [] });
expect(sameRun.status).toBe(403);
expect(sameRun.body).toMatchObject({ code: "interaction_creator_excluded" });
mockIssueService.getById.mockResolvedValueOnce(createIssue({ status: "todo" }));
const missingRunApp = await createApp({
type: "agent",
agentId: ASSIGNEE_AGENT_ID,
companyId: "company-1",
});
const missingRun = await request(missingRunApp)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions/interaction-2/respond")
.send({ answers: [] });
expect(missingRun.status).toBe(422);
expect(missingRun.body).toMatchObject({ code: "interaction_run_attribution_required" });
});
it("rejects an invalid run attribution before resolving or waking", async () => {
mockInteractionService.getForIssue.mockResolvedValueOnce({
id: "interaction-invalid-run",
kind: "ask_user_questions",
status: "pending",
createdByAgentId: CREATED_AGENT_ID,
sourceRunId: RUN_1,
requestedResolverPolicy: "anyone",
effectiveResolverPolicy: "anyone",
payload: { version: 1, questions: [] },
});
const app = await createApp({
type: "agent",
agentId: ASSIGNEE_AGENT_ID,
companyId: "company-1",
runId: RUN_INVALID,
});
mockRunAttribution.value = null;
const res = await request(app)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions/interaction-invalid-run/respond")
.send({ answers: [] });
expect(res.status).toBe(422);
expect(res.body).toMatchObject({ code: "interaction_run_attribution_required" });
expect(mockInteractionService.answerQuestions).not.toHaveBeenCalled();
expect(mockHeartbeatService.wakeup).not.toHaveBeenCalled();
expect(mockLogActivity).not.toHaveBeenCalled();
});
it("hides cross-company interactions before resolver evaluation", async () => {
mockAccessDecide.mockResolvedValue({
allowed: false,
action: "issue:read",
reason: "deny_company_boundary",
explanation: "Denied by company boundary.",
});
const app = await createApp({
type: "agent",
agentId: ASSIGNEE_AGENT_ID,
companyId: "company-2",
runId: RUN_CROSS_COMPANY,
});
const res = await request(app)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions/interaction-2/respond")
.send({ answers: [] });
expect(res.status).toBe(404);
expect(mockInteractionService.getForIssue).not.toHaveBeenCalled();
expect(mockInteractionService.answerQuestions).not.toHaveBeenCalled();
});
it("contains task-bridge resolvers at the ordinary issue mutation boundary", async () => {
mockAccessDecide.mockResolvedValue({
allowed: false,
action: "issue:mutate",
reason: "deny_task_bridge_scope",
explanation: "Denied by task bridge scope.",
});
const app = await createApp({
type: "agent",
agentId: ASSIGNEE_AGENT_ID,
companyId: "company-1",
runId: RUN_TASK_BRIDGE,
source: "agent_key",
keyId: "bridge-key",
keyScope: { kind: "task_bridge" },
});
const res = await request(app)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions/interaction-2/respond")
.send({ answers: [] });
expect(res.status).toBe(403);
expect(res.body).toMatchObject({ code: "interaction_scope_denied" });
expect(mockInteractionService.answerQuestions).not.toHaveBeenCalled();
expect(mockHeartbeatService.wakeup).not.toHaveBeenCalled();
});
it("reauthorizes suggested-task effects before resolving the interaction", async () => {
mockInteractionService.getForIssue.mockResolvedValueOnce({
id: "interaction-suggest-assigned",
kind: "suggest_tasks",
status: "pending",
createdByAgentId: CREATED_AGENT_ID,
sourceRunId: RUN_1,
requestedResolverPolicy: "anyone",
effectiveResolverPolicy: "anyone",
payload: {
version: 1,
tasks: [{
clientKey: "assigned",
title: "Assigned child",
assigneeAgentId: UNRELATED_AGENT_ID,
}],
},
});
mockAccessDecide.mockImplementation(async (input: { action?: string }) => input.action === "tasks:assign"
? {
allowed: false,
action: input.action,
reason: "deny_assignment",
explanation: "Task assignment is not authorized.",
}
: {
allowed: true,
action: input.action,
reason: "allow_explicit_grant",
explanation: "Allowed by test grant.",
});
const app = await createApp({
type: "agent",
agentId: ASSIGNEE_AGENT_ID,
companyId: "company-1",
runId: RUN_2,
});
const res = await request(app)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions/interaction-suggest-assigned/accept")
.send({ selectedClientKeys: ["assigned"] });
expect(res.status).toBe(403);
expect(res.body).toMatchObject({ code: "interaction_governed_action_denied" });
expect(mockInteractionService.acceptInteraction).not.toHaveBeenCalled();
expect(mockHeartbeatService.wakeup).not.toHaveBeenCalled();
expect(mockLogActivity).not.toHaveBeenCalled();
});
describe("suggested-task effect authorization", () => {
const path = `/api/issues/${ISSUE_ID}/interactions/interaction-suggest/accept`;
const owner = { type: "board", userId: "local-board", companyIds: ["company-1"], source: "session" };
function suggest(task: Record<string, unknown> = {}, payload: Record<string, unknown> = {}) {
mockInteractionService.getForIssue.mockResolvedValue({
id: "interaction-suggest", kind: "suggest_tasks", status: "pending",
createdByAgentId: CREATED_AGENT_ID, sourceRunId: RUN_1,
requestedResolverPolicy: "anyone", effectiveResolverPolicy: "anyone",
payload: { version: 1, tasks: [{ clientKey: "task-1", title: "Write the launch post", ...task }], ...payload },
});
}
function denyAction(action: string) {
mockAccessDecide.mockImplementation(async (input: { action?: string }) => ({
allowed: input.action !== action, action: input.action,
reason: input.action === action ? "deny_explicit_policy" : "allow_explicit_grant",
explanation: "Test resource policy",
}));
}
function expectNoEffects() {
expect(mockInteractionService.acceptInteraction).not.toHaveBeenCalled();
expect(mockLogActivity).not.toHaveBeenCalled();
expect(mockHeartbeatService.wakeup).not.toHaveBeenCalled();
}
beforeEach(() => {
mockIssueService.getById.mockResolvedValue(createIssue({
conversationAgentId: ASSIGNEE_AGENT_ID, conversationUserId: owner.userId,
}));
suggest();
});
it.each([
{}, { assigneeAgentId: null }, { assigneeAgentId: CREATED_AGENT_ID }, { assigneeUserId: "reviewer" },
])("requires board assignment authorization for %j", async (assignment) => {
suggest(assignment);
denyAction("tasks:assign");
const response = await request(await createApp(owner)).post(path).send({ selectedClientKeys: ["task-1"] });
expect(response.status).toBe(403);
expect(response.body).toMatchObject({ code: "interaction_governed_action_denied" });
expect(mockAccessDecide).toHaveBeenCalledWith(expect.objectContaining({ action: "tasks:assign", actor: owner }));
expectNoEffects();
});
it("authorizes the conversation agent before accepting an ownerless proposal", async () => {
expect((await request(await createApp(owner)).post(path).send({})).status).toBe(200);
expect(mockAccessDecide).toHaveBeenCalledWith(expect.objectContaining({
action: "tasks:assign", resource: expect.objectContaining({ assigneeAgentId: ASSIGNEE_AGENT_ID, assigneeUserId: null }),
}));
});
it.each(["implicit", "task-parent", "default-parent"])("authorizes a standalone commitment for a conversation's %s parent", async (parentMode) => {
suggest({ assigneeAgentId: CREATED_AGENT_ID, ...(parentMode === "task-parent" ? { parentId: ISSUE_ID } : {}) },
parentMode === "default-parent" ? { defaultParentId: ISSUE_ID } : {});
const response = await request(await createApp(owner)).post(path).send({ selectedClientKeys: ["task-1"] });
expect(response.status).toBe(200);
expect(mockAccessDecide).toHaveBeenCalledWith(expect.objectContaining({
action: "tasks:assign", resource: expect.objectContaining({ parentIssueId: null, assigneeAgentId: CREATED_AGENT_ID }),
scope: expect.objectContaining({ parentIssueId: null }),
}));
expect(mockInteractionService.acceptInteraction).toHaveBeenCalledWith(
expect.anything(), "interaction-suggest", expect.anything(), expect.objectContaining({ suggestedTaskEffectsAuthorized: true }),
);
});
it("retains a real source task as the implicit parent", async () => {
mockIssueService.getById.mockResolvedValue(createIssue());
suggest({ assigneeAgentId: CREATED_AGENT_ID });
expect((await request(await createApp(owner)).post(path).send({})).status).toBe(200);
expect(mockAccessDecide).toHaveBeenCalledWith(expect.objectContaining({
action: "tasks:assign", scope: expect.objectContaining({ parentIssueId: ISSUE_ID }),
}));
});
it.each([false, true])("authorizes inherited proposal projects independently of array order (child first: %s)", async (childFirst) => {
const projectId = "cccccccc-cccc-4ccc-8ccc-cccccccccccc";
const tasks = [
{ clientKey: "root", title: "Launch", projectId },
{ clientKey: "child", parentClientKey: "root", title: "Write copy" },
{ clientKey: "grandchild", parentClientKey: "child", title: "Review copy" },
];
suggest({}, { tasks: childFirst ? tasks.toReversed() : tasks });
mockAccessDecide.mockImplementation(async (input: { action?: string; resource?: { projectId?: string } }) => ({
allowed: input.action !== "tasks:assign" || input.resource?.projectId === projectId,
action: input.action, reason: "project_scoped_grant", explanation: "Only project A assignments are permitted",
}));
expect((await request(await createApp(owner)).post(path).send({})).status).toBe(200);
const assignments = mockAccessDecide.mock.calls.filter(([input]) => input.action === "tasks:assign");
expect(assignments).toHaveLength(3);
expect(assignments.every(([input]) => input.resource.projectId === projectId)).toBe(true);
expect(mockInteractionService.acceptInteraction).toHaveBeenCalledOnce();
});
it("rejects a nested proposal's explicit project outside the assignment grant", async () => {
const allowedProject = "cccccccc-cccc-4ccc-8ccc-cccccccccccc";
suggest({}, { tasks: [
{ clientKey: "root", title: "Launch", projectId: allowedProject },
{ clientKey: "child", parentClientKey: "root", title: "Write copy", projectId: OTHER_ISSUE_ID },
] });
mockAccessDecide.mockImplementation(async (input: { action?: string; resource?: { projectId?: string } }) => ({
allowed: input.action !== "tasks:assign" || input.resource?.projectId === allowedProject,
action: input.action, reason: "project_scoped_grant", explanation: "Only project A assignments are permitted",
}));
expect((await request(await createApp(owner)).post(path).send({})).status).toBe(403);
expectNoEffects();
});
it("checks the selected subset without authorizing an unselected assignment", async () => {
suggest({}, { tasks: [
{ clientKey: "task-1", title: "Unassigned draft" },
{ clientKey: "task-2", title: "Restricted assignment", assigneeAgentId: CREATED_AGENT_ID },
] });
mockAccessDecide.mockImplementation(async (input: { action?: string; resource?: { assigneeAgentId?: string } }) => ({
allowed: input.action !== "tasks:assign" || input.resource?.assigneeAgentId === ASSIGNEE_AGENT_ID,
action: input.action, reason: "test_policy", explanation: "Only the conversation agent may be assigned",
}));
expect((await request(await createApp(owner)).post(path).send({ selectedClientKeys: ["task-1"] })).status).toBe(200);
expect(mockAccessDecide.mock.calls.filter(([input]) => input.action === "tasks:assign").map(([input]) => input.resource.assigneeAgentId)).toEqual([ASSIGNEE_AGENT_ID]);
});
it("denies an inaccessible explicit parent before creating an unassigned task", async () => {
const conversation = createIssue({ conversationAgentId: ASSIGNEE_AGENT_ID, conversationUserId: owner.userId });
mockIssueService.getById.mockImplementation(async (id: string) => id === OTHER_ISSUE_ID
? createIssue({ id: OTHER_ISSUE_ID }) : conversation);
suggest({ parentId: OTHER_ISSUE_ID });
mockAccessDecide.mockImplementation(async (input: { action?: string; resource?: { issueId?: string } }) => ({
allowed: !(input.action === "issue:read" && input.resource?.issueId === OTHER_ISSUE_ID),
action: input.action, reason: "deny_issue_private", explanation: "Parent is private",
}));
expect((await request(await createApp(owner)).post(path).send({})).status).toBe(404);
expectNoEffects();
});
it.each(["denied", "missing", "other-company"])("denies a %s project before creating an unassigned task", async (condition) => {
suggest({ projectId: OTHER_ISSUE_ID });
if (condition === "denied") denyAction("project:read");
if (condition === "missing") mockProjectService.getById.mockResolvedValue(null);
if (condition === "other-company") mockProjectService.getById.mockResolvedValue({ id: OTHER_ISSUE_ID, companyId: "company-2" });
const response = await request(await createApp(owner)).post(path).send({});
expect(response.status).toBe(403);
expect(response.body).toMatchObject({ code: "interaction_governed_action_denied" });
expectNoEffects();
});
it("authorizes the source project that creation retains when an explicit parent has another project", async () => {
const sourceProject = "cccccccc-cccc-4ccc-8ccc-cccccccccccc";
const parentProject = "dddddddd-dddd-4ddd-8ddd-dddddddddddd";
mockIssueService.getById.mockImplementation(async (id: string) => createIssue(id === OTHER_ISSUE_ID
? { id: OTHER_ISSUE_ID, projectId: parentProject }
: { projectId: sourceProject }));
suggest({ parentId: OTHER_ISSUE_ID, assigneeAgentId: CREATED_AGENT_ID });
expect((await request(await createApp(owner)).post(path).send({})).status).toBe(200);
expect(mockAccessDecide).toHaveBeenCalledWith(expect.objectContaining({
action: "project:read", resource: expect.objectContaining({ projectId: sourceProject }),
}));
expect(mockAccessDecide).toHaveBeenCalledWith(expect.objectContaining({
action: "tasks:assign", scope: expect.objectContaining({ projectId: sourceProject, parentIssueId: OTHER_ISSUE_ID }),
}));
});
it("keeps agents subject to assignment authorization even for an unassigned task", async () => {
mockIssueService.getById.mockResolvedValue(createIssue({ status: "todo" }));
denyAction("tasks:assign");
const actor = { type: "agent", agentId: ASSIGNEE_AGENT_ID, companyId: "company-1", runId: RUN_2 };
const response = await request(await createApp(actor)).post(path).send({});
expect(response.status).toBe(403);
expect(mockAccessDecide).toHaveBeenCalledWith(expect.objectContaining({ action: "tasks:assign", actor }));
expectNoEffects();
});
it("does not let a watchdog create an unrelated standalone task from a conversation", async () => {
mockResolveTaskWatchdogMutationScope.mockResolvedValue({
kind: "watchdog", watchdogId: "watchdog-1", companyId: "company-1",
watchedIssueId: ISSUE_ID, watchdogIssueId: null, stopFingerprint: "stop-1",
});
const response = await request(await createApp({
type: "agent", agentId: ASSIGNEE_AGENT_ID, companyId: "company-1", runId: RUN_2,
})).post(path).send({});
expect(response.status).toBe(403);
expectNoEffects();
});
});
it("blocks human-only and tool-action interactions for agents", async () => {
mockInteractionService.getForIssue
.mockResolvedValueOnce({
id: "interaction-1",
kind: "request_confirmation",
createdByAgentId: CREATED_AGENT_ID,
sourceRunId: RUN_1,
requestedResolverPolicy: "board_or_agents",
effectiveResolverPolicy: "board_only",
payload: { version: 1, prompt: "Proceed?" },
})
.mockResolvedValueOnce({
id: "interaction-tool",
kind: "request_confirmation",
createdByAgentId: CREATED_AGENT_ID,
sourceRunId: RUN_1,
requestedResolverPolicy: "board_or_agents",
effectiveResolverPolicy: "board_or_agents",
payload: { version: 1, prompt: "Run?", toolAction: { actionRequestId: "action-1" } },
});
mockIssueService.getById
.mockResolvedValueOnce(createIssue({ status: "todo" }))
.mockResolvedValueOnce(createIssue({ status: "todo" }));
const app = await createApp({
type: "agent",
agentId: ASSIGNEE_AGENT_ID,
companyId: "company-1",
runId: RUN_2,
});
const capped = await request(app)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions/interaction-1/accept")
.send({});
expect(capped.status).toBe(403);
expect(capped.body.error).toContain("human-only");
const toolAction = await request(app)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions/interaction-tool/accept")
.send({});
expect(toolAction.status).toBe(403);
expect(toolAction.body).toMatchObject({ code: "interaction_governed_action_denied" });
});
it("lets watchdog-scoped agents use the ordinary resolver and contains low-trust agents", async () => {
mockIssueService.getById.mockResolvedValue(createIssue({ status: "todo" }));
const actor = {
type: "agent",
agentId: ASSIGNEE_AGENT_ID,
companyId: "company-1",
runId: RUN_2,
};
mockResolveTaskWatchdogMutationScope.mockResolvedValueOnce({
kind: "watchdog",
watchdogId: "watchdog-1",
companyId: "company-1",
watchedIssueId: "aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa",
watchdogIssueId: null,
stopFingerprint: "stop-1",
});
const watchdogApp = await createApp(actor);
const watchdog = await request(watchdogApp)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions/interaction-2/respond")
.send({ answers: [] });
expect(watchdog.status).toBe(200);
expect(mockInteractionService.answerQuestions).toHaveBeenCalledTimes(1);
mockResolveCoreTrustPreset.mockReturnValueOnce({ kind: "low_trust_review" });
const lowTrustApp = await createApp(actor);
const lowTrust = await request(lowTrustApp)
.post("/api/issues/aaaaaaaa-aaaa-4aaa-8aaa-aaaaaaaaaaaa/interactions/interaction-2/respond")
.send({ answers: [] });
expect(lowTrust.status).toBe(403);
expect(lowTrust.body).toMatchObject({ code: "interaction_scope_denied" });
expect(mockInteractionService.answerQuestions).toHaveBeenCalledTimes(1);
});
const CROSS_ISSUE_RESOLUTION_ROUTES = [
{
route: "accept",
kind: "request_confirmation",
body: {},
payload: { version: 1, prompt: "Proceed?" },
service: "acceptInteraction",
},
{
route: "reject",
kind: "request_confirmation",
body: { reason: "Not now" },
payload: { version: 1, prompt: "Proceed?" },
service: "rejectInteraction",
},
{
route: "respond",
kind: "ask_user_questions",
body: { answers: [] },
payload: { version: 1, questions: [] },
service: "answerQuestions",
},
{
route: "verdicts",
kind: "request_item_verdicts",
body: { verdicts: [{ id: "item-1", verdict: "approve" }] },
payload: {
version: 1,
prompt: "Review",
items: [{ id: "item-1", label: "Item" }],
verdicts: ["approve", "reject"],
},
service: "submitItemVerdicts",
},
] as const;
function stubCrossIssueResolution(
{ kind, payload }: { kind: string; payload: Record<string, unknown> },
) {
// The run authenticated below is working OTHER_ISSUE_ID, so resolving this
// card is a cross-issue mutation even though the audience is open.
mockIssueService.getById.mockResolvedValueOnce(createIssue({ status: "todo" }));
mockInteractionService.getForIssue.mockResolvedValueOnce({
id: `interaction-cross-${kind}`,
kind,
status: "pending",
createdByAgentId: CREATED_AGENT_ID,
sourceRunId: RUN_1,
requestedResolverPolicy: "anyone",
effectiveResolverPolicy: "anyone",
payload,
});
}
it.each(CROSS_ISSUE_RESOLUTION_ROUTES)(
"charges one shared cross-issue observation for an allowed $route",
async ({ route, kind, body, payload, service }) => {
stubCrossIssueResolution({ kind, payload });
mockCrossIssueInfluence.sourceIssueId = OTHER_ISSUE_ID;
const app = await createApp({
type: "agent",
agentId: ASSIGNEE_AGENT_ID,
companyId: "company-1",
runId: RUN_2,
});
const res = await request(app)
.post(`/api/issues/${ISSUE_ID}/interactions/interaction-cross-${kind}/${route}`)
.send(body);
expect(res.status).toBe(200);
expect(mockInteractionService[service]).toHaveBeenCalledTimes(1);
expect(mockCrossIssueInfluence.inserted).toEqual([
expect.objectContaining({
action: "issue.cross_issue_influence_observed",
entityId: ISSUE_ID,
details: expect.objectContaining({
kind: "interaction_resolution",
sourceIssueId: OTHER_ISSUE_ID,
targetIssueId: ISSUE_ID,
count: 1,
}),
}),
]);
},
);
});