Skip to content

fix(bwcopilot): mount the shared skills tree at /nsls2/agents/skills - #26

Merged
Thomas A Caswell (tacaswell) merged 1 commit into
NSLS2:mainfrom
stuwilkins:fix/bwcopilot-skills-path
Oct 22, 2026
Merged

Thomas A Caswell (tacaswell) merged 1 commit into
NSLS2:mainfrom
stuwilkins:fix/bwcopilot-skills-path

Conversation

@stuwilkins

Copy link
Copy Markdown
Contributor

Skills are now published to /nsls2/agents/skills and the old per-host clone is no longer deployed. bwcopilot mounts the new tree read-only; list it in Copilot's skillDirectories setting. Update the bwopencode comments and the docs.

Assisted-by: opencode:claude-sonnet-5-5

Summary

Replace this with a brief summary of what this PR accomplishes.
For example, what functionality is added/removed, and why.

Description

More details could be added here with background info,
explanation of design choices, open questions for reviewers, etc.

Testing

Summarize how these changes were tested and on which hosts.
Include or link to test output here.
DO NOT POST SENSITIVE INFORMATION HERE.

Checklist

  • Edit the sections above with relevant information
  • Update in-repo documentation, if applicable
  • Run pixi run -e dev lint locally (or wait for CI) — see .github/workflows/lint.yml
  • Test the changes on a representative managed host (or runansible deployment)

Skills are now published to /nsls2/agents/skills and the old per-host clone is no longer deployed. bwcopilot mounts the new tree read-only; list it in Copilot's skillDirectories setting. Update the bwopencode comments and the docs.

Assisted-by: opencode:claude-sonnet-5-5
@stuwilkins
Stuart B. Wilkins (stuwilkins) marked this pull request as ready for review October 21, 2026 12:44
Copilot AI balanced review requested due to automatic review settings October 21, 2026 12:44

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟢 Approval recommended

The focused path migration preserves existing mount protections, with no unresolved findings.

No open issues 🎉

What changed in this PR

Updates bwcopilot to use the deployment’s new shared skills location while preserving optional, read-only mounting.

Changes:

  • Mounts /nsls2/agents/skills instead of the retired skills path.
  • Documents Copilot configuration and updates OpenCode wrapper comments.
File Description
docs/​bw-wrappers.md Documents shared skills paths and Copilot configuration.
bin/​bwopencode2 Updates the shared skills path in a comment.
bin/​bwopencode Updates the shared skills path in a comment.
bin/​bwcopilot Switches the read-only skills mount to the new path.

Review effort: Balanced

@tacaswell
Thomas A Caswell (tacaswell) merged commit ec503bb into NSLS2:main Oct 22, 2026
1 check 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

Development

Successfully merging this pull request may close these issues.

3 participants