Skip to content

fix(skills): validate Agent Skills frontmatter - #235

Merged
harlan-zw merged 2 commits into
mainfrom
fix/skill-metadata
Oct 10, 2026
Merged

harlan-zw merged 2 commits into
mainfrom
fix/skill-metadata

Conversation

@harlan-zw

@harlan-zw harlan-zw commented Oct 10, 2026 •

Copy link
Copy Markdown
Collaborator

🔗 Linked issue

❓ Type of change

  • 📖 Documentation
  • 🐞 Bug fix
  • 👌 Enhancement
  • ✨ New feature
  • 🧹 Chore
  • ⚠️ Breaking change

📚 Description

The generation contract rejects standard optional fields and encourages generic compatibility claims. Support all six Agent Skills fields, keep compatibility for specific requirements, and move account instructions into a linked reference.

Generation and validation support standard Skill frontmatter with specific environment requirements

🤖 AI disclosure: Harlan Agent Kit modified this description. My AI open-source policy.

@harlan-zw harlan-zw added the harlan-agent-review Approve automated work for the current issue state or pull request head commit. label Oct 10, 2026
@harlan-github-agent harlan-github-agent Bot added harlan-agent-review-required Pull request triage requires an adversarial Review for this head commit. and removed harlan-agent-review Approve automated work for the current issue state or pull request head commit. labels Oct 10, 2026
@harlan-zw

Copy link
Copy Markdown
Collaborator Author

🤖 Harlan Agent Kit Agent assessment of head 83f8a86.

Independent review found no confirmed blocker. All 24 focused output tests passed. The author diagram was visually checked. Confidence: 95/100.

The controller queue exceeded free host slots. Its Review was stopped for this head before independent review.

Harlan Agent Kit posted this assessment. Human merge decision required.

@harlan-zw harlan-zw changed the title fix(skills): support license and compatibility metadata fix(skills): validate Agent Skills frontmatter Oct 10, 2026
@harlan-zw harlan-zw added the harlan-agent-review Approve automated work for the current issue state or pull request head commit. label Oct 10, 2026
@harlan-github-agent harlan-github-agent Bot removed the harlan-agent-review Approve automated work for the current issue state or pull request head commit. label Oct 10, 2026
@harlan-zw harlan-zw added the harlan-agent-review Approve automated work for the current issue state or pull request head commit. label Oct 10, 2026
@harlan-github-agent harlan-github-agent Bot removed the harlan-agent-review Approve automated work for the current issue state or pull request head commit. label Oct 10, 2026
@harlan-zw

Copy link
Copy Markdown
Collaborator Author

🤖 Harlan Agent Kit Agent assessment of head aca221c.

Independent review found no confirmed blocker. Thirty output regression tests passed independently. Boundary calls accepted all six fields together and rejected duplicate keys, malformed metadata, unknown YAML tags, and oversized Unicode values. The moved account instructions remain linked and byte-identical.

The controller queue exceeded free host slots. Its Review was stopped for this head before independent review.

Harlan Agent Kit posted this assessment. Human merge decision required.

@harlan-zw
harlan-zw merged commit a246e3a into main Oct 10, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

harlan-agent-review-required Pull request triage requires an adversarial Review for this head commit.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant