Skip to content

Conversation

@zanesq
Copy link
Collaborator

@zanesq zanesq commented Aug 29, 2025

Pull Request Description

remove localstorage getconfig fallbacks since they aren't needed

  • Configuration is now passed entirely through process.argv in webPreferences.additionalArguments
  • No need for localStorage fallback - the main process provides complete config

Verified things are loading locally and in the bundle.

@zanesq
Copy link
Collaborator Author

zanesq commented Aug 29, 2025

.bundle

@github-actions
Copy link
Contributor

macOS ARM64 Desktop App (Apple Silicon)

📱 Download macOS Desktop App (arm64, unsigned)

Instructions:
After downloading, unzip the file and drag the Goose.app to a location you prefer. The app is unsigned, so to run it run xattr -r -d com.apple.quarantine '/path/to/Goose.app' and then open the app

@zanesq
Copy link
Collaborator Author

zanesq commented Sep 2, 2025

Verified working in bundle

@zanesq zanesq marked this pull request as draft September 4, 2025 15:17
@zanesq
Copy link
Collaborator Author

zanesq commented Sep 4, 2025

This doesn't fix the refresh issue but I still think we should remove these getConfig fallbacks anyway. App is working as expected otherwise.

@zanesq zanesq marked this pull request as ready for review September 4, 2025 15:36
@zanesq zanesq requested a review from jamadeo September 4, 2025 15:53
@zanesq zanesq merged commit 429aeff into main Sep 4, 2025
11 checks passed
@zanesq zanesq deleted the zane/remove-localstorage-config branch September 4, 2025 16:01
michaelneale added a commit that referenced this pull request Sep 5, 2025
* main:
  feat: make tests for rmcp based developer server consistent with former implementation (#4519)
  worlds simplest logging to see where things are blocked (#3888)
  docs: update quickstart and install topics (#4378)
  feat: Add configurable Bedrock retry parameters (#4316)
  remove localstorage getconfig fallbacks (#4432)
  chore(deps-dev): bump electron from 37.2.6 to 37.4.0 in /ui/desktop (#4516)
  Fix databricks streaming errors  (#4506)
  docs: malware check for uvx and npx extensions (#4508)
katzdave added a commit that referenced this pull request Sep 8, 2025
* 'main' of github.com:block/goose:
  docs: add ampersand to link (#4560)
  Add video link to README for user guidance (#4553)
  docs: social channels (#4552)
  feat: simplify navigation, make reload work (#4498)
  docs: new recipe warning (#4545)
  Add AGENTS.md for AI coding assistant support (#4539)
  docs: non-interactive compact now (#4543)
  fixed css classes and added some accessibility fixes (#4492)
  feat(acp): Read files (#4531)
  Add YouTube Short to Auto Visualiser Tutorial (#4536)
  Fix/settings page (#4520)
  update to RMCP 0.6.2 (#4523)
  docs: nested goosehints (#4528)
  feat: Agent Client Protocol implementation of goose (#4511)
  feat: make tests for rmcp based developer server consistent with former implementation (#4519)
  worlds simplest logging to see where things are blocked (#3888)
  docs: update quickstart and install topics (#4378)
  feat: Add configurable Bedrock retry parameters (#4316)
  remove localstorage getconfig fallbacks (#4432)
  chore(deps-dev): bump electron from 37.2.6 to 37.4.0 in /ui/desktop (#4516)
This was referenced Sep 9, 2025
thebristolsound pushed a commit to thebristolsound/goose that referenced this pull request Sep 11, 2025
Signed-off-by: Matt Donovan <mattddonovan@protonmail.com>
HikaruEgashira pushed a commit to HikaruEgashira/goose that referenced this pull request Oct 3, 2025
Signed-off-by: HikaruEgashira <hikaru-egashira@c-fo.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants