rulesync 9.5.0 → 9.6.1

package/README.md

@@ -97,7 +97,7 @@ The tables below show whether each tool supports a given feature (✅ = supporte
 | Takt                   |  ✅   |        | ✅  |    ✅    |    ✅     |   ✅   |       |     ✅      |
 | Vibe Code              |  ✅   |   ✅   | ✅  |          |    ✅     |   ✅   |  ✅   |     ✅      |
 | Qwen Code              |  ✅   |   ✅   | ✅  |    ✅    |    ✅     |   ✅   |  ✅   |     ✅      |
-| Reasonix               |       |        | ✅  |    ✅    |           |        |  ✅   |     ✅      |
+| Reasonix               |  ✅   |        | ✅  |    ✅    |           |   ✅   |  ✅   |     ✅      |
 | Kiro ⚠️                |  ✅   |   ✅   | ✅  |    ✅    |    ✅     |   ✅   |  ✅   |     ✅      |
 | Kiro CLI               |  ✅   |   ✅   | ✅  |    ✅    |    ✅     |   ✅   |  ✅   |     ✅      |
 | Kiro IDE               |  ✅   |   ✅   | ✅  |    ✅    |    ✅     |   ✅   |  ✅   |     ✅      |

package/dist/cli/index.cjs

@@ -1,5 +1,5 @@
 #!/usr/bin/env node
-const require_import = require("../import-DngV1i49.cjs");
+const require_import = require("../import-DW-a1SJq.cjs");
 let commander = require("commander");
 let zod_mini = require("zod/mini");
 let node_path = require("node:path");
@@ -1563,12 +1563,13 @@ const gitignoreCommand = async (logger, options) => {
 	const gitignorePath = (0, node_path.join)(process.cwd(), ".gitignore");
 	const gitattributesPath = (0, node_path.join)(process.cwd(), ".gitattributes");
 	const config = await require_import.ConfigResolver.resolve({});
+	const resolvedEntries = resolveGitignoreEntries({
+		targets: options?.targets,
+		features: options?.features,
+		logger
+	});
 	const { gitignore: gitignoreEntries, gitattributes: gitattributesEntries } = groupEntriesByDestination({
-		entries: resolveGitignoreEntries({
-			targets: options?.targets,
-			features: options?.features,
-			logger
-		}),
+		entries: resolvedEntries,
 		resolveDestination: (target, feature) => {
 			if (feature === void 0 || feature === "general") return config.getGitignoreDestination(target);
 			return config.getGitignoreDestination(target, feature);
@@ -2023,7 +2024,7 @@ function parseApmLock(content) {
 	if (!content.trim()) return null;
 	let loaded;
 	try {
-		loaded = (0, js_yaml.load)(content);
+		loaded = require_import.loadYaml(content);
 	} catch {
 		return null;
 	}
@@ -2093,7 +2094,7 @@ async function apmManifestExists(projectRoot) {
 * or any dependency entry fails normalization.
 */
 function parseApmManifest(content) {
-	const loaded = (0, js_yaml.load)(content);
+	const loaded = require_import.loadYaml(content);
 	if (loaded === void 0 || loaded === null) return { dependencies: [] };
 	const parsed = ApmManifestSchema.safeParse(loaded);
 	if (!parsed.success) throw new Error(`Invalid apm.yml: ${parsed.error.message}`);
@@ -2562,11 +2563,14 @@ function injectSourceMetadata(params) {
 	if (content.startsWith(`${FRONTMATTER_FENCE}\r\n`)) openFenceLen = 5;
 	else if (content.startsWith(`${FRONTMATTER_FENCE}\n`)) openFenceLen = 4;
 	else if (content === FRONTMATTER_FENCE) openFenceLen = 3;
-	else return `${FRONTMATTER_FENCE}\n${(0, js_yaml.dump)(provenance, {
-		noRefs: true,
-		lineWidth: -1,
-		sortKeys: false
-	})}${FRONTMATTER_FENCE}\n${content}`;
+	else {
+		const yaml = (0, js_yaml.dump)(provenance, {
+			noRefs: true,
+			lineWidth: -1,
+			sortKeys: false
+		});
+		return `${FRONTMATTER_FENCE}\n${yaml}${FRONTMATTER_FENCE}\n${content}`;
+	}
 	const afterOpen = content.substring(openFenceLen);
 	let fmBody;
 	let rest;
@@ -2582,24 +2586,28 @@ function injectSourceMetadata(params) {
 	}
 	let loaded;
 	try {
-		loaded = (0, js_yaml.load)(fmBody);
+		loaded = require_import.loadYaml(fmBody);
 	} catch {
 		throw new Error("invalid frontmatter");
 	}
-	if (loaded === null || loaded === void 0) return `${FRONTMATTER_FENCE}\n${(0, js_yaml.dump)(provenance, {
-		noRefs: true,
-		lineWidth: -1,
-		sortKeys: false
-	})}${FRONTMATTER_FENCE}\n${rest}`;
+	if (loaded === null || loaded === void 0) {
+		const yaml = (0, js_yaml.dump)(provenance, {
+			noRefs: true,
+			lineWidth: -1,
+			sortKeys: false
+		});
+		return `${FRONTMATTER_FENCE}\n${yaml}${FRONTMATTER_FENCE}\n${rest}`;
+	}
 	if (typeof loaded !== "object" || Array.isArray(loaded)) throw new Error("invalid frontmatter");
-	return `${FRONTMATTER_FENCE}\n${(0, js_yaml.dump)({
+	const yaml = (0, js_yaml.dump)({
 		...loaded,
 		...provenance
 	}, {
 		noRefs: true,
 		lineWidth: -1,
 		sortKeys: false
-	})}${FRONTMATTER_FENCE}\n${rest}`;
+	});
+	return `${FRONTMATTER_FENCE}\n${yaml}${FRONTMATTER_FENCE}\n${rest}`;
 }
 //#endregion
 //#region src/lib/gh/gh-lock.ts
@@ -2668,7 +2676,7 @@ function parseGhLock(content) {
 	if (!content.trim()) return null;
 	let loaded;
 	try {
-		loaded = (0, js_yaml.load)(content);
+		loaded = require_import.loadYaml(content);
 	} catch {
 		return null;
 	}
@@ -3803,6 +3811,21 @@ function getFirstPathSeparatorIndex(path) {
 	if (backslashIndex === -1) return slashIndex;
 	return Math.min(slashIndex, backslashIndex);
 }
+/**
+* Decide whether a repository's root-level files should be installed as the
+* single requested skill (the "root fallback").
+*
+* A root fallback fires only when a single, non-wildcard skill was requested,
+* that skill's own directory is absent, and the repository root actually carries
+* a `SKILL.md`. Both the git transport (`groupRemoteFilesBySkillRoot`) and the
+* GitHub transport (`discoverGithubSkillDirs`) gate on these same conditions, so
+* the decision lives here to keep the two paths from drifting.
+*/
+function shouldUseRootFallback(params) {
+	const { skillFilter, isWildcard, hasRootSkillFile, hasRequestedSkillDir } = params;
+	const [singleSkillName] = skillFilter;
+	return !isWildcard && skillFilter.length === 1 && singleSkillName !== void 0 && hasRootSkillFile && !hasRequestedSkillDir;
+}
 function groupRemoteFilesBySkillRoot(params) {
 	const { remoteFiles, skillFilter, isWildcard } = params;
 	const grouped = /* @__PURE__ */ new Map();
@@ -3825,7 +3848,12 @@ function groupRemoteFilesBySkillRoot(params) {
 	}
 	const [singleSkillName] = skillFilter;
 	const hasRootSkillFile = rootLevelFiles.some((file) => file.relativePath === require_import.SKILL_FILE_NAME);
-	if (!isWildcard && skillFilter.length === 1 && singleSkillName !== void 0 && hasRootSkillFile && !grouped.has(singleSkillName)) grouped.set(singleSkillName, rootLevelFiles);
+	if (singleSkillName !== void 0 && shouldUseRootFallback({
+		skillFilter,
+		isWildcard,
+		hasRootSkillFile,
+		hasRequestedSkillDir: grouped.has(singleSkillName)
+	})) grouped.set(singleSkillName, rootLevelFiles);
 	return grouped;
 }
 /**
@@ -3962,7 +3990,12 @@ async function discoverGithubSkillDirs(params) {
 		}));
 		const [singleSkillName] = skillFilter;
 		const hasRequestedSkillDir = singleSkillName !== void 0 && remoteSkillDirs.some((d) => d.name === singleSkillName);
-		if (!isWildcard && skillFilter.length === 1 && !hasRequestedSkillDir) {
+		if (shouldUseRootFallback({
+			skillFilter,
+			isWildcard,
+			hasRootSkillFile: entries.some((entry) => entry.type === "file" && entry.name === "SKILL.md"),
+			hasRequestedSkillDir
+		})) {
 			const fallback = await fetchRootLevelFallbackSkill({
 				entries,
 				parsed,
@@ -6428,7 +6461,7 @@ function wrapCommand$1({ name, errorCode, handler, getVersion, loggerFactory = c
 }
 //#endregion
 //#region src/cli/index.ts
-const getVersion = () => "9.5.0";
+const getVersion = () => "9.6.1";
 function wrapCommand(name, errorCode, handler) {
 	return wrapCommand$1({
 		name,

package/dist/cli/index.js

@@ -1,12 +1,12 @@
 #!/usr/bin/env node
-import { $ as toPosixPath, A as RulesyncCommand, B as createTempDirectory, C as RulesyncHooks, Ct as RULESYNC_SKILLS_RELATIVE_DIR_PATH, D as CLAUDECODE_MEMORIES_DIR_NAME, Dt as ALL_FEATURES, E as CLAUDECODE_LOCAL_RULE_FILE_NAME, Et as formatError, F as ConsoleLogger, G as getFileSize, H as ensureDir, I as JsonLogger, J as listDirectoryFiles, K as getHomeDirectory, L as CLIError, M as stringifyFrontmatter, N as ConfigResolver, O as CLAUDECODE_SETTINGS_LOCAL_FILE_NAME, Ot as ALL_FEATURES_WITH_WILDCARD, P as findControlCharacter, Q as removeTempDirectory, R as ErrorCodes, S as HooksProcessor, St as RULESYNC_RULES_RELATIVE_DIR_PATH, T as CLAUDECODE_DIR, Tt as RULESYNC_SUBAGENTS_RELATIVE_DIR_PATH, U as fileExists, V as directoryExists, W as findFilesByGlobs, X as removeDirectory, Y as readFileContent, Z as removeFile, _ as RulesyncPermissions, _t as RULESYNC_MCP_SCHEMA_URL, a as convertFromTool, at as RULESYNC_AIIGNORE_FILE_NAME, b as IgnoreProcessor, bt as RULESYNC_PERMISSIONS_RELATIVE_FILE_PATH, c as RulesyncRuleFrontmatterSchema, ct as RULESYNC_CONFIG_RELATIVE_FILE_PATH, d as RulesyncSubagentFrontmatterSchema, dt as RULESYNC_HOOKS_FILE_NAME, et as writeFileContent, f as SkillsProcessor, ft as RULESYNC_HOOKS_RELATIVE_FILE_PATH, g as SKILL_FILE_NAME$1, gt as RULESYNC_MCP_RELATIVE_FILE_PATH, h as RulesyncSkillFrontmatterSchema, ht as RULESYNC_MCP_FILE_NAME, i as getProcessorRegistryEntry, it as MAX_FILE_SIZE, j as RulesyncCommandFrontmatterSchema, k as CLAUDECODE_SKILLS_DIR_PATH, l as SubagentsProcessor, lt as RULESYNC_CONFIG_SCHEMA_URL, m as RulesyncSkill, mt as RULESYNC_LOCAL_CONFIG_RELATIVE_FILE_PATH, n as checkRulesyncDirExists, nt as ALL_TOOL_TARGETS_WITH_WILDCARD, o as RulesProcessor, ot as RULESYNC_AIIGNORE_RELATIVE_FILE_PATH, p as getLocalSkillDirNames, pt as RULESYNC_IGNORE_RELATIVE_FILE_PATH, q as isSymlink, r as generate, rt as ToolTargetSchema, s as RulesyncRule, st as RULESYNC_COMMANDS_RELATIVE_DIR_PATH, t as importFromTool, tt as ALL_TOOL_TARGETS, u as RulesyncSubagent, ut as RULESYNC_CURATED_SKILLS_RELATIVE_DIR_PATH, v as McpProcessor, vt as RULESYNC_OVERVIEW_FILE_NAME, w as CommandsProcessor, wt as RULESYNC_SOURCES_LOCK_RELATIVE_FILE_PATH, x as RulesyncIgnore, xt as RULESYNC_RELATIVE_DIR_PATH, y as RulesyncMcp, yt as RULESYNC_PERMISSIONS_FILE_NAME, z as checkPathTraversal } from "../import-BdJG1fyb.js";
+import { $ as removeTempDirectory, A as RulesyncCommand, B as checkPathTraversal, C as RulesyncHooks, Ct as RULESYNC_RULES_RELATIVE_DIR_PATH, D as CLAUDECODE_MEMORIES_DIR_NAME, Dt as formatError, E as CLAUDECODE_LOCAL_RULE_FILE_NAME, Et as RULESYNC_SUBAGENTS_RELATIVE_DIR_PATH, F as findControlCharacter, G as findFilesByGlobs, H as directoryExists, I as ConsoleLogger, J as isSymlink, K as getFileSize, L as JsonLogger, M as stringifyFrontmatter, N as loadYaml, O as CLAUDECODE_SETTINGS_LOCAL_FILE_NAME, Ot as ALL_FEATURES, P as ConfigResolver, Q as removeFile, R as CLIError, S as HooksProcessor, St as RULESYNC_RELATIVE_DIR_PATH, T as CLAUDECODE_DIR, Tt as RULESYNC_SOURCES_LOCK_RELATIVE_FILE_PATH, U as ensureDir, V as createTempDirectory, W as fileExists, X as readFileContent, Y as listDirectoryFiles, Z as removeDirectory, _ as RulesyncPermissions, _t as RULESYNC_MCP_RELATIVE_FILE_PATH, a as convertFromTool, at as MAX_FILE_SIZE, b as IgnoreProcessor, bt as RULESYNC_PERMISSIONS_FILE_NAME, c as RulesyncRuleFrontmatterSchema, ct as RULESYNC_COMMANDS_RELATIVE_DIR_PATH, d as RulesyncSubagentFrontmatterSchema, dt as RULESYNC_CURATED_SKILLS_RELATIVE_DIR_PATH, et as toPosixPath, f as SkillsProcessor, ft as RULESYNC_HOOKS_FILE_NAME, g as SKILL_FILE_NAME$1, gt as RULESYNC_MCP_FILE_NAME, h as RulesyncSkillFrontmatterSchema, ht as RULESYNC_LOCAL_CONFIG_RELATIVE_FILE_PATH, i as getProcessorRegistryEntry, it as ToolTargetSchema, j as RulesyncCommandFrontmatterSchema, k as CLAUDECODE_SKILLS_DIR_PATH, kt as ALL_FEATURES_WITH_WILDCARD, l as SubagentsProcessor, lt as RULESYNC_CONFIG_RELATIVE_FILE_PATH, m as RulesyncSkill, mt as RULESYNC_IGNORE_RELATIVE_FILE_PATH, n as checkRulesyncDirExists, nt as ALL_TOOL_TARGETS, o as RulesProcessor, ot as RULESYNC_AIIGNORE_FILE_NAME, p as getLocalSkillDirNames, pt as RULESYNC_HOOKS_RELATIVE_FILE_PATH, q as getHomeDirectory, r as generate, rt as ALL_TOOL_TARGETS_WITH_WILDCARD, s as RulesyncRule, st as RULESYNC_AIIGNORE_RELATIVE_FILE_PATH, t as importFromTool, tt as writeFileContent, u as RulesyncSubagent, ut as RULESYNC_CONFIG_SCHEMA_URL, v as McpProcessor, vt as RULESYNC_MCP_SCHEMA_URL, w as CommandsProcessor, wt as RULESYNC_SKILLS_RELATIVE_DIR_PATH, x as RulesyncIgnore, xt as RULESYNC_PERMISSIONS_RELATIVE_FILE_PATH, y as RulesyncMcp, yt as RULESYNC_OVERVIEW_FILE_NAME, z as ErrorCodes } from "../import-tTMN0WRi.js";
 import { Command } from "commander";
 import { nonnegative, optional, refine, z } from "zod/mini";
 import * as path$1 from "node:path";
 import { basename, dirname, isAbsolute, join, posix, relative, resolve, sep } from "node:path";
 import * as os$1 from "node:os";
 import { promisify } from "node:util";
-import { dump, load } from "js-yaml";
+import { dump } from "js-yaml";
 import { Semaphore } from "es-toolkit/promise";
 import { RequestError } from "@octokit/request-error";
 import { Octokit } from "@octokit/rest";
@@ -1561,12 +1561,13 @@ const gitignoreCommand = async (logger, options) => {
 	const gitignorePath = join(process.cwd(), ".gitignore");
 	const gitattributesPath = join(process.cwd(), ".gitattributes");
 	const config = await ConfigResolver.resolve({});
+	const resolvedEntries = resolveGitignoreEntries({
+		targets: options?.targets,
+		features: options?.features,
+		logger
+	});
 	const { gitignore: gitignoreEntries, gitattributes: gitattributesEntries } = groupEntriesByDestination({
-		entries: resolveGitignoreEntries({
-			targets: options?.targets,
-			features: options?.features,
-			logger
-		}),
+		entries: resolvedEntries,
 		resolveDestination: (target, feature) => {
 			if (feature === void 0 || feature === "general") return config.getGitignoreDestination(target);
 			return config.getGitignoreDestination(target, feature);
@@ -2021,7 +2022,7 @@ function parseApmLock(content) {
 	if (!content.trim()) return null;
 	let loaded;
 	try {
-		loaded = load(content);
+		loaded = loadYaml(content);
 	} catch {
 		return null;
 	}
@@ -2091,7 +2092,7 @@ async function apmManifestExists(projectRoot) {
 * or any dependency entry fails normalization.
 */
 function parseApmManifest(content) {
-	const loaded = load(content);
+	const loaded = loadYaml(content);
 	if (loaded === void 0 || loaded === null) return { dependencies: [] };
 	const parsed = ApmManifestSchema.safeParse(loaded);
 	if (!parsed.success) throw new Error(`Invalid apm.yml: ${parsed.error.message}`);
@@ -2560,11 +2561,14 @@ function injectSourceMetadata(params) {
 	if (content.startsWith(`${FRONTMATTER_FENCE}\r\n`)) openFenceLen = 5;
 	else if (content.startsWith(`${FRONTMATTER_FENCE}\n`)) openFenceLen = 4;
 	else if (content === FRONTMATTER_FENCE) openFenceLen = 3;
-	else return `${FRONTMATTER_FENCE}\n${dump(provenance, {
-		noRefs: true,
-		lineWidth: -1,
-		sortKeys: false
-	})}${FRONTMATTER_FENCE}\n${content}`;
+	else {
+		const yaml = dump(provenance, {
+			noRefs: true,
+			lineWidth: -1,
+			sortKeys: false
+		});
+		return `${FRONTMATTER_FENCE}\n${yaml}${FRONTMATTER_FENCE}\n${content}`;
+	}
 	const afterOpen = content.substring(openFenceLen);
 	let fmBody;
 	let rest;
@@ -2580,24 +2584,28 @@ function injectSourceMetadata(params) {
 	}
 	let loaded;
 	try {
-		loaded = load(fmBody);
+		loaded = loadYaml(fmBody);
 	} catch {
 		throw new Error("invalid frontmatter");
 	}
-	if (loaded === null || loaded === void 0) return `${FRONTMATTER_FENCE}\n${dump(provenance, {
-		noRefs: true,
-		lineWidth: -1,
-		sortKeys: false
-	})}${FRONTMATTER_FENCE}\n${rest}`;
+	if (loaded === null || loaded === void 0) {
+		const yaml = dump(provenance, {
+			noRefs: true,
+			lineWidth: -1,
+			sortKeys: false
+		});
+		return `${FRONTMATTER_FENCE}\n${yaml}${FRONTMATTER_FENCE}\n${rest}`;
+	}
 	if (typeof loaded !== "object" || Array.isArray(loaded)) throw new Error("invalid frontmatter");
-	return `${FRONTMATTER_FENCE}\n${dump({
+	const yaml = dump({
 		...loaded,
 		...provenance
 	}, {
 		noRefs: true,
 		lineWidth: -1,
 		sortKeys: false
-	})}${FRONTMATTER_FENCE}\n${rest}`;
+	});
+	return `${FRONTMATTER_FENCE}\n${yaml}${FRONTMATTER_FENCE}\n${rest}`;
 }
 //#endregion
 //#region src/lib/gh/gh-lock.ts
@@ -2666,7 +2674,7 @@ function parseGhLock(content) {
 	if (!content.trim()) return null;
 	let loaded;
 	try {
-		loaded = load(content);
+		loaded = loadYaml(content);
 	} catch {
 		return null;
 	}
@@ -3801,6 +3809,21 @@ function getFirstPathSeparatorIndex(path) {
 	if (backslashIndex === -1) return slashIndex;
 	return Math.min(slashIndex, backslashIndex);
 }
+/**
+* Decide whether a repository's root-level files should be installed as the
+* single requested skill (the "root fallback").
+*
+* A root fallback fires only when a single, non-wildcard skill was requested,
+* that skill's own directory is absent, and the repository root actually carries
+* a `SKILL.md`. Both the git transport (`groupRemoteFilesBySkillRoot`) and the
+* GitHub transport (`discoverGithubSkillDirs`) gate on these same conditions, so
+* the decision lives here to keep the two paths from drifting.
+*/
+function shouldUseRootFallback(params) {
+	const { skillFilter, isWildcard, hasRootSkillFile, hasRequestedSkillDir } = params;
+	const [singleSkillName] = skillFilter;
+	return !isWildcard && skillFilter.length === 1 && singleSkillName !== void 0 && hasRootSkillFile && !hasRequestedSkillDir;
+}
 function groupRemoteFilesBySkillRoot(params) {
 	const { remoteFiles, skillFilter, isWildcard } = params;
 	const grouped = /* @__PURE__ */ new Map();
@@ -3823,7 +3846,12 @@ function groupRemoteFilesBySkillRoot(params) {
 	}
 	const [singleSkillName] = skillFilter;
 	const hasRootSkillFile = rootLevelFiles.some((file) => file.relativePath === SKILL_FILE_NAME$1);
-	if (!isWildcard && skillFilter.length === 1 && singleSkillName !== void 0 && hasRootSkillFile && !grouped.has(singleSkillName)) grouped.set(singleSkillName, rootLevelFiles);
+	if (singleSkillName !== void 0 && shouldUseRootFallback({
+		skillFilter,
+		isWildcard,
+		hasRootSkillFile,
+		hasRequestedSkillDir: grouped.has(singleSkillName)
+	})) grouped.set(singleSkillName, rootLevelFiles);
 	return grouped;
 }
 /**
@@ -3960,7 +3988,12 @@ async function discoverGithubSkillDirs(params) {
 		}));
 		const [singleSkillName] = skillFilter;
 		const hasRequestedSkillDir = singleSkillName !== void 0 && remoteSkillDirs.some((d) => d.name === singleSkillName);
-		if (!isWildcard && skillFilter.length === 1 && !hasRequestedSkillDir) {
+		if (shouldUseRootFallback({
+			skillFilter,
+			isWildcard,
+			hasRootSkillFile: entries.some((entry) => entry.type === "file" && entry.name === "SKILL.md"),
+			hasRequestedSkillDir
+		})) {
 			const fallback = await fetchRootLevelFallbackSkill({
 				entries,
 				parsed,
@@ -6426,7 +6459,7 @@ function wrapCommand$1({ name, errorCode, handler, getVersion, loggerFactory = c
 }
 //#endregion
 //#region src/cli/index.ts
-const getVersion = () => "9.5.0";
+const getVersion = () => "9.6.1";
 function wrapCommand(name, errorCode, handler) {
 	return wrapCommand$1({
 		name,