Skip to content

feat: add agent validation and improve CI workflow#40

Merged
quuu merged 3 commits intovercel-labs:mainfrom
bensabic:misc-updates
Jan 19, 2026
Merged

feat: add agent validation and improve CI workflow#40
quuu merged 3 commits intovercel-labs:mainfrom
bensabic:misc-updates

Conversation

@bensabic
Copy link
Contributor

@bensabic bensabic commented Jan 18, 2026

Summary

  • Fix OpenCode paths to use skills (plural) instead of skill

  • Add validate-agents script to prevent duplicate agent entries (checks displayName and skillsDir conflicts)

  • Rename update-readme to sync-agents, now updates both README and package.json keywords, multiple changes to the script structure

  • Single workflow for handling the agent validation and sync jobs

  • README and package.json kept in sync with validated agents

    • Remove CI-only scripts from package.json
    • Add telemetry documentation to README
    • Support importing skills from non-default branches via tree URLs
  • Added source parser unit test

    Test Plan

    • Run npx tsx scripts/validate-agents.ts locally, should catch existing Gemini duplicates
    • Verify OpenCode paths show .opencode/skills/ in README
    • Check package.json keywords are auto-generated from agents

Closes #22
Closes #36
Closes #37
Closes #42

@bensabic
Copy link
Contributor Author

@quuu - addressed the conflicts from the other commits and added one more addition

- Fix OpenCode paths to use 'skills' (plural) instead of 'skill'
- Add validate-agents script to check for duplicate displayNames and skillsDirs
- Rename update-readme to sync-agents (now also updates package.json keywords)
- Combine workflows into single agents.yml with validate and sync jobs
- Remove scripts from package.json (CI-only)
- Add telemetry documentation to README
- Support importing skills from non-default branches via tree URLs

Closes #36
Closes #37
Closes #42
Copy link
Collaborator

@quuu quuu left a comment

Choose a reason for hiding this comment

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

@bensabic overall am excited to merge this!

just a few comments to make sure things work for all platforms and that the code is maintainable/ well understood after it's merged :)

- Fix homedir normalization to work on macOS/Windows (use os.homedir())
- Add unit tests for source-parser.ts
@bensabic
Copy link
Contributor Author

@quuu Thank you so much and so sorry about all that as well! The changes have been made and functionality tested accordingly.

Additionally, I updated the sync-agents script to incorporate what PR #43 introduced with the manual update.

Copy link
Collaborator

@quuu quuu left a comment

Choose a reason for hiding this comment

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

the revisions look good! thanks a lot

@quuu quuu merged commit 8c2c13c into vercel-labs:main Jan 19, 2026
2 checks 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

2 participants