local-skills-mcp 1.0.0 → 1.0.2

package/CHANGELOG.md

@@ -2,6 +2,14 @@
 
 All notable changes to this project will be documented in this file. See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## <small>1.0.2 (2026-08-24)</small>
+
+- fix: remove SIGINT listener on close, and collapse the duplicated changelog (#108) ([ab657dc](https://github.com/kdpa-llc/local-skills-mcp/commit/ab657dc)), closes [#108](https://github.com/kdpa-llc/local-skills-mcp/issues/108)
+
+## <small>1.0.1 (2026-08-24)</small>
+
+- fix(deps): regenerate the lockfile so npm ci works on Node 22 (#107) ([31bd1c1](https://github.com/kdpa-llc/local-skills-mcp/commit/31bd1c1)), closes [#107](https://github.com/kdpa-llc/local-skills-mcp/issues/107)
+
 ## 1.0.0 (2026-08-20)
 
 - feat!: signal the Node.js 22 requirement and make "!" mark a breaking change ([599a0fd](https://github.com/kdpa-llc/local-skills-mcp/commit/599a0fd)), closes [#98](https://github.com/kdpa-llc/local-skills-mcp/issues/98) [#98](https://github.com/kdpa-llc/local-skills-mcp/issues/98) [#98](https://github.com/kdpa-llc/local-skills-mcp/issues/98)
@@ -151,25 +159,11 @@ Claude-Session: https://claude.ai/code/session_015oP5dZ2WxDSSSpMoRtbASi
 - chore(deps-dev): bump the development-dependencies group with 2 updates (#37) ([6ce77d2](https://github.com/kdpa-llc/local-skills-mcp/commit/6ce77d2)), closes [#37](https://github.com/kdpa-llc/local-skills-mcp/issues/37)
 - docs: replace hardcoded values with dynamic references (#20) ([45a8d80](https://github.com/kdpa-llc/local-skills-mcp/commit/45a8d80)), closes [#20](https://github.com/kdpa-llc/local-skills-mcp/issues/20) [#20](https://github.com/kdpa-llc/local-skills-mcp/issues/20)
 
-# Changelog
-
-All notable changes to this project will be documented in this file.
-
-The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
-and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
-
-## [Unreleased]
-
-### Added
-
-- Package built-in skills directory as base default for all MCP instances
-- Self-documenting capabilities: `local-skills-mcp-usage`, `local-skills-mcp-guide`, and `skill-creator` skills
-- Skills are now available immediately after installation with zero configuration
-
-### Changed
+## Earlier history
 
-- Directory aggregation priority updated to include package built-in skills as lowest priority (first in list)
-- Updated documentation to reflect new built-in skills feature
+Entries below predate semantic-release taking over this file. They are kept
+verbatim; the `[Unreleased]` block that used to sit here described the
+built-in skills feature, which shipped in 0.2.0 and is covered above.
 
 ## [0.1.0] - 2025-11-01
 

package/dist/index.d.ts

@@ -103,6 +103,29 @@ export declare class LocalSkillsServer {
      */
     constructor(skillsDirs?: string[]);
     private setupErrorHandling;
+    /**
+     * SIGINT handler, kept as a stable reference so {@link close} can remove it.
+     *
+     * Registering an inline listener per instance leaked one listener per server
+     * that was ever constructed: nothing removed them, so a process creating
+     * several servers — every test run — walked into
+     * MaxListenersExceededWarning.
+     */
+    private readonly handleSigint;
+    /**
+     * Close the server and set the exit status, without calling process.exit().
+     *
+     * close() removes the SIGINT listener and closes the transport, leaving
+     * nothing to hold the event loop, so the process ends on its own with this
+     * status. Forcing the exit truncates whatever else is still unwinding, and
+     * is untestable here: process.exit() inside a fire-and-forget handler
+     * escapes any spy a test installs, because the handler returns before the
+     * promise settles. Vitest then reports "process.exit unexpectedly called"
+     * and fails the run even though every test passed.
+     *
+     * Returned so callers - and tests - can await the shutdown deterministically.
+     */
+    shutdown(): Promise<void>;
     private setupHandlers;
     private handleValidateSkill;
     private handleEvaluateSkill;

package/dist/index.js

@@ -195,11 +195,41 @@ export class LocalSkillsServer {
         this.server.onerror = (error) => {
             console.error("[MCP Error]", error);
         };
-        // eslint-disable-next-line @typescript-eslint/no-misused-promises
-        process.on("SIGINT", async () => {
-            await this.server.close();
-            process.exit(0);
-        });
+        process.on("SIGINT", this.handleSigint);
+    }
+    /**
+     * SIGINT handler, kept as a stable reference so {@link close} can remove it.
+     *
+     * Registering an inline listener per instance leaked one listener per server
+     * that was ever constructed: nothing removed them, so a process creating
+     * several servers — every test run — walked into
+     * MaxListenersExceededWarning.
+     */
+    handleSigint = () => {
+        void this.shutdown();
+    };
+    /**
+     * Close the server and set the exit status, without calling process.exit().
+     *
+     * close() removes the SIGINT listener and closes the transport, leaving
+     * nothing to hold the event loop, so the process ends on its own with this
+     * status. Forcing the exit truncates whatever else is still unwinding, and
+     * is untestable here: process.exit() inside a fire-and-forget handler
+     * escapes any spy a test installs, because the handler returns before the
+     * promise settles. Vitest then reports "process.exit unexpectedly called"
+     * and fails the run even though every test passed.
+     *
+     * Returned so callers - and tests - can await the shutdown deterministically.
+     */
+    async shutdown() {
+        try {
+            await this.close();
+            process.exitCode = 0;
+        }
+        catch (error) {
+            console.error("[MCP Error] Failed to close cleanly", error);
+            process.exitCode = 1;
+        }
     }
     setupHandlers() {
         // List available tools (dynamically generated to include current skill list)
@@ -460,6 +490,9 @@ export class LocalSkillsServer {
      * ```
      */
     async close() {
+        // Drop the signal handler first so repeated construct/close cycles do not
+        // accumulate listeners on the process.
+        process.off("SIGINT", this.handleSigint);
         await this.server.close();
         // Allow time for all handles to be released (important on Windows)
         await new Promise((resolve) => setTimeout(resolve, 100));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "local-skills-mcp",
-  "version": "1.0.0",
+  "version": "1.0.2",
   "description": "Universal MCP server enabling any LLM or AI agent to utilize expert skills from your local filesystem. Reduces context consumption through lazy loading - only skill names visible initially, full content loads on-demand. Works with Claude, Cline, custom agents, and any MCP-compatible client.",
   "type": "module",
   "main": "dist/index.js",