ai-ml/skills-library
SHA256
13
6
Fork 4
Code Issues 7 Pull requests 2 Projects Activity Actions

Onboard repo to skillsaw linting #10

Merged
cverna merged 1 commit from cverna/skills-library:introduce_skillsaw into main 2026-07-19 16:27:20 +00:00
Conversation 5 Commits 1 Files changed 7 +425 -41
cverna commented 2026-07-08 18:19:30 +00:00
Member
Copy link

This introduce some CI for our skills, using https://github.com/stbenjam/skillsaw. It will be run against each PR.

In this PR we are creating the baseline which fixes weak-language and context-budget violations in skill files.

This introduce some CI for our skills, using https://github.com/stbenjam/skillsaw. It will be run against each PR. In this PR we are creating the baseline which fixes weak-language and context-budget violations in skill files.
Onboard repo to skillsaw linting
Some checks failed
Lint / skillsaw (pull_request) Has been cancelled
Details
b4c15482e4 
Creates skillsaw configuration and CI workflow for Forgejo.
Fixes weak-language and context-budget violations in skill files.

Assisted-by: opencode deepseek-v4-flash-free
Signed-off-by: Clement Verna <cverna@tutanota.com>
cverna reviewed 2026-07-08 18:19:52 +00:00
.forgejo/workflows/lint.yml Outdated
@ -0,0 +7,4 @@
jobs:
skillsaw:
runs-on: ubuntu-latest
cverna commented 2026-07-08 18:19:52 +00:00
Author
Member
Copy link

Need to change once we have runner available.

Need to change once we have runner available.
cverna marked this conversation as resolved
cverna force-pushed introduce_skillsaw from b4c15482e4
Some checks failed
Lint / skillsaw (pull_request) Has been cancelled
Details
to dad445d5b9
Some checks failed
Lint / skillsaw (pull_request) Failing after 0s
Details
2026-07-08 18:43:36 +00:00 Compare
cverna force-pushed introduce_skillsaw from dad445d5b9
Some checks failed
Lint / skillsaw (pull_request) Failing after 0s
Details
to 6f3f709ef8
Some checks failed
Lint / skillsaw (pull_request) Failing after 5s
Details
2026-07-08 18:48:44 +00:00 Compare
cverna force-pushed introduce_skillsaw from 6f3f709ef8
Some checks failed
Lint / skillsaw (pull_request) Failing after 5s
Details
to 8ce8a61ab7
Some checks failed
Lint / skillsaw (pull_request) Has been cancelled
Details
2026-07-08 18:54:30 +00:00 Compare
cverna force-pushed introduce_skillsaw from 8ce8a61ab7
Some checks failed
Lint / skillsaw (pull_request) Has been cancelled
Details
to 6f02847aa3
Some checks failed
Lint / skillsaw (pull_request) Failing after 41s
Details
2026-07-08 18:55:43 +00:00 Compare
Fix checkout ref to use branch name instead of SHA
Some checks failed
Lint / skillsaw (pull_request) Failing after 35s
Details
43261af8b1 
Forgejo passes a SHA256-based commit hash that doesn't exist on the remote.
Checkout by branch name to avoid the resolution issue.

Signed-off-by: Clement Verna <cverna@tutanota.com>
cverna force-pushed introduce_skillsaw from 43261af8b1
Some checks failed
Lint / skillsaw (pull_request) Failing after 35s
Details
to 52ed503b6b
Some checks failed
Lint / skillsaw (pull_request) Failing after 5s
Details
2026-07-08 19:06:32 +00:00 Compare
cverna force-pushed introduce_skillsaw from 52ed503b6b
Some checks failed
Lint / skillsaw (pull_request) Failing after 5s
Details
to 7cec6159d6
Some checks failed
Validate manifests / validate (pull_request) Failing after 3s
Details
2026-07-08 19:13:41 +00:00 Compare
cverna force-pushed introduce_skillsaw from 7cec6159d6
Some checks failed
Validate manifests / validate (pull_request) Failing after 3s
Details
to a8d04fd281
Some checks failed
Validate manifests / validate (pull_request) Failing after 13s
Details
2026-07-08 19:15:04 +00:00 Compare
cverna force-pushed introduce_skillsaw from a8d04fd281
Some checks failed
Validate manifests / validate (pull_request) Failing after 13s
Details
to 080c8c72ca
Some checks failed
Validate manifests / validate (pull_request) Failing after 17s
Details
2026-07-08 19:18:03 +00:00 Compare
cverna force-pushed introduce_skillsaw from 080c8c72ca
Some checks failed
Validate manifests / validate (pull_request) Failing after 17s
Details
to 200c56fcf2
Some checks failed
Validate manifests / validate (pull_request) Failing after 15s
Details
2026-07-08 19:18:46 +00:00 Compare
cverna force-pushed introduce_skillsaw from 200c56fcf2
Some checks failed
Validate manifests / validate (pull_request) Failing after 15s
Details
to 44e6ec5b82
All checks were successful
Validate manifests / validate (pull_request) Successful in 17s
Details
2026-07-08 19:19:53 +00:00 Compare
cverna commented 2026-07-09 18:53:44 +00:00
Author
Member
Copy link

Discussion about this in #13

Discussion about this in https://forge.fedoraproject.org/ai-ml/skills-library/issues/13
cverna force-pushed introduce_skillsaw from 44e6ec5b82
All checks were successful
Validate manifests / validate (pull_request) Successful in 17s
Details
to 694d87dec2
Some checks failed
Validate manifests / validate (pull_request) Failing after 23s
Details
2026-07-11 11:31:04 +00:00 Compare
ttomecek approved these changes 2026-07-16 13:06:25 +00:00
ttomecek left a comment
Copy link

LGTM very nice! it even improved the skills already

LGTM very nice! it even improved the skills already
.forgejo/workflows/lint.yml Outdated
@ -0,0 +19,4 @@
steps:
- name: "Install required packages"
run: |
dnf config-manager disable fedora-cisco-openh264
ttomecek commented 2026-07-16 13:05:13 +00:00
Member
Copy link

well done! 😁

well done! 😁
ttomecek commented 2026-07-16 13:09:43 +00:00
Member
Copy link

The CI failure:

Warnings:
⚠ WARNING (context-budget) [skills/fedora-compose-triage/SKILL.md]: Estimated 3,083 tokens exceeds skill warn limit of 3,000
Rule docs (or run skillsaw explain <rule-id>):
https://skillsaw.org/rules/context-budget/

I personally wouldn't mind bumping that limit even to 10k. Since we now have 1M context window models, 3k limit sounds pretty strict to me.

The CI failure: > Warnings: > ⚠ WARNING (context-budget) [skills/fedora-compose-triage/SKILL.md]: Estimated 3,083 tokens exceeds skill warn limit of 3,000 > Rule docs (or run `skillsaw explain <rule-id>`): > https://skillsaw.org/rules/context-budget/ I personally wouldn't mind bumping that limit even to 10k. Since we now have 1M context window models, 3k limit sounds pretty strict to me.
👍 1
cverna force-pushed introduce_skillsaw from 694d87dec2
Some checks failed
Validate manifests / validate (pull_request) Failing after 23s
Details
to e3f27d1fb6
All checks were successful
Validate manifests / validate (pull_request) Successful in 24s
Details
2026-07-19 16:25:28 +00:00 Compare
cverna commented 2026-07-19 16:27:09 +00:00
Author
Member
Copy link

I personally wouldn't mind bumping that limit even to 10k. Since we now have 1M context window models, 3k limit sounds pretty strict to me.

Updated to warning at 6k and error at 10k.

> I personally wouldn't mind bumping that limit even to 10k. Since we now have 1M context window models, 3k limit sounds pretty strict to me. Updated to warning at 6k and error at 10k.
cverna merged commit e3f27d1fb6 into main 2026-07-19 16:27:20 +00:00
cverna deleted branch introduce_skillsaw 2026-07-19 16:27:20 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
ttomecek
Labels
Clear labels   
bug
Something is not working

  
duplicate
This issue or pull request already exists

  
enhancement
New feature

  
help wanted
Need some help

  
invalid
Something is wrong

  
question
More information is needed

  
wontfix
This won't be fixed

No labels
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
ai-ml/skills-library!10
No description provided.