mosaicstack/stack
1
0
Fork 0
Code Issues 42 Pull Requests Actions Packages Projects Releases 15 Wiki Activity

fix(tess): route bare Discord approvals #733

Merged
jason.woltje merged 1 commits from feat/tess-m3-integration into main 2026-07-13 10:29:11 +00:00
Conversation 1 Commits 1 Files Changed 2 +43 -10
jason.woltje commented 2026-07-13 10:11:59 +00:00
Owner
Copy Link

Summary

  • route the gateway-accepted bare Discord /approve command to discord:approve
  • exercise the production DiscordPlugin.handleDiscordMessage ingress path through durable approval and stop

Verification

  • TURBO_FORCE=true pnpm typecheck
  • TURBO_FORCE=true pnpm lint
  • TURBO_FORCE=true pnpm format:check
  • targeted cross-surface and Discord ingress suite: 17 tests passed

Follow-up to merged #732.

## Summary - route the gateway-accepted bare Discord `/approve` command to `discord:approve` - exercise the production `DiscordPlugin.handleDiscordMessage` ingress path through durable approval and stop ## Verification - `TURBO_FORCE=true pnpm typecheck` - `TURBO_FORCE=true pnpm lint` - `TURBO_FORCE=true pnpm format:check` - targeted cross-surface and Discord ingress suite: 17 tests passed Follow-up to merged #732.
jason.woltje added 2 commits 2026-07-13 10:12:00 +00:00
feat(tess): wire durable interaction surfaces
All checks were successful
ci/woodpecker/pr/ci Pipeline was successful
Details
451f7e04ec
fix(tess): route bare Discord approvals
Some checks failed
ci/woodpecker/pr/ci Pipeline was canceled
Details
a02f526d0a
jason.woltje force-pushed feat/tess-m3-integration from a02f526d0a to ed1d985c90 2026-07-13 10:12:53 +00:00 Compare
jason.woltje commented 2026-07-13 10:17:53 +00:00
Author
Owner
Copy Link

VERIFIED APPROVE reviewer-of-record [W-jarvis:reviewer-gpt5.5] head ed1d985c90

Findings: command-authorization.service.ts is byte-identical to origin/main (hash a9f829e7ecec721c6c585fe0da56fb5d6a6441e9), preserving the 7-field runtime action digest. Bare /approve is now routed through the real Discord plugin path and the gateway resolves provider/runtime session from getSnapshot(conversationId), not raw message text.
Anti-masking checks pass: the integration test uses real plugin routing with bare /approve, durable conversation handle Nova:discord:channel-1, and runtime session runtime-1; no hardcoded Tess literal or live credentials were introduced. CI is currently pending; Mos merges on green.

Marker: ROR-733-APPROVE-ed1d985c

VERIFIED APPROVE reviewer-of-record [W-jarvis:reviewer-gpt5.5] head ed1d985c90e24ca3046bd570a556802927d80ed8 Findings: `command-authorization.service.ts` is byte-identical to `origin/main` (hash `a9f829e7ecec721c6c585fe0da56fb5d6a6441e9`), preserving the 7-field runtime action digest. Bare `/approve` is now routed through the real Discord plugin path and the gateway resolves provider/runtime session from `getSnapshot(conversationId)`, not raw message text. Anti-masking checks pass: the integration test uses real plugin routing with bare `/approve`, durable conversation handle `Nova:discord:channel-1`, and runtime session `runtime-1`; no hardcoded `Tess` literal or live credentials were introduced. CI is currently pending; Mos merges on green. Marker: ROR-733-APPROVE-ed1d985c
jason.woltje merged commit f1c6b37b46 into main 2026-07-13 10:29:11 +00:00
jason.woltje deleted branch feat/tess-m3-integration 2026-07-13 10:29:12 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
fleet-enhancement
Fleet-wide reliability/speed/cost improvements surfaced by the nightly enhance pass
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
jason.woltje
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: mosaicstack/stack#733
Delete Branch "feat/tess-m3-integration"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?