Skip to content

Commit d15bd72

Browse files
committed
fix(install): allow cbm-integrations.json in the install.ps1 archive namespace
The prior fix put cbm-integrations.json into the Windows archive, and install.ps1 promptly rejected it: its pre-extraction namespace guard permits only the exact release entries in $WindowsArchiveNames and throws "archive contains an unexpected root entry" on anything else. The asset is now a real release member, so it belongs in that allow-list. Added in archive order (binary, cbm-integrations.json, LICENSE, install.ps1, THIRD_PARTY_NOTICES.md), matching package-release.sh and the windows-bundle contract's windows_archive_names. The guard also REQUIRES each listed entry exactly once, so this both permits and mandates the asset — an archive missing it now fails install.ps1 up front rather than at first hook render. install.sh has no equivalent namespace allow-list (it extracts the tarball directly), which is why the Unix smoke passed; no change needed there. install.ps1 stays pure ASCII (0 non-ASCII bytes); windows-bundle contract passes. This was the Windows smoke advancing PAST the archive-staging fix (asset now present) to the installer's own entry validation. Signed-off-by: Martin Vogel <martin.vogel.tech@gmail.com>
1 parent 1c16786 commit d15bd72

1 file changed

Lines changed: 1 addition & 0 deletions

File tree

‎install.ps1‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@ $InstallDir = "$env:LOCALAPPDATA\Programs\codebase-memory-mcp"
1616
$BinName = "codebase-memory-mcp.exe"
1717
$WindowsArchiveNames = @(
1818
$BinName,
19+
"cbm-integrations.json",
1920
"LICENSE",
2021
"install.ps1",
2122
"THIRD_PARTY_NOTICES.md"

0 commit comments

Comments
 (0)