@wrongstack/core 0.68.0 → 0.73.1

package/dist/index.js

@@ -2677,7 +2677,8 @@ When reviewing code:
 - Verify test coverage for critical paths
 - Ensure naming conventions are followed`,
     tags: ["review", "quality", "security"],
-    toolPreferences: ["read", "grep", "git", "diff", "test"]
+    toolPreferences: ["read", "grep", "git", "diff", "test"],
+    suggestedSkills: ["bug-hunter", "security-scanner", "typescript-strict", "testing"]
   },
   {
     id: "code-auditor",
@@ -2694,7 +2695,8 @@ When auditing code for security:
 - Assess input validation and output encoding
 - Look for timing attacks and information leakage`,
     tags: ["security", "audit", "compliance"],
-    toolPreferences: ["grep", "read", "audit", "bash"]
+    toolPreferences: ["grep", "read", "audit", "bash"],
+    suggestedSkills: ["security-scanner", "bug-hunter", "audit-log"]
   },
   {
     id: "architect",
@@ -2712,7 +2714,8 @@ When designing or reviewing architecture:
 - Assess API design and contract stability
 - Consider operational aspects (monitoring, logging, deployment)`,
     tags: ["architecture", "design", "scalability"],
-    toolPreferences: ["read", "glob", "tree", "diff"]
+    toolPreferences: ["read", "glob", "tree", "diff"],
+    suggestedSkills: ["api-design", "refactor-planner", "node-modern", "docker-deploy"]
   },
   {
     id: "debugger",
@@ -2730,7 +2733,8 @@ When investigating bugs:
 - Use binary search to isolate the root cause
 - Verify fixes with tests before considering done`,
     tags: ["debug", "investigation", "error-resolution"],
-    toolPreferences: ["read", "grep", "bash", "logs", "test"]
+    toolPreferences: ["read", "grep", "bash", "logs", "test"],
+    suggestedSkills: ["bug-hunter", "audit-log", "observability"]
   },
   {
     id: "tester",
@@ -2748,7 +2752,8 @@ When testing or writing tests:
 - Check for integration test gaps
 - Verify test isolation and cleanup`,
     tags: ["testing", "qa", "quality"],
-    toolPreferences: ["read", "grep", "test", "bash"]
+    toolPreferences: ["read", "grep", "test", "bash"],
+    suggestedSkills: ["testing", "bug-hunter", "typescript-strict"]
   },
   {
     id: "devops",
@@ -2766,7 +2771,8 @@ When working on infrastructure:
 - Assess secrets management
 - Check for resource limits and quotas`,
     tags: ["devops", "infrastructure", "operations"],
-    toolPreferences: ["read", "bash", "grep", "logs", "git"]
+    toolPreferences: ["read", "bash", "grep", "logs", "git"],
+    suggestedSkills: ["docker-deploy", "observability", "security-scanner"]
   },
   {
     id: "refactorer",
@@ -2784,7 +2790,8 @@ When refactoring code:
 - Don't mix formatting changes with logic changes
 - Keep performance in mind \u2014 don't regress`,
     tags: ["refactor", "modernization", "improvement"],
-    toolPreferences: ["read", "edit", "test", "git", "grep"]
+    toolPreferences: ["read", "edit", "test", "git", "grep"],
+    suggestedSkills: ["refactor-planner", "typescript-strict", "node-modern", "testing"]
   },
   {
     id: "brief",
@@ -2814,7 +2821,8 @@ Get to the point \u2014 read files, run commands, make changes.
 - One-liner sufficient? One liner.
 - Max 3 sentences per paragraph.`,
     tags: ["fast", "concise", "direct"],
-    toolPreferences: ["read", "edit", "bash"]
+    toolPreferences: ["read", "edit", "bash"],
+    suggestedSkills: []
   },
   {
     id: "teach",
@@ -2879,7 +2887,8 @@ You follow these principles, but always with explanation:
 
 Remember: your job is to make the user a better developer, not just to complete tasks faster.`,
     tags: ["teaching", "mentor", "learning"],
-    toolPreferences: ["read", "edit", "explain"]
+    toolPreferences: ["read", "edit", "explain"],
+    suggestedSkills: ["prompt-engineering", "skill-creator", "node-modern", "typescript-strict"]
   }
 ];
 
@@ -3512,7 +3521,7 @@ ${errorDetails}`,
       const decision = await this.opts.permissionPolicy.evaluate(tool, use.input, ctx);
       let effectivePermission = decision.permission;
       const policy = this.opts.permissionPolicy;
-      const yolo = policy.getYolo?.() === true || policy.getYoloDestructive?.() === true || policy.getForceAllYolo?.() === true;
+      const yolo = policy.getYolo?.() === true || policy.getYoloDestructive?.() === true;
       if (toolDangerousCaps.length > 0 && effectivePermission === "auto" && !yolo) {
         effectivePermission = "confirm";
       }
@@ -4654,7 +4663,8 @@ function isGlob(p) {
   return false;
 }
 function globToRegex(pat) {
-  let i = 0, re = "^";
+  let i = 0;
+  let re = "^";
   while (i < pat.length) {
     const c = pat[i];
     if (c === "*") {
@@ -5922,8 +5932,17 @@ var BEHAVIOR_DEFAULTS = {
     modelsRegistry: true,
     skills: true
   },
+  indexing: {
+    onSessionStart: true,
+    onEdit: true,
+    watchExternal: true,
+    debounceMs: 400
+  },
   session: { ...DEFAULT_SESSION_LOGGING_CONFIG }
 };
+function envBool(v) {
+  return !/^(0|false|no|off)$/i.test(v.trim());
+}
 var ENV_MAP = {
   WRONGSTACK_PROVIDER: (c, v) => {
     c.provider = v;
@@ -5948,8 +5967,23 @@ var ENV_MAP = {
   WRONGSTACK_LOG_LEVEL: (c, v) => {
     if (!c.log) c.log = { level: "info" };
     c.log.level = v;
+  },
+  WRONGSTACK_INDEX_ON_START: (c, v) => {
+    c.indexing = { ...defaultIndexing, ...c.indexing, onSessionStart: envBool(v) };
+  },
+  WRONGSTACK_INDEX_ON_EDIT: (c, v) => {
+    c.indexing = { ...defaultIndexing, ...c.indexing, onEdit: envBool(v) };
+  },
+  WRONGSTACK_INDEX_WATCH: (c, v) => {
+    c.indexing = { ...defaultIndexing, ...c.indexing, watchExternal: envBool(v) };
   }
 };
+var defaultIndexing = {
+  onSessionStart: true,
+  onEdit: true,
+  watchExternal: true,
+  debounceMs: 400
+};
 function isPrimitiveArray(a) {
   return a.every((v) => v === null || typeof v !== "object");
 }
@@ -7174,14 +7208,6 @@ var DefaultPermissionPolicy = class {
   getConfirmDestructive() {
     return this.confirmDestructive;
   }
-  /** @deprecated Use `setYoloDestructive`. */
-  setForceAllYolo(enabled) {
-    this.setYoloDestructive(enabled);
-  }
-  /** @deprecated Use `getYoloDestructive`. */
-  getForceAllYolo() {
-    return this.getYoloDestructive();
-  }
   async reload() {
     try {
       const raw = await fsp3.readFile(this.trustFile, "utf8");
@@ -12899,7 +12925,7 @@ var DefaultMultiAgentCoordinator = class _DefaultMultiAgentCoordinator extends E
   }
   async spawn(subagent) {
     const id = subagent.id || randomUUID();
-    subagent = this.withNickname(subagent, id);
+    const cfg = this.withNickname(subagent, id);
     if (this.subagents.has(id)) {
       throw new Error(`Subagent id "${id}" already exists \u2014 refusing to overwrite`);
     }
@@ -12914,12 +12940,12 @@ var DefaultMultiAgentCoordinator = class _DefaultMultiAgentCoordinator extends E
       maxConcurrent: this.config.maxConcurrent ?? 16
     };
     this.subagents.set(id, {
-      config: { ...subagent, id },
+      config: { ...cfg, id },
       context,
       status: "idle",
       abortController: new AbortController()
     });
-    this.emit("subagent.started", { subagent: { ...subagent, id } });
+    this.emit("subagent.started", { subagent: { ...cfg, id } });
     this.fleetBus?.emit({
       subagentId: id,
       ts: Date.now(),
@@ -14604,6 +14630,10 @@ A scratchpad shared with the rest of the fleet is mounted at \`${parts.sharedScr
 - Use stable filenames (one file per concern); overwrite instead of appending so the Director sees the latest state.`
     );
   }
+  if (parts.skills && parts.skills.trim().length > 0) {
+    sections.push(`Domain knowledge:
+${parts.skills.trim()}`);
+  }
   if (parts.override && parts.override.trim().length > 0) {
     sections.push(parts.override.trim());
   }
@@ -16448,6 +16478,7 @@ var Director = class _Director {
       role: config.prompt,
       task: taskBrief,
       sharedScratchpad: this.sharedScratchpadPath ?? void 0,
+      skills: config.skillContent,
       override: config.systemPromptOverride
     });
   }
@@ -20753,6 +20784,7 @@ var MAX_TARBALL_SIZE = 50 * 1024 * 1024;
 async function downloadGitHubTarball(parsed) {
   const url = `https://api.github.com/repos/${parsed.owner}/${parsed.repo}/tarball/${parsed.ref}`;
   const response = await fetch(url, {
+    signal: AbortSignal.timeout(3e4),
     headers: {
       Accept: "application/vnd.github+json",
       "User-Agent": "wrongstack-skill-installer"
@@ -21588,7 +21620,6 @@ var SessionRecovery = class {
           }
           return null;
         } catch {
-          continue;
         }
       }
       return null;
@@ -21819,7 +21850,7 @@ var ToolAuditLog = class {
     await fsp3.appendFile(fp, line, "utf8");
     const count = (this.unSyncedWrites.get(sessionId) ?? 0) + 1;
     this.unSyncedWrites.set(sessionId, count);
-    if (this.fsyncEvery !== Infinity && count % this.fsyncEvery === 0) {
+    if (this.fsyncEvery !== Number.POSITIVE_INFINITY && count % this.fsyncEvery === 0) {
       await this.sync(sessionId, fp);
     }
   }
@@ -22258,6 +22289,7 @@ var CloudSync = class {
   async githubFetch(token, owner, repo, method, pathSegment, body) {
     const url = `https://api.github.com/repos/${owner}/${repo}${pathSegment}`;
     const res = await fetch(url, {
+      signal: AbortSignal.timeout(15e3),
       method,
       headers: {
         Authorization: `Bearer ${token}`,
@@ -26390,7 +26422,7 @@ var InputBuilder = class {
       meta: { label: paragraphLabel(text) }
     });
     this.refs.push(ref);
-    const lines = text.split("\n").length;
+    const lines = text.split(/\r?\n|\r/).length;
     return `[pasted #${ref.seq}, ${lines} lines]`;
   }
   /**
@@ -26447,14 +26479,16 @@ var InputBuilder = class {
     if (text.length >= this.pasteCharThreshold) return true;
     let lines = 1;
     for (let i = 0; i < text.length; i++) {
-      if (text.charCodeAt(i) === 10) lines++;
+      const c = text.charCodeAt(i);
+      if (c === 10) lines++;
+      else if (c === 13 && text.charCodeAt(i + 1) !== 10) lines++;
       if (lines >= this.pasteLineThreshold) return true;
     }
     return false;
   }
 };
 function paragraphLabel(text) {
-  const lines = text.split("\n").length;
+  const lines = text.split(/\r?\n|\r/).length;
   const bytes = Buffer.byteLength(text, "utf8");
   if (bytes < 1024) return `${lines} lines, ${bytes} B`;
   return `${lines} lines, ${(bytes / 1024).toFixed(1)} KB`;
@@ -26465,23 +26499,34 @@ var PROMPT = `You are WrongStack, a command-line AI coding agent.
 
 You operate inside the user's terminal with direct read and write access to their working directory, the ability to run shell commands, and access to the web. You assist a developer who knows what they're doing \u2014 your job is to accelerate them, not to second-guess them.
 
+These are your baseline instructions. When an active mode prompt (Teach, Brief, Code Reviewer, etc.) is present in your context, its instructions **override** conflicting defaults below \u2014 the mode layer always wins on conflict.
+
 ## Core principles
 
 1. **Read before you write.** Always inspect the relevant files before proposing changes. Assumptions about code you haven't read are bugs in waiting.
 2. **Prefer surgical edits over rewrites.** When modifying existing files, use the edit tool with str_replace; only use write for new files or full replacements explicitly requested.
 3. **Show your work.** Before non-trivial changes, briefly state what you're about to do \u2014 one sentence, not a wall of text. After tool calls, summarize what happened, not what you did mechanically.
 4. **Be honest about limits.** If you don't know, say so. If something failed, say what failed and what you'll try next. Never fabricate file contents, API responses, or test results.
-5. **Be concise.** The user is a developer in a terminal. No marketing language, no "great question!", no bullet-point lists when prose works. If a one-liner answers, a one-liner is the answer.
+5. **Be concise.** The user is a developer in a terminal. No marketing language, no "great question!", no bullet-point lists when prose works. If a one-liner answers, a one-liner is the answer. (Active modes may override verbosity \u2014 follow the mode's guidance on depth.)
 6. **Ask when blocked, proceed when not.** If the task is ambiguous in a way that meaningfully changes the approach, ask. If it's ambiguous in a way that doesn't, pick a reasonable default and proceed, stating the assumption.
 7. **Trust the tools.** If a permission prompt is shown, the user will answer. Do not preemptively explain that you "would like to" do something \u2014 call the tool, let the permission flow decide.
 8. **Format for scanability.** Use code blocks for code, backticks for file paths, bold for key terms. One-liners stay one line. Paragraphs max 3 sentences.
-9. **Recover explicitly.** When a tool fails, state: (1) what failed, (2) what you tried, (3) what you'll attempt next. Never silently skip.
+9. **Match the user's language.** Respond in the same language the user writes in. If they write in Turkish, reply in Turkish. If they mix languages, follow the dominant one.
+10. **Recover explicitly.** When a tool fails, distinguish the failure type and respond accordingly:
+
+| Failure type | Examples | Strategy |
+|---|---|---|
+| **Transient** | Timeout, rate limit, network hiccup | Retry once with adjusted params, then report |
+| **Permanent** | Syntax error, missing file, type error, permission denied | Do NOT retry \u2014 diagnose and report the root cause |
+| **Validation** | Invalid argument, out-of-range value, schema mismatch | State what was rejected and what range/format is accepted |
+
+Never silently skip a failure \u2014 always report it, even when you choose not to retry.
 
 ## Decision heuristics
 
 - **Task is ambiguous** (unclear which file, conflicting requirements) \u2192 ask before proceeding
 - **Task is clear, approach is unknown** \u2192 try one approach, report what happened
-- **Tool fails** \u2192 retry once with adjusted params, then report failure
+- **Tool fails** \u2192 classify the failure (transient/permanent/validation), then apply the appropriate recovery
 - **Permission prompt shown** \u2192 wait for user, do not act unilaterally
 - **Tool denied by user** \u2192 do NOT retry the same tool in the next iteration. If the user denies a write, bash, or any tool, respect that decision. The user's "no" is final \u2014 acknowledge it and ask if they'd like to clarify what they actually want.
 - **Context window filling up** \u2192 use context_manager proactively; don't wait to be told
@@ -26512,6 +26557,8 @@ var DefaultSystemPromptBuilder = class {
    */
   envCacheByRoot = /* @__PURE__ */ new Map();
   skillCache;
+  /** Cached full skill bodies (after frontmatter), built once per session. */
+  skillBodyCache;
   async build(ctx) {
     if (this.opts.skillLoader && !this.skillCache) {
       try {
@@ -26519,8 +26566,8 @@ var DefaultSystemPromptBuilder = class {
         if (entries.length > 0) {
           const lines = [];
           for (const e of entries) {
-            const scopeTag = e.scope.length > 0 ? ` \u2014 ${e.scope.slice(0, 4).join(", ")}` : "";
-            lines.push(`- **${e.name}**${scopeTag}  (${e.trigger})`);
+            const shortTrigger = compactTrigger(e.trigger);
+            lines.push(`- **${e.name}**  (${shortTrigger})`);
           }
           this.skillCache = lines.join("\n");
         }
@@ -26552,6 +26599,24 @@ var DefaultSystemPromptBuilder = class {
         cache_control: { type: "ephemeral" }
       });
     }
+    if (this.opts.modeStore && this.opts.skillLoader) {
+      try {
+        const activeMode = await this.opts.modeStore.getActiveMode();
+        if (activeMode?.suggestedSkills && activeMode.suggestedSkills.length > 0) {
+          const skills = await this.opts.skillLoader.list();
+          const loadedNames = new Set(skills.map((s) => s.name));
+          const available = activeMode.suggestedSkills.filter((n) => loadedNames.has(n));
+          if (available.length > 0) {
+            blocks.push({
+              type: "text",
+              text: `Mode "${activeMode.id}" works best with these skills: ${available.join(", ")}. Their full instructions are in the Active Skills block above.`,
+              cache_control: { type: "ephemeral" }
+            });
+          }
+        }
+      } catch {
+      }
+    }
     if (layer6.trim()) {
       blocks.push({
         type: "text",
@@ -26782,7 +26847,13 @@ summarize it, and let the tool result hold only the summary.`);
       );
     }
     if (this.skillCache) {
-      lines.push("", "## Skills in scope for this session", this.skillCache);
+      lines.push(
+        "",
+        "## Skills in scope for this session",
+        this.skillCache,
+        "",
+        "Full skill instructions are injected in the Active Skills block below."
+      );
     }
     const text = lines.join("\n");
     this.envCacheByRoot.set(ctx.projectRoot, text);
@@ -26799,6 +26870,40 @@ ${mem}`);
       } catch {
       }
     }
+    if (this.opts.skillLoader && this.skillBodyCache === void 0) {
+      try {
+        const skills = await this.opts.skillLoader.list();
+        if (skills.length > 0) {
+          const bodies = [];
+          for (const s of skills) {
+            try {
+              const raw = await this.opts.skillLoader.readBody(s.name);
+              const body = stripFrontmatter(raw);
+              if (body.trim()) {
+                bodies.push(`## Skill: ${s.name}
+
+${body.trim()}`);
+              }
+            } catch {
+            }
+          }
+          if (bodies.length > 0) {
+            this.skillBodyCache = bodies.join("\n\n---\n\n");
+          } else {
+            this.skillBodyCache = "";
+          }
+        } else {
+          this.skillBodyCache = "";
+        }
+      } catch {
+        this.skillBodyCache = "";
+      }
+    }
+    if (this.skillBodyCache) {
+      parts.push(`# Active Skills
+
+${this.skillBodyCache}`);
+    }
     return parts.join("\n\n");
   }
   async buildMode() {
@@ -26888,6 +26993,22 @@ ${mem}`);
     return langs.size === 0 ? "unknown" : Array.from(langs).join(", ");
   }
 };
+function stripFrontmatter(raw) {
+  if (!raw.startsWith("---")) return raw;
+  const end = raw.indexOf("\n---", 4);
+  if (end === -1) return raw;
+  let body = raw.slice(end + 4);
+  if (body.startsWith("\n")) body = body.slice(1);
+  return body;
+}
+function compactTrigger(trigger) {
+  let s = trigger.replace(/^Use this skill when /i, "").replace(/^Use this skill for /i, "").replace(/^Use when /i, "").replace(/\.$/, "");
+  if (s.length > 72) {
+    const cut = s.lastIndexOf(" ", 68);
+    s = cut > 50 ? s.slice(0, cut) + "\u2026" : s.slice(0, 68) + "\u2026";
+  }
+  return s;
+}
 
 // src/registry/tool-registry.ts
 var ToolRegistry = class _ToolRegistry {