Skip to content

Point the plugin config at Morrison-Lab/ai-config, add Posit's Quarto skills - #53

Merged
d-morrison merged 1 commit into
mainfrom
claude/clever-bardeen-nepiz5
Sep 10, 2026
Merged

d-morrison merged 1 commit into
mainfrom
claude/clever-bardeen-nepiz5

Conversation

@d-morrison

Copy link
Copy Markdown
Collaborator

Why

.claude/settings.json already registered the ai-config plugin, but under the key d-morrison pointing at d-morrison/ai-config. That repo now lives at Morrison-Lab/ai-config, and its .claude-plugin/marketplace.json declares the marketplace name Morrison-Lab — so both the marketplace key and the plugin reference ai-config@d-morrison disagreed with the marketplace they resolve to. The GitHub redirect keeps the clone working, but the plugin's skills (/mwc, /ardi, the PR-workflow skills) did not load in this repo's Claude Code sessions.

What changed

  • Renamed the marketplace key to Morrison-Lab, pointed its source at Morrison-Lab/ai-config, and updated the plugin reference to ai-config@Morrison-Lab — matching the install snippet in that repo's README.
  • Registered posit-dev/skills (marketplace name posit-dev-skills) and enabled its quarto plugin, which bundles the quarto-authoring, brand-yml, and alt-text skills. This repo is a Quarto website, so those apply directly.

Not enabled, but available from the same marketplace if you want them: posit-dev (general dev skills), github (PR create / review-thread workflows), r-lib (R package development), shiny, connect, open-source, ggsql.

Verification

.claude/settings.json parses as JSON; the marketplace keys and plugin references match the name fields in each upstream .claude-plugin/marketplace.json (Morrison-Lab and posit-dev-skills). Plugin loading itself only takes effect in a fresh session, so it can't be exercised from this one.

🤖 Generated with Claude Code

https://claude.ai/code/session_01KCdy9ZGLe3uScg74jogJDF


Generated by Claude Code

… skills

The ai-config marketplace entry still named `d-morrison/ai-config` under the
key `d-morrison`. That repo now lives at `Morrison-Lab/ai-config`, and its
`.claude-plugin/marketplace.json` declares the marketplace name
`Morrison-Lab`, so the key and the plugin reference `ai-config@d-morrison`
no longer matched the marketplace they resolve to — the plugin's skills
(`/mwc`, `/ardi`, the PR-workflow skills) did not load in this repo's
sessions.

Also register posit-dev/skills and enable its `quarto` plugin, which bundles
the quarto-authoring, brand-yml, and alt-text skills — this repo is a Quarto
website, so those apply directly.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01KCdy9ZGLe3uScg74jogJDF
@github-actions

Copy link
Copy Markdown

Claude finished reviewView run


All unrelated to the plugin marketplace config — different repos (qwt, psw itself, rme) that are out of scope for this PR. No cleanup needed there.

The change itself is minimal, correct, and well-scoped: it renames the marketplace key to match the actual marketplace.json name at Morrison-Lab/ai-config, updates the plugin reference to match, and additively registers posit-dev/skills with the quarto plugin enabled. I independently verified every factual claim in the PR description against the live repos:

  • d-morrison/ai-config 301-redirects to github.com/Morrison-Lab/ai-config (repo was renamed/transferred).
  • Morrison-Lab/ai-config's .claude-plugin/marketplace.json declares "name": "Morrison-Lab", matching the new key.
  • posit-dev/skills's marketplace.json declares the marketplace name posit-dev-skills and defines a quarto plugin bundling brand-yml, quarto-authoring, and alt-text — exactly as described, and directly relevant since this repo is a Quarto website.
  • The resulting .claude/settings.json is valid JSON.

No .qmd chapters, prose, citations, or Quarto config are touched by this PR (it's confined to .claude/settings.json), so the SERG lab-manual prose/coding-style/practices sections don't apply here — this is tooling/CI configuration, not R or Quarto content. No hallucinated APIs, packages, or URLs were found.

One neutral observation (not a blocker): I noticed the local working tree has an unstaged, uncommitted reversion of this exact file back to the pre-PR content — this is local session/environment state, not part of the PR's committed diff, so I did not treat it as a finding and left it untouched per review-only instructions.

Verdict

Ready for merge

Structured Review Data (JSON)
{
  "schema_version": "1.1",
  "reviewer": "claude",
  "commit_sha": "aa3f6a68fcff442ca17aab3cb996dd90dd22f150",
  "verdict": "CLEAN",
  "findings": [],
  "detailed_assessment": "Reviewed .claude/settings.json changes and confirmed the renamed Morrison-Lab marketplace key, updated ai-config plugin reference, and newly added posit-dev-skills entry all resolve correctly with no JSON syntax defect.",
  "holistic_assessment": "Change is narrowly scoped tooling configuration with no Quarto content, no regression risk to site build or chapters, and requirements verified live against both upstream marketplace.json sources."
}

Reviewed commit: aa3f6a6

@github-actions

Copy link
Copy Markdown

💰 Cost: $0.2716 (review) — run

@d-morrison
d-morrison merged commit d7e7f29 into main Sep 10, 2026
12 checks passed
@d-morrison
d-morrison deleted the claude/clever-bardeen-nepiz5 branch September 10, 2026 21:55
@github-actions

Copy link
Copy Markdown
PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-09-10 21:57 UTC

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