Skip to content

Show hosted usage dashboard by default - #165

Closed
lawrencecchen wants to merge 3 commits into
mainfrom
fix/gcp-dashboard-add
Closed

Show hosted usage dashboard by default#165
lawrencecchen wants to merge 3 commits into
mainfrom
fix/gcp-dashboard-add

Conversation

@lawrencecchen

@lawrencecchen lawrencecchen commented Aug 5, 2026 •
edited by cubic-dev-ai Bot
Loading

Copy link
Copy Markdown
Contributor

Summary

  • route bare hosted sr and sr status through the full usage dashboard
  • keep sr-gcp account-add instructions after the dashboard

Testing

  • go test ./cmd/subrouter -count=1
  • go test -p 1 ./... -count=1
  • public staging dashboard smoke against https://staging.sr.cmux.com

Deployment

Issues

View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

Summary by cubic

When using hosted accounts, sr and sr status now open the full usage dashboard by default. sr-gcp account-add instructions are shown after the dashboard.

Written for commit 9fbba90. Summary will update on new commits.

Review in cubic

@coderabbitai

coderabbitai Bot commented Aug 5, 2026 •
edited
Loading

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 7 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used all 2 included reviews currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: a1418485-2873-48f0-af66-f25fc4c08da0

📥 Commits

Reviewing files that changed from the base of the PR and between 40f29cc and 9fbba90.

📒 Files selected for processing (2)
  • cmd/subrouter/sr.go
  • cmd/subrouter/sr_hosted_login_test.go

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@lawrencecchen lawrencecchen mentioned this pull request Aug 5, 2026
Merged
@teamleaderleo
Merge remote-tracking branch 'origin/main' into fix/gcp-dashboard-add
9fbba90 
# Conflicts:
#	cmd/subrouter/sr.go
#	cmd/subrouter/sr_hosted_login_test.go
@github-actions

github-actions Bot commented Sep 26, 2026

Copy link
Copy Markdown


Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.

I have read the CLA Document v2.2 and I hereby sign the CLA

1 out of 2 committers have signed the CLA.
✅ lawrencecchen
❌ teamleaderleo
You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

@teamleaderleo teamleaderleo mentioned this pull request Sep 26, 2026
Merged
@teamleaderleo

teamleaderleo commented Sep 26, 2026

Copy link
Copy Markdown
Contributor

Thank you @lawrencecchen. This is carried in #397 on current main, with you as co-author. It also prints how to add an account when a hosted tenant has none. It had to be a new PR: this repo's CLA check requires the PR opener to author every commit, so the merge-main commit I pushed here turned CLA red. Please disregard that. I've left this PR open for you or a maintainer to close once #397 lands.

teamleaderleo added a commit that referenced this pull request Sep 26, 2026
@teamleaderleo
Show the hosted usage dashboard for bare sr
4e5b51f 
Bare sr on a hosted cmux credential source fell through to the local
account store: it listed local accounts, offered a local switch, and
could auto-import ~/.codex auth. Route it, and sr status, to the hosted
usage dashboard as team mode already does, and tell a hosted user with no
accounts how to add one.

Carries #165 by Lawrence Chen.
teamleaderleo added a commit that referenced this pull request Sep 28, 2026
@teamleaderleo
Show the hosted usage dashboard for bare sr (#397)
f385677 
* Show the hosted usage dashboard for bare sr

Bare sr on a hosted cmux credential source fell through to the local
account store: it listed local accounts, offered a local switch, and
could auto-import ~/.codex auth. Route it, and sr status, to the hosted
usage dashboard as team mode already does, and tell a hosted user with no
accounts how to add one.

Carries #165 by Lawrence Chen.

* Keep sr status honoring an explicit local server override; isolate hosted default tests from server env

* Keep bare sr and argless sr switch local under SUBROUTER_SERVER=local on hosted storage
@teamleaderleo

teamleaderleo commented Sep 28, 2026

Copy link
Copy Markdown
Contributor

Thanks @lawrencecchen. This landed in #397, which carries your change (hosted usage dashboard for bare sr) plus a fix so SUBROUTER_SERVER=local still wins, and credits you in the description.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@lawrencecchen @teamleaderleo