rulesync 16.30.0 → 16.30.2

package/dist/{import-BDlmyTcr.js → import-BknrMODR.js}

@@ -69862,6 +69862,78 @@ const getProcessorRegistryEntry = (feature) => {
 	return entry;
 };
 //#endregion
+//#region src/lib/fold-root-overwrite-watch.ts
+/**
+* Whether a target folds every non-root rule into its root file, i.e. its tool
+* reads the one root file and nothing else.
+*/
+function foldsIntoRoot({ toolTarget }) {
+	return RulesProcessor.getFactory(toolTarget)?.meta.collisionPolicy === "fold";
+}
+/**
+* Names the root file the way the rest of the run does: relative to the
+* working directory when it lives under it (so `AGENTS.md`, or
+* `packages/app/AGENTS.md` for a second output root), absolute otherwise.
+*/
+function displayPath({ filePath }) {
+	const rel = relative(process.cwd(), filePath);
+	return rel === "" || isAbsolute(rel) || rel === ".." || rel.startsWith(`..${sep}`) ? filePath : toPosixPath(rel);
+}
+/**
+* Watches for a later target overwriting a fold target's root file with
+* different content, which silently loses every non-root rule for that tool.
+*
+* Several targets write the same root file (`AGENTS.md` above all), and the
+* documented rule is that the last target in config order wins. For a target
+* that files non-root rules in its own directory that is harmless: it keeps the
+* root body either way. A `collisionPolicy: "fold"` target (codexcli and the
+* others whose tool reads only the one root file) has nowhere else to put its
+* non-root rules, so when a sibling that emits the root body alone comes later
+* in config order — `["codexcli", "zoocode"]` — the fold is overwritten and
+* Codex CLI is left with the root rule only, in a diff that reads as a large
+* deletion of `AGENTS.md`. See issue #3022.
+*
+* The overwrite itself is not prevented (last-wins is what the docs promise);
+* it is named, once per root file, together with the reordering that keeps the
+* folded content. `observe` only records what each target would write; the
+* verdict is `report`'s, once every target has been seen, because only the
+* final writer decides what is on disk: with `["codexcli", "zoocode", "pi"]`
+* the fold target `pi` wins the file with every non-root body in it, so nothing
+* is lost and nothing is reported. Files are keyed by their resolved output
+* path, so a project with several output roots is compared root by root
+* however each root was spelled. In `--check` mode nothing is
+* written, but the sentence describes the same outcome of the same config, so
+* it is worded the same.
+*/
+function createFoldRootOverwriteWatch({ logger }) {
+	const writesByPath = /* @__PURE__ */ new Map();
+	return {
+		observe: ({ toolTarget, toolFiles }) => {
+			const folds = foldsIntoRoot({ toolTarget });
+			for (const file of toolFiles) {
+				if (!(file instanceof ToolRule) || !file.isRoot()) continue;
+				const path = resolve(file.getFilePath());
+				const writes = writesByPath.get(path) ?? [];
+				writes.push({
+					target: toolTarget,
+					folds,
+					content: file.getFileContent()
+				});
+				writesByPath.set(path, writes);
+			}
+		},
+		report: () => {
+			for (const [path, writes] of writesByPath) {
+				const last = writes.at(-1);
+				if (last === void 0 || last.folds) continue;
+				const fold = writes.findLast((write) => write.folds && write.content !== last.content);
+				if (fold === void 0) continue;
+				logger.warn(`Target '${last.target}' overwrites ${displayPath({ filePath: path })}, the file target '${fold.target}' folds every non-root rule into, so '${fold.target}' is left with the root rule only. The last target in config order wins a shared file: list '${fold.target}' after '${last.target}' to keep the folded content (see "Target Order and File Conflicts" in the configuration guide).`);
+			}
+		}
+	};
+}
+//#endregion
 //#region src/lib/orphan-sweep.ts
 function createOrphanSweepPlan() {
 	const generatedPaths = /* @__PURE__ */ new Set();
@@ -70171,17 +70243,19 @@ async function processEmptyFeatureGeneration(params) {
 * based on whether rulesync files exist.
 */
 async function processFeatureWithRulesyncFiles(params) {
-	const { config, processor, rulesyncFiles, sweepPlan, skipFilePaths } = params;
+	const { config, processor, rulesyncFiles, sweepPlan, skipFilePaths, onToolFiles } = params;
 	if (rulesyncFiles.length === 0) return processEmptyFeatureGeneration({
 		config,
 		processor,
 		sweepPlan,
 		skipFilePaths
 	});
+	const toolFiles = await processor.convertRulesyncFilesToToolFiles(rulesyncFiles);
+	onToolFiles?.(toolFiles);
 	return processFeatureGeneration({
 		config,
 		processor,
-		toolFiles: await processor.convertRulesyncFilesToToolFiles(rulesyncFiles),
+		toolFiles,
 		sweepPlan,
 		skipFilePaths
 	});
@@ -70599,6 +70673,7 @@ async function generateRulesCore(params) {
 		featureName: "rules",
 		logger
 	});
+	const foldRootOverwriteWatch = createFoldRootOverwriteWatch({ logger });
 	const isCheck = config.getCheck();
 	const rootFileOwner = isCheck ? computeRootFileOwnership({
 		targets: config.getConfigFileTargets(),
@@ -70635,13 +70710,18 @@ async function generateRulesCore(params) {
 			processor,
 			rulesyncFiles,
 			sweepPlan,
-			skipFilePaths: skipFilePaths.size > 0 ? skipFilePaths : void 0
+			skipFilePaths: skipFilePaths.size > 0 ? skipFilePaths : void 0,
+			onToolFiles: (toolFiles) => foldRootOverwriteWatch.observe({
+				toolTarget,
+				toolFiles
+			})
 		});
 		totalCount += result.count;
 		allPaths.push(...result.paths);
 		if (result.hasDiff) hasDiff = true;
 		if (result.sourceLoadFailed) sourceLoadFailed = true;
 	}
+	foldRootOverwriteWatch.report();
 	return {
 		count: totalCount,
 		paths: allPaths,
@@ -71396,4 +71476,4 @@ async function importChecksCore(params) {
 //#endregion
 export { RulesyncCheck as $, ALL_TOOL_TARGETS as $t, CLAUDECODE_SKILLS_DIR_PATH as A, RULESYNC_SOURCES_LOCK_RELATIVE_FILE_PATH as An, ensureDir as At, RulesyncSkill as B, quoteForLog as Bn, pathEscapesRoot as Bt, ChecksProcessor as C, RULESYNC_PERMISSIONS_FILE_NAME as Cn, applyFileMode as Ct, CLAUDECODE_LOCAL_RULE_FILE_NAME as D, RULESYNC_RELATIVE_DIR_PATH as Dn, checkPathTraversal as Dt, CLAUDECODE_DIR as E, RULESYNC_PERMISSIONS_SCHEMA_URL as En, assertWritablePathInsideRoot as Et, AUGMENTCODE_DIR as F, DEPRECATED_FEATURE_REPLACEMENTS as Fn, isFileSystemError as Ft, RulesyncMcp as G, removeFile as Gt, RulesyncRule as H, stripControlCharactersKeepingLineFeeds as Hn, readFileContentOrNull as Ht, AUGMENTCODE_SETTINGS_LOCAL_FILE_NAME as I, formatError as In, isSymlink as It, getRulesyncSourceCandidates as J, resolvePath as Jt, RulesyncIgnore as K, removeFileStrict as Kt, getLocalSkillDirNames as L, truncateText as Ln, listDirectoryEntryNames as Lt, FACTORYDROID_SETTINGS_LOCAL_FILE_NAME as M, parseCommaSeparatedList as Mn, getFileSize as Mt, caseFoldIdentity as N, ALL_FEATURES as Nn, getHomeDirectory as Nt, CLAUDECODE_MEMORIES_DIR_NAME as O, RULESYNC_RULES_RELATIVE_DIR_PATH as On, createTempDirectory as Ot, groupSpellingsByCaseFoldedIdentity as P, ALL_FEATURES_WITH_WILDCARD as Pn, isFileNotFoundError as Pt, RulesyncCommandFrontmatterSchema as Q, writeFileContent as Qt, RulesyncSubagent as R, hasDeceptiveHiddenCharacters as Rn, listFilePathsRecursively as Rt, QWENCODE_LOCAL_RULE_FILE_NAME as S, RULESYNC_NPM_SOURCES_LOCK_RELATIVE_FILE_PATH as Sn, ErrorCodes as St, CODEXCLI_DIR as T, RULESYNC_PERMISSIONS_RELATIVE_FILE_PATH as Tn, assertTreeContainsNoSymlinks as Tt, RulesyncRuleFrontmatterSchema as U, stripHiddenCharacters as Un, removeDirectory as Ut, RulesyncSkillFrontmatterSchema as V, stripControlCharacters as Vn, readFileContent as Vt, RulesyncPermissions as W, removeDirectoryStrict as Wt, parseJsonc as X, toPosixPath as Xt, resolveRulesyncSourceWritePath as Y, runWithDirectoryRollback as Yt, RulesyncCommand as Z, writeFileBuffer as Zt, IgnoreProcessor as _, RULESYNC_LOCAL_CONFIG_RELATIVE_FILE_PATH as _n, warnOnConflictingFlags as _t, getProcessorRegistryEntry as a, RULESYNC_AIIGNORE_FILE_NAME as an, ConfigResolver as at, CommandsProcessor as b, RULESYNC_MCP_RELATIVE_FILE_PATH as bn, withWarnOnceScope as bt, RulesProcessor as c, RULESYNC_COMMANDS_RELATIVE_DIR_PATH as cn, CONFLICTING_TARGET_PAIRS as ct, CODEBUDDY_DIR as d, RULESYNC_CURATED_RULES_RELATIVE_DIR_PATH as dn, SourceEntrySchema as dt, ALL_TOOL_TARGETS_WITH_WILDCARD as en, RulesyncCheckFrontmatterSchema as et, CODEBUDDY_LOCAL_RULE_FILE_NAME as f, RULESYNC_CURATED_SKILLS_RELATIVE_DIR_PATH as fn, findControlCharacter as ft, McpProcessor as g, RULESYNC_IGNORE_RELATIVE_FILE_PATH as gn, fallbackLogger as gt, shortenToWidth as h, RULESYNC_HOOKS_RELATIVE_FILE_PATH as hn, WarningCollectingLogger as ht, inspectInputRoots as i, MAX_FILE_SIZE as in, SKILL_FILE_NAME as it, FACTORYDROID_DIR as j, RULESYNC_SUBAGENTS_RELATIVE_DIR_PATH as jn, fileExists as jt, CLAUDECODE_SETTINGS_LOCAL_FILE_NAME as k, RULESYNC_SKILLS_RELATIVE_DIR_PATH as kn, directoryExists as kt, SubagentsProcessor as l, RULESYNC_CONFIG_RELATIVE_FILE_PATH as ln, ConfigFileSchema as lt, displayWidthOf as m, RULESYNC_HOOKS_LEGACY_FILE_NAME as mn, JsonLogger as mt, formatSourceLoadFailure as n, ToolTargetSchema as nn, loadYaml as nt, convertFromTool as o, RULESYNC_AIIGNORE_RELATIVE_FILE_PATH as on, mergeInputRootConfigs as ot, ELLIPSIS_WIDTH as p, RULESYNC_HOOKS_FILE_NAME as pn, ConsoleLogger as pt, RulesyncHooks as q, removeTempDirectory as qt, generate as r, CURATED_RULES_FEATURE_SUBDIR as rn, SHARED_USER_MANAGED_CONFIG_PATHS as rt, isPackagingToolTarget as s, RULESYNC_CHECKS_RELATIVE_DIR_PATH as sn, resolveEffectiveInputRoots as st, importFromTool as t, PACKAGING_TOOL_TARGETS as tn, stringifyFrontmatter as tt, SkillsProcessor as u, RULESYNC_CONFIG_SCHEMA_URL as un, GITIGNORE_DESTINATION_KEY as ut, CRUSH_LOCAL_RULE_FILE_NAME as v, RULESYNC_MCP_FILE_NAME as vn, withFallbackLoggerTarget as vt, CODEXCLI_BASH_RULES_FILE_NAME as w, RULESYNC_PERMISSIONS_LEGACY_FILE_NAME as wn, assertDirectoryIfExists as wt, QWENCODE_DIR as x, RULESYNC_MCP_SCHEMA_URL as xn, CLIError as xt, HooksProcessor as y, RULESYNC_MCP_LEGACY_FILE_NAME as yn, resetRunWarningState as yt, RulesyncSubagentFrontmatterSchema as z, hasEnclosingMarkOutsideKeycap as zn, listSubdirectoryNames as zt };
 
-//# sourceMappingURL=import-BDlmyTcr.js.map
+//# sourceMappingURL=import-BknrMODR.js.map