Add native Windows installer and documentation - #678
Open
nsxdavid wants to merge 5 commits into
Open
Conversation
justrach
added a commit
that referenced
this pull request
Jul 30, 2026
…tinel fix Merges nsxdavid's install/install.ps1 (verbatim) plus its Windows docs, on top of 6387fe3 which already fixed #677 a different way. Conflict resolution in the three tracked copies of the bash installer (install/, website/app/, website/dist/ — kept byte-identical): - detect_platform keeps our "windows" sentinel. The PR's `{ ...; } >&2; return 10` variant works, but the sentinel is the narrower fix: the function still only ever reports a platform, and main() owns all user output and control flow. - main()'s windows branch keeps the sentinel check and adopts the PR's better guidance — the native PowerShell one-liner first, WSL2 demoted to the alternative. The one-liner points at raw.githubusercontent.com as the PR wrote it; codedb.codegraff.com has no /install.ps1 route. - All of #658 survives: the no-hooks marker, the hooks-registered receipt, pretooluse_entry_present(), the gated block-legacy script write, and the README hook opt-out paragraph. Docs take the PR's Windows coverage with the stale claims corrected: - npm: the launcher already resolves codedb-windows-x86_64.exe, but the published codedeebee predates that release asset, so `npx -y codedeebee mcp` is documented as arriving with the next published release rather than as working today. Dropped the named 0.2.5830 target. - Removed the pointer to issue #501 (closed). Verified: bash -n clean on all three copies; stubbed MINGW uname + a curl that prints NETWORK-TOUCHED and exits 7 — all three exit 0, print the ps1 guidance, and never reach curl; darwin-arm64 still detects normally; zig build test green. pwsh unavailable locally, so install.ps1 rides on the PR's recorded 5.1 + 7.5 parse checks. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01TD448QW4vLZ2UqPic6gR5a
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #677
Summary
Why
codedb already publishes
codedb-windows-x86_64.exe, but its primary installation surfaces do not provide a consistent native Windows installation path.If npm is the maintainer's preferred long-term distribution and update mechanism for Windows, this PowerShell installer is intended as an interim bootstrap path—not a replacement for proper npm publishing. It makes the already-published Windows binary easy to install safely now, without committing the project to PowerShell as its permanent Windows packaging strategy.
No MCP runtime behavior is changed.
Before
Under simulated MINGW Windows detection, the existing Bash installer:
Result:
The reproduction replaced
unamewith MINGW/x86_64 results and replacedcurlwith a network-blocking stub.After
The same reproduction against this branch:
Validation
Native Windows validation:
5.1.26100.8737: installer parse passed7.5.8: installer parse passed0.2.5830codedb-windows-x86_64.exematched its published SHA256 checksumcodedb 0.2.5830codedb.exeremained running passedirm ... | iexpipeline shape was validated against the checked-out installercodedeebee0.2.5830 installed and launched the Windows binaryNeighboring checks:
install/install.shwebsite/app/install_script.shwebsite/dist/install.shgit diff --checkpassedScope and repository state
release/0.2.5829b6b4f7fTracked website deployment outputs changed:
website/dist/install.shwebsite/dist/privacy.htmlwebsite/dist/quickstart.htmlThese are updated alongside their corresponding website sources. No
.zig-cache,zig-out, benchmark logs, or native build outputs are included.CI note
The fork's
bench-regressionworkflow currently fails before building repository code because its pinned Zig archive URL returns HTTP 404:The PR base guard accepts
release/0.2.5829, and the branch is mergeable.Contributing checklist
CONTRIBUTING.md; the release target follows the repository's active PR base guard