Bump skillsaw to v0.11.4

[stbenjam]

Small bump to skillsaw, that fixes @zaneb's comment here: #487 (comment)

Summary

• Bumps skillsaw from v0.11.2 to v0.11.4 across Makefile, GitHub Actions workflows, and .skillsaw.yaml
• Regenerates docs/index.html with the new version

Changes

• Makefile: update container image tag
• .github/workflows/lint-plugins.yml: update action SHA and version input
• .github/workflows/lint-review.yml: update action SHA
• .github/workflows/update-docs.yml: update pip install version
• .skillsaw.yaml: update version field
• docs/index.html: regenerated

🤖 Generated with Claude Code

Summary by CodeRabbit

• Chores
  • Updated build and linting tooling versions from 0.11.2 to 0.11.4 across CI/CD workflows, configuration files, and development setup.

[coderabbitai]

Walkthrough

Skillsaw tool version is updated uniformly from 0.11.2 to 0.11.4 across configuration files, GitHub Actions workflows, and the Makefile container image reference. All references to the tool are bumped consistently.

Changes

Skillsaw upgrade

Layer / File(s)	Summary
Version and reference updates across workflows and configuration .skillsaw.yaml, Makefile, .github/workflows/lint-plugins.yml, .github/workflows/lint-review.yml, .github/workflows/update-docs.yml	Skillsaw is bumped from 0.11.2 to 0.11.4 in the local configuration, Makefile container image, and pinned action references across three GitHub Actions workflows.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

• openshift-eng/ai-helpers#505: Both PRs overlap by updating the project's pinned skillsaw version across GitHub Actions and the Makefile/.skillsaw.yaml.
• openshift-eng/ai-helpers#456: Both PRs modify the Skillsaw GitHub Actions usage in .github/workflows/lint-plugins.yml, updating pinned Skillsaw versions.
• openshift-eng/ai-helpers#471: Both PRs update the pinned stbenjam/skillsaw action versions and the .skillsaw.yaml version value as part of Skillsaw configuration changes.

Suggested labels

area/dependency, ok-to-test

Suggested reviewers

• cblecker
• bryan-cox

---

Important

Pre-merge checks failed

Please resolve all errors before merging. Addressing warnings is optional.

❌ Failed checks (1 error)

Check name	Status	Explanation	Resolution
No Real People Names In Style References	❌ Error	PR contains real people names in plugin documentation: "Alice Developer," "Bob Engineer," "Charlie Contributor," "Diana Developer," "Eve Technical Writer," and "John Doe" in examples and tests.	Replace real names with generic placeholders (e.g., "Developer Name," "Engineer Name") in suggest-reviewers SKILL.md, hello-world evals, and jira status-analysis SKILL.md.

✅ Passed checks (9 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title 'Bump skillsaw to v0.11.4' directly and clearly describes the main change across all modified files.
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.
No Assumed Git Remote Names	✅ Passed	PR modifies only version configuration and workflows (skillsaw 0.11.2→0.11.4); none of the 6 modified files contain hardcoded git remote names without discovery.
Git Push Safety Rules	✅ Passed	PR contains only version bumps and configuration updates; no git push commands, force push operations, or pushes to main/master found in any modified files.
No Untrusted Mcp Servers	✅ Passed	PR only updates skillsaw version from 0.11.2 to 0.11.4 across Makefile, workflows, and .skillsaw.yaml. No new MCP server installations from any source detected.
Ai-Helpers Overlap Detection	✅ Passed	PR only updates tool versions and configuration (Makefile, workflows, .skillsaw.yaml), not ai-helpers files (plugins//commands/.md, skills, agents), so overlap detection check does not apply.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

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

[coderabbitai]

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)

.github/workflows/lint-review.yml (1)

17-25: ⚠️ Potential issue | 🟠 Major | ⚡ Quick win

Add contents: read to review job permissions for actions/checkout.

Job-level permissions currently grants only pull-requests: write, which makes all other GITHUB_TOKEN permissions none; actions/checkout requires contents: read (unless you pass an explicit token/ssh-key).

Proposed fix

   review:
     if: github.event.workflow_run.event == 'pull_request'
     runs-on: ubuntu-latest
     permissions:
+      contents: read
       pull-requests: write

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In @.github/workflows/lint-review.yml around lines 17 - 25, The job-level
permissions lack contents: read, which prevents actions/checkout from working
with the implicit GITHUB_TOKEN; update the permissions block in the lint-review
job to include "contents: read" alongside "pull-requests: write" so
actions/checkout (uses: actions/checkout@...) can read the repository and the
review action (uses: stbenjam/skillsaw/review@...) can post comments; locate the
permissions stanza in .github/workflows/lint-review.yml and add the contents:
read entry at the same indentation level as pull-requests: write.

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Outside diff comments:
In @.github/workflows/lint-review.yml:
- Around line 17-25: The job-level permissions lack contents: read, which
prevents actions/checkout from working with the implicit GITHUB_TOKEN; update
the permissions block in the lint-review job to include "contents: read"
alongside "pull-requests: write" so actions/checkout (uses:
actions/checkout@...) can read the repository and the review action (uses:
stbenjam/skillsaw/review@...) can post comments; locate the permissions stanza
in .github/workflows/lint-review.yml and add the contents: read entry at the
same indentation level as pull-requests: write.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: ba5f13d2-5704-4a9a-80f8-37829d738c7a

📥 Commits

Reviewing files that changed from the base of the PR and between 744b3d5 and 40ee07c.

📒 Files selected for processing (6)

• .github/workflows/lint-plugins.yml
• .github/workflows/lint-review.yml
• .github/workflows/update-docs.yml
• .skillsaw.yaml
• Makefile
• docs/index.html

[cblecker]

/lgtm

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: cblecker, stbenjam

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details

Needs approval from an approver in each of these files:

• .github/OWNERS [cblecker,stbenjam]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[openshift-ci]

@stbenjam: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name	Commit	Details	Required	Rerun command
ci/prow/images	40ee07c	link	unknown	/test images

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.