wave-agent-sdk 0.6.4 → 0.7.0

package/dist/managers/permissionManager.js

@@ -10,20 +10,22 @@ import { minimatch } from "minimatch";
 import { RESTRICTED_TOOLS } from "../types/permissions.js";
 import { splitBashCommand, stripEnvVars, stripRedirections, getSmartPrefix, DANGEROUS_COMMANDS, } from "../utils/bashParser.js";
 import { isPathInside } from "../utils/pathSafety.js";
-import { BASH_TOOL_NAME, EDIT_TOOL_NAME, MULTI_EDIT_TOOL_NAME, DELETE_FILE_TOOL_NAME, WRITE_TOOL_NAME, READ_TOOL_NAME, LS_TOOL_NAME, } from "../constants/tools.js";
+import { BASH_TOOL_NAME, EDIT_TOOL_NAME, WRITE_TOOL_NAME, READ_TOOL_NAME, LS_TOOL_NAME, } from "../constants/tools.js";
 const SAFE_COMMANDS = ["cd", "ls", "pwd", "true", "false"];
+import { logger } from "../utils/globalLogger.js";
 export class PermissionManager {
-    constructor(options = {}) {
+    constructor(container, options = {}) {
+        this.container = container;
         this.allowedRules = [];
         this.deniedRules = [];
         this.temporaryRules = [];
         this.additionalDirectories = [];
-        this.logger = options.logger;
         this.configuredDefaultMode = options.configuredDefaultMode;
         this.allowedRules = options.allowedRules || [];
         this.deniedRules = options.deniedRules || [];
         this.workdir = options.workdir;
         this.planFilePath = options.planFilePath;
+        this._logger = options.logger;
         this.updateAdditionalDirectories(options.additionalDirectories || []);
     }
     /**
@@ -37,7 +39,7 @@ export class PermissionManager {
      */
     updateConfiguredDefaultMode(defaultMode) {
         const oldEffectiveMode = this.getCurrentEffectiveMode();
-        this.logger?.debug("Updating configured default permission mode", {
+        logger?.debug("Updating configured default permission mode", {
             previous: this.configuredDefaultMode,
             new: defaultMode,
         });
@@ -45,7 +47,7 @@ export class PermissionManager {
         const newEffectiveMode = this.getCurrentEffectiveMode();
         if (oldEffectiveMode !== newEffectiveMode &&
             this.onConfiguredDefaultModeChange) {
-            this.logger?.debug("Effective permission mode changed due to configuration update", {
+            logger?.debug("Effective permission mode changed due to configuration update", {
                 oldMode: oldEffectiveMode,
                 newMode: newEffectiveMode,
             });
@@ -62,7 +64,7 @@ export class PermissionManager {
      * Update the allowed rules (e.g., when configuration reloads)
      */
     updateAllowedRules(rules) {
-        this.logger?.debug("Updating allowed permission rules", {
+        logger?.debug("Updating allowed permission rules", {
             count: rules.length,
         });
         this.allowedRules = rules;
@@ -71,7 +73,7 @@ export class PermissionManager {
      * Update the denied rules (e.g., when configuration reloads)
      */
     updateDeniedRules(rules) {
-        this.logger?.debug("Updating denied permission rules", {
+        logger?.debug("Updating denied permission rules", {
             count: rules.length,
         });
         this.deniedRules = rules;
@@ -80,7 +82,7 @@ export class PermissionManager {
      * Add temporary rules for the current session
      */
     addTemporaryRules(rules) {
-        this.logger?.debug("Adding temporary permission rules", {
+        logger?.debug("Adding temporary permission rules", {
             count: rules.length,
             rules,
         });
@@ -90,14 +92,14 @@ export class PermissionManager {
      * Clear all temporary rules
      */
     clearTemporaryRules() {
-        this.logger?.debug("Clearing temporary permission rules");
+        logger?.debug("Clearing temporary permission rules");
         this.temporaryRules = [];
     }
     /**
      * Update the additional directories (e.g., when configuration reloads)
      */
     updateAdditionalDirectories(directories) {
-        this.logger?.debug("Updating additional directories", {
+        logger?.debug("Updating additional directories", {
             count: directories.length,
         });
         this.additionalDirectories = directories.map((dir) => {
@@ -111,7 +113,7 @@ export class PermissionManager {
      * Update the working directory
      */
     updateWorkdir(workdir) {
-        this.logger?.debug("Updating working directory", {
+        logger?.debug("Updating working directory", {
             workdir,
         });
         this.workdir = workdir;
@@ -120,7 +122,7 @@ export class PermissionManager {
      * Set the current plan file path
      */
     setPlanFilePath(path) {
-        this.logger?.debug("Setting plan file path", { path });
+        logger?.debug("Setting plan file path", { path });
         this.planFilePath = path;
     }
     /**
@@ -148,7 +150,7 @@ export class PermissionManager {
                 return { isInside: true, resolvedPath: absolutePath };
             }
         }
-        this.logger?.debug("Path is outside Safe Zone", {
+        logger?.debug("Path is outside Safe Zone", {
             absolutePath,
             workdir: effectiveWorkdir,
             additionalDirectories: this.additionalDirectories,
@@ -167,7 +169,7 @@ export class PermissionManager {
     resolveEffectivePermissionMode(cliPermissionMode) {
         // CLI override takes highest precedence
         if (cliPermissionMode !== undefined) {
-            this.logger?.debug("Using CLI permission mode override", {
+            logger?.debug("Using CLI permission mode override", {
                 cliMode: cliPermissionMode,
                 configuredDefault: this.configuredDefaultMode,
             });
@@ -175,13 +177,13 @@ export class PermissionManager {
         }
         // Use configured default mode if available
         if (this.configuredDefaultMode !== undefined) {
-            this.logger?.debug("Using configured default permission mode", {
+            logger?.debug("Using configured default permission mode", {
                 configuredDefault: this.configuredDefaultMode,
             });
             return this.configuredDefaultMode;
         }
         // Fall back to system default
-        this.logger?.debug("Using system default permission mode");
+        logger?.debug("Using system default permission mode");
         return "default";
     }
     /**
@@ -189,7 +191,7 @@ export class PermissionManager {
      * Called by individual tools after validation/diff, before real operation
      */
     async checkPermission(context) {
-        this.logger?.debug("Checking permission for tool", {
+        logger?.debug("Checking permission for tool", {
             toolName: context.toolName,
             permissionMode: context.permissionMode,
             hasCallback: !!context.canUseToolCallback,
@@ -197,7 +199,7 @@ export class PermissionManager {
         // 0. Check denied rules first - Deny always takes precedence
         for (const rule of this.deniedRules) {
             if (this.matchesRule(context, rule)) {
-                this.logger?.warn("Permission denied by rule", {
+                logger?.warn("Permission denied by rule", {
                     toolName: context.toolName,
                     rule,
                 });
@@ -209,28 +211,22 @@ export class PermissionManager {
         }
         // 1. If bypassPermissions mode, always allow
         if (context.permissionMode === "bypassPermissions") {
-            this.logger?.debug("Permission bypassed for tool", {
+            logger?.debug("Permission bypassed for tool", {
                 toolName: context.toolName,
             });
             return { behavior: "allow" };
         }
-        // 1.1 If acceptEdits mode, allow Edit, MultiEdit, Delete, Write
+        // 1.1 If acceptEdits mode, allow Edit, Write
         if (context.permissionMode === "acceptEdits") {
-            const autoAcceptedTools = [
-                EDIT_TOOL_NAME,
-                MULTI_EDIT_TOOL_NAME,
-                DELETE_FILE_TOOL_NAME,
-                WRITE_TOOL_NAME,
-            ];
+            const autoAcceptedTools = [EDIT_TOOL_NAME, WRITE_TOOL_NAME];
             if (autoAcceptedTools.includes(context.toolName)) {
                 // Enforce Safe Zone for file operations
-                const targetPath = (context.toolInput?.file_path ||
-                    context.toolInput?.target_file);
+                const targetPath = context.toolInput?.file_path;
                 const workdir = context.toolInput?.workdir;
                 if (targetPath) {
                     const { isInside, resolvedPath } = this.isInsideSafeZone(targetPath, workdir);
                     if (!isInside) {
-                        this.logger?.info("File operation outside the Safe Zone in acceptEdits mode, falling back to manual confirmation", {
+                        logger?.info("File operation outside the Safe Zone in acceptEdits mode, falling back to manual confirmation", {
                             toolName: context.toolName,
                             targetPath,
                             resolvedPath,
@@ -238,7 +234,7 @@ export class PermissionManager {
                         // Fall through to normal permission check flow to trigger confirmation prompt
                     }
                     else {
-                        this.logger?.debug("Permission automatically accepted for tool in acceptEdits mode", {
+                        logger?.debug("Permission automatically accepted for tool in acceptEdits mode", {
                             toolName: context.toolName,
                         });
                         return { behavior: "allow" };
@@ -248,33 +244,21 @@ export class PermissionManager {
         }
         // 1.2 Check if tool call is allowed by persistent or temporary rules
         if (this.isAllowedByRule(context)) {
-            this.logger?.debug("Permission allowed by persistent rule", {
+            logger?.debug("Permission allowed by persistent rule", {
                 toolName: context.toolName,
             });
             return { behavior: "allow" };
         }
         // 1.3 If plan mode, allow Read-only tools and Edit/Write for plan file
         if (context.permissionMode === "plan") {
-            const writeTools = [
-                EDIT_TOOL_NAME,
-                MULTI_EDIT_TOOL_NAME,
-                WRITE_TOOL_NAME,
-                DELETE_FILE_TOOL_NAME,
-            ];
-            if (context.toolName === DELETE_FILE_TOOL_NAME) {
-                return {
-                    behavior: "deny",
-                    message: "Delete operations are not allowed in plan mode.",
-                };
-            }
+            const writeTools = [EDIT_TOOL_NAME, WRITE_TOOL_NAME];
             if (writeTools.includes(context.toolName)) {
-                const targetPath = (context.toolInput?.file_path ||
-                    context.toolInput?.target_file);
+                const targetPath = context.toolInput?.file_path;
                 if (this.planFilePath && targetPath) {
                     const absoluteTargetPath = path.resolve(targetPath);
                     const absolutePlanPath = path.resolve(this.planFilePath);
                     if (absoluteTargetPath === absolutePlanPath) {
-                        this.logger?.debug("Allowing write to plan file in plan mode", {
+                        logger?.debug("Allowing write to plan file in plan mode", {
                             toolName: context.toolName,
                             targetPath,
                         });
@@ -289,7 +273,7 @@ export class PermissionManager {
         }
         // 2. If not a restricted tool, always allow
         if (!this.isRestrictedTool(context.toolName)) {
-            this.logger?.debug("Tool is not restricted, allowing", {
+            logger?.debug("Tool is not restricted, allowing", {
                 toolName: context.toolName,
             });
             return { behavior: "allow" };
@@ -297,11 +281,11 @@ export class PermissionManager {
         // 3. If custom callback provided, call it and return result
         if (context.canUseToolCallback) {
             try {
-                this.logger?.debug("Calling custom permission callback for tool", {
+                logger?.debug("Calling custom permission callback for tool", {
                     toolName: context.toolName,
                 });
                 const decision = await context.canUseToolCallback(context);
-                this.logger?.debug("Custom callback returned decision", {
+                logger?.debug("Custom callback returned decision", {
                     toolName: context.toolName,
                     decision,
                 });
@@ -309,7 +293,7 @@ export class PermissionManager {
             }
             catch (error) {
                 const errorMessage = error instanceof Error ? error.message : String(error);
-                this.logger?.error("Error in permission callback", {
+                logger?.error("Error in permission callback", {
                     toolName: context.toolName,
                     error: errorMessage,
                 });
@@ -321,7 +305,7 @@ export class PermissionManager {
         }
         // 4. For default mode on restricted tools without callback, integrate with CLI confirmation
         // Note: CLI confirmation integration will be implemented in Phase 2
-        this.logger?.warn("No permission callback provided for restricted tool in default mode", {
+        logger?.warn("No permission callback provided for restricted tool in default mode", {
             toolName: context.toolName,
         });
         return {
@@ -334,7 +318,7 @@ export class PermissionManager {
      */
     isRestrictedTool(toolName) {
         const isRestricted = RESTRICTED_TOOLS.includes(toolName);
-        this.logger?.debug("Checking if tool is restricted", {
+        logger?.debug("Checking if tool is restricted", {
             toolName,
             isRestricted,
         });
@@ -362,14 +346,9 @@ export class PermissionManager {
             suggestedPrefix,
         };
         // Set hidePersistentOption for out-of-bounds file operations
-        const fileTools = [
-            EDIT_TOOL_NAME,
-            MULTI_EDIT_TOOL_NAME,
-            DELETE_FILE_TOOL_NAME,
-            WRITE_TOOL_NAME,
-        ];
+        const fileTools = [EDIT_TOOL_NAME, WRITE_TOOL_NAME];
         if (fileTools.includes(toolName)) {
-            const targetPath = (toolInput?.file_path || toolInput?.target_file);
+            const targetPath = toolInput?.file_path;
             const workdir = toolInput?.workdir;
             if (targetPath) {
                 const { isInside } = this.isInsideSafeZone(targetPath, workdir);
@@ -409,7 +388,7 @@ export class PermissionManager {
                 context.hidePersistentOption = true;
             }
         }
-        this.logger?.debug("Created permission context", {
+        logger?.debug("Created permission context", {
             toolName,
             permissionMode,
             hasCallback: !!callback,
@@ -453,13 +432,10 @@ export class PermissionManager {
             READ_TOOL_NAME,
             WRITE_TOOL_NAME,
             EDIT_TOOL_NAME,
-            MULTI_EDIT_TOOL_NAME,
-            DELETE_FILE_TOOL_NAME,
             LS_TOOL_NAME,
         ];
         if (pathTools.includes(toolName)) {
             const targetPath = (context.toolInput?.file_path ||
-                context.toolInput?.target_file ||
                 context.toolInput?.path);
             if (targetPath) {
                 return minimatch(targetPath, pattern, { dot: true });

package/dist/managers/planManager.d.ts

@@ -1,11 +1,9 @@
-import type { Logger } from "../types/core.js";
-/**
- * Manages plan files for plan mode
- */
+import { Container } from "../utils/container.js";
 export declare class PlanManager {
-    private logger?;
+    private container;
     private planDir;
-    constructor(logger?: Logger | undefined);
+    private currentPlanFilePath;
+    constructor(container: Container);
     /**
      * Ensures the plan directory exists and generates a new plan file path with a random name
      */

package/dist/managers/planManager.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"planManager.d.ts","sourceRoot":"","sources":["../../src/managers/planManager.ts"],"names":[],"mappings":"AAIA,OAAO,KAAK,EAAE,MAAM,EAAE,MAAM,kBAAkB,CAAC;AAE/C;;GAEG;AACH,qBAAa,WAAW;IAGV,OAAO,CAAC,MAAM,CAAC;IAF3B,OAAO,CAAC,OAAO,CAAS;gBAEJ,MAAM,CAAC,EAAE,MAAM,YAAA;IAInC;;OAEG;IACU,yBAAyB,CAAC,IAAI,CAAC,EAAE,MAAM,GAAG,OAAO,CAAC;QAC7D,IAAI,EAAE,MAAM,CAAC;QACb,IAAI,EAAE,MAAM,CAAC;KACd,CAAC;IAgBF;;OAEG;IACI,UAAU,IAAI,MAAM;CAG5B"}
+{"version":3,"file":"planManager.d.ts","sourceRoot":"","sources":["../../src/managers/planManager.ts"],"names":[],"mappings":"AAKA,OAAO,EAAE,SAAS,EAAE,MAAM,uBAAuB,CAAC;AAOlD,qBAAa,WAAW;IAIV,OAAO,CAAC,SAAS;IAH7B,OAAO,CAAC,OAAO,CAAS;IACxB,OAAO,CAAC,mBAAmB,CAAuB;gBAE9B,SAAS,EAAE,SAAS;IAIxC;;OAEG;IACU,yBAAyB,CAAC,IAAI,CAAC,EAAE,MAAM,GAAG,OAAO,CAAC;QAC7D,IAAI,EAAE,MAAM,CAAC;QACb,IAAI,EAAE,MAAM,CAAC;KACd,CAAC;IA+BF;;OAEG;IACI,UAAU,IAAI,MAAM;CAG5B"}

package/dist/managers/planManager.js

@@ -5,9 +5,11 @@ import { generateRandomName } from "../utils/nameGenerator.js";
 /**
  * Manages plan files for plan mode
  */
+import { logger } from "../utils/globalLogger.js";
 export class PlanManager {
-    constructor(logger) {
-        this.logger = logger;
+    constructor(container) {
+        this.container = container;
+        this.currentPlanFilePath = null;
         this.planDir = path.join(os.homedir(), ".wave", "plans");
     }
     /**
@@ -18,12 +20,24 @@ export class PlanManager {
             await fs.mkdir(this.planDir, { recursive: true });
         }
         catch (error) {
-            this.logger?.error(`Failed to create plan directory: ${this.planDir}`, error);
+            logger?.error(`Failed to create plan directory: ${this.planDir}`, error);
             throw error;
         }
         const name = generateRandomName(seed);
         const filePath = path.join(this.planDir, `${name}.md`);
-        this.logger?.info(`Generated plan file path: ${filePath}`);
+        if (this.currentPlanFilePath !== filePath) {
+            try {
+                await fs.unlink(filePath);
+            }
+            catch (error) {
+                if (error instanceof Error &&
+                    error.code !== "ENOENT") {
+                    logger?.error(`Failed to remove existing plan file: ${filePath}`, error);
+                }
+            }
+            this.currentPlanFilePath = filePath;
+        }
+        logger?.info(`Generated plan file path: ${filePath}`);
         return { path: filePath, name };
     }
     /**

package/dist/managers/pluginManager.d.ts

@@ -1,33 +1,21 @@
-import { Plugin, PluginConfig, Logger } from "../types/index.js";
-import { SkillManager } from "./skillManager.js";
-import { HookManager } from "./hookManager.js";
-import { LspManager } from "./lspManager.js";
-import { McpManager } from "./mcpManager.js";
-import { SlashCommandManager } from "./slashCommandManager.js";
-import { ConfigurationService } from "../services/configurationService.js";
+import { Plugin, PluginConfig } from "../types/index.js";
+import { Container } from "../utils/container.js";
 export interface PluginManagerOptions {
     workdir: string;
-    logger?: Logger;
-    skillManager?: SkillManager;
-    hookManager?: HookManager;
-    lspManager?: LspManager;
-    mcpManager?: McpManager;
-    slashCommandManager?: SlashCommandManager;
     enabledPlugins?: Record<string, boolean>;
-    configurationService?: ConfigurationService;
 }
 export declare class PluginManager {
+    private container;
     private plugins;
     private workdir;
-    private logger?;
-    private skillManager?;
-    private hookManager?;
-    private lspManager?;
-    private mcpManager?;
-    private slashCommandManager?;
     private enabledPlugins;
-    private configurationService?;
-    constructor(options: PluginManagerOptions);
+    constructor(container: Container, options: PluginManagerOptions);
+    private get skillManager();
+    private get hookManager();
+    private get lspManager();
+    private get mcpManager();
+    private get slashCommandManager();
+    private get configurationService();
     /**
      * Update enabled plugins configuration
      */

package/dist/managers/pluginManager.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"pluginManager.d.ts","sourceRoot":"","sources":["../../src/managers/pluginManager.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,MAAM,EAAE,YAAY,EAAE,MAAM,EAAE,MAAM,mBAAmB,CAAC;AAGjE,OAAO,EAAE,YAAY,EAAE,MAAM,mBAAmB,CAAC;AACjD,OAAO,EAAE,WAAW,EAAE,MAAM,kBAAkB,CAAC;AAC/C,OAAO,EAAE,UAAU,EAAE,MAAM,iBAAiB,CAAC;AAC7C,OAAO,EAAE,UAAU,EAAE,MAAM,iBAAiB,CAAC;AAC7C,OAAO,EAAE,mBAAmB,EAAE,MAAM,0BAA0B,CAAC;AAE/D,OAAO,EAAE,oBAAoB,EAAE,MAAM,qCAAqC,CAAC;AAE3E,MAAM,WAAW,oBAAoB;IACnC,OAAO,EAAE,MAAM,CAAC;IAChB,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,YAAY,CAAC,EAAE,YAAY,CAAC;IAC5B,WAAW,CAAC,EAAE,WAAW,CAAC;IAC1B,UAAU,CAAC,EAAE,UAAU,CAAC;IACxB,UAAU,CAAC,EAAE,UAAU,CAAC;IACxB,mBAAmB,CAAC,EAAE,mBAAmB,CAAC;IAC1C,cAAc,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC;IACzC,oBAAoB,CAAC,EAAE,oBAAoB,CAAC;CAC7C;AAED,qBAAa,aAAa;IACxB,OAAO,CAAC,OAAO,CAA6B;IAC5C,OAAO,CAAC,OAAO,CAAS;IACxB,OAAO,CAAC,MAAM,CAAC,CAAS;IACxB,OAAO,CAAC,YAAY,CAAC,CAAe;IACpC,OAAO,CAAC,WAAW,CAAC,CAAc;IAClC,OAAO,CAAC,UAAU,CAAC,CAAa;IAChC,OAAO,CAAC,UAAU,CAAC,CAAa;IAChC,OAAO,CAAC,mBAAmB,CAAC,CAAsB;IAClD,OAAO,CAAC,cAAc,CAA0B;IAChD,OAAO,CAAC,oBAAoB,CAAC,CAAuB;gBAExC,OAAO,EAAE,oBAAoB;IAYzC;;OAEG;IACH,oBAAoB,CAAC,cAAc,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GAAG,IAAI;IAInE;;OAEG;YACW,oBAAoB;IA2BlC;;OAEG;YACW,gBAAgB;IA2D9B;;;OAGG;IACG,WAAW,CAAC,OAAO,EAAE,YAAY,EAAE,GAAG,OAAO,CAAC,IAAI,CAAC;IAmBzD;;OAEG;IACH,UAAU,IAAI,MAAM,EAAE;IAItB;;OAEG;IACH,SAAS,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,GAAG,SAAS;CAG5C"}
+{"version":3,"file":"pluginManager.d.ts","sourceRoot":"","sources":["../../src/managers/pluginManager.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,MAAM,EAAE,YAAY,EAAE,MAAM,mBAAmB,CAAC;AAUzD,OAAO,EAAE,SAAS,EAAE,MAAM,uBAAuB,CAAC;AAElD,MAAM,WAAW,oBAAoB;IACnC,OAAO,EAAE,MAAM,CAAC;IAChB,cAAc,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC;CAC1C;AAED,qBAAa,aAAa;IAMtB,OAAO,CAAC,SAAS;IALnB,OAAO,CAAC,OAAO,CAA6B;IAC5C,OAAO,CAAC,OAAO,CAAS;IACxB,OAAO,CAAC,cAAc,CAA0B;gBAGtC,SAAS,EAAE,SAAS,EAC5B,OAAO,EAAE,oBAAoB;IAM/B,OAAO,KAAK,YAAY,GAEvB;IAED,OAAO,KAAK,WAAW,GAEtB;IAED,OAAO,KAAK,UAAU,GAErB;IAED,OAAO,KAAK,UAAU,GAErB;IAED,OAAO,KAAK,mBAAmB,GAE9B;IAED,OAAO,KAAK,oBAAoB,GAE/B;IAED;;OAEG;IACH,oBAAoB,CAAC,cAAc,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GAAG,IAAI;IAInE;;OAEG;YACW,oBAAoB;IAyBlC;;OAEG;YACW,gBAAgB;IAyD9B;;;OAGG;IACG,WAAW,CAAC,OAAO,EAAE,YAAY,EAAE,GAAG,OAAO,CAAC,IAAI,CAAC;IAmBzD;;OAEG;IACH,UAAU,IAAI,MAAM,EAAE;IAItB;;OAEG;IACH,SAAS,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,GAAG,SAAS;CAG5C"}

package/dist/managers/pluginManager.js

@@ -1,18 +1,31 @@
+import { logger } from "../utils/globalLogger.js";
 import { PluginLoader } from "../services/pluginLoader.js";
 import * as path from "path";
 import { MarketplaceService } from "../services/MarketplaceService.js";
 export class PluginManager {
-    constructor(options) {
+    constructor(container, options) {
+        this.container = container;
         this.plugins = new Map();
         this.workdir = options.workdir;
-        this.logger = options.logger;
-        this.skillManager = options.skillManager;
-        this.hookManager = options.hookManager;
-        this.lspManager = options.lspManager;
-        this.mcpManager = options.mcpManager;
-        this.slashCommandManager = options.slashCommandManager;
         this.enabledPlugins = options.enabledPlugins || {};
-        this.configurationService = options.configurationService;
+    }
+    get skillManager() {
+        return this.container.get("SkillManager");
+    }
+    get hookManager() {
+        return this.container.get("HookManager");
+    }
+    get lspManager() {
+        return this.container.get("LspManager");
+    }
+    get mcpManager() {
+        return this.container.get("McpManager");
+    }
+    get slashCommandManager() {
+        return this.container.get("SlashCommandManager");
+    }
+    get configurationService() {
+        return this.container.get("ConfigurationService");
     }
     /**
      * Update enabled plugins configuration
@@ -34,14 +47,14 @@ export class PluginManager {
             for (const p of installedRegistry.plugins) {
                 const pluginId = `${p.name}@${p.marketplace}`;
                 if (this.enabledPlugins[pluginId] !== true) {
-                    this.logger?.info(`Plugin ${pluginId} is not enabled via configuration`);
+                    logger?.info(`Plugin ${pluginId} is not enabled via configuration`);
                     continue;
                 }
                 await this.loadSinglePlugin(p.cachePath);
             }
         }
         catch (error) {
-            this.logger?.error("Failed to load installed plugins:", error);
+            logger?.error("Failed to load installed plugins:", error);
         }
     }
     /**
@@ -52,7 +65,7 @@ export class PluginManager {
             const manifest = await PluginLoader.loadManifest(absolutePath);
             if (this.plugins.has(manifest.name)) {
                 // If already loaded (e.g. via explicit config), skip
-                this.logger?.warn(`Plugin with name '${manifest.name}' is already loaded`);
+                logger?.warn(`Plugin with name '${manifest.name}' is already loaded`);
                 return;
             }
             const plugin = {
@@ -85,10 +98,10 @@ export class PluginManager {
                 this.hookManager.registerPluginHooks(plugin.hooksConfig);
             }
             this.plugins.set(manifest.name, plugin);
-            this.logger?.info(`Loaded plugin: ${manifest.name} v${manifest.version}`);
+            logger?.info(`Loaded plugin: ${manifest.name} v${manifest.version}`);
         }
         catch (error) {
-            this.logger?.error(`Failed to load plugin from ${absolutePath}`, error);
+            logger?.error(`Failed to load plugin from ${absolutePath}`, error);
         }
     }
     /**
@@ -99,7 +112,7 @@ export class PluginManager {
         // Load plugins from configuration (e.g. --plugin-dir) first to give them higher priority
         for (const config of configs) {
             if (config.type !== "local") {
-                this.logger?.warn(`Unsupported plugin type: ${config.type}`);
+                logger?.warn(`Unsupported plugin type: ${config.type}`);
                 continue;
             }
             const absolutePath = path.isAbsolute(config.path)

package/dist/managers/reversionManager.d.ts

@@ -1,9 +1,10 @@
 import { FileSnapshot } from "../types/reversion.js";
-import { ReversionService } from "../services/reversionService.js";
+import { Container } from "../utils/container.js";
 export declare class ReversionManager {
+    private container;
     private buffer;
-    private reversionService;
-    constructor(reversionService: ReversionService);
+    constructor(container: Container);
+    private get reversionService();
     /**
      * Records the current state of a file into a temporary buffer.
      * Returns a snapshotId.

package/dist/managers/reversionManager.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"reversionManager.d.ts","sourceRoot":"","sources":["../../src/managers/reversionManager.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,YAAY,EAAE,MAAM,uBAAuB,CAAC;AACrD,OAAO,EAAE,gBAAgB,EAAE,MAAM,iCAAiC,CAAC;AAEnE,qBAAa,gBAAgB;IAC3B,OAAO,CAAC,MAAM,CAAwC;IACtD,OAAO,CAAC,gBAAgB,CAAmB;gBAE/B,gBAAgB,EAAE,gBAAgB;IAI9C;;;OAGG;IACG,cAAc,CAClB,SAAS,EAAE,MAAM,EACjB,QAAQ,EAAE,MAAM,EAChB,SAAS,EAAE,QAAQ,GAAG,QAAQ,GAAG,QAAQ,GACxC,OAAO,CAAC,MAAM,CAAC;IAuBlB;;;OAGG;IACG,oBAAoB,CACxB,SAAS,EAAE,MAAM,EACjB,QAAQ,EAAE,MAAM,EAChB,SAAS,EAAE,QAAQ,GAAG,QAAQ,GAAG,QAAQ,GACxC,OAAO,CAAC,MAAM,CAAC;IAIlB;;;OAGG;IACG,cAAc,CAAC,UAAU,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC;IAoBvD,OAAO,CAAC,kBAAkB,CAAsB;IAEhD;;OAEG;IACH,6BAA6B,IAAI,YAAY,EAAE;IAM/C;;;OAGG;IACH,eAAe,CAAC,UAAU,EAAE,MAAM,GAAG,IAAI;IAIzC;;;;OAIG;IACG,QAAQ,CACZ,UAAU,EAAE,MAAM,EAAE,EACpB,WAAW,EAAE,OAAO,mBAAmB,EAAE,OAAO,EAAE,GACjD,OAAO,CAAC,MAAM,CAAC;CAqEnB"}
+{"version":3,"file":"reversionManager.d.ts","sourceRoot":"","sources":["../../src/managers/reversionManager.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,YAAY,EAAE,MAAM,uBAAuB,CAAC;AAErD,OAAO,EAAE,SAAS,EAAE,MAAM,uBAAuB,CAAC;AAElD,qBAAa,gBAAgB;IAGf,OAAO,CAAC,SAAS;IAF7B,OAAO,CAAC,MAAM,CAAwC;gBAElC,SAAS,EAAE,SAAS;IAExC,OAAO,KAAK,gBAAgB,GAE3B;IAED;;;OAGG;IACG,cAAc,CAClB,SAAS,EAAE,MAAM,EACjB,QAAQ,EAAE,MAAM,EAChB,SAAS,EAAE,QAAQ,GAAG,QAAQ,GAAG,QAAQ,GACxC,OAAO,CAAC,MAAM,CAAC;IAuBlB;;;OAGG;IACG,oBAAoB,CACxB,SAAS,EAAE,MAAM,EACjB,QAAQ,EAAE,MAAM,EAChB,SAAS,EAAE,QAAQ,GAAG,QAAQ,GAAG,QAAQ,GACxC,OAAO,CAAC,MAAM,CAAC;IAIlB;;;OAGG;IACG,cAAc,CAAC,UAAU,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC;IAoBvD,OAAO,CAAC,kBAAkB,CAAsB;IAEhD;;OAEG;IACH,6BAA6B,IAAI,YAAY,EAAE;IAM/C;;;OAGG;IACH,eAAe,CAAC,UAAU,EAAE,MAAM,GAAG,IAAI;IAIzC;;;;OAIG;IACG,QAAQ,CACZ,UAAU,EAAE,MAAM,EAAE,EACpB,WAAW,EAAE,OAAO,mBAAmB,EAAE,OAAO,EAAE,GACjD,OAAO,CAAC,MAAM,CAAC;CAqEnB"}

package/dist/managers/reversionManager.js

@@ -1,9 +1,12 @@
 import fs from "fs/promises";
 export class ReversionManager {
-    constructor(reversionService) {
+    constructor(container) {
+        this.container = container;
         this.buffer = new Map();
         this.committedSnapshots = [];
-        this.reversionService = reversionService;
+    }
+    get reversionService() {
+        return this.container.get("ReversionService");
     }
     /**
      * Records the current state of a file into a temporary buffer.

package/dist/managers/skillManager.d.ts

@@ -1,16 +1,17 @@
 import type { SkillManagerOptions, SkillMetadata, Skill, SkillToolArgs, SkillInvocationContext } from "../types/index.js";
+import { Container } from "../utils/container.js";
 /**
  * Manages skill discovery and loading
  */
 export declare class SkillManager {
+    private container;
     private personalSkillsPath;
     private scanTimeout;
-    private logger?;
     private workdir;
     private skillMetadata;
     private skillContent;
     private initialized;
-    constructor(options?: SkillManagerOptions);
+    constructor(container: Container, options?: SkillManagerOptions);
     /**
      * Initialize the skill manager by discovering available skills
      */

package/dist/managers/skillManager.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"skillManager.d.ts","sourceRoot":"","sources":["../../src/managers/skillManager.ts"],"names":[],"mappings":"AAGA,OAAO,KAAK,EACV,mBAAmB,EACnB,aAAa,EACb,KAAK,EAGL,aAAa,EACb,sBAAsB,EAEvB,MAAM,mBAAmB,CAAC;AAG3B;;GAEG;AACH,qBAAa,YAAY;IACvB,OAAO,CAAC,kBAAkB,CAAS;IACnC,OAAO,CAAC,WAAW,CAAS;IAC5B,OAAO,CAAC,MAAM,CAAC,CAAS;IACxB,OAAO,CAAC,OAAO,CAAS;IAExB,OAAO,CAAC,aAAa,CAAoC;IACzD,OAAO,CAAC,YAAY,CAA4B;IAChD,OAAO,CAAC,WAAW,CAAS;gBAEhB,OAAO,GAAE,mBAAwB;IAQ7C;;OAEG;IACG,UAAU,IAAI,OAAO,CAAC,IAAI,CAAC;IAwCjC;;OAEG;IACH,aAAa,IAAI,OAAO;IAIxB;;OAEG;IACH,kBAAkB,IAAI,aAAa,EAAE;IAQrC;;;OAGG;IACG,SAAS,CAAC,SAAS,EAAE,MAAM,GAAG,OAAO,CAAC,KAAK,GAAG,IAAI,CAAC;IAgBzD;;OAEG;YACW,cAAc;IAkB5B;;OAEG;YACW,uBAAuB;IAiFrC;;OAEG;YACW,oBAAoB;IAkBlC;;OAEG;IACG,YAAY,CAChB,IAAI,EAAE,aAAa,GAClB,OAAO,CAAC;QAAE,OAAO,EAAE,MAAM,CAAC;QAAC,OAAO,CAAC,EAAE,sBAAsB,CAAA;KAAE,CAAC;IAqCjE;;OAEG;IACH,OAAO,CAAC,kBAAkB;IAY1B;;OAEG;IACH,OAAO,CAAC,qBAAqB;IAc7B;;OAEG;IACH,oBAAoB,CAAC,MAAM,EAAE,KAAK,EAAE,GAAG,IAAI;CAc5C"}
+{"version":3,"file":"skillManager.d.ts","sourceRoot":"","sources":["../../src/managers/skillManager.ts"],"names":[],"mappings":"AAGA,OAAO,KAAK,EACV,mBAAmB,EACnB,aAAa,EACb,KAAK,EAGL,aAAa,EACb,sBAAsB,EACvB,MAAM,mBAAmB,CAAC;AAG3B,OAAO,EAAE,SAAS,EAAE,MAAM,uBAAuB,CAAC;AAGlD;;GAEG;AACH,qBAAa,YAAY;IAUrB,OAAO,CAAC,SAAS;IATnB,OAAO,CAAC,kBAAkB,CAAS;IACnC,OAAO,CAAC,WAAW,CAAS;IAC5B,OAAO,CAAC,OAAO,CAAS;IAExB,OAAO,CAAC,aAAa,CAAoC;IACzD,OAAO,CAAC,YAAY,CAA4B;IAChD,OAAO,CAAC,WAAW,CAAS;gBAGlB,SAAS,EAAE,SAAS,EAC5B,OAAO,GAAE,mBAAwB;IAQnC;;OAEG;IACG,UAAU,IAAI,OAAO,CAAC,IAAI,CAAC;IAoCjC;;OAEG;IACH,aAAa,IAAI,OAAO;IAIxB;;OAEG;IACH,kBAAkB,IAAI,aAAa,EAAE;IAQrC;;;OAGG;IACG,SAAS,CAAC,SAAS,EAAE,MAAM,GAAG,OAAO,CAAC,KAAK,GAAG,IAAI,CAAC;IAgBzD;;OAEG;YACW,cAAc;IAkB5B;;OAEG;YACW,uBAAuB;IAiFrC;;OAEG;YACW,oBAAoB;IAkBlC;;OAEG;IACG,YAAY,CAChB,IAAI,EAAE,aAAa,GAClB,OAAO,CAAC;QAAE,OAAO,EAAE,MAAM,CAAC;QAAC,OAAO,CAAC,EAAE,sBAAsB,CAAA;KAAE,CAAC;IAqCjE;;OAEG;IACH,OAAO,CAAC,kBAAkB;IAY1B;;OAEG;IACH,OAAO,CAAC,qBAAqB;IAc7B;;OAEG;IACH,oBAAoB,CAAC,MAAM,EAAE,KAAK,EAAE,GAAG,IAAI;CAc5C"}