Windows launch helpers for Skill Recorder - #67
Open
AIxingbuxing wants to merge 2 commits into
Open
Conversation
…d WINDOWS.md Provides one-click Windows setup for Skill Recorder from source: - start.bat: launch dev mode with Node 24 check + ELECTRON_MIRROR - stop.bat: kill only this project's Electron processes (by command line) - copilot-login.bat: reliable browser OAuth sign-in for Copilot - push-pr.bat: proxy-aware push with hard timeout (fixes silent hang) - WINDOWS.md: Windows-specific install/launch/login notes Also documents the .bat no-BOM/ASCII encoding requirement that otherwise causes cmd.exe to flash-and-close on Chinese Windows (GBK code page).
|
AIxingbuxing please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
Contributor License AgreementContribution License AgreementThis Contribution License Agreement (“Agreement”) is agreed to by the party signing below (“You”),
|
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.
Windows launch helpers for Skill Recorder
This adds a small set of Windows-only convenience scripts and a setup guide so contributors can build and run Skill Recorder from source on Windows with minimal friction.
start.bat — One-click dev launcher. Verifies Node.js ≥ 24 (a hard requirement for this project), sets ELECTRON_MIRROR so the reviewed Electron binary downloads from a reachable mirror, then runs npm run dev. Ends with pause so any startup error is actually visible instead of flashing by.
stop.bat — Cleanly stops only Skill Recorder's own Electron process. It matches processes by their command line (skill-recorder) and explicitly excludes cmd.exe, powershell.exe, and conhost.exe, so it will never accidentally kill VS Code, Discord, or other Electron-based apps running on the same machine.
copilot-login.bat — Runs the GitHub Copilot CLI login from a plain cmd.exe. Copilot authenticates through a browser OAuth flow (the authorization happens in your browser; no code is ever shown in the terminal), which is the reliable way to sign in on Windows.
WINDOWS.md — A Windows-specific install/launch/login guide. It documents the required reviewed-Electron install step (npm run electron:install-reviewed), the Copilot web-login flow, and—importantly—the rule that these .bat files must be pure ASCII with no UTF-8 BOM. On Chinese Windows (default GBK code page), a BOM makes cmd.exe misread the first line echo (@echo) off, causing the script to flash open and immediately close. Keeping the scripts ASCII-only with no BOM avoids that entirely.