VERIFIED APPROVE reviewer-of-record [W-jarvis:reviewer] head b4fcf139a7
Reviewed exact live head b4fcf139a73678e9e59c8f6b63c108c095a87b3a; Woodpecker pipeline 1776 is SUCCESS for that commit.
Findings:
Scope is limited to packages/agent + packages/mosaic and two barrel exports. No gateway, Discord, or runtime-registry wiring changed, so Matrix is not promoted/defaulted in this PR.
MatrixNativeRuntimeProvider implements the normalized AgentRuntimeProvider shape and declares only concretely wired runtime ops (session.list/tree/stream/send/attach/terminate). Matrix room/event/credential mechanics stay transport-local.
Authority boundaries are fail-closed: default read/write authorities deny, writes require both preflight and exact authorization, read attachments are immutable-scope-bound, and control attach is rejected before transport verification.
The parity suite runs the same contract scenarios against both existing tmux/fleet and Matrix/native fixtures (it.each(fixtures())): shared capabilities/list shape, empty send rejection before transport, empty termination approval rejection before transport, and read-only attach + cross-scope detach denial.
MatrixNativeRuntimeTransport is concrete Matrix CS API code: HTTPS homeserver validation, configured session→room bindings only, whoami configured-user checks before ops, deterministic idempotency-key-derived transaction IDs, trusted scope metadata in outbound events, and stream filtering to the configured remote identity/session.
apps/gateway/src/commands/command-authorization.service.ts is byte-identical to main, hash a9f829e7ecec721c6c585fe0da56fb5d6a6441e9.
No live credentials or hardcoded Tess identity found; test-token is test fixture data. Transport errors do not include token values.
No merge performed by reviewer.
Marker: ROR-744-VERIFIED-APPROVE-b4fcf139
VERIFIED APPROVE reviewer-of-record [W-jarvis:reviewer] head b4fcf139a73678e9e59c8f6b63c108c095a87b3a
Reviewed exact live head `b4fcf139a73678e9e59c8f6b63c108c095a87b3a`; Woodpecker pipeline `1776` is SUCCESS for that commit.
Findings:
- Scope is limited to `packages/agent` + `packages/mosaic` and two barrel exports. No gateway, Discord, or runtime-registry wiring changed, so Matrix is not promoted/defaulted in this PR.
- `MatrixNativeRuntimeProvider` implements the normalized `AgentRuntimeProvider` shape and declares only concretely wired runtime ops (`session.list/tree/stream/send/attach/terminate`). Matrix room/event/credential mechanics stay transport-local.
- Authority boundaries are fail-closed: default read/write authorities deny, writes require both preflight and exact authorization, read attachments are immutable-scope-bound, and control attach is rejected before transport verification.
- The parity suite runs the same contract scenarios against both existing tmux/fleet and Matrix/native fixtures (`it.each(fixtures())`): shared capabilities/list shape, empty send rejection before transport, empty termination approval rejection before transport, and read-only attach + cross-scope detach denial.
- `MatrixNativeRuntimeTransport` is concrete Matrix CS API code: HTTPS homeserver validation, configured session→room bindings only, whoami configured-user checks before ops, deterministic idempotency-key-derived transaction IDs, trusted scope metadata in outbound events, and stream filtering to the configured remote identity/session.
- `apps/gateway/src/commands/command-authorization.service.ts` is byte-identical to main, hash `a9f829e7ecec721c6c585fe0da56fb5d6a6441e9`.
- No live credentials or hardcoded Tess identity found; `test-token` is test fixture data. Transport errors do not include token values.
No merge performed by reviewer.
Marker: ROR-744-VERIFIED-APPROVE-b4fcf139
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Summary
Security boundaries
whoami; inbound events require configured remote identityVerification
TURBO_FORCE=true pnpm typecheckTURBO_FORCE=true pnpm lintTURBO_FORCE=true pnpm format:checkTURBO_FORCE=true pnpm test(42 tasks)Closes #711; implements TESS-M5-001 / TESS-TRN-001.
VERIFIED APPROVE reviewer-of-record [W-jarvis:reviewer] head
b4fcf139a7Reviewed exact live head
b4fcf139a73678e9e59c8f6b63c108c095a87b3a; Woodpecker pipeline1776is SUCCESS for that commit.Findings:
packages/agent+packages/mosaicand two barrel exports. No gateway, Discord, or runtime-registry wiring changed, so Matrix is not promoted/defaulted in this PR.MatrixNativeRuntimeProviderimplements the normalizedAgentRuntimeProvidershape and declares only concretely wired runtime ops (session.list/tree/stream/send/attach/terminate). Matrix room/event/credential mechanics stay transport-local.it.each(fixtures())): shared capabilities/list shape, empty send rejection before transport, empty termination approval rejection before transport, and read-only attach + cross-scope detach denial.MatrixNativeRuntimeTransportis concrete Matrix CS API code: HTTPS homeserver validation, configured session→room bindings only, whoami configured-user checks before ops, deterministic idempotency-key-derived transaction IDs, trusted scope metadata in outbound events, and stream filtering to the configured remote identity/session.apps/gateway/src/commands/command-authorization.service.tsis byte-identical to main, hasha9f829e7ecec721c6c585fe0da56fb5d6a6441e9.test-tokenis test fixture data. Transport errors do not include token values.No merge performed by reviewer.
Marker: ROR-744-VERIFIED-APPROVE-b4fcf139