Skip to content

feat: wrap backstage-cli for auth and actions commands - #167

Merged
kadel merged 1 commit into
redhat-developer:mainfrom
yangcao77:rhdh-cli-wrapper
Aug 26, 2026
Merged

kadel merged 1 commit into
redhat-developer:mainfrom
yangcao77:rhdh-cli-wrapper

Conversation

@yangcao77

Copy link
Copy Markdown
Contributor

Expose auth, actions, and action-source commands on rhdh-cli by passing them through to the bundled backstage-cli binary.

Expose auth, actions, and action-source commands on rhdh-cli by
passing them through to the bundled backstage-cli binary.

Signed-off-by: Stephanie <yangcao@redhat.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
@sonarqubecloud

Copy link
Copy Markdown

@yangcao77

Copy link
Copy Markdown
Contributor Author

@kadel PTAL

@kadel

kadel commented Aug 26, 2026

Copy link
Copy Markdown
Member

Took a closer look at the rebranding in client.ts. There's a small bug where backstage-cli isn't rewritten to rhdh-cli when the name lands across a stream chunk boundary (e.g. one write ends with run backs and the next starts with tage-cli), so the branding occasionally leaks through. It's cosmetic and non-blocking, so no need to hold the PR for it. I've filed RHDHBUGS-3698 to track the fix plus a regression test that splits the name across chunks (the current tests emit output in a single event, so that path isn't exercised yet).

https://issues.redhat.com/browse/RHDHBUGS-3698

@kadel
kadel enabled auto-merge (squash) August 26, 2026 10:08
@kadel
kadel merged commit 0229ca3 into redhat-developer:main Aug 26, 2026
16 checks 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.

2 participants