Remove personal hostname reference from install-workflow doc#31
Conversation
|
Warning Rate limit exceeded
You’ve run out of usage credits. Purchase more in the billing tab. ⌛ 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: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (3)
📝 WalkthroughWalkthroughA patch version is released for the indigo plugin, incrementing from 1.8.0 to 1.8.1 across both plugin manifest files. Documentation is updated to clarify cross-machine mount handling behavior when the skill runs on a different Mac than the Indigo server. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ 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 |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In @.claude-plugin/marketplace.json:
- Line 11: Update the "version" value in the .claude-plugin marketplace metadata
to exactly match the new patch version from the plugin manifest (the "version"
field in plugin.json); find the "version" key in marketplace.json (currently
"1.8.1") and change it to the same patch bump (e.g., "1.8.2") so the CI
version-sync rule passes.
In @.claude-plugin/plugin.json:
- Line 3: Update the "version" field from "1.8.1" to the next patch (e.g.,
"1.8.2") in the plugin manifest and apply the identical change to the
marketplace manifest so both "version" keys match; ensure you follow semantic
versioning and bump the patch portion in both plugin.json and marketplace.json
before pushing.
🪄 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: defaults
Review profile: CHILL
Plan: Pro
Run ID: a7e35ca9-ee38-40a8-a08c-e76c16b65c9d
📒 Files selected for processing (3)
.claude-plugin/marketplace.json.claude-plugin/plugin.jsonskills/update-plugins/references/install-workflow.md
| { | ||
| "name": "indigo", | ||
| "version": "1.8.0", | ||
| "version": "1.8.1", |
There was a problem hiding this comment.
Version bump is effectively a no-op against main and fails CI
"1.8.1" appears to match origin/main, so the PR does not satisfy the required per-PR version bump. Please increment to the next patch (e.g., 1.8.2) and keep both manifest files in sync.
Suggested fix
--- a/.claude-plugin/plugin.json
+++ b/.claude-plugin/plugin.json
@@
- "version": "1.8.1",
+ "version": "1.8.2",--- a/.claude-plugin/marketplace.json
+++ b/.claude-plugin/marketplace.json
@@
- "version": "1.8.1",
+ "version": "1.8.2",As per coding guidelines, “Bump version in both .claude-plugin/plugin.json and .claude-plugin/marketplace.json with every PR and ensure they match using semantic versioning (patch for fixes/tweaks, minor for new features).”
📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| "version": "1.8.1", | |
| "version": "1.8.2", |
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In @.claude-plugin/plugin.json at line 3, Update the "version" field from
"1.8.1" to the next patch (e.g., "1.8.2") in the plugin manifest and apply the
identical change to the marketplace manifest so both "version" keys match;
ensure you follow semantic versioning and bump the patch portion in both
plugin.json and marketplace.json before pushing.
jarvis.local was the author's own Indigo server hostname used as a parenthetical example in the cross-machine mount-handling section. Replaced with generic phrasing — the example adds no value and unnecessarily personalises shipped docs. All other workspace-specific strings (/Volumes/Macintosh HD-1, /Volumes/Macintosh HD) are standard macOS mount names and stay in place — they're the probe prefixes the skill uses before asking the user, not identifying info. Bumps plugin version 1.8.0 → 1.8.1 (patch: doc cleanup, no user-visible behaviour change). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
simons-plugins
force-pushed
the
chore/remove-jarvis-ref
branch
from
7624d66 to
5f29667
Compare
1 participant
Summary
Removes the
jarvis.localreference inskills/update-plugins/references/install-workflow.md— the author's own Indigo server hostname used only as a parenthetical example in the cross-machine mount-handling section. Replaced with generic phrasing.Other workspace-specific strings (
/Volumes/Macintosh HD-1,/Volumes/Macintosh HD) stay — they're standard macOS mount names used as probe prefixes before the skill asks the user, not identifying info.Version
Test plan
grep -ri jarvisacross the repo returns zero hits🤖 Generated with Claude Code
Summary by CodeRabbit
Chores
Documentation