Build: stabilize container/native pipeline and test stack

Stage native builds to C:\Temp and enforce NativeBuild → traversal order
Hyper-V isolation + named/global NuGet caches to fix MoveFile and path errors
Isolate container Obj/OutDir/IntDir, suppress MSB3026 copy noise, guard config mismatches
Bump Palaso/SIL deps to 17.0.0-beta0082; restore legacy/localization assets when needed
Fix native corruption and COM activation (Views.dll/TestViews.exe, VwSelection)
Replace crashy StackWalk64 with CaptureStackBackTrace + SEH guards
ScriptureProvider injection and Utilities ref removal; ParatextHelper test cleanup
Migrate RhinoMocks/NMock → Moq; add helper scripts and warning suppressions
Repair C# test projects (missing refs, duplicate attributes, ignored flaky cases)
Add parse_msbuild_warnings/remove_duplicate_assemblyinfo and native Invoke-CppTest switch
Improve native test runner staging and DirectoryFinder fallback
MCP server for PowerShell wrappers; steer Copilot to approved scripts
Spec-kit agents + AgentConfiguration module for worktrees/containers
Streamline VS Code tasks and auto-approval; kill omnisharp/dotnet/fieldworks on cleanup
File-lock retries, process killing, and more aggressive cleanup to speed builds
Defender/antivirus exclusion scripts and documentation updates
NuGet vulnerability upgrades and invalid lib cleanup
Regen_midl/Post-Install fixes and container path hygiene
Exclude SIL.LCModel.Tests run; settings tidy
Build speed-up touch-ups and git hash cleaning fix

No more container build
* Remove docker container worktree logic (will use VSCode agent worktrees)
* Remove spinup, teardown
* Added tracing and logging for debug version of FieldWorks

Fix FieldWorks startup crashes and RegFree COM registration

Summary:
Resolved the "Class not registered" (0x80040154) startup crash by correcting the Registration-Free COM configuration and fixing initialization order.

Details:
- Build: Added `<NoWin32Manifest>true` to FieldWorks.csproj. This prevents the .NET SDK from embedding a default manifest in Debug builds, allowing the external `FieldWorks.exe.manifest` (which contains the required COM entries) to be used by the Windows Loader.
- COM: Explicitly registered managed COM assemblies (LexTextDll, xWorks, FwUtils, etc.) in `BuildInclude.targets` so they are included in the generated application manifest.
- COM: Decorated `LexTextApp` with `[ComVisible(true)]` and a stable GUID to ensure it can be instantiated via COM.
- Stability: Moved ICU initialization earlier in the startup sequence (`FieldWorks.cs`) to prevent "ICU not initialized" errors.
- Dev Env: Improved `FwDirectoryFinder` to reliably locate `DistFiles` relative to the executable in developer builds.
- Resources: Restored missing `ErrorReporter.resx` (renamed from ErrorReport.resx) to ensure crash reporting functions correctly.
- Config: Added a default layout for `CmObject` in `Cellar.fwlayout` to prevent UI crashes on unknown object types.
- Deps: Updated `SharpZipLib` and `System.Net.Fix FieldWorks startup crashes and RegFree COM registration

Summary:
Resolved the "Class not registered" (0x80040154) startup crash by correcting the Registration-Free COM configuration and fixing initialization order.

Details:
- Build: Added `<NoWin32Manifest>true` to FieldWorks.csproj. This prevents the .NET SDK from embedding a default manifest in Debug builds, allowing the external `FieldWorks.exe.manifest` (which contains the required COM entries) to be used by the Windows Loader.
- COM: Explicitly registered managed COM assemblies (LexTextDll, xWorks, FwUtils, etc.) in `BuildInclude.targets` so they are included in the generated application manifest.
- COM: Decorated `LexTextApp` with `[ComVisible(true)]` and a stable GUID to ensure it can be instantiated via COM.
- Stability: Moved ICU initialization earlier in the startup sequence (`FieldWorks.cs`) to prevent "ICU not initialized" errors.
- Dev Env: Improved `FwDirectoryFinder` to reliably locate `DistFiles` relative to the executable in developer builds.
- Resources: Restored missing `ErrorReporter.resx` (renamed from ErrorReport.resx) to ensure crash reporting functions correctly.
- Config: Added a default layout for `CmObject` in `Cellar.fwlayout` to prevent UI crashes on unknown object types.
- Deps: Updated `SharpZipLib` and `System.Net.

Fix build issue

Initial spec

Ready to implement

Get rid of old docker stuff

Worktree colorization and container documentation removal

feat: Migrate installer to WiX v6 and .NET 4.8

Migrates the FieldWorks installer infrastructure from WiX 3.11 to WiX v6,
moving from a legacy "PatchableInstaller" structure to a standard SDK-style
"FLExInstaller" layout.

Key Changes:
- Initialize `FLExInstaller` directory structure and port shared assets.
- Convert `CustomActions` and `ProcRunner` to SDK-style projects targeting .NET 4.8.
- Update Custom Actions code to use `WixToolset.Dtf` (WiX v6) namespaces.
- Create `FieldWorks.Installer.wixproj` and `FieldWorks.Bundle.wixproj` using WiX v6 SDK.
- Rewrite `Framework.wxs` and `Bundle.wxs` to use WiX v6 `<Package>` and `<Bundle>` syntax.
- Update XML namespaces in all `.wxs` and `.wxl` files to `http://wixtoolset.org/schemas/v4/wxs`.
- Replace legacy `heat.exe` command-line harvesting with MSBuild `<Harvestfeat: Migrate installer to WiX v6 and .NET 4.8

Migrates the FieldWorks installer infrastructure from WiX 3.11 to WiX v6,
moving from a legacy "PatchableInstaller" structure to a standard SDK-style
"FLExInstaller" layout.

Key Changes:
- Initialize `FLExInstaller` directory structure and port shared assets.
- Convert `CustomActions` and `ProcRunner` to SDK-style projects targeting .NET 4.8.
- Update Custom Actions code to use `WixToolset.Dtf` (WiX v6) namespaces.
- Create `FieldWorks.Installer.wixproj` and `FieldWorks.Bundle.wixproj` using WiX v6 SDK.
- Rewrite `Framework.wxs` and `Bundle.wxs` to use WiX v6 `<Package>` and `<Bundle>` syntax.
- Update XML namespaces in all `.wxs` and `.wxl` files to `http://wixtoolset.org/schemas/v4/wxs`.
- Replace legacy `heat.exe` command-line harvesting with MSBuild `<Harvest

Finally - unit tests are running in VSCode!

Add custom agents

Fix native tests:

The native test failures in `VwTextSelection.OnProblemDeletion` and `VwTextSelection.StTextEditing` were caused by `CreateInstance(CLSID_ActionHandler)` failing. This is likely because `ActionHandler` is not registered as a COM server in the test environment, or the test environment (TestViews) links directly to the object files but doesn't register the classes.

The fix was to use `ActionHandler::CreateCom` to instantiate the object directly, bypassing COM registration. This matches the pattern used for `VwRootBox` in other tests.

Changes made:
1.  Modified `Src/views/Test/TestVwSelection.h`:
    *   Added `#include "ActionHandler.h"` to access the static `CreateCom` method.
    *   In `testOnProblemDeletion`, replaced `qah.CreateInstance(CLSID_ActionHandler)` with `ActionHandler::CreateCom(NULL, __uuidof(IActionHandler), (void **)&qah)`.
    *   In `testStTextEditing`, replaced `qah.CreateInstance(CLSID_ActionHandler)` with `ActionHandler::CreateCom(NULL, __uuidof(IActionHandler), (void **)&qah)`.

Update worktree colorizations

Remove extra files

Streamline copilot files

remove fw.code-workspace for clarity

Fix colorization

Close existing VSCode instance when spawning new

Update colorization script

Remove Docker and container references from FieldWorks build system

This commit removes all Docker and container-related artifacts and references from the FieldWorks codebase. The changes include:

- Deleted Docker-specific files: Dockerfile.windows, .dockerignore, and Docker build workflow
- Removed Docker-related scripts and documentation including Build/Agent/Docker/ folder
- Eliminated container-specific build logic and environment variables
- Removed references to DOTNET_RUNNING_IN_CONTAINER, FW_CONTAINER, and related conditional code
- Cleaned up Vagrant references while preserving the vagrant/ directory structure
- Updated build scripts and configuration files to remove container-specific paths and logic
- Removed agent-container related references from documentation and specs
- Updated project documentation to reflect the removal of container builds

This cleanup ensures that the build system only uses native Windows build processes and removes all container-specific dependencies and configurations that were never fully implemented or used in production.

Fix installer bundle resources, vcredist staging, and UI dialog flow

Update the installer to fix Burn bundle localization and payload binding, adjust UI dialog flow and feature definitions, and ensure deterministic VC++ redistributable staging so the bundle compiles reliably.
- Correct Bundle theme/logo/license/icon and reference the MSI output path; remove duplicate localization inputs to avoid WIX localization conflicts.
- Add deterministic names and explicit SourceDir SourceFile paths for VC++ redistributables and a staging/Download target so payloads exist at WiX bind time.
- Tidy dialog flow and framework changes and adjust CustomFeatures.wxi to match updated install feature behavior.
- Verified by running .[build.ps1](http://_vscodecontentref_/0) -BuildInstaller -Configuration Release; installer build completes successfully.

Update worktree and colorization handling

More refinement

Merge settings

context7 updates

Test update

Fix colorization script - settings generation issue

feat: drop test-time dependency on registry-backed SilEncConverters40 EncConverters

- Replace direct static look‑ups of SilEncConverters40.EncConverters in unit tests
  with Moq fakes (IEncConverter/IEncConverters) and lazy‑init seams.
- Guard EncConverters creation in SCTextEnum so it only occurs when a
  LegacyMapping is actually required, eliminating eager registry access.
- Update affected Paratext import and Interlinear SFM import tests to use the
  new fakes and verify behavior without relying on global process/registry state.

This removes non‑deterministic HKLM\Software\... registry dependencies from the
test suite, making CI runs repeatable and allowing the code to be built/run in
environments where those keys are absent.

VSTest migration + managed test stabilization

Switch managed test execution to VSTest with shared runsettings and NUnit adapter
Fix/modernize test infrastructure to be worktree/CI-friendly (build/test scripts, output TRX/logs)
Stabilize brittle tests across modules (xWorks/DetailControls/LexText/Utilities) and correct assertion direction issues
Address dependency/binding/runtime issues found during migration (collator sort-key, XSL URI resolution, safer finalization, lazy init)
Update test modernization specs/checklists to reflect current green state (4048 total, 0 failed)

Installer(wix6): complete WiX 6 build pipeline + parity/evidence tooling
Resolve Heat from repo/NuGet, normalize paths, and generate harvest .wxs before compile
Stabilize bundle StdBA payloads (theme/localization/logo/license) and align branding/version outputs
Improve MSI upgrade/ARP behavior and fix bundle UI/title + MSI-cancel handling
Add installer evidence capture + compare scripts, plus VS Code tasks for end-to-end installer checks
Add deterministic Hyper-V parity lane (WiX3 vs WiX6) with checkpoint restore, runs, and evidence publishing
Add installer artifact validation tests; set C# 8 defaults; update docs/specs and fix startup crash ordering

All tests pass
Full ledger of all test changes made to pass
No new tests ignored

add to gitignore

Clean up build after rebase

Adding some more agents and prompts

Add WiX 3 legacy target wiring and guardrails in Installer.Wix3.targets and Installer.legacy.targets.
Route installer orchestration and toolset selection updates in Installer.targets and Orchestrator.proj.
Make localization downloads safer by default in Localize.targets.
Split WiX 6 assets and projects into wix6 while keeping WiX 3 in FLExInstaller.
Update installer WiX authoring inputs in CustomComponents.wxi and related .wxi files.
Refresh guidance in ReadMe.md and installer-build-guide.md.
Update migration status and specs under spec.md and related files.
CI/instructions updates in CI.yml and installer.instructions.md.
Version/build metadata adjustments in build.ps1, CommonAssemblyInfoTemplate.cs, and bldinc.h.

Resolved all three warning classes and verified with a Release x64 installer build.
* Conditioned transform DLL references to avoid MSB3246 when the compiled transform assemblies are missing: ParserCore.csproj, ParserUI.csproj, ParserUITests.csproj.
* Added the missing LexText dependency so RegFree sees the DLL: FieldWorks.csproj.
* Suppressed known merge‑module collisions for LGHT1055/1056 in the WiX 3 MSI linker: buildMsi.bat.

fix: Correct clrClass placement in SxS manifests and enhance installer

Fix Windows SxS "side-by-side configuration is incorrect" error by
placing clrClass elements as direct children of assembly, not nested
under file elements. Also sanitize version strings with non-numeric
suffixes (e.g. 9.3.5.local_20260119) in manifest generation.

- RegFreeCreator: Pass parent (assembly) instead of fileNode to
  AddOrReplaceClrClass, fixing manifest structure per Windows SxS spec
- RegFree: Always sanitize version to 4 numeric parts for SxS compliance
- Revert LaunchUnicodeCharEditor to Return="check" (both WiX 3 and 6)
  so installer fails on manifest errors before release
- Add -SignInstaller switch for local signing control
- Add Invoke-InstallerWithLog.ps1 helper for installer testing
- Update unit tests to verify correct clrClass placement

Move analysis files under spec and revise ignored tests analysis

Points to review in the test update analysis

Author: johnml1135

.GitHub/AI_GOVERNANCE.md

@@ -0,0 +1,66 @@
+# Copilot and AI guidance governance
+
+## Purpose
+This repo uses a **Copilot-first** documentation strategy:
+- Component knowledge lives with the component (`Src/**/COPILOT.md`).
+- A small set of scoped instruction files in `.github/instructions/` provides **prescriptive, enforceable constraints**.
+- `.github/copilot-instructions.md` is the short “front door” that links to the right places.
+- Agent definitions in `.github/agents/` and role chatmodes in `.github/chatmodes/` describe **behavior/persona**, not system architecture.
+
+## Source of truth
+- **Component architecture & entry points**: `Src/<Component>/COPILOT.md`
+- **Repo-wide workflow** (how to build/test, safety constraints): `.github/copilot-instructions.md`
+- **Non-negotiable rules** (security, terminal restrictions, installer rules, etc.): `.github/instructions/*.instructions.md`
+
+## No duplication rule
+- Do not copy component descriptions into `.github/instructions/`.
+- Do not restate rules in multiple places. Prefer linking.
+- If a rule must be enforced by Copilot for a subtree, add a scoped `.instructions.md`; otherwise document it in the relevant `COPILOT.md`.
+
+## What goes where
+
+### `.github/copilot-instructions.md`
+Use for:
+- One-page onboarding for Copilot: build/test commands, repo constraints, and links.
+- Pointers to the curated instruction set and the component docs.
+
+### `.github/instructions/*.instructions.md`
+Use for:
+- Prescriptive constraints that must be applied during editing/review.
+- Cross-cutting rules that prevent expensive mistakes (security, terminal command restrictions, installer rules, managed/native boundary rules).
+
+**Curated keep set (intentionally small):**
+- `build.instructions.md`
+- `debugging.instructions.md`
+- `installer.instructions.md`
+- `managed.instructions.md`
+- `native.instructions.md`
+- `powershell.instructions.md`
+- `repo.instructions.md`
+- `security.instructions.md`
+- `terminal.instructions.md`
+- `testing.instructions.md`
+
+### `Src/**/COPILOT.md`
+Use for:
+- Where to start (entry points, key projects, typical workflows).
+- Dependencies and cross-component links.
+- Tests (where they live, how to run them).
+
+Baseline expectations for a component COPILOT doc:
+- **Where to start** (projects, primary entry points)
+- **Dependencies** (other components/layers)
+- **Tests** (test projects and the recommended `./test.ps1` invocation)
+
+### `.github/agents/` and `.github/chatmodes/`
+Use for:
+- Role definitions, boundaries, and tool preferences.
+- Do not put component architecture here; link to the component `COPILOT.md`.
+
+## Adding a new scoped instruction file
+Add a new `.github/instructions/<name>.instructions.md` only when:
+- The guidance is prescriptive (MUST/DO NOT), and
+- It applies broadly or to a subtree, and
+- It would be harmful if Copilot ignored it.
+
+Otherwise, update the appropriate `Src/**/COPILOT.md`.