@huaqiu/dsh-kicad 0.4.3 → 0.4.5

package/lib/index.d.mts

@@ -230,6 +230,12 @@ interface SkillRegistration {
   description: string;
   /** Full SKILL.md body. */
   content: string;
+  /**
+   * Discovery source. The DSH registry requires it on every loaded skill and
+   * rejects a missing value with "source must be a string" at load time.
+   * `'runtime'` is the registry-owned source for plugin-bundled skills.
+   */
+  source: 'runtime';
   /** Where `references/` and `scripts/` live for progressive disclosure. */
   resourceBase: {
     kind: 'directory';

package/lib/index.mjs

@@ -1,6 +1,7 @@
 import { getLogger } from "@huaqiu/dsh-plugin-log";
 import { existsSync, readFileSync } from "node:fs";
 import { dirname, join, resolve } from "node:path";
+import { fileURLToPath } from "node:url";
 import { defineTool } from "@deepseek-ai/dsh-tools";
 import { spawn } from "node:child_process";
 const DEFAULT_TIMEOUT_MS = 3e4;
@@ -160,7 +161,7 @@ function resolveSkillDir(moduleUrl, override) {
 	if (override && override.trim().length > 0) return resolve(override);
 	const envOverride = process.env["DSH_KICAD_SKILLS_DIR"];
 	if (envOverride && envOverride.trim().length > 0) return resolve(envOverride);
-	const here = dirname(new URL(moduleUrl).pathname);
+	const here = dirname(fileURLToPath(moduleUrl));
 	return resolve(here, "..", "skills", KICAD_SKILL_NAME);
 }
 /**
@@ -963,6 +964,7 @@ function apply(ctx, config = {}) {
 		name: skill.name,
 		description: skill.description,
 		content: skill.content,
+		source: "runtime",
 		resourceBase: {
 			kind: "directory",
 			path: skill.dir

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@huaqiu/dsh-kicad",
-  "version": "0.4.3",
+  "version": "0.4.5",
   "type": "module",
   "main": "./lib/index.mjs",
   "types": "./lib/index.d.mts",
@@ -22,7 +22,7 @@
     "@deepseek-ai/dsh-tools": "^0.1.0-rc.0"
   },
   "dependencies": {
-    "@huaqiu/dsh-plugin-log": "0.4.3"
+    "@huaqiu/dsh-plugin-log": "0.4.5"
   },
   "files": [
     "lib",

package/src/index.ts

@@ -79,6 +79,12 @@ export interface SkillRegistration {
   description: string
   /** Full SKILL.md body. */
   content: string
+  /**
+   * Discovery source. The DSH registry requires it on every loaded skill and
+   * rejects a missing value with "source must be a string" at load time.
+   * `'runtime'` is the registry-owned source for plugin-bundled skills.
+   */
+  source: 'runtime'
   /** Where `references/` and `scripts/` live for progressive disclosure. */
   resourceBase: { kind: 'directory'; path: string }
 }
@@ -244,6 +250,7 @@ export function apply(ctx: Context, config: KicadConfigInput = {}): () => void {
         name: skill.name,
         description: skill.description,
         content: skill.content,
+        source: 'runtime',
         resourceBase: { kind: 'directory', path: skill.dir },
       }),
     )

package/src/paths.ts

@@ -17,6 +17,7 @@
  */
 import { existsSync } from 'node:fs'
 import { dirname, join, resolve } from 'node:path'
+import { fileURLToPath } from 'node:url'
 
 import { KICAD_SKILL_NAME } from './scripts.js'
 
@@ -35,7 +36,12 @@ export function resolveSkillDir(moduleUrl: string, override?: string): string {
   const envOverride = process.env['DSH_KICAD_SKILLS_DIR']
   if (envOverride && envOverride.trim().length > 0) return resolve(envOverride)
 
-  const here = dirname(new URL(moduleUrl).pathname)
+  // `new URL(moduleUrl).pathname` keeps a leading slash on Windows
+  // (`/C:/Users/...`), which `resolve()` then roots at the drive root and turns
+  // into `C:\C:\Users\...` — a doubled drive prefix. `fileURLToPath` decodes the
+  // file URL into a native path on every platform, so it is the correct input
+  // for `dirname`/`resolve`.
+  const here = dirname(fileURLToPath(moduleUrl))
   // One level up is right for both `lib/` (built) and `src/` (vitest) because
   // both sit directly under the package root next to `skills/`.
   return resolve(here, '..', 'skills', KICAD_SKILL_NAME)