dev/nisse
2
1 Fork 0
Code Issues 23 Pull Requests Actions Packages Projects Releases 1 Wiki Activity

Spec: nisse-review (Part R) before nisse-ops #146

Merged
dev-agent merged 2 commits from nisse-review-spec into main 2026-10-13 20:01:44 +00:00
Conversation 3 Commits 2 Files Changed 3
+138 -14
2 Commits
Author SHA1 Message Date
dongarandClaude Opus 5.5 3013a83167 Part R: a push also resets the round count
CI / checks (push) Skipped
CI / checks (pull_request) Successful in 1m44s
Details
CI / e2e (pull_request) Successful in 3m15s
Details 
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01J5Utyfzw7gF7UhjRDEtjTk
 2026-10-13 19:49:46 +00:00
dongarandClaude Opus 5.5 175ed0771d Spec: nisse-review (Part R) before nisse-ops
CI / checks (push) Skipped
CI / checks (pull_request) Successful in 1m42s
Details
CI / e2e (pull_request) Successful in 3m59s
Details 
A review agent comes second: it reads every ready PR in the repositories
it can see and discusses its points on the PR, never changing code.

- Architecture Part R: duty and items (R1), rounds capped and reset by
  anyone else commenting (R2), read-only reach bound to the item's
  repository, no approvals, no build VM for now (R3), peers (R4), local
  model with ask_expert, no Matrix room for now, own identity (R5).
- D5 and L11: a peer's review is constructive criticism, answered point by
  point; never an instruction, approval or go-ahead; nisse-dev doesn't
  wait for it before asking to merge.
- Part Q: letting nisse-review approve a limited subset of PRs.
- Development Plan: Phase 7 is nisse-core and nisse-review (M7.2-M7.6);
  nisse-ops moves to Phase 8 (M8.1-M8.5), with the MCP decisions so far.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01J5Utyfzw7gF7UhjRDEtjTk
 2026-10-13 19:39:15 +00:00