modwright 0.1.3 → 0.1.5

package/dist/server.js

@@ -4,7 +4,7 @@ import { StdioServerTransport } from "@modelcontextprotocol/sdk/server/stdio.js"
 import { z } from "zod";
 import { pathExists, tail } from "./core/fsutil.js";
 import { describeProject, findProjectFile, loadProject, locateProjectFile } from "./core/project/index.js";
-import { userDataDir } from "./core/home.js";
+import { packageRoot, userDataDir } from "./core/home.js";
 import { applyDataFetch, applyManagedInstall, checkToolchain, featureReadiness, locateTool, planDataFetch, planDataMemory, planManagedInstall, planToolMemory, readinessSummary, TOOL_SPECS, toolsToLocate } from "./core/toolchain/index.js";
 import { loadUserConfig } from "./core/userconfig.js";
 import { describeNeeds, isProjectTrusted, planTrust, projectTrustNeeds, trustRefusal } from "./core/trust.js";
@@ -14,6 +14,8 @@ import { buildContext, createValidatorRegistry, runValidators } from "./core/val
 import { runBuild } from "./core/build/index.js";
 import { classifyReload, detectRedHotTools, runDeploy } from "./core/deploy/index.js";
 import { armBridge, bridgeCandidateDirs, bridgeStatus, cetLogCandidates, listPendingArms, PROTOCOL_VERSION as BRIDGE_PROTOCOL_VERSION, pollCyberpunk, pollForResult, probeSchema as bridgeProbeSchema, readPendingArm, stageBridge } from "./core/bridge/index.js";
+import { removeBridge } from "./core/bridge/remove.js";
+import { BACKUP_RETENTION, pruneDated } from "./core/safety/retention.js";
 import { runTestPlan } from "./core/testplan/index.js";
 import { applyBuildLedgerHook, applyClaimTransition, applyDecisionRecord, applyValidateLedgerHook, checkLedger, claimsFilePath, listClaims, loadClaims, loadDecisions, saveClaims, saveDecisions, showClaim, } from "./core/ledger/index.js";
 import { IndexStore, defaultCacheDir, findLegacyIndex, indexFile, legacyIndexHint, loadKnowledge, resolveIndexFile, searchFacts, } from "./core/knowledge/index.js";
@@ -29,7 +31,27 @@ import { SURFACES, getSurface, installGameVersion, resolveInstall, surfaceIds }
 import { CONVERTERS, convert as runConvert, defaultLocateTool as defaultTextLocateTool, extractTemplate, fieldDiff, getTemplate, invalidateTemplate, listTemplates, } from "./core/text/index.js";
 import { bootstrapProject, describe as describeScaffold, findScaffold, findSequence, generateClaudeAssets, loadAllScaffolds, loadAllSequences, planSequence, runScaffold, summarize, summarizeSequences } from "./core/scaffold/index.js";
 import { loadSkillPromptCatalog, renderSkillPrompt } from "./core/prompts.js";
-const server = new McpServer({ name: "modwright", version: "0.1.0" });
+const server = new McpServer({ name: "modwright", version: await serverVersion() });
+async function serverVersion() {
+    try {
+        const pkg = JSON.parse(await fs.readFile(path.join(packageRoot(), "package.json"), "utf8"));
+        return pkg.version ?? "0.0.0";
+    }
+    catch {
+        return "0.0.0";
+    }
+}
+{
+    const register = server.registerTool.bind(server);
+    server.registerTool = (name, config, handler) => register(name, config, async (...args) => {
+        try {
+            return await handler(...args);
+        }
+        catch (error) {
+            return fail(`${String(name)} failed: ${error.message}`);
+        }
+    });
+}
 const registry = new SignatureRegistry();
 for (const surface of SURFACES) {
     if (surface.logSignatures?.length)
@@ -312,6 +334,7 @@ server.registerTool("who_touches", {
     const collection = await collectWrites(surface, install, rootIds ? { rootIds } : {});
     const { errors, ...coverage } = collection.coverage;
     const copies = duplicateCopies(collection);
+    const repeats = (collection.repeatedKeys ?? []).filter((r) => key ? r.key.toLowerCase().startsWith(key.toLowerCase()) : mod ? r.writerId.toLowerCase().includes(mod.toLowerCase()) || r.name.toLowerCase().includes(mod.toLowerCase()) : true);
     const result = key
         ? { mode: "key", ...whoTouches(collection, key, limit) }
         : mod
@@ -327,6 +350,16 @@ server.registerTool("who_touches", {
             errors: errors.slice(0, 25),
         },
         ...(copies.length > 0 ? { duplicateCopies: copies } : {}),
+        ...(repeats.length > 0
+            ? {
+                repeatedKeys: {
+                    total: repeats.length,
+                    items: repeats.slice(0, 50),
+                    note: "Each key is repeated within one mapping in that file: invalid YAML that TweakXL reads without an error. Which " +
+                        "occurrence takes effect is not established, so one of the edits may be silently lost.",
+                },
+            }
+            : {}),
         ...result,
         note: "Static: read from the mods' files, never from the running game. 'contested' means two or more mods write the " +
             "key and not only by appending; which one the game ends up with depends on load order, which this tool does " +
@@ -1299,19 +1332,59 @@ server.registerTool("rollback", {
         "allowOutsideRoots is set.",
     inputSchema: {
         path: z.string().optional().describe("modwright.json path or a directory to search from. Default: current directory."),
+        action: z
+            .enum(["list", "restore", "prune"])
+            .optional()
+            .describe("list (the default without backupId), restore (the default with one), or prune: delete old backup runs, keeping the newest 20 and anything under 14 days old. Only ModWright's own dated folders are ever pruned."),
         backupId: z.string().optional().describe("The stamp naming one backup directory under .modwright/backups (from this tool's own listing). Omit to list."),
-        mode: z.enum(["dry-run", "apply"]).default("dry-run").describe("dry-run reports what would be restored without writing a byte."),
+        backupRoot: z
+            .string()
+            .optional()
+            .describe("prune only: another backups folder to prune instead of the project's, e.g. one an older version left under a working directory. Must be a .modwright/backups folder or ModWright's per-user backups."),
+        mode: z.enum(["dry-run", "apply"]).default("dry-run").describe("dry-run reports what would be restored or pruned without changing anything."),
         installPath: installPathArg,
         allowOutsideRoots: z
             .boolean()
             .default(false)
             .describe("Restore entries outside the project and the game's mod roots too. Off by default: a backup that names any other path is refused whole."),
     },
-}, async ({ path: start, backupId, mode, installPath, allowOutsideRoots }) => {
+}, async ({ path: start, action, backupId, backupRoot: otherRoot, mode, installPath, allowOutsideRoots }) => {
+    if (action === "prune") {
+        let root;
+        if (otherRoot) {
+            const abs = path.resolve(otherRoot);
+            const posix = abs.split(path.sep).join("/").toLowerCase();
+            const perUser = path.join(userDataDir(), "backups").split(path.sep).join("/").toLowerCase();
+            if (!posix.endsWith("/.modwright/backups") && posix !== perUser) {
+                return fail(`rollback prune: ${abs} is not a .modwright/backups folder or ModWright's per-user backups (${path.join(userDataDir(), "backups")}).`);
+            }
+            root = abs;
+        }
+        else {
+            const project = await projectOrFail(start);
+            if ("isError" in project)
+                return project;
+            root = path.join(project.root, ".modwright", "backups");
+        }
+        const report = await pruneDated(root, "backup", BACKUP_RETENTION, { mode });
+        return json({
+            action,
+            ...report,
+            expired: report.expired.slice(0, 100),
+            expiredCount: report.expired.length,
+            note: report.expired.length === 0
+                ? `Nothing to prune: ${report.runs} backup run(s), all within the newest ${BACKUP_RETENTION.keepRuns} or under ${BACKUP_RETENTION.maxAgeDays} days old.`
+                : report.applied
+                    ? `Pruned ${report.expired.length - (report.failed?.length ?? 0)} folder(s); ${report.keptCount} kept.`
+                    : `Dry run: ${report.expired.length} folder(s) would be deleted outright (they are the backups); ${report.keptCount} kept. Repeat with mode 'apply'.`,
+        });
+    }
     const resolved = await projectOrFail(start);
     if ("isError" in resolved)
         return resolved;
     const backupRoot = path.join(resolved.root, ".modwright", "backups");
+    if (action === "restore" && !backupId)
+        return fail("rollback restore needs backupId (from this tool's listing).");
     if (!backupId) {
         const backups = await listBackups(backupRoot);
         return json({
@@ -1401,10 +1474,13 @@ server.registerTool("bridge", {
         "(dry-run by default for both; refuses on a protocolVersion mismatch rather than guessing). 'poll': wait for " +
         "that request's result up to a budget of max(15s, 3s x probe count) + window, capped at 300s, " +
         "reporting one of four named timeout outcomes rather than a bare 'timed out' — plus, on the CET log transport, " +
-        "'truncated' when the result line was cut mid-write. Test-plan execution is run_tests; claims are the " +
+        "'truncated' when the result line was cut mid-write. 'remove': take a deployed bridge back out of the game " +
+        "(every folder the project's withBridge deploy targets name), backed up under the project; dry-run by default. " +
+        "A deploy without the withBridge variant leaves an installed bridge in place. " +
+        "Test-plan execution is run_tests; claims are the " +
         "ledger tool.",
     inputSchema: {
-        action: z.enum(["status", "arm", "poll", "stage"]),
+        action: z.enum(["status", "arm", "poll", "stage", "remove"]),
         game: z
             .enum(BRIDGE_GAMES)
             .optional()
@@ -1475,6 +1551,38 @@ server.registerTool("bridge", {
             return fail(`bridge stage failed: ${error.message}`);
         }
     }
+    if (action === "remove") {
+        if (!start)
+            return fail('bridge remove requires "path" (the project whose deployed bridge to remove).');
+        const surfaceId = game ?? projectGame;
+        const removeSurface = surfaceId ? getSurface(surfaceId) : undefined;
+        if (!removeSurface)
+            return fail(`Could not determine the game for ${start}; pass game explicitly.`);
+        const install = await resolveInstall(removeSurface, installPath);
+        if (!install)
+            return fail(`No install detected for ${removeSurface.name}.`);
+        try {
+            const resolved = await loadProject(start);
+            const report = await removeBridge(resolved, removeSurface.modRoots(install), { mode });
+            const toRemove = report.removals.filter((r) => r.action === "remove").length;
+            return json({
+                action,
+                game: removeSurface.id,
+                project: resolved.project.name,
+                ...report,
+                note: report.removals.length === 0
+                    ? "The project has no withBridge deploy target: nothing to remove."
+                    : toRemove === 0
+                        ? "No deployed bridge folder was found: nothing to remove."
+                        : report.applied
+                            ? `Removed ${toRemove} bridge folder(s); the backup is under the project's .modwright/backups (rollback restores it).`
+                            : `Dry run: ${toRemove} bridge folder(s) would be removed, with a backup. Repeat with mode 'apply'.`,
+            });
+        }
+        catch (error) {
+            return fail(`bridge remove failed: ${error.message}`);
+        }
+    }
     const gameId = game ?? projectGame;
     if (!gameId)
         return fail('bridge status/arm/poll requires "game" (or a "path" to a project that names one).');
@@ -1650,7 +1758,7 @@ server.registerTool("ledger", {
         scopeFiles: z.array(z.string()).optional().describe("'record' kind:\"claim\": files this claim is about, for a new claim."),
         scopeEntries: z.array(z.string()).optional().describe("'record' kind:\"claim\": named entries (stat/tweak ids) this claim is about, for a new claim."),
         toStatus: z.enum(["authored", "checker-clean", "built", "verified", "contradicted"]).optional().describe("'record' kind:\"claim\": the status to move this claim to."),
-        tool: z.enum(["validate", "build", "run_tests", "ledger"]).optional().describe("'record' kind:\"claim\": which tool this write claims to be — defaults to \"ledger\" (a manual record/correction). Passing another tool's name does not skip that tool's own gate; it only labels the write."),
+        tool: z.enum(["validate", "build", "run_tests", "ledger"]).optional().describe("'record' kind:\"claim\": whose transition rule applies. \"ledger\" (the default) is a manual record or correction: any move is allowed, with a note when it skips a stage. Naming validate, build or run_tests applies that tool's rule instead: only the status that tool writes, and only forward. A name never widens what is allowed and never stands in for the tool's own checks."),
         by: z.string().optional().describe("'record': who or what made this write. Default \"ledger\"."),
         note: z.string().optional().describe("'record': why, for the history entry."),
         evidence: z

package/dist/surfaces/baldursgate3/validators/bridge.js

@@ -2,6 +2,6 @@ import { bridgeNeverShipValidator } from "../../../core/bridge/index.js";
 export const bridgeNeverShipsValidator = bridgeNeverShipValidator({
     id: "bg3.history.bridge-never-ships",
     game: "baldursgate3",
-    evidence: ["ModWright research"],
+    evidence: ["ModWright's own rule, decided 2026-09-08: the in-game verification bridge is an authoring aid and never ships in a release build."],
 });
 export const BRIDGE_VALIDATORS = [bridgeNeverShipsValidator];

package/dist/surfaces/baldursgate3/validators/templates.js

@@ -2,6 +2,6 @@ import { vanillaExtractNeverShipValidator } from "../../../core/text/validate.js
 export const vanillaExtractNeverShipsValidator = vanillaExtractNeverShipValidator({
     id: "bg3.history.vanilla-extract-never-ships",
     game: "baldursgate3",
-    evidence: ["ModWright research"],
+    evidence: ["ModWright's own rule, decided 2026-09-08: a template extracted from the user's own install is for authoring only, and a release never ships one verbatim (redistributing game data)."],
 });
 export const TEMPLATE_VALIDATORS = [vanillaExtractNeverShipsValidator];

package/dist/surfaces/cyberpunk2077/index/parsers/tweakxl-yaml.js

@@ -181,3 +181,32 @@ export function parseTweakXLYaml(text, file, options = {}) {
     }
     return ctx.records;
 }
+export function findRepeatedKeys(text) {
+    const lineCounter = new YAML.LineCounter();
+    const doc = YAML.parseDocument(text, { lineCounter, uniqueKeys: false });
+    if (doc.errors.length > 0)
+        return [];
+    const out = [];
+    const visit = (node, prefix) => {
+        if (YAML.isMap(node)) {
+            const seen = new Map();
+            for (const pair of node.items) {
+                if (!YAML.isScalar(pair.key))
+                    continue;
+                const name = String(pair.key.value);
+                const line = pair.key.range ? lineCounter.linePos(pair.key.range[0]).line : 0;
+                seen.set(name, [...(seen.get(name) ?? []), line]);
+                visit(pair.value, prefix ? `${prefix}.${name}` : name);
+            }
+            for (const [name, lines] of seen)
+                if (lines.length > 1)
+                    out.push({ key: prefix ? `${prefix}.${name}` : name, lines });
+        }
+        else if (YAML.isSeq(node)) {
+            for (const item of node.items)
+                visit(item, prefix);
+        }
+    };
+    visit(doc.contents, "");
+    return out.sort((a, b) => a.lines[0] - b.lines[0]);
+}

package/dist/surfaces/cyberpunk2077/validators/bridge.js

@@ -2,6 +2,6 @@ import { bridgeNeverShipValidator } from "../../../core/bridge/index.js";
 export const bridgeNeverShipsValidator = bridgeNeverShipValidator({
     id: "cp2077.history.bridge-never-ships",
     game: "cyberpunk2077",
-    evidence: ["ModWright research"],
+    evidence: ["ModWright's own rule, decided 2026-09-08: the in-game verification bridge is an authoring aid and never ships in a release build."],
 });
 export const BRIDGE_VALIDATORS = [bridgeNeverShipsValidator];

package/dist/surfaces/cyberpunk2077/validators/cr2w.js

@@ -264,7 +264,7 @@ function checkResourcePathShape(root, file) {
 const GAMEVERSION_EVIDENCE = [
     "cp2077.toolchain.gameversion-header",
     "cp2077.packaging.game-version-compatibility",
-    "observed: a WolvenKit 8.20.0 en-us.source.json header carries GameVersion 2310",
+    "observed by 2026-09-07: a WolvenKit 8.20.0 en-us.source.json header carries GameVersion 2310",
 ];
 function expectedGameVersion(marketingVersion) {
     const m = /^(\d+)\.(\d{1,2})$/.exec(marketingVersion.trim());

package/dist/surfaces/cyberpunk2077/validators/modsettings.js

@@ -1,6 +1,6 @@
 import * as path from "node:path";
 const ID = "cp2077.history.modsettings-annotation-keys";
-const EVIDENCE = ["ModWright research"];
+const EVIDENCE = ["cp2077.ui.modsettings.universal-annotation-keys", "cp2077.ui.modsettings.type-specific-annotation-keys", "cp2077.ui.modsettings.unannotated-property-is-invisible"];
 const REQUIRED_KEYS = ["mod", "category", "displayName"];
 function isWithin(parent, child) {
     const rel = path.relative(parent, child);

package/dist/surfaces/cyberpunk2077/validators/templates.js

@@ -2,6 +2,6 @@ import { vanillaExtractNeverShipValidator } from "../../../core/text/validate.js
 export const vanillaExtractNeverShipsValidator = vanillaExtractNeverShipValidator({
     id: "cp2077.history.vanilla-extract-never-ships",
     game: "cyberpunk2077",
-    evidence: ["ModWright research"],
+    evidence: ["ModWright's own rule, decided 2026-09-08: a template extracted from the user's own install is for authoring only, and a release never ships one verbatim (redistributing game data)."],
 });
 export const TEMPLATE_VALIDATORS = [vanillaExtractNeverShipsValidator];

package/dist/surfaces/cyberpunk2077/validators/tweaks.js

@@ -85,7 +85,11 @@ function finding(validatorId, severity, file, message, evidence, extra = {}) {
 const YAML_LINT_ID = "cp2077.tweaks.yaml-lint";
 const YAML_LINT_EVIDENCE = [
     "cp2077.tweakdb.melee.tweakxl-log-required-after-yaml-changes",
+    "cp2077.tweakxl.yaml-syntax.duplicate-keys-do-not-fail-the-parse",
 ];
+const DUPLICATE_KEY_NOTE = "That is invalid YAML. TweakXL reads the file without an error, but which occurrence takes effect is not established, " +
+    "so one of the two edits can be silently lost.";
+const DUPLICATE_KEY_FIX = "Write everything this key sets in one block, and delete the repeat.";
 export const yamlLintValidator = {
     id: YAML_LINT_ID,
     game: GAME,
@@ -113,9 +117,12 @@ export const yamlLintValidator = {
         const doc = YAML.parseDocument(text, { lineCounter });
         return doc.errors.map((err) => {
             const pos = err.pos ? lineCounter.linePos(err.pos[0]) : undefined;
-            return finding(YAML_LINT_ID, "block", file, err.message.split("\n")[0] ?? err.message, YAML_LINT_EVIDENCE, {
+            const first = err.message.split("\n")[0] ?? err.message;
+            const duplicate = /map keys must be unique/i.test(first);
+            return finding(YAML_LINT_ID, "block", file, duplicate ? `${first} ${DUPLICATE_KEY_NOTE}` : first, YAML_LINT_EVIDENCE, {
                 line: pos?.line,
                 column: pos?.col,
+                ...(duplicate ? { fix: DUPLICATE_KEY_FIX } : {}),
             });
         });
     },
@@ -164,7 +171,7 @@ export const baseExistsValidator = {
     },
 };
 const TYPE_KNOWN_ID = "cp2077.tweaks.type-known";
-const TYPE_KNOWN_EVIDENCE = ["ModWright research"];
+const TYPE_KNOWN_EVIDENCE = ["cp2077.tweakxl.yaml-syntax.record-names-must-be-unique", "https://github.com/psiberx/cp2077-tweak-xl/wiki/YAML-Tweaks"];
 export const typeKnownValidator = {
     id: TYPE_KNOWN_ID,
     game: GAME,
@@ -252,7 +259,7 @@ export const appendOnListFlatValidator = {
     },
 };
 const INSTANCES_ID = "cp2077.tweaks.instances-resolve";
-const INSTANCES_EVIDENCE = ["ModWright research"];
+const INSTANCES_EVIDENCE = ["https://github.com/psiberx/cp2077-tweak-xl/wiki/YAML-Tweaks"];
 const VAR_RE = /\$\(([^)]+)\)/g;
 function collectVarNamesFromText(text, out) {
     for (const m of text.matchAll(VAR_RE))
@@ -553,7 +560,7 @@ const REPLACER_BASE = "Character.Player_Replacer_Puppet_Base";
 const REPLACER_TEMPLATE_EVIDENCE = [
     "cp2077.player.control.replacers-are-other-player-bodies",
     "cp2077.player.control.android-replacer-is-v-in-the-android-app",
-    "cyberpunk2077.tweakxl.missing-resource (triage pattern: `<record>.entityTemplatePath refers to a non-existent resource.`)",
+    "observed 2026-10-04 in a TweakXL 1.11.4 log: `<record>.entityTemplatePath refers to a non-existent resource.` is logged as a warning naming the record",
 ];
 async function derivesFromReplacerBase(ctx, start, projectBases) {
     let current = start;

package/dist/surfaces/cyberpunk2077/writes.js

@@ -2,7 +2,7 @@ import { promises as fs } from "node:fs";
 import * as path from "node:path";
 import { TweakParseError } from "./index/model.js";
 import { parseTweakFile } from "./index/parsers/tweak.js";
-import { parseTweakXLYaml } from "./index/parsers/tweakxl-yaml.js";
+import { findRepeatedKeys, parseTweakXLYaml } from "./index/parsers/tweakxl-yaml.js";
 const TWEAK_EXTENSIONS = new Set([".yaml", ".yml", ".tweak"]);
 const MAX_TWEAK_BYTES = 8 * 1024 * 1024;
 export const TWEAKS_UNIT_NOTE = "r6/tweaks is one shared folder and TweakXL has no notion of mods, so each top-level folder there " +
@@ -38,7 +38,9 @@ async function parseFile(abs) {
     if (stat.size > MAX_TWEAK_BYTES)
         throw new Error(`larger than ${MAX_TWEAK_BYTES} bytes; skipped`);
     const text = await fs.readFile(abs, "utf8");
-    return path.extname(abs).toLowerCase() === ".tweak" ? parseTweakFile(text, abs) : parseTweakXLYaml(text, abs, { allowDuplicateKeys: true });
+    if (path.extname(abs).toLowerCase() === ".tweak")
+        return { records: parseTweakFile(text, abs), repeats: [] };
+    return { records: parseTweakXLYaml(text, abs, { allowDuplicateKeys: true }), repeats: findRepeatedKeys(text) };
 }
 export async function extractCyberpunkWrites(mod, root) {
     let files;
@@ -56,7 +58,10 @@ export async function extractCyberpunkWrites(mod, root) {
         const abs = path.join(mod.path, rel);
         const unit = root.id === "tweaks" ? rel.split("/")[0] : undefined;
         try {
-            extraction.writes.push(...recordWrites(await parseFile(abs), abs, unit));
+            const { records, repeats } = await parseFile(abs);
+            extraction.writes.push(...recordWrites(records, abs, unit));
+            for (const r of repeats)
+                (extraction.repeatedKeys ??= []).push({ file: abs, ...(unit ? { unit } : {}), key: r.key, lines: r.lines });
         }
         catch (error) {
             const message = error instanceof TweakParseError ? error.message : `${abs}: ${error.message}`;

package/dist/surfaces/skyrimse/validators/archives.js

@@ -1,6 +1,6 @@
 import * as path from "node:path";
 import { BSA_VERSION_OBLIVION, BSA_VERSION_SKYRIM_LE, BSA_VERSION_SKYRIM_SE, BsaHeaderError, readBsaHeader } from "../archives.js";
-const EVIDENCE = ["ModWright research"];
+const HEADER_EVIDENCE = ["sse.archives.bsa.version-105"];
 const APPLIES = "every .bsa file under a project source";
 function isWithin(parent, child) {
     const rel = path.relative(parent, child);
@@ -23,11 +23,12 @@ async function headerOrFinding(id, file) {
             file,
             message: `${path.basename(file)} has no readable BSA header (${reason}): ${err.message}`,
             fix: "Repack the archive with BSArch or the Creation Kit's archive tool.",
-            evidence: EVIDENCE,
+            evidence: HEADER_EVIDENCE,
         };
     }
 }
 const isFinding = (v) => v.validatorId !== undefined;
+const ARCHIVEVERSION_EVIDENCE = ["sse.archives.bsa.version-105", "sse.archives.bsa.v105-uses-lz4", "sse.archives.bsa.folder-record-is-24-bytes-in-v105"];
 const archiveVersion = {
     id: "sse.archives.version",
     game: "skyrimse",
@@ -39,7 +40,7 @@ const archiveVersion = {
     needs: { project: true },
     cost: "edit",
     severity: "block",
-    evidence: EVIDENCE,
+    evidence: ARCHIVEVERSION_EVIDENCE,
     async run(ctx) {
         if (ctx.target.kind !== "file")
             return [];
@@ -59,11 +60,12 @@ const archiveVersion = {
                     ? `${name} is a version ${header.version} archive, ${known}; Skyrim SE reads version 105.`
                     : `${name} has BSA version ${header.version}; Skyrim SE reads version 105.`,
                 fix: "Extract and repack the archive for Skyrim SE (BSArch with the SSE format, or the Creation Kit).",
-                evidence: EVIDENCE,
+                evidence: ARCHIVEVERSION_EVIDENCE,
             },
         ];
     },
 };
+const ARCHIVENAMEFLAGS_EVIDENCE = ["sse.archives.bsa.directory-and-file-name-flags-are-required"];
 const archiveNameFlags = {
     id: "sse.archives.name-flags",
     game: "skyrimse",
@@ -75,7 +77,7 @@ const archiveNameFlags = {
     needs: { project: true },
     cost: "edit",
     severity: "warn",
-    evidence: EVIDENCE,
+    evidence: ARCHIVENAMEFLAGS_EVIDENCE,
     async run(ctx) {
         if (ctx.target.kind !== "file")
             return [];
@@ -91,7 +93,7 @@ const archiveNameFlags = {
                 file: ctx.target.path,
                 message: `${path.basename(ctx.target.path)} lacks ${missing.join(" and ")} in its archive flags; UESP notes the game may not load a BSA without them, and every official archive sets both.`,
                 fix: "Repack with names included (BSArch's default).",
-                evidence: EVIDENCE,
+                evidence: ARCHIVENAMEFLAGS_EVIDENCE,
             });
         }
         if (header.xbox360) {
@@ -101,7 +103,7 @@ const archiveNameFlags = {
                 file: ctx.target.path,
                 message: `${path.basename(ctx.target.path)} has the Xbox 360 flag (0x40): big-endian numbers and hashes, which the PC game does not read.`,
                 fix: "Repack for PC.",
-                evidence: EVIDENCE,
+                evidence: ARCHIVENAMEFLAGS_EVIDENCE,
             });
         }
         return findings;