Closed
Conversation
- Remove UTF-8 BOM from install.ps1 to fix PowerShell execution error - Update logger to default to ~/.gaia/gaia.log instead of current directory - Add graceful PermissionError handling with helpful user messages - Prevents permission errors when running from protected directories like C:\windows\system32
kovtcharov-amd
approved these changes
Feb 12, 2026
6 tasks
27 tasks
kovtcharov
added a commit
that referenced
this pull request
Apr 9, 2026
Comprehensive plan for the GAIA Agent UI desktop installer — the primary install path for non-developer end users. Ground-up implementation using Lemonade Server's installer as a reference architecture (not vendored). Key decisions: - Distribute via GitHub Releases initially (R2 deferred) - NSIS on Windows, DMG on macOS, DEB + AppImage on Linux - Remove the existing Squirrel installer entirely (no migration — v0.17.1 users uninstall manually via release-notes step) - Reuse `bin/gaia-ui.mjs` install logic via a shared `services/backend-installer.cjs` module — both the npm CLI and the Electron app call it (eliminates the broken Squirrel UX) - Auto-update via electron-updater + GitHub provider - `gaia uninstall` CLI as the unifying piece for all four platforms - Drag-to-trash for macOS uninstall (DMG convention) - Documentation as a first-class deliverable (Phase I) - Code signing via SignPath OSS (Windows) + Apple Developer ID (macOS) Plan is structured as 9 phases (A–I) with explicit dependency ordering. Phase A is independently mergeable and ships first. Refs: #530 Related: #341 (folded into Phase B), #597 (left as polish) Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
kovtcharov
added a commit
that referenced
this pull request
Apr 9, 2026
The file began with a UTF-8 BOM (EF BB BF), which causes PowerShell to fail with: "The term '#' is not recognized as the name of a cmdlet". Removing the BOM so the script runs cleanly when piped through iex. Folds in the first of three fixes from the stale PR #341. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
kovtcharov
added a commit
that referenced
this pull request
Apr 9, 2026
…llback (#341) GaiaLogger used to default to a relative "gaia.log" in the current working directory, which had two problems: 1. It littered CWD with log files on every invocation. 2. It crashed with PermissionError when GAIA was invoked from a read-only or system directory (e.g. `cd / && gaia ...`), which broke the Windows PowerShell one-liner installer from PR #341 whenever it ran in restricted shells. Fix: - Default log_file is now ~/.gaia/gaia.log (created on demand). - File-handler creation is wrapped in try/except for both PermissionError and OSError. - On failure we try ~/.gaia/gaia.log, then tempfile.gettempdir() /gaia.log, then fall back to console-only logging with a friendly warning on stderr instead of crashing. Folds in the logger portion of the stale PR #341. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
kovtcharov
added a commit
that referenced
this pull request
Apr 9, 2026
4 tasks
itomek
approved these changes
Apr 15, 2026
Collaborator
Author
|
Closing as superseded — both fixes from this PR have already landed in `main`: 1. UTF-8 BOM removed from install.ps1 ✅ 2. Logger writes to user space with permission fallback ✅ This PR's branch is also significantly out of date (37+ commits behind, conflicts on the moved files). No need to rebase — work is done. |
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
3 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.
Summary
Changes
Install Script Fix
install.ps1that caused PowerShell execution errorThe term '#' is not recognizederror when running installerLogger Improvements
~/.gaia/gaia.logC:\windows\system32)Testing
These fixes improve the first-run experience and prevent common permission issues on Windows.