Repository navigation
Claude settings template: claude.ai skill sync and claude.ai MCP servers off (user-directed, measured unused) - #367
Merged
Conversation
…ervers The user asked on 2026-09-26 to "disable the unused claude.ai skills and the docs connector". The template now sets "syncClaudeAiSkills": false (Claude Code 2.1.275; per the skills docs, the client then moves skills it already synced to ~/.claude/skills/.trash/ and no longer loads them) and env ENABLE_CLAUDEAI_MCP_SERVERS "false" (2.1.63). tools/adoption/apply_claude_settings.py carries both into applied hosts. - docs/decisions/2026-09-25-skills-trial-and-usage.md: dated addendum with the user's sentence, the upstream sources (CHANGELOG pinned at anthropics/claude-code 7779afb12e36), measured use (the 2026-09-25 /skill-doctor baseline and a read-only transcript scout), the host read-back, the effect on selection rule 4, and the overturn. - adoption/bootstrap.md, adoption/platforms/macos-arm64.md: the template changed after `v2026.09.26.2`. - tests/test_render_config.py: pins both values in the rendered template. - manifests/evidence.json: re-registers the changed files. skillOverrides, adoption/skills/manifest.json and the Codex skill configuration are unchanged. The user withdrew the same-day request to turn off never-used skills in favour of improving their use. Co-Authored-By: Claude Opus 5.5 (1M context) <[email protected]>
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
seathatflowsinourveins
deleted the
claude/skills-context-trim-20260926
branch
September 26, 2026 21:19
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Scope
"syncClaudeAiSkills": false) and the claude.ai MCP servers (env.ENABLE_CLAUDEAI_MCP_SERVERS: "false"), as the user directed on 2026-09-26: "disable the unused claude.ai skills and the docs connector". A dated addendum records the decision, both new-host pages mark the template change, and one test pins both values.dde28cc2005bff54e4be61195e76d63b166db4f4lane:foundation(themanifests/evidence.jsonchange only re-registers hashes, which docs/lanes.md says does not make a PR shared)adoption/templates/claude.settings.template.json,adoption/bootstrap.md,adoption/platforms/macos-arm64.md,docs/decisions/2026-09-25-skills-trial-and-usage.md,tests/test_render_config.py,manifests/evidence.json(re-registration only)Not in this PR: every
skillOverridesstate,adoption/skills/manifest.jsonand the Codex skill configuration are unchanged. The user replaced the same-day request to turn off never-used user and plugin skills with "don't just mindlessly delete, but improve their usage". Role-based skill wiring comes in a separate change.SOTA sources
syncClaudeAiSkillstofalsein your user settings. Claude Code stops downloading, and the next time it starts it moves the skills it already synced to~/.claude/skills/.trash/and no longer loads them."anthropics/claude-codeCHANGELOG.md at7779afb12e3635f46f56ec823979d68350ae000b(2026-09-25T21:49:55Z; the copy read here has the same sha256, prefixae8f617c42850f48b143):syncClaudeAiSkills: falseorsyncClaudeAiPlugins: false"ENABLE_CLAUDEAI_MCP_SERVERS=falseenv var to opt out from making claude.ai MCP servers available"tools/adoption/apply_claude_settings.pydeep-merges the rendered template into user settings (template scalars win; keys are never deleted), perdocs/decisions/2026-09-25-skills-trial-and-usage.md"Mechanics"./skill-doctorrow in the same decision record (8 synced skills, none ever invoked).Evidence-class table
syncClaudeAiSkillsfalse(boolean) andENABLE_CLAUDEAI_MCP_SERVERS"false"(string)local_integrationtests/test_render_config.pytest_claude_template_turns_off_claudeai_skill_sync_and_mcp_servers; it fails on all five mutants tried (synctrue, sync removed, env booleanfalse, env"true", env removed)source_review/skill-doctorrun (Claude Code 2.1.283, 0 turns, $0) starts with 8 MCP servers, none from claude.ai, noclaude_aitool and noanthropic-skills:*skill; its table has noclaude.ai syncrownative_proven)mcp__claude_ai_Claude_Docs__*toolsmcp__claude_ai_*tool calls, 0 of 384Skillcalls naming ananthropic-skills:*or synced skillv2026.09.26.2local_integrationtests/test_adoption_docs_consistency.py(links, markers and release-pin quotes pass)Local commands run
All with the CI dependency venv (
ci-venv/bin/python), in the branch worktree:The full suite ran before one prose-only edit, which pinned the CHANGELOG commit in the addendum. After it, the decision record was re-registered, and
validate.py,evidence_manifest.py --check,build_ecosystem.py --checkand the six touched modules above were run again, all with rc=0.Decision record
docs/decisions/2026-09-25-skills-trial-and-usage.md, section "Addendum 2026-09-26: claude.ai skill sync and MCP servers off" (#addendum-2026-09-26-claudeai-skill-sync-and-mcp-servers-off): the user's sentence, the upstream sources, the measured use, the host read-back, the effect on selection rule 4, and the overturn (change the template value, not only the host).Host evidence
Not applicable: no file under
evidence/hosts/changes.Checklist
version comment (no floating tags). (None changed.)
permissions: contents: read(or a narrower, explicitly justified addition). (None changed.)
added without a documented owner.
moved or overwritten).
Residuals
adoption/skills/manifest.jsonexcluded[]still gives "Duplicates of the synced anthropic-skills" (pdf,docx,xlsx,pptx) and the syncedskill-creatorcopy as exclusion reasons. Where this template applies, those synced copies no longer load. The addendum records that they stay excluded on the user's finding that they went unused, and leaves re-wording the manifest text to the 2026-10-25 trial review.syncClaudeAiPluginsis deliberately unset: the user did not ask for it, and the host's init event lists no claude.ai plugin.apply_claude_settings.pyrun adds both keys. A host-only value is overwritten there because template scalars win, so re-enabling either one means changing the template (the addendum's overturn).🤖 Generated with Claude Code