Skip to content

docs(plugins): drop unreachable repo-root LICENSE pointer from 13 READMEs (wave 1) - #755

Merged
kyle-sexton merged 1 commit into
mainfrom
docs/537-license-pointer-wave1
Jul 20, 2026
Merged

docs(plugins): drop unreachable repo-root LICENSE pointer from 13 READMEs (wave 1)#755
kyle-sexton merged 1 commit into
mainfrom
docs/537-license-pointer-wave1

Conversation

@kyle-sexton

@kyle-sexton kyle-sexton commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

No linked issue — wave 1 of 3; #537 stays open until all 39 plugins are done.

Deletes the dangling "See the LICENSE file at the root of the repository" sentence from the first 13 plugin READMEs (alphabetical), keeping the inline MIT (SPDX-License-Identifier: MIT) statement — an installed consumer runs from the isolated plugin cache and cannot reach a marketplace-root file. Patch bump + CHANGELOG entry per plugin, mirroring the merged #426 two-plugin fix.

Verification: repro grep (at the root of|root of (the )?melodic) returns 0 hits over the 13 wave-1 READMEs, 26 hits over the untouched remainder; all 13 manifest versions match their CHANGELOG top entry.

Related

🤖 Generated with Claude Code

…DMEs (wave 1)

Installed consumers run each plugin from an isolated cache and cannot
reach a file at the marketplace-repo root, so the "See the LICENSE file
at the root" sentence dangles. Keep the inline MIT/SPDX statement; patch
bump + CHANGELOG entry per plugin, mirroring the merged two-plugin fix.

Wave 1 of 3 (alphabetical): actionlint, architecture, bash-format,
biome-format, bug-report, claude-config, claude-memory, claude-ops,
code-tidying, codebase-health, context7, debugging, desktop-notification.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@claude

claude Bot commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Claude finished @kyle-sexton's task in 1m 0s —— View job


Security Review

  • Gather PR diff and changed files
  • Review README.md changes for security issues
  • Review plugin.json version bumps for supply-chain risk
  • Review CHANGELOG.md entries
  • Post findings

No security issues found.

This PR is a documentation-only change across 13 plugins, touching three file types per plugin:

README.md — Each file removes a two-line License section pointer to a LICENSE file at the repository root that is unreachable when a plugin runs from the isolated consumer cache. The replacement is the inline SPDX identifier MIT (SPDX-License-Identifier: MIT). No executable content, no URLs to external resources, no user-controlled input.

plugin.json — Each manifest receives a patch version bump (e.g., 0.4.1 → 0.4.2). The only field changed is version; no dependency pins, MCP server endpoints, tool definitions, or permission entries are added or modified. A version bump does prompt consumers to pull an update, but the update itself carries only the documentation changes above — there is no supply-chain risk.

CHANGELOG.md — Plain Markdown entries describing the documentation change. No executable content, no external URLs.

No GitHub Actions workflows are touched, so there are no pull_request_target / workflow_run misuse concerns, no script injection surfaces, and no permission-widening changes. No credentials or tokens appear anywhere in the diff.

@claude

claude Bot commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Claude finished @kyle-sexton's task in 1m 32s —— View job


PR Review

  • Gather context and read CLAUDE.md
  • Review the PR diff
  • Spot-check README changes across several plugins
  • Verify CHANGELOG format and version bumps
  • Check plugin.json version increments
  • Post review findings

This is a clean, well-executed mechanical cleanup. Here's what I verified:

What the PR does: Removes the unreachable See the LICENSE file at the root of the melodic-software/claude-code-plugins repository. line from the License section of 13 plugin READMEs. The motivation is correct — plugins installed to the isolated plugin cache at ${CLAUDE_PLUGIN_DATA} cannot reach the repo root, so the pointer is a dead reference for installed consumers.

Version bumps — all correct. Every plugin gets a patch increment (docs-only → patch per semver), and every bump is correctly applied:

Plugin Before After
actionlint 0.4.1 0.4.2
architecture 0.3.3 0.3.4
bash-format 0.5.1 0.5.2
biome-format 0.4.1 0.4.2
bug-report 0.5.0 0.5.1
claude-config 0.7.0 0.7.1
claude-memory 0.2.1 0.2.2
claude-ops 0.15.1 0.15.2
code-tidying 0.6.0 0.6.1
codebase-health 0.6.1 0.6.2
context7 0.4.0 0.4.1
debugging 0.4.0 0.4.1
desktop-notification 0.4.1 0.4.2

CHANGELOG entries — consistent and accurate. The entry wording correctly describes this as documentation-only with no behavior change. The date-free version header format (e.g., ## [0.4.2] without - YYYY-MM-DD) is pre-existing across all plugins and not introduced here.

After-state consistency. The before-state had minor inconsistencies (some used backtick-quoted `LICENSE`, others didn't; one also backtick-quoted the repo name). After this PR, all 13 end uniformly with MIT (SPDX-License-Identifier: MIT). — clean.

REVIEW.md blocking checks — all N/A for a docs-only diff; no secrets, no handlers, no data access, no tenant data. Nothing blocking.

One observation (not a blocker): 26 plugins on main still carry the old pointer (grep confirms a wave 2+ will be needed). That's consistent with the "wave 1" naming and no action needed here.

Verdict: ✅ Looks good to merge. No issues found.

kyle-sexton added a commit that referenced this pull request Jul 20, 2026
…DMEs (wave 2) (#758)

No linked issue — wave 2 of 3; #537 stays open until all 39 plugins are
done.

Same recipe as wave-1 PR #755 for the next 13 plugins alphabetically;
guardrails is deferred to wave 3 because two in-flight guardrails
version bumps (#757 lane) serialize ahead of it. Per-plugin patch bump +
CHANGELOG entry; discovery and docs-hygiene entries carry dates per
those plugins' own changelog convention.

Verification: repro grep returns 0 hits over the 13 wave-2 READMEs; all
13 manifest versions match their CHANGELOG top entry; base predates #755
so wave-1 plugins intentionally still carry the pointer here (no file
overlap between waves).

## Related

- #537 (wave 2 of 3)
- #755 (wave 1)
- #426 (the two-plugin fix this mirrors)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
@kyle-sexton
kyle-sexton merged commit 9a17c91 into main Jul 20, 2026
22 of 23 checks passed
@kyle-sexton
kyle-sexton deleted the docs/537-license-pointer-wave1 branch July 20, 2026 20:42
kyle-sexton added a commit that referenced this pull request Jul 21, 2026
…13 READMEs (wave 3) (#771)

Closes #537 — final wave; repo-wide repro grep (`at the root of|root of
(the )?melodic`) returns zero across all `plugins/*/README.md` on this
branch.

Supersedes #766 (identical change, rebased): #737 landed toolchain 0.5.0
/ verification 0.3.0 mid-grace, so those two stack as 0.5.1 / 0.3.1;
guardrails (0.9.2→0.9.3) and work-items (0.17.1→0.17.2) stack above
their own mid-flight entries as before. All 13 manifests match their
CHANGELOG top entry; songwriting's methodology-attribution note
preserved.

## Related

- #537 (wave 3 of 3 — closes)
- #766 (superseded — pre-rebase head of this same change)
- #755 / #758 (waves 1–2)
- #426 (the two-plugin fix this mirrors)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

---------

Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
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