rulesync 9.6.1 → 9.6.3

package/dist/{import-tTMN0WRi.js → import-A1yGuzwv.js}

@@ -12617,6 +12617,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)) {
@@ -12639,8 +12646,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)) {
@@ -12654,7 +12666,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;
 }
@@ -36610,30 +36625,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: join(outputRoot, ROVODEV_DIR, ROVODEV_RULE_FILE_NAME),
-			mirrorGlob: 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: join(outputRoot, ROVODEV_DIR, ROVODEV_RULE_FILE_NAME),
+				mirrorGlob: join(outputRoot, ROVODEV_RULE_FILE_NAME)
+			})
 		};
 	}
 	/** Glob for the `separate-local-file` deletion; rovodev writes it at project root, not under `.rovodev/`. */
@@ -37256,8 +37272,7 @@ const toolRuleFactories = /* @__PURE__ */ new Map([
 				skills: { skillClass: RovodevSkill }
 			},
 			localRootMode: "separate-local-file",
-			localRootFileName: "AGENTS.local.md",
-			mirrorsRootToAgentsMd: true
+			localRootFileName: "AGENTS.local.md"
 		}
 	}],
 	["takt", {
@@ -37444,7 +37459,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
@@ -37686,8 +37702,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);
@@ -38125,13 +38142,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 ?? ".",
@@ -38141,16 +38158,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
@@ -38526,7 +38549,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;
 }
@@ -39127,4 +39150,4 @@ async function importPermissionsCore(params) {
 //#endregion
 export { removeTempDirectory as $, RulesyncCommand as A, checkPathTraversal as B, RulesyncHooks as C, RULESYNC_RULES_RELATIVE_DIR_PATH as Ct, CLAUDECODE_MEMORIES_DIR_NAME as D, formatError as Dt, CLAUDECODE_LOCAL_RULE_FILE_NAME as E, RULESYNC_SUBAGENTS_RELATIVE_DIR_PATH as Et, findControlCharacter as F, findFilesByGlobs as G, directoryExists as H, ConsoleLogger as I, isSymlink as J, getFileSize as K, JsonLogger as L, stringifyFrontmatter as M, loadYaml as N, CLAUDECODE_SETTINGS_LOCAL_FILE_NAME as O, ALL_FEATURES as Ot, ConfigResolver as P, removeFile as Q, CLIError as R, HooksProcessor as S, RULESYNC_RELATIVE_DIR_PATH as St, CLAUDECODE_DIR as T, RULESYNC_SOURCES_LOCK_RELATIVE_FILE_PATH as Tt, ensureDir as U, createTempDirectory as V, fileExists as W, readFileContent as X, listDirectoryFiles as Y, removeDirectory as Z, RulesyncPermissions as _, RULESYNC_MCP_RELATIVE_FILE_PATH as _t, convertFromTool as a, MAX_FILE_SIZE as at, IgnoreProcessor as b, RULESYNC_PERMISSIONS_FILE_NAME as bt, RulesyncRuleFrontmatterSchema as c, RULESYNC_COMMANDS_RELATIVE_DIR_PATH as ct, RulesyncSubagentFrontmatterSchema as d, RULESYNC_CURATED_SKILLS_RELATIVE_DIR_PATH as dt, toPosixPath as et, SkillsProcessor as f, RULESYNC_HOOKS_FILE_NAME as ft, SKILL_FILE_NAME as g, RULESYNC_MCP_FILE_NAME as gt, RulesyncSkillFrontmatterSchema as h, RULESYNC_LOCAL_CONFIG_RELATIVE_FILE_PATH as ht, getProcessorRegistryEntry as i, ToolTargetSchema as it, RulesyncCommandFrontmatterSchema as j, CLAUDECODE_SKILLS_DIR_PATH as k, ALL_FEATURES_WITH_WILDCARD as kt, SubagentsProcessor as l, RULESYNC_CONFIG_RELATIVE_FILE_PATH as lt, RulesyncSkill as m, RULESYNC_IGNORE_RELATIVE_FILE_PATH as mt, checkRulesyncDirExists as n, ALL_TOOL_TARGETS as nt, RulesProcessor as o, RULESYNC_AIIGNORE_FILE_NAME as ot, getLocalSkillDirNames as p, RULESYNC_HOOKS_RELATIVE_FILE_PATH as pt, getHomeDirectory as q, generate as r, ALL_TOOL_TARGETS_WITH_WILDCARD as rt, RulesyncRule as s, RULESYNC_AIIGNORE_RELATIVE_FILE_PATH as st, importFromTool as t, writeFileContent as tt, RulesyncSubagent as u, RULESYNC_CONFIG_SCHEMA_URL as ut, McpProcessor as v, RULESYNC_MCP_SCHEMA_URL as vt, CommandsProcessor as w, RULESYNC_SKILLS_RELATIVE_DIR_PATH as wt, RulesyncIgnore as x, RULESYNC_PERMISSIONS_RELATIVE_FILE_PATH as xt, RulesyncMcp as y, RULESYNC_OVERVIEW_FILE_NAME as yt, ErrorCodes as z };
 
-//# sourceMappingURL=import-tTMN0WRi.js.map
+//# sourceMappingURL=import-A1yGuzwv.js.map