Skip to content

plugins/mattstack: README lists process-digraphs; certify plugin/skills in CI - #547

Merged
m4ttheweric merged 2 commits into
mainfrom
dg-certify-plugin-skills
Sep 28, 2026
Merged

m4ttheweric merged 2 commits into
mainfrom
dg-certify-plugin-skills

Conversation

@m4ttheweric

@m4ttheweric m4ttheweric commented Sep 28, 2026 •

Copy link
Copy Markdown
Collaborator

Follow-up to #544: the README now lists process-digraphs as public, and CI certifies the skills under plugin/skills/, which its certify loop never reached before.

What changed

Verification

  • The five plugin/skills dirs were never certified by CI; each passes sh tests/certify.sh (exit 0), so no skill needed a fix.
  • The full new loop (skills/*/ plugin/skills/*/ attachments/*/ attachments/*/*/) runs locally with no failures.
  • scripts/repo-purity.sh passes.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • New Features
    • Added mattstack:process-digraphs for creating or revising process workflows as diagrams, with support for rendering and linting.
  • Documentation
    • Updated the list of directly invocable skills to include process diagram workflows.

m4ttheweric and others added 2 commits September 28, 2026 08:23
…kills

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Sep 28, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: 5b2f8298-78a2-40ce-b4e0-f2d87cbdd17d

📥 Commits

Reviewing files that changed from the base of the PR and between a24b734 and d3fb1f8.

📒 Files selected for processing (3)
  • .github/workflows/checks.yml
  • plugins/mattstack/.claude-plugin/plugin.json
  • plugins/mattstack/README.md

Included review availability: This review used your included allowance. 0 included reviews remain after this review. Your included PR review attempts over the past 7 days set your current allowance at 1 review per hour.


📝 Walkthrough

Walkthrough

The README documents mattstack:process-digraphs and its tools. The certification loop now searches plugin skill directories, and the plugin version changes to 0.27.5.

Changes

Process digraph skill registration

Layer / File(s) Summary
Document and certify the process-digraphs skill
.github/workflows/checks.yml, plugins/mattstack/README.md, plugins/mattstack/.claude-plugin/plugin.json
The README lists and describes the skill. The certification loop searches plugin/skills/*/, and the plugin version changes to 0.27.5.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to d3fb1

The documentation, skill certification paths, and plugin version update appear consistent. No material merge risk is evident.

Architecture Summary

Architecture risk: 🔵 Low · up to d3fb1

The change affects 1 system.

Changed systems: plugins

Architecture concerns
No architecture-level concerns identified.

Review details

Systems and components

  • observed — plugins (service) was modified; 2 changed files map to changed impact.

Before / after behavior

  • observed — Modified behavior in plugins/mattstack/.claude-plugin/plugin.json: The plugin metadata version changes from 0.27.3 to 0.27.5.
  • observed — Modified behavior in plugins/mattstack/README.md: The directly invocable skill list now names mattstack:process-digraphs; the surrounding description of other skills as engines remains.
  • observed — Modified behavior in plugins/mattstack/README.md: The Infra section adds mattstack:process-digraphs, describing the digraph format and its rendering and linting tools.
  • observed — Modified behavior in .github/workflows/checks.yml: The certification loop now includes plugin/skills/*/ in its directory search; all existing search paths remain.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly identifies the README update and CI certification-loop change, which are the main changes in the pull request.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

Comment @coderabbitai help to get the list of available commands.

@m4ttheweric
m4ttheweric merged commit 07d50f1 into main Sep 28, 2026
16 checks passed
@m4ttheweric
m4ttheweric deleted the dg-certify-plugin-skills branch September 28, 2026 13:33
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