chore: refine Rollbar triage skill - #17
Conversation
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (3)
📝 WalkthroughWalkthroughThe Rollbar CLI skill is rewritten as a read-only investigation playbook with authorised mutation rules, evidence and safety guidance, completion reporting, agent metadata, and a command reference covering inspection, deployment, user, mutation, and filtering operations. ChangesRollbar CLI playbook
Estimated code review effort: 2 (Simple) | ~10 minutes Poem
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Pull request overview
Refactors the rollbar-cli Copilot skill documentation from a command-heavy catalogue into an evidence-led production triage workflow, with a separate progressive command reference and added agent UI metadata.
Changes:
- Replaced the main skill doc with a structured investigation workflow, automation rules, and privacy/safety guidance.
- Added a dedicated command reference markdown for exact flags and mutation commands.
- Added
agents/openai.yamlmetadata for UI display and a default prompt.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| .ai/skills/rollbar-cli/SKILL.md | Rewrites the skill into a step-by-step triage workflow with explicit safety/authorization boundaries. |
| .ai/skills/rollbar-cli/references/command-reference.md | Introduces a focused command/flags reference to keep the main workflow concise. |
| .ai/skills/rollbar-cli/agents/openai.yaml | Adds agent UI metadata and a default prompt for the skill. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| - You need to inspect raw occurrences for a specific item or fetch one occurrence directly. | ||
| - You need to look up Rollbar account users before assigning an item. | ||
| Use `rollbar-cli` to move from an error signal to an evidence-backed classification or code fix. Keep diagnosis read-only | ||
| unless the user asks for a fix or explicitly authorises a Rollbar state change. |
|
|
||
| ## Privacy And Safety | ||
|
|
||
| - Redact access tokens, cookies, authorisation headers, session identifiers, and direct personal data. |
| rollbar-cli users list --fields id,username,email --no-headers | ||
| ``` | ||
|
|
||
| Mutate an item only with explicit user authorisation: |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 7cb61eea3a
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
|
||
| # All statuses in the same window | ||
| rollbar-cli items list \ | ||
| --status all \ |
There was a problem hiding this comment.
For the all-status baseline, this command passes all as a non-empty item status; cmd/items.go forwards --status verbatim to the Rollbar API, and Rollbar's List all items docs say valid status values are active, resolved, muted, and archived (multiple status filters are separate valid values, not all). Following this reference will either error or fail to retrieve the resolved/muted regressions the workflow is trying to catch, so the all-status example should omit --status or use valid status values.
Useful? React with 👍 / 👎.
Summary
Refactors the Rollbar skill from a long command catalogue into an evidence-led production triage workflow.
The revised workflow covers time-window baselines, active and all-status queries, impact ranking, deduplication, representative occurrence review, deploy correlation, root-cause classification, regression-tested code fixes, browser-noise guardrails, post-deploy verification, and explicit authorization for Rollbar mutations. Detailed commands move into a progressively loaded reference, and Codex UI metadata is added.
Validation
git diff --checkpassedSummary by CodeRabbit
Documentation
New Features