Skip to content

Support internal: true metadata to skip installing certain skills#520

Merged
jakemac53 merged 4 commits into
mainfrom
internal-meta
Jul 15, 2026
Merged

Support internal: true metadata to skip installing certain skills#520
jakemac53 merged 4 commits into
mainfrom
internal-meta

Conversation

@jakemac53

Copy link
Copy Markdown
Contributor

Closes #502

  • Adds a simple parser for skill frontmatter as well.
  • Uses that to extract the internal metadata field if present and skip skills unless the environment variable is set
  • Cleans up some older tests that I noticed were set up wrong or invalid.

@jakemac53 jakemac53 requested a review from jwren July 13, 2026 15:48
@github-actions

Copy link
Copy Markdown

Package publishing

If you have publishing permissions, you can use the links below to publish the changes after merging this PR.

Package Version Status Publish tag (post-merge)
package:dart_mcp 0.5.2 already published at pub.dev
package:dart_mcp_server 1.1.0-wip WIP (no publish necessary)
package:skills 0.4.0-dev ready to publish skills-v0.4.0-dev

Documentation at https://github.com/dart-lang/ecosystem/wiki/Publishing-automation.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request adds support for filtering internal skills by parsing internal: true metadata from the frontmatter of skill files, skipping them during scanning unless the INSTALL_INTERNAL_SKILLS=1 environment variable is set. The feedback suggests optimizing frontmatter parsing to avoid unnecessary string allocations and recommends using local parameters or fields instead of directly referencing the global environment variable check within the scanners.

Comment thread pkgs/skills/lib/src/core/frontmatter.dart Outdated
Comment thread pkgs/skills/lib/src/core/git_scanner.dart
Comment thread pkgs/skills/lib/src/core/skill_scanner.dart
@github-actions

github-actions Bot commented Jul 13, 2026

Copy link
Copy Markdown

PR Health

Changelog Entry ✔️
Package Changed Files

Changes to files need to be accounted for in their respective changelogs.

This check can be disabled by tagging the PR with skip-changelog-check.

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
@jakemac53 jakemac53 merged commit a067ab8 into main Jul 15, 2026
22 checks passed
@jakemac53 jakemac53 deleted the internal-meta branch July 15, 2026 16:43
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.

Support internal: true metadata when installing skills

2 participants