@esneiderbravo/speclaw 2.0.14 → 2.0.15

package/dist/cli/commands/init.js

@@ -24,17 +24,16 @@ export function detectProjectName(cwd) {
     return path.basename(cwd);
 }
 /**
- * Interactive setup: pick agents (and packs when any exist), scaffold, index,
+ * Setup: create `.agents/` (plus any `--agents`), pick packs when any exist, scaffold, index,
  * and print the handoff prompt.
  *
- * @param flags - Parsed flags; runs interactively on a TTY unless `--agents`, `--yes`, or
- *   `-y` is set. Honors `--project-name`, `--packs`, and `--no-index`.
+ * @param flags - Parsed flags; only the pack picker is interactive (on a TTY unless
+ *   `--agents`, `--yes`, or `-y` is set). Honors `--project-name`, `--packs`, and `--no-index`.
  */
 export async function runInit(flags) {
     const cwd = process.cwd();
     const projectName = flags["project-name"] || detectProjectName(cwd);
     const interactive = Boolean(process.stdin.isTTY) && !flags.agents && !flags.yes && !flags.y;
-    let agents;
     let packs;
     const availablePacks = loadPacks();
     const packIds = Object.keys(availablePacks);
@@ -50,36 +49,26 @@ export async function runInit(flags) {
             `Later, ${ui.code("speclaw update")} upgrades itself through npx.`);
         ui.plain();
     }
-    if (interactive) {
-        // Covers: req~remove-agents-pack~1
-        const prompts = {
-            agents: () => clack.multiselect({
-                message: "Which agents do you use? (space to select)",
-                options: AGENTS.map((a) => ({ value: a.id, label: a.label })),
-                initialValues: ["agents"],
-                required: true,
-            }),
-        };
-        if (packIds.length > 0) {
-            prompts.packs = () => clack.multiselect({
-                message: "Which tool packs to install?",
-                options: packIds.map((id) => ({
-                    value: id,
-                    label: id,
-                    hint: availablePacks[id].description.slice(0, 50),
-                })),
-                initialValues: [],
-                required: false,
-            });
-        }
-        const answers = await clack.group(prompts, { onCancel: () => process.exit(1) });
-        agents = answers.agents;
-        packs = answers.packs ?? [];
-    }
-    else {
-        // Default: one agent-facing folder, `.agents/`, linked into ai-specs/.
-        agents = list(flags.agents).length ? list(flags.agents) : ["agents"];
-        packs = list(flags.packs);
+    // Agents are never prompted for: init creates one agent-facing folder,
+    // `.agents/`, linked into ai-specs/. Other IDE folders only via `--agents`
+    // or `speclaw agent add`.
+    const agents = list(flags.agents).length ? list(flags.agents) : ["agents"];
+    packs = list(flags.packs);
+    // Covers: req~remove-agents-pack~1
+    if (interactive && packIds.length > 0 && !packs.length) {
+        const answer = await clack.multiselect({
+            message: "Which tool packs to install?",
+            options: packIds.map((id) => ({
+                value: id,
+                label: id,
+                hint: availablePacks[id].description.slice(0, 50),
+            })),
+            initialValues: [],
+            required: false,
+        });
+        if (clack.isCancel(answer))
+            process.exit(1);
+        packs = answer;
     }
     const unknownAgents = agents.filter((a) => !agentById(a));
     if (unknownAgents.length) {

package/dist/cli/lib/help.js

@@ -13,7 +13,7 @@ Install globally so the command is always available:
   npm i -g @esneiderbravo/speclaw     (or run npx @esneiderbravo/speclaw@latest init)
 
 Setup
-  init                     Interactive setup: pick agents, scaffold, index, get the prompt
+  init                     Setup: .agents/ linked into ai-specs/, scaffold, index, get the prompt
                            (--minimal omits setup/lifecycle MCP tools)
   update                   Re-run itself at the latest version via npx, then apply project
                            migrations (--check reports version only; --no-self-update
@@ -123,8 +123,9 @@ you rarely run it by hand.
         name: "init",
         usage: `Usage: speclaw init [options]
 
-Interactive setup: pick agents, scaffold the constitution and lawbook, build the
-Compass index, and print the starting prompt. Prefer
+Setup without an agent picker: create .agents/ linked into ai-specs/ (gitignored),
+scaffold the constitution and lawbook, build the Compass index, and print the
+starting prompt. Prefer
 npx @esneiderbravo/speclaw@latest init so the scaffold matches the latest release.
 
 Options

package/dist/shared/agents.js

@@ -11,6 +11,7 @@ export const AGENTS = [
         linkTargets: ["skills", "commands", "agents"],
         mcpFile: ".mcp.json",
         hooks: { file: ".claude/settings.json", key: "hooks" },
+        ownedPaths: [".claude/rules/speclaw"],
     },
     {
         id: "cursor",
@@ -141,11 +142,23 @@ export function configureAgent(projectPath, agentId, report) {
     if (!agent)
         throw new Error(`Unknown agent "${agentId}". Known: ${AGENTS.map((a) => a.id).join(", ")}`);
     const ideRoot = path.join(projectPath, agent.ideDir);
-    // An agent folder speclaw creates holds only links into the gitignored
-    // ai-specs/ and per-developer wiring: keep it out of version control. A
-    // folder that already existed is the user's and is left as it is.
+    // Everything speclaw puts in an agent folder is links into the gitignored
+    // ai-specs/ plus per-developer wiring: keep it out of version control. A
+    // folder speclaw creates is ignored whole; in a folder that already existed
+    // (the IDE's own, e.g. `.claude/settings.local.json`) only speclaw's entries are.
+    const comment = `speclaw ${agent.label} links (regenerated by init; never commit)`;
     if (!fs.existsSync(ideRoot)) {
-        ensureGitignore(projectPath, `${agent.ideDir}/`, `speclaw ${agent.label} links (regenerated by init; never commit)`, report);
+        ensureGitignore(projectPath, `${agent.ideDir}/`, comment, report);
+    }
+    else {
+        const owned = agent.linkTargets
+            .filter((t) => fs.existsSync(path.join(projectPath, "ai-specs", t)))
+            .map((t) => `${agent.ideDir}/${t}`)
+            .concat(agent.ownedPaths ?? []);
+        if (agent.hooks && !fs.existsSync(path.join(projectPath, agent.hooks.file)))
+            owned.push(agent.hooks.file);
+        for (const entry of owned)
+            ensureGitignore(projectPath, entry, comment, report);
     }
     for (const target of agent.linkTargets) {
         const sourceDir = path.join(projectPath, "ai-specs", target);
@@ -164,13 +177,25 @@ export function configureAgent(projectPath, agentId, report) {
         writeMcpConfig(projectPath, agent.mcpFile, report);
 }
 /**
- * List the agents already set up in this project (their IDE dir exists).
+ * List the agents speclaw set up in this project: those whose IDE dir holds at
+ * least one speclaw link into ai-specs/. A bare IDE dir is not enough — Claude
+ * Code and Cursor create `.claude/` / `.cursor/` on their own, and treating
+ * that as "configured" made `update` populate folders nobody chose.
  *
  * @param projectPath - Project root to inspect.
- * @returns The ids of agents whose IDE directory is present.
+ * @returns The ids of agents with a speclaw link in their IDE directory.
  */
 export function detectConfiguredAgents(projectPath) {
-    return AGENTS.filter((a) => fs.existsSync(path.join(projectPath, a.ideDir))).map((a) => a.id);
+    return AGENTS.filter((a) => a.linkTargets.some((t) => isAiSpecsLink(path.join(projectPath, a.ideDir, t), t))).map((a) => a.id);
+}
+/** True when `p` is a symlink whose target ends in `ai-specs/<target>`. */
+function isAiSpecsLink(p, target) {
+    try {
+        return fs.readlinkSync(p).replace(/\\/g, "/").endsWith(`ai-specs/${target}`);
+    }
+    catch {
+        return false;
+    }
 }
 /**
  * Re-run configuration for every already-configured agent (e.g. after adding a pack).

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@esneiderbravo/speclaw",
-  "version": "2.0.14",
+  "version": "2.0.15",
   "publishConfig": {
     "access": "public"
   },