Skip to content

Add sbx db sidekiq#57

Merged
naterod-reverb merged 4 commits into
mainfrom
naterod/sidekiq-ui
Mar 31, 2026
Merged

Add sbx db sidekiq#57
naterod-reverb merged 4 commits into
mainfrom
naterod/sidekiq-ui

Conversation

@naterod-reverb

@naterod-reverb naterod-reverb commented Mar 26, 2026

Copy link
Copy Markdown
Contributor

Adding sbx db sidekiq and sbx db s alias for easier sidekiq UI access in sandboxes.

Also included a small edit to the README to include instructions on making a temp local build for testing on orchestra

@wenn

wenn commented Mar 27, 2026

Copy link
Copy Markdown
Contributor

@copilot fix the vet failure

Copilot AI commented Mar 27, 2026

Copy link
Copy Markdown
Contributor

@copilot fix the vet failure

Fixed in bc44805. The TestPrint/it_prints_the_summary test in summary/summary_test.go was missing the Sidekiq: sbx db s | sbx db sidekiq line in the expected output string.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds a new sbx db subcommand for opening the Sidekiq UI, and surfaces it in the sandbox summary output.

Changes:

  • Added sidekiq (s) subcommand to sbx db command routing and help text.
  • Added a new sidekiq package with an Open() implementation that opens the Sidekiq URL in a browser.
  • Updated the sbx info/summary output to list the new Sidekiq shortcut.

Reviewed changes

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

File Description
summary/summary.go Adds Sidekiq entry to the displayed sandbox command summary.
db/db.go Wires sidekiq/s into sbx db subcommands and help text.
sidekiq/sidekiq.go Implements Sidekiq URL construction and browser open behavior.
sidekiq/sidekiq_test.go Adds tests verifying URL construction and error propagation from the opener.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread sidekiq/sidekiq.go
Copilot finished work on behalf of wenn March 27, 2026 15:47
Copilot AI requested a review from wenn March 27, 2026 15:47
@naterod-reverb naterod-reverb merged commit cf76351 into main Mar 31, 2026
1 check passed
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.

4 participants