opencode-swarm 7.29.0 → 7.29.1

package/dist/cli/index.js

@@ -34,7 +34,7 @@ var package_default;
 var init_package = __esm(() => {
   package_default = {
     name: "opencode-swarm",
-    version: "7.29.0",
+    version: "7.29.1",
     description: "Architect-centric agentic swarm plugin for OpenCode - hub-and-spoke orchestration with SME consultation, code generation, and QA review",
     main: "dist/index.js",
     types: "dist/index.d.ts",
@@ -40260,19 +40260,9 @@ function validateConfigKey(path24, value, _config) {
     case "guardrails.profiles": {
       const profiles = value;
       if (profiles) {
-        const validAgents = [
-          "architect",
-          "coder",
-          "test_engineer",
-          "explorer",
-          "reviewer",
-          "critic",
-          "sme",
-          "docs",
-          "designer"
-        ];
+        const validAgents = new Set(ALL_AGENT_NAMES);
         for (const [agentName, profile] of Object.entries(profiles)) {
-          if (!validAgents.includes(agentName)) {
+          if (!validAgents.has(agentName)) {
             findings.push({
               id: "unknown-agent-profile",
               title: "Unknown agent profile",
@@ -40433,23 +40423,23 @@ function validateConfigKey(path24, value, _config) {
     case "swarms": {
       const swarms = value;
       if (swarms && typeof swarms === "object") {
+        const validAgents = new Set(ALL_AGENT_NAMES);
         for (const [swarmId, swarmConfig] of Object.entries(swarms)) {
           const swarm = swarmConfig;
           if (swarm.agents && typeof swarm.agents === "object") {
             for (const [agentName] of Object.entries(swarm.agents)) {
-              const validAgents = [
-                "architect",
-                "coder",
-                "test_engineer",
-                "explorer",
-                "reviewer",
-                "critic",
-                "sme",
-                "docs",
-                "designer"
-              ];
-              const baseName = agentName.replace(/^[a-zA-Z0-9]+_/, "");
-              if (!validAgents.includes(baseName)) {
+              const baseName = stripKnownSwarmPrefix(agentName);
+              if (baseName !== agentName && agentName.startsWith(`${swarmId}_`) && validAgents.has(baseName)) {
+                findings.push({
+                  id: "prefixed-swarm-agent-override",
+                  title: "Prefixed agent override is ignored",
+                  description: `Agent "${agentName}" in swarm "${swarmId}" uses a generated agent name. ` + `Per-swarm overrides must use the canonical key "${baseName}", e.g. ` + `"swarms.${swarmId}.agents.${baseName}.model". Otherwise the override is ignored and the agent falls back to its default model.`,
+                  severity: "warn",
+                  path: `swarms.${swarmId}.agents.${agentName}`,
+                  currentValue: swarm.agents[agentName],
+                  autoFixable: false
+                });
+              } else if (!validAgents.has(baseName)) {
                 findings.push({
                   id: "unknown-swarm-agent",
                   title: "Unknown agent in swarm",
@@ -40800,6 +40790,8 @@ function removeStraySwarmDir(projectRoot, strayPath) {
 }
 var VALID_CONFIG_PATTERNS, DANGEROUS_PATH_SEGMENTS;
 var init_config_doctor = __esm(() => {
+  init_constants();
+  init_schema();
   init_utils();
   VALID_CONFIG_PATTERNS = [
     /^\.config[\\/]opencode[\\/]opencode-swarm\.json$/,

package/dist/index.js

@@ -48,7 +48,7 @@ var package_default;
 var init_package = __esm(() => {
   package_default = {
     name: "opencode-swarm",
-    version: "7.29.0",
+    version: "7.29.1",
     description: "Architect-centric agentic swarm plugin for OpenCode - hub-and-spoke orchestration with SME consultation, code generation, and QA review",
     main: "dist/index.js",
     types: "dist/index.d.ts",
@@ -61164,19 +61164,9 @@ function validateConfigKey(path31, value, _config) {
     case "guardrails.profiles": {
       const profiles = value;
       if (profiles) {
-        const validAgents = [
-          "architect",
-          "coder",
-          "test_engineer",
-          "explorer",
-          "reviewer",
-          "critic",
-          "sme",
-          "docs",
-          "designer"
-        ];
+        const validAgents = new Set(ALL_AGENT_NAMES);
         for (const [agentName, profile] of Object.entries(profiles)) {
-          if (!validAgents.includes(agentName)) {
+          if (!validAgents.has(agentName)) {
             findings.push({
               id: "unknown-agent-profile",
               title: "Unknown agent profile",
@@ -61337,23 +61327,23 @@ function validateConfigKey(path31, value, _config) {
     case "swarms": {
       const swarms = value;
       if (swarms && typeof swarms === "object") {
+        const validAgents = new Set(ALL_AGENT_NAMES);
         for (const [swarmId, swarmConfig] of Object.entries(swarms)) {
           const swarm = swarmConfig;
           if (swarm.agents && typeof swarm.agents === "object") {
             for (const [agentName] of Object.entries(swarm.agents)) {
-              const validAgents = [
-                "architect",
-                "coder",
-                "test_engineer",
-                "explorer",
-                "reviewer",
-                "critic",
-                "sme",
-                "docs",
-                "designer"
-              ];
-              const baseName = agentName.replace(/^[a-zA-Z0-9]+_/, "");
-              if (!validAgents.includes(baseName)) {
+              const baseName = stripKnownSwarmPrefix(agentName);
+              if (baseName !== agentName && agentName.startsWith(`${swarmId}_`) && validAgents.has(baseName)) {
+                findings.push({
+                  id: "prefixed-swarm-agent-override",
+                  title: "Prefixed agent override is ignored",
+                  description: `Agent "${agentName}" in swarm "${swarmId}" uses a generated agent name. ` + `Per-swarm overrides must use the canonical key "${baseName}", e.g. ` + `"swarms.${swarmId}.agents.${baseName}.model". Otherwise the override is ignored and the agent falls back to its default model.`,
+                  severity: "warn",
+                  path: `swarms.${swarmId}.agents.${agentName}`,
+                  currentValue: swarm.agents[agentName],
+                  autoFixable: false
+                });
+              } else if (!validAgents.has(baseName)) {
                 findings.push({
                   id: "unknown-swarm-agent",
                   title: "Unknown agent in swarm",
@@ -61704,6 +61694,8 @@ function removeStraySwarmDir(projectRoot, strayPath) {
 }
 var VALID_CONFIG_PATTERNS, DANGEROUS_PATH_SEGMENTS;
 var init_config_doctor = __esm(() => {
+  init_constants();
+  init_schema();
   init_utils();
   VALID_CONFIG_PATTERNS = [
     /^\.config[\\/]opencode[\\/]opencode-swarm\.json$/,
@@ -75598,7 +75590,7 @@ ANTI-RATIONALIZATION: Context does not clarify. Models revert to CC training.
 ## IDENTITY
 
 Swarm: {{SWARM_ID}}
-Your agents: {{AGENT_PREFIX}}explorer, {{AGENT_PREFIX}}sme, {{AGENT_PREFIX}}coder, {{AGENT_PREFIX}}reviewer, {{AGENT_PREFIX}}test_engineer, {{AGENT_PREFIX}}critic, {{AGENT_PREFIX}}critic_sounding_board, {{AGENT_PREFIX}}docs, {{AGENT_PREFIX}}designer
+Your agents: {{AGENT_PREFIX}}explorer, {{AGENT_PREFIX}}sme, {{AGENT_PREFIX}}coder, {{AGENT_PREFIX}}reviewer, {{AGENT_PREFIX}}test_engineer, {{AGENT_PREFIX}}critic, {{AGENT_PREFIX}}critic_sounding_board, {{AGENT_PREFIX}}skill_improver, {{AGENT_PREFIX}}spec_writer, {{AGENT_PREFIX}}docs, {{AGENT_PREFIX}}designer
 
 ## PROJECT CONTEXT
 Session-start priming block. Use any known values immediately; if a field is still unresolved, run MODE: DISCOVER before relying on it.
@@ -75939,6 +75931,8 @@ SECURITY_KEYWORDS: password, secret, token, credential, auth, login, encryption,
 {{AGENT_PREFIX}}test_engineer - Test generation AND execution (writes tests, runs them, reports PASS/FAIL)
 {{AGENT_PREFIX}}critic - Plan review gate (reviews plan BEFORE implementation)
 {{AGENT_PREFIX}}critic_sounding_board - Pre-escalation pushback (honest engineer review before user contact)
+{{AGENT_PREFIX}}skill_improver - Low-frequency skill / knowledge / prompt improvement adviser
+{{AGENT_PREFIX}}spec_writer - .swarm/spec.md authoring via spec_write
 {{AGENT_PREFIX}}docs - Documentation updates (README, API docs, guides — NOT .swarm/ files)
 {{AGENT_PREFIX}}designer - UI/UX design specs (scaffold generation for UI components — runs BEFORE coder on UI tasks)
 
@@ -75989,30 +75983,20 @@ For every applicable directive in the block:
 
 You may also call the \`knowledge_ack\` tool to record an outcome explicitly when chat-text markers would be ambiguous (e.g. inside structured tool args).
 
-## SKILL IMPROVER (low-frequency, expensive-model adviser)
+## SKILL IMPROVER
 
-The \`skill_improver\` agent and the \`skill_improve\` tool exist for rare, deep
-review of accumulated knowledge / skills / spec / architect prompt. They are
-quota-bounded (default 10 calls/day) and disabled by default. Suggest running
-\`skill_improve\` only after one of:
-- repeated reviewer rejections in a row,
-- many \`KNOWLEDGE_IGNORED\` outcomes for the same cluster,
-- stale skills (no updates while their target area changed),
-- a fresh spec mismatch with shipped behaviour.
+\`{{AGENT_PREFIX}}skill_improver\` / \`skill_improve\`: rare, quota-bounded,
+disabled by default, proposal-only. Use for repeated rejections,
+\`KNOWLEDGE_IGNORED\`, stale skills, or spec drift.
 
 When \`skill_improver.require_user_approval\` is true (default), ASK the user
 before running. Default outputs are proposals only — they never modify source.
 
 ## SPEC WRITER
 
-For substantial spec authoring or revision, prefer delegating to the
-\`spec_writer\` agent (independent model from architect). It writes only via
-the safe \`spec_write\` tool. Use it when:
-- the user requests a new spec or major spec revision,
-- requirements decomposition is non-trivial,
-- you would otherwise inline-author \`.swarm/spec.md\` yourself.
-
-Continue handling small touch-ups (typos, cross-references) inline.
+For substantial spec authoring/revision, prefer \`{{AGENT_PREFIX}}spec_writer\`;
+it writes via \`spec_write\`. Use for major specs or non-trivial
+decomposition. Handle small touch-ups.
 
 ### ANTI-RATIONALIZATION
 - ✗ "The coder already knows these conventions" → Skills contain project-specific rules the model cannot know from training. Always pass.
@@ -76193,11 +76177,12 @@ MODE: BRAINSTORM runs seven phases in strict order. Do not skip phases. Do not c
 - Exit with a design outline the user can skim in under two minutes.
 
 **Phase 5: SPEC WRITE + SELF-REVIEW (architect + reviewer).**
-- Generate \`.swarm/spec.md\` following the same SPEC CONTENT RULES that MODE: SPECIFY uses: WHAT/WHY only, no tech stack, no implementation details, FR-### / SC-### numbering, Given/When/Then scenarios, \`[NEEDS CLARIFICATION]\` markers (max 3).
+- Delegate substantial spec drafting to \`{{AGENT_PREFIX}}spec_writer\` with the chosen design, dialogue notes, SME context, and SPEC CONTENT RULES. The spec writer must persist \`.swarm/spec.md\` through \`spec_write\`.
+- The spec must follow the same SPEC CONTENT RULES that MODE: SPECIFY uses: WHAT/WHY only, no tech stack, no implementation details, FR-### / SC-### numbering, Given/When/Then scenarios, \`[NEEDS CLARIFICATION]\` markers (max 3).
 - Cross-reference design sections by name where relevant context helps (but keep HOW out of the spec).
 - Delegate to \`{{AGENT_PREFIX}}reviewer\` for an independent review of the draft spec. Reviewer must flag: requirements that encode HOW, untestable requirements, missing edge cases, silent assumptions.
 - Apply reviewer feedback. If reviewer rejects, iterate once and re-review. After two rounds, surface remaining disagreements to the user.
-- Write the final spec to \`.swarm/spec.md\`.
+- Read back and lint the final spec after \`{{AGENT_PREFIX}}spec_writer\` writes it.
 - Exit when reviewer signs off (or user explicitly accepts remaining disagreements).
 
 **Phase 6: QA GATE SELECTION (architect, dialogue only).**
@@ -76269,7 +76254,7 @@ Activates when: user asks to "specify", "define requirements", "write a spec", o
 1b. Run CODEBASE REALITY CHECK for any codebase references mentioned by the user or implied by the feature. Skip if work is purely greenfield (no existing codebase to check). Report discrepancies before proceeding to explorer.
 2. Delegate to \`{{AGENT_PREFIX}}explorer\` to scan the codebase for relevant context (existing patterns, related code, affected areas).
 3. Delegate to \`{{AGENT_PREFIX}}sme\` for domain research on the feature area to surface known constraints, best practices, and integration concerns.
-4. Generate \`.swarm/spec.md\` capturing:
+4. Delegate substantial spec drafting to \`{{AGENT_PREFIX}}spec_writer\`. Include the user requirements, explorer findings, SME constraints, and these required contents:
    - First line must be: \`# Specification: <feature-name>\`
    - Feature description: WHAT users need and WHY — never HOW to implement
    - User scenarios with acceptance criteria (Given/When/Then format)
@@ -76278,7 +76263,7 @@ Activates when: user asks to "specify", "define requirements", "write a spec", o
    - Key entities if data is involved (no schema or field definitions — entity names only)
    - Edge cases and known failure modes
    - \`[NEEDS CLARIFICATION]\` markers (max 3) for items where uncertainty could change scope, security, or core behavior; prefer informed defaults over asking
-5. Write the spec to \`.swarm/spec.md\`.
+5. Require \`{{AGENT_PREFIX}}spec_writer\` to write the spec via \`spec_write\`, then read back and lint \`.swarm/spec.md\`.
 5b. **QA GATE SELECTION (dialogue only).**
 {{QA_GATE_DIALOGUE_SPECIFY}}
 
@@ -79481,8 +79466,24 @@ function createSwarmAgents(swarmId, swarmConfig, isDefault, pluginConfig, projec
     const swarmIdentity = isDefault ? "default" : swarmId;
     const agentPrefix = prefix;
     architect.config.prompt = architect.config.prompt?.replace(/\{\{SWARM_ID\}\}/g, swarmIdentity).replace(/\{\{AGENT_PREFIX\}\}/g, agentPrefix).replace(/\{\{QA_RETRY_LIMIT\}\}/g, String(qaRetryLimit)).replace(/\{\{PROJECT_LANGUAGE\}\}/g, projectContext.PROJECT_LANGUAGE).replace(/\{\{PROJECT_FRAMEWORK\}\}/g, projectContext.PROJECT_FRAMEWORK).replace(/\{\{BUILD_CMD\}\}/g, projectContext.BUILD_CMD).replace(/\{\{TEST_CMD\}\}/g, projectContext.TEST_CMD).replace(/\{\{LINT_CMD\}\}/g, projectContext.LINT_CMD).replace(/\{\{ENTRY_POINTS\}\}/g, projectContext.ENTRY_POINTS).replace(/\{\{CODER_CONSTRAINTS\}\}/g, projectContext.CODER_CONSTRAINTS).replace(/\{\{TEST_CONSTRAINTS\}\}/g, projectContext.TEST_CONSTRAINTS).replace(/\{\{REVIEWER_CHECKLIST\}\}/g, projectContext.REVIEWER_CHECKLIST).replace(/\{\{PROJECT_CONTEXT_SECONDARY_LANGUAGES\}\}/g, projectContext.PROJECT_CONTEXT_SECONDARY_LANGUAGES);
+    const skillImproverEnabled = !isAgentDisabled("skill_improver", swarmAgents, swarmPrefix);
+    const specWriterEnabled = !isAgentDisabled("spec_writer", swarmAgents, swarmPrefix);
+    if (!skillImproverEnabled) {
+      architect.config.prompt = architect.config.prompt?.replace(`, ${agentPrefix}skill_improver`, "").replace(`
+${agentPrefix}skill_improver - Low-frequency skill / knowledge / prompt improvement adviser`, "").replace(/\n## SKILL IMPROVER[\s\S]*?(?=\n\n## SPEC WRITER)/, "");
+    }
+    if (!specWriterEnabled) {
+      const escapedAgentPrefix = agentPrefix.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
+      architect.config.prompt = architect.config.prompt?.replace(`, ${agentPrefix}spec_writer`, "").replace(`
+${agentPrefix}spec_writer - .swarm/spec.md authoring via spec_write`, "").replace(/\n## SPEC WRITER[\s\S]*?(?=\n\n### ANTI-RATIONALIZATION)/, "").replace(new RegExp(`- Delegate substantial spec drafting to \`${escapedAgentPrefix}spec_writer\`[^\\n]*\\n`, "g"), "- spec_writer is disabled. Ask the user to enable the spec_writer agent before creating or revising `.swarm/spec.md`.\n").replace(new RegExp(`4\\. Delegate substantial spec drafting to \`${escapedAgentPrefix}spec_writer\`[^\\n]*`), "4. spec_writer is disabled. Ask the user to enable the spec_writer agent before creating or revising `.swarm/spec.md`.").replace(new RegExp(`5\\. Require \`${escapedAgentPrefix}spec_writer\` to write the spec via \`spec_write\`, then read back and lint \`\\.swarm/spec\\.md\`\\.`), "5. Do not continue SPECIFY until spec_writer is available.").replace(new RegExp(`- Read back and lint the final spec after \`${escapedAgentPrefix}spec_writer\` writes it\\.`), "- Do not continue BRAINSTORM spec writing until spec_writer is available.");
+    }
     if (!isDefault) {
       architect.description = `[${swarmName}] ${architect.description}`;
+      const optionalAgentLines = [
+        skillImproverEnabled ? `- @${swarmId}_skill_improver (not @skill_improver)` : undefined,
+        specWriterEnabled ? `- @${swarmId}_spec_writer (not @spec_writer)` : undefined
+      ].filter((line) => Boolean(line)).join(`
+`);
       const swarmHeader = `## ⚠️ YOU ARE THE ${swarmName.toUpperCase()} SWARM ARCHITECT
 
 Your swarm ID is "${swarmId}". ALL your agents have the "${swarmId}_" prefix:
@@ -79490,7 +79491,8 @@ Your swarm ID is "${swarmId}". ALL your agents have the "${swarmId}_" prefix:
 - @${swarmId}_coder (not @coder)
 - @${swarmId}_sme (not @sme)
 - @${swarmId}_reviewer (not @reviewer)
-- etc.
+${optionalAgentLines ? `${optionalAgentLines}
+` : ""}- etc.
 
 CRITICAL: Agents without the "${swarmId}_" prefix DO NOT EXIST or belong to a DIFFERENT swarm.
 If you call @coder instead of @${swarmId}_coder, the call will FAIL or go to the wrong swarm.

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "opencode-swarm",
-	"version": "7.29.0",
+	"version": "7.29.1",
 	"description": "Architect-centric agentic swarm plugin for OpenCode - hub-and-spoke orchestration with SME consultation, code generation, and QA review",
 	"main": "dist/index.js",
 	"types": "dist/index.d.ts",