Skip to content

feat(config): support Skill include and exclude patterns - #171

Merged
roark47 merged 1 commit into
mainfrom
fix/exclude-qoder-metadata
Aug 7, 2026
Merged

feat(config): support Skill include and exclude patterns#171
roark47 merged 1 commit into
mainfrom
fix/exclude-qoder-metadata

Conversation

@zpzjzj

@zpzjzj zpzjzj commented Aug 7, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • add optional doublestar include and exclude patterns to local Skill references
  • apply the same filters to both run-agent and judge Skill installation, with exclude taking precedence
  • keep the default behavior of installing all Skill content except evals/, including hidden Agent memory/metadata and ordinary YAML files
  • validate patterns, preserve judge filter metadata in reports, and document the configuration in English and Chinese

For the reported Qoder RepoWiki case, users can now configure:

skills:
  - source: local_path
    path: .
    exclude:
      - .qoder/repowiki/**

Verification

  • make verify
  • make test
  • regression coverage confirms .yaml and .yamlx have identical default handling
  • installation coverage confirms .qoder/repowiki/** is pruned while .claude/memory.md remains available

@zpzjzj
zpzjzj requested a review from hittyt as a code owner August 7, 2026 01:18
@zpzjzj
zpzjzj force-pushed the fix/exclude-qoder-metadata branch from 8fa6762 to 706e956 Compare August 7, 2026 01:33
@zpzjzj zpzjzj changed the title fix(agent): exclude Qoder repository metadata fix(agent): exclude hidden Skill metadata Aug 7, 2026
@zpzjzj
zpzjzj force-pushed the fix/exclude-qoder-metadata branch from 706e956 to 0a9404a Compare August 7, 2026 03:11
@zpzjzj zpzjzj changed the title fix(agent): exclude hidden Skill metadata feat(config): support Skill include and exclude patterns Aug 7, 2026
@zpzjzj
zpzjzj requested a review from roark47 August 7, 2026 03:15
@zpzjzj zpzjzj self-assigned this Aug 7, 2026
@zpzjzj zpzjzj added the feature label Aug 7, 2026

@roark47 roark47 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@roark47
roark47 added this pull request to the merge queue Aug 7, 2026
Merged via the queue into main with commit 4494b3b Aug 7, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants