Skip to content

Correct false "plugin manager detects updates" versioning rationale - #11

Merged
cchacs merged 1 commit into
mainfrom
claude/open-issues-todos-uj3tmu
Sep 8, 2026
Merged

cchacs merged 1 commit into
mainfrom
claude/open-issues-todos-uj3tmu

Conversation

@cchacs

@cchacs cchacs commented Sep 8, 2026

Copy link
Copy Markdown

Summary

feedBack core's own CLAUDE.md is explicit that the plugin loader does not consume a plugin's manifest version for update detection — it's only used as a cache-busting query param (?v=<version>) on served JS/CSS URLs. This repo's Versioning section repeated the inaccurate "the plugin manager uses this to detect updates" claim (duplicated verbatim across ~15 plugin repos per feedback core#70); replaced with the real rationale.

Doc-only change — no code touched.

🤖 Generated with Claude Code

https://claude.ai/code/session_016cfE1wg6F2VT6eqJEp4LBz


Generated by Claude Code

…nale

feedBack core's own CLAUDE.md is explicit that the plugin loader does
not consume a plugin's manifest version for update detection — it's
only used as a cache-busting query param on served JS/CSS URLs. This
repo's Versioning section repeated the inaccurate "the plugin manager
uses this to detect updates" claim; replaced with the real rationale
(feedback core issue #70).

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_016cfE1wg6F2VT6eqJEp4LBz
@coderabbitai

coderabbitai Bot commented Sep 8, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: c2714235-732d-4b41-8383-d45e04c442a8

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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.

@pullfrog pullfrog Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ No new issues found.

Reviewed changes — Reviewed the single doc-only commit in CLAUDE.md.

  • Correct the Versioning rationale — Replaced the inaccurate "the plugin manager uses this to detect updates" claim with the real reason (the plugin.json version is used for cache-busting the served JS/CSS URL). This matches feedBack core's documented behavior and corroborates the repo's own notes that the Host cache-busts assets by plugin version.

Doc-only change; no code touched and no plugin.json bump warranted.

Pullfrog  | View workflow run | Using Big Pickle (free) | 𝕏

@cchacs
cchacs merged commit d7e3dc0 into main Sep 8, 2026
9 checks passed
@cchacs
cchacs deleted the claude/open-issues-todos-uj3tmu branch September 8, 2026 12:00
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.

2 participants