Skip to content

feat: add skills_lint reusable workflow - #450

Open
unicoderbot[bot] wants to merge 8 commits into
mainfrom
vgv-ai-bot/issue-449
Open

feat: add skills_lint reusable workflow#450
unicoderbot[bot] wants to merge 8 commits into
mainfrom
vgv-ai-bot/issue-449

Conversation

@unicoderbot

@unicoderbot unicoderbot Bot commented May 29, 2026

Copy link
Copy Markdown
Contributor

Summary

Adds a new reusable workflow for validating agent skill directories using the dart_skills_lint tool. This workflow performs static analysis checks to ensure skill definitions comply with the Agent Skills Specification.

Changes

  • Added .github/workflows/dart_skills_lint.yml reusable workflow with workflow_call trigger
  • Configurable inputs for:
    • skills_directories: Space-separated list of skill root directories (defaults to .claude/skills and .agents/skills)
    • dart_skills_lint_version: Version constraint for the tool (defaults to latest)
    • dart_sdk: Dart SDK version (defaults to stable)
    • runs_on: Runner environment (defaults to ubuntu-latest)
    • working_directory: Working directory (defaults to .)
  • Workflow steps:
    1. Checkout code
    2. Setup Dart SDK
    3. Activate dart_skills_lint
    4. Run skill validation with configured directories
  • Updated README.md with workflow documentation

Validation

This workflow serves as a presubmit check for skill definitions, validating:

  • SKILL.md file presence and structure
  • YAML frontmatter compliance
  • Directory structure integrity
  • Relative path validity

Closes #449

@unicoderbot
unicoderbot Bot requested a review from a team as a code owner May 29, 2026 12:55
@unicoderbot unicoderbot Bot mentioned this pull request May 29, 2026
Open
6 tasks
@marcossevilla marcossevilla moved this from Needs Triage to In Progress in VGV Open Source 🦄 🧙🌟 Jun 3, 2026
@marcossevilla

marcossevilla commented Jun 3, 2026

Copy link
Copy Markdown
Member

@vgvbot check ci failures and try to fix them

@unicoderbot

unicoderbot Bot commented Jun 3, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

On it, I'll address the review comments and push the changes.

  • Preparing environment
    • Cloning repository
    • Setting up dev environment
  • Reviewing feedback
  • Working on changes
  • Pushing updates

I've pushed changes to address the feedback.

unicoderbot Bot added a commit that referenced this pull request Jun 3, 2026
@unicoderbot @marcossevilla
fix: address review feedback on PR #450
f812711 
Co-authored-by: marcossevilla <marcossevilla@users.noreply.github.com>
@marcossevilla

marcossevilla commented Jun 3, 2026

Copy link
Copy Markdown
Member

@vgvbot formatting is still failing for docs/workflows/dart_skills_lint.md. use prettier to fix this.

@unicoderbot

unicoderbot Bot commented Jun 3, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

On it, I'll address the review comments and push the changes.

  • Preparing environment
    • Cloning repository
    • Setting up dev environment
  • Reviewing feedback
  • Working on changes
  • Pushing updates

I've pushed changes to address the feedback.

unicoderbot Bot added a commit that referenced this pull request Jun 3, 2026
@unicoderbot @marcossevilla
fix: address review feedback on PR #450
411d63f 
Co-authored-by: marcossevilla <marcossevilla@users.noreply.github.com>
@marcossevilla
marcossevilla requested a review from ryzizub June 3, 2026 11:29
@marcossevilla

marcossevilla commented Jun 3, 2026

Copy link
Copy Markdown
Member

@vgvbot also add a workflow to verify this new reusable workflow you introduced, take inspiration in verify-dart

@unicoderbot

unicoderbot Bot commented Jun 3, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

On it, I'll address the review comments and push the changes.

  • Preparing environment
    • Cloning repository
    • Setting up dev environment
  • Reviewing feedback
  • Working on changes
  • Pushing updates

I've pushed changes to address the feedback.

unicoderbot Bot added a commit that referenced this pull request Jun 3, 2026
@unicoderbot @marcossevilla
fix: address review feedback on PR #450
cb9bd9d 
Co-authored-by: marcossevilla <marcossevilla@users.noreply.github.com>
@marcossevilla

marcossevilla commented Jun 4, 2026

Copy link
Copy Markdown
Member

@vgvbot fix the failing ci workflow by activating dart_skills_lint as a git ref dependency

@unicoderbot

unicoderbot Bot commented Jun 4, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

On it, I'll address the review comments and push the changes.

  • Preparing environment
    • Cloning repository
    • Setting up dev environment
  • Reviewing feedback
  • Working on changes
  • Pushing updates

I've pushed changes to address the feedback.

unicoderbot Bot added a commit that referenced this pull request Jun 4, 2026
@unicoderbot @marcossevilla
fix: address review feedback on PR #450
c0cbf22 
Co-authored-by: marcossevilla <marcossevilla@users.noreply.github.com>
@marcossevilla marcossevilla mentioned this pull request Jun 19, 2026
Open
@unicoderbot

unicoderbot Bot commented Jun 19, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

CI failed on this PR. I'm looking into it and will push a fix (attempt 1 of 3).

  • Preparing environment
    • Cloning repository
    • Setting up dev environment
  • Inspecting CI failures
  • Working on fix
  • Pushing fix

I've pushed a fix for the failing CI (attempt 1/3). Re-run CI to confirm.

unicoderbot Bot added a commit that referenced this pull request Jun 19, 2026
@unicoderbot
fix(ci): address failing checks on PR #450 (attempt 1/3)
7b2ba0d
@erickzanardo

erickzanardo commented Sep 1, 2026

Copy link
Copy Markdown
Member

/unicoder rebase this PR

@unicoderbot

unicoderbot Bot commented Sep 1, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

On it, I'll rebase this PR onto the target branch and resolve any conflicts.

  • Preparing environment
    • Cloning repository
    • Setting up dev environment
  • Fetching target branch
  • Rebasing onto target
  • Replaying commits
  • Updating PR branch

I tried to rebase this PR onto main but couldn't safely resolve the conflicts. Reason: conflict in generated or binary file site/package-lock.json is unsafe to auto-resolve. The branch is unchanged — please rebase manually.

unicoderbot Bot and others added 6 commits September 1, 2026 10:34
@unicoderbot @marcossevilla @erickzanardo
feat: add dart_skills_lint reusable workflow
56b6af8 
Closes #449

Co-authored-by: marcossevilla <marcossevilla@users.noreply.github.com>
@unicoderbot @marcossevilla @erickzanardo
fix: address review feedback on PR #450
c797402 
Co-authored-by: marcossevilla <marcossevilla@users.noreply.github.com>
@unicoderbot @marcossevilla @erickzanardo
fix: address review feedback on PR #450
7e16774 
Co-authored-by: marcossevilla <marcossevilla@users.noreply.github.com>
@unicoderbot @marcossevilla @erickzanardo
fix: address review feedback on PR #450
19b877c 
Co-authored-by: marcossevilla <marcossevilla@users.noreply.github.com>
@unicoderbot @marcossevilla @erickzanardo
fix: address review feedback on PR #450
b4473dd 
Co-authored-by: marcossevilla <marcossevilla@users.noreply.github.com>
@unicoderbot @erickzanardo
fix(ci): address failing checks on PR #450 (attempt 1/3)
e63ff52
@unicoderbot

unicoderbot Bot commented Sep 1, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

CI failed on this PR. I'm looking into it and will push a fix (attempt 1 of 3).

  • Preparing environment
    • Cloning repository
    • Setting up dev environment
  • Inspecting CI failures
  • Working on fix
  • Pushing fix

I stopped before starting: the SDK this repository is configured with cannot build it.

flutter 3.44.2 ships Dart 3.12.2, but examples/dart_package/pubspec.yaml requires Dart 3.13.0 or newer, so pub get and flutter analyze would fail.

Update the flutter version in this repository's bot settings, or set it to auto and I will pick the newest stable release that satisfies the repository. Then run this task again.

🔍 View this task run

@erickzanardo @claude
fix(skills_lint): track upstream move to the skills_lint package
6c9f9b2 
The tool this workflow wraps moved out from under the PR:

- flutter/skills was renamed to flutter/agent-plugins
- the tool was extracted to google/skills_lint.dart
- it is now published on pub.dev as skills_lint (dart_skills_lint is
  unpublished), and the executable is skills_lint, not dart_skills_lint:cli

The git-source activate therefore failed with "Could not find a file named
tool/dart_skills_lint/pubspec.yaml".

Activate from pub.dev instead, which is what issue #449 originally
specified, and replace the dart_skills_lint_ref input with
skills_lint_version. Rename the workflow, example, docs page, CI job and
semantic-PR scope to skills_lint to match the tool, since the workflow
filename is part of the public @v1 API.

Also point the Agent Skills Specification links at agentskills.io, as the
old SPECIFICATION.md URL now 404s.

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

alestiago commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

🦄 💻

@marcossevilla marcossevilla changed the title feat: add dart_skills_lint reusable workflow feat: add skills_lint reusable workflow Sep 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@marcossevilla marcossevilla marcossevilla approved these changes
@ryzizub ryzizub Awaiting requested review from ryzizub

Assignees

No one assigned

Labels

None yet

Projects

VGV Open Source 🦄 🧙🌟
Status: In Progress

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

feat: add dart_skills_lint reusable workflow

3 participants

@marcossevilla @erickzanardo @alestiago