fix(build): sync Configuration and config properties, update output paths

The build system has two configuration properties that must stay in sync:
- Configuration: MSBuild property for managed projects (Debug, Release)
- config: Legacy property for native C++ makefiles

This commit:

1. Auto-derive config from Configuration when not explicitly set
   - Prevents accidental mismatches like /p:Configuration=Debug /p:config=release
   - Validates they match when both are specified, with clear error message

2. Update vcxproj files to use configuration-specific paths
   - Kernel.vcxproj: Output\Common → Output\$(Configuration)\Common
   - views.vcxproj: same pattern
   - TestViews.vcxproj: same pattern
   - Fixes IntelliSense paths when switching Debug/Release

3. Fix regen_midl.cmd
   - Remove hardcoded worktree path
   - Accept optional Configuration parameter (default: Debug)
   - Auto-detect VS 2022 Community or BuildTools installation

Author: johnml1135

Build/SetupInclude.targets

@@ -34,6 +34,33 @@
 		>
 		<fwrt Condition="'$(fwrt)' == ''">$(MSBuildThisFileDirectory)..</fwrt>
 	</PropertyGroup>
+	<!--
+		Configuration Synchronization:
+		There are two configuration properties that must stay in sync:
+		- Configuration: Standard MSBuild property used by .NET/managed projects (Debug, Release)
+		- config: Legacy property used by native C++ makefiles (debug, release, bounds, profile)
+
+		To prevent mismatches (e.g., Configuration=Debug with config=release), we:
+		1. Default config from Configuration if not explicitly set
+		2. Validate they match when both are specified
+	-->
+	<PropertyGroup>
+		<!-- If config is not set, derive it from Configuration (default to Debug if neither set) -->
+		<config Condition="'$(config)' == '' AND '$(Configuration)' != ''">$(Configuration)</config>
+		<config Condition="'$(config)' == ''">Debug</config>
+	</PropertyGroup>
+	<Target Name="ValidateConfigurationSync"
+		BeforeTargets="Setup"
+		Condition="'$(Configuration)' != '' AND '$(config)' != ''">
+		<!-- Normalize both to compare (handle case differences) -->
+		<PropertyGroup>
+			<_ConfigLower>$(Configuration.ToLowerInvariant())</_ConfigLower>
+			<_configLower>$(config.ToLowerInvariant())</_configLower>
+		</PropertyGroup>
+		<Error
+			Condition="'$(_ConfigLower)' != '$(_configLower)'"
+			Text="Configuration mismatch detected!%0A%0A  Configuration = $(Configuration)%0A  config = $(config)%0A%0AThese must match to ensure managed and native builds use the same output paths.%0A%0ASolutions:%0A  1. Use only /p:Configuration=Release (config will be derived automatically)%0A  2. Or explicitly set both to match: /p:Configuration=Release /p:config=Release" />
+	</Target>
 	<!-- config is generally either Debug or Release, but might be Bounds or Profile. -->
 	<!-- if the user gives it lowercase, fix it to be capitalized -->
 	<Choose>

Src/Kernel/Kernel.vcxproj

@@ -51,7 +51,7 @@
     <NMakeCleanCommandLine Condition="'$(Configuration)|$(Platform)'=='Debug|x64'" />
     <NMakeOutput Condition="'$(Configuration)|$(Platform)'=='Debug|x64'">FwKernel.dll</NMakeOutput>
     <NMakePreprocessorDefinitions Condition="'$(Configuration)|$(Platform)'=='Debug|x64'">x64;$(NMakePreprocessorDefinitions)</NMakePreprocessorDefinitions>
-    <NMakeIncludeSearchPath Condition="'$(Configuration)|$(Platform)'=='Debug|x64'">..\Generic;..\..\Output\Common;..\views\lib;..\Graphite\lib;..\Cellar;..\AppCore;..\..\Include;..\DebugProcs;..\..\Lib\src\graphite2\include;$(NMakeIncludeSearchPath)</NMakeIncludeSearchPath>
+    <NMakeIncludeSearchPath Condition="'$(Configuration)|$(Platform)'=='Debug|x64'">..\Generic;..\..\Output\$(Configuration)\Common;..\views\lib;..\Graphite\lib;..\Cellar;..\AppCore;..\..\Include;..\DebugProcs;..\..\Lib\src\graphite2\include;$(NMakeIncludeSearchPath)</NMakeIncludeSearchPath>
     <NMakeForcedIncludes Condition="'$(Configuration)|$(Platform)'=='Debug|x64'">$(NMakeForcedIncludes)</NMakeForcedIncludes>
     <NMakeAssemblySearchPath Condition="'$(Configuration)|$(Platform)'=='Debug|x64'">$(NMakeAssemblySearchPath)</NMakeAssemblySearchPath>
     <NMakeForcedUsingAssemblies Condition="'$(Configuration)|$(Platform)'=='Debug|x64'">$(NMakeForcedUsingAssemblies)</NMakeForcedUsingAssemblies>

Src/views/Test/TestViews.vcxproj

@@ -50,7 +50,7 @@
     <NMakeCleanCommandLine Condition="'$(Configuration)|$(Platform)'=='Debug|x64'">nmake /nologo BUILD_CONFIG=Debug BUILD_TYPE=d BUILD_ROOT=$(ProjectDir)..\..\..\  BUILD_ARCH=x64 /f testViews.mak clean</NMakeCleanCommandLine>
     <NMakeOutput Condition="'$(Configuration)|$(Platform)'=='Debug|x64'">$(ProjectDir)..\..\..\Output\Debug\TestViews.exe</NMakeOutput>
     <NMakePreprocessorDefinitions Condition="'$(Configuration)|$(Platform)'=='Debug|x64'">WIN64;$(NMakePreprocessorDefinitions)</NMakePreprocessorDefinitions>
-    <NMakeIncludeSearchPath Condition="'$(Configuration)|$(Platform)'=='Debug|x64'">$(ProjectDir)..\..\..\Src\views;$(ProjectDir)..\..\Views;$(ProjectDir)..\..\Views\Lib;$(ProjectDir)..\..\Kernel;$(ProjectDir)..\..\Generic;$(ProjectDir)..\..\..\Lib\src\unit++;$(ProjectDir)..\..\..\Output\Common;$(ProjectDir)..\..\..\Output\Common\Raw;$(NMakeIncludeSearchPath)</NMakeIncludeSearchPath>
+    <NMakeIncludeSearchPath Condition="'$(Configuration)|$(Platform)'=='Debug|x64'">$(ProjectDir)..\..\..\Src\views;$(ProjectDir)..\..\Views;$(ProjectDir)..\..\Views\Lib;$(ProjectDir)..\..\Kernel;$(ProjectDir)..\..\Generic;$(ProjectDir)..\..\..\Lib\src\unit++;$(ProjectDir)..\..\..\Output\$(Configuration)\Common;$(ProjectDir)..\..\..\Output\$(Configuration)\Common\Raw;$(NMakeIncludeSearchPath)</NMakeIncludeSearchPath>
     <NMakeForcedIncludes Condition="'$(Configuration)|$(Platform)'=='Debug|x64'">$(NMakeForcedIncludes)</NMakeForcedIncludes>
     <NMakeAssemblySearchPath Condition="'$(Configuration)|$(Platform)'=='Debug|x64'">$(NMakeAssemblySearchPath)</NMakeAssemblySearchPath>
     <NMakeForcedUsingAssemblies Condition="'$(Configuration)|$(Platform)'=='Debug|x64'">$(NMakeForcedUsingAssemblies)</NMakeForcedUsingAssemblies>

Src/views/views.vcxproj

@@ -84,7 +84,7 @@
     <NMakeCleanCommandLine Condition="'$(Configuration)|$(Platform)'=='Debug|x64'">..\..\Bin\mkvw.bat d e</NMakeCleanCommandLine>
     <NMakeOutput Condition="'$(Configuration)|$(Platform)'=='Debug|x64'">..\..\Output\Debug\views.dll</NMakeOutput>
     <NMakePreprocessorDefinitions Condition="'$(Configuration)|$(Platform)'=='Debug|x64'">x64;$(NMakePreprocessorDefinitions)</NMakePreprocessorDefinitions>
-    <NMakeIncludeSearchPath Condition="'$(Configuration)|$(Platform)'=='Debug|x64'">..\..\Output\Common\Raw;..\..\Output\Common;..\Kernel;..\Generic;.;$(NMakeIncludeSearchPath)</NMakeIncludeSearchPath>
+    <NMakeIncludeSearchPath Condition="'$(Configuration)|$(Platform)'=='Debug|x64'">..\..\Output\$(Configuration)\Common\Raw;..\..\Output\$(Configuration)\Common;..\Kernel;..\Generic;.;$(NMakeIncludeSearchPath)</NMakeIncludeSearchPath>
     <NMakeForcedIncludes Condition="'$(Configuration)|$(Platform)'=='Debug|x64'">$(NMakeForcedIncludes)</NMakeForcedIncludes>
     <NMakeAssemblySearchPath Condition="'$(Configuration)|$(Platform)'=='Debug|x64'">$(NMakeAssemblySearchPath)</NMakeAssemblySearchPath>
     <NMakeForcedUsingAssemblies Condition="'$(Configuration)|$(Platform)'=='Debug|x64'">$(NMakeForcedUsingAssemblies)</NMakeForcedUsingAssemblies>

regen_midl.cmd

@@ -1,6 +1,28 @@
 @echo off
-call C:\BuildTools\VC\Auxiliary\Build\vcvarsall.bat x64
-cd /d C:\fw-mounts\C\Users\johnm\Documents\repos\fw-worktrees\agent-1\Output\Common
-echo Running MIDL for x64...
+REM Usage: regen_midl.cmd [Configuration]
+REM   Configuration: Debug or Release (default: Debug)
+setlocal
+set CONFIG=%~1
+if "%CONFIG%"=="" set CONFIG=Debug
+
+REM Find vcvarsall.bat - try VS 2022 Community, then BuildTools
+if exist "C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Auxiliary\Build\vcvarsall.bat" (
+    call "C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Auxiliary\Build\vcvarsall.bat" x64
+) else if exist "C:\BuildTools\VC\Auxiliary\Build\vcvarsall.bat" (
+    call "C:\BuildTools\VC\Auxiliary\Build\vcvarsall.bat" x64
+) else (
+    echo ERROR: Cannot find vcvarsall.bat
+    exit /b 1
+)
+
+REM Navigate to configuration-specific output directory
+cd /d "%~dp0Output\%CONFIG%\Common"
+if errorlevel 1 (
+    echo ERROR: Directory Output\%CONFIG%\Common does not exist
+    exit /b 1
+)
+
+echo Running MIDL for x64 (%CONFIG% configuration)...
 midl /env x64 /Oicf /out Raw /dlldata FwKernelPs_d.c FwKernelPs.idl
 echo MIDL exit code: %ERRORLEVEL%
+endlocal