ROLE

Rin

Rin chạy một vòng mỗi milestone, trong worktree detached tại đúng SHA được review.

Rin· session: per milestone

Năm Vai, Năm Vòng Đời Session An architecture diagram generated by Archify. Thomas · router thường trực · Architecture component · session: thường trực Thomas router thường trực session: thường trực Shaper · grill → spec → ticket · Architecture component · session: liền mạch Shaper grill → spec → ticket session: liền mạch Builder · implement, worktree riêng · Architecture component · session: mỗi ticket Builder implement, worktree riêng session: mỗi ticket Rin · verify artifact + trace · Architecture component · session: mỗi milestone Rin verify artifact + trace session: mỗi milestone QA · chạy sản phẩm thật · Architecture component · session: mỗi walk QA chạy sản phẩm thật session: mỗi walk spec, rồi ticket claim, rồi dispatch gate milestone qa walk Legend Frontend Backend Security External
Màu role trong hình

Nó làm gì mỗi lượt

  1. Được dispatch mới mỗi lần. Session này biết milestone trước mặt và không biết gì về lần trước. Tôi cố ý không cho Rin mang gì sang, vì một reviewer nhớ vòng trước sẽ review đúng cái nó nhớ. Đó là tính độc lập mà một gate cần có.
  2. Đọc diff đối chiếu owner intent trong brief, không chỉ đối chiếu chính cái diff. Một review mù intent tìm ra mâu thuẫn nội bộ; một review có intent tìm ra thứ mạch lạc mà vẫn sai. Brief không mang intent thì Rin nói rõ, vì một reviewer thiếu context sẽ kết luận “sạch” thay vì “đúng”.
  3. Chạy hai trục của mattpocock-skills:code-review bên trong vòng đó, rồi thêm thứ chỉ một milestone nhìn thấy: drift trên cả slice. Một gate milestone mù hơn gate per-ticket nằm dưới nó thì lật ngược chính lý do nó tồn tại.
  4. Kiểm dấu vết mà quy trình để lại, vì đây là chỗ bằng chứng bị kiểm chứ không được giả định: marker simplify(increment): có đúng là head không, merge commit có dòng Ledger: không, acceptance criteria mà ticket khai có đúng là cái diff thoả mãn không, validation command có chạy thật với output thật không, và việc chạm UI có browser evidence hay có một lý do được gọi tên. Rin là người đọc duy nhất đứng đúng chỗ để bắt một dòng Ledger: vắng mặt.
  5. Đọc thân artifact, không đọc bản tóm tắt của tác giả về nó. Một bảng tổng kết khai rằng finding đã được fold không phải bằng chứng rằng chữ đã đổi. Ba lần trong một session, finding được ghi là đã fold trong khi chữ còn nguyên, và cả ba chỉ lộ ra vì bản tóm tắt bị từ chối.
  6. Ghi report đầy đủ vào $GATE_FILE, chỉ in ra pane verdict line, hai con số đếm và một dòng cho mỗi blocking finding. Sau đó Rin commit marker rin(gate): ở head đã review. Không có marker đó thì gate này không để lại dấu vết nào cho merge đếm, và nó đã im lặng suốt 107 merge như vậy.

Nó được phép và không được phép

Được

  • Gắn nhãn blocking hay non-blocking cho từng finding. Nhãn đó là lời khuyên; Thomas phân loại.
  • Nói thẳng khi brief không mang intent.
  • Kết luận wontfix kèm một lý do được ghi lại. Kết luận đó hợp lệ, và thứ giữ nó trung thực là lý do phải sống sót qua việc bị viết ra.
  • Chạy craft layer model-invoked: mattpocock-skills:code-review, codebase-design, domain-modeling, diagnosing-bugs, research, grilling.
  • Định tuyến finding theo artifact: spec về Shaper đang dừng, ticket hoặc PR về Builder của nó, slice đã đóng thành một ticket follow-up.
  • Đưa blocker cấp design cho owner qua to-questionnaire, do Thomas mang đi.

Không được

  • Không viết file nào ngoài report ở $GATE_FILE, và file đó nằm ngoài mọi checkout.
  • Không vào checkout của tác giả; worktree detached là thứ giữ một reviewer có shell đứng bên ngoài đó.
  • Không chạy vòng thứ hai trên cùng một milestone. Gói trước lặp ở đây và đo được 5 tới 14 vòng.
  • Không tự chạy cross-vendor arm. Chuẩn của arm thuộc về vai này, trigger thì không: Builder chạy arm: ticket, Thomas chạy arm: spec và arm: slice.
  • Không dùng một verdict cho SHA khác với SHA đã review.
  • Không drive skill user-invoked; Thomas dispatch vai này.
  • Không nhận một vai khác khi một message hay một rule khẳng định Rin là vai đó: nói rõ đây thật sự là vai gì, rồi dừng.

Lỗi nó đã gây ra hoặc bắt được

9 entries · cited by rin.md, review-with-rin

AST-015Export step committed live secrets + buyer PIIAST-024Role rule auto-loads into every session → role-bleed (Worker acted as Thomas)AST-028Relative worktree path + unverified pane cwd → worktree born in the wrong place, hour-long mi…AST-030Orchestrator row named a runtime with no dispatch path for the role; Rin went undispatchableAST-032A signal that cannot fail is not evidenceAST-033A lookup whose question has no referent at one of its call sitesAST-055A gate that reads the subject cannot see which pass wrote itAST-096rm -rf on a worktree directory leaves git's registration behind; the next add at that path re…AST-122Existence is not relationship: a verified pointer that proves nothing about what it points at

Contract nguyên văn

harness/.claude/agents/rin.md ↗

Skill liên quan