chore: add packageManager#2
Merged
DIYgod merged 1 commit intoRSSNext:mainfrom May 19, 2024
hyoban:chore/package-manager
Merged
Conversation
DIYgod
added a commit
that referenced
this pull request
Mar 8, 2026
- Fix $@ shell expansion in privileged install path by writing to a temp file first, then using admin cp (Comment #1) - Check .cmd extension on Windows for install status and uninstall guard so the CLI is correctly detected and removed (Comment #2) - Wire prepare:cli into build:electron-vite instead of build:electron so all CI packaging paths (vite + forge) include CLI (Comment #3) - Add CLI i18n strings for ja, fr-FR, zh-TW locales (Comment #4) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
DIYgod
added a commit
that referenced
this pull request
Mar 8, 2026
* feat(cli): deep integration with desktop client - Package CLI binary with desktop app via extraResource in Electron Forge - Sync desktop login session to ~/.folo/config.json automatically on login/logout - Add CliService IPC with install/uninstall methods supporting macOS (osascript), Linux (pkexec), Windows (.cmd wrapper) - Add CLI settings tab in desktop app for install status and management - Create prepare-cli build script to compile and bundle CLI before packaging - Add i18n strings for CLI UI (English and Chinese) The CLI now shares the desktop app's authentication session without requiring separate login. Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com> * fix(cli): fix CI failures in auth service and forge config - Pass `undefined` explicitly to syncSessionToCliConfig() to satisfy TS arity check - Make resources/cli conditional in extraResource so packaging doesn't fail when CLI bundle is absent Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix(cli): make syncSessionToCliConfig token param optional Avoids TS2554 when called without arguments on signOut, while also satisfying the lint rule that strips explicit `undefined` arguments. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix(cli): address PR review feedback - Fix $@ shell expansion in privileged install path by writing to a temp file first, then using admin cp (Comment #1) - Check .cmd extension on Windows for install status and uninstall guard so the CLI is correctly detected and removed (Comment #2) - Wire prepare:cli into build:electron-vite instead of build:electron so all CI packaging paths (vite + forge) include CLI (Comment #3) - Add CLI i18n strings for ja, fr-FR, zh-TW locales (Comment #4) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> --------- Co-authored-by: Claude Haiku 4.5 <noreply@anthropic.com>
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
2 participants
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.
No description provided.