@ionivetech/mugiwara 0.6.1 → 0.6.2

package/scripts/savepoint.sh

@@ -7,6 +7,14 @@ die() { echo "savepoint: $*" >&2; exit 1; }
 
 MUGIWARA_DIR="${MUGIWARA_DIR:-.mugiwara}"
 
+# --- git identity for actor attribution ---
+# Resolve once from repo git config; used when no actor is passed explicitly.
+# GIT_AUTHOR_NAME is an env var usually unset — falling through to $USER
+# attributes the savepoint to the OS user instead of the git identity.
+GIT_NAME="$(git config user.name 2>/dev/null || true)"
+GIT_EMAIL="$(git config user.email 2>/dev/null || true)"
+if [ -n "$GIT_NAME" ] && [ -n "$GIT_EMAIL" ]; then GIT_ID="$GIT_NAME <$GIT_EMAIL>"; else GIT_ID="$GIT_NAME"; fi
+
 # --- parse mission args ---
 # --branch flag must parse FIRST — it shifts positionals
 BRANCH_MODE=0
@@ -14,13 +22,13 @@ if [ "${1:-}" = "--branch" ]; then
   BRANCH_MODE=1
   shift
   MISSION="${1:-${STATE_MISSION:-}}"
-  ACTOR="${2:-${STATE_ACTOR:-${GIT_AUTHOR_NAME:-${USER:-}}}}"
+  ACTOR="${2:-${STATE_ACTOR:-${GIT_AUTHOR_NAME:-${GIT_ID:-${USER:-}}}}}"
   BRANCH="${3:-$(git branch --show-current 2>/dev/null || echo 'unknown')}"
   WAVE="${4:-${STATE_WAVE:-1}}"
   MODE="${5:-${STATE_MODE:-guided}}"
 else
   MISSION="${1:-${STATE_MISSION:-}}"
-  ACTOR="${2:-${STATE_ACTOR:-${GIT_AUTHOR_NAME:-${USER:-}}}}"
+  ACTOR="${2:-${STATE_ACTOR:-${GIT_AUTHOR_NAME:-${GIT_ID:-${USER:-}}}}}"
   BRANCH="${3:-$(git branch --show-current 2>/dev/null || echo 'unknown')}"
   WAVE="${4:-${STATE_WAVE:-1}}"
   MODE="${5:-${STATE_MODE:-guided}}"

package/src/mission.ts

@@ -80,15 +80,12 @@ export function archiveMission(projectDir: string, mission: string, opts: { dryR
     if (f) report = join('reports', f);
   }
 
-  // remove per-mission wave intermediates from results/<mission>/, EXCEPT
-  // 06-closure.md and 07-pr-verdict.md (PR material + closure stay)
+  // step results 01..05 + todos.md are evidence — kept; archive removes
+  // only spec/review/issues/logs/continue.md
   const resultsDir = join(root, 'results', mission);
   if (existsSync(resultsDir)) {
     for (const f of readdirSync(resultsDir)) {
-      if (f === '06-closure.md' || f === '07-pr-verdict.md') { kept.push(join('results', mission, f)); continue; }
-      const p = join(resultsDir, f);
-      if (!dryRun) rmSync(p, { recursive: true, force: true });
-      removed.push(join('results', mission, f));
+      kept.push(join('results', mission, f));
     }
   }
 

package/src/targets/claude.ts

@@ -10,8 +10,10 @@ const HOOKS_SRC = join(here, '..', '..', 'hooks');
 const COMMANDS_SRC = join(here, '..', '..', '.claude', 'commands');
 
 // Claude Code has no path-scoped permission. write-scope maps to a partial
-// `tools:` list: artifacts agents lose Edit (cannot modify existing source)
-// but keep Write (must create .mugiwara/**); source agents get the default set.
+// `tools:` list — applied to internal subagent-only agents only: artifacts
+// agents lose Edit (cannot modify existing source) but keep Write (must create
+// .mugiwara/**). User-facing crew run inline in the main thread and keep the
+// default toolset (incl. Edit); discipline is enforced by rules, not tools.
 function toolsFromScope(scope?: string): string | undefined {
   if (scope === 'artifacts') return 'Read, Grep, Glob, Write, Bash, WebFetch, WebSearch';
   return undefined;
@@ -35,7 +37,7 @@ export const target: Target = {
   transformAgent(data: FrontmatterData, body: string) {
     const fm: FrontmatterData = { name: data.name, description: data.description };
     if (data.tools) fm.tools = data.tools;
-    else {
+    else if (data['internal-agent'] === 'true') {
       const generated = toolsFromScope(data['write-scope']);
       if (generated) fm.tools = generated;
     }

package/src/targets/opencode.ts

@@ -32,23 +32,27 @@ const CREW: Record<string, CrewConfig> = {
   'memory-keeper': { color: '#d946ef', temperature: 0.2, steps: 8 },
 };
 
-// write-scope is the single source of truth (content/agents/*.md frontmatter).
-// The path boundary (artifacts vs source) IS expressible in opencode:
-// permission.edit accepts glob/pattern -> action, last match wins. Artifacts
-// agents get deny-all-edit except .mugiwara/**; source agents (zoro, brook)
-// get full edit allow. Derived at install time from the frontmatter field.
+// write-scope is the single source of truth (content/agents/*.md frontmatter),
+// but it is a RULE for user-facing crew agents: they run inline in the main
+// thread, so a runtime permission bound to the active-agent identity would
+// force tab-switching per wave and break auto mode + resume. Runtime
+// enforcement stays for internal subagent-only agents (internal: true) — the
+// path boundary IS expressible in opencode: permission.edit accepts
+// glob/pattern -> action, last match wins. Artifacts internal agents get
+// deny-all-edit except .mugiwara/**; source internal agents get full edit
+// allow. Derived at install time from the frontmatter field.
 function permissionFromScope(scope: string | undefined): Record<string, string | Record<string, string>> | undefined {
   if (scope === 'source') return { edit: 'allow' };
   if (scope === 'artifacts') return { edit: { '*': 'deny', '.mugiwara/**': 'allow' } };
   return undefined;
 }
 
-function agentFrontmatter(name: string, description: string, writeScope?: string) {
+function agentFrontmatter(name: string, description: string, internal: boolean, writeScope?: string) {
   const crew = CREW[name];
   const lines = [`description: ${description}`, `mode: all`];
   if (crew) {
     lines.push(`color: '${crew.color}'`, `temperature: ${crew.temperature}`, `steps: ${crew.steps}`);
-    const perm = permissionFromScope(writeScope);
+    const perm = internal ? permissionFromScope(writeScope) : undefined;
     if (perm) {
       lines.push('permission:');
       for (const [k, v] of Object.entries(perm)) {
@@ -81,7 +85,7 @@ export const target: Target = {
   },
   transformAgent(data: FrontmatterData, body: string) {
     const desc = data['internal-agent'] === 'true' ? `[INTERNAL] ${data.description}` : data.description;
-    const fm = agentFrontmatter(data.name, desc, data['write-scope']);
+    const fm = agentFrontmatter(data.name, desc, data['internal-agent'] === 'true', data['write-scope']);
     return { relPath: `${data.name}.md`, text: `---\n${fm}\n---\n${body}` };
   },
   refsDir({ scope, projectDir, home }, skillName: string) {