@x18n/core 0.1.9 → 0.1.10

package/dist/index.d.mts

@@ -561,6 +561,8 @@ interface CheckContext {
   readonly usages: () => Iterable<Usage>;
   readonly coverage: () => readonly LocaleCoverage[];
   readonly unused: () => readonly KeyString[];
+  /** Fully dynamic usages in a scope (`keyPrefix`, namespace) that may reach the key. `planUnused` keeps a key they reach. */
+  readonly dynamicUsages: (key: KeyString) => readonly Usage[];
   /** Usages whose key is not defined in the source locale. */
   readonly undefinedUsages: () => readonly Usage[];
   readonly placeholders: (value: string) => readonly Placeholder[];
@@ -597,6 +599,20 @@ interface ReportInput {
   readonly diagnostics: readonly Diagnostic[];
   readonly coverage: readonly LocaleCoverage[];
   readonly summary: ReportSummary;
+  /** Set with `x18n check --fix`; `diagnostics` are then the ones left after the fixes (or before them, in a dry run). */
+  readonly fix?: FixSummary | undefined;
+}
+/** What `x18n check --fix` did in a project. */
+interface FixSummary {
+  /** False in a dry run. */
+  readonly applied: boolean;
+  /** Diagnostics whose fix is in the plan. */
+  readonly fixed: number;
+  /** Diagnostics left out because their only bulk fix is `unsafe` (pass `--fix=unsafe`). */
+  readonly skipped: number;
+  /** Files the plan writes, relative to the project root. */
+  readonly files: readonly string[];
+  readonly warnings: readonly string[];
 }
 /** What a check baseline left out (`x18n check --baseline`). */
 interface BaselineSummary {
@@ -1157,11 +1173,13 @@ interface Usage {
   readonly scopePrefix: string | undefined;
 }
 type Severity = "error" | "warning" | "info" | "hint";
+/** A `set` with `replace: false` is skipped, with a warning, when it would replace other keys or a message on its path. */
 type EditOp = {
   readonly type: "set";
   readonly key: KeyString;
   readonly locale: LocaleCode;
   readonly value: MessageLeaf;
+  readonly replace?: boolean;
 } | {
   readonly type: "rename";
   readonly from: KeyString;
@@ -1215,9 +1233,15 @@ interface ExtractItem {
   /** Lines to add once per file when missing, for example an import. */
   readonly imports?: readonly string[];
 }
+/**
+ * How `x18n check --fix` may apply a fix. `safe` fixes only add, so `--fix` applies them; `unsafe` fixes can lose data, so
+ * only `--fix=unsafe` applies them. A fix without a level is offered in the editor and never applied in bulk.
+ */
+type FixLevel = "safe" | "unsafe";
 interface Fix {
   readonly title: string;
   readonly ops: readonly EditOp[];
+  readonly level?: FixLevel | undefined;
 }
 interface Location {
   readonly path: string;
@@ -1659,6 +1683,20 @@ interface UnusedDeletion {
   /** Sorted by key. */
   readonly kept: readonly KeptUnusedKey[];
 }
+/** Which fixes `planFixes` applies. */
+interface FixQuery {
+  /** Also apply `unsafe` fixes (`x18n check --fix=unsafe`). */
+  readonly unsafe?: boolean | undefined;
+}
+/** The fixes of many diagnostics, as one plan: review it (dry run), then pass `plan` to `apply`. */
+interface FixPlan {
+  /** Deletes first, then the other ops, each planned once. */
+  readonly plan: Plan;
+  /** The diagnostics whose fix is in the plan. */
+  readonly fixed: readonly Diagnostic[];
+  /** Diagnostics left out because their only bulk fix is `unsafe` and the query does not allow it. */
+  readonly skipped: readonly Diagnostic[];
+}
 interface ProjectExplanation {
   readonly name: string;
   readonly root: string;
@@ -1804,6 +1842,8 @@ interface Project {
   readonly plan: (ops: readonly EditOp[]) => Plan;
   /** Plans the deletion of the unused keys (`analyze().unused`), minus `ignoreKeys` and keys a scoped fully dynamic usage may reach. */
   readonly planUnused: (query?: UnusedQuery) => UnusedDeletion;
+  /** Plans the first fix of each diagnostic that has a `safe` fix, or also an `unsafe` one with `unsafe: true` (`x18n check --fix`). */
+  readonly planFixes: (diagnostics: readonly Diagnostic[], query?: FixQuery) => FixPlan;
   readonly apply: (plan: Plan, options?: {
     readonly answers?: Answers | undefined;
   }) => Promise<ApplyResult>;
@@ -1940,10 +1980,14 @@ interface LineIndex {
   readonly line: (line: number) => string;
   readonly lineCount: number;
 }
-/** Line index of a text, cached for the few texts used at a time (reporters, LSP ranges). */
+/**
+ * Line index of a text, cached for the few texts used at a time (reporters, LSP ranges).
+ * The least recently used text goes first, so a large resource hit by many diagnostics stays
+ * cached while ranges in many small source files pass through.
+ */
 export declare function createLineIndex(text: string): LineIndex;
 export declare function rangeContains(range: TextRange, offset: number): boolean;
 /** Fast non-cryptographic string hash (cyrb53), as hex. */
 export declare function hashText(text: string): string;
 //#endregion
-export type { Analysis, Answers, ApplyResult, ArgSource, BaselineSummary, Builtins, CacheStore, CallSpec, CapabilityConflict, Check, CheckContext, CheckIds, CheckLevel, CheckSetting, ConfigContext, ConfigDraft, ConfigOrigin, CreateProjectOptions, Decision, DecisionOption, DetectContext, Diagnostic, DiagnosticInput, Disposable, EditOp, Envelope, ErrorCode, ErrorInfo, ExtractItem, ExtractOptions, ExtractTemplate, Extractor, FileChange, FileEvent, FileScan, FileWrite, FinderContext, Fix, Format, FormatContext, FormatEntry, GlobOptions, GlossaryEntry, HardString, HistoryConfigs, HistoryContext, HistoryOptions, HistoryProvider, HistorySettings, Host, JsonObject, JsonPrimitive, JsonValue, KeptUnusedKey, KeyContext, KeyFile, KeyId, KeyInfo, KeyPattern, KeyQuery, KeySeparator, KeyString, KeyTarget, LazyFormat, LazyScanner, LibraryOptions, LineIndex, LinePosition, LocaleCode, LocaleCoverage, LocaleFinder, LocaleInfo, Location, Logger, ManifestReader, MemoryHost, Message, MessageChange, MessageHistory, MessageLeaf, MessageOp, MessageState, MessageTree, NamespaceFromPath, NamespaceOptions, NodeRange, OverlayHost, ParamsSource, ParseResult, Placeholder, PlaceholderComparer, PlaceholderProblem, PlaceholderSyntax, Plan, Plugin, PluginConfigs, PluginFile, PluginLoader, PluginOption, PluginRegistry, PluginSpecifier, PluralForm, Preset, PresetDetection, PresetEntry, PresetExplanation, PresetFactory, PresetLoader, PresetOption, Project, ProjectChange, ProjectExplanation, PromptContext, PromptData, PromptDraft, PromptIssue, PromptItem, PromptOptions, RegexScannerSpec, RelatedValue, ReportInput, ReportSummary, Reporter, ReporterContext, ReporterLoader, ResolveConfigOptions, ResolvedConfig, Resource, ResourceBlock, ResourceCandidate, ResourceId, ResourceNode, ResourceSpec, ResourceTransform, ResourceTransformContext, ReviewAuthor, ReviewComment, ReviewData, ReviewEdit, ReviewEditContext, ReviewFile, ReviewFilter, ReviewOptions, ReviewState, ReviewStore, ReviewThread, ReviewType, ScanContext, ScanResult, Scanner, ScannerEntry, SchemaIssue, Scope, SearchMatch, SearchOptions, SerializableConfig, Setup, Severity, SourceFile, TextEdit, TextRange, ThreadComment, TreeLeaf, UnusedDeletion, UnusedKeyDeletion, UnusedQuery, Usage, UsageCandidate, UsageContext, UsageKind, UsageMatch, UsageOptions, UsageQuery, UserConfig, WireDiagnostic, WireRange, WriteOptions, X18nError };
+export type { Analysis, Answers, ApplyResult, ArgSource, BaselineSummary, Builtins, CacheStore, CallSpec, CapabilityConflict, Check, CheckContext, CheckIds, CheckLevel, CheckSetting, ConfigContext, ConfigDraft, ConfigOrigin, CreateProjectOptions, Decision, DecisionOption, DetectContext, Diagnostic, DiagnosticInput, Disposable, EditOp, Envelope, ErrorCode, ErrorInfo, ExtractItem, ExtractOptions, ExtractTemplate, Extractor, FileChange, FileEvent, FileScan, FileWrite, FinderContext, Fix, FixLevel, FixPlan, FixQuery, FixSummary, Format, FormatContext, FormatEntry, GlobOptions, GlossaryEntry, HardString, HistoryConfigs, HistoryContext, HistoryOptions, HistoryProvider, HistorySettings, Host, JsonObject, JsonPrimitive, JsonValue, KeptUnusedKey, KeyContext, KeyFile, KeyId, KeyInfo, KeyPattern, KeyQuery, KeySeparator, KeyString, KeyTarget, LazyFormat, LazyScanner, LibraryOptions, LineIndex, LinePosition, LocaleCode, LocaleCoverage, LocaleFinder, LocaleInfo, Location, Logger, ManifestReader, MemoryHost, Message, MessageChange, MessageHistory, MessageLeaf, MessageOp, MessageState, MessageTree, NamespaceFromPath, NamespaceOptions, NodeRange, OverlayHost, ParamsSource, ParseResult, Placeholder, PlaceholderComparer, PlaceholderProblem, PlaceholderSyntax, Plan, Plugin, PluginConfigs, PluginFile, PluginLoader, PluginOption, PluginRegistry, PluginSpecifier, PluralForm, Preset, PresetDetection, PresetEntry, PresetExplanation, PresetFactory, PresetLoader, PresetOption, Project, ProjectChange, ProjectExplanation, PromptContext, PromptData, PromptDraft, PromptIssue, PromptItem, PromptOptions, RegexScannerSpec, RelatedValue, ReportInput, ReportSummary, Reporter, ReporterContext, ReporterLoader, ResolveConfigOptions, ResolvedConfig, Resource, ResourceBlock, ResourceCandidate, ResourceId, ResourceNode, ResourceSpec, ResourceTransform, ResourceTransformContext, ReviewAuthor, ReviewComment, ReviewData, ReviewEdit, ReviewEditContext, ReviewFile, ReviewFilter, ReviewOptions, ReviewState, ReviewStore, ReviewThread, ReviewType, ScanContext, ScanResult, Scanner, ScannerEntry, SchemaIssue, Scope, SearchMatch, SearchOptions, SerializableConfig, Setup, Severity, SourceFile, TextEdit, TextRange, ThreadComment, TreeLeaf, UnusedDeletion, UnusedKeyDeletion, UnusedQuery, Usage, UsageCandidate, UsageContext, UsageKind, UsageMatch, UsageOptions, UsageQuery, UserConfig, WireDiagnostic, WireRange, WriteOptions, X18nError };

package/dist/index.mjs

@@ -29,14 +29,19 @@ function applyTextEdits(text, edits) {
 	return out;
 }
 const lineIndexes = /* @__PURE__ */ new Map();
-/** Line index of a text, cached for the few texts used at a time (reporters, LSP ranges). */
+/**
+* Line index of a text, cached for the few texts used at a time (reporters, LSP ranges).
+* The least recently used text goes first, so a large resource hit by many diagnostics stays
+* cached while ranges in many small source files pass through.
+*/
 function createLineIndex(text) {
 	let index = lineIndexes.get(text);
-	if (!index) {
+	if (index) lineIndexes.delete(text);
+	else {
 		if (lineIndexes.size >= 16) lineIndexes.delete(lineIndexes.keys().next().value);
 		index = buildLineIndex(text);
-		lineIndexes.set(text, index);
 	}
+	lineIndexes.set(text, index);
 	return index;
 }
 function buildLineIndex(text) {
@@ -1119,6 +1124,9 @@ function renderExplanation(e, configFile) {
 	return lines.join("\n");
 }
 //#endregion
+//#region package.json
+var version = "0.1.10";
+//#endregion
 //#region src/project/history.ts
 const localChange = { updatedAt: Number.POSITIVE_INFINITY };
 /** A target message is stale when it last changed before its source message. */
@@ -1172,6 +1180,90 @@ function createHistory(input) {
 	};
 }
 //#endregion
+//#region src/project/unused.ts
+/**
+* True when a fully dynamic usage (`t(key)`) may reach `key`: the key lies in the usage's scope namespace and under its
+* scope key prefix. Undefined for a usage without a scope, which may reach any key.
+*/
+function scopeMatcher(usage, namespaced) {
+	const namespace = namespaced ? usage.scopeNamespace : void 0;
+	const prefix = usage.scopePrefix;
+	if (namespace === void 0 && !prefix) return void 0;
+	return (key) => {
+		const id = splitKey(key, namespaced);
+		if (namespace !== void 0 && id.namespace !== namespace) return false;
+		if (!prefix) return true;
+		const path = id.path.join(".");
+		return path === prefix || path.startsWith(`${prefix}.`);
+	};
+}
+function dynamicReach(usages, namespaced) {
+	const scoped = [];
+	const unscoped = [];
+	for (const usage of usages) {
+		if (usage.dynamic !== "unknown") continue;
+		const reaches = scopeMatcher(usage, namespaced);
+		if (reaches) scoped.push({
+			usage,
+			reaches
+		});
+		else unscoped.push(usage);
+	}
+	return {
+		reaching: (key) => scoped.filter((s) => s.reaches(key)).map((s) => s.usage),
+		unscoped
+	};
+}
+/** The plan warning for deleting unused keys while fully dynamic usages without a scope may reach any key. */
+function unscopedWarning(unscoped) {
+	const files = [...new Set(unscoped.map((usage) => usage.path))].sort();
+	const shown = files.length > 5 ? `${files.slice(0, 5).join(", ")} and ${files.length - 5} more files` : files.join(", ");
+	return `${unscoped.length} usage${unscoped.length === 1 ? " has" : "s have"} a fully dynamic key (in ${shown}) that may reach any key; list the keys they use in \`usage.keysInUse\` to keep them.`;
+}
+function planUnusedDeletion(options, query = {}) {
+	const ignored = keyMatcher(options.ignoreKeys);
+	const match = query.glob ? keyMatcher([query.glob]) : void 0;
+	const candidates = [...options.unused].filter((key) => !ignored(key) && (match === void 0 || match(key)) && (query.namespace === void 0 || splitKey(key, options.namespaced).namespace === query.namespace)).sort();
+	const reach = dynamicReach(options.usages, options.namespaced);
+	const keys = [];
+	const kept = [];
+	for (const key of candidates) {
+		const usages = reach.reaching(key);
+		if (usages.length > 0) {
+			kept.push({
+				key,
+				usages
+			});
+			continue;
+		}
+		keys.push({
+			key,
+			messages: options.locations(key).filter((node) => node.leaf).map((node) => ({
+				locale: node.locale,
+				path: node.path
+			}))
+		});
+	}
+	const plan = options.plan(keys.map(({ key }) => ({
+		type: "delete",
+		key,
+		locales: "all"
+	})));
+	if (reach.unscoped.length === 0 || keys.length === 0) return {
+		plan,
+		keys,
+		kept
+	};
+	return {
+		plan: {
+			...plan,
+			warnings: [...plan.warnings, unscopedWarning(reach.unscoped)]
+		},
+		keys,
+		kept
+	};
+}
+//#endregion
 //#region src/project/usages.ts
 function innermost(scopes, range) {
 	let best;
@@ -1510,6 +1602,7 @@ function createAnalyzer(options) {
 			return prefixes.some((prefix) => key.startsWith(prefix));
 		};
 		const unused = hasScanners ? allKeys.filter((key) => !isUsed(key)) : [];
+		const reach = dynamicReach(state.usages, options.namespaced());
 		const undefinedUsages = state.usages.filter((usage) => usage.dynamic === false && usage.key !== void 0 && !isDefined(usage.key));
 		const placeholders = (value) => {
 			let found = placeholderCache.get(value);
@@ -1537,6 +1630,7 @@ function createAnalyzer(options) {
 			usages: () => state.usages,
 			coverage: () => coverage,
 			unused: () => unused,
+			dynamicUsages: reach.reaching,
 			undefinedUsages: () => undefinedUsages,
 			placeholders,
 			placeholderSyntaxes: registry.placeholders,
@@ -1572,6 +1666,55 @@ function createAnalyzer(options) {
 	return { analyze };
 }
 //#endregion
+//#region src/project/fixes.ts
+/** The first fix of a diagnostic that the query allows: `safe`, or also `unsafe` with `unsafe: true`. */
+function pick(diagnostic, unsafe) {
+	return diagnostic.fixes.find((fix) => fix.level === "safe" || unsafe && fix.level === "unsafe");
+}
+/** Identifies ops that do the same: a second `set` of a key in a locale is dropped, so the first value wins. */
+function opId(op) {
+	return op.type === "set" ? JSON.stringify([
+		op.type,
+		op.key,
+		op.locale
+	]) : JSON.stringify(op);
+}
+/**
+* Plans the fixes of many diagnostics as one plan. Deletes go first, so a deleted message makes room for a key that a
+* later `set` with `replace: false` adds under it. Identical ops are planned once.
+*/
+function planFixes(diagnostics, options, query = {}) {
+	const unsafe = query.unsafe === true;
+	const fixed = [];
+	const skipped = [];
+	const deletes = /* @__PURE__ */ new Map();
+	const others = /* @__PURE__ */ new Map();
+	for (const diagnostic of diagnostics) {
+		const fix = pick(diagnostic, unsafe);
+		if (!fix) {
+			if (diagnostic.fixes.some((f) => f.level === "unsafe")) skipped.push(diagnostic);
+			continue;
+		}
+		fixed.push(diagnostic);
+		for (const op of fix.ops) {
+			const ops = op.type === "delete" ? deletes : others;
+			const id = opId(op);
+			if (!ops.has(id)) ops.set(id, op);
+		}
+	}
+	const plan = options.plan([...deletes.values(), ...others.values()]);
+	const deletesEverywhere = [...deletes.values()].some((op) => op.type === "delete" && (op.locales === void 0 || op.locales === "all"));
+	const { unscoped } = dynamicReach(options.usages, options.namespaced);
+	return {
+		plan: deletesEverywhere && unscoped.length > 0 ? {
+			...plan,
+			warnings: [...plan.warnings, unscopedWarning(unscoped)]
+		} : plan,
+		fixed,
+		skipped
+	};
+}
+//#endregion
 //#region src/project/reviews.ts
 const noReviews = {
 	threads: {},
@@ -2004,10 +2147,14 @@ var PlanBuilder = class {
 			if ((before !== void 0 && !isMessageLeaf(before)) !== (after !== void 0 && !isMessageLeaf(after)) || before !== void 0 && after === void 0) entry.flat = void 0;
 		}
 	}
-	setValue(key, locale, value) {
-		this.touched.add(key);
+	/**
+	* Returns the value the set replaced in `before`. Undefined when `replace` is false and the value would replace other keys
+	* or a message on its path: the set is then skipped with a warning.
+	*/
+	setValue(key, locale, value, replace = true) {
 		const existing = this.options.message(key, locale);
 		if (existing) {
+			this.touched.add(key);
 			const entry = this.workFor(this.targetOf(this.writable(existing.resource)));
 			const before = leafAt(entry.tree, existing.storedPath);
 			this.pushOp(entry, {
@@ -2015,20 +2162,36 @@ var PlanBuilder = class {
 				path: existing.storedPath,
 				value
 			});
-			return before;
+			return { before };
 		}
 		const target = this.pickTarget(key, locale);
 		if (target.state) this.writable(target.state.resource.id);
 		const entry = this.workFor(target);
 		const stored = this.storedPathFor(entry, this.relativePath(key, target));
 		const occupied = nodeAt(entry.tree, stored);
-		if (occupied !== void 0 && !isMessageLeaf(occupied)) this.warnings.push(`"${key}" is a parent of other keys in ${target.path}; they are replaced.`);
+		const parent = occupied !== void 0 && !isMessageLeaf(occupied);
+		if (!replace) {
+			const leaf = stored.slice(0, -1).findIndex((_, i) => {
+				const node = nodeAt(entry.tree, stored.slice(0, i + 1));
+				return node !== void 0 && isMessageLeaf(node);
+			});
+			if (leaf !== -1) {
+				const message = stored.slice(0, leaf + 1).join(keyJoiner(this.separatorOf(target)));
+				this.warnings.push(`"${key}" is not added: "${message}" is a message in ${target.path}, and adding "${key}" would replace it.`);
+				return;
+			}
+			if (parent) {
+				this.warnings.push(`"${key}" is not added: it is a parent of other keys in ${target.path}, and adding it would replace them.`);
+				return;
+			}
+		} else if (parent) this.warnings.push(`"${key}" is a parent of other keys in ${target.path}; they are replaced.`);
+		this.touched.add(key);
 		this.pushOp(entry, {
 			type: "set",
 			path: stored,
 			value
 		});
-		return isMessageLeaf(occupied) ? occupied : void 0;
+		return { before: isMessageLeaf(occupied) ? occupied : void 0 };
 	}
 	deleteMessage(m) {
 		const s = this.writable(m.resource);
@@ -2102,7 +2265,7 @@ var PlanBuilder = class {
 		const own = this.options.resolveKey(to, m.locale);
 		if (!relative || own && own.path !== s.resource.path) {
 			this.deleteMessage(m);
-			return this.setValue(to, m.locale, m.value);
+			return this.setValue(to, m.locale, m.value)?.before;
 		}
 		const entry = this.workFor(this.targetOf(s));
 		const separator = this.separatorOf(s.resource);
@@ -2313,11 +2476,12 @@ var PlanBuilder = class {
 		for (const locale of locales) this.recordSource(op.key, locale, true);
 	}
 	set(op) {
-		this.setValue(op.key, op.locale, op.value);
+		const replace = op.replace !== false;
+		if (!this.setValue(op.key, op.locale, op.value, replace)) return;
 		if (op.locale === this.state.sourceLocale) this.plannedSource.set(op.key, op.value);
 		else if (op.value !== "") this.recordSource(op.key, op.locale);
 		if (!this.config.write.keepFilled) return;
-		for (const locale of this.state.locales) if (locale !== op.locale && !this.state.fallbacks[locale] && !this.options.message(op.key, locale)) this.setValue(op.key, locale, "");
+		for (const locale of this.state.locales) if (locale !== op.locale && !this.state.fallbacks[locale] && !this.options.message(op.key, locale)) this.setValue(op.key, locale, "", replace);
 	}
 	run(ops) {
 		for (const op of ops) switch (op.type) {
@@ -2837,80 +3001,6 @@ function renderPrompt(data) {
 	return out.join("\n");
 }
 //#endregion
-//#region src/project/unused.ts
-/**
-* True when a fully dynamic usage (`t(key)`) may reach `key`: the key lies in the usage's scope namespace and under its
-* scope key prefix. Undefined for a usage without a scope, which may reach any key.
-*/
-function scopeMatcher(usage, namespaced) {
-	const namespace = namespaced ? usage.scopeNamespace : void 0;
-	const prefix = usage.scopePrefix;
-	if (namespace === void 0 && !prefix) return void 0;
-	return (key) => {
-		const id = splitKey(key, namespaced);
-		if (namespace !== void 0 && id.namespace !== namespace) return false;
-		if (!prefix) return true;
-		const path = id.path.join(".");
-		return path === prefix || path.startsWith(`${prefix}.`);
-	};
-}
-function planUnusedDeletion(options, query = {}) {
-	const ignored = keyMatcher(options.ignoreKeys);
-	const match = query.glob ? keyMatcher([query.glob]) : void 0;
-	const candidates = [...options.unused].filter((key) => !ignored(key) && (match === void 0 || match(key)) && (query.namespace === void 0 || splitKey(key, options.namespaced).namespace === query.namespace)).sort();
-	const scoped = [];
-	const unscoped = [];
-	for (const usage of options.usages) {
-		if (usage.dynamic !== "unknown") continue;
-		const reaches = scopeMatcher(usage, options.namespaced);
-		if (reaches) scoped.push({
-			usage,
-			reaches
-		});
-		else unscoped.push(usage);
-	}
-	const keys = [];
-	const kept = [];
-	for (const key of candidates) {
-		const usages = scoped.filter((s) => s.reaches(key)).map((s) => s.usage);
-		if (usages.length > 0) {
-			kept.push({
-				key,
-				usages
-			});
-			continue;
-		}
-		keys.push({
-			key,
-			messages: options.locations(key).filter((node) => node.leaf).map((node) => ({
-				locale: node.locale,
-				path: node.path
-			}))
-		});
-	}
-	const plan = options.plan(keys.map(({ key }) => ({
-		type: "delete",
-		key,
-		locales: "all"
-	})));
-	if (unscoped.length === 0 || keys.length === 0) return {
-		plan,
-		keys,
-		kept
-	};
-	const files = [...new Set(unscoped.map((usage) => usage.path))].sort();
-	const shown = files.length > 5 ? `${files.slice(0, 5).join(", ")} and ${files.length - 5} more files` : files.join(", ");
-	const warning = `${unscoped.length} usage${unscoped.length === 1 ? " has" : "s have"} a fully dynamic key (in ${shown}) that may reach any key; list the keys they use in \`usage.keysInUse\` to keep them.`;
-	return {
-		plan: {
-			...plan,
-			warnings: [...plan.warnings, warning]
-		},
-		keys,
-		kept
-	};
-}
-//#endregion
 //#region src/project/project.ts
 const ioConcurrency = 64;
 async function forEachLimit(items, limit, fn) {
@@ -2950,8 +3040,10 @@ function createProject(options) {
 	const { basesOf, formOf } = derivedKeys(config.usage.derivedKeys);
 	const scanSignature = hashText(JSON.stringify({
 		usage: config.usage,
+		extract: config.extract,
 		namespace: config.namespace,
 		scanners: registry.scanners.map((s) => s.id),
+		version,
 		v: 1
 	}));
 	const localeFilter = (code) => (!config.locales || config.locales.includes(code)) && !config.ignoreLocales.includes(code);
@@ -3529,8 +3621,14 @@ function createProject(options) {
 		basesOf,
 		formOf,
 		localeInfo,
-		text: (path) => fileTexts.get(path) ?? files.get(path)?.text
+		text: (path) => fileTexts.get(path) ?? files.get(path)?.text,
+		namespaced: () => namespaced
 	});
+	let lastAnalysis;
+	function analyze() {
+		if (lastAnalysis?.revision !== revision) lastAnalysis = analyzer.analyze();
+		return lastAnalysis;
+	}
 	function keys(query = {}) {
 		const { messages, keyPaths } = idx();
 		const match = query.glob ? keyMatcher([query.glob], config.keySeparator) : void 0;
@@ -3545,7 +3643,7 @@ function createProject(options) {
 			});
 		}
 		if (query.unused) {
-			const unused = new Set(analyzer.analyze().unused);
+			const unused = new Set(analyze().unused);
 			list = list.filter((key) => unused.has(key));
 		}
 		return list.sort();
@@ -3573,6 +3671,24 @@ function createProject(options) {
 		}
 		return found.sort((a, b) => b.score - a.score || a.value.length - b.value.length || a.key.localeCompare(b.key));
 	}
+	let diagnosticIndex;
+	function diagnosticsOf(key) {
+		const analysis = analyze();
+		if (diagnosticIndex?.analysis !== analysis) {
+			const byKey = /* @__PURE__ */ new Map();
+			for (const d of analysis.diagnostics) {
+				if (d.key === void 0) continue;
+				const list = byKey.get(d.key);
+				if (list) list.push(d);
+				else byKey.set(d.key, [d]);
+			}
+			diagnosticIndex = {
+				analysis,
+				byKey
+			};
+		}
+		return [...diagnosticIndex.byKey.get(key) ?? []];
+	}
 	function get(key) {
 		const { messages, keyPaths } = idx();
 		const byLocale = messages.get(key);
@@ -3587,7 +3703,7 @@ function createProject(options) {
 			messages: list,
 			missing: locales.filter((locale) => !byLocale.has(locale) && needs(key, locale)),
 			usages: usageMatches({ key }),
-			diagnostics: analyzer.analyze().diagnostics.filter((d) => d.key === key),
+			diagnostics: diagnosticsOf(key),
 			reviews: reviews({ key })
 		};
 	}
@@ -3809,7 +3925,7 @@ function createProject(options) {
 			...h,
 			path: p
 		}))),
-		analyze: () => analyzer.analyze(),
+		analyze,
 		async loadHistory(historyOptions = {}) {
 			await ready;
 			await history.load(historyOptions);
@@ -3832,13 +3948,18 @@ function createProject(options) {
 		owns: (path) => fileTexts.has(path) || files.has(path) || usageEligible(path),
 		plan: (ops) => planner.plan(ops),
 		planUnused: (query) => planUnusedDeletion({
-			unused: analyzer.analyze().unused,
+			unused: analyze().unused,
 			usages: idx().usages,
 			namespaced,
 			ignoreKeys: config.ignoreKeys,
 			locations: (key) => project.locations(key),
 			plan: (ops) => planner.plan(ops)
 		}, query),
+		planFixes: (diagnostics, query) => planFixes(diagnostics, {
+			usages: idx().usages,
+			namespaced,
+			plan: (ops) => planner.plan(ops)
+		}, query),
 		apply: (plan, applyOptions) => planner.apply(plan, applyOptions?.answers ?? {}),
 		update,
 		watch() {

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@x18n/core",
   "type": "module",
-  "version": "0.1.9",
+  "version": "0.1.10",
   "description": "Runtime-neutral engine of x18n: domain model, plugin API and project analysis",
   "author": "Anthony Fu <anthonyfu117@hotmail.com>",
   "license": "MIT",