Skip to content

Fix /build bootstrap: matlab/matplotlib via gh from scientific-agent-skills - #4

Open
PaulinaDotor wants to merge 1 commit into
IvanGomezMartinez:mainfrom
PaulinaDotor:fix/bootstrap-scientific-agent-skills
Open

PaulinaDotor wants to merge 1 commit into
IvanGomezMartinez:mainfrom
PaulinaDotor:fix/bootstrap-scientific-agent-skills

Conversation

@PaulinaDotor

Copy link
Copy Markdown

Summary

 scripts/bootstrap-skills.sh
- npx … k-dense-ai/claude-scientific-skills --skill matlab
+ gh skill install K-Dense-AI/scientific-agent-skills skills/matlab
+ gh skill install K-Dense-AI/scientific-agent-skills skills/matplotlib

 .agents/skills/
+ matplotlib/          # vendored from scientific-agent-skills@v2.69.0
~ matlab/              # refreshed metadata (github-repo + tree sha)

 scripts/lib/build_import.py
- move only top-level files
+ move nested dirs/files when cataloging import/ → codes/

/build Step 0 was hanging/failing on the old claude-scientific-skills URL. Setup now installs matlab and matplotlib with gh skill install from the current K-Dense repo, and catalog can move nested bundle folders.

Evidence

  • Before: npx skills add …/claude-scientific-skills failed / hung during /build bootstrap.
    After: gh skill install K-Dense-AI/scientific-agent-skills skills/matlab|matplotlib succeeds; ./scripts/test-bootstrap-skills.sh → bootstrap-skills tests: OK.

Merge Danger

Door: two-way

Reverts cleanly; no data migration. Downstream forks need gh + auth for first-time setup of these two skills.

Blast Radius: bootstrap setup path

Affects /build Step 0 setup only (not SKIPPED_FRESH / update). Adds vendored matplotlib skill tree to the template.

Made with Cursor

…c-agent-skills.

Replace the dead claude-scientific-skills URL with gh skill install from K-Dense-AI/scientific-agent-skills, vendor matplotlib, and allow nested folders when cataloging import bundles.

Co-authored-by: Cursor <cursoragent@cursor.com>
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