@agents-inc/cli 0.35.0 → 0.41.1

package/dist/stores/wizard-store.test.js

@@ -1,7 +1,8 @@
 #!/usr/bin/env node
 import {
-  createMockCategory
-} from "../chunk-YN35L5NE.js";
+  createMockMatrix,
+  createMockSkill
+} from "../chunk-AH7XHAKN.js";
 import {
   beforeEach,
   describe,
@@ -10,13 +11,16 @@ import {
 } from "../chunk-XY3XDVMI.js";
 import {
   useWizardStore
-} from "../chunk-OGXSTJP2.js";
-import "../chunk-WMVGRAFB.js";
-import "../chunk-T4EXUIBY.js";
-import "../chunk-OI4WBRC7.js";
+} from "../chunk-423MJ6DT.js";
+import "../chunk-SILUTTV7.js";
+import "../chunk-TTXV55NQ.js";
+import {
+  typedKeys
+} from "../chunk-T4EXUIBY.js";
+import "../chunk-XJXJZ2MJ.js";
 import {
   DEFAULT_PRESELECTED_SKILLS
-} from "../chunk-YCS7GF6Y.js";
+} from "../chunk-BK7TANUV.js";
 import {
   init_esm_shims
 } from "../chunk-DHET7RCE.js";
@@ -151,41 +155,41 @@ describe("WizardStore", () => {
   describe("technology selection", () => {
     it("should toggle technology in exclusive mode", () => {
       const store = useWizardStore.getState();
-      store.toggleTechnology("web", "framework", "web-framework-react", true);
+      store.toggleTechnology("web", "web-framework", "web-framework-react", true);
       const { domainSelections } = useWizardStore.getState();
-      globalExpect(domainSelections.web.framework).toEqual(["web-framework-react"]);
+      globalExpect(domainSelections.web["web-framework"]).toEqual(["web-framework-react"]);
     });
     it("should replace technology in exclusive mode", () => {
       const store = useWizardStore.getState();
-      store.toggleTechnology("web", "framework", "web-framework-react", true);
-      store.toggleTechnology("web", "framework", "web-framework-vue", true);
+      store.toggleTechnology("web", "web-framework", "web-framework-react", true);
+      store.toggleTechnology("web", "web-framework", "web-framework-vue", true);
       const { domainSelections } = useWizardStore.getState();
-      globalExpect(domainSelections.web.framework).toEqual(["web-framework-vue"]);
+      globalExpect(domainSelections.web["web-framework"]).toEqual(["web-framework-vue"]);
     });
     it("should toggle off technology in exclusive mode", () => {
       const store = useWizardStore.getState();
-      store.toggleTechnology("web", "framework", "web-framework-react", true);
-      store.toggleTechnology("web", "framework", "web-framework-react", true);
+      store.toggleTechnology("web", "web-framework", "web-framework-react", true);
+      store.toggleTechnology("web", "web-framework", "web-framework-react", true);
       const { domainSelections } = useWizardStore.getState();
-      globalExpect(domainSelections.web.framework).toEqual([]);
+      globalExpect(domainSelections.web["web-framework"]).toEqual([]);
     });
     it("should allow multiple selections in non-exclusive mode", () => {
       const store = useWizardStore.getState();
-      store.toggleTechnology("web", "testing", "web-testing-vitest", false);
-      store.toggleTechnology("web", "testing", "web-testing-playwright-e2e", false);
+      store.toggleTechnology("web", "web-testing", "web-testing-vitest", false);
+      store.toggleTechnology("web", "web-testing", "web-testing-playwright-e2e", false);
       const { domainSelections } = useWizardStore.getState();
-      globalExpect(domainSelections.web.testing).toEqual([
+      globalExpect(domainSelections.web["web-testing"]).toEqual([
         "web-testing-vitest",
         "web-testing-playwright-e2e"
       ]);
     });
     it("should toggle off technology in non-exclusive mode", () => {
       const store = useWizardStore.getState();
-      store.toggleTechnology("web", "testing", "web-testing-vitest", false);
-      store.toggleTechnology("web", "testing", "web-testing-playwright-e2e", false);
-      store.toggleTechnology("web", "testing", "web-testing-vitest", false);
+      store.toggleTechnology("web", "web-testing", "web-testing-vitest", false);
+      store.toggleTechnology("web", "web-testing", "web-testing-playwright-e2e", false);
+      store.toggleTechnology("web", "web-testing", "web-testing-vitest", false);
       const { domainSelections } = useWizardStore.getState();
-      globalExpect(domainSelections.web.testing).toEqual(["web-testing-playwright-e2e"]);
+      globalExpect(domainSelections.web["web-testing"]).toEqual(["web-testing-playwright-e2e"]);
     });
   });
   describe("domain navigation", () => {
@@ -261,11 +265,11 @@ describe("WizardStore", () => {
       const { installMode } = useWizardStore.getState();
       globalExpect(installMode).toBe("local");
     });
-    it("should toggle show descriptions", () => {
+    it("should toggle show labels", () => {
       const store = useWizardStore.getState();
-      store.toggleShowDescriptions();
-      const { showDescriptions } = useWizardStore.getState();
-      globalExpect(showDescriptions).toBe(true);
+      store.toggleShowLabels();
+      const { showLabels } = useWizardStore.getState();
+      globalExpect(showLabels).toBe(true);
     });
     it("should toggle help on", () => {
       const store = useWizardStore.getState();
@@ -343,9 +347,9 @@ describe("WizardStore", () => {
   describe("computed getters", () => {
     it("should get all selected technologies", () => {
       const store = useWizardStore.getState();
-      store.toggleTechnology("web", "framework", "web-framework-react", true);
-      store.toggleTechnology("web", "styling", "web-styling-scss-modules", true);
-      store.toggleTechnology("api", "api", "api-framework-hono", true);
+      store.toggleTechnology("web", "web-framework", "web-framework-react", true);
+      store.toggleTechnology("web", "web-styling", "web-styling-scss-modules", true);
+      store.toggleTechnology("api", "api-api", "api-framework-hono", true);
       const technologies = store.getAllSelectedTechnologies();
       globalExpect(technologies).toContain("web-framework-react");
       globalExpect(technologies).toContain("web-styling-scss-modules");
@@ -360,9 +364,9 @@ describe("WizardStore", () => {
     });
     it("should get selected technologies per domain", () => {
       const store = useWizardStore.getState();
-      store.toggleTechnology("web", "framework", "web-framework-react", true);
-      store.toggleTechnology("web", "styling", "web-styling-scss-modules", true);
-      store.toggleTechnology("api", "api", "api-framework-hono", true);
+      store.toggleTechnology("web", "web-framework", "web-framework-react", true);
+      store.toggleTechnology("web", "web-styling", "web-styling-scss-modules", true);
+      store.toggleTechnology("api", "api-api", "api-framework-hono", true);
       const perDomain = store.getSelectedTechnologiesPerDomain();
       globalExpect(perDomain.web).toEqual(["web-framework-react", "web-styling-scss-modules"]);
       globalExpect(perDomain.api).toEqual(["api-framework-hono"]);
@@ -375,16 +379,16 @@ describe("WizardStore", () => {
     });
     it("should omit domains with empty subcategory arrays from getSelectedTechnologiesPerDomain", () => {
       const store = useWizardStore.getState();
-      store.toggleTechnology("web", "framework", "web-framework-react", true);
-      store.toggleTechnology("web", "framework", "web-framework-react", true);
+      store.toggleTechnology("web", "web-framework", "web-framework-react", true);
+      store.toggleTechnology("web", "web-framework", "web-framework-react", true);
       const perDomain = store.getSelectedTechnologiesPerDomain();
       globalExpect(perDomain.web).toBeUndefined();
     });
     it("should get technology count", () => {
       const store = useWizardStore.getState();
-      store.toggleTechnology("web", "framework", "web-framework-react", true);
-      store.toggleTechnology("web", "styling", "web-styling-scss-modules", true);
-      store.toggleTechnology("api", "api", "api-framework-hono", true);
+      store.toggleTechnology("web", "web-framework", "web-framework-react", true);
+      store.toggleTechnology("web", "web-styling", "web-styling-scss-modules", true);
+      store.toggleTechnology("api", "api-api", "api-framework-hono", true);
       globalExpect(store.getTechnologyCount()).toBe(3);
     });
     it("should return 0 for getTechnologyCount with no selections", () => {
@@ -426,65 +430,6 @@ describe("WizardStore", () => {
       globalExpect(store.canGoToPreviousDomain()).toBe(false);
     });
   });
-  describe("parent domain selectors", () => {
-    const matrixWithParentDomain = {
-      categories: {
-        "error-handling": createMockCategory("error-handling", "Error Handling", {
-          domain: "web-extras",
-          parent_domain: "web",
-          exclusive: false
-        }),
-        framework: createMockCategory("framework", "Framework", {
-          domain: "web"
-        })
-      }
-    };
-    const matrixWithoutParent = {
-      categories: {
-        framework: createMockCategory("framework", "Framework", {
-          domain: "web"
-        })
-      }
-    };
-    it("should find parent domain for a domain with parent_domain", () => {
-      const store = useWizardStore.getState();
-      const parent = store.getParentDomain(
-        "web-extras",
-        matrixWithParentDomain
-      );
-      globalExpect(parent).toBe("web");
-    });
-    it("should return undefined for a domain without parent_domain", () => {
-      const store = useWizardStore.getState();
-      const parent = store.getParentDomain("web", matrixWithoutParent);
-      globalExpect(parent).toBeUndefined();
-    });
-    it("should get parent domain selections when parent exists", () => {
-      const store = useWizardStore.getState();
-      store.toggleTechnology("web", "framework", "web-framework-react", true);
-      const parentSelections = store.getParentDomainSelections(
-        "web-extras",
-        matrixWithParentDomain
-      );
-      globalExpect(parentSelections).toEqual({ framework: ["web-framework-react"] });
-    });
-    it("should return undefined for getParentDomainSelections when no parent", () => {
-      const store = useWizardStore.getState();
-      const parentSelections = store.getParentDomainSelections(
-        "web",
-        matrixWithoutParent
-      );
-      globalExpect(parentSelections).toBeUndefined();
-    });
-    it("should return undefined for getParentDomainSelections when parent has no selections", () => {
-      const store = useWizardStore.getState();
-      const parentSelections = store.getParentDomainSelections(
-        "web-extras",
-        matrixWithParentDomain
-      );
-      globalExpect(parentSelections).toBeUndefined();
-    });
-  });
   describe("reset", () => {
     it("should reset to initial state", () => {
       const store = useWizardStore.getState();
@@ -509,17 +454,17 @@ describe("WizardStore", () => {
       const store = useWizardStore.getState();
       const stack = {
         agents: {
-          web: { framework: [sa("web-framework-react", true)] }
+          web: { "web-framework": [sa("web-framework-react", true)] }
         }
       };
       const categories = {
-        framework: { domain: "web" }
+        "web-framework": { domain: "web" }
       };
       store.populateFromStack(stack, categories);
       const { selectedDomains, domainSelections } = useWizardStore.getState();
-      globalExpect(selectedDomains).toEqual(["web", "web-extras", "api", "cli", "mobile", "shared"]);
+      globalExpect(selectedDomains).toEqual(["web", "api", "cli", "mobile", "shared"]);
       globalExpect(domainSelections.web).toBeDefined();
-      globalExpect(domainSelections.web.framework).toEqual(["web-framework-react"]);
+      globalExpect(domainSelections.web["web-framework"]).toEqual(["web-framework-react"]);
       globalExpect(domainSelections.api).toBeUndefined();
     });
     it("should populate domainSelections from stack agents", () => {
@@ -527,43 +472,43 @@ describe("WizardStore", () => {
       const stack = {
         agents: {
           web: {
-            framework: [sa("web-framework-react", true)],
-            "client-state": [sa("web-state-zustand")]
+            "web-framework": [sa("web-framework-react", true)],
+            "web-client-state": [sa("web-state-zustand")]
           },
-          api: { api: [sa("api-framework-hono", true)] }
+          api: { "api-api": [sa("api-framework-hono", true)] }
         }
       };
       const categories = {
-        framework: { domain: "web" },
-        "client-state": { domain: "web" },
-        api: { domain: "api" }
+        "web-framework": { domain: "web" },
+        "web-client-state": { domain: "web" },
+        "api-api": { domain: "api" }
       };
       store.populateFromStack(stack, categories);
       const { domainSelections } = useWizardStore.getState();
-      globalExpect(domainSelections.web.framework).toEqual(["web-framework-react"]);
-      globalExpect(domainSelections.web["client-state"]).toEqual(["web-state-zustand"]);
-      globalExpect(domainSelections.api.api).toEqual(["api-framework-hono"]);
+      globalExpect(domainSelections.web["web-framework"]).toEqual(["web-framework-react"]);
+      globalExpect(domainSelections.web["web-client-state"]).toEqual(["web-state-zustand"]);
+      globalExpect(domainSelections.api["api-api"]).toEqual(["api-framework-hono"]);
     });
     it("should skip entries without a domain", () => {
       const store = useWizardStore.getState();
       const stack = {
         agents: {
-          misc: { methodology: [sa("meta-methodology-vitest")] }
+          misc: { "shared-methodology": [sa("meta-methodology-vitest")] }
         }
       };
       const categories = {
-        methodology: {}
+        "shared-methodology": {}
       };
       store.populateFromStack(stack, categories);
       const { domainSelections } = useWizardStore.getState();
-      globalExpect(Object.keys(domainSelections)).toHaveLength(0);
+      globalExpect(typedKeys(domainSelections)).toHaveLength(0);
     });
     it("should populate multiple skills from array-valued subcategories", () => {
       const store = useWizardStore.getState();
       const stack = {
         agents: {
           "pattern-scout": {
-            methodology: [
+            "shared-methodology": [
               sa("meta-methodology-investigation-requirements", true),
               sa("meta-methodology-anti-over-engineering", true),
               sa("meta-methodology-success-criteria", true)
@@ -572,11 +517,11 @@ describe("WizardStore", () => {
         }
       };
       const categories = {
-        methodology: { domain: "shared" }
+        "shared-methodology": { domain: "shared" }
       };
       store.populateFromStack(stack, categories);
       const { domainSelections } = useWizardStore.getState();
-      globalExpect(domainSelections.shared.methodology).toEqual([
+      globalExpect(domainSelections.shared["shared-methodology"]).toEqual([
         "meta-methodology-investigation-requirements",
         "meta-methodology-anti-over-engineering",
         "meta-methodology-success-criteria"
@@ -587,41 +532,41 @@ describe("WizardStore", () => {
       const stack = {
         agents: {
           web: {
-            framework: [sa("web-framework-react", true)],
-            methodology: [
+            "web-framework": [sa("web-framework-react", true)],
+            "shared-methodology": [
               sa("meta-methodology-investigation-requirements", true),
               sa("meta-methodology-anti-over-engineering", true)
             ]
           },
-          api: { api: [sa("api-framework-hono", true)] }
+          api: { "api-api": [sa("api-framework-hono", true)] }
         }
       };
       const categories = {
-        framework: { domain: "web" },
-        methodology: { domain: "shared" },
-        api: { domain: "api" }
+        "web-framework": { domain: "web" },
+        "shared-methodology": { domain: "shared" },
+        "api-api": { domain: "api" }
       };
       store.populateFromStack(stack, categories);
       const { domainSelections } = useWizardStore.getState();
-      globalExpect(domainSelections.web.framework).toEqual(["web-framework-react"]);
-      globalExpect(domainSelections.shared.methodology).toEqual([
+      globalExpect(domainSelections.web["web-framework"]).toEqual(["web-framework-react"]);
+      globalExpect(domainSelections.shared["shared-methodology"]).toEqual([
         "meta-methodology-investigation-requirements",
         "meta-methodology-anti-over-engineering"
       ]);
-      globalExpect(domainSelections.api.api).toEqual(["api-framework-hono"]);
+      globalExpect(domainSelections.api["api-api"]).toEqual(["api-framework-hono"]);
     });
     it("should deduplicate skills from arrays across multiple agents", () => {
       const store = useWizardStore.getState();
       const stack = {
         agents: {
           agent1: {
-            methodology: [
+            "shared-methodology": [
               sa("meta-methodology-investigation-requirements", true),
               sa("meta-methodology-anti-over-engineering", true)
             ]
           },
           agent2: {
-            methodology: [
+            "shared-methodology": [
               sa("meta-methodology-anti-over-engineering", true),
               sa("meta-methodology-success-criteria", true)
             ]
@@ -629,11 +574,11 @@ describe("WizardStore", () => {
         }
       };
       const categories = {
-        methodology: { domain: "shared" }
+        "shared-methodology": { domain: "shared" }
       };
       store.populateFromStack(stack, categories);
       const { domainSelections } = useWizardStore.getState();
-      globalExpect(domainSelections.shared.methodology).toEqual([
+      globalExpect(domainSelections.shared["shared-methodology"]).toEqual([
         "meta-methodology-investigation-requirements",
         "meta-methodology-anti-over-engineering",
         "meta-methodology-success-criteria"
@@ -661,28 +606,283 @@ describe("WizardStore", () => {
       store.toggleDomain("web");
       store.toggleDomain("api");
       store.setStep("build");
-      store.toggleTechnology("web", "framework", "web-framework-react", true);
-      store.toggleTechnology("web", "styling", "web-styling-scss-modules", true);
-      store.toggleTechnology("api", "api", "api-framework-hono", true);
+      store.toggleTechnology("web", "web-framework", "web-framework-react", true);
+      store.toggleTechnology("web", "web-styling", "web-styling-scss-modules", true);
+      store.toggleTechnology("api", "api-api", "api-framework-hono", true);
       store.setStep("confirm");
       const state = useWizardStore.getState();
       globalExpect(state.step).toBe("confirm");
       globalExpect(state.approach).toBe("scratch");
       globalExpect(state.selectedDomains).toEqual(["web", "api"]);
-      globalExpect(state.domainSelections.web.framework).toEqual(["web-framework-react"]);
-      globalExpect(state.domainSelections.web.styling).toEqual(["web-styling-scss-modules"]);
-      globalExpect(state.domainSelections.api.api).toEqual(["api-framework-hono"]);
+      globalExpect(state.domainSelections.web["web-framework"]).toEqual(["web-framework-react"]);
+      globalExpect(state.domainSelections.web["web-styling"]).toEqual(["web-styling-scss-modules"]);
+      globalExpect(state.domainSelections.api["api-api"]).toEqual(["api-framework-hono"]);
     });
     it("should preserve selections when going back", () => {
       const store = useWizardStore.getState();
       store.setApproach("scratch");
       store.toggleDomain("web");
       store.setStep("build");
-      store.toggleTechnology("web", "framework", "web-framework-react", true);
+      store.toggleTechnology("web", "web-framework", "web-framework-react", true);
       store.goBack();
       const state = useWizardStore.getState();
       globalExpect(state.selectedDomains).toContain("web");
-      globalExpect(state.domainSelections.web.framework).toEqual(["web-framework-react"]);
+      globalExpect(state.domainSelections.web["web-framework"]).toEqual(["web-framework-react"]);
+    });
+  });
+  describe("buildSourceRows sort order", () => {
+    function makeSource(overrides) {
+      return {
+        type: "private",
+        installed: false,
+        ...overrides
+      };
+    }
+    it("should sort local sources before scoped marketplace sources", () => {
+      const store = useWizardStore.getState();
+      const skill = createMockSkill("web-framework-react", "web-framework", {
+        displayName: "react",
+        availableSources: [
+          makeSource({ name: "Photoroom", type: "private", primary: true }),
+          makeSource({ name: "local", type: "local", installed: true, installMode: "local" })
+        ]
+      });
+      const matrix = createMockMatrix(
+        { "web-framework-react": skill },
+        {
+          displayNames: { "web-framework-react": "react" }
+        }
+      );
+      store.toggleTechnology("web", "web-framework", "web-framework-react", true);
+      const rows = store.buildSourceRows(matrix);
+      globalExpect(rows).toHaveLength(1);
+      globalExpect(rows[0].options[0].id).toBe("local");
+      globalExpect(rows[0].options[1].id).toBe("Photoroom");
+    });
+    it("should sort scoped marketplace before default public marketplace", () => {
+      const store = useWizardStore.getState();
+      const skill = createMockSkill("web-framework-react", "web-framework", {
+        displayName: "react",
+        availableSources: [
+          makeSource({ name: "Agents Inc", type: "public" }),
+          makeSource({ name: "Photoroom", type: "private", primary: true })
+        ]
+      });
+      const matrix = createMockMatrix(
+        { "web-framework-react": skill },
+        {
+          displayNames: { "web-framework-react": "react" }
+        }
+      );
+      store.toggleTechnology("web", "web-framework", "web-framework-react", true);
+      const rows = store.buildSourceRows(matrix);
+      globalExpect(rows).toHaveLength(1);
+      globalExpect(rows[0].options[0].id).toBe("Photoroom");
+      globalExpect(rows[0].options[1].id).toBe("Agents Inc");
+    });
+    it("should sort default public marketplace before third-party sources", () => {
+      const store = useWizardStore.getState();
+      const skill = createMockSkill("web-framework-react", "web-framework", {
+        displayName: "react",
+        availableSources: [
+          makeSource({ name: "Extra Corp", type: "private" }),
+          makeSource({ name: "Agents Inc", type: "public" })
+        ]
+      });
+      const matrix = createMockMatrix(
+        { "web-framework-react": skill },
+        {
+          displayNames: { "web-framework-react": "react" }
+        }
+      );
+      store.toggleTechnology("web", "web-framework", "web-framework-react", true);
+      const rows = store.buildSourceRows(matrix);
+      globalExpect(rows).toHaveLength(1);
+      globalExpect(rows[0].options[0].id).toBe("Agents Inc");
+      globalExpect(rows[0].options[1].id).toBe("Extra Corp");
+    });
+    it("should sort all four tiers in correct order", () => {
+      const store = useWizardStore.getState();
+      const skill = createMockSkill("web-framework-react", "web-framework", {
+        displayName: "react",
+        availableSources: [
+          makeSource({ name: "Extra Corp", type: "private" }),
+          makeSource({ name: "Agents Inc", type: "public" }),
+          makeSource({ name: "Photoroom", type: "private", primary: true }),
+          makeSource({ name: "local", type: "local", installed: true, installMode: "local" })
+        ]
+      });
+      const matrix = createMockMatrix(
+        { "web-framework-react": skill },
+        {
+          displayNames: { "web-framework-react": "react" }
+        }
+      );
+      store.toggleTechnology("web", "web-framework", "web-framework-react", true);
+      const rows = store.buildSourceRows(matrix);
+      globalExpect(rows).toHaveLength(1);
+      const sourceNames = rows[0].options.map((opt) => opt.id);
+      globalExpect(sourceNames).toEqual(["local", "Photoroom", "Agents Inc", "Extra Corp"]);
+    });
+  });
+  describe("agent selection", () => {
+    it("should start with empty selectedAgents", () => {
+      const { selectedAgents } = useWizardStore.getState();
+      globalExpect(selectedAgents).toEqual([]);
+    });
+    it("should toggle agent on", () => {
+      const store = useWizardStore.getState();
+      store.toggleAgent("web-developer");
+      const { selectedAgents } = useWizardStore.getState();
+      globalExpect(selectedAgents).toContain("web-developer");
+    });
+    it("should toggle agent off", () => {
+      const store = useWizardStore.getState();
+      store.toggleAgent("web-developer");
+      store.toggleAgent("web-developer");
+      const { selectedAgents } = useWizardStore.getState();
+      globalExpect(selectedAgents).not.toContain("web-developer");
+    });
+    it("should allow multiple agents to be selected", () => {
+      const store = useWizardStore.getState();
+      store.toggleAgent("web-developer");
+      store.toggleAgent("api-developer");
+      store.toggleAgent("web-reviewer");
+      const { selectedAgents } = useWizardStore.getState();
+      globalExpect(selectedAgents).toEqual(["web-developer", "api-developer", "web-reviewer"]);
+    });
+    it("should reset selectedAgents on reset", () => {
+      const store = useWizardStore.getState();
+      store.toggleAgent("web-developer");
+      store.reset();
+      const { selectedAgents } = useWizardStore.getState();
+      globalExpect(selectedAgents).toEqual([]);
+    });
+  });
+  describe("preselectAgentsFromDomains", () => {
+    it("should preselect web-related agents when web domain is selected", () => {
+      const store = useWizardStore.getState();
+      store.toggleDomain("web");
+      store.preselectAgentsFromDomains();
+      const { selectedAgents } = useWizardStore.getState();
+      globalExpect(selectedAgents).toContain("web-developer");
+      globalExpect(selectedAgents).toContain("web-reviewer");
+      globalExpect(selectedAgents).toContain("web-researcher");
+      globalExpect(selectedAgents).toContain("web-tester");
+      globalExpect(selectedAgents).toContain("web-pm");
+      globalExpect(selectedAgents).toContain("web-architecture");
+    });
+    it("should preselect api-related agents when api domain is selected", () => {
+      const store = useWizardStore.getState();
+      store.toggleDomain("api");
+      store.preselectAgentsFromDomains();
+      const { selectedAgents } = useWizardStore.getState();
+      globalExpect(selectedAgents).toContain("api-developer");
+      globalExpect(selectedAgents).toContain("api-reviewer");
+      globalExpect(selectedAgents).toContain("api-researcher");
+      globalExpect(selectedAgents).not.toContain("web-developer");
+    });
+    it("should preselect cli agents when cli domain is selected", () => {
+      const store = useWizardStore.getState();
+      store.toggleDomain("cli");
+      store.preselectAgentsFromDomains();
+      const { selectedAgents } = useWizardStore.getState();
+      globalExpect(selectedAgents).toContain("cli-developer");
+      globalExpect(selectedAgents).toContain("cli-tester");
+      globalExpect(selectedAgents).toContain("cli-reviewer");
+      globalExpect(selectedAgents).toContain("cli-migrator");
+    });
+    it("should never include optional agents regardless of domains", () => {
+      const store = useWizardStore.getState();
+      store.toggleDomain("web");
+      store.toggleDomain("api");
+      store.toggleDomain("cli");
+      store.preselectAgentsFromDomains();
+      const { selectedAgents } = useWizardStore.getState();
+      globalExpect(selectedAgents).not.toContain("agent-summoner");
+      globalExpect(selectedAgents).not.toContain("skill-summoner");
+      globalExpect(selectedAgents).not.toContain("documentor");
+      globalExpect(selectedAgents).not.toContain("pattern-scout");
+      globalExpect(selectedAgents).not.toContain("web-pattern-critique");
+    });
+    it("should return empty agents when no domains are selected", () => {
+      const store = useWizardStore.getState();
+      store.preselectAgentsFromDomains();
+      const { selectedAgents } = useWizardStore.getState();
+      globalExpect(selectedAgents).toEqual([]);
+    });
+    it("should produce union of agents for multiple domains", () => {
+      const store = useWizardStore.getState();
+      store.toggleDomain("web");
+      store.toggleDomain("api");
+      store.preselectAgentsFromDomains();
+      const { selectedAgents } = useWizardStore.getState();
+      globalExpect(selectedAgents).toContain("web-developer");
+      globalExpect(selectedAgents).toContain("api-developer");
+      globalExpect(selectedAgents).toContain("web-reviewer");
+      globalExpect(selectedAgents).toContain("api-reviewer");
+    });
+    it("should not preselect api agents when only web domain is selected", () => {
+      const store = useWizardStore.getState();
+      store.toggleDomain("web");
+      store.preselectAgentsFromDomains();
+      const { selectedAgents } = useWizardStore.getState();
+      globalExpect(selectedAgents).toContain("web-developer");
+      globalExpect(selectedAgents).not.toContain("api-developer");
+      globalExpect(selectedAgents).not.toContain("api-reviewer");
+    });
+    it("should return sorted agents", () => {
+      const store = useWizardStore.getState();
+      store.toggleDomain("web");
+      store.toggleDomain("api");
+      store.preselectAgentsFromDomains();
+      const { selectedAgents } = useWizardStore.getState();
+      const sorted = [...selectedAgents].sort();
+      globalExpect(selectedAgents).toEqual(sorted);
+    });
+    it("should replace previous agent selection", () => {
+      const store = useWizardStore.getState();
+      store.toggleAgent("documentor");
+      store.toggleDomain("web");
+      store.preselectAgentsFromDomains();
+      const { selectedAgents } = useWizardStore.getState();
+      globalExpect(selectedAgents).not.toContain("documentor");
+    });
+  });
+  describe("step progress with agents step", () => {
+    it("should include agents in completed steps when on confirm", () => {
+      const store = useWizardStore.getState();
+      store.setApproach("scratch");
+      store.setStep("build");
+      store.setStep("sources");
+      store.setStep("agents");
+      store.setStep("confirm");
+      const { completedSteps } = store.getStepProgress();
+      globalExpect(completedSteps).toContain("agents");
+      globalExpect(completedSteps).toContain("sources");
+      globalExpect(completedSteps).toContain("build");
+    });
+    it("should include sources in completed steps when on agents step", () => {
+      const store = useWizardStore.getState();
+      store.setApproach("scratch");
+      store.setStep("build");
+      store.setStep("sources");
+      store.setStep("agents");
+      const { completedSteps } = store.getStepProgress();
+      globalExpect(completedSteps).toContain("build");
+      globalExpect(completedSteps).toContain("sources");
+      globalExpect(completedSteps).not.toContain("agents");
+    });
+    it("should skip agents step when using stack defaults", () => {
+      const store = useWizardStore.getState();
+      store.setApproach("stack");
+      store.selectStack("nextjs-fullstack");
+      store.setStackAction("defaults");
+      store.setStep("confirm");
+      const { skippedSteps } = store.getStepProgress();
+      globalExpect(skippedSteps).toContain("agents");
+      globalExpect(skippedSteps).toContain("build");
+      globalExpect(skippedSteps).toContain("sources");
     });
   });
 });