Skip to content

Conversation

@jonandersen
Copy link
Contributor

Improves recipe loading error handling by ensuring failed recipes don't crash the UI and display helpful error messages with filenames. Prior to this change an invalid goose recipe would crash the UI forcing the user to restart goose.

Before After
Screenshot 2025-08-26 at 7 55 40 PM Screenshot 2025-08-26 at 7 55 35 PM
Recipe errors crash the entire UI Graceful error display with filename

Changes:

  • Added filename property to SavedRecipe interface for better error reporting
  • Updated recipe storage functions to always inject filename information
  • Implemented try-catch error handling in RecipesView to gracefully display "Failed to read file: {filename}" for corrupted recipes
  • Keeps the main recipes list functional even when individual recipes fail to load

@jonandersen jonandersen force-pushed the jonandersen/recipe-filename-error-handling branch from b041865 to 61b0e81 Compare August 27, 2025 00:03
@michaelneale michaelneale requested a review from zanesq August 27, 2025 04:57
@zanesq zanesq merged commit e79c766 into block:main Aug 27, 2025
10 checks passed
zanesq added a commit that referenced this pull request Aug 27, 2025
…nscribing

* 'main' of github.com:block/goose:
  update tetrate blog date to today (#4368)
  tetrate signup: blog/launch post (#4313)
  Implement graceful recipe error handling with filename display (#4363)
  docs: airgapped operation by bypassing hermit for desktop app (#4063)
  remove Ollama card from welcome screen (#4348)
  feat: initial implementation of extension malware check (#4272)
  Add Tetrate Agent Router Service to Provider Registry (#4354)
  Goose Simple Compact UX (#4202)
  Refactor Extensions Install Modal (#4328)
  fix: url path trailing slash for custom-providers (#4345)
  docs: update available and onboarding providers list (#4356)
katzdave added a commit that referenced this pull request Aug 27, 2025
* 'main' of github.com:block/goose:
  Fix eleven labs audio transcription and added more logging (#4358)
  feat: re-introduce session sharing (#4370)
  remove duplicate blog post (#4369)
  fix focus ring under form submits (#4332)
  Trigger docs deployment
  update tetrate blog date to today (#4368)
  tetrate signup: blog/launch post (#4313)
  Implement graceful recipe error handling with filename display (#4363)
  docs: airgapped operation by bypassing hermit for desktop app (#4063)
  remove Ollama card from welcome screen (#4348)
  feat: initial implementation of extension malware check (#4272)
  Add Tetrate Agent Router Service to Provider Registry (#4354)
  Goose Simple Compact UX (#4202)
michaelneale added a commit that referenced this pull request Aug 28, 2025
* main: (38 commits)
  feat: linux computer control for android (termux) (#3890)
  feat: Added scroll state support for chat-session-list navigation (#4360)
  docs: typo fix (#4376)
  blog: goose janitor (#4131)
  Fix eleven labs audio transcription and added more logging (#4358)
  feat: re-introduce session sharing (#4370)
  remove duplicate blog post (#4369)
  fix focus ring under form submits (#4332)
  Trigger docs deployment
  update tetrate blog date to today (#4368)
  tetrate signup: blog/launch post (#4313)
  Implement graceful recipe error handling with filename display (#4363)
  docs: airgapped operation by bypassing hermit for desktop app (#4063)
  remove Ollama card from welcome screen (#4348)
  feat: initial implementation of extension malware check (#4272)
  Add Tetrate Agent Router Service to Provider Registry (#4354)
  Goose Simple Compact UX (#4202)
  Refactor Extensions Install Modal (#4328)
  fix: url path trailing slash for custom-providers (#4345)
  docs: update available and onboarding providers list (#4356)
  ...
dorien-koelemeijer pushed a commit to dorien-koelemeijer/goose that referenced this pull request Sep 2, 2025
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.

2 participants