toga-ai 1.0.32 → 1.0.33

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "toga-ai",
-  "version": "1.0.32",
+  "version": "1.0.33",
   "description": "TOGA Technology Team Claude Knowledge System — shared AI coding harness with skills, knowledge base CLI, and project installer for Claude Code.",
   "keywords": [
     "claude",

package/scripts/install.js

@@ -661,13 +661,28 @@ function main() {
   const ok = mergeSettings(claudeDir, harnessDir);
   console.log('  ' + (ok ? '✓' : '✗') + ' Hooks     (' + hooksCount + ' scripts in .claude/hooks/toga/, settings.json merged)');
 
-  // MCP example — skip if already present
+  // Contexts — mode context files (dev/research/review), update if changed
+  const contextsSrc = path.join(harnessDir, 'contexts');
+  const contextsDest = path.join(claudeDir, 'contexts', 'toga');
+  if (fs.existsSync(contextsSrc)) {
+    let contextsStats = { added: 0, updated: 0, unchanged: 0 };
+    try { contextsStats = copyDir(contextsSrc, contextsDest, { updateIfChanged: true }); }
+    catch (e) { errors.push('contexts: ' + e.message); }
+    console.log('  ✓ Contexts  (' + countMd(contextsDest) + '): ' + fmtStats(contextsStats));
+  }
+
+  // MCP example — update if changed so teammates get new server configs
   const mcpSrc = path.join(harnessDir, 'mcp-configs', 'mcp-servers.json');
   const mcpDest = path.join(claudeDir, 'mcp-servers.example.json');
-  if (fs.existsSync(mcpSrc) && !fs.existsSync(mcpDest)) {
-    try { fs.copyFileSync(mcpSrc, mcpDest); }
-    catch (e) { errors.push('mcp: ' + e.message); }
-    console.log('  ✓ MCP       (.claude/mcp-servers.example.json — merge into .mcp.json to enable)');
+  if (fs.existsSync(mcpSrc)) {
+    try {
+      const firstInstall = !fs.existsSync(mcpDest);
+      const incoming = fs.readFileSync(mcpSrc);
+      if (firstInstall || !incoming.equals(fs.readFileSync(mcpDest))) {
+        fs.copyFileSync(mcpSrc, mcpDest);
+        console.log('  ✓ MCP       (.claude/mcp-servers.example.json ' + (firstInstall ? 'created' : 'updated') + ' — merge into .mcp.json to enable)');
+      }
+    } catch (e) { errors.push('mcp: ' + e.message); }
   }
 
   // Knowledge — git repo if available (has team-captured docs), else npm bundle
@@ -697,6 +712,26 @@ function main() {
   }
   console.log('  ✓ Knowledge (' + countMd(knowledgeDest) + ' docs): ' + fmtStats(knowledgeStats));
 
+  // registry.json — the repo↔project↔framework map every skill depends on.
+  // Force-sync explicitly from the newest source (git if pulled, else bundle) so a
+  // partial knowledge copy can never leave a machine without/with a stale registry.
+  const registrySrc = path.join(knowledgeDir, 'knowledge', 'registry.json');
+  const registryDest = path.join(knowledgeDest, 'registry.json');
+  if (fs.existsSync(registrySrc)) {
+    try {
+      fs.copyFileSync(registrySrc, registryDest);
+      let repoCount = 0;
+      try {
+        const reg = JSON.parse(fs.readFileSync(registryDest, 'utf8'));
+        repoCount = Array.isArray(reg) ? reg.length : 0;
+      } catch (e) {}
+      console.log('  ✓ Registry  (' + repoCount + ' repos): synced from ' + (knowledgeDir !== PACKAGE_ROOT ? 'git repo' : 'npm bundle'));
+    } catch (e) { errors.push('registry.json: ' + e.message); }
+  } else {
+    errors.push('registry.json: missing from source ' + registrySrc);
+    console.log('  ✗ Registry  MISSING from source — kickoff will not work. Re-run: npm cache clean --force && npx toga-ai@latest');
+  }
+
   // CLAUDE.md
   const action = updateClaudeMd(projectRoot, skillNames, harnessDir, 'npm bundle v' + bundleVersion);
   console.log('  ✓ CLAUDE.md ' + action);

package/skills/kickoff/SKILL.md

@@ -5,7 +5,18 @@ description: Start-of-session context loader for TOGA Technology projects. Run t
 
 # Kickoff — prime a coding session from the team knowledge base
 
-## Step 0 — Auto-update check (runs before anything else)
+## Arguments — text passed after `/kickoff` never skips any step
+
+`/kickoff` may be invoked with trailing text (e.g. `/kickoff worker2 backend fix for Compass`).
+That text is the developer's description of today's work — it is **not** permission to
+shortcut the flow.
+
+- **Step 0 (auto-update check) ALWAYS runs first**, with or without arguments.
+- Use the argument text to **pre-fill answers** to the Step 2 interview (framework, layer,
+  repo, client, task). Only ask about whatever is still missing or ambiguous.
+- Never treat the argument as an instruction to start coding before Steps 0–5 complete.
+
+## Step 0 — Auto-update check (runs before anything else, even with arguments)
 
 Before loading any context, check whether the installed harness is up to date:
 
@@ -20,17 +31,33 @@ LATEST=$(node -e "const {execSync}=require('child_process');try{process.stdout.w
 
 **If `INSTALLED` equals `LATEST`** — already up to date. Continue to Step 1.
 
-**If `INSTALLED` differs from `LATEST`** — auto-upgrade now, then continue:
+**If `INSTALLED` differs from `LATEST`** — auto-upgrade now, then **verify** before continuing:
 
 ```bash
 npx toga-ai@latest
 ```
 
-Tell the developer:
-> "Updated toga-ai from v`INSTALLED` → v`LATEST`. Skills, agents, and rules refreshed."
+After the upgrade, re-read the version file to confirm it actually updated — npx can
+serve a stale cached package (common on Windows), in which case the installer ran but
+installed the old version again:
+
+```bash
+NOW=$(cat .claude/toga-ai.version 2>/dev/null || echo "unknown")
+```
 
-If the upgrade fails, warn and continue — a stale install beats a blocked session:
-> "⚠ Auto-update to v`LATEST` failed. Run `npx toga-ai@latest` manually when convenient."
+- **If `NOW` equals `LATEST`** — success. Tell the developer:
+  > "Updated toga-ai from v`INSTALLED` → v`LATEST`. Skills, agents, knowledge, and registry refreshed."
+- **If `NOW` still differs from `LATEST`** — stale npx cache. Retry once with a clean cache:
+  ```bash
+  npm cache clean --force && npx toga-ai@latest
+  ```
+  Re-read the version file again. If it now equals `LATEST`, report success as above.
+- **If still stale after the cache-clean retry, or the upgrade command failed** — warn and
+  continue; a stale install beats a blocked session:
+  > "⚠ Auto-update to v`LATEST` failed (still on v`NOW`). Run `npm cache clean --force && npx toga-ai@latest` manually when convenient."
+
+Never report the update as done based on the command exiting cleanly — only the version
+file equaling `LATEST` counts as success.