Added skill and skill installation guide#18
Conversation
|
Warning Rate limit exceeded
To keep reviews running without waiting, you can enable usage-based add-on for your organization. This allows additional reviews beyond the hourly cap. Account admins can enable it under billing. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughAdds comprehensive documentation for the Bifrost Rune Workflow skill, including a planning-and-execution guide with CLI commands, core concepts, routing, and operational constraints. Updates README to introduce the skill and link to the full documentation. ChangesBifrost Rune Workflow Documentation
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Tip 💬 Introducing Slack Agent: The best way for teams to turn conversations into code.Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.
Built for teams:
One agent for your entire SDLC. Right inside Slack. 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.
Actionable comments posted: 4
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@README.md`:
- Line 186: The installation command uses an invalid GitHub URL (npx skills add
https://github.com/devzeebo/bifrost/skills/bifrost-rune-workflow.md) which 404s;
update the URL to a valid location for the skill file (for example the
repository's raw file URL or the correct blob path) so the npx skills add
command can fetch it. Locate the README line containing the literal "npx skills
add" and "bifrost-rune-workflow.md" and replace the broken GitHub URL with the
correct raw.githubusercontent.com or https://github.com/.../blob/... URL
pointing to the actual file in the repo (confirm the branch/name), or
alternatively change the instruction to reference the published skill package
name if applicable.
- Around line 195-199: The glossary table in README has malformed Markdown using
double pipes at row starts (e.g., the header and rows containing **Rune**,
**Saga**, **Realm**); fix it by replacing the leading "||" with single "|" for
the header, separator, and each content row and ensure the separator row uses
the correct pipe/dash pattern (e.g., "|---|---|") so the table renders correctly
in Markdown.
In `@skills/bifrost-rune-workflow.md`:
- Around line 30-34: The fenced code blocks containing the rune state diagram
("draft → [forge] → open → [claim] → claimed → [fulfill] → fulfilled ... [seal]
→ sealed") and the CLI examples beginning with "bf create <title> ..." should
include a language identifier (use "text") after the opening triple backticks;
update both code fences (the diagram block and the CLI block) to ```text so
markdownlint passes and readability improves.
- Line 335: Replace the incorrect status name `in_progress` with the documented
lifecycle status `claimed` in the sentence "Only one rune can be in_progress at
a time" (change it to "Only one rune can be claimed at a time"), and scan the
surrounding workflow text for any other occurrences of `in_progress` to ensure
all references use `claimed` consistently (look for the exact token
`in_progress` and update it to `claimed` where it refers to rune lifecycle
state).
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository UI
Review profile: CHILL
Plan: Pro
Run ID: 83dc46c0-89f1-4311-a369-d1c8bca68ae8
📒 Files selected for processing (2)
README.mdskills/bifrost-rune-workflow.md
Update installation command to use raw GitHub URL
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
No description provided.