Skip to content

fix(cli): use correct dashboard url#142

Open
Vunovati wants to merge 1 commit into
mainfrom
fix/dashboard-url
Open

fix(cli): use correct dashboard url#142
Vunovati wants to merge 1 commit into
mainfrom
fix/dashboard-url

Conversation

@Vunovati
Copy link
Copy Markdown
Collaborator

@Vunovati Vunovati commented May 18, 2026

Summary by CodeRabbit

  • New Features

    • Dashboard create command now returns a resolved url field in the response, computed from configuration settings, CLI flags, or defaults.
  • Documentation

    • Updated skill documentation to display the dashboard URL directly from the response rather than constructing it from separate fields.

Review Change Stack

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 18, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 8e3f2ae8-a160-4ded-8a51-5dcbaf39b7c3

📥 Commits

Reviewing files that changed from the base of the PR and between 3a9729e and b786320.

📒 Files selected for processing (8)
  • .changeset/dashboard-url-in-response.md
  • packages/cli/AGENTS.md
  • packages/cli/src/commands/dashboards.test.ts
  • packages/cli/src/commands/dashboards.ts
  • skills/create-dashboard/SKILL.md
  • skills/create-dashboard/rules/workflow.md
  • skills/create-dashboard/tile.json
  • skills/root-cause-analysis/rules/workflow-identify-cause.md

📝 Walkthrough

Walkthrough

The PR adds URL resolution to the dashboards create CLI command, computing a fully resolved metrics dashboard URL from configuration and CLI flags, and returning it in the command response. Consuming skills are updated to display this returned URL instead of constructing it themselves. Versions are incremented and documentation is updated accordingly.

Changes

Dashboard URL Resolution & Skill Integration

Layer / File(s) Summary
CLI Dashboard URL Resolution Implementation and Testing
packages/cli/src/commands/dashboards.ts, packages/cli/src/commands/dashboards.test.ts
The dashboards create command extracts baseUrl from connection options and includes a computed metrics URL in the response. Comprehensive test suite validates URL construction scenarios: using configured .kopairc URL, overriding with --url flag, falling back to http://localhost:8000, stripping /signals suffix, and URL-encoding dashboard IDs.
Skill Workflow and Metadata Updates
skills/create-dashboard/rules/workflow.md, skills/create-dashboard/SKILL.md, skills/create-dashboard/tile.json, skills/root-cause-analysis/rules/workflow-identify-cause.md
The create-dashboard skill's Post-Creation section now displays the resolved url directly from the CLI response instead of constructing it. Skill version bumped to 1.2.0. The root-cause-analysis skill's dashboard link creation is similarly updated to consume the returned url field.
Documentation and Release Notes
.changeset/dashboard-url-in-response.md, packages/cli/AGENTS.md
Changeset documents the new url field returned by dashboards create and instructs callers to display it directly. Agent guide updated to reflect that dashboard command responses include the fully resolved url.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

Poem

A CLI once asked, "Where's my dashboard URL?"
It spun out a thread and replied, "I'll do—
Just check my response, no need to compute,
The URL's right there, resolved absolute!" 🐰✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'fix(cli): use correct dashboard url' clearly and concisely describes the main change: fixing the dashboard URL generation in the CLI command.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/dashboard-url

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 and usage tips.

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.

1 participant