Skip to content

chore: add .skillsaw.yaml to exclude vendored gws skills#41

Merged
cblecker merged 1 commit into
mainfrom
chore/skillsaw-exclude-vendored-skills
May 27, 2026
Merged

chore: add .skillsaw.yaml to exclude vendored gws skills#41
cblecker merged 1 commit into
mainfrom
chore/skillsaw-exclude-vendored-skills

Conversation

@cblecker
Copy link
Copy Markdown
Owner

@cblecker cblecker commented May 27, 2026

Summary

  • Adds .skillsaw.yaml config to exclude gws/skills/** from skillsaw linting
  • The gws skills are auto-generated from upstream via gws generate-skills — content-rule violations on vendored text should be suppressed at the config level rather than by modifying the files (which creates drift)
  • Mirrors the existing .markdownlint-cli2.jsonc ignore pattern for the same path
  • Closes the approach taken in fix(gws): correct skillsaw content violations via skillsaw fix --llm #40

Test plan

  • uvx skillsaw --strict passes with 0 errors, 0 warnings

Summary by CodeRabbit

  • Chores
    • Updated Skillsaw to version 0.11.1
    • Modified configuration to exclude specific paths from processing

Review Change Stack

The gws skills are auto-generated from upstream via `gws generate-skills`.
Exclude them from skillsaw linting to avoid content-rule violations on
text we don't control, matching the existing markdownlint ignore pattern.

Assisted-by: Claude:claude-opus-4-6
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 27, 2026

Caution

Review failed

Pull request was closed or merged during review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 00374d05-6e82-42ff-ae86-dc05c1101913

📥 Commits

Reviewing files that changed from the base of the PR and between 42e8562 and c0784a3.

📒 Files selected for processing (1)
  • .skillsaw.yaml

📝 Walkthrough

Walkthrough

The .skillsaw.yaml configuration file is updated to version 0.11.1 and a new exclude rule is added to filter out paths matching gws/skills/**.

Changes

Configuration Update

Layer / File(s) Summary
Skillsaw version and exclude rule
.skillsaw.yaml
Version is bumped to 0.11.1 and an exclude rule is added to filter paths under gws/skills/**.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • cblecker/claude-plugins#39: Switches the project to use Skillsaw for linting via uvx skillsaw; this PR bumps the Skillsaw version to 0.11.1 in direct support of that migration.

Poem

🐰 A config file tweak, so neat and tight,
Skillsaw climbs to point-one-one tonight!
Exclude the gws paths with a gentle sway,
Linting tools dance in their helpful way! ✨

🚥 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 and specifically describes the main change: adding a .skillsaw.yaml configuration file to exclude vendored gws skills from linting.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/skillsaw-exclude-vendored-skills

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@cblecker cblecker merged commit 782617c into main May 27, 2026
7 of 8 checks passed
@cblecker cblecker deleted the chore/skillsaw-exclude-vendored-skills branch May 27, 2026 21:47
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