co-maintainer 0.4.1 → 0.4.2

package/README.md

@@ -1,5 +1,7 @@
 # co-maintainer
 
+[![NPM Version](https://img.shields.io/npm/v/co-maintainer)](https://www.npmjs.com/package/co-maintainer)
+
 `co-maintainer` analyzes a GitHub repository and writes repository-specific
 `SKILL.md` guidance that helps developers contribute changes more reliably.
 

package/dist/package.json

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

package/dist/src/knowledge/facts.js

@@ -314,9 +314,12 @@ export function extractFacts(source, options) {
                 .join("; ")}.`, "commit history", 2));
         }
     }
-    if (options.includePullRequestChanges &&
-        source.pullRequests.some((pr) => pr.changedFiles.length)) {
-        const files = counts(source.pullRequests.flatMap((pr) => pr.changedFiles)).slice(0, 10);
+    const currentPaths = new Set(source.tree);
+    const changedFiles = source.pullRequests
+        .flatMap((pr) => pr.changedFiles)
+        .filter((path) => currentPaths.has(path));
+    if (options.includePullRequestChanges && changedFiles.length) {
+        const files = counts(changedFiles).slice(0, 10);
         add(facts, fact("review-bar", `Frequently changed files include ${files
             .map(([path]) => `\`${path}\``)
             .join(", ")}; check nearby tests and workflows before editing.`, "pull request files", 2));

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

@@ -2,5 +2,6 @@ import type { Source } from "./types.ts";
 export type ValidationResult = {
     valid: boolean;
     errors: string[];
+    warnings: string[];
 };
-export declare function validateSkill(markdown: string, outputDirectory: string, source?: Source): Promise<ValidationResult>;
+export declare function validateSkill(markdown: string, outputDirectory: string, source?: Source, referenceIssues?: "error" | "warning"): Promise<ValidationResult>;

package/dist/src/knowledge/validate.js

@@ -1,6 +1,8 @@
 import { stat } from "../util/runtime.js";
-export async function validateSkill(markdown, outputDirectory, source) {
+export async function validateSkill(markdown, outputDirectory, source, referenceIssues = "error") {
     const errors = [];
+    const warnings = [];
+    const referenceProblems = referenceIssues === "error" ? errors : warnings;
     if (!markdown.startsWith("---\n"))
         errors.push("missing YAML frontmatter");
     if (!/^name:\s+\S+/m.test(markdown))
@@ -104,14 +106,14 @@ export async function validateSkill(markdown, outputDirectory, source) {
                     ![...paths].some((path) => wildcardPattern
                         ? wildcardPattern.test(path)
                         : path.startsWith(`${normalized}/`))) {
-                    errors.push(`referenced path is absent from 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} `))) {
-                errors.push(`referenced command is absent from source: ${reference}`);
+                referenceProblems.push(`referenced command is absent from source: ${reference}`);
             }
         }
     }
@@ -130,5 +132,5 @@ export async function validateSkill(markdown, outputDirectory, source) {
             errors.push(`linked file does not exist: ${link}`);
         }
     }
-    return { valid: errors.length === 0, errors };
+    return { valid: errors.length === 0, errors, warnings };
 }

package/dist/src/services/setup.js

@@ -211,7 +211,10 @@ export async function runInitOrRemake(options) {
         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));
+    const finalValidation = await timed("final skill validation", options.logTime, () => validateSkill(result.markdown, `${reposDir()}/${options.repo}`, 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("; ")}`);
     }

package/package.json

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