@tanstack/intent 0.0.33 → 0.0.34

package/dist/cli.mjs

@@ -1,12 +1,12 @@
 #!/usr/bin/env node
 import "./utils-COlDcU72.mjs";
-import "./workspace-patterns-BN2A_60g.mjs";
-import { i as parseSkillUse } from "./skill-use-uwGleSOz.mjs";
-import { r as resolveSkillUse } from "./resolver-D2CgIYGg.mjs";
-import { t as resolveProjectContext } from "./project-context-IDLpJU3S.mjs";
-import { a as scanIntentsOrFail, c as fail, i as resolveStaleTargets, l as isCliFailure, n as runInstallCommand, o as scanOptionsFromGlobalFlags, r as getMetaDir, s as printWarnings } from "./install-PUnIfBNC.mjs";
-import { existsSync, readFileSync, readdirSync, realpathSync } from "node:fs";
-import { isAbsolute, join, relative, resolve, sep } from "node:path";
+import { n as findWorkspacePackages } from "./workspace-patterns-BN2A_60g.mjs";
+import { i as parseSkillUse } from "./skill-use-BzuuvLM7.mjs";
+import { r as resolveSkillUse } from "./resolver-Whd12ksO.mjs";
+import { t as resolveProjectContext } from "./project-context-alYMNoNa.mjs";
+import { a as scanIntentsOrFail, c as fail, i as resolveStaleTargets, l as isCliFailure, n as runInstallCommand, o as scanOptionsFromGlobalFlags, r as getMetaDir, s as printWarnings } from "./install-QjryhQtg.mjs";
+import { appendFileSync, existsSync, readFileSync, readdirSync, realpathSync } from "node:fs";
+import { basename, dirname, isAbsolute, join, relative, resolve, sep } from "node:path";
 import { fileURLToPath } from "node:url";
 import { cac } from "cac";
 
@@ -43,7 +43,7 @@ async function runListCommand(options, scanIntentsOrFail$1) {
 		console.log(JSON.stringify(result, null, 2));
 		return;
 	}
-	const { computeSkillNameWidth, printSkillTree, printTable } = await import("./display-DvLbcWzq.mjs");
+	const { computeSkillNameWidth, printSkillTree, printTable } = await import("./display-B3vkG99D.mjs");
 	const scanCoverage = formatScanCoverage(result);
 	if (result.packages.length === 0) {
 		console.log("No intent-enabled packages found.");
@@ -245,6 +245,10 @@ async function runSetupGithubActionsCommand(root, metaDir) {
 //#endregion
 //#region src/commands/stale.ts
 async function runStaleCommand(targetDir, options, resolveStaleTargets$1) {
+	if (options.githubReview) {
+		await runGithubReview(targetDir, options, resolveStaleTargets$1);
+		return;
+	}
 	const { reports, workflowAdvisories = [] } = await resolveStaleTargets$1(targetDir);
 	if (options.json) {
 		console.log(JSON.stringify(reports, null, 2));
@@ -273,9 +277,26 @@ async function runStaleCommand(targetDir, options, resolveStaleTargets$1) {
 		console.log();
 	}
 }
+async function runGithubReview(targetDir, options, resolveStaleTargets$1) {
+	const { collectStaleReviewItems, createFailedStaleReviewItem, createWorkflowAdvisoryReviewItems, writeStaleReviewWorkflowFiles } = await import("./workflow-review-Dz_ofcYQ.mjs");
+	const packageLabel = options.packageLabel ?? "workspace";
+	try {
+		const { reports, workflowAdvisories = [] } = await resolveStaleTargets$1(targetDir);
+		const items = [...collectStaleReviewItems(reports), ...createWorkflowAdvisoryReviewItems(packageLabel, workflowAdvisories)];
+		writeStaleReviewWorkflowFiles(items);
+		if (items.length === 0) console.log("No stale skills or coverage gaps found.");
+		else console.log(`Wrote ${items.length} intent skill review item(s).`);
+	} catch (err) {
+		writeStaleReviewWorkflowFiles([createFailedStaleReviewItem(packageLabel)]);
+		const message = err instanceof Error ? err.message : String(err);
+		console.log(`Intent stale check failed: ${message}`);
+		console.log("Wrote a review PR body so maintainers can inspect the logs.");
+	}
+}
 
 //#endregion
 //#region src/commands/validate.ts
+const agentSkillNamePattern = /^[a-z0-9]+(?:-[a-z0-9]+)*$/;
 function buildValidationFailure(errors, warnings) {
 	const lines = [
 		"",
@@ -310,114 +331,230 @@ function collectPackagingWarnings(context) {
 	}
 	return warnings;
 }
-async function runValidateCommand(dir) {
-	const [{ parse: parseYaml }, { findSkillFiles }] = await Promise.all([import("yaml"), import("./utils-dkVvY7D7.mjs")]);
-	const targetDir = dir ?? "skills";
-	const context = resolveProjectContext({
-		cwd: process.cwd(),
-		targetPath: targetDir
-	});
-	const skillsDir = context.targetSkillsDir ?? resolve(process.cwd(), targetDir);
-	if (!existsSync(skillsDir)) fail(`Skills directory not found: ${skillsDir}`);
-	const errors = [];
-	const skillFiles = findSkillFiles(skillsDir);
-	if (skillFiles.length === 0) fail("No SKILL.md files found");
-	for (const filePath of skillFiles) {
-		const rel = relative(process.cwd(), filePath);
-		const content = readFileSync(filePath, "utf8");
-		const match = content.match(/^---\r?\n([\s\S]*?)\r?\n---\r?\n([\s\S]*)/);
-		if (!match) {
-			errors.push({
-				file: rel,
-				message: "Missing or invalid frontmatter"
-			});
-			continue;
-		}
-		if (!match[1]) {
-			errors.push({
-				file: rel,
-				message: "Missing YAML frontmatter"
-			});
-			continue;
-		}
-		let fm;
-		try {
-			fm = parseYaml(match[1]);
-		} catch (err) {
-			const detail = err instanceof Error ? err.message : String(err);
-			errors.push({
+function formatWarning({ file, message }) {
+	return `${file}: ${message}`;
+}
+function isRecord(value) {
+	return !!value && typeof value === "object" && !Array.isArray(value);
+}
+function collectAgentSkillSpecWarnings({ filePath, fm, rel }) {
+	const warnings = [];
+	if (typeof fm.name === "string") {
+		if (fm.name.length > 64) warnings.push({
+			file: rel,
+			message: `Agent Skills spec warning: name exceeds 64 characters (${fm.name.length} chars)`
+		});
+		for (const segment of fm.name.split("/")) if (!agentSkillNamePattern.test(segment)) {
+			warnings.push({
 				file: rel,
-				message: `Invalid YAML frontmatter: ${detail}`
+				message: "Agent Skills spec warning: each name segment should use lowercase letters, numbers, and single hyphens only"
 			});
-			continue;
+			break;
 		}
-		if (!fm.name) errors.push({
+		const parentDir = basename(dirname(filePath));
+		if (!fm.name.includes("/") && fm.name !== parentDir) warnings.push({
 			file: rel,
-			message: "Missing required field: name"
+			message: "Agent Skills spec warning: name should match the parent directory name"
 		});
-		if (!fm.description) errors.push({
+	}
+	if (fm.license !== void 0 && (typeof fm.license !== "string" || fm.license.trim().length === 0)) warnings.push({
+		file: rel,
+		message: "Agent Skills spec warning: license should be a non-empty string"
+	});
+	if (fm.compatibility !== void 0) {
+		if (typeof fm.compatibility !== "string" || fm.compatibility.trim().length === 0) warnings.push({
 			file: rel,
-			message: "Missing required field: description"
+			message: "Agent Skills spec warning: compatibility should be a non-empty string"
 		});
-		if (typeof fm.name === "string") {
-			const expectedPath = relative(skillsDir, filePath).replace(/[/\\]SKILL\.md$/, "").split(sep).join("/");
-			if (fm.name !== expectedPath) errors.push({
-				file: rel,
-				message: `name "${fm.name}" does not match directory path "${expectedPath}"`
-			});
-		}
-		if (typeof fm.description === "string" && fm.description.length > 1024) errors.push({
+		else if (fm.compatibility.length > 500) warnings.push({
 			file: rel,
-			message: `Description exceeds 1024 character limit (${fm.description.length} chars)`
+			message: `Agent Skills spec warning: compatibility exceeds 500 characters (${fm.compatibility.length} chars)`
 		});
-		if (fm.type === "framework" && !Array.isArray(fm.requires)) errors.push({
+	}
+	if (fm.metadata !== void 0) {
+		if (!isRecord(fm.metadata)) warnings.push({
 			file: rel,
-			message: "Framework skills must have a \"requires\" field"
+			message: "Agent Skills spec warning: metadata should be a mapping"
 		});
-		const lineCount = content.split(/\r?\n/).length;
-		if (lineCount > 500) errors.push({
+		else if (Object.values(fm.metadata).some((value) => typeof value !== "string")) warnings.push({
 			file: rel,
-			message: `Exceeds 500 line limit (${lineCount} lines). Rewrite for conciseness: move API tables to references/, trim verbose examples, and remove content an agent already knows. Do not simply raise the limit.`
+			message: "Agent Skills spec warning: metadata values should be strings"
+		});
+	}
+	if (fm["allowed-tools"] !== void 0 && typeof fm["allowed-tools"] !== "string") warnings.push({
+		file: rel,
+		message: "Agent Skills spec warning: allowed-tools should be a space-separated string"
+	});
+	return warnings;
+}
+async function runValidateCommand(dir, options = {}) {
+	if (!options.githubSummary) {
+		await runValidateCommandInternal(dir);
+		return;
+	}
+	try {
+		await runValidateCommandInternal(dir);
+		writeGithubValidationSummary({ ok: true });
+	} catch (err) {
+		writeGithubValidationSummary({
+			ok: false,
+			message: validationErrorMessage(err)
 		});
+		throw err;
+	}
+}
+async function runValidateCommandInternal(dir) {
+	const [{ parse: parseYaml }, { findSkillFiles }] = await Promise.all([import("yaml"), import("./utils-dkVvY7D7.mjs")]);
+	const context = resolveProjectContext({
+		cwd: process.cwd(),
+		targetPath: dir
+	});
+	const explicitDir = dir !== void 0;
+	const skillsDirs = explicitDir ? [context.targetSkillsDir ?? resolve(process.cwd(), dir)] : collectDefaultSkillsDirs(context, findSkillFiles);
+	if (explicitDir && !existsSync(skillsDirs[0])) fail(`Skills directory not found: ${skillsDirs[0]}`);
+	const errors = [];
+	const warnings = [];
+	let validatedCount = 0;
+	if (explicitDir && findSkillFiles(skillsDirs[0]).length === 0) fail("No SKILL.md files found");
+	if (skillsDirs.length === 0) {
+		console.log("No skills/ directory found — skipping validation.");
+		return;
 	}
-	const artifactsDir = join(skillsDir, "_artifacts");
-	if (!context.isMonorepo && existsSync(artifactsDir)) for (const fileName of [
-		"domain_map.yaml",
-		"skill_spec.md",
-		"skill_tree.yaml"
-	]) {
-		const artifactPath = join(artifactsDir, fileName);
-		if (!existsSync(artifactPath)) {
-			errors.push({
-				file: relative(process.cwd(), artifactPath),
-				message: "Missing required artifact"
+	for (const skillsDir of skillsDirs) {
+		const skillFiles = findSkillFiles(skillsDir);
+		const validateContext = resolveProjectContext({
+			cwd: process.cwd(),
+			targetPath: skillsDir
+		});
+		for (const filePath of skillFiles) {
+			const rel = relative(process.cwd(), filePath);
+			const content = readFileSync(filePath, "utf8");
+			const match = content.match(/^---\r?\n([\s\S]*?)\r?\n---\r?\n([\s\S]*)/);
+			if (!match) {
+				errors.push({
+					file: rel,
+					message: "Missing or invalid frontmatter"
+				});
+				continue;
+			}
+			if (!match[1]) {
+				errors.push({
+					file: rel,
+					message: "Missing YAML frontmatter"
+				});
+				continue;
+			}
+			let fm;
+			try {
+				fm = parseYaml(match[1]);
+			} catch (err) {
+				const detail = err instanceof Error ? err.message : String(err);
+				errors.push({
+					file: rel,
+					message: `Invalid YAML frontmatter: ${detail}`
+				});
+				continue;
+			}
+			if (!fm.name) errors.push({
+				file: rel,
+				message: "Missing required field: name"
 			});
-			continue;
-		}
-		const content = readFileSync(artifactPath, "utf8");
-		if (content.trim().length === 0) {
-			errors.push({
-				file: relative(process.cwd(), artifactPath),
-				message: "Artifact file is empty"
+			if (!fm.description) errors.push({
+				file: rel,
+				message: "Missing required field: description"
 			});
-			continue;
-		}
-		if (fileName.endsWith(".yaml")) try {
-			parseYaml(content);
-		} catch (err) {
-			const detail = err instanceof Error ? err.message : String(err);
-			errors.push({
-				file: relative(process.cwd(), artifactPath),
-				message: `Invalid YAML in artifact file: ${detail}`
+			if (typeof fm.name === "string") {
+				const expectedPath = relative(skillsDir, filePath).replace(/[/\\]SKILL\.md$/, "").split(sep).join("/");
+				if (fm.name !== expectedPath) errors.push({
+					file: rel,
+					message: `name "${fm.name}" does not match directory path "${expectedPath}"`
+				});
+			}
+			if (typeof fm.description === "string" && fm.description.length > 1024) errors.push({
+				file: rel,
+				message: `Description exceeds 1024 character limit (${fm.description.length} chars)`
+			});
+			if (fm.type === "framework" && !Array.isArray(fm.requires)) errors.push({
+				file: rel,
+				message: "Framework skills must have a \"requires\" field"
+			});
+			warnings.push(...collectAgentSkillSpecWarnings({
+				filePath,
+				fm,
+				rel
+			}).map(formatWarning));
+			const lineCount = content.split(/\r?\n/).length;
+			if (lineCount > 500) errors.push({
+				file: rel,
+				message: `Exceeds 500 line limit (${lineCount} lines). Rewrite for conciseness: move API tables to references/, trim verbose examples, and remove content an agent already knows. Do not simply raise the limit.`
 			});
 		}
+		const artifactsDir = join(skillsDir, "_artifacts");
+		if (!validateContext.isMonorepo && existsSync(artifactsDir)) for (const fileName of [
+			"domain_map.yaml",
+			"skill_spec.md",
+			"skill_tree.yaml"
+		]) {
+			const artifactPath = join(artifactsDir, fileName);
+			if (!existsSync(artifactPath)) {
+				errors.push({
+					file: relative(process.cwd(), artifactPath),
+					message: "Missing required artifact"
+				});
+				continue;
+			}
+			const content = readFileSync(artifactPath, "utf8");
+			if (content.trim().length === 0) {
+				errors.push({
+					file: relative(process.cwd(), artifactPath),
+					message: "Artifact file is empty"
+				});
+				continue;
+			}
+			if (fileName.endsWith(".yaml")) try {
+				parseYaml(content);
+			} catch (err) {
+				const detail = err instanceof Error ? err.message : String(err);
+				errors.push({
+					file: relative(process.cwd(), artifactPath),
+					message: `Invalid YAML in artifact file: ${detail}`
+				});
+			}
+		}
+		validatedCount += skillFiles.length;
+		warnings.push(...collectPackagingWarnings(validateContext));
 	}
-	const warnings = collectPackagingWarnings(context);
 	if (errors.length > 0) fail(buildValidationFailure(errors, warnings));
-	console.log(`✅ Validated ${skillFiles.length} skill files — all passed`);
+	console.log(`✅ Validated ${validatedCount} skill files — all passed`);
 	if (warnings.length > 0) console.log();
 	printWarnings(warnings);
 }
+function validationErrorMessage(err) {
+	if (isCliFailure(err)) return err.message;
+	if (err instanceof Error) return err.message;
+	return String(err);
+}
+function writeGithubValidationSummary({ message, ok }) {
+	const summaryPath = process.env.GITHUB_STEP_SUMMARY;
+	if (!summaryPath) return;
+	const lines = ["### Intent skill validation", ""];
+	if (ok) lines.push("Skill validation passed.", "");
+	else lines.push("Skill validation failed.", "", "Why this failed:", "", "Intent validates SKILL.md frontmatter, skill names, required fields, size limits, framework requirements, and artifact files.", "The command output below contains the exact file-level reasons to fix.", "", "Run locally:", "", "```bash", "npx @tanstack/intent@latest validate", "```", "", "Command output:", "", "```text", message ?? "Unknown validation error.", "```", "");
+	appendFileSync(summaryPath, lines.join("\n"));
+}
+function collectDefaultSkillsDirs(context, findSkillFiles) {
+	const skillsDirs = [];
+	const addSkillsDir = (skillsDir) => {
+		if (existsSync(skillsDir) && findSkillFiles(skillsDir).length > 0) skillsDirs.push(skillsDir);
+	};
+	if (context.workspaceRoot && context.cwd === context.workspaceRoot) {
+		addSkillsDir(join(context.workspaceRoot, "skills"));
+		for (const packageDir of findWorkspacePackages(context.workspaceRoot)) addSkillsDir(join(packageDir, "skills"));
+		return [...new Set(skillsDirs)].sort((a, b) => a.localeCompare(b));
+	}
+	addSkillsDir(context.targetSkillsDir ?? (context.packageRoot ? join(context.packageRoot, "skills") : resolve(context.cwd, "skills")));
+	return skillsDirs;
+}
 
 //#endregion
 //#region src/cli.ts
@@ -433,8 +570,8 @@ function createCli() {
 	cli.command("meta [name]", "List meta-skills, or print one by name").usage("meta [name]").example("meta").example("meta domain-discovery").action(async (name) => {
 		await runMetaCommand(name, getMetaDir());
 	});
-	cli.command("validate [dir]", "Validate skill files").usage("validate [dir]").example("validate").example("validate packages/query/skills").action(async (dir) => {
-		await runValidateCommand(dir);
+	cli.command("validate [dir]", "Validate skill files").usage("validate [dir] [--github-summary]").option("--github-summary", "Write a GitHub Actions step summary").example("validate").example("validate packages/query/skills").action(async (dir, options) => {
+		await runValidateCommand(dir, options);
 	});
 	cli.command("install", "Create or update skill loading guidance in an agent config file").usage("install [--map] [--dry-run] [--print-prompt] [--global] [--global-only]").option("--map", "Write explicit skill-to-task mappings").option("--dry-run", "Print the generated block without writing").option("--print-prompt", "Print the legacy agent setup prompt instead of writing").option("--global", "Include global packages after project packages").option("--global-only", "Install mappings from global packages only").example("install").example("install --map").example("install --dry-run").example("install --print-prompt").example("install --global").action(async (options) => {
 		await runInstallCommand(options, scanIntentsOrFail);
@@ -442,7 +579,7 @@ function createCli() {
 	cli.command("scaffold", "Print maintainer scaffold prompt").usage("scaffold").action(() => {
 		runScaffoldCommand(getMetaDir());
 	});
-	cli.command("stale [dir]", "Check skills for staleness in the current package or workspace").usage("stale [dir] [--json]").option("--json", "Output JSON").example("stale").example("stale packages/query").example("stale --json").action(async (targetDir, options) => {
+	cli.command("stale [dir]", "Check skills for staleness in the current package or workspace").usage("stale [dir] [--json] [--github-review]").option("--json", "Output JSON").option("--github-review", "Write GitHub Actions review PR files").option("--package-label <label>", "Fallback package label for review PRs").example("stale").example("stale packages/query").example("stale --json").action(async (targetDir, options) => {
 		await runStaleCommand(targetDir, options, resolveStaleTargets);
 	});
 	cli.command("edit-package-json", "Update package.json files so skills are published").usage("edit-package-json").action(async () => {

package/dist/{display-DvLbcWzq.mjs → display-B3vkG99D.mjs}

@@ -1,5 +1,5 @@
 import "./utils-COlDcU72.mjs";
 import "./skill-paths-8k9K9y26.mjs";
-import { n as printSkillTree, r as printTable, t as computeSkillNameWidth } from "./display-BTZWCjzT.mjs";
+import { n as printSkillTree, r as printTable, t as computeSkillNameWidth } from "./display-CAof6doy.mjs";
 
 export { computeSkillNameWidth, printSkillTree, printTable };

package/dist/index.d.mts

@@ -1,5 +1,5 @@
-import { _ as StalenessReport, a as IntentArtifactSet, c as IntentConfig, d as MetaFeedbackPayload, f as MetaSkillName, g as SkillStaleness, h as SkillEntry, i as IntentArtifactFile, l as IntentPackage, m as ScanResult, n as FeedbackPayload, o as IntentArtifactSkill, p as ScanOptions, r as IntentArtifactCoverageIgnore, s as IntentArtifactWarning, t as AgentName, u as IntentProjectConfig, v as StalenessSignal, y as VersionConflict } from "./types-_y9b00bI.mjs";
-import { i as runEditPackageJson, o as runSetupGithubActions, r as SetupGithubActionsResult, t as EditPackageJsonResult } from "./setup-D2CGdTsx.mjs";
+import { _ as StalenessReport, a as IntentArtifactSet, c as IntentConfig, d as MetaFeedbackPayload, f as MetaSkillName, g as SkillStaleness, h as SkillEntry, i as IntentArtifactFile, l as IntentPackage, m as ScanResult, n as FeedbackPayload, o as IntentArtifactSkill, p as ScanOptions, r as IntentArtifactCoverageIgnore, s as IntentArtifactWarning, t as AgentName, u as IntentProjectConfig, v as StalenessSignal, y as VersionConflict } from "./types-DT7Y6TFz.mjs";
+import { i as runEditPackageJson, o as runSetupGithubActions, r as SetupGithubActionsResult, t as EditPackageJsonResult } from "./setup-t1i2o2-h.mjs";
 
 //#region src/scanner.d.ts
 declare function scanForIntents(root?: string, options?: ScanOptions): ScanResult;

package/dist/index.mjs

@@ -1,133 +1,18 @@
 import { a as parseFrontmatter, n as findSkillFiles, o as resolveDepDir, r as getDeps } from "./utils-COlDcU72.mjs";
 import "./skill-paths-8k9K9y26.mjs";
-import { t as scanForIntents } from "./scanner-CW59cxE_.mjs";
+import { t as scanForIntents } from "./scanner-Dav1tzQK.mjs";
 import "./workspace-patterns-BN2A_60g.mjs";
 import { t as readIntentArtifacts } from "./artifact-coverage-wLNVX8yC.mjs";
 import { n as checkStaleness } from "./staleness-PdgakrCQ.mjs";
-import { i as parseSkillUse, n as formatSkillUse, r as isSkillUseParseError, t as SkillUseParseError } from "./skill-use-uwGleSOz.mjs";
-import { n as isResolveSkillUseError, r as resolveSkillUse, t as ResolveSkillUseError } from "./resolver-D2CgIYGg.mjs";
-import "./project-context-IDLpJU3S.mjs";
-import { r as runSetupGithubActions, t as runEditPackageJson } from "./setup-DfLsziXU.mjs";
+import { n as collectStaleReviewItems, r as createFailedStaleReviewItem, t as buildStaleReviewBody } from "./workflow-review-CwkPVIQf.mjs";
+import { i as parseSkillUse, n as formatSkillUse, r as isSkillUseParseError, t as SkillUseParseError } from "./skill-use-BzuuvLM7.mjs";
+import { n as isResolveSkillUseError, r as resolveSkillUse, t as ResolveSkillUseError } from "./resolver-Whd12ksO.mjs";
+import "./project-context-alYMNoNa.mjs";
+import { r as runSetupGithubActions, t as runEditPackageJson } from "./setup-Dp-W8y0Y.mjs";
 import { readFileSync, writeFileSync } from "node:fs";
 import { join } from "node:path";
 import { execFileSync, execSync } from "node:child_process";
 
-//#region src/workflow-review.ts
-function collectStaleReviewItems(reports) {
-	const items = [];
-	for (const report of reports) {
-		for (const skill of report.skills ?? []) {
-			if (!skill?.needsReview) continue;
-			items.push({
-				type: "stale-skill",
-				library: report.library,
-				subject: skill.name,
-				reasons: skill.reasons ?? []
-			});
-		}
-		for (const signal of report.signals ?? []) {
-			if (signal?.needsReview === false) continue;
-			items.push({
-				type: signal?.type ?? "review-signal",
-				library: signal?.library ?? report.library,
-				subject: signal?.packageName ?? signal?.packageRoot ?? signal?.skill ?? signal?.artifactPath ?? signal?.subject ?? report.library,
-				reasons: signal?.reasons ?? [],
-				artifactPath: signal?.artifactPath,
-				packageName: signal?.packageName,
-				packageRoot: signal?.packageRoot,
-				skill: signal?.skill
-			});
-		}
-	}
-	return items;
-}
-function createFailedStaleReviewItem(library) {
-	return {
-		type: "stale-check-failed",
-		library,
-		subject: "intent stale --json",
-		reasons: ["The stale check command failed. Review the workflow logs before updating skills."]
-	};
-}
-function buildStaleReviewBody(items) {
-	const grouped = /* @__PURE__ */ new Map();
-	for (const item of items) grouped.set(item.type, (grouped.get(item.type) ?? 0) + 1);
-	const signalRows = [...grouped.entries()].sort(([a], [b]) => a.localeCompare(b)).map(([type, count]) => `| \`${type}\` | ${count} |`);
-	const itemRows = items.map((item) => {
-		const subject = item.subject ? `\`${item.subject}\`` : "-";
-		const reasons = item.reasons?.length ? item.reasons.join("; ") : "-";
-		return `| \`${item.type}\` | ${subject} | \`${item.library}\` | ${reasons} |`;
-	});
-	const prompt = [
-		"You are helping maintain Intent skills for this repository.",
-		"",
-		"Goal:",
-		"Resolve the Intent skill review signals below while preserving the existing scope, taxonomy, and maintainer-reviewed artifacts.",
-		"",
-		"Review signals:",
-		JSON.stringify(items, null, 2),
-		"",
-		"Required workflow:",
-		"1. Read the existing `_artifacts/*domain_map.yaml`, `_artifacts/*skill_tree.yaml`, and generated `skills/**/SKILL.md` files.",
-		"2. Read each flagged package package.json, public exports, README/docs if present, and source entry points.",
-		"3. Compare flagged packages against the existing domains, skills, tasks, packages, covers, sources, tensions, and cross-references in the artifacts.",
-		"4. For each signal, decide whether it means existing skill coverage, a missing generated skill, a new skill candidate, out-of-scope coverage, or deferred work.",
-		"",
-		"Maintainer questions:",
-		"Before editing skills or artifacts, ask the maintainer:",
-		"1. For each flagged package, is this package user-facing enough to need agent guidance?",
-		"2. If yes, should it extend an existing skill or become a new skill?",
-		"3. If it extends an existing skill, which current skill should own it?",
-		"4. If it is out of scope, what short reason should be recorded in artifact coverage ignores?",
-		"5. Are any of these packages experimental or unstable enough to exclude for now?",
-		"",
-		"Decision rules:",
-		"- Do not auto-generate skills.",
-		"- Do not create broad new skill areas without maintainer confirmation.",
-		"- Prefer adding package coverage to an existing skill when the package is an implementation variant of an existing domain.",
-		"- Create a new skill only when the package introduces a distinct developer task or failure mode.",
-		"- Preserve current naming, path, and package layout conventions.",
-		"- Keep generated skills under the package-local `skills/` directory.",
-		"- Keep repo-root `_artifacts` as the reviewed plan.",
-		"",
-		"If maintainer confirms updates:",
-		"1. Update the relevant `_artifacts/*domain_map.yaml` or `_artifacts/*skill_tree.yaml`.",
-		"2. Update or create `SKILL.md` files only for confirmed coverage changes.",
-		"3. Keep `sources` aligned between artifact skill entries and SKILL frontmatter.",
-		"4. Bump `library_version` only for skills whose covered source package version changed.",
-		"5. Run `npx @tanstack/intent@latest validate` on touched skill directories.",
-		"6. Summarize every package as one of: existing-skill coverage, new skill, ignored, or deferred."
-	].join("\n");
-	return [
-		"## Intent Skill Review Needed",
-		"",
-		"Intent found skills, artifact coverage, or workspace package coverage that need maintainer review.",
-		"",
-		"### Summary",
-		"",
-		"| Signal | Count |",
-		"| --- | ---: |",
-		...signalRows,
-		"",
-		"### Review Items",
-		"",
-		"| Signal | Subject | Library | Reason |",
-		"| --- | --- | --- | --- |",
-		...itemRows,
-		"",
-		"### Agent Prompt",
-		"",
-		"Paste this into your coding agent:",
-		"",
-		"```text",
-		prompt,
-		"```",
-		"",
-		"This PR is a review reminder only. It does not update skills automatically."
-	].join("\n");
-}
-
-//#endregion
 //#region src/feedback.ts
 const META_FEEDBACK_REPO = "TanStack/intent";
 const SECRET_PATTERNS = [

package/dist/{install-PUnIfBNC.mjs → install-QjryhQtg.mjs}

@@ -1,5 +1,5 @@
-import { i as parseSkillUse, n as formatSkillUse } from "./skill-use-uwGleSOz.mjs";
-import { t as resolveProjectContext } from "./project-context-IDLpJU3S.mjs";
+import { i as parseSkillUse, n as formatSkillUse } from "./skill-use-BzuuvLM7.mjs";
+import { t as resolveProjectContext } from "./project-context-alYMNoNa.mjs";
 import { existsSync, mkdirSync, readFileSync, writeFileSync } from "node:fs";
 import { dirname, join, relative, resolve } from "node:path";
 import { parse } from "yaml";
@@ -28,7 +28,7 @@ function printWarnings(warnings) {
 
 //#endregion
 //#region src/cli-support.ts
-const INTENT_CHECK_SKILLS_WORKFLOW_VERSION = 2;
+const INTENT_CHECK_SKILLS_WORKFLOW_VERSION = 3;
 function getMetaDir() {
 	return join(dirname(fileURLToPath(import.meta.url)), "..", "meta");
 }
@@ -46,7 +46,7 @@ function getCheckSkillsWorkflowAdvisories(root) {
 	return [`Intent workflow update available: run \`npx @tanstack/intent@latest setup\` to refresh ${relative(process.cwd(), workflowPath) || workflowPath}.`];
 }
 async function scanIntentsOrFail(options) {
-	const { scanForIntents } = await import("./scanner-DlkcbVye.mjs");
+	const { scanForIntents } = await import("./scanner-BAZxWeUk.mjs");
 	try {
 		return scanForIntents(void 0, options);
 	} catch (err) {

package/dist/intent-library.mjs

@@ -2,10 +2,10 @@
 import "./utils-COlDcU72.mjs";
 import "./skill-paths-8k9K9y26.mjs";
 import "./workspace-patterns-BN2A_60g.mjs";
-import "./project-context-IDLpJU3S.mjs";
-import { t as INSTALL_PROMPT } from "./install-PUnIfBNC.mjs";
-import { n as printSkillTree, r as printTable, t as computeSkillNameWidth } from "./display-BTZWCjzT.mjs";
-import { t as scanLibrary } from "./library-scanner-Cl-XPEMf.mjs";
+import "./project-context-alYMNoNa.mjs";
+import { t as INSTALL_PROMPT } from "./install-QjryhQtg.mjs";
+import { n as printSkillTree, r as printTable, t as computeSkillNameWidth } from "./display-CAof6doy.mjs";
+import { t as scanLibrary } from "./library-scanner-fexXlPXb.mjs";
 
 //#region src/intent-library.ts
 function cmdList() {

package/dist/library-scanner.d.mts

@@ -1,4 +1,4 @@
-import { h as SkillEntry } from "./types-_y9b00bI.mjs";
+import { h as SkillEntry } from "./types-DT7Y6TFz.mjs";
 
 //#region src/library-scanner.d.ts
 interface LibraryPackage {

package/dist/library-scanner.mjs

@@ -1,5 +1,5 @@
 import "./utils-COlDcU72.mjs";
 import "./skill-paths-8k9K9y26.mjs";
-import { t as scanLibrary } from "./library-scanner-Cl-XPEMf.mjs";
+import { t as scanLibrary } from "./library-scanner-fexXlPXb.mjs";
 
 export { scanLibrary };

package/dist/{resolver-D2CgIYGg.mjs → resolver-Whd12ksO.mjs}

@@ -1,4 +1,4 @@
-import { i as parseSkillUse } from "./skill-use-uwGleSOz.mjs";
+import { i as parseSkillUse } from "./skill-use-BzuuvLM7.mjs";
 
 //#region src/resolver.ts
 var ResolveSkillUseError = class extends Error {

package/dist/{scanner-DlkcbVye.mjs → scanner-BAZxWeUk.mjs}

@@ -1,6 +1,6 @@
 import "./utils-COlDcU72.mjs";
 import "./skill-paths-8k9K9y26.mjs";
-import { t as scanForIntents } from "./scanner-CW59cxE_.mjs";
+import { t as scanForIntents } from "./scanner-Dav1tzQK.mjs";
 import "./workspace-patterns-BN2A_60g.mjs";
 
 export { scanForIntents };

package/dist/{scanner-CW59cxE_.mjs → scanner-Dav1tzQK.mjs}

@@ -1,8 +1,9 @@
 import { a as parseFrontmatter, i as listNodeModulesPackageDirs, o as resolveDepDir, r as getDeps, s as toPosixPath, t as detectGlobalNodeModules } from "./utils-COlDcU72.mjs";
 import { r as rewriteSkillLoadPaths } from "./skill-paths-8k9K9y26.mjs";
 import { a as resolveWorkspacePackages, i as readWorkspacePatterns, r as findWorkspaceRoot } from "./workspace-patterns-BN2A_60g.mjs";
+import { createRequire } from "node:module";
 import { existsSync, readFileSync, readdirSync } from "node:fs";
-import { join, relative, sep } from "node:path";
+import { dirname, join, relative, resolve, sep } from "node:path";
 
 //#region src/discovery/register.ts
 function isLocalToProject(dirPath, projectRoot) {
@@ -132,13 +133,31 @@ function createDependencyWalker(opts) {
 
 //#endregion
 //#region src/scanner.ts
-function detectPackageManager(root) {
-	if (existsSync(join(root, ".pnp.cjs")) || existsSync(join(root, ".pnp.js"))) throw new Error("Yarn PnP is not yet supported. Add `nodeLinker: node-modules` to your .yarnrc.yml to use intent.");
+const requireFromHere = createRequire(import.meta.url);
+function findPnpFile(start) {
+	let dir = resolve(start);
+	while (true) {
+		for (const fileName of [".pnp.cjs", ".pnp.js"]) {
+			const pnpPath = join(dir, fileName);
+			if (existsSync(pnpPath)) return pnpPath;
+		}
+		const next = dirname(dir);
+		if (next === dir) return null;
+		dir = next;
+	}
+}
+function isYarnPnpProject(root) {
+	return findPnpFile(root) !== null;
+}
+function assertLocalNodeModulesSupported(root) {
 	if (existsSync(join(root, "deno.json")) && !existsSync(join(root, "node_modules"))) throw new Error("Deno without node_modules is not yet supported. Add `\"nodeModulesDir\": \"auto\"` to your deno.json to use intent.");
+}
+function detectPackageManager(root) {
 	const dirsToCheck = [root];
 	const wsRoot = findWorkspaceRoot(root);
 	if (wsRoot && wsRoot !== root) dirsToCheck.push(wsRoot);
 	for (const dir of dirsToCheck) {
+		if (isYarnPnpProject(dir)) return "yarn";
 		if (existsSync(join(dir, "pnpm-lock.yaml"))) return "pnpm";
 		if (existsSync(join(dir, "bun.lockb")) || existsSync(join(dir, "bun.lock"))) return "bun";
 		if (existsSync(join(dir, "yarn.lock"))) return "yarn";
@@ -146,6 +165,35 @@ function detectPackageManager(root) {
 	}
 	return "unknown";
 }
+function loadPnpApi(root) {
+	const pnpPath = findPnpFile(root);
+	if (!pnpPath) return null;
+	try {
+		const foundApi = requireFromHere("node:module").findPnpApi?.(root);
+		if (foundApi) return foundApi;
+		const pnpModule = requireFromHere(pnpPath);
+		if (typeof pnpModule.setup === "function") pnpModule.setup();
+		if (typeof pnpModule.getDependencyTreeRoots === "function" && typeof pnpModule.getPackageInformation === "function") return pnpModule;
+		return createRequire(join(dirname(pnpPath), "package.json"))("pnpapi");
+	} catch (err) {
+		const msg = err instanceof Error ? err.message : String(err);
+		throw new Error(`Yarn PnP project detected, but Intent could not load Yarn's PnP API from ${pnpPath}: ${msg}`);
+	}
+}
+function getPnpLocatorKey(locator) {
+	return `${locator.name ?? "<top>"}@${locator.reference ?? "<top>"}`;
+}
+function getPnpDependencyLocator(dependencyName, target) {
+	if (target === null) return null;
+	if (Array.isArray(target)) return {
+		name: target[0],
+		reference: target[1]
+	};
+	return {
+		name: dependencyName,
+		reference: target
+	};
+}
 function validateIntentField(_pkgName, intent) {
 	if (!intent || typeof intent !== "object") return null;
 	const pb = intent;
@@ -304,6 +352,7 @@ function scanForIntents(root, options = {}) {
 	const projectRoot = root ?? process.cwd();
 	const scanScope = getScanScope(options);
 	const packageManager = detectPackageManager(projectRoot);
+	const pnpApi = scanScope === "global" ? null : loadPnpApi(projectRoot);
 	const nodeModulesDir = join(projectRoot, "node_modules");
 	const explicitGlobalNodeModules = process.env.INTENT_GLOBAL_NODE_MODULES?.trim() || null;
 	const packages = [];
@@ -378,7 +427,33 @@ function scanForIntents(root, options = {}) {
 		tryRegister,
 		warnings
 	});
+	function scanPnpPackages() {
+		if (!pnpApi) return;
+		const api = pnpApi;
+		const visited = /* @__PURE__ */ new Set();
+		const workspaceRoot = findWorkspaceRoot(projectRoot);
+		const projectLocator = api.findPackageLocator?.(projectRoot.endsWith(sep) ? projectRoot : `${projectRoot}${sep}`);
+		const roots = workspaceRoot && workspaceRoot !== projectRoot && projectLocator ? [projectLocator] : api.getDependencyTreeRoots?.() ?? (api.topLevel ? [api.topLevel] : []);
+		function visit(locator) {
+			const key = getPnpLocatorKey(locator);
+			if (visited.has(key)) return;
+			visited.add(key);
+			const info = api.getPackageInformation(locator);
+			if (!info) return;
+			tryRegister(info.packageLocation.replace(/[\\/]$/, ""), locator.name ?? "unknown");
+			for (const [dependencyName, target] of info.packageDependencies) {
+				const dependencyLocator = getPnpDependencyLocator(dependencyName, target);
+				if (dependencyLocator) visit(dependencyLocator);
+			}
+		}
+		for (const locator of roots) visit(locator);
+	}
 	function scanLocalPackages() {
+		if (pnpApi && !nodeModules.local.exists) {
+			scanPnpPackages();
+			return;
+		}
+		assertLocalNodeModulesSupported(projectRoot);
 		scanTarget(nodeModules.local);
 		walkWorkspacePackages();
 		walkKnownPackages();

package/dist/{setup-DfLsziXU.mjs → setup-Dp-W8y0Y.mjs}

@@ -1,5 +1,5 @@
 import { i as readWorkspacePatterns, r as findWorkspaceRoot, t as findPackagesWithSkills } from "./workspace-patterns-BN2A_60g.mjs";
-import { t as resolveProjectContext } from "./project-context-IDLpJU3S.mjs";
+import { t as resolveProjectContext } from "./project-context-alYMNoNa.mjs";
 import { existsSync, mkdirSync, readFileSync, readdirSync, writeFileSync } from "node:fs";
 import { basename, join, relative } from "node:path";
 

package/dist/setup.d.mts

@@ -1,2 +1,2 @@
-import { a as runEditPackageJsonAll, c as findWorkspaceRoot, i as runEditPackageJson, l as readWorkspacePatterns, n as MonorepoResult, o as runSetupGithubActions, r as SetupGithubActionsResult, s as findPackagesWithSkills, t as EditPackageJsonResult, u as resolveWorkspacePackages } from "./setup-D2CGdTsx.mjs";
+import { a as runEditPackageJsonAll, c as findWorkspaceRoot, i as runEditPackageJson, l as readWorkspacePatterns, n as MonorepoResult, o as runSetupGithubActions, r as SetupGithubActionsResult, s as findPackagesWithSkills, t as EditPackageJsonResult, u as resolveWorkspacePackages } from "./setup-t1i2o2-h.mjs";
 export { EditPackageJsonResult, MonorepoResult, SetupGithubActionsResult, findPackagesWithSkills, findWorkspaceRoot, readWorkspacePatterns, resolveWorkspacePackages, runEditPackageJson, runEditPackageJsonAll, runSetupGithubActions };

package/dist/setup.mjs

@@ -1,6 +1,6 @@
 import "./utils-COlDcU72.mjs";
 import { a as resolveWorkspacePackages, i as readWorkspacePatterns, r as findWorkspaceRoot, t as findPackagesWithSkills } from "./workspace-patterns-BN2A_60g.mjs";
-import "./project-context-IDLpJU3S.mjs";
-import { n as runEditPackageJsonAll, r as runSetupGithubActions, t as runEditPackageJson } from "./setup-DfLsziXU.mjs";
+import "./project-context-alYMNoNa.mjs";
+import { n as runEditPackageJsonAll, r as runSetupGithubActions, t as runEditPackageJson } from "./setup-Dp-W8y0Y.mjs";
 
 export { findPackagesWithSkills, findWorkspaceRoot, readWorkspacePatterns, resolveWorkspacePackages, runEditPackageJson, runEditPackageJsonAll, runSetupGithubActions };

package/dist/workflow-review-CwkPVIQf.mjs

@@ -0,0 +1,153 @@
+import { appendFileSync, writeFileSync } from "node:fs";
+
+//#region src/workflow-review.ts
+function collectStaleReviewItems(reports) {
+	const items = [];
+	for (const report of reports) {
+		for (const skill of report.skills ?? []) {
+			if (!skill?.needsReview) continue;
+			items.push({
+				type: "stale-skill",
+				library: report.library,
+				subject: skill.name,
+				reasons: skill.reasons ?? []
+			});
+		}
+		for (const signal of report.signals ?? []) {
+			if (signal?.needsReview === false) continue;
+			items.push({
+				type: signal?.type ?? "review-signal",
+				library: signal?.library ?? report.library,
+				subject: signal?.packageName ?? signal?.packageRoot ?? signal?.skill ?? signal?.artifactPath ?? signal?.subject ?? report.library,
+				reasons: signal?.reasons ?? [],
+				artifactPath: signal?.artifactPath,
+				packageName: signal?.packageName,
+				packageRoot: signal?.packageRoot,
+				skill: signal?.skill
+			});
+		}
+	}
+	return items;
+}
+function createFailedStaleReviewItem(library) {
+	return {
+		type: "stale-check-failed",
+		library,
+		subject: "intent stale --json",
+		reasons: ["The stale check command failed. Review the workflow logs before updating skills."]
+	};
+}
+function createWorkflowAdvisoryReviewItems(library, advisories) {
+	return advisories.map((advisory) => ({
+		type: "workflow-advisory",
+		library,
+		subject: "check-skills.yml",
+		reasons: [advisory]
+	}));
+}
+function buildStaleReviewBody(items) {
+	const grouped = /* @__PURE__ */ new Map();
+	for (const item of items) grouped.set(item.type, (grouped.get(item.type) ?? 0) + 1);
+	const signalRows = [...grouped.entries()].sort(([a], [b]) => a.localeCompare(b)).map(([type, count]) => `| \`${type}\` | ${count} |`);
+	const itemRows = items.map((item) => {
+		const subject = item.subject ? `\`${item.subject}\`` : "-";
+		const reasons = item.reasons?.length ? item.reasons.join("; ") : "-";
+		return `| \`${item.type}\` | ${subject} | \`${item.library}\` | ${reasons} |`;
+	});
+	const reasonBullets = items.map((item) => {
+		const subject = item.subject ? `\`${item.subject}\`` : "`unknown`";
+		const reasons = item.reasons?.length ? item.reasons.join("; ") : "Intent did not emit a detailed reason for this signal.";
+		return `- \`${item.type}\` for ${subject}: ${reasons}`;
+	});
+	const prompt = [
+		"You are helping maintain Intent skills for this repository.",
+		"",
+		"Goal:",
+		"Resolve the Intent skill review signals below while preserving the existing scope, taxonomy, and maintainer-reviewed artifacts.",
+		"",
+		"Review signals:",
+		JSON.stringify(items, null, 2),
+		"",
+		"Required workflow:",
+		"1. Read the existing `_artifacts/*domain_map.yaml`, `_artifacts/*skill_tree.yaml`, and generated `skills/**/SKILL.md` files.",
+		"2. Read each flagged package package.json, public exports, README/docs if present, and source entry points.",
+		"3. Compare flagged packages against the existing domains, skills, tasks, packages, covers, sources, tensions, and cross-references in the artifacts.",
+		"4. For each signal, decide whether it means existing skill coverage, a missing generated skill, a new skill candidate, out-of-scope coverage, or deferred work.",
+		"",
+		"Maintainer questions:",
+		"Before editing skills or artifacts, ask the maintainer:",
+		"1. For each flagged package, is this package user-facing enough to need agent guidance?",
+		"2. If yes, should it extend an existing skill or become a new skill?",
+		"3. If it extends an existing skill, which current skill should own it?",
+		"4. If it is out of scope, what short reason should be recorded in artifact coverage ignores?",
+		"5. Are any of these packages experimental or unstable enough to exclude for now?",
+		"",
+		"Decision rules:",
+		"- Do not auto-generate skills.",
+		"- Do not create broad new skill areas without maintainer confirmation.",
+		"- Prefer adding package coverage to an existing skill when the package is an implementation variant of an existing domain.",
+		"- Create a new skill only when the package introduces a distinct developer task or failure mode.",
+		"- Preserve current naming, path, and package layout conventions.",
+		"- Keep generated skills under the package-local `skills/` directory.",
+		"- Keep repo-root `_artifacts` as the reviewed plan.",
+		"",
+		"If maintainer confirms updates:",
+		"1. Update the relevant `_artifacts/*domain_map.yaml` or `_artifacts/*skill_tree.yaml`.",
+		"2. Update or create `SKILL.md` files only for confirmed coverage changes.",
+		"3. Keep `sources` aligned between artifact skill entries and SKILL frontmatter.",
+		"4. Bump `library_version` only for skills whose covered source package version changed.",
+		"5. Run `npx @tanstack/intent@latest validate` on touched skill directories.",
+		"6. Summarize every package as one of: existing-skill coverage, new skill, ignored, or deferred."
+	].join("\n");
+	return [
+		"## Intent Skill Review Needed",
+		"",
+		"Intent found skills, artifact coverage, or workspace package coverage that need maintainer review.",
+		"",
+		"### Summary",
+		"",
+		"| Signal | Count |",
+		"| --- | ---: |",
+		...signalRows,
+		"",
+		"### Why This PR Opened",
+		"",
+		...reasonBullets,
+		"",
+		"### Review Items",
+		"",
+		"| Signal | Subject | Library | Reason |",
+		"| --- | --- | --- | --- |",
+		...itemRows,
+		"",
+		"### Agent Prompt",
+		"",
+		"Paste this into your coding agent:",
+		"",
+		"```text",
+		prompt,
+		"```",
+		"",
+		"This PR is a review reminder only. It does not update skills automatically."
+	].join("\n");
+}
+function writeStaleReviewWorkflowFiles(items, options = {}) {
+	const outputPath = options.outputPath ?? process.env.GITHUB_OUTPUT;
+	const summaryPath = options.summaryPath ?? process.env.GITHUB_STEP_SUMMARY;
+	const reviewItemsPath = options.reviewItemsPath ?? "review-items.json";
+	const prBodyPath = options.prBodyPath ?? "pr-body.md";
+	const hasReview = items.length > 0;
+	writeFileSync(reviewItemsPath, JSON.stringify(items, null, 2) + "\n");
+	if (outputPath) appendFileSync(outputPath, `has_review=${hasReview ? "true" : "false"}\n`);
+	const summary = hasReview ? buildStaleReviewBody(items) + "\n" : [
+		"### Intent skill review",
+		"",
+		"No stale skills or coverage gaps found.",
+		""
+	].join("\n");
+	if (hasReview) writeFileSync(prBodyPath, summary);
+	if (summaryPath) appendFileSync(summaryPath, summary);
+}
+
+//#endregion
+export { writeStaleReviewWorkflowFiles as a, createWorkflowAdvisoryReviewItems as i, collectStaleReviewItems as n, createFailedStaleReviewItem as r, buildStaleReviewBody as t };

package/dist/workflow-review-Dz_ofcYQ.mjs

@@ -0,0 +1,3 @@
+import { a as writeStaleReviewWorkflowFiles, i as createWorkflowAdvisoryReviewItems, n as collectStaleReviewItems, r as createFailedStaleReviewItem, t as buildStaleReviewBody } from "./workflow-review-CwkPVIQf.mjs";
+
+export { collectStaleReviewItems, createFailedStaleReviewItem, createWorkflowAdvisoryReviewItems, writeStaleReviewWorkflowFiles };

package/meta/templates/workflows/check-skills.yml

@@ -1,11 +1,13 @@
 # check-skills.yml — Drop this into your library repo's .github/workflows/
 #
-# Checks intent skills after a release and opens or updates one review PR when
-# existing skills, artifact coverage, or workspace package coverage need review.
+# Validates intent skills on PRs. After a release or manual run, opens or
+# updates one review PR when existing skills, artifact coverage, or workspace
+# package coverage need review.
 #
-# Triggers: new release published, or manual workflow_dispatch.
+# Triggers: pull requests touching skills/artifacts, new release published, or
+# manual workflow_dispatch.
 #
-# intent-workflow-version: 2
+# intent-workflow-version: 3
 #
 # Template variables (replaced by `intent setup`):
 #   {{PACKAGE_LABEL}} — e.g. @tanstack/query or my-workspace workspace
@@ -13,6 +15,12 @@
 name: Check Skills
 
 on:
+  pull_request:
+    paths:
+      - 'skills/**'
+      - '**/skills/**'
+      - '_artifacts/**'
+      - '**/_artifacts/**'
   release:
     types: [published]
   workflow_dispatch: {}
@@ -22,8 +30,28 @@ permissions:
   pull-requests: write
 
 jobs:
-  check:
+  validate:
+    name: Validate intent skills
+    if: github.event_name == 'pull_request'
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v4
+
+      - name: Setup Node
+        uses: actions/setup-node@v4
+        with:
+          node-version: 20
+
+      - name: Install intent
+        run: npm install -g @tanstack/intent
+
+      - name: Validate skills
+        run: intent validate --github-summary
+
+  review:
     name: Check intent skill coverage
+    if: github.event_name != 'pull_request'
     runs-on: ubuntu-latest
     steps:
       - name: Checkout
@@ -42,181 +70,7 @@ jobs:
       - name: Check skills
         id: stale
         run: |
-          set +e
-          intent stale --json > stale.json
-          STATUS=$?
-          set -e
-
-          cat stale.json
-
-          if [ "$STATUS" -ne 0 ]; then
-            echo "has_review=true" >> "$GITHUB_OUTPUT"
-            echo "check_failed=true" >> "$GITHUB_OUTPUT"
-            cat > review-items.json <<'JSON'
-          [
-            {
-              "type": "stale-check-failed",
-              "library": "{{PACKAGE_LABEL}}",
-              "subject": "intent stale --json",
-              "reasons": ["The stale check command failed. Review the workflow logs before updating skills."]
-            }
-          ]
-          JSON
-          else
-            node <<'NODE'
-          const fs = require('fs')
-          const reports = JSON.parse(fs.readFileSync('stale.json', 'utf8'))
-          const items = []
-
-          for (const report of reports) {
-            for (const skill of report.skills ?? []) {
-              if (!skill?.needsReview) continue
-              items.push({
-                type: 'stale-skill',
-                library: report.library,
-                subject: skill.name,
-                reasons: skill.reasons ?? [],
-              })
-            }
-
-            for (const signal of report.signals ?? []) {
-              if (signal?.needsReview === false) continue
-              items.push({
-                type: signal?.type ?? 'review-signal',
-                library: signal?.library ?? report.library,
-                subject:
-                  signal?.packageName ??
-                  signal?.packageRoot ??
-                  signal?.skill ??
-                  signal?.artifactPath ??
-                  signal?.subject ??
-                  report.library,
-                reasons: signal?.reasons ?? [],
-                artifactPath: signal?.artifactPath,
-                packageName: signal?.packageName,
-                packageRoot: signal?.packageRoot,
-                skill: signal?.skill,
-              })
-            }
-          }
-
-          fs.writeFileSync('review-items.json', JSON.stringify(items, null, 2) + '\n')
-          fs.appendFileSync(
-            process.env.GITHUB_OUTPUT,
-            `has_review=${items.length > 0 ? 'true' : 'false'}\n`,
-          )
-          NODE
-          fi
-
-          {
-            echo "review_items<<EOF"
-            cat review-items.json
-            echo "EOF"
-          } >> "$GITHUB_OUTPUT"
-
-      - name: Write clean summary
-        if: steps.stale.outputs.has_review == 'false'
-        run: |
-          {
-            echo "### Intent skill review"
-            echo ""
-            echo "No stale skills or coverage gaps found."
-          } >> "$GITHUB_STEP_SUMMARY"
-
-      - name: Build review PR body
-        if: steps.stale.outputs.has_review == 'true'
-        run: |
-          node <<'NODE'
-          const fs = require('fs')
-          const items = JSON.parse(fs.readFileSync('review-items.json', 'utf8'))
-          const grouped = new Map()
-
-          for (const item of items) {
-            grouped.set(item.type, (grouped.get(item.type) ?? 0) + 1)
-          }
-
-          const signalRows = [...grouped.entries()]
-            .sort(([a], [b]) => a.localeCompare(b))
-            .map(([type, count]) => `| \`${type}\` | ${count} |`)
-
-          const itemRows = items.map((item) => {
-            const subject = item.subject ? `\`${item.subject}\`` : '-'
-            const reasons = item.reasons?.length ? item.reasons.join('; ') : '-'
-            return `| \`${item.type}\` | ${subject} | \`${item.library}\` | ${reasons} |`
-          })
-
-          const prompt = [
-            'You are helping maintain Intent skills for this repository.',
-            '',
-            'Goal:',
-            'Resolve the Intent skill review signals below while preserving the existing scope, taxonomy, and maintainer-reviewed artifacts.',
-            '',
-            'Review signals:',
-            JSON.stringify(items, null, 2),
-            '',
-            'Required workflow:',
-            '1. Read the existing `_artifacts/*domain_map.yaml`, `_artifacts/*skill_tree.yaml`, and generated `skills/**/SKILL.md` files.',
-            '2. Read each flagged package package.json, public exports, README/docs if present, and source entry points.',
-            '3. Compare flagged packages against the existing domains, skills, tasks, packages, covers, sources, tensions, and cross-references in the artifacts.',
-            '4. For each signal, decide whether it means existing skill coverage, a missing generated skill, a new skill candidate, out-of-scope coverage, or deferred work.',
-            '',
-            'Maintainer questions:',
-            'Before editing skills or artifacts, ask the maintainer:',
-            '1. For each flagged package, is this package user-facing enough to need agent guidance?',
-            '2. If yes, should it extend an existing skill or become a new skill?',
-            '3. If it extends an existing skill, which current skill should own it?',
-            '4. If it is out of scope, what short reason should be recorded in artifact coverage ignores?',
-            '5. Are any of these packages experimental or unstable enough to exclude for now?',
-            '',
-            'Decision rules:',
-            '- Do not auto-generate skills.',
-            '- Do not create broad new skill areas without maintainer confirmation.',
-            '- Prefer adding package coverage to an existing skill when the package is an implementation variant of an existing domain.',
-            '- Create a new skill only when the package introduces a distinct developer task or failure mode.',
-            '- Preserve current naming, path, and package layout conventions.',
-            '- Keep generated skills under the package-local `skills/` directory.',
-            '- Keep repo-root `_artifacts` as the reviewed plan.',
-            '',
-            'If maintainer confirms updates:',
-            '1. Update the relevant `_artifacts/*domain_map.yaml` or `_artifacts/*skill_tree.yaml`.',
-            '2. Update or create `SKILL.md` files only for confirmed coverage changes.',
-            '3. Keep `sources` aligned between artifact skill entries and SKILL frontmatter.',
-            '4. Bump `library_version` only for skills whose covered source package version changed.',
-            '5. Run `npx @tanstack/intent@latest validate` on touched skill directories.',
-            '6. Summarize every package as one of: existing-skill coverage, new skill, ignored, or deferred.',
-          ].join('\n')
-
-          const body = [
-            '## Intent Skill Review Needed',
-            '',
-            'Intent found skills, artifact coverage, or workspace package coverage that need maintainer review.',
-            '',
-            '### Summary',
-            '',
-            '| Signal | Count |',
-            '| --- | ---: |',
-            ...signalRows,
-            '',
-            '### Review Items',
-            '',
-            '| Signal | Subject | Library | Reason |',
-            '| --- | --- | --- | --- |',
-            ...itemRows,
-            '',
-            '### Agent Prompt',
-            '',
-            'Paste this into your coding agent:',
-            '',
-            '```text',
-            prompt,
-            '```',
-            '',
-            'This PR is a review reminder only. It does not update skills automatically.',
-          ].join('\n')
-
-          fs.writeFileSync('pr-body.md', body + '\n')
-          fs.writeFileSync(process.env.GITHUB_STEP_SUMMARY, body + '\n')
-          NODE
+          intent stale --github-review --package-label "{{PACKAGE_LABEL}}"
 
       - name: Open or update review PR
         if: steps.stale.outputs.has_review == 'true'

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tanstack/intent",
-  "version": "0.0.33",
+  "version": "0.0.34",
   "description": "Ship compositional knowledge for AI coding agents alongside your npm packages",
   "license": "MIT",
   "type": "module",

package/meta/templates/workflows/validate-skills.yml

@@ -1,52 +0,0 @@
-# validate-skills.yml — Drop this into your library repo's .github/workflows/
-#
-# Validates skill files on PRs that touch the skills/ directory.
-# Ensures frontmatter is correct, names match paths, and files stay under
-# the 500-line limit.
-
-name: Validate Skills
-
-on:
-  pull_request:
-    paths:
-      - 'skills/**'
-      - '**/skills/**'
-
-jobs:
-  validate:
-    name: Validate skill files
-    runs-on: ubuntu-latest
-    steps:
-      - name: Checkout
-        uses: actions/checkout@v4
-
-      - name: Setup Node
-        uses: actions/setup-node@v4
-        with:
-          node-version: 20
-
-      - name: Install intent CLI
-        run: npm install -g @tanstack/intent
-
-      - name: Find and validate skills
-        run: |
-          # Find all directories containing SKILL.md files
-          SKILLS_DIR=""
-          if [ -d "skills" ]; then
-            SKILLS_DIR="skills"
-          elif [ -d "packages" ]; then
-            # Monorepo — find skills/ under packages
-            for dir in packages/*/skills; do
-              if [ -d "$dir" ]; then
-                echo "Validating $dir..."
-                intent validate "$dir"
-              fi
-            done
-            exit 0
-          fi
-
-          if [ -n "$SKILLS_DIR" ]; then
-            intent validate "$SKILLS_DIR"
-          else
-            echo "No skills/ directory found — skipping validation."
-          fi