rulesync 9.6.1 → 9.6.3

package/dist/{import-DW-a1SJq.cjs → import-DnpMhZOY.cjs}

@@ -12644,6 +12644,13 @@ function mapOauthFromCodex(oauth) {
 	}
 	return result;
 }
+const CODEX_MCP_SERVER_NAME_PATTERN = /^[a-zA-Z0-9_-]+$/;
+function normalizeCodexMcpServerName(name) {
+	if (PROTOTYPE_POLLUTION_KEYS.has(name)) return null;
+	if (CODEX_MCP_SERVER_NAME_PATTERN.test(name)) return name;
+	const normalizedName = name.toLowerCase().replace(/[^a-z0-9]+/g, "_").replace(/^_+|_+$/g, "");
+	return normalizedName && !PROTOTYPE_POLLUTION_KEYS.has(normalizedName) ? normalizedName : null;
+}
 function convertFromCodexFormat(codexMcp) {
 	const result = {};
 	for (const [name, config] of Object.entries(codexMcp)) {
@@ -12666,8 +12673,13 @@ function convertFromCodexFormat(codexMcp) {
 }
 function convertToCodexFormat(mcpServers) {
 	const result = {};
+	const originalNames = /* @__PURE__ */ new Map();
 	for (const [name, config] of Object.entries(mcpServers)) {
-		if (PROTOTYPE_POLLUTION_KEYS.has(name)) continue;
+		const codexName = normalizeCodexMcpServerName(name);
+		if (codexName === null) {
+			warnWithFallback(void 0, `MCP server "${name}" could not be normalized to a valid Codex MCP server name and was skipped.`);
+			continue;
+		}
 		if (!isRecord(config)) continue;
 		const converted = {};
 		for (const [key, value] of Object.entries(config)) {
@@ -12681,7 +12693,10 @@ function convertToCodexFormat(mcpServers) {
 				else warnWithFallback(void 0, `[CodexCliMcp] Skipping invalid value type for mapped key '${key}': expected string array, got ${typeof value}`);
 			} else converted[key] = value;
 		}
-		result[name] = converted;
+		const previousName = originalNames.get(codexName);
+		if (previousName !== void 0) warnWithFallback(void 0, `Codex MCP server name collision: "${previousName}" and "${name}" both normalize to "${codexName}". Only the last processed server will be used.`);
+		originalNames.set(codexName, name);
+		result[codexName] = converted;
 	}
 	return result;
 }
@@ -36637,30 +36652,31 @@ var RovodevRule = class RovodevRule extends ToolRule {
 		});
 	}
 	/**
-	* Mirror the primary `.rovodev/AGENTS.md` root rule to `./AGENTS.md` so project
-	* memory stays discoverable at the repo root. Empty when `rootRule` is not the primary root.
+	* Root-mirror contract: rovodev mirrors its primary `.rovodev/AGENTS.md` root
+	* rule to `./AGENTS.md` so project memory stays discoverable at the repo root.
+	* Generation (`getMirrorFiles`) and deletion (`getMirrorDeletionGlobs`) are
+	* bundled in one object so they cannot drift out of symmetry. The processor
+	* keys off this method's mere presence to decide the tool mirrors at all.
 	*/
-	static getRootMirrorFiles({ outputRoot, rootRule, content }) {
-		if (!(rootRule instanceof RovodevRule)) return [];
-		const primary = this.getSettablePaths({ global: false }).root;
-		if (rootRule.getRelativeDirPath() !== primary.relativeDirPath || rootRule.getRelativeFilePath() !== primary.relativeFilePath) return [];
-		return [new RovodevRule({
-			outputRoot,
-			relativeDirPath: ".",
-			relativeFilePath: ROVODEV_RULE_FILE_NAME,
-			fileContent: content,
-			validate: true,
-			root: true
-		})];
-	}
-	/**
-	* Globs for mirror deletion: the `./AGENTS.md` mirror (`mirrorGlob`) is deleted
-	* only when the primary `.rovodev/AGENTS.md` (`primaryGlob`) still exists.
-	*/
-	static getRootMirrorDeletionGlobs({ outputRoot }) {
+	static getRootMirror() {
 		return {
-			primaryGlob: (0, node_path.join)(outputRoot, ROVODEV_DIR, ROVODEV_RULE_FILE_NAME),
-			mirrorGlob: (0, node_path.join)(outputRoot, ROVODEV_RULE_FILE_NAME)
+			getMirrorFiles: ({ outputRoot, rootRule, content }) => {
+				if (!(rootRule instanceof RovodevRule)) return [];
+				const primary = RovodevRule.getSettablePaths({ global: false }).root;
+				if (rootRule.getRelativeDirPath() !== primary.relativeDirPath || rootRule.getRelativeFilePath() !== primary.relativeFilePath) return [];
+				return [new RovodevRule({
+					outputRoot,
+					relativeDirPath: ".",
+					relativeFilePath: ROVODEV_RULE_FILE_NAME,
+					fileContent: content,
+					validate: true,
+					root: true
+				})];
+			},
+			getMirrorDeletionGlobs: ({ outputRoot }) => ({
+				primaryGlob: (0, node_path.join)(outputRoot, ROVODEV_DIR, ROVODEV_RULE_FILE_NAME),
+				mirrorGlob: (0, node_path.join)(outputRoot, ROVODEV_RULE_FILE_NAME)
+			})
 		};
 	}
 	/** Glob for the `separate-local-file` deletion; rovodev writes it at project root, not under `.rovodev/`. */
@@ -37283,8 +37299,7 @@ const toolRuleFactories = /* @__PURE__ */ new Map([
 				skills: { skillClass: RovodevSkill }
 			},
 			localRootMode: "separate-local-file",
-			localRootFileName: "AGENTS.local.md",
-			mirrorsRootToAgentsMd: true
+			localRootFileName: "AGENTS.local.md"
 		}
 	}],
 	["takt", {
@@ -37471,7 +37486,8 @@ var RulesProcessor = class extends FeatureProcessor {
 		if (!rootRule) return;
 		const newContent = this.generateReferenceSectionFromMeta(meta, toolRules) + (!meta.createsSeparateConventionsRule && meta.additionalConventions ? this.generateAdditionalConventionsSectionFromMeta(meta) : "") + rootRule.getFileContent();
 		rootRule.setFileContent(newContent);
-		if (meta.mirrorsRootToAgentsMd && !this.global && factory.class.getRootMirrorFiles) toolRules.push(...factory.class.getRootMirrorFiles({
+		const rootMirror = factory.class.getRootMirror?.();
+		if (rootMirror && !this.global) toolRules.push(...rootMirror.getMirrorFiles({
 			outputRoot: this.outputRoot,
 			rootRule,
 			content: newContent
@@ -37713,8 +37729,9 @@ As this project's AI coding tool, you must follow the additional conventions bel
 			})();
 			this.logger.debug(`Found ${localRootToolRules.length} local root tool rule files for deletion`);
 			const rootMirrorDeletionRules = await (async () => {
-				if (!forDeletion || this.global || !factory.meta.mirrorsRootToAgentsMd || !factory.class.getRootMirrorDeletionGlobs) return [];
-				const { primaryGlob, mirrorGlob } = factory.class.getRootMirrorDeletionGlobs({ outputRoot: this.outputRoot });
+				const rootMirror = factory.class.getRootMirror?.();
+				if (!forDeletion || this.global || !rootMirror) return [];
+				const { primaryGlob, mirrorGlob } = rootMirror.getMirrorDeletionGlobs({ outputRoot: this.outputRoot });
 				if ((await findFilesByGlobs(primaryGlob)).length === 0) return [];
 				const mirrorPaths = await findFilesByGlobs(mirrorGlob);
 				return buildDeletionRulesFromPaths(mirrorPaths);
@@ -38152,13 +38169,13 @@ const sharedFileKey = (path) => {
 	const file = path.relativeFilePath.replace(/\\/g, "/");
 	return dir === "" || dir === "." ? file : `${dir}/${file}`;
 };
-const settablePathsForScope = (cls, global) => {
+const settablePathsForScope = ({ cls, global }) => {
 	const paths = [];
 	let settable;
 	try {
 		settable = cls.getSettablePaths?.({ global });
 	} catch {
-		return paths;
+		settable = void 0;
 	}
 	if (settable?.relativeFilePath) paths.push({
 		relativeDirPath: settable.relativeDirPath ?? ".",
@@ -38168,16 +38185,22 @@ const settablePathsForScope = (cls, global) => {
 		paths.push(settable.root);
 		for (const alt of settable.alternativeRoots ?? []) paths.push(alt);
 	}
-	let extra;
+	let extra = [];
 	try {
 		extra = cls.getExtraSharedWritePaths?.({ global }) ?? [];
 	} catch {
-		return paths;
+		extra = [];
 	}
 	for (const path of extra) if (path.relativeFilePath) paths.push(path);
 	return paths;
 };
-const collectFactoryPaths = (factory) => [...settablePathsForScope(factory.class, false), ...settablePathsForScope(factory.class, true)];
+const collectFactoryPaths = (factory) => [...settablePathsForScope({
+	cls: factory.class,
+	global: false
+}), ...settablePathsForScope({
+	cls: factory.class,
+	global: true
+})];
 /**
 * Derive, from the processor registry, every on-disk file that two or more
 * features read-modify-write. Source of truth the generation step graph's
@@ -38553,7 +38576,7 @@ function computeRootFileOwnership(params) {
 		const paths = factory.class.getSettablePaths({ global: params.global });
 		if ("root" in paths && paths.root) register(paths.root.relativeDirPath, paths.root.relativeFilePath, target);
 		if ("alternativeRoots" in paths && paths.alternativeRoots) for (const alt of paths.alternativeRoots) register(alt.relativeDirPath, alt.relativeFilePath, target);
-		if (!params.global && factory.meta.mirrorsRootToAgentsMd) register(".", AGENTSMD_RULE_FILE_NAME, target);
+		if (!params.global && factory.class.getRootMirror) register(".", AGENTSMD_RULE_FILE_NAME, target);
 	}
 	return ownerByPath;
 }

package/dist/index.cjs

@@ -1,5 +1,5 @@
 Object.defineProperty(exports, Symbol.toStringTag, { value: "Module" });
-const require_import = require("./import-DW-a1SJq.cjs");
+const require_import = require("./import-DnpMhZOY.cjs");
 //#region src/index.ts
 async function generate(options = {}) {
 	const { silent = true, verbose = false, ...rest } = options;

package/dist/index.js

@@ -1,4 +1,4 @@
-import { I as ConsoleLogger, Ot as ALL_FEATURES, P as ConfigResolver, a as convertFromTool$1, n as checkRulesyncDirExists, nt as ALL_TOOL_TARGETS, r as generate$1, t as importFromTool$1 } from "./import-tTMN0WRi.js";
+import { I as ConsoleLogger, Ot as ALL_FEATURES, P as ConfigResolver, a as convertFromTool$1, n as checkRulesyncDirExists, nt as ALL_TOOL_TARGETS, r as generate$1, t as importFromTool$1 } from "./import-A1yGuzwv.js";
 //#region src/index.ts
 async function generate(options = {}) {
 	const { silent = true, verbose = false, ...rest } = options;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "rulesync",
-  "version": "9.6.1",
+  "version": "9.6.3",
   "description": "Unified AI rules management CLI tool that generates configuration files for various AI development tools",
   "keywords": [
     "ai",