Skip to content

ci: add skills-ref lint step for SKILL.md validation#626

Merged
jpoehnelt merged 1 commit intogoogleworkspace:mainfrom
ytausch:add-skills-ref-ci
Mar 25, 2026
Merged

ci: add skills-ref lint step for SKILL.md validation#626
jpoehnelt merged 1 commit intogoogleworkspace:mainfrom
ytausch:add-skills-ref-ci

Conversation

@ytausch
Copy link
Contributor

@ytausch ytausch commented Mar 25, 2026

Description

Adds a lint-skills CI job that validates all SKILL.md files using skills-ref validate. Runs via uvx (no install step needed) and triggers on skills/** changes or pushes to main.

Dry Run Output:

Not applicable — CI-only change, no new CLI command or API call.

Example output if a skill fails validation:

Validation failed for skills/gws-calendar:
  - Unexpected fields in frontmatter: version. Only ['allowed-tools', 'compatibility', 'description', 'license', 'metadata', 'name'] are allowed.

Checklist:

  • My code follows the AGENTS.md guidelines (no generated google-* crates).
  • I have run cargo fmt --all to format the code perfectly.
  • I have run cargo clippy -- -D warnings and resolved all warnings.
  • I have added tests that prove my fix is effective or that my feature works.
  • I have provided a Changeset file (e.g. via pnpx changeset) to document my changes.

Note: No Rust code was changed, so fmt/clippy are no-ops. No changeset is needed as this is a CI-only change that doesn't affect the published package. The CI job itself serves as the test.

@changeset-bot
Copy link

changeset-bot bot commented Mar 25, 2026

⚠️ No Changeset found

Latest commit: a612f02

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@ytausch ytausch marked this pull request as ready for review March 25, 2026 12:05
@gemini-code-assist
Copy link
Contributor

Note

Gemini is unable to generate a summary for this pull request due to the file types involved not being currently supported.

@jpoehnelt jpoehnelt merged commit b45cdf8 into googleworkspace:main Mar 25, 2026
1 check passed
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.

2 participants