@yogiadhik/skillcheck 0.2.0 → 0.2.1

package/README.md

@@ -94,11 +94,12 @@ because the point is the pattern, and the pattern repeats:
   official and community both. Quality is achievable; it's just not
   the default.
 
-Two honest footnotes. First: twice, the loudest "errors" in an audit
-turned out to be this linter's own false positives — links inside
-example code blocks, and templated {baseDir} targets that resolve at
-runtime. Both fixed (0.1.1, 0.1.3), both caught by checking findings
-against the accused code before believing them. Do the same with
+Two honest footnotes. First: three times now, the loudest "errors" in
+an audit turned out to be this linter's own false positives — links
+inside example code blocks, templated {baseDir} targets that resolve
+at runtime, and placeholder links in quoted prose like
+"[Team roster](its link)". All fixed (0.1.1, 0.1.3, 0.2.1), all caught
+by checking findings against the accused code before believing them. Do the same with
 anything this tool tells you. Second: these audits are how skillcheck
 gets tested — every rule here has drawn blood on real files or been
 fixed for crying wolf.

package/dist/lint/rules.js

@@ -18,8 +18,10 @@ export const descriptionQuality = (s) => {
     const out = [];
     if (d.length < 20)
         out.push(finding(s, 'description-quality', 'warning', 'description under 20 chars; unlikely to trigger reliably'));
-    if (d.length > 1000)
-        out.push(finding(s, 'description-quality', 'warning', 'description over 1000 chars; gets truncated in listings'));
+    // claude code truncates description + when_to_use at 1536 chars combined
+    const whenToUse = typeof s.frontmatter.when_to_use === 'string' ? s.frontmatter.when_to_use : '';
+    if (d.length + whenToUse.length > 1536)
+        out.push(finding(s, 'description-quality', 'warning', 'description over 1536 chars (counting when_to_use); gets truncated in listings'));
     if (!/\b(use|when|for|trigger)\b/i.test(d))
         out.push(finding(s, 'description-quality', 'warning', 'description has no trigger language ("use when ...") — the agent decides from this text alone'));
     return out;
@@ -53,6 +55,9 @@ export const relativeLinks = (s) => {
         // templated targets like {baseDir}/x.md resolve at runtime; can't judge them
         if (target.includes('{'))
             continue;
+        // a destination with a raw space never parses as a markdown link; it's placeholder prose
+        if (/\s/.test(target))
+            continue;
         if (!existsSync(resolve(s.dir, target.split('#')[0])))
             out.push(finding(s, 'relative-links', 'error', `broken relative link: ${target}`));
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yogiadhik/skillcheck",
-  "version": "0.2.0",
+  "version": "0.2.1",
   "description": "lint and behavioral tests for agent skills",
   "license": "MIT",
   "author": "Yogesh Adhikari",
@@ -29,6 +29,6 @@
     "@types/cross-spawn": "^6.0.6",
     "@types/node": "^26.1.1",
     "typescript": "^7.0.2",
-    "vitest": "^4.1.10"
+    "vitest": "^5.0.2"
   }
 }