Federate CLAUDE.md into per-project files; refresh README.md

[TheValiantOne]

Summary

• Splits the root CLAUDE.md (which had grown to cover all four projects — WitcherScriptMerger.Core, the WinForms host, the Linux-capable WitcherScriptMerger.Headless host, and WitcherScriptMerger.Tests — in one file) into a short root CLAUDE.md plus one CLAUDE.md per project, each scoped to what's needed to work in that folder:
  • WitcherScriptMerger.Core/CLAUDE.md — folder map, AppState/IMergeNotifier design, FileMerger's interactive/headless split, DiffPlexMergeEngine (including the confirmed upstream DiffPlex bug and its measured failure rates), text-merge encoding, Paths.ValidateTextMergeDependencies/ValidateBundleDependencies, hash format, and the shared CLI/MCP orchestration (Cli/MergeOperations.cs, Mcp/WsmMcpTools.cs).
  • WitcherScriptMerger/CLAUDE.md — the WinForms host's own folder map, MainForm's IMergeNotifier translation, MessageBoxManager's known-broken relabeling hook, InteractiveMergeRunner, Program.cs's CLI/GUI/MCP startup flow for this host, the TFM 7.0-suffix constraint, and its build/run/publish commands.
  • WitcherScriptMerger.Headless/CLAUDE.md — its own routing, what it deliberately omits (GUI, console-attach, KDiff3), the two real cross-platform path-separator bugs its own WSL2 verification found and fixed, and its publish commands.
  • WitcherScriptMerger.Tests/CLAUDE.md — what's covered, the AppState.Settings-safety constraints tests must respect (moved here from the root file), and the WSM_TEST_GAME_DIR-gated live-install cross-check tests.
• WitcherScriptMerger.Core/Mcp/CLAUDE.md (pre-existing, already correctly scoped) had two pointers into the old root file's now-deleted sections — retargeted to the new Core-level file, and extended with a Categories.BundleText internal-path allow-listing caveat the split would otherwise have silently dropped.
• CONTRIBUTING.md's CLAUDE.md cross-references were also retargeted, since federation broke several of them (e.g. "see CLAUDE.md's Compatibility constraints" — that section no longer exists at root). This file is outside this unit's stated scope, but its pointers break as a direct, mechanical consequence of this change, so fixing them is included here rather than left dangling.
• README.md rewritten from its original, largely unmodified upstream-era text (first-person "I threw together this tool" voice, no mention of the fork/CLI/MCP/DiffPlex/Headless host) to reflect the current state: this is a fork of AnotherSymbiote/WitcherScriptMerger, GUI + headless CLI + MCP + a Linux-capable host, corrected xxHash attribution (System.IO.Hashing, not the old hand-ported implementation), DiffPlex as an ordinary MIT-licensed NuGet dependency vs. QuickBMS/wcc_lite's unresolved-license external-binary status (needed only for bundle-content conflicts), and WSM's own GPLv2 license.

Every substantive fact from the original root file was traced to its new home rather than dropped — including the hash-format stability constraint, the DiffPlex bug's measured failure rates (0.35%–38.89% across edit densities, 100,000 trials), the text-encoding rationale, the historical gh pr create fork mistake, get_status's independently-reported textMergeDependenciesValid/bundleDependenciesValid fields, and the CustomLoadOrder.Refresh() null-ref fix IMergeNotifier provided.

AI assistance disclosure

This PR (the doc restructuring, content redistribution, and README rewrite) was substantially produced by Claude Code. Verified via:

• dotnet build WitcherScriptMerger.sln — succeeds.
• dotnet format whitespace WitcherScriptMerger.sln --verify-no-changes — passes (docs-only change; included per repo convention).
• Every see X/CLAUDE.md cross-link walked and confirmed to resolve to a real file and (where quoted) an actual heading.
• A fact inventory built from the original root CLAUDE.md, checked against each new file to confirm nothing was silently dropped.
• Ran this repo's code-review skill against the diff; it surfaced three real gaps (the WinForms host's publish command being dropped, get_status's exact return-shape not being documented anywhere, and the CustomLoadOrder null-ref history being lost) — all three are fixed in this PR.

Known follow-up, not fixed here: six .cs files in WitcherScriptMerger.Core (AppState.cs, DiffPlexMergeEngine.cs, FileEncoding.cs, WsmMcpTools.cs, MergeOperations.cs, FileMerger.cs) have inline comments referencing pre-federation root CLAUDE.md section names (e.g. "CLAUDE.md's Compatibility constraints"). All six still resolve to the right file (the nearest CLAUDE.md from Core is now Core's own), and most of the referenced content moved there under a renamed heading, but two comments are now imprecise. Left out of this docs-only PR as a separate, small follow-up rather than expanding this diff into source-code comment edits.

Test plan

• dotnet build WitcherScriptMerger.sln succeeds
• dotnet format whitespace WitcherScriptMerger.sln --verify-no-changes passes
• Every cross-link between the new/edited CLAUDE.md files resolves to a real file/heading
• Content diffed against the original root CLAUDE.md fact-by-fact; nothing substantive dropped
• code-review skill run against the diff; all three findings addressed

🤖 Generated with Claude Code

https://claude.ai/code/session_01GXAuGMLB44T5Zv5o5ZzKah