wave-agent-sdk 1.0.9 → 1.1.0

package/dist/utils/builtinEmbed.js

@@ -0,0 +1,53 @@
+/**
+ * Builtin content embedding.
+ *
+ * Builtin skills/subagents/plugins ship as TS source (src/builtin/*.ts) and are
+ * compiled into the bundle. At runtime they are lazily materialized to a
+ * content-hashed cache directory under the OS temp dir, so consumers that
+ * readdirSync real directories keep working unchanged — including inside
+ * esbuild bundles that never ship a `builtin/` dir on disk.
+ */
+import { createHash } from "crypto";
+import { existsSync, mkdirSync, writeFileSync } from "fs";
+import { tmpdir } from "os";
+import { dirname, join } from "path";
+import { BUILTIN_CONTENT } from "../builtin/index.js";
+const BUILTIN_CACHE_ROOT = "wave-builtin";
+const BUILTIN_CACHE_COMPLETE = ".wave-builtin-complete";
+/**
+ * Content-hash key: sha256 of the sorted `rel\0content` entries, truncated.
+ * Any change to builtin content yields a new key, auto-invalidating stale caches.
+ */
+export function getBuiltinCacheKey() {
+    const entries = Object.keys(BUILTIN_CONTENT)
+        .sort()
+        .map((rel) => `${rel}\0${BUILTIN_CONTENT[rel]}`);
+    return createHash("sha256")
+        .update(entries.join("\0"))
+        .digest("hex")
+        .slice(0, 16);
+}
+let _builtinDir;
+/**
+ * Materialize builtin content to `{tmpdir}/wave-builtin/{contentHash}/` and
+ * return the directory. Memoized per process; idempotent across processes via
+ * the `.wave-builtin-complete` marker written last (a torn write leaves no
+ * marker, so the next run rewrites).
+ */
+export function ensureBuiltinMaterialized() {
+    if (_builtinDir)
+        return _builtinDir;
+    const dir = join(tmpdir(), BUILTIN_CACHE_ROOT, getBuiltinCacheKey());
+    const marker = join(dir, BUILTIN_CACHE_COMPLETE);
+    if (!existsSync(marker)) {
+        mkdirSync(dir, { recursive: true, mode: 0o700 });
+        for (const [rel, content] of Object.entries(BUILTIN_CONTENT)) {
+            const filePath = join(dir, rel);
+            mkdirSync(dirname(filePath), { recursive: true, mode: 0o700 });
+            writeFileSync(filePath, content);
+        }
+        writeFileSync(marker, "");
+    }
+    _builtinDir = dir;
+    return dir;
+}

package/dist/utils/configPaths.d.ts

@@ -9,7 +9,6 @@
  * - Local configs: {workdir}/.wave/settings.local.json > {workdir}/.wave/settings.json
  * - Project configs override user configs (existing behavior)
  */
-export declare function getPackageRoot(): string;
 /**
  * Get the builtin skills directory path
  */

package/dist/utils/configPaths.js

@@ -9,48 +9,27 @@
  * - Local configs: {workdir}/.wave/settings.local.json > {workdir}/.wave/settings.json
  * - Project configs override user configs (existing behavior)
  */
-import { join, dirname } from "path";
+import { join } from "path";
 import { homedir } from "os";
 import { existsSync } from "fs";
-import { fileURLToPath } from "url";
-import { findUpSync } from "find-up";
-const __filename = fileURLToPath(import.meta.url);
-const __dirname = dirname(__filename);
-/**
- * Resolve the package root directory by finding the nearest package.json.
- * Works correctly even when bundled (e.g. esbuild into a VS Code extension),
- * as long as vendor/ and builtin/ remain alongside package.json on disk.
- */
-let _packageRoot;
-export function getPackageRoot() {
-    if (_packageRoot)
-        return _packageRoot;
-    const pkgPath = findUpSync("package.json", { cwd: __dirname });
-    if (pkgPath) {
-        _packageRoot = dirname(pkgPath);
-        return _packageRoot;
-    }
-    // Fallback: relative to this file (works during development)
-    _packageRoot = join(__dirname, "..", "..");
-    return _packageRoot;
-}
+import { ensureBuiltinMaterialized } from "./builtinEmbed.js";
 /**
  * Get the builtin skills directory path
  */
 export function getBuiltinSkillsDir() {
-    return join(getPackageRoot(), "builtin", "skills");
+    return join(ensureBuiltinMaterialized(), "skills");
 }
 /**
  * Get the builtin subagents directory path
  */
 export function getBuiltinSubagentsDir() {
-    return join(getPackageRoot(), "builtin", "subagents");
+    return join(ensureBuiltinMaterialized(), "subagents");
 }
 /**
  * Get the builtin plugins directory path
  */
 export function getBuiltinPluginsDir() {
-    return join(getPackageRoot(), "builtin", "plugins");
+    return join(ensureBuiltinMaterialized(), "plugins");
 }
 /**
  * Get the user-specific configuration file path (legacy function)

package/dist/utils/containerSetup.js

@@ -27,6 +27,7 @@ import { MemoryService } from "../services/memory.js";
 import { AutoMemoryService } from "../services/autoMemoryService.js";
 import { USER_MEMORY_FILE } from "./constants.js";
 import { getGitMainRepoRoot } from "./gitUtils.js";
+import { AsyncWorkRegistry } from "./asyncWorkRegistry.js";
 import { logger } from "./globalLogger.js";
 import { authService } from "../services/authService.js";
 import { remoteSettingsService } from "../services/remoteSettingsService.js";
@@ -45,6 +46,10 @@ export function setupAgentContainer(setupOptions) {
     container.register("WorktreeSession", null);
     const messageQueue = new MessageQueue();
     container.register("MessageQueue", messageQueue);
+    // Registry of live async work that must drain before the agent is destroyed
+    // (dispatch, background subagents, fork subagents — the fire-and-forget work
+    // sites). destroy() awaits its drain() so no async work can outlive the agent.
+    container.register("AsyncWorkRegistry", new AsyncWorkRegistry());
     const foregroundTaskManager = new ForegroundTaskManager(container);
     container.register("ForegroundTaskManager", foregroundTaskManager);
     container.register("ConfigurationService", configurationService);

package/dist/utils/convertMessagesForAPI.js

@@ -102,10 +102,29 @@ export function convertMessagesForAPI(messages, options) {
                                 });
                             }
                             else {
-                                // Non-vision model: replace images with a text placeholder
+                                // Non-vision model: replace images with a text placeholder and
+                                // append [Image source: <path>] metadata for MCP images that
+                                // were persisted to temp files, so the main model can delegate
+                                // recognition to a vision subagent (which reads the path with
+                                // the Read tool). Images without a persisted path (e.g. legacy
+                                // history) fall back to the placeholder only.
+                                const contentParts = [
+                                    {
+                                        type: "text",
+                                        text: "[Tool returned an image, but the current model does not support image recognition]",
+                                    },
+                                ];
+                                toolBlock.images.forEach((image) => {
+                                    if (image.path) {
+                                        contentParts.push({
+                                            type: "text",
+                                            text: `[Image source: ${image.path}]`,
+                                        });
+                                    }
+                                });
                                 imageUserMessages.push({
                                     role: "user",
-                                    content: "[Tool returned an image, but the current model does not support image recognition]",
+                                    content: contentParts,
                                 });
                             }
                         }

package/dist/utils/messageOperations.d.ts

@@ -37,6 +37,7 @@ export interface UpdateToolBlockParams {
     images?: Array<{
         data: string;
         mediaType?: string;
+        path?: string;
     }>;
     compactParams?: string;
     parametersChunk?: string;

package/dist/utils/skillParser.js

@@ -147,7 +147,6 @@ export function validateSkillMetadata(metadata) {
     // Import SKILL_DEFAULTS dynamically to avoid circular imports
     const NAME_PATTERN = /^[a-z0-9-]+$/;
     const MAX_NAME_LENGTH = 64;
-    const MAX_DESCRIPTION_LENGTH = 1024;
     const MIN_DESCRIPTION_LENGTH = 1;
     // Validate name
     if (!metadata.name) {
@@ -161,7 +160,8 @@ export function validateSkillMetadata(metadata) {
             errors.push("Skill name must contain only lowercase letters, numbers, and hyphens");
         }
     }
-    // Validate description
+    // Validate description (no length limit, aligned with Claude Code which
+    // truncates long descriptions at render time instead of rejecting the skill)
     if (!metadata.description) {
         errors.push("Skill description is required");
     }
@@ -169,9 +169,6 @@ export function validateSkillMetadata(metadata) {
         if (metadata.description.length < MIN_DESCRIPTION_LENGTH) {
             errors.push(`Skill description must be at least ${MIN_DESCRIPTION_LENGTH} character`);
         }
-        if (metadata.description.length > MAX_DESCRIPTION_LENGTH) {
-            errors.push(`Skill description must be ${MAX_DESCRIPTION_LENGTH} characters or less`);
-        }
     }
     return errors;
 }
@@ -194,7 +191,7 @@ export function formatSkillError(skillPath, errors) {
         "  1. Ensure SKILL.md has valid YAML frontmatter (---...---)",
         "  2. Include required fields: name and description",
         "  3. Use lowercase letters, numbers, and hyphens only for name",
-        "  4. Keep name under 64 characters and description under 1024 characters",
+        "  4. Keep name under 64 characters",
     ].join("\n");
     return `${header}\n${errorList}\n\n${suggestions}`;
 }

package/dist/utils/toolImagePersistence.d.ts

@@ -0,0 +1,25 @@
+/**
+ * Persistence for MCP tool-returned images.
+ *
+ * MCP tools can return image content blocks as in-memory base64. When the
+ * agent's model does not support vision, the image is written to a temp file
+ * so convertMessagesForAPI can attach `[Image source: <path>]` metadata and
+ * the main model can delegate recognition to the vision subagent (which reads
+ * the path with the Read tool). Vision-capable models keep the inline base64
+ * and never write to disk.
+ */
+export interface PersistedToolImage {
+    data: string;
+    mediaType?: string;
+    path?: string;
+}
+/**
+ * Persist base64 images to temp files under /tmp/wave-mcp-images/.
+ * Uses the OS tmpdir for simplicity and automatic OS cleanup (same convention
+ * as /tmp/wave-tool-results/). Returns each image unchanged (no path) when the
+ * write fails, so the caller degrades to the placeholder-only behavior.
+ */
+export declare function persistToolImages(images: Array<{
+    data: string;
+    mediaType?: string;
+}>): PersistedToolImage[];

package/dist/utils/toolImagePersistence.js

@@ -0,0 +1,56 @@
+/**
+ * Persistence for MCP tool-returned images.
+ *
+ * MCP tools can return image content blocks as in-memory base64. When the
+ * agent's model does not support vision, the image is written to a temp file
+ * so convertMessagesForAPI can attach `[Image source: <path>]` metadata and
+ * the main model can delegate recognition to the vision subagent (which reads
+ * the path with the Read tool). Vision-capable models keep the inline base64
+ * and never write to disk.
+ */
+import * as fs from "fs";
+import * as os from "os";
+import * as path from "path";
+import { logger } from "./globalLogger.js";
+const MCP_IMAGES_DIR = path.join(os.tmpdir(), "wave-mcp-images");
+/** Map MCP mimeType to a file extension. Unknown types fall back to .png. */
+const MIME_TO_EXT = {
+    "image/png": ".png",
+    "image/jpeg": ".jpg",
+    "image/webp": ".webp",
+    "image/gif": ".gif",
+    "image/bmp": ".bmp",
+    "image/svg+xml": ".svg",
+    "image/avif": ".avif",
+    "image/x-icon": ".ico",
+    "image/tiff": ".tiff",
+};
+/**
+ * Persist base64 images to temp files under /tmp/wave-mcp-images/.
+ * Uses the OS tmpdir for simplicity and automatic OS cleanup (same convention
+ * as /tmp/wave-tool-results/). Returns each image unchanged (no path) when the
+ * write fails, so the caller degrades to the placeholder-only behavior.
+ */
+export function persistToolImages(images) {
+    return images.map((image) => {
+        try {
+            fs.mkdirSync(MCP_IMAGES_DIR, { recursive: true });
+            // Strip a data: URL prefix if present (MCP spec carries raw base64, but
+            // be defensive about dataURL-formatted data).
+            let data = image.data;
+            if (data.startsWith("data:")) {
+                const commaIndex = data.indexOf(",");
+                data = commaIndex >= 0 ? data.slice(commaIndex + 1) : data;
+            }
+            const ext = MIME_TO_EXT[image.mediaType || ""] || ".png";
+            const id = `${Date.now()}_${Math.random().toString(36).substring(2, 9)}`;
+            const filePath = path.join(MCP_IMAGES_DIR, `mcp-image_${id}${ext}`);
+            fs.writeFileSync(filePath, Buffer.from(data, "base64"));
+            return { ...image, path: filePath };
+        }
+        catch (error) {
+            logger?.error("Failed to persist MCP tool image:", error);
+            return image;
+        }
+    });
+}

package/dist/utils/windowsPaths.d.ts

@@ -0,0 +1,28 @@
+/**
+ * Windows path conversion utilities for Git Bash (POSIX shell) execution.
+ *
+ * Hook commands on Windows are executed via Git Bash instead of cmd.exe
+ * (cmd.exe does not strip quotes from arguments after the first token, so
+ * `node "C:\path\script.js"` silently fails — see issue #1773). Git Bash
+ * cannot resolve Windows paths like `C:\Users\foo`: backslashes are treated
+ * as escape characters and the drive letter breaks POSIX semantics. These
+ * helpers convert Windows paths to the POSIX form Git Bash expects
+ * (`C:\Users\foo` → `/c/Users/foo`), matching Claude Code's
+ * `windowsPathToPosixPath`.
+ */
+/**
+ * Convert a single Windows path to POSIX form for Git Bash:
+ *   - `C:\Users\foo`  → `/c/Users/foo`
+ *   - `C:/Users/foo`  → `/c/Users/foo`
+ *   - `\\server\share` → `//server/share` (UNC preserved)
+ *   - Already POSIX or relative paths are returned with slashes flipped.
+ */
+export declare function windowsPathToPosixPath(p: string): string;
+/**
+ * Convert every Windows absolute path inside a shell command string to POSIX
+ * form so Git Bash can parse it. Handles both quoted paths (which may contain
+ * spaces, e.g. `node "C:\Program Files\script.js"`) and bare paths
+ * (`cd C:\path\x`). Drive letters preceded by another alphanumeric character
+ * (e.g. URLs like `http://`) are left untouched.
+ */
+export declare function toPosixCommand(command: string): string;

package/dist/utils/windowsPaths.js

@@ -0,0 +1,47 @@
+/**
+ * Windows path conversion utilities for Git Bash (POSIX shell) execution.
+ *
+ * Hook commands on Windows are executed via Git Bash instead of cmd.exe
+ * (cmd.exe does not strip quotes from arguments after the first token, so
+ * `node "C:\path\script.js"` silently fails — see issue #1773). Git Bash
+ * cannot resolve Windows paths like `C:\Users\foo`: backslashes are treated
+ * as escape characters and the drive letter breaks POSIX semantics. These
+ * helpers convert Windows paths to the POSIX form Git Bash expects
+ * (`C:\Users\foo` → `/c/Users/foo`), matching Claude Code's
+ * `windowsPathToPosixPath`.
+ */
+/**
+ * Convert a single Windows path to POSIX form for Git Bash:
+ *   - `C:\Users\foo`  → `/c/Users/foo`
+ *   - `C:/Users/foo`  → `/c/Users/foo`
+ *   - `\\server\share` → `//server/share` (UNC preserved)
+ *   - Already POSIX or relative paths are returned with slashes flipped.
+ */
+export function windowsPathToPosixPath(p) {
+    // UNC paths: \\server\share -> //server/share
+    if (p.startsWith("\\\\")) {
+        return p.replace(/\\/g, "/");
+    }
+    // Drive letter paths: C:\Users\foo -> /c/Users/foo
+    const match = p.match(/^([A-Za-z]):[\\/]/);
+    if (match) {
+        const driveLetter = match[1].toLowerCase();
+        return `/${driveLetter}${p.slice(2).replace(/\\/g, "/")}`;
+    }
+    // Already POSIX or relative — just flip slashes
+    return p.replace(/\\/g, "/");
+}
+/**
+ * Convert every Windows absolute path inside a shell command string to POSIX
+ * form so Git Bash can parse it. Handles both quoted paths (which may contain
+ * spaces, e.g. `node "C:\Program Files\script.js"`) and bare paths
+ * (`cd C:\path\x`). Drive letters preceded by another alphanumeric character
+ * (e.g. URLs like `http://`) are left untouched.
+ */
+export function toPosixCommand(command) {
+    // Quoted paths first (may contain spaces) — keep the surrounding quotes.
+    let converted = command.replace(/"([A-Za-z]):[\\/][^"]*"/g, (quoted) => `"${windowsPathToPosixPath(quoted.slice(1, -1))}"`);
+    // Bare (unquoted) paths.
+    converted = converted.replace(/(?<![A-Za-z0-9])([A-Za-z]):[\\/][^\s"']*/g, (match) => windowsPathToPosixPath(match));
+    return converted;
+}

package/dist/utils/worktreeSession.d.ts

@@ -22,4 +22,10 @@ export interface WorktreeSession {
     repoRoot: string;
     /** The HEAD commit of the original branch at worktree creation time */
     originalHeadCommit?: string;
+    /**
+     * True when this worktree was created by a WorktreeCreate hook (not by git).
+     * Deletion delegates to the WorktreeRemove hook; wave never runs
+     * `git worktree remove` for hook-based worktrees (aligned with Claude Code).
+     */
+    hookBased?: boolean;
 }

package/dist/utils/worktreeUtils.d.ts

@@ -41,6 +41,13 @@ export declare function createWorktree(name: string, cwd: string, options?: {
 export declare function performPostCreationSetup(worktreePath: string, repoRoot: string): Promise<void>;
 /**
  * Remove a git worktree and its branch.
+ *
+ * Removal is best-effort: `git worktree remove --force` deletes the worktree
+ * metadata before the working directory, and on Windows its recursive deletion
+ * is MAX_PATH-limited — deep paths (e.g. node_modules) can fail with "Filename
+ * too long", leaving an orphan directory. When git fails we fall back to
+ * fs.rmSync with an extended-length path (bypasses MAX_PATH) and prune stale
+ * metadata. Failures are logged but never block branch deletion.
  */
 export declare function removeWorktree(info: WorktreeInfo): void;
 /**

package/dist/utils/worktreeUtils.js

@@ -498,32 +498,77 @@ export async function performPostCreationSetup(worktreePath, repoRoot) {
     await copyLocalSettingsToWorktree(repoRoot, worktreePath);
     await copyWorktreeIncludeFiles(repoRoot, worktreePath);
 }
+/**
+ * On Windows, prefix an absolute path with the extended-length marker (`\\?\`)
+ * so recursive removal bypasses the 260-char MAX_PATH limit. POSIX paths are
+ * returned unchanged.
+ */
+function toExtendedLengthPath(worktreePath) {
+    if (process.platform !== "win32") {
+        return worktreePath;
+    }
+    const absolute = path.win32.resolve(worktreePath);
+    if (absolute.startsWith("\\\\?\\")) {
+        return absolute;
+    }
+    if (absolute.startsWith("\\\\")) {
+        // UNC path: \\server\share -> \\?\UNC\server\share
+        return `\\\\?\\UNC\\${absolute.slice(2)}`;
+    }
+    return `\\\\?\\${absolute}`;
+}
 /**
  * Remove a git worktree and its branch.
+ *
+ * Removal is best-effort: `git worktree remove --force` deletes the worktree
+ * metadata before the working directory, and on Windows its recursive deletion
+ * is MAX_PATH-limited — deep paths (e.g. node_modules) can fail with "Filename
+ * too long", leaving an orphan directory. When git fails we fall back to
+ * fs.rmSync with an extended-length path (bypasses MAX_PATH) and prune stale
+ * metadata. Failures are logged but never block branch deletion.
  */
 export function removeWorktree(info) {
     const repoRoot = info.repoRoot;
+    // Get current branch in worktree before removing
+    let currentBranch;
+    try {
+        currentBranch = execFileSync("git", ["rev-parse", "--abbrev-ref", "HEAD"], {
+            cwd: info.path,
+            encoding: "utf8",
+            stdio: ["ignore", "pipe", "ignore"],
+        }).trim();
+    }
+    catch {
+        // Ignore errors
+    }
+    // Remove worktree
     try {
-        // Get current branch in worktree before removing
-        let currentBranch;
-        try {
-            currentBranch = execFileSync("git", ["rev-parse", "--abbrev-ref", "HEAD"], {
-                cwd: info.path,
-                encoding: "utf8",
-                stdio: ["ignore", "pipe", "ignore"],
-            }).trim();
-        }
-        catch {
-            // Ignore errors
-        }
-        // Remove worktree
         execFileSync("git", ["worktree", "remove", "--force", info.path], {
             cwd: repoRoot,
             stdio: ["ignore", "pipe", "pipe"],
         });
-        // Delete worktree branch
+    }
+    catch (error) {
+        logger.warn("git worktree remove failed, falling back to fs.rmSync:", {
+            error: error instanceof Error ? error.message : String(error),
+            worktreePath: info.path,
+        });
+        try {
+            fs.rmSync(toExtendedLengthPath(info.path), {
+                recursive: true,
+                force: true,
+            });
+        }
+        catch (rmError) {
+            logger.error("Failed to remove worktree or branch:", {
+                error: rmError instanceof Error ? rmError.message : String(rmError),
+                worktreePath: info.path,
+            });
+        }
+        // git removes worktree metadata before the working directory; prune any
+        // leftovers in case git failed before deleting them.
         try {
-            execFileSync("git", ["branch", "-D", info.branch], {
+            execFileSync("git", ["worktree", "prune"], {
                 cwd: repoRoot,
                 stdio: ["ignore", "pipe", "pipe"],
             });
@@ -531,33 +576,36 @@ export function removeWorktree(info) {
         catch {
             // Ignore errors
         }
-        // Delete current branch if different and not protected
-        if (currentBranch &&
-            currentBranch !== info.branch &&
-            currentBranch !== "HEAD") {
-            const defaultRemoteBranch = getDefaultRemoteBranch(repoRoot);
-            const defaultBranchName = defaultRemoteBranch.split("/").pop();
-            if (currentBranch !== defaultBranchName &&
-                currentBranch !== "main" &&
-                currentBranch !== "master") {
-                try {
-                    execFileSync("git", ["branch", "-D", currentBranch], {
-                        cwd: repoRoot,
-                        stdio: ["ignore", "pipe", "pipe"],
-                    });
-                }
-                catch {
-                    // Ignore errors
-                }
-            }
-        }
     }
-    catch (error) {
-        logger.error("Failed to remove worktree or branch:", {
-            error: error instanceof Error ? error.message : String(error),
-            worktreePath: info.path,
+    // Delete worktree branch
+    try {
+        execFileSync("git", ["branch", "-D", info.branch], {
+            cwd: repoRoot,
+            stdio: ["ignore", "pipe", "pipe"],
         });
-        throw error;
+    }
+    catch {
+        // Ignore errors
+    }
+    // Delete current branch if different and not protected
+    if (currentBranch &&
+        currentBranch !== info.branch &&
+        currentBranch !== "HEAD") {
+        const defaultRemoteBranch = getDefaultRemoteBranch(repoRoot);
+        const defaultBranchName = defaultRemoteBranch.split("/").pop();
+        if (currentBranch !== defaultBranchName &&
+            currentBranch !== "main" &&
+            currentBranch !== "master") {
+            try {
+                execFileSync("git", ["branch", "-D", currentBranch], {
+                    cwd: repoRoot,
+                    stdio: ["ignore", "pipe", "pipe"],
+                });
+            }
+            catch {
+                // Ignore errors
+            }
+        }
     }
 }
 /**

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "wave-agent-sdk",
-  "version": "1.0.9",
+  "version": "1.1.0",
   "description": "SDK for building AI-powered development tools and agents",
   "keywords": [
     "ai",
@@ -36,7 +36,6 @@
   },
   "files": [
     "dist",
-    "builtin",
     "README.md"
   ],
   "dependencies": {
@@ -51,7 +50,6 @@
     "@vscode/ripgrep": "^1.18.0",
     "chokidar": "^4.0.3",
     "cron-parser": "^5.5.0",
-    "find-up": "^8.0.0",
     "fuzzysort": "^3.1.0",
     "glob": "^13.0.0",
     "lru-cache": "^11.3.5",
@@ -65,11 +63,10 @@
     "@vitest/coverage-v8": "^4.1.7",
     "rimraf": "^6.1.2",
     "tsc-alias": "^1.8.16",
-    "tsx": "^4.20.4",
     "vitest": "^4.1.7"
   },
   "engines": {
-    "node": ">=20"
+    "node": ">=22"
   },
   "license": "MIT",
   "scripts": {
@@ -78,6 +75,9 @@
     "watch": "tsc -p tsconfig.build.json --watch & tsc-alias -p tsconfig.build.json --watch",
     "test": "vitest run --reporter=dot",
     "test:coverage": "vitest run --coverage --reporter=dot",
+    "test:unit": "vitest run --reporter=dot --exclude 'tests/integration/**' --exclude '**/*.integration.test.ts'",
+    "test:unit:coverage": "vitest run --coverage --reporter=dot --exclude 'tests/integration/**' --exclude '**/*.integration.test.ts'",
+    "test:integration": "vitest run --reporter=dot tests/integration .integration.test",
     "lint": "eslint --cache",
     "format": "prettier --write ."
   }

package/builtin/plugins/sdd/.wave-plugin/plugin.json

@@ -1,8 +0,0 @@
-{
-  "name": "sdd",
-  "description": "Spec-first workflow: specify skill, SessionStart guidance, and spec-count validation.",
-  "version": "1.0.0",
-  "author": {
-    "name": "Wave Team"
-  }
-}

package/builtin/plugins/sdd/hooks/hooks.json

@@ -1,14 +0,0 @@
-{
-  "hooks": {
-    "SessionStart": [
-      {
-        "hooks": [
-          {
-            "type": "command",
-            "command": "node \"${WAVE_PLUGIN_ROOT}/scripts/session-start.js\""
-          }
-        ]
-      }
-    ]
-  }
-}

package/builtin/plugins/sdd/scripts/session-start.js

@@ -1,24 +0,0 @@
-#!/usr/bin/env node
-// SessionStart hook for the sdd built-in plugin.
-// Emits the spec-first workflow guidance as additionalContext (JSON form),
-// resolving the absolute path to the plugin's spec-count validator so the
-// agent can run it from its bash tool (which does not carry WAVE_PLUGIN_ROOT).
-import path from "node:path";
-
-const root =
-  process.env.WAVE_PLUGIN_ROOT ||
-  path.dirname(new URL("..", import.meta.url).pathname);
-const specCount = `node ${JSON.stringify(path.join(root, "scripts", "spec-count.js"))}`;
-
-const guidance = [
-  "Spec-First Workflow（规格优先工作流）：",
-  "- 需求增加或变更时，优先更新 spec：先更新对应规格说明（新增用户故事、验收场景），待用户确认 spec 后再实现代码。spec 是功能设计的权威来源，不是 changelog。",
-  "- 边界模糊时也先写 spec 草稿请用户确认，不要直接改代码。",
-  "- 规格编写技能（specify）由 AI 自动触发：对话中涉及新需求或需求变更时主动创建或更新规格文件，不需要用户手动调用（不出现在斜杠命令列表中）。",
-  `- 新增或修改 spec 后运行校验：${specCount}（自动检测 docs/specs/，否则 specs/，否则退出）。`,
-].join("\n");
-
-// JSON form → parsed as hookSpecificOutput.additionalContext by the hook manager.
-console.log(
-  JSON.stringify({ hookSpecificOutput: { additionalContext: guidance } }),
-);