load-skills 0.2.0 → 0.2.2

package/README.md

@@ -58,5 +58,4 @@ These warnings appear on `report.invalidSkills[*].warnings`.
 - `invalid_meta_name`: `name` is present but not a non-empty string.
 - `invalid_meta_description`: `description` is present but not a non-empty string.
 - `skill_md_content_size_limit_exceeded`: `SKILL.md` body exceeds 500 lines.
-- `reference_large_without_toc`: A `references/` file exceeds 300 lines without a table-of-contents marker.
 - `resource_read_error`: A resource file or directory could not be read during scanning.

package/dist/discovery.js

@@ -60,10 +60,15 @@ async function discoverNonRecursive(rootPath) {
     const entries = await readdir(rootPath, { withFileTypes: true });
     const paths = [];
     for (const entry of entries.sort((a, b) => a.name.localeCompare(b.name))) {
-        if (!entry.isDirectory()) {
+        if (!entry.isDirectory() && !entry.isSymbolicLink()) {
             continue;
         }
-        const candidate = path.join(rootPath, entry.name, "SKILL.md");
+        const candidateDir = path.join(rootPath, entry.name);
+        const candidateDirStat = await stat(candidateDir).catch(() => null);
+        if (!candidateDirStat?.isDirectory()) {
+            continue;
+        }
+        const candidate = path.join(candidateDir, "SKILL.md");
         if (await isReadableFile(candidate)) {
             paths.push(candidate);
         }
@@ -80,7 +85,11 @@ async function discoverRecursive(rootPath) {
                 paths.push(fullPath);
                 continue;
             }
-            if (entry.isDirectory()) {
+            if (entry.isDirectory() || entry.isSymbolicLink()) {
+                const directoryStat = await stat(fullPath).catch(() => null);
+                if (!directoryStat?.isDirectory()) {
+                    continue;
+                }
                 await walk(fullPath);
             }
         }

package/dist/loadSkills.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"loadSkills.d.ts","sourceRoot":"","sources":["../src/loadSkills.ts"],"names":[],"mappings":"AAMA,OAAO,KAAK,EAGV,gBAAgB,EAChB,gBAAgB,EAKjB,MAAM,YAAY,CAAC;AAOpB,wBAAsB,UAAU,CAC9B,MAAM,CAAC,EAAE,gBAAgB,GACxB,OAAO,CAAC,gBAAgB,CAAC,CAgH3B"}
+{"version":3,"file":"loadSkills.d.ts","sourceRoot":"","sources":["../src/loadSkills.ts"],"names":[],"mappings":"AAMA,OAAO,KAAK,EAGV,gBAAgB,EAChB,gBAAgB,EAKjB,MAAM,YAAY,CAAC;AAIpB,wBAAsB,UAAU,CAC9B,MAAM,CAAC,EAAE,gBAAgB,GACxB,OAAO,CAAC,gBAAgB,CAAC,CA2G3B"}

package/dist/loadSkills.js

@@ -4,7 +4,7 @@ import path from "node:path";
 import { discoverSkillFiles } from "./discovery.js";
 import { parseSkillDocument } from "./parseSkill.js";
 import { getErrorMessage, pathExists } from "./utils.js";
-import { applyValidationRules, validateLargeReferences, } from "./validateSkill.js";
+import { applyValidationRules } from "./validateSkill.js";
 export async function loadSkills(config) {
     const resolvedConfig = config ?? {};
     const discovery = await discoverSkillFiles(resolvedConfig);
@@ -22,8 +22,6 @@ export async function loadSkills(config) {
         skillWarnings.push(...parsed.warnings);
         const resourceScan = await collectResources(skillPath);
         skillWarnings.push(...resourceScan.warnings);
-        const largeReferenceWarnings = validateLargeReferences(resourceScan.referenceContents);
-        skillWarnings.push(...largeReferenceWarnings);
         const validated = applyValidationRules({
             meta: parsed.meta,
             content: parsed.content,
@@ -101,15 +99,9 @@ async function collectResources(skillPath) {
     const warnings = [];
     const referenceFiles = await listFilesRecursively(referencesRoot, warnings);
     const scriptFiles = await listFilesRecursively(scriptsRoot, warnings);
-    const referenceContents = [];
     for (const filePath of referenceFiles) {
         try {
-            const content = await readFile(filePath, "utf8");
-            referenceContents.push({
-                path: filePath,
-                content,
-                lineCount: content.length === 0 ? 0 : content.split(/\r?\n/).length,
-            });
+            await readFile(filePath, "utf8");
         }
         catch (error) {
             warnings.push({
@@ -127,7 +119,6 @@ async function collectResources(skillPath) {
     }
     return {
         references: referenceFiles.sort((a, b) => a.localeCompare(b)),
-        referenceContents,
         scripts: scripts.sort((a, b) => a.path.localeCompare(b.path)),
         warnings,
     };

package/dist/types.d.ts

@@ -1,5 +1,5 @@
 export type SkillScriptType = "javascript" | "typescript" | "python" | "shell" | "ruby" | "other";
-export type SkillWarningCode = "missing_frontmatter" | "invalid_yaml_frontmatter" | "missing_required_meta_name" | "missing_required_meta_description" | "invalid_meta_name" | "invalid_meta_description" | "skill_md_content_size_limit_exceeded" | "reference_large_without_toc" | "resource_read_error";
+export type SkillWarningCode = "missing_frontmatter" | "invalid_yaml_frontmatter" | "missing_required_meta_name" | "missing_required_meta_description" | "invalid_meta_name" | "invalid_meta_description" | "skill_md_content_size_limit_exceeded" | "resource_read_error";
 export interface SkillWarning {
     code: SkillWarningCode;
     message: string;

package/dist/types.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"types.d.ts","sourceRoot":"","sources":["../src/types.ts"],"names":[],"mappings":"AAAA,MAAM,MAAM,eAAe,GACvB,YAAY,GACZ,YAAY,GACZ,QAAQ,GACR,OAAO,GACP,MAAM,GACN,OAAO,CAAC;AAEZ,MAAM,MAAM,gBAAgB,GACxB,qBAAqB,GACrB,0BAA0B,GAC1B,4BAA4B,GAC5B,mCAAmC,GACnC,mBAAmB,GACnB,0BAA0B,GAC1B,sCAAsC,GACtC,6BAA6B,GAC7B,qBAAqB,CAAC;AAE1B,MAAM,WAAW,YAAY;IAC3B,IAAI,EAAE,gBAAgB,CAAC;IACvB,OAAO,EAAE,MAAM,CAAC;CACjB;AAED,MAAM,WAAW,WAAW;IAC1B,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,EAAE,eAAe,CAAC;CACvB;AAED,MAAM,MAAM,SAAS,GAAG,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GAAG;IAChD,IAAI,EAAE,MAAM,CAAC;IACb,WAAW,EAAE,MAAM,CAAC;CACrB,CAAC;AAEF,MAAM,WAAW,KAAK;IACpB,IAAI,EAAE,SAAS,CAAC;IAChB,OAAO,EAAE,MAAM,CAAC;IAChB,UAAU,EAAE,MAAM,EAAE,CAAC;IACrB,OAAO,EAAE,WAAW,EAAE,CAAC;IACvB,SAAS,EAAE,MAAM,CAAC;IAClB,aAAa,EAAE,MAAM,CAAC;CACvB;AAED,MAAM,WAAW,YAAY;IAC3B,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC;IAC9B,OAAO,EAAE,MAAM,CAAC;IAChB,UAAU,EAAE,MAAM,EAAE,CAAC;IACrB,OAAO,EAAE,WAAW,EAAE,CAAC;IACvB,QAAQ,EAAE,YAAY,EAAE,CAAC;IACzB,SAAS,EAAE,MAAM,CAAC;IAClB,aAAa,EAAE,MAAM,CAAC;CACvB;AAED,MAAM,WAAW,gBAAgB;IAC/B,KAAK,CAAC,EAAE,MAAM,EAAE,CAAC;IACjB,SAAS,CAAC,EAAE,OAAO,CAAC;IACpB,GAAG,CAAC,EAAE,MAAM,CAAC;CACd;AAED,MAAM,MAAM,mBAAmB,GAAG,gBAAgB,GAAG,oBAAoB,CAAC;AAE1E,MAAM,WAAW,oBAAoB;IACnC,SAAS,EAAE,MAAM,CAAC;IAClB,YAAY,EAAE,MAAM,CAAC;IACrB,KAAK,EAAE,MAAM,CAAC;IACd,UAAU,EAAE,MAAM,EAAE,CAAC;IACrB,KAAK,CAAC,EAAE,mBAAmB,CAAC;CAC7B;AAED,MAAM,WAAW,qBAAqB;IACpC,SAAS,EAAE,MAAM,CAAC;IAClB,mBAAmB,EAAE,MAAM,CAAC;IAC5B,gBAAgB,EAAE,MAAM,CAAC;IACzB,oBAAoB,EAAE,MAAM,CAAC;IAC7B,oBAAoB,EAAE,MAAM,CAAC;IAC7B,aAAa,EAAE,MAAM,CAAC;IACtB,iBAAiB,EAAE,MAAM,CAAC;IAC1B,iBAAiB,EAAE,MAAM,CAAC;CAC3B;AAED,MAAM,WAAW,gBAAgB;IAC/B,KAAK,EAAE,oBAAoB,EAAE,CAAC;IAC9B,iBAAiB,EAAE,MAAM,CAAC,MAAM,EAAE,qBAAqB,EAAE,CAAC,CAAC;IAC3D,aAAa,EAAE,YAAY,EAAE,CAAC;CAC/B;AAED,MAAM,WAAW,gBAAgB;IAC/B,MAAM,EAAE,MAAM,CAAC,MAAM,EAAE,KAAK,CAAC,CAAC;IAC9B,MAAM,EAAE,gBAAgB,CAAC;CAC1B;AAED,MAAM,WAAW,gBAAgB;IAC/B,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC;IAC9B,OAAO,EAAE,MAAM,CAAC;IAChB,QAAQ,EAAE,YAAY,EAAE,CAAC;CAC1B"}
+{"version":3,"file":"types.d.ts","sourceRoot":"","sources":["../src/types.ts"],"names":[],"mappings":"AAAA,MAAM,MAAM,eAAe,GACvB,YAAY,GACZ,YAAY,GACZ,QAAQ,GACR,OAAO,GACP,MAAM,GACN,OAAO,CAAC;AAEZ,MAAM,MAAM,gBAAgB,GACxB,qBAAqB,GACrB,0BAA0B,GAC1B,4BAA4B,GAC5B,mCAAmC,GACnC,mBAAmB,GACnB,0BAA0B,GAC1B,sCAAsC,GACtC,qBAAqB,CAAC;AAE1B,MAAM,WAAW,YAAY;IAC3B,IAAI,EAAE,gBAAgB,CAAC;IACvB,OAAO,EAAE,MAAM,CAAC;CACjB;AAED,MAAM,WAAW,WAAW;IAC1B,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,EAAE,eAAe,CAAC;CACvB;AAED,MAAM,MAAM,SAAS,GAAG,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GAAG;IAChD,IAAI,EAAE,MAAM,CAAC;IACb,WAAW,EAAE,MAAM,CAAC;CACrB,CAAC;AAEF,MAAM,WAAW,KAAK;IACpB,IAAI,EAAE,SAAS,CAAC;IAChB,OAAO,EAAE,MAAM,CAAC;IAChB,UAAU,EAAE,MAAM,EAAE,CAAC;IACrB,OAAO,EAAE,WAAW,EAAE,CAAC;IACvB,SAAS,EAAE,MAAM,CAAC;IAClB,aAAa,EAAE,MAAM,CAAC;CACvB;AAED,MAAM,WAAW,YAAY;IAC3B,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC;IAC9B,OAAO,EAAE,MAAM,CAAC;IAChB,UAAU,EAAE,MAAM,EAAE,CAAC;IACrB,OAAO,EAAE,WAAW,EAAE,CAAC;IACvB,QAAQ,EAAE,YAAY,EAAE,CAAC;IACzB,SAAS,EAAE,MAAM,CAAC;IAClB,aAAa,EAAE,MAAM,CAAC;CACvB;AAED,MAAM,WAAW,gBAAgB;IAC/B,KAAK,CAAC,EAAE,MAAM,EAAE,CAAC;IACjB,SAAS,CAAC,EAAE,OAAO,CAAC;IACpB,GAAG,CAAC,EAAE,MAAM,CAAC;CACd;AAED,MAAM,MAAM,mBAAmB,GAAG,gBAAgB,GAAG,oBAAoB,CAAC;AAE1E,MAAM,WAAW,oBAAoB;IACnC,SAAS,EAAE,MAAM,CAAC;IAClB,YAAY,EAAE,MAAM,CAAC;IACrB,KAAK,EAAE,MAAM,CAAC;IACd,UAAU,EAAE,MAAM,EAAE,CAAC;IACrB,KAAK,CAAC,EAAE,mBAAmB,CAAC;CAC7B;AAED,MAAM,WAAW,qBAAqB;IACpC,SAAS,EAAE,MAAM,CAAC;IAClB,mBAAmB,EAAE,MAAM,CAAC;IAC5B,gBAAgB,EAAE,MAAM,CAAC;IACzB,oBAAoB,EAAE,MAAM,CAAC;IAC7B,oBAAoB,EAAE,MAAM,CAAC;IAC7B,aAAa,EAAE,MAAM,CAAC;IACtB,iBAAiB,EAAE,MAAM,CAAC;IAC1B,iBAAiB,EAAE,MAAM,CAAC;CAC3B;AAED,MAAM,WAAW,gBAAgB;IAC/B,KAAK,EAAE,oBAAoB,EAAE,CAAC;IAC9B,iBAAiB,EAAE,MAAM,CAAC,MAAM,EAAE,qBAAqB,EAAE,CAAC,CAAC;IAC3D,aAAa,EAAE,YAAY,EAAE,CAAC;CAC/B;AAED,MAAM,WAAW,gBAAgB;IAC/B,MAAM,EAAE,MAAM,CAAC,MAAM,EAAE,KAAK,CAAC,CAAC;IAC9B,MAAM,EAAE,gBAAgB,CAAC;CAC1B;AAED,MAAM,WAAW,gBAAgB;IAC/B,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC;IAC9B,OAAO,EAAE,MAAM,CAAC;IAChB,QAAQ,EAAE,YAAY,EAAE,CAAC;CAC1B"}

package/dist/validateSkill.d.ts

@@ -8,9 +8,4 @@ export declare function applyValidationRules(input: {
     isValid: boolean;
     meta?: SkillMeta;
 };
-export declare function validateLargeReferences(references: Array<{
-    path: string;
-    content: string;
-    lineCount: number;
-}>): SkillWarning[];
 //# sourceMappingURL=validateSkill.d.ts.map

package/dist/validateSkill.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"validateSkill.d.ts","sourceRoot":"","sources":["../src/validateSkill.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,YAAY,EAAE,MAAM,YAAY,CAAC;AAc1D,wBAAgB,oBAAoB,CAAC,KAAK,EAAE;IAC1C,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC;IAC9B,OAAO,EAAE,MAAM,CAAC;IAChB,QAAQ,EAAE,YAAY,EAAE,CAAC;CAC1B,GAAG;IACF,QAAQ,EAAE,YAAY,EAAE,CAAC;IACzB,OAAO,EAAE,OAAO,CAAC;IACjB,IAAI,CAAC,EAAE,SAAS,CAAC;CAClB,CAoDA;AAED,wBAAgB,uBAAuB,CACrC,UAAU,EAAE,KAAK,CAAC;IAAE,IAAI,EAAE,MAAM,CAAC;IAAC,OAAO,EAAE,MAAM,CAAC;IAAC,SAAS,EAAE,MAAM,CAAA;CAAE,CAAC,GACtE,YAAY,EAAE,CAqBhB"}
+{"version":3,"file":"validateSkill.d.ts","sourceRoot":"","sources":["../src/validateSkill.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,YAAY,EAAE,MAAM,YAAY,CAAC;AAa1D,wBAAgB,oBAAoB,CAAC,KAAK,EAAE;IAC1C,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC;IAC9B,OAAO,EAAE,MAAM,CAAC;IAChB,QAAQ,EAAE,YAAY,EAAE,CAAC;CAC1B,GAAG;IACF,QAAQ,EAAE,YAAY,EAAE,CAAC;IACzB,OAAO,EAAE,OAAO,CAAC;IACjB,IAAI,CAAC,EAAE,SAAS,CAAC;CAClB,CAoDA"}

package/dist/validateSkill.js

@@ -6,7 +6,6 @@ const INVALID_WARNING_CODES = new Set([
     "invalid_meta_name",
     "invalid_meta_description",
     "skill_md_content_size_limit_exceeded",
-    "reference_large_without_toc",
     "resource_read_error",
 ]);
 export function applyValidationRules(input) {
@@ -50,23 +49,6 @@ export function applyValidationRules(input) {
         isValid: false,
     };
 }
-export function validateLargeReferences(references) {
-    const warnings = [];
-    for (const reference of references) {
-        if (reference.lineCount <= 300) {
-            continue;
-        }
-        const normalized = reference.content.toLowerCase();
-        const hasTocMarker = normalized.includes("table of contents") || normalized.includes("[toc]");
-        if (!hasTocMarker) {
-            warnings.push({
-                code: "reference_large_without_toc",
-                message: `Large reference file is missing a table of contents hint: ${reference.path}`,
-            });
-        }
-    }
-    return warnings;
-}
 function dedupeWarnings(warnings) {
     const keys = new Set();
     const deduped = [];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "load-skills",
-  "version": "0.2.0",
+  "version": "0.2.2",
   "description": "Load and validate agent skills from configurable paths.",
   "type": "module",
   "main": "./dist/index.js",