@fitlab-ai/agent-infra 0.7.0 → 0.7.2

package/templates/.agents/skills/update-agent-infra/scripts/sync-templates.js

@@ -42,6 +42,9 @@ const DEFAULTS = {
       "disk": null
     }
   },
+  "task": {
+    "shortIdLength": 2
+  },
   "labels": {
     "in": {}
   },
@@ -88,6 +91,39 @@ const PACKAGE_NAME = '@fitlab-ai/agent-infra';
 const KNOWN_PLATFORMS = new Set(['github']);
 const KNOWN_LANGUAGES = new Set(['en', 'zh-CN']);
 
+// Single source of truth for built-in TUI ids and owned path prefixes.
+// Keep in sync with lib/builtin-tuis.ts (enforced by tests/unit/scripts/sync-templates-consts.test.ts).
+const BUILTIN_TUI_IDS = ['claude-code', 'codex', 'gemini-cli', 'opencode'];
+const BUILTIN_TUI_OWNED_PATH_PREFIXES = {
+  'claude-code': ['.claude/'],
+  'codex': ['.codex/'],
+  'gemini-cli': ['.gemini/'],
+  'opencode': ['.opencode/']
+};
+
+function resolveEnabledTUIs(value) {
+  // Missing field / null / non-array → full set (backward compat).
+  if (!Array.isArray(value)) return new Set(BUILTIN_TUI_IDS);
+  // Empty array is a meaningful user choice: no built-in TUI managed.
+  const set = new Set();
+  for (const v of value) {
+    if (typeof v === 'string' && BUILTIN_TUI_IDS.includes(v)) set.add(v);
+  }
+  return set;
+}
+
+function isPathOwnedByDisabledTUI(rel, enabledSet) {
+  const normalized = String(rel || '').replace(/\\/g, '/').replace(/^\.\//, '');
+  for (const tui of BUILTIN_TUI_IDS) {
+    if (enabledSet.has(tui)) continue;
+    for (const prefix of BUILTIN_TUI_OWNED_PATH_PREFIXES[tui]) {
+      const trimmed = prefix.replace(/\/$/, '');
+      if (normalized === trimmed || normalized.startsWith(prefix)) return true;
+    }
+  }
+  return false;
+}
+
 function norm(p) { return p.replace(/\\/g, '/'); }
 
 function normDir(p) {
@@ -654,26 +690,32 @@ function learnAndGenerateCommands(projectRoot, customSkills, tool, templateSkill
   }
 }
 
-function generateCustomCommands(projectRoot, customSkills, project, lang, report, customTUIs, templateSkillNames) {
+function generateCustomCommands(projectRoot, customSkills, project, lang, report, customTUIs, templateSkillNames, enabledTUIs) {
   for (const skill of customSkills) {
-    writeIfChanged(
-      projectRoot,
-      `.claude/commands/${skill.dirName}.md`,
-      generateClaudeCommand(skill, lang),
-      report.custom.commands
-    );
-    writeIfChanged(
-      projectRoot,
-      '.gemini/commands/' + project + '/' + skill.dirName + '.toml',
-      generateGeminiCommand(skill, lang),
-      report.custom.commands
-    );
-    writeIfChanged(
-      projectRoot,
-      `.opencode/commands/${skill.dirName}.md`,
-      generateOpenCodeCommand(skill, lang),
-      report.custom.commands
-    );
+    if (enabledTUIs.has('claude-code')) {
+      writeIfChanged(
+        projectRoot,
+        `.claude/commands/${skill.dirName}.md`,
+        generateClaudeCommand(skill, lang),
+        report.custom.commands
+      );
+    }
+    if (enabledTUIs.has('gemini-cli')) {
+      writeIfChanged(
+        projectRoot,
+        '.gemini/commands/' + project + '/' + skill.dirName + '.toml',
+        generateGeminiCommand(skill, lang),
+        report.custom.commands
+      );
+    }
+    if (enabledTUIs.has('opencode')) {
+      writeIfChanged(
+        projectRoot,
+        `.opencode/commands/${skill.dirName}.md`,
+        generateOpenCodeCommand(skill, lang),
+        report.custom.commands
+      );
+    }
   }
 
   const tools = Array.isArray(customTUIs) ? customTUIs : [];
@@ -969,6 +1011,7 @@ function syncTemplates(projectRoot, templateRootOverride) {
 
   const { project, org, language: lang = 'en' } = cfg;
   const platformType = cfg.platform?.type || DEFAULTS.platform.type;
+  const enabledTUIs = resolveEnabledTUIs(cfg.tuis);
   const customTUIsConfig = Array.isArray(cfg.customTUIs) ? cfg.customTUIs : [];
   const vars = { project, org };
   const templateSkillNames = listTemplateSkillNames(templateRoot);
@@ -989,7 +1032,7 @@ function syncTemplates(projectRoot, templateRootOverride) {
       errors: [],
       conflicts: []
     },
-    managed: { written: [], created: [], unchanged: [], skippedMerged: [], skippedPlatform: [], removed: [] },
+    managed: { written: [], created: [], unchanged: [], skippedMerged: [], skippedPlatform: [], skippedTUI: [], removed: [] },
     custom: {
       detected: [],
       generated: [],
@@ -1016,10 +1059,12 @@ function syncTemplates(projectRoot, templateRootOverride) {
   const known = new Set([...managed, ...merged, ...ejected]);
   for (const e of (DEFAULTS.files.managed || [])) {
     if (isPathOwnedByOtherPlatform(e, platformType)) continue;
+    if (isPathOwnedByDisabledTUI(e, enabledTUIs)) continue;
     if (!known.has(e)) { managed.push(e); known.add(e); report.registryAdded.push({ entry: e, list: 'managed' }); }
   }
   for (const e of (DEFAULTS.files.merged || [])) {
     if (isPathOwnedByOtherPlatform(e, platformType)) continue;
+    if (isPathOwnedByDisabledTUI(e, enabledTUIs)) continue;
     if (!known.has(e)) { merged.push(e); known.add(e); report.registryAdded.push({ entry: e, list: 'merged' }); }
   }
 
@@ -1050,11 +1095,50 @@ function syncTemplates(projectRoot, templateRootOverride) {
     report.managed.removed.push(norm(path.relative(projectRoot, target)));
   }
 
+  // Cleanup files owned by disabled built-in TUIs. Iterates managed + merged
+  // only (ejected entries are explicitly user-retained, see ejected loop below).
+  //
+  // Protection rule: only skip files registered as customTUI command targets
+  // (e.g. a customTUI configured with dir=.codex/commands/ when codex is
+  // disabled). Built-in TUI custom-skill commands like
+  // .gemini/commands/<project>/<dirName>.toml are intentionally NOT protected
+  // here so that disabling gemini-cli actually frees the gemini directory —
+  // they will be regenerated only for still-enabled TUIs (see
+  // generateCustomCommands).
+  for (const entry of [...managed, ...merged]) {
+    if (!isPathOwnedByDisabledTUI(entry, enabledTUIs)) continue;
+
+    if (entry.endsWith('/')) {
+      const dir = path.join(projectRoot, entry);
+      if (!fs.existsSync(dir)) continue;
+
+      for (const filePath of walkDir(dir)) {
+        const relProj = norm(path.relative(projectRoot, filePath));
+        if (customTUICommandTargets.has(relProj)) continue;
+        fs.unlinkSync(filePath);
+        report.managed.removed.push(relProj);
+      }
+      removeEmptyDirs(dir);
+      continue;
+    }
+
+    const target = path.join(projectRoot, renderPathname(entry, project));
+    if (!fs.existsSync(target)) continue;
+    const relProj = norm(path.relative(projectRoot, target));
+    if (customTUICommandTargets.has(relProj)) continue;
+    fs.unlinkSync(target);
+    report.managed.removed.push(relProj);
+  }
+
   for (const entry of managed) {
     if (isPathOwnedByOtherPlatform(entry, platformType)) {
       report.managed.skippedPlatform.push(entry);
       continue;
     }
+    if (isPathOwnedByDisabledTUI(entry, enabledTUIs)) {
+      report.managed.skippedTUI.push(entry);
+      continue;
+    }
 
     const isDir = entry.endsWith('/');
     let entryRels;
@@ -1140,7 +1224,7 @@ function syncTemplates(projectRoot, templateRootOverride) {
 
   const customSkills = detectCustomSkills(projectRoot, templateSkillNames);
   report.custom.detected = customSkills.map((skill) => skill.dirName);
-  generateCustomCommands(projectRoot, customSkills, project, lang, report, customTUIs, templateSkillNames);
+  generateCustomCommands(projectRoot, customSkills, project, lang, report, customTUIs, templateSkillNames, enabledTUIs);
 
   for (const entry of ejected) {
     const dstFull = path.join(projectRoot, entry);
@@ -1148,6 +1232,9 @@ function syncTemplates(projectRoot, templateRootOverride) {
       report.ejected.skipped.push(entry);
       continue;
     }
+    // Do not (re)create ejected files for disabled TUIs. Existing files are
+    // never touched by sync (handled above); this guard only blocks creation.
+    if (isPathOwnedByDisabledTUI(entry, enabledTUIs)) continue;
 
     const selected = platformSelect(langSelect(entryVariantRels(entry, allSet, platformType), lang, allSet, project), platformType, project);
     const target = norm(renderPathname(entry, project));
@@ -1168,6 +1255,10 @@ function syncTemplates(projectRoot, templateRootOverride) {
       report.managed.skippedPlatform.push(entry);
       continue;
     }
+    if (isPathOwnedByDisabledTUI(entry, enabledTUIs)) {
+      report.managed.skippedTUI.push(entry);
+      continue;
+    }
 
     if (entry.includes('*')) {
       const hits = allRels.filter(r => {

package/templates/.agents/workflows/bug-fix.en.yaml

@@ -139,7 +139,7 @@ steps:
           rule: "Read the highest-round plan review artifact and confirm feedback is approved or handled"
         - name: review-code
           pattern: "review-code.md | review-code-r{N}.md"
-          rule: "In fix mode, read the highest-round code review artifact and verify it matches the latest Code Review entry in task.md Activity Log"
+          rule: "In fix mode, read the highest-round code review artifact and verify it matches the latest Review Code entry in task.md Activity Log"
       outputs:
         - name: code
           pattern: "code.md | code-r{N}.md"

package/templates/.agents/workflows/bug-fix.zh-CN.yaml

@@ -139,7 +139,7 @@ steps:
           rule: "读取最高轮次的方案审查产物，确认方案已获通过或已处理反馈"
         - name: review-code
           pattern: "review-code.md | review-code-r{N}.md"
-          rule: "修复模式读取最高轮次的代码审查产物，并校验其与 task.md Activity Log 中最近一条 Code Review 记录一致"
+          rule: "修复模式读取最高轮次的代码审查产物，并校验其与 task.md Activity Log 中最近一条 Review Code 记录一致"
       outputs:
         - name: code
           pattern: "code.md | code-r{N}.md"

package/templates/.agents/workflows/feature-development.en.yaml

@@ -139,7 +139,7 @@ steps:
           rule: "Read the highest-round plan review artifact and confirm feedback is approved or handled"
         - name: review-code
           pattern: "review-code.md | review-code-r{N}.md"
-          rule: "In fix mode, read the highest-round code review artifact and verify it matches the latest Code Review entry in task.md Activity Log"
+          rule: "In fix mode, read the highest-round code review artifact and verify it matches the latest Review Code entry in task.md Activity Log"
       outputs:
         - name: code
           pattern: "code.md | code-r{N}.md"

package/templates/.agents/workflows/feature-development.zh-CN.yaml

@@ -139,7 +139,7 @@ steps:
           rule: "读取最高轮次的方案审查产物，确认方案已获通过或已处理反馈"
         - name: review-code
           pattern: "review-code.md | review-code-r{N}.md"
-          rule: "修复模式读取最高轮次的代码审查产物，并校验其与 task.md Activity Log 中最近一条 Code Review 记录一致"
+          rule: "修复模式读取最高轮次的代码审查产物，并校验其与 task.md Activity Log 中最近一条 Review Code 记录一致"
       outputs:
         - name: code
           pattern: "code.md | code-r{N}.md"

package/templates/.agents/workflows/refactoring.en.yaml

@@ -141,7 +141,7 @@ steps:
           rule: "Read the highest-round plan review artifact and confirm feedback is approved or handled"
         - name: review-code
           pattern: "review-code.md | review-code-r{N}.md"
-          rule: "In fix mode, read the highest-round code review artifact and verify it matches the latest Code Review entry in task.md Activity Log"
+          rule: "In fix mode, read the highest-round code review artifact and verify it matches the latest Review Code entry in task.md Activity Log"
       outputs:
         - name: code
           pattern: "code.md | code-r{N}.md"

package/templates/.agents/workflows/refactoring.zh-CN.yaml

@@ -141,7 +141,7 @@ steps:
           rule: "读取最高轮次的方案审查产物，确认方案已获通过或已处理反馈"
         - name: review-code
           pattern: "review-code.md | review-code-r{N}.md"
-          rule: "修复模式读取最高轮次的代码审查产物，并校验其与 task.md Activity Log 中最近一条 Code Review 记录一致"
+          rule: "修复模式读取最高轮次的代码审查产物，并校验其与 task.md Activity Log 中最近一条 Review Code 记录一致"
       outputs:
         - name: code
           pattern: "code.md | code-r{N}.md"

package/templates/.claude/settings.json

@@ -29,6 +29,17 @@
           }
         ]
       }
+    ],
+    "StopFailure": [
+      {
+        "matcher": "",
+        "hooks": [
+          {
+            "type": "command",
+            "command": "sh \"$(git rev-parse --show-toplevel)/.agents/hooks/auto-resume.sh\""
+          }
+        ]
+      }
     ]
   }
 }