chati-dev 4.2.1 → 4.2.2

package/src/orchestrator/cli.js

@@ -381,24 +381,33 @@ async function handleNext(projectDir) {
   // Check for parallel group — with Agent Teams override (Article XXI)
   let action, spawnCommand = null, parallelSpawnCommand = null, parallelAgents = [];
   let teamData = null;
+  const teamsEnabled = isAgentTeamsEnabled(projectDir);
+
   if (nextInfo.isParallel && nextInfo.group.length > 1) {
-    // When agent_teams is enabled, substitute spawn_parallel with spawn_team
-    if (isAgentTeamsEnabled(projectDir)) {
-      const teamType = nextInfo.group.includes('dev') ? 'build' : 'planning';
-      const teamConfig = TEAM_CONFIGS[teamType];
+    // Planning Team auto-spawn: detail + architect + ux are marked parallel: true
+    // in agent-selector. When agent_teams enabled, swap spawn_parallel for spawn_team.
+    if (teamsEnabled) {
+      const teamConfig = TEAM_CONFIGS.planning;
       const teamId = generateTeamId(teamConfig.slug);
       action = 'spawn_team';
       parallelAgents = teamConfig.members;
-      teamData = {
-        team_id: teamId,
-        team_type: teamType,
-        members: teamConfig.members,
-      };
+      teamData = { team_id: teamId, team_type: 'planning', members: teamConfig.members };
     } else {
       action = 'spawn_parallel';
       parallelAgents = nextInfo.group;
       parallelSpawnCommand = buildParallelSpawnCommand(nextInfo.group, projectDir, lastAgent, modelInfo.provider, 900000);
     }
+  } else if (nextAgent === 'dev' && teamsEnabled) {
+    // Build Team auto-spawn: dev + qa-implementation are NOT parallel in the
+    // classic sense (dev writes, qa reviews per-task), so they're not in a
+    // parallel group. But under Agent Teams, they coordinate via mailbox + shared
+    // task list, which IS the team mechanism. Spawn Build Team here when dev
+    // would otherwise spawn solo.
+    const teamConfig = TEAM_CONFIGS.build;
+    const teamId = generateTeamId(teamConfig.slug);
+    action = 'spawn_team';
+    parallelAgents = teamConfig.members;
+    teamData = { team_id: teamId, team_type: 'build', members: teamConfig.members };
   } else if (isInteractive) {
     action = 'activate_interactive';
   } else {
@@ -978,12 +987,33 @@ function generateTeamId(slug) {
   return `TM-${date}-${slug}`;
 }
 
+/**
+ * Agent Teams require BOTH the feature flag AND a Claude provider.
+ *
+ * Why provider gate: Agent Teams uses Claude Code's native Task tool / subagent
+ * spawn API. Gemini and Codex have no equivalent — calling spawn_team for those
+ * providers would fail. We always fall back silently to spawn_parallel
+ * (Gemini) or spawn_autonomous (Codex) when the active provider is not claude.
+ */
 function isAgentTeamsEnabled(projectDir) {
   const configPath = join(projectDir, 'chati.dev', 'config.yaml');
   if (!existsSync(configPath)) return false;
   const raw = readFileSync(configPath, 'utf-8');
-  const match = raw.match(/agent_teams:\s*(true|false)/);
-  return match ? match[1] === 'true' : false;
+
+  // Feature flag check
+  const flagMatch = raw.match(/agent_teams:\s*(true|false)/);
+  const flagOn = flagMatch ? flagMatch[1] === 'true' : false;
+  if (!flagOn) return false;
+
+  // Provider gate — Agent Teams is Claude-only.
+  // Read primary_provider from session.yaml (preferred) or active_provider
+  // field. Default to 'claude' if neither exists (early bootstrap).
+  const sessionPath = join(projectDir, '.chati', 'session.yaml');
+  if (!existsSync(sessionPath)) return true; // No session yet — assume claude
+  const sessionRaw = readFileSync(sessionPath, 'utf-8');
+  const providerMatch = sessionRaw.match(/^\s*(?:active_provider|primary_provider):\s*["']?(\w+)/m);
+  const provider = providerMatch ? providerMatch[1] : 'claude';
+  return provider === 'claude';
 }
 
 async function handleSpawnTeam(projectDir, args) {