chatroom-cli 1.132.0 → 1.132.1

package/dist/index.js

@@ -20412,6 +20412,10 @@ var init_agent_role = __esm(() => {
         { teamId: "duo", order: 45 },
         { teamId: "solo", order: 45 }
       ]
+    },
+    reviewer: {
+      tags: ["ephemeral" /* Ephemeral */],
+      teams: [{ teamId: "duo", order: 35 }]
     }
   };
   AGENT_ROLE_DEFINITIONS = Object.fromEntries(Object.entries(roleMetadata).map(([role, metadata]) => [role, { role, ...metadata }]));
@@ -100027,7 +100031,7 @@ var init_team_presets = __esm(() => {
     duo: {
       structureId: TEAM_STRUCTURE_IDS.duo,
       name: "Duo",
-      description: "A planner and builder working as a pair, planner as coordinator, with optional ephemeral architect, researcher, triage, uiux-engineer, and verifier roles",
+      description: "A planner and builder working as a pair, planner as coordinator, with optional ephemeral architect, researcher, triage, reviewer, uiux-engineer, and verifier roles",
       roles: getBuiltinRolesForTeam("duo"),
       entryPoint: "planner"
     },
@@ -100992,6 +100996,75 @@ var init_planner_researcher_chat = __esm(() => {
   init_planner_researcher_code();
 });
 
+// ../../services/backend/prompts/teams/reviewer-handoff-templates.ts
+function getEntryPointToReviewerHandoffTemplate(query) {
+  const entryPoint = query.fromRole;
+  return `Pass the review target and the review focus. State the exact commit range, branch, pull request, or file list to inspect, the focus areas (for example security, correctness, or test quality), and any known context or constraints. Ask the reviewer for a report only: do not edit files, run builds or tests, or implement fixes.
+
+### When to review
+
+Review before delivery for code-producing work:
+- a builder slice has just completed and is about to be delivered;
+- the user asks for a code review;
+- a change is large, risky, or touches security-sensitive or shared code.
+
+Skip review for a trivial edit that has already been verified.
+
+After completing the review, the reviewer must return its issue report to \`${entryPoint}\` with the normal handoff command.
+
+\`\`\`bash
+${handoffCommand({
+    chatroomId: "<chatroom-id>",
+    role: entryPoint,
+    nextRole: "reviewer",
+    cliEnvPrefix: ""
+  })}
+\`\`\``;
+}
+function getReviewerToEntryPointHandoffTemplate(query) {
+  const entryPoint = query.toRole;
+  return `Instructions (not part of the message): Return a complete review report for the target the planner named. The reviewer is advisory: do not edit files, implement fixes, run builds or tests, or propose architecture rewrites beyond each issue's suggested fix. Report only what you observed and mark anything unverified as such. Label every issue with a severity: \`blocker\`, \`major\`, \`minor\`, or \`nit\`. Group the issues by severity with blockers first, and cite the repository-relative file and line or line range for each. Use the exact sections below, in this order.
+
+## Summary
+<one to three sentences: overall state of the target and the count of blocker/major issues>
+
+## Review scope
+<commits, paths, and PR/branch inspected; commands run, if any>
+
+## Issues
+<numbered list grouped by severity (blocker, major, minor, nit); each item: \`file:line\` — summary — evidence — suggested fix; write exactly \`None observed.\` if none>
+
+## Not reviewed
+<anything out of scope or unreadable; write exactly \`None.\` if none>
+
+Instructions after the body (not part of the message): run the normal handoff command to \`${entryPoint}\` with the complete body above, then stop.
+
+\`\`\`bash
+${handoffCommand({
+    chatroomId: "<chatroom-id>",
+    role: "reviewer",
+    nextRole: entryPoint,
+    cliEnvPrefix: ""
+  })}
+\`\`\``;
+}
+var init_reviewer_handoff_templates = __esm(() => {
+  init_command2();
+});
+
+// ../../services/backend/prompts/teams/duo/handoff-templates/planner-reviewer-code.ts
+function getPlannerReviewerCodeTemplate(query) {
+  return getEntryPointToReviewerHandoffTemplate(query);
+}
+var init_planner_reviewer_code = __esm(() => {
+  init_reviewer_handoff_templates();
+});
+
+// ../../services/backend/prompts/teams/duo/handoff-templates/planner-reviewer-chat.ts
+var init_planner_reviewer_chat = __esm(() => {
+  init_planner_reviewer_code();
+});
+
 // ../../services/backend/prompts/teams/triage-handoff-templates.ts
 function getEntryPointToTriageHandoffTemplate(query) {
   const entryPoint = query.fromRole;
@@ -101491,6 +101564,8 @@ var init_planner = __esm(() => {
   init_planner_builder_code();
   init_planner_researcher_chat();
   init_planner_researcher_code();
+  init_planner_reviewer_chat();
+  init_planner_reviewer_code();
   init_planner_triage_chat();
   init_planner_triage_code();
   init_planner_uiux_engineer_chat();
@@ -101509,9 +101584,19 @@ var init_planner = __esm(() => {
       "uiux-engineer",
       "triage",
       "researcher",
+      "reviewer",
+      "verifier"
+    ],
+    returnsTo: [
+      "user",
+      "builder",
+      "architect",
+      "uiux-engineer",
+      "triage",
+      "researcher",
+      "reviewer",
       "verifier"
     ],
-    returnsTo: ["user", "builder", "architect", "uiux-engineer", "triage", "researcher", "verifier"],
     outboundTemplates: {
       builder: handoffTemplateByMode({
         chat: getPlannerBuilderCodeTemplate,
@@ -101533,6 +101618,10 @@ var init_planner = __esm(() => {
         chat: getPlannerResearcherCodeTemplate,
         code: getPlannerResearcherCodeTemplate
       }),
+      reviewer: handoffTemplateByMode({
+        chat: getPlannerReviewerCodeTemplate,
+        code: getPlannerReviewerCodeTemplate
+      }),
       verifier: handoffTemplateByMode({
         chat: getPlannerVerifierCodeTemplate,
         code: getPlannerVerifierCodeTemplate
@@ -101577,6 +101666,38 @@ var init_researcher = __esm(() => {
   };
 });
 
+// ../../services/backend/prompts/teams/duo/handoff-templates/reviewer-planner-code.ts
+function getReviewerPlannerCodeTemplate(query) {
+  return getReviewerToEntryPointHandoffTemplate(query);
+}
+var init_reviewer_planner_code = __esm(() => {
+  init_reviewer_handoff_templates();
+});
+
+// ../../services/backend/prompts/teams/duo/handoff-templates/reviewer-planner-chat.ts
+var init_reviewer_planner_chat = __esm(() => {
+  init_reviewer_planner_code();
+});
+
+// ../../services/backend/prompts/teams/duo/handoff-templates/reviewer.ts
+var duoReviewerHandoffContract;
+var init_reviewer = __esm(() => {
+  init_reviewer_planner_chat();
+  init_reviewer_planner_code();
+  init_contracts();
+  duoReviewerHandoffContract = {
+    role: "reviewer",
+    receivesFrom: ["planner"],
+    returnsTo: ["planner"],
+    outboundTemplates: {
+      planner: handoffTemplateByMode({
+        chat: getReviewerPlannerCodeTemplate,
+        code: getReviewerPlannerCodeTemplate
+      })
+    }
+  };
+});
+
 // ../../services/backend/prompts/teams/duo/handoff-templates/triage-planner-code.ts
 function getTriagePlannerCodeTemplate(query) {
   return getTriageToEntryPointHandoffTemplate(query);
@@ -101683,6 +101804,7 @@ var init_handoff_templates = __esm(() => {
   init_builder();
   init_planner();
   init_researcher();
+  init_reviewer();
   init_triage();
   init_uiux_engineer();
   init_verifier();
@@ -101692,6 +101814,7 @@ var init_handoff_templates = __esm(() => {
     builder: duoBuilderHandoffContract,
     architect: duoArchitectHandoffContract,
     researcher: duoResearcherHandoffContract,
+    reviewer: duoReviewerHandoffContract,
     "uiux-engineer": duoUiuxEngineerHandoffContract,
     triage: duoTriageHandoffContract,
     verifier: duoVerifierHandoffContract
@@ -102536,6 +102659,30 @@ chatroom handoff --chatroom-id="<chatroom-id>" --role="researcher" --next-role="
 \`\`\``;
 });
 
+// ../../services/backend/prompts/cli/roles/reviewer.ts
+var REVIEWER_GUIDANCE;
+var init_reviewer2 = __esm(() => {
+  init_sections();
+  REVIEWER_GUIDANCE = `## Reviewer Operating Model
+
+You are the reviewer responsible for inspecting the code artifacts the planner names and reporting the issues you observe. You are advisory only: do not edit files, implement fixes, or act as the builder or planner.
+
+${"{SESSION_CONTINUITY}"}
+
+${getInterRoleHandoffRule()}
+
+1. Recover the review target and focus from the injected task and the \`<handoff-templates>\` brief. If the target is not explicit, ask the planner through the handoff; do not guess a range.
+2. Inspect the target: the diff or commits, the changed files, and the surrounding code they depend on. Read the actual files; do not review from summaries.
+3. Look for correctness bugs, security issues, missing or weak tests, unclear ownership or error handling, dead code, duplication, and type drift. Stay within the stated focus when one is given.
+4. Label every issue with a severity (\`blocker\`, \`major\`, \`minor\`, \`nit\`) and cite the file and line. Only report what you observed; mark anything unverified as such.
+5. Follow the injected \`<handoff-templates>\` review report exactly, in its order.
+6. Hand the report to \`{ENTRY_POINT_ROLE}\` with the normal command, then stop:
+
+\`\`\`bash
+chatroom handoff --chatroom-id="<chatroom-id>" --role="reviewer" --next-role="{ENTRY_POINT_ROLE}"
+\`\`\``;
+});
+
 // ../../services/backend/prompts/cli/roles/triage.ts
 var TRIAGE_GUIDANCE;
 var init_triage3 = __esm(() => {
@@ -102638,6 +102785,7 @@ var init_fromContext2 = __esm(() => {
   init_builder2();
   init_planner2();
   init_researcher2();
+  init_reviewer2();
   init_triage3();
   init_uiux_engineer3();
   init_verifier3();
@@ -102731,6 +102879,19 @@ var init_templates = __esm(() => {
       ],
       defaultHandoffTarget: "planner"
     },
+    reviewer: {
+      role: "reviewer",
+      title: "Reviewer",
+      description: "You are the reviewer responsible for inspecting the code artifacts the planner names and reporting the issues you observe; you are advisory only and do not implement fixes.",
+      responsibilities: [
+        "Recover the review target and focus from the planner brief",
+        "Inspect the diff or commits and the surrounding code they depend on",
+        "Look for correctness, security, test quality, ownership, dead code, duplication, and type drift",
+        "Label each issue with a severity and cite the file and line",
+        "Hand one review report to the planner, separating reviewed scope from what was not reviewed"
+      ],
+      defaultHandoffTarget: "planner"
+    },
     verifier: {
       role: "verifier",
       title: "Verifier",
@@ -139141,4 +139302,4 @@ program2.hook("preAction", async (_thisCommand, actionCommand) => {
 });
 program2.parse();
 
-//# debugId=430CD3ADC73C742264756E2164756E21
+//# debugId=BCEF986EF291738664756E2164756E21