@starlein/paperclip-plugin-company-wizard 0.4.6 → 0.4.7

package/dist/worker.js

@@ -9546,7 +9546,7 @@ async function assembleCompany({
     }
     return assignee;
   };
-  const resolveReviewGate = (reviewGate) => {
+  const resolveReviewGate = (reviewGate, assignTo) => {
     if (!reviewGate || typeof reviewGate !== "object") return null;
     const reviewersRaw = Array.isArray(reviewGate.reviewers) ? reviewGate.reviewers : [];
     const reviewers = [];
@@ -9560,22 +9560,12 @@ async function assembleCompany({
     }
     const approver = typeof reviewGate.approver === "string" && allRoles.has(reviewGate.approver) ? reviewGate.approver : void 0;
     let mergeGate = typeof reviewGate.mergeGate === "string" && allRoles.has(reviewGate.mergeGate) ? reviewGate.mergeGate : void 0;
-    if (!mergeGate) {
-      const mergeGateFallbacks = [
-        "code-reviewer",
-        "devops",
-        "ui-designer",
-        "ux-researcher",
-        "security-engineer",
-        "qa"
-      ];
-      mergeGate = mergeGateFallbacks.find((role) => allRoles.has(role) && role !== approver);
-    }
+    if (mergeGate === assignTo) mergeGate = void 0;
     if (approver) {
       const idx = reviewers.indexOf(approver);
       if (idx !== -1) reviewers.splice(idx, 1);
     }
-    if (reviewers.length === 0 && !approver && !mergeGate) return null;
+    if (!mergeGate) return null;
     return { reviewers, approver, mergeGate };
   };
   const hasCi = moduleNames.includes("ci-cd");
@@ -10309,7 +10299,7 @@ Read: \`docs/${doc}\`
         ["goalId", goalRef ? `\u2192 "${goalRef}"` : void 0],
         ["labelIds", `\u2192 [${issueLabelNames.map((name) => `"${name}"`).join(", ")}]`]
       ]);
-      const issueReviewGate = resolveReviewGate(issue.reviewGate);
+      const issueReviewGate = resolveReviewGate(issue.reviewGate, issue.assignTo);
       if (issueReviewGate) {
         bootstrap += renderReviewGate(issueReviewGate);
       }
@@ -10334,7 +10324,7 @@ Read: \`docs/${doc}\`
 `;
     if (moduleNames.includes("pr-review")) {
       const ciClause = hasCi ? "CI (lint/test/build) must be green before the merge gate merges \u2014 this is the hard gate and cannot be skipped" : "no CI is configured, so the merge-gate agent must run the test suite/build and paste the real output into the merge-gate verdict before merging \u2014 this is the hard gate";
-      bootstrap += `- Required PR reviews use the issue's \`executionPolicy\`. The substantive gate is execution, not opinion: ${ciClause}. Stages, in order: a \`review\` stage for QA when present (test adequacy / running the tests), a \`review\` stage for the Security Engineer **only when the change is security-relevant** (auth, secrets, input boundaries, crypto, dependencies, infra exposure), an \`approval\` stage for the Product Owner (intent/scope), then a final \`approval\` merge-gate stage for the **Code Reviewer** (a non-author who satisfies the hard gate above, merges the PR, then records approval to close the issue). **Never list the issue's executor/author as a participant in any stage** \u2014 Paperclip excludes the original executor from review/approval, so a stage whose only participant is the author has no eligible participant and the issue stalls in \`in_review\` (422 No eligible approval participant); this is why the merge gate is the Code Reviewer (or another present non-author), not the engineer who wrote the code. The merge gate must be last so the Product Owner's approval does not auto-close the issue with the PR still open. Other domain reviewers may add advisory, non-blocking comments but do not gate the merge. Every verdict must cite executed verification. Resolve each role to its agentId. Model review stages in executionPolicy rather than child issues or @-mentions.
+      bootstrap += `- Required PR reviews use the issue's \`executionPolicy\`. The substantive gate is execution, not opinion: ${ciClause}. Stages, in order: a \`review\` stage for QA when present (test adequacy / running the tests), a \`review\` stage for the Security Engineer **only when the change is security-relevant** (auth, secrets, input boundaries, crypto, dependencies, infra exposure), an \`approval\` stage for the Product Owner when present (intent/scope), then a final \`approval\` merge-gate stage for the **Code Reviewer** (a non-author who satisfies the hard gate above, merges the PR, then records approval to close the issue). **Never list the issue's executor/author as a participant in any stage** \u2014 Paperclip excludes the original executor from review/approval, so a stage whose only participant is the author has no eligible participant and the issue stalls in \`in_review\` (422 No eligible approval participant); this is why the merge gate is the Code Reviewer (a non-author), not the engineer who wrote the code. The merge gate must be last so the Product Owner's approval does not auto-close the issue with the PR still open. When no Code Reviewer is on the team, do not set executionPolicy stages at all \u2014 use the PR-Self-Merge path (the engineer opens the PR and merges via \`gh pr merge <N> --merge\`); other review roles may leave advisory comments but do not block. Other domain reviewers may add advisory, non-blocking comments but do not gate the merge. Every verdict must cite executed verification. Resolve each role to its agentId. Model review stages in executionPolicy rather than child issues or @-mentions.
 `;
     }
     bootstrap += `
@@ -10613,6 +10603,17 @@ var PaperclipClient = class {
       })
     });
   }
+  async getInstructionsBundle(agentId) {
+    return this._fetch(`/api/agents/${agentId}/instructions-bundle`, {
+      method: "GET"
+    });
+  }
+  async deleteInstructionsBundleFile(agentId, { path: path3 }) {
+    return this._fetch(`/api/agents/${agentId}/instructions-bundle/file`, {
+      method: "DELETE",
+      body: JSON.stringify({ path: path3 })
+    });
+  }
   async createAgent(companyId, agent) {
     const {
       name,
@@ -11002,7 +11003,7 @@ var __dirname = path2.dirname(fileURLToPath2(import.meta.url));
 var DEFAULT_TEMPLATES_REPO_URL = "https://github.com/starlein/paperclip-plugin-company-wizard/tree/main/templates";
 var BUNDLED_TEMPLATES_DIR = path2.resolve(__dirname, "..", "templates");
 var PLUGIN_PACKAGE_NAME = "@starlein/paperclip-plugin-company-wizard";
-var CURRENT_PLUGIN_VERSION = "0.4.6";
+var CURRENT_PLUGIN_VERSION = "0.4.7";
 var NPM_LATEST_URL = "https://registry.npmjs.org/@starlein%2Fpaperclip-plugin-company-wizard/latest";
 function copyDirSync(src, dest) {
   fs2.mkdirSync(dest, { recursive: true });
@@ -11351,11 +11352,57 @@ async function syncAgentInstructionsIntoManagedBundle({
     );
   }
 }
+function preserveExistingSkillSync(existingAgent, nextAdapterConfig) {
+  const existingSync = existingAgent?.adapterConfig && typeof existingAgent.adapterConfig === "object" && "paperclipSkillSync" in existingAgent.adapterConfig ? existingAgent.adapterConfig.paperclipSkillSync : void 0;
+  if (!existingSync) {
+    return nextAdapterConfig;
+  }
+  return {
+    ...nextAdapterConfig,
+    paperclipSkillSync: existingSync
+  };
+}
 function routineTemplateTitle(routine) {
   if (typeof routine?.title === "string" && routine.title.trim()) return routine.title.trim();
   if (typeof routine?.name === "string" && routine.name.trim()) return routine.name.trim();
   return "";
 }
+function buildWizardManifest(params) {
+  const routineTitles = params.initialRoutines.map((r) => routineTemplateTitle(r)).filter(Boolean);
+  const generatedFilePaths = {};
+  if (params.assembleResult?.allFiles && typeof params.assembleResult.allFiles === "object") {
+    for (const relativePath of Object.keys(params.assembleResult.allFiles)) {
+      const match = relativePath.match(/^agents\/([^/]+)\//);
+      if (match) {
+        const agentRole = match[1];
+        if (!generatedFilePaths[agentRole]) generatedFilePaths[agentRole] = [];
+        generatedFilePaths[agentRole].push(relativePath);
+      } else if (relativePath.startsWith("docs/")) {
+        if (!generatedFilePaths["docs"]) generatedFilePaths["docs"] = [];
+        generatedFilePaths["docs"].push(relativePath);
+      }
+    }
+  }
+  return {
+    pluginVersion: CURRENT_PLUGIN_VERSION,
+    preset: params.presetName,
+    modules: params.selectedModules,
+    roles: params.selectedRoleNames,
+    generatedFilePaths,
+    routineTitles,
+    updatedAt: (/* @__PURE__ */ new Date()).toISOString()
+  };
+}
+var WIZARD_PLUGIN_KEY = "starlein.paperclip-plugin-company-wizard";
+async function findPluginId(client) {
+  try {
+    const plugins = await client._fetch("/api/plugins");
+    const match = Array.isArray(plugins) ? plugins.find((p) => p.pluginKey === WIZARD_PLUGIN_KEY) : null;
+    return match?.id ?? null;
+  } catch {
+    return null;
+  }
+}
 async function syncRoutineTrigger({
   client,
   routineId,
@@ -11656,6 +11703,204 @@ var plugin = definePlugin({
         }
       }
     });
+    ctx.actions.register("preview-company-update", async (params) => {
+      let tmpDir;
+      try {
+        let countFiles2 = function(dir) {
+          if (!fs2.existsSync(dir)) return;
+          for (const entry of fs2.readdirSync(dir, { withFileTypes: true })) {
+            const full = path2.join(dir, entry.name);
+            if (entry.isDirectory()) countFiles2(full);
+            else if (entry.isFile()) plannedFiles++;
+          }
+        };
+        var countFiles = countFiles2;
+        const existingCompanyId = typeof params.existingCompanyId === "string" && params.existingCompanyId.trim() ? params.existingCompanyId.trim() : "";
+        if (!existingCompanyId) {
+          return { error: "existingCompanyId is required for preview." };
+        }
+        const cfg = await ctx.config.get() ?? {};
+        const paperclipUrl = cfg.paperclipUrl || process.env.PAPERCLIP_PUBLIC_URL || "http://localhost:3100";
+        const paperclipEmail = cfg.paperclipEmail || "";
+        const paperclipPassword = cfg.paperclipPassword || "";
+        const companyName = typeof params.companyName === "string" && params.companyName.trim() ? params.companyName.trim() : "Preview";
+        const templatesDir = await ensureTemplatesDir(cfg);
+        tmpDir = path2.join(os.tmpdir(), `company-wizard-preview-update-${Date.now()}`);
+        const [presets, allModules, roleTemplates] = await Promise.all([
+          loadPresets(templatesDir),
+          loadModules(templatesDir),
+          loadRoles(templatesDir)
+        ]);
+        const roleTemplateByName = new Map(
+          (Array.isArray(roleTemplates) ? roleTemplates : []).filter((role) => role && typeof role.name === "string").map((role) => [role.name, role])
+        );
+        const selectedPreset = presets.find((p) => p.name === params.presetName) || null;
+        const effectiveModules = resolveEffectiveModules(
+          selectedPreset,
+          allModules,
+          params.selectedModules ?? []
+        );
+        const goals = collectGoals(selectedPreset, allModules, new Set(effectiveModules));
+        const presetBootstrapData = collectPresetBootstrapData(selectedPreset);
+        const previewGoals = Array.isArray(params.goals) ? params.goals : params.goal ? [params.goal] : [];
+        const previewProjects = Array.isArray(params.projects) ? params.projects : [];
+        const previewIssues = Array.isArray(params.issues) ? params.issues : [];
+        const assembleResult = await assembleCompany({
+          companyName,
+          userGoals: previewGoals,
+          userProjects: previewProjects,
+          moduleNames: effectiveModules,
+          extraRoleNames: params.selectedRoles ?? [],
+          inlineGoals: goals,
+          userIssues: previewIssues,
+          presetIssues: presetBootstrapData.issues,
+          presetRoutines: presetBootstrapData.routines,
+          presetLabels: presetBootstrapData.labels,
+          enableIsolatedWorktrees: await resolveEnableIsolatedWorkspacesFromInstance(cfg),
+          enableEnrichedPersonas: true,
+          outputDir: tmpDir,
+          templatesDir
+        });
+        const allRoles = Array.isArray(params.allRoles) ? params.allRoles : [...assembleResult.allRoles ?? []].filter(Boolean);
+        const teamRoles = allRoles.filter((r) => r && r !== "ceo");
+        let plannedFiles = 0;
+        countFiles2(assembleResult.companyDir);
+        const client = new PaperclipClient(paperclipUrl, {
+          email: paperclipEmail,
+          password: paperclipPassword
+        });
+        await client.connect();
+        const company = await client.getCompany(existingCompanyId);
+        const existingAgents = await client.listAgents(existingCompanyId);
+        const existingRoutines = await client.listRoutines(existingCompanyId);
+        let existingManifest = null;
+        try {
+          const pluginId = await findPluginId(client);
+          if (pluginId) {
+            const settings = await client._fetch(
+              `/api/plugins/${pluginId}/company-settings/${existingCompanyId}`
+            );
+            const manifestData = settings?.settingsJson?.wizardManifest ?? settings?.settings_json?.wizardManifest;
+            if (manifestData && typeof manifestData === "object") {
+              existingManifest = manifestData;
+            }
+          }
+        } catch {
+        }
+        const existingCeo = Array.isArray(existingAgents) ? existingAgents.find((a) => a?.role === "ceo" && a?.status !== "terminated") : null;
+        const existingByTemplateRole = /* @__PURE__ */ new Map();
+        if (Array.isArray(existingAgents)) {
+          for (const a of existingAgents) {
+            const tr = a?.metadata?.templateRole;
+            if (tr && a?.status !== "terminated" && a?.role !== "ceo") {
+              existingByTemplateRole.set(tr, a);
+            }
+          }
+        }
+        const agents = [];
+        const ceoTemplate = roleTemplateByName.get("ceo") || {};
+        const ceoTitle = typeof ceoTemplate.title === "string" && ceoTemplate.title.trim() ? ceoTemplate.title.trim() : "CEO";
+        agents.push({
+          role: "ceo",
+          title: existingCeo?.title || ceoTitle,
+          action: existingCeo ? "update" : "hire"
+        });
+        for (const roleName of teamRoles) {
+          const roleTemplate = roleTemplateByName.get(roleName) || {};
+          const roleTitle = typeof roleTemplate.title === "string" && roleTemplate.title.trim() ? roleTemplate.title.trim() : formatRoleName(roleName);
+          const existing = existingByTemplateRole.get(roleName);
+          agents.push({
+            role: roleName,
+            title: roleTitle,
+            action: existing ? "update" : "hire"
+          });
+        }
+        const selectedRoleSet = new Set(allRoles);
+        const manifestRoleSet = existingManifest?.roles ? new Set(existingManifest.roles) : null;
+        for (const [tr, agent] of existingByTemplateRole.entries()) {
+          if (!selectedRoleSet.has(tr)) {
+            if (manifestRoleSet && !manifestRoleSet.has(tr)) {
+              continue;
+            }
+            const roleTemplate = roleTemplateByName.get(tr) || {};
+            const roleTitle = typeof roleTemplate.title === "string" && roleTemplate.title.trim() ? roleTemplate.title.trim() : formatRoleName(tr);
+            agents.push({
+              role: tr,
+              title: agent?.title || roleTitle,
+              action: "retire"
+            });
+          }
+        }
+        const existingRoutineByTitle = /* @__PURE__ */ new Map();
+        if (Array.isArray(existingRoutines)) {
+          for (const r of existingRoutines) {
+            const title = routineTemplateTitle(r);
+            if (title) existingRoutineByTitle.set(title.toLowerCase(), r);
+          }
+        }
+        const plannedRoutines = [
+          ...assembleResult.initialRoutines ?? [],
+          ...presetBootstrapData.routines ?? []
+        ];
+        const seenRoutineTitles = /* @__PURE__ */ new Set();
+        const routines = [];
+        for (const routine of plannedRoutines) {
+          const title = routineTemplateTitle(routine);
+          if (!title) continue;
+          const key = title.toLowerCase();
+          if (seenRoutineTitles.has(key)) continue;
+          seenRoutineTitles.add(key);
+          const existing = existingRoutineByTitle.get(key);
+          routines.push({
+            title,
+            action: existing ? "update" : "create",
+            assignTo: routine.assignTo || void 0
+          });
+        }
+        const desiredSkillsPreserved = [];
+        if (Array.isArray(existingAgents)) {
+          for (const a of existingAgents) {
+            if (a?.adapterConfig && typeof a.adapterConfig === "object" && "paperclipSkillSync" in a.adapterConfig) {
+              const sync = a.adapterConfig.paperclipSkillSync;
+              if (sync && typeof sync === "object" && "desiredSkills" in sync) {
+                const skills = sync.desiredSkills;
+                if (Array.isArray(skills) && skills.length > 0) {
+                  desiredSkillsPreserved.push({
+                    agentId: a.id,
+                    agentName: a.title || a.name || a.id,
+                    skills
+                  });
+                }
+              }
+            }
+          }
+        }
+        try {
+          fs2.rmSync(tmpDir, { recursive: true, force: true });
+        } catch {
+        }
+        tmpDir = void 0;
+        return {
+          diff: {
+            companyId: existingCompanyId,
+            companyName: company?.name || companyName,
+            agents,
+            routines,
+            desiredSkillsPreserved,
+            plannedFiles,
+            existingManifest
+          }
+        };
+      } catch (err) {
+        if (tmpDir) {
+          try {
+            fs2.rmSync(tmpDir, { recursive: true, force: true });
+          } catch {
+          }
+        }
+        return { error: err instanceof Error ? err.message : String(err) };
+      }
+    });
     ctx.actions.register("check-auth", async () => {
       const cfg = await ctx.config.get() ?? {};
       const paperclipUrl = cfg.paperclipUrl || process.env.PAPERCLIP_PUBLIC_URL || "http://localhost:3100";
@@ -11842,8 +12087,27 @@ var plugin = definePlugin({
         let boardOperationsIssue = null;
         let hiringPlanIssue = null;
         let bootstrapIssue;
+        let allRoleNames = [];
+        let existingManifest = null;
+        let existingByTemplateRole = /* @__PURE__ */ new Map();
+        let routines = [];
         try {
-          const allRoleNames = [...assembleResult.allRoles ?? []].filter(Boolean).sort();
+          allRoleNames = [...assembleResult.allRoles ?? []].filter(Boolean).sort();
+          if (existingCompanyId) {
+            try {
+              const pluginId = await findPluginId(client);
+              if (pluginId) {
+                const settings = await client._fetch(
+                  `/api/plugins/${pluginId}/company-settings/${companyId}`
+                );
+                const manifestData = settings?.settingsJson?.wizardManifest ?? settings?.settings_json?.wizardManifest;
+                if (manifestData && typeof manifestData === "object") {
+                  existingManifest = manifestData;
+                }
+              }
+            } catch {
+            }
+          }
           const repositoryMode = userProjects.some(
             (project) => project?.repoUrl || project?.workspace?.sourceType === "git_repo"
           ) ? "existing git repository" : "fresh local repository";
@@ -11930,7 +12194,7 @@ var plugin = definePlugin({
               try {
                 const ceoPatch = {
                   adapterType,
-                  adapterConfig,
+                  adapterConfig: preserveExistingSkillSync(existingCeo, adapterConfig),
                   runtimeConfig: ceoRuntimeConfig,
                   ...ceoMetadata ? { metadata: { ...existingCeo.metadata ?? {}, ...ceoMetadata } } : {}
                 };
@@ -12015,7 +12279,7 @@ var plugin = definePlugin({
           const teamRoles = [...assembleResult.allRoles ?? []].filter(
             (r) => r && r !== "ceo"
           );
-          let existingByTemplateRole = /* @__PURE__ */ new Map();
+          existingByTemplateRole = /* @__PURE__ */ new Map();
           if (existingCompanyId && teamRoles.length > 0) {
             const agents = await client.listAgents(companyId);
             if (Array.isArray(agents)) {
@@ -12050,7 +12314,7 @@ var plugin = definePlugin({
               try {
                 await client.updateAgent(existingAgent.id, {
                   adapterType,
-                  adapterConfig: roleAdapterConfig,
+                  adapterConfig: preserveExistingSkillSync(existingAgent, roleAdapterConfig),
                   runtimeConfig: roleRuntimeConfig,
                   metadata: { ...existingAgent.metadata ?? {}, ...roleMetadata },
                   ...!existingAgent.title && roleTitle ? { title: roleTitle } : {},
@@ -12094,7 +12358,7 @@ var plugin = definePlugin({
               );
             }
           }
-          const routines = Array.isArray(assembleResult.initialRoutines) ? assembleResult.initialRoutines : [];
+          routines = Array.isArray(assembleResult.initialRoutines) ? assembleResult.initialRoutines : [];
           if (!existingCompanyId) {
             let mainProjectId;
             const mainProject = assembleResult.mainProject;
@@ -12196,6 +12460,73 @@ var plugin = definePlugin({
         const issueIds = [boardOperationsIssue?.id, hiringPlanIssue?.id, bootstrapIssue.id].filter(
           Boolean
         );
+        try {
+          const pluginId = await findPluginId(client);
+          if (pluginId) {
+            const wizardManifest = buildWizardManifest({
+              presetName: selectedPreset?.name ?? null,
+              selectedModules: effectiveModules,
+              selectedRoleNames: allRoleNames,
+              assembleResult,
+              initialRoutines: routines
+            });
+            await client._fetch(`/api/plugins/${pluginId}/company-settings/${companyId}`, {
+              method: "PUT",
+              body: JSON.stringify({ settingsJson: { wizardManifest } })
+            });
+            log("\u2713 Wizard manifest saved");
+          } else {
+            log("\u26A0 Could not find plugin ID \u2014 manifest not saved");
+          }
+        } catch (manifestErr) {
+          log(
+            `\u26A0 Could not save wizard manifest: ${manifestErr instanceof Error ? manifestErr.message : String(manifestErr)}`
+          );
+        }
+        if (existingCompanyId && existingManifest && Array.isArray(existingManifest.roles)) {
+          const newRoleSet = new Set(allRoleNames);
+          const retiredRoles = existingManifest.roles.filter(
+            (role) => !newRoleSet.has(role)
+          );
+          for (const role of retiredRoles) {
+            try {
+              const agentData = existingByTemplateRole.get(role);
+              const agentInfo = agentData ? `**Agent ID:** ${agentData.id}
+**Agent name:** ${agentData.title || agentData.name || agentData.id}` : "_No active agent found for this role._";
+              const issueTitle = `Review retired template role: ${role}`;
+              const issueDescription = [
+                `## Retired Role: \`${role}\``,
+                "",
+                "This template role was removed from the company configuration during an update.",
+                "The agent is still active but no longer managed by the wizard.",
+                "",
+                "### Cleanup Checklist",
+                "",
+                "- [ ] Review the agent's current work and open issues",
+                "- [ ] Reassign any in-progress issues to other team members",
+                "- [ ] Consider pausing the agent's heartbeat",
+                "- [ ] Remove module-specific skill files if applicable",
+                "- [ ] Consider terminating the agent if no longer needed",
+                "",
+                agentInfo
+              ].join("\n");
+              const createdIssue = await client.createIssue(companyId, {
+                title: issueTitle,
+                description: issueDescription,
+                priority: "low",
+                status: "todo",
+                ...boardOperationsIssue?.id ? { projectId: boardOperationsIssue.id, goalId: boardOperationsIssue.id } : {}
+              });
+              log(
+                `\u2713 Retired-role review issue created for "${role}": ${createdIssue.identifier || createdIssue.id}`
+              );
+            } catch (retiredRoleErr) {
+              log(
+                `\u26A0 Could not create retired-role review issue for "${role}": ${retiredRoleErr instanceof Error ? retiredRoleErr.message : String(retiredRoleErr)}`
+              );
+            }
+          }
+        }
         return {
           companyId,
           issuePrefix: company.issuePrefix,