@senomas/pi-git-hat 0.2.0 → 0.2.2

package/git-hat.ts

@@ -60,7 +60,7 @@ interface RolesConfig {
   defaultRole?: string;
   fileDir?: string;
   caseInsensitive?: boolean;
-  redetectOnInput?: boolean;
+
   postSwitchLog?: boolean;
 }
 
@@ -68,7 +68,7 @@ interface MergedConfig {
   roles: Record<string, RoleDef>;
   fileDir: string;
   caseInsensitive: boolean;
-  redetectOnInput: boolean;
+
   postSwitchLog: boolean;
   configFile: string;
 }
@@ -229,7 +229,7 @@ function loadConfig(): MergedConfig {
     roles: {},
     fileDir: ".pi",
     caseInsensitive: true,
-    redetectOnInput: true,
+
     postSwitchLog: true,
   };
 
@@ -252,7 +252,7 @@ function loadConfig(): MergedConfig {
         }
         if (raw.fileDir) merged.fileDir = raw.fileDir;
         if (raw.caseInsensitive !== undefined) merged.caseInsensitive = raw.caseInsensitive;
-        if (raw.redetectOnInput !== undefined) merged.redetectOnInput = raw.redetectOnInput;
+
         if (raw.postSwitchLog !== undefined) merged.postSwitchLog = raw.postSwitchLog;
       } catch {
         // ignore parse errors
@@ -497,6 +497,22 @@ You are **ADMIN**. Switching branch does not switch your role.
 - Edit \`README.md\` and \`AGENTS.md\`
 - Explore the codebase (read-only)
 
+## What you do NOT do
+- Edit source code
+- Create or modify \`todo/\`, \`plan/\`, \`report/\` files
+- Switch branches (the user handles that)
+`,
+
+  researcher: `
+
+## Your Role: Researcher
+
+You are **RESEARCHER**. Switching branch does not switch your role.
+
+## What you do
+- Research with \`find\` / \`grep\` / \`ls\` / \`web_search\` / \`read\`
+- Write findings to \`docs/*.md\` and root \`*.md\` files
+
 ## What you do NOT do
 - Edit source code
 - Create or modify \`todo/\`, \`plan/\`, \`report/\` files
@@ -723,6 +739,7 @@ function roleIcon(role: string): string {
   if (lower === "implementor") return "\uD83D\uDEE0";
   if (lower === "reviewer") return "\uD83D\uDD0D";
   if (lower === "admin") return "\u2699";
+  if (lower === "researcher") return "\uD83D\uDD2C";
   return "\uD83E\uDDE2";
 }
 
@@ -1200,18 +1217,14 @@ export default function (pi: ExtensionAPI) {
   });
 
   pi.on("input", async (event, ctx) => {
-    if (config.redetectOnInput) {
-      await updateRole(ctx);
-    }
+    await updateRole(ctx);
     return { action: "continue" };
   });
 
   // -- Re-detect after every turn (catches git checkout/switch) --
 
   pi.on("turn_end", async (_event, ctx) => {
-    if (config.redetectOnInput) {
-      await updateRole(ctx);
-    }
+    await updateRole(ctx);
   });
 
   // -- System prompt injection ----------------------------------
@@ -1312,13 +1325,19 @@ If the user asks you to make changes, tell them to switch to an appropriate bran
         if (isBash) {
           const cmd = ((event.input as { command?: string }).command ?? "").trim();
 
-          // Always allow rebase master/main (required to fix ancestry)
-          if (/^git\s+rebase\s+(master|main)\b/.test(cmd)) {
+          // Require confirmation for git rebase (any target)
+          if (/\bgit\s+rebase\b/.test(cmd)) {
+            const confirmed = await ctx.ui.confirm(
+              `Rebase onto master? Running: \`${cmd}\``,
+            );
+            if (!confirmed) {
+              return { block: true, reason: "Rebase cancelled by user." };
+            }
             return;
           }
 
           // Require confirmation for git switch/checkout
-          if (/^git\s+(switch|checkout)\b/.test(cmd)) {
+          if (/\bgit\s+(switch|checkout)\b/.test(cmd)) {
             const confirmed = await ctx.ui.confirm(
               `Switch branches? Running: \`${cmd}\``,
             );
@@ -1336,15 +1355,16 @@ If the user asks you to make changes, tell them to switch to an appropriate bran
       }
     }
 
-    // For known roles: require confirmation for git branch switching
+    // For known roles: require confirmation for git rebase/switch/checkout
     if (isBash) {
       const cmd = ((event.input as { command?: string }).command ?? "").trim();
-      if (/^git\s+(switch|checkout)\b/.test(cmd)) {
+      if (/\bgit\s+(rebase|switch|checkout)\b/.test(cmd)) {
+        const action = /rebase/.test(cmd) ? "Rebase" : "Switch";
         const confirmed = await ctx.ui.confirm(
-          `Switch branches? Running: \`${cmd}\``,
+          `${action} branches? Running: \`${cmd}\``,
         );
         if (!confirmed) {
-          return { block: true, reason: "Branch switch cancelled by user." };
+          return { block: true, reason: `${action} cancelled by user.` };
         }
         return;
       }
@@ -1408,6 +1428,15 @@ If the user asks you to make changes, tell them to switch to an appropriate bran
       };
     }
 
+    // Researcher: docs/ and root *.md only
+    if (lowerRole === "researcher") {
+      if (isInside(path, ["docs"]) || (path.endsWith(".md") && !path.includes("/"))) return;
+      return {
+        block: true,
+        reason: `\uD83D\uDD2C Researcher: can only write to docs/ and root *.md. Blocked: ${rawPath}`,
+      };
+    }
+
     // Custom roles: no hard write restrictions (base-enforce handles it)
     return;
   });

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@senomas/pi-git-hat",
-	"version": "0.2.0",
+	"version": "0.2.2",
 	"description": "Pi extension for role-based Git branch workflows — wear different hats by switching branches",
 	"type": "module",
 	"keywords": ["pi-package", "git", "workflow", "branching", "roles"],

package/roles/planner.md

@@ -6,10 +6,11 @@ You are **PLANNER**. Your sole responsibility is to research (just collecting da
 ## What you do
 1. Research with `read` / `grep` / `find` / `ls`
 2. Create `todo/NN-name.md` with sequenced, actionable items
-3. Present the plan to the user
+3. Write to `docs/*.md` files (design docs, research notes, architecture decisions)
+4. Present the plan to the user
 
 ## What you do NOT do
-- `edit`/`write` any file outside `todo/`
+- `edit`/`write` any file outside `todo/` or `docs/*.md`
 - Switch branches (the user handles that)
 - Implement anything (that's the implementor)
 - Write reports (that's the implementor)

package/roles/researcher.md

@@ -0,0 +1,12 @@
+# Researcher
+
+You are **RESEARCHER**. Switching branch does not switch your role.
+
+## What you do
+- Research with `find` / `grep` / `ls` / `web_search` / `read`
+- Write findings to `docs/*.md` and root `*.md` files
+
+## What you do NOT do
+- Edit source code
+- Create or modify `todo/`, `plan/`, `report/` files
+- Switch branches (the user handles that)

package/roles/roles.json

@@ -3,6 +3,7 @@
     "planner":      { "pattern": "^plan(-|/|$)", "description": "Plan work by creating todo/plan files" },
     "implementor":  { "pattern": "^(implementor$|feature|feat|impl$|work$)(-|/|$)", "description": "Feature/impl/work branches" },
     "reviewer":     { "pattern": "^review(-|/|$)", "description": "Review implementations against todos" },
-    "admin":        { "pattern": "^(main|master)$", "description": "Project configuration and docs" }
+    "admin":        { "pattern": "^(main|master)$", "description": "Project configuration and docs" },
+    "researcher":   { "pattern": "^research(-|/|$)", "description": "Research topics and document findings" }
   }
 }