co-maintainer 0.4.10 → 0.4.11

package/dist/package.json

@@ -1,6 +1,6 @@
 {
     "name": "co-maintainer",
-    "version": "0.4.10",
+    "version": "0.4.11",
     "description": "Analyzes a GitHub repository and writes repository-specific contribution guidance.",
     "license": "MIT",
     "repository": {

package/dist/src/knowledge/validate.d.ts

@@ -4,4 +4,4 @@ export type ValidationResult = {
     errors: string[];
     warnings: string[];
 };
-export declare function validateSkill(markdown: string, outputDirectory: string, source?: Source, referenceIssues?: "error" | "warning"): Promise<ValidationResult>;
+export declare function validateSkill(markdown: string, source?: Source, referenceIssues?: "error" | "warning"): Promise<ValidationResult>;

package/dist/src/knowledge/validate.js

@@ -1,5 +1,4 @@
-import { stat } from "../util/runtime.js";
-export async function validateSkill(markdown, outputDirectory, source, referenceIssues = "error") {
+export async function validateSkill(markdown, source, referenceIssues = "error") {
     const errors = [];
     const warnings = [];
     const referenceProblems = referenceIssues === "error" ? errors : warnings;
@@ -33,54 +32,6 @@ export async function validateSkill(markdown, outputDirectory, source, reference
     if (source) {
         const paths = new Set([...source.tree, ...Object.keys(source.files)]);
         const repositoryName = String(source.repo.full_name ?? "");
-        const commands = new Set();
-        for (const [path, content] of Object.entries(source.files)) {
-            if (path.endsWith("package.json")) {
-                try {
-                    const scripts = JSON.parse(content)
-                        .scripts ?? {};
-                    const prefix = source.tree.some((item) => /pnpm-lock\.yaml$/.test(item))
-                        ? "pnpm"
-                        : source.tree.some((item) => /yarn\.lock$/.test(item))
-                            ? "yarn"
-                            : "npm";
-                    for (const name of Object.keys(scripts)) {
-                        commands.add(`${prefix} ${name}`);
-                        commands.add(`${prefix} run ${name}`);
-                        commands.add(String(scripts[name]));
-                    }
-                }
-                catch {
-                    // Ignore malformed manifests; syntax validation is outside this gate.
-                }
-            }
-            if (/deno\.jsonc?$/.test(path)) {
-                try {
-                    const tasks = JSON.parse(content).tasks ??
-                        {};
-                    for (const [name, task] of Object.entries(tasks)) {
-                        commands.add(`deno task ${name}`);
-                        commands.add(String(task));
-                    }
-                }
-                catch {
-                    // Ignore malformed manifests; syntax validation is outside this gate.
-                }
-            }
-            for (const match of content.matchAll(/^\s*run:\s*([^\s#].*?)\s*$/gm)) {
-                commands.add(match[1].replace(/^['"]|['"]$/g, ""));
-            }
-            for (const match of content.matchAll(/`((?:npm|pnpm|yarn|bun|deno|cargo|make|go|python|node|git)\s+[^`\n]+)`/g)) {
-                commands.add(match[1]);
-            }
-            for (const match of content.matchAll(/^\s*((?:npm|pnpm|yarn|bun|deno|cargo|make|go|python|node|git)\s+[^\n`]+)$/gm)) {
-                commands.add(match[1].trim());
-            }
-        }
-        if (source.tree.some((path) => /Cargo\.toml$/.test(path))) {
-            commands.add("cargo test");
-            commands.add("cargo build");
-        }
         const references = [...markdown.matchAll(/`([^`\n]+)`/g)].map((match) => match[1]);
         for (const reference of references) {
             const isQualifier = /^[a-z][a-z\d_-]*:/i.test(reference);
@@ -109,27 +60,14 @@ export async function validateSkill(markdown, outputDirectory, source, reference
                     referenceProblems.push(`referenced path is absent from source: ${reference}`);
                 }
             }
-            if (/^(?:npm|pnpm|yarn|bun|deno|cargo|make|go|python|node|git)\s/.test(reference) &&
-                ![...commands].some((command) => reference === command ||
-                    reference.startsWith(`${command} `) ||
-                    command.startsWith(`${reference} `))) {
-                referenceProblems.push(`referenced command is absent from source: ${reference}`);
-            }
         }
     }
-    const links = [...markdown.matchAll(/\[[^\]]+\]\(([^)]+)\)/g)].map((match) => match[1]);
-    for (const link of links) {
-        if (/^[a-z][a-z\d+.-]*:/i.test(link) ||
-            link.startsWith("/") ||
-            link.includes("..")) {
-            errors.push(`link is not relative: ${link}`);
-            continue;
-        }
-        try {
-            await stat(`${outputDirectory}/${link}`);
-        }
-        catch {
-            errors.push(`linked file does not exist: ${link}`);
+    for (const link of markdown.matchAll(/\[[^\]]+\]\(([^)]+)\)/g)) {
+        const target = link[1];
+        if (/^[a-z][a-z\d+.-]*:/i.test(target) ||
+            target.startsWith("/") ||
+            target.includes("..")) {
+            errors.push(`link is not relative: ${target}`);
         }
     }
     return { valid: errors.length === 0, errors, warnings };

package/dist/src/services/setup.js

@@ -204,19 +204,19 @@ export async function runInitOrRemake(options) {
     let result = await timed("assemble skill", options.logTime, () => assembleSkill(options.repo, facts, previousMarkdown, previous?.sectionHashes ?? {}, overrides));
     const reviewDocuments = buildReviewDocuments(facts);
     result.markdown = addReviewLink(result.markdown, reviewDocuments);
-    const validation = await timed("validate skill", options.logTime, () => validateSkill(result.markdown, `${reposDir()}/${options.repo}`, source));
+    const validation = await timed("validate skill", options.logTime, () => validateSkill(result.markdown, source));
     if (!validation.valid && Object.keys(overrides).length) {
         log("validate", `AI output rejected: ${validation.errors.join("; ")}`);
         overrides = {};
         result = await timed("reassemble valid skill", options.logTime, () => assembleSkill(options.repo, facts, previousMarkdown, previous?.sectionHashes ?? {}, overrides));
         result.markdown = addReviewLink(result.markdown, reviewDocuments);
     }
-    const finalValidation = await timed("final skill validation", options.logTime, () => validateSkill(result.markdown, `${reposDir()}/${options.repo}`, source, "warning"));
+    const finalValidation = await timed("final skill validation", options.logTime, () => validateSkill(result.markdown, source, "warning"));
     if (finalValidation.warnings.length) {
         log("validate", `stale references kept: ${finalValidation.warnings.join(", ")}`);
     }
     if (!finalValidation.valid) {
-        throw new Error(`generated skill is invalid: ${finalValidation.errors.join("; ")}`);
+        log("validate", `skill problems kept: ${finalValidation.errors.join("; ")}`);
     }
     await timed("write skill and state", options.logTime, async () => {
         const { withKnowledgeLock } = await import("../review/guides.js");

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "co-maintainer",
-  "version": "0.4.10",
+  "version": "0.4.11",
   "description": "Analyzes a GitHub repository and writes repository-specific contribution guidance.",
   "license": "MIT",
   "repository": {