Skip to content

feat: add Science Platform agent skills - #309

Open
sfabbro wants to merge 3 commits into
opencadc:mainfrom
sfabbro:feat/agent-skills
Open

feat: add Science Platform agent skills#309
sfabbro wants to merge 3 commits into
opencadc:mainfrom
sfabbro:feat/agent-skills

Conversation

@sfabbro

@sfabbro sfabbro commented Sep 4, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Add 22 in-tree CANFAR Science Platform skills (skills/, catalog.json) for agents, with no AstroAI product CLI, images, or $WORK paths.
  • Install path is npx skills add opencadc/canfar. Cursor plugin metadata lives in .cursor-plugin/plugin.json; the validator is python3 scripts/validate_skills.py.
  • Document the skills in docs/agents/platform-skills.md and run the validator as a CI job. Lint exclusions cover scripts/ so the CLI checker does not fight ruff/interrogate/radon.
  • Correct the transfers overview example: vos: is Vault, so ARC copies use arc: (or canfar data). The validator also scans catalog/plugin/docs metadata for product leaks.
  • README install is npx-only. Leak scan treats capital-R Ray (not X-ray) and canfar-lab. Replica sharding points at canfar.helpers.distributed; Session.create GPU kwarg is gpu=.

Test plan

  • python3 scripts/validate_skills.py (22/22)
  • uv run --group docs mkdocs build (exit 0; git-committers 403 is non-fatal)
  • uv run pre-commit run --files on the new/changed files
  • CI skills job on this PR
  • CI on the vos:/arc follow-up commit
  • CI on the install-docs / leak-scan / distributed / gpu= commit

Ship 22 OpenCADC-grounded agent skills in-tree so npx skills add
opencadc/canfar installs the platform guides next to the client.
vostools rewrites the vos scheme to vault, so the transfers table sent
ARC uploads to the wrong service. Scan skill metadata for the same class
of product leak.
README was mixing npx install with the clone-only validator. The leak
regex treated X-ray as Ray and missed canfar-lab. Point replica sharding
at canfar.helpers.distributed and document Session.create gpu=.
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.

1 participant