Skip to content

chore(claude-settings): reduce the mirrored enabledPlugins block to deltas - #570

Merged
kyle-sexton merged 1 commit into
mainfrom
chore/enabled-plugins-deltas
Sep 7, 2026
Merged

chore(claude-settings): reduce the mirrored enabledPlugins block to deltas#570
kyle-sexton merged 1 commit into
mainfrom
chore/enabled-plugins-deltas

Conversation

@kyle-sexton

Copy link
Copy Markdown
Contributor

No related issue: fleet fan-out of melodic-software/claude-code-plugins#3688 (closed); this repo's part of that remediation.

Summary

The fleet plugin list (standards components/cloud-environment/fleet-plugins.json, standards#542) is what every cloud snapshot installs, and the canonical bootstrap reads it overlaid with each repo's .claude/settings.json. A committed block that mirrors the whole catalog is therefore redundant in the cloud and, locally, writes one project-scope install record per entry into the user's installed_plugins.json on every session start in every checkout (claude-code-plugins#3813). This repo declares no per-repo deltas, so its block is reduced to {}.

Fix

  • .claude/settings.json: enabledPlugins reduced to {}. extraKnownMarketplaces and everything else in the file is unchanged.

A repo that later needs a plugin disabled adds a single "<plugin>@melodic-software": false entry; a repo that needs one outside the fleet list adds a true entry. Template and rationale: claude-code-plugins docs/CLOUD-FLEET-SETUP.md.

Verification

  • jq -e . .claude/settings.json parses; git diff touches only the enabledPlugins block.
  • The cloud install path is unaffected: the snapshot installs the fleet list at build time (verified on stamp 2026-09-06.2 for claude-code-plugins and medley), and the canonical bootstrap's drift repair reads the same list from /opt/melodic-fleet-plugins.json.

Related

🤖 Generated with Claude Code

https://claude.ai/code/session_016e6sBmnTGcidwvSrM86NYp

…eltas

The fleet plugin list (standards components/cloud-environment/fleet-plugins.json)
is what every cloud snapshot installs, and the canonical bootstrap reads it
overlaid with this repo's .claude/settings.json. A block that mirrors the whole
catalog is redundant in the cloud and, locally, writes one project-scope install
record per entry on every session start in every checkout. This repo declares
no deltas, so the block becomes {}.

Fleet fan-out of melodic-software/claude-code-plugins#3688.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_016e6sBmnTGcidwvSrM86NYp
@kyle-sexton
kyle-sexton marked this pull request as ready for review September 7, 2026 15:30
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 7, 2026

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review Completed 2026-09-07T15:32:48.556810Z 2591643 Draft marked ready
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@claude

claude Bot commented Sep 7, 2026

Copy link
Copy Markdown

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


Code review complete

Scope: .claude/settings.json, enabledPlugins reduced from the full mirrored plugin catalog to {} (+1/-72).

Findings: None. This is a minimal, low-risk data change:

  • The commit at 2591643 touches only the enabledPlugins block; extraKnownMarketplaces and hooks are byte-for-byte unchanged.
  • Verified the resulting file is valid JSON (jq -e .).
  • Traced the consumer, install_plugins_from in .claude/cloud-bootstrap.sh: jq -r '.enabledPlugins // {} | to_entries[] | select(.value == true) | .key' is a correct no-op against {}, and the catalog-inventory missing computation merges the fleet list with the repo's block, so an empty repo block degrades cleanly to "fleet list is authoritative" rather than erroring or silently dropping plugins.
  • No other file in the repo reads or enumerates enabledPlugins entries by name, so nothing else depends on the removed keys.
  • PR body already satisfies this repo's PR body contract (No related issue: ... line, all four required sections non-empty).

Nothing to flag — the change matches its stated intent and the described verification checks out.

@github-actions

github-actions Bot commented Sep 7, 2026

Copy link
Copy Markdown

Claude has reviewed this PR 1 time. The lane skips further automatic reviews after 5; deleting this comment resets the count.

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 2591643bc3

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .claude/settings.json
@kyle-sexton
kyle-sexton merged commit 6e63e21 into main Sep 7, 2026
32 of 34 checks passed
@kyle-sexton
kyle-sexton deleted the chore/enabled-plugins-deltas branch September 7, 2026 15:34
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