@lousy-agents/mcp 5.18.1 → 5.19.0

package/dist/mcp-server.js

@@ -34967,8 +34967,9 @@ async function readVersionFileContent(targetDir, relativePath) {
                 continue;
             }
             if (await file_system_utils_pathExistsWithinRoot(targetDir, pmConfig.manifestFile)) {
-                const lockfilePath = pmConfig.lockfile ? await file_system_utils_resolveSafePath(targetDir, pmConfig.lockfile) : undefined;
-                const hasLockfile = lockfilePath ? await file_system_utils_pathExistsWithinRoot(targetDir, pmConfig.lockfile) : false;
+                const { lockfile } = pmConfig;
+                const lockfilePath = lockfile ? await file_system_utils_resolveSafePath(targetDir, lockfile) : undefined;
+                const hasLockfile = lockfilePath && lockfile ? await file_system_utils_pathExistsWithinRoot(targetDir, lockfile) : false;
                 if (pmConfig.requiresLockfile && !hasLockfile) {
                     continue;
                 }
@@ -34985,8 +34986,9 @@ async function readVersionFileContent(targetDir, relativePath) {
         const packageManagers = [];
         for (const pmConfig of otherPackageManagers){
             if (await file_system_utils_pathExistsWithinRoot(targetDir, pmConfig.manifestFile)) {
-                const lockfilePath = pmConfig.lockfile ? await file_system_utils_resolveSafePath(targetDir, pmConfig.lockfile) : undefined;
-                const hasLockfile = lockfilePath ? await file_system_utils_pathExistsWithinRoot(targetDir, pmConfig.lockfile) : false;
+                const { lockfile } = pmConfig;
+                const lockfilePath = lockfile ? await file_system_utils_resolveSafePath(targetDir, lockfile) : undefined;
+                const hasLockfile = lockfilePath && lockfile ? await file_system_utils_pathExistsWithinRoot(targetDir, lockfile) : false;
                 if (pmConfig.requiresLockfile && !hasLockfile) {
                     continue;
                 }
@@ -35148,11 +35150,18 @@ function defaultExec(command, args, options) {
             throw new Error("Not authenticated");
         }
         try {
-            await this.octokit.rest.repos.createRepoRuleset({
+            const params = {
                 owner,
                 repo,
-                ...payload
-            });
+                name: payload.name,
+                enforcement: payload.enforcement,
+                target: payload.target,
+                // biome-ignore lint/style/useNamingConvention: GitHub API schema requires snake_case
+                bypass_actors: payload.bypass_actors,
+                conditions: payload.conditions,
+                rules: payload.rules
+            };
+            await this.octokit.rest.repos.createRepoRuleset(params);
         } catch (error) {
             const details = formatOctokitError(error);
             throw new Error(`Failed to create ruleset for ${owner}/${repo}: ${details}`);
@@ -37626,13 +37635,13 @@ const PackageJsonSchema = schemas_object({
             "shared"
         ]
     },
-    // Doctor-only skill locations
+    // Additional skill roots (SKILL.md convention, no alternate layout documented)
     {
         id: "skill-pi",
         path: ".pi/skills",
         matchKind: "directory-prefix",
         primaryConstruct: "skill",
-        lintTarget: "none",
+        lintTarget: "skills",
         harnessHints: [
             "pi"
         ]
@@ -37642,7 +37651,7 @@ const PackageJsonSchema = schemas_object({
         path: ".pi/prompts",
         matchKind: "directory-prefix",
         primaryConstruct: "skill",
-        lintTarget: "none",
+        lintTarget: "skills",
         harnessHints: [
             "pi"
         ]
@@ -37762,13 +37771,13 @@ const PackageJsonSchema = schemas_object({
         ],
         instructionFormat: "claude-md"
     },
-    // Doctor-only construct locations
+    // Subagent lint root
     {
         id: "subagent-claude",
         path: ".claude/agents",
         matchKind: "directory-prefix",
         primaryConstruct: "subagent",
-        lintTarget: "none",
+        lintTarget: "subagents",
         harnessHints: [
             "claude"
         ]
@@ -37804,11 +37813,14 @@ const PackageJsonSchema = schemas_object({
  * Normalizes a repo-relative path for catalog matching.
  * Collapses `.` / `..` segments. Returns "" when the path is absolute or
  * escapes the repository root — callers treat "" as non-matching.
- */ function normalizeRepoRelativePath(path) {
-    const normalized = path.replaceAll("\\", "/");
+ */ function isAbsoluteRepoPath(normalized) {
     // POSIX absolute, UNC-style leading slash after backslash conversion,
     // and Windows drive-letter absolute (C:/..., d:\...).
-    if (normalized.startsWith("/") || /^[A-Za-z]:(\/|$)/.test(normalized)) {
+    return normalized.startsWith("/") || /^[A-Za-z]:(\/|$)/.test(normalized);
+}
+function normalizeRepoRelativePath(path) {
+    const normalized = path.replaceAll("\\", "/");
+    if (isAbsoluteRepoPath(normalized)) {
         return "";
     }
     const resolved = [];
@@ -37816,10 +37828,10 @@ const PackageJsonSchema = schemas_object({
         if (segment === "" || segment === ".") {
             continue;
         }
+        if (segment === ".." && resolved.length === 0) {
+            return "";
+        }
         if (segment === "..") {
-            if (resolved.length === 0) {
-                return "";
-            }
             resolved.pop();
             continue;
         }
@@ -37867,14 +37879,15 @@ function constructTypesForPath(path, catalog = AGENTIC_LOCATION_CATALOG) {
     if (!best) {
         return [];
     }
-    if (best.secondaryConstructs && best.secondaryConstructs.length > 0) {
+    const secondary = best.secondaryConstructs ?? [];
+    if (secondary.length === 0) {
         return [
-            best.primaryConstruct,
-            ...best.secondaryConstructs
+            best.primaryConstruct
         ];
     }
     return [
-        best.primaryConstruct
+        best.primaryConstruct,
+        ...secondary
     ];
 }
 function lintTargetEntries(target, catalog = agentic_location_catalog_AGENTIC_LOCATION_CATALOG) {
@@ -37886,8 +37899,14 @@ function skillDirectoryRoots(catalog = agentic_location_catalog_AGENTIC_LOCATION
 function agentDirectoryRoots(catalog = AGENTIC_LOCATION_CATALOG) {
     return lintTargetEntries("agents", catalog).filter((entry)=>entry.matchKind === "directory-prefix").map((entry)=>entry.path);
 }
+function subagentDirectoryRoots(catalog = AGENTIC_LOCATION_CATALOG) {
+    return lintTargetEntries("subagents", catalog).filter((entry)=>entry.matchKind === "directory-prefix").map((entry)=>entry.path);
+}
+function isHookConfigEntry(entry) {
+    return entry.matchKind === "exact" && entry.hookPlatform != null;
+}
 function hookConfigPaths(catalog = AGENTIC_LOCATION_CATALOG) {
-    return lintTargetEntries("hooks", catalog).filter((entry)=>entry.matchKind === "exact" && entry.hookPlatform != null).map((entry)=>({
+    return lintTargetEntries("hooks", catalog).filter(isHookConfigEntry).map((entry)=>({
             relativePath: entry.path,
             platform: entry.hookPlatform
         }));
@@ -57474,120 +57493,6 @@ function lib_isUint8Array(value) {
  */
 const remark = unified().use(remarkParse).use(remarkStringify).freeze()
 
-;// CONCATENATED MODULE: ../core/src/gateways/markdown-ast-gateway.ts
-/**
- * Gateway for parsing Markdown files into AST using remark.
- * Extracts structural features like headings, code blocks, and paragraphs.
- */ 
-
-
-/** Maximum instruction file size: 1 MB */ const markdown_ast_gateway_MAX_INSTRUCTION_FILE_BYTES = 1_048_576;
-/**
- * Remark-based implementation of the Markdown AST gateway.
- */ class RemarkMarkdownAstGateway {
-    async parseFile(filePath) {
-        const content = await read_file_no_follow_readFileNoFollow(filePath, markdown_ast_gateway_MAX_INSTRUCTION_FILE_BYTES);
-        return this.parseContent(content);
-    }
-    parseContent(content) {
-        const processor = remark();
-        const ast = processor.parse(content);
-        const headings = [];
-        const codeBlocks = [];
-        const inlineCodes = [];
-        lib_visit(ast, "heading", (node)=>{
-            const text = this.extractHeadingText(node);
-            headings.push({
-                text,
-                depth: node.depth,
-                position: {
-                    line: node.position?.start.line ?? 0
-                }
-            });
-        });
-        // Track code blocks with their node index in root children
-        for(let i = 0; i < ast.children.length; i++){
-            const child = ast.children[i];
-            if (child.type === "code") {
-                const codeNode = child;
-                codeBlocks.push({
-                    value: codeNode.value,
-                    lang: codeNode.lang ?? undefined,
-                    position: {
-                        line: codeNode.position?.start.line ?? 0
-                    },
-                    nodeIndex: i
-                });
-            }
-        }
-        lib_visit(ast, "inlineCode", (node)=>{
-            inlineCodes.push({
-                value: node.value,
-                position: {
-                    line: node.position?.start.line ?? 0
-                }
-            });
-        });
-        return {
-            headings,
-            codeBlocks,
-            inlineCodes,
-            ast
-        };
-    }
-    findConditionalKeywordsInProximity(structure, codeBlockNodeIndex, proximityWindow, keywords) {
-        const ast = structure.ast;
-        const endIndex = Math.min(ast.children.length, codeBlockNodeIndex + 1 + proximityWindow);
-        for(let i = codeBlockNodeIndex + 1; i < endIndex; i++){
-            const sibling = ast.children[i];
-            const text = extractTextFromNode(sibling);
-            const lowerText = text.toLowerCase();
-            for (const keyword of keywords){
-                if (containsWordBoundary(lowerText, keyword.toLowerCase())) {
-                    return true;
-                }
-            }
-        }
-        return false;
-    }
-    extractHeadingText(node) {
-        const parts = [];
-        lib_visit(node, "text", (textNode)=>{
-            parts.push(textNode.value);
-        });
-        return parts.join("");
-    }
-}
-/**
- * Checks if text contains a keyword at a word boundary (safe string matching, no regex).
- */ function containsWordBoundary(text, keyword) {
-    const index = text.indexOf(keyword);
-    if (index === -1) {
-        return false;
-    }
-    const charBefore = index > 0 ? text[index - 1] : " ";
-    const charAfter = index + keyword.length < text.length ? text[index + keyword.length] : " ";
-    const isWordBoundaryBefore = !/[a-z0-9]/i.test(charBefore);
-    const isWordBoundaryAfter = !/[a-z0-9]/i.test(charAfter);
-    return isWordBoundaryBefore && isWordBoundaryAfter;
-}
-/**
- * Extracts plain text content from any AST node recursively.
- */ function extractTextFromNode(node) {
-    if (node.value && typeof node.value === "string") {
-        return node.value;
-    }
-    if (Array.isArray(node.children)) {
-        return node.children.map((child)=>extractTextFromNode(child)).join(" ");
-    }
-    return "";
-}
-/**
- * Creates and returns the default Markdown AST gateway.
- */ function createMarkdownAstGateway() {
-    return new RemarkMarkdownAstGateway();
-}
-
 ;// CONCATENATED MODULE: ../core/src/entities/instruction-quality.ts
 /**
  * Core domain entities for instruction quality analysis.
@@ -57639,6 +57544,9 @@ const HEADING_PATTERN_DESCRIPTIONS = new Map(Object.entries(HEADING_PATTERN_DESC
  * Orchestrates discovery, AST parsing, and scoring of feedback loop documentation.
  */ 
 
+/** Type guard narrowing an `unknown` AST child to {@link AstNodeLike}. */ function isAstNodeLike(node) {
+    return typeof node === "object" && node !== null && "type" in node && typeof node.type === "string";
+}
 /** Maximum number of raw heading pattern entries accepted before deduplication. */ const MAX_RAW_HEADING_PATTERNS = 1000;
 /** Maximum number of heading patterns accepted in a single execute() call. */ const MAX_HEADING_PATTERNS = 50;
 /** Maximum character length for a single heading pattern. */ const analyze_instruction_quality_MAX_PATTERN_LENGTH = 200;
@@ -57994,6 +57902,9 @@ const HEADING_PATTERN_DESCRIPTIONS = new Map(Object.entries(HEADING_PATTERN_DESC
         return parts.join(" ");
     }
     collectText(node, parts) {
+        if (!isAstNodeLike(node)) {
+            return;
+        }
         if (node.type === "text" && typeof node.value === "string") {
             parts.push(node.value);
         }
@@ -58140,6 +58051,124 @@ const HEADING_PATTERN_DESCRIPTIONS = new Map(Object.entries(HEADING_PATTERN_DESC
     }
 }
 
+;// CONCATENATED MODULE: ../core/src/gateways/markdown-ast-gateway.ts
+/**
+ * Gateway for parsing Markdown files into AST using remark.
+ * Extracts structural features like headings, code blocks, and paragraphs.
+ */ 
+
+
+
+/** Maximum instruction file size: 1 MB */ const markdown_ast_gateway_MAX_INSTRUCTION_FILE_BYTES = 1_048_576;
+/**
+ * Remark-based implementation of the Markdown AST gateway.
+ */ class RemarkMarkdownAstGateway {
+    async parseFile(filePath) {
+        const content = await read_file_no_follow_readFileNoFollow(filePath, markdown_ast_gateway_MAX_INSTRUCTION_FILE_BYTES);
+        return this.parseContent(content);
+    }
+    parseContent(content) {
+        const processor = remark();
+        const ast = processor.parse(content);
+        const headings = [];
+        const codeBlocks = [];
+        const inlineCodes = [];
+        lib_visit(ast, "heading", (node)=>{
+            const text = this.extractHeadingText(node);
+            headings.push({
+                text,
+                depth: node.depth,
+                position: {
+                    line: node.position?.start.line ?? 0
+                }
+            });
+        });
+        // Track code blocks with their node index in root children
+        for(let i = 0; i < ast.children.length; i++){
+            const child = ast.children[i];
+            if (child.type === "code") {
+                const codeNode = child;
+                codeBlocks.push({
+                    value: codeNode.value,
+                    lang: codeNode.lang ?? undefined,
+                    position: {
+                        line: codeNode.position?.start.line ?? 0
+                    },
+                    nodeIndex: i
+                });
+            }
+        }
+        lib_visit(ast, "inlineCode", (node)=>{
+            inlineCodes.push({
+                value: node.value,
+                position: {
+                    line: node.position?.start.line ?? 0
+                }
+            });
+        });
+        return {
+            headings,
+            codeBlocks,
+            inlineCodes,
+            ast
+        };
+    }
+    findConditionalKeywordsInProximity(structure, codeBlockNodeIndex, proximityWindow, keywords) {
+        const ast = structure.ast;
+        const endIndex = Math.min(ast.children.length, codeBlockNodeIndex + 1 + proximityWindow);
+        for(let i = codeBlockNodeIndex + 1; i < endIndex; i++){
+            const sibling = ast.children[i];
+            const text = extractTextFromNode(sibling);
+            const lowerText = text.toLowerCase();
+            for (const keyword of keywords){
+                if (containsWordBoundary(lowerText, keyword.toLowerCase())) {
+                    return true;
+                }
+            }
+        }
+        return false;
+    }
+    extractHeadingText(node) {
+        const parts = [];
+        lib_visit(node, "text", (textNode)=>{
+            parts.push(textNode.value);
+        });
+        return parts.join("");
+    }
+}
+/**
+ * Checks if text contains a keyword at a word boundary (safe string matching, no regex).
+ */ function containsWordBoundary(text, keyword) {
+    const index = text.indexOf(keyword);
+    if (index === -1) {
+        return false;
+    }
+    const charBefore = index > 0 ? text[index - 1] : " ";
+    const charAfter = index + keyword.length < text.length ? text[index + keyword.length] : " ";
+    const isWordBoundaryBefore = !/[a-z0-9]/i.test(charBefore);
+    const isWordBoundaryAfter = !/[a-z0-9]/i.test(charAfter);
+    return isWordBoundaryBefore && isWordBoundaryAfter;
+}
+/**
+ * Extracts plain text content from any AST node recursively.
+ */ function extractTextFromNode(node) {
+    if (!isAstNodeLike(node)) {
+        return "";
+    }
+    if (node.value && typeof node.value === "string") {
+        return node.value;
+    }
+    if (Array.isArray(node.children)) {
+        return node.children.map((child)=>extractTextFromNode(child)).join(" ");
+    }
+    return "";
+}
+/**
+ * Creates and returns the default Markdown AST gateway.
+ */ function createMarkdownAstGateway() {
+    return new RemarkMarkdownAstGateway();
+}
+
 ;// CONCATENATED MODULE: ./src/tools/analyze-instruction-quality.ts
 /**
  * MCP tool handler for analyzing instruction quality.

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@lousy-agents/mcp",
-    "version": "5.18.1",
+    "version": "5.19.0",
     "description": "MCP server for lousy-agents - provides AI coding assistant tools via the Model Context Protocol",
     "type": "module",
     "repository": {
@@ -33,7 +33,8 @@
         "lousy-agents-mcp": "dist/mcp-server.js"
     },
     "scripts": {
-        "build": "rspack build --config rspack.config.ts && chmod +x dist/mcp-server.js"
+        "build": "rspack build --config rspack.config.ts && chmod +x dist/mcp-server.js",
+        "typecheck": "tsc --noEmit"
     },
     "dependencies": {
         "@modelcontextprotocol/sdk": "1.30.0",