Skip to content

fix: fill the plugin's homepage, docs, repository, support and terms links - #20

Merged
STRML merged 2 commits into
mainfrom
fix/plugin-links
Oct 1, 2026
Merged

STRML merged 2 commits into
mainfrom
fix/plugin-links

Conversation

@STRML

@STRML STRML commented Oct 1, 2026 •

Copy link
Copy Markdown
Contributor

The Claude directory listing step showed homepage, documentation, repository, support and terms of service as not set. Adds them to plugin.json. claude plugin validate passes.

Summary by CodeRabbit

  • Documentation
    • Plugin information now includes links to its homepage, documentation, source repository, support resources, and terms of service.
    • The privacy policy link remains unchanged. These links are available alongside the plugin’s other information.

…links

The Claude directory listing showed each as not set.
@coderabbitai

coderabbitai Bot commented Oct 1, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

Warning

Review limit reached

Your organization has reached its usage spending cap. Adjust your spending cap in the billing tab.

Next included review available in 24 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used all 2 included reviews currently available. Your 80 included PR review attempts over the past 7 days set your current allowance at 2 reviews per hour.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Team

Run ID: 7fbd40a0-e12c-4dd4-8ab8-1ebf218dae4e

📥 Commits

Reviewing files that changed from the base of the PR and between b6c1bfc and e7e714f.

📒 Files selected for processing (1)
  • plugin/README.md

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Team

Run ID: 02768662-cbdf-4cbb-8868-9e286089dafb

📥 Commits

Reviewing files that changed from the base of the PR and between 01142b8 and b6c1bfc.

📒 Files selected for processing (1)
  • plugin/.claude-plugin/plugin.json

Included review availability: This review used your included allowance. 0 included reviews remain after this review. Your included PR review attempts over the past 7 days set your current allowance at 3 reviews per hour.

📜 Recent review details
⏰ Context from checks skipped due to timeout. (4)
  • GitHub Check: server (maintenance)
  • GitHub Check: packages
  • GitHub Check: converter
  • GitHub Check: server (laps)
🔇 Additional comments (1)
plugin/.claude-plugin/plugin.json (1)

12-12: 🎯 Functional Correctness

The repository does not establish whether the Claude Code directory consumes documentationUrl, supportUrl, or termsOfServiceUrl. The comment’s contract concern remains undecided without the applicable directory manifest contract or implementation.


📝 Walkthrough

Walkthrough

The plugin manifest retains its privacy policy URL and adds homepage, documentation, repository, support, and terms-of-service URLs.

Changes

Plugin manifest metadata

Layer / File(s) Summary
Manifest URL metadata
plugin/.claude-plugin/plugin.json
The manifest adds homepage, documentation, repository, support, and terms-of-service URLs. The privacy policy URL remains unchanged.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Bug fix

Merge Risk: ⚪ Minimal · up to b6c1b

The change adds project and support links without an established functional regression. It is mergeable after normal checks; directory display remains unconfirmed.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the main change: adding the plugin homepage, documentation, repository, support, and terms links.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
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.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Autopilot is currently an internal CodeRabbit preview.


Comment @coderabbitai help to get the list of available commands.

@STRML
STRML merged commit 404be3c into main Oct 1, 2026
5 checks passed
@STRML
STRML deleted the fix/plugin-links branch October 1, 2026 19:52
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