Skip to content

Conversation

@DOsinga
Copy link
Collaborator

@DOsinga DOsinga commented Jan 9, 2026

Summary

Add the option to the diagnostics module to file a bug immediately including the version, extensions & model/provider

@DOsinga DOsinga requested review from alexhancock and Copilot and removed request for Copilot January 9, 2026 13:12
Copilot AI review requested due to automatic review settings January 9, 2026 13:32
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds a "File Bug on GitHub" feature to the diagnostics modal, allowing users to directly create GitHub issues with pre-filled system information including version, extensions, and model/provider details.

Key changes:

  • Created a new /system_info API endpoint that returns structured system information
  • Enhanced the diagnostics modal with a second button that opens GitHub's issue creation form with pre-populated data
  • Refactored file structure from DownloadDiagnostics.tsx to Diagnostics.tsx to better reflect the expanded functionality

Reviewed changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
ui/desktop/src/components/ui/Diagnostics.tsx New component replacing DownloadDiagnostics.tsx, adds GitHub issue filing functionality alongside existing diagnostics download
ui/desktop/src/components/ui/DownloadDiagnostics.tsx Removed file, functionality moved to Diagnostics.tsx
ui/desktop/src/components/ChatInput.tsx Updated import path to reference new Diagnostics component
ui/desktop/src/api/types.gen.ts Added SystemInfo type and SystemInfoData/SystemInfoResponses for new endpoint
ui/desktop/src/api/sdk.gen.ts Added systemInfo API function and updated imports
ui/desktop/openapi.json Added /system_info endpoint definition and SystemInfo schema
crates/goose/src/session/mod.rs Exported new get_system_info function and SystemInfo type
crates/goose/src/session/diagnostics.rs Implemented SystemInfo struct with collect() and to_text() methods, refactored generate_diagnostics to use SystemInfo
crates/goose-server/src/routes/status.rs Added system_info route handler
crates/goose-server/src/openapi.rs Registered SystemInfo schema and system_info path in OpenAPI spec

Copilot AI review requested due to automatic review settings January 9, 2026 13:36
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 10 out of 10 changed files in this pull request and generated no new comments.

@DOsinga DOsinga merged commit 5a460fd into main Jan 12, 2026
26 checks passed
@DOsinga DOsinga deleted the file-bug-directly branch January 12, 2026 17:13
fbalicchia pushed a commit to fbalicchia/goose that referenced this pull request Jan 13, 2026
Co-authored-by: Douwe Osinga <[email protected]>
Co-authored-by: Copilot <[email protected]>
zanesq added a commit that referenced this pull request Jan 13, 2026
…ased

* 'main' of github.com:block/goose: (23 commits)
  Use Intl.NumberFormat for token formatting in SessionsInsights (#6466)
  feat(ui): format large and small token counts for readability (#6449)
  fix: apply subrecipes when using slash commands (#6460)
  Fix: exclude platform_schedule_tool in CLI (#6442)
  Fix: Small update in how ML-based prompt injection determines final result (#6439)
  docs: remove SSE transport and rename to Streamable HTTP (#6319)
  fix: correct Cloudinary extension command and env variable (#6453)
  fix: add gap between buttons in MacDesktopInstallButtons.js (#6452)
  refactor: include hidden dotfiles folders in file picker search (#6315)
  upgraded safe npm packages (#6450)
  chore(deps): bump react-router and react-router-dom in /ui/desktop (#6408)
  chore(deps): bump lru from 0.12.5 to 0.16.3 (#6379)
  chore(deps-dev): bump @modelcontextprotocol/sdk from 1.24.0 to 1.25.2 in /ui/desktop (#6375)
  fix: inconsistent API url requirement between desktop and CLI versions (#6419)
  feat(vertexai): Add streaming support (#6409)
  fix deeplink recipe launch cold start (#6210)
  Spell check setting (#6446)
  File bug directly (#6413)
  fix(cli): incorrect bin name in shell completions (#6444)
  Use crunchy from crates instead of git fork (#6415)
  ...
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