Skip to content

chore: Add copilot instructions and skills for building and testing plugins#1303

Open
mofojed wants to merge 3 commits intodeephaven:mainfrom
mofojed:copilot-skills
Open

chore: Add copilot instructions and skills for building and testing plugins#1303
mofojed wants to merge 3 commits intodeephaven:mainfrom
mofojed:copilot-skills

Conversation

@mofojed
Copy link
Member

@mofojed mofojed commented Feb 12, 2026

Tested by running the following prompts:

  1. "Run tests for deephaven.ui"
  • Without the skills, keeps trying to run using pytest, with the wrong .venv, or in the wrong directory
  • With the skills, it runs the tests using tox correctly the first time
  1. "Build deephaven.ui and start a server"
  • Sometimes it would start up fine, other times it would end up running npm run docker instead
  • With the skill, it sources the .venv and runs plugin_builder
  1. "Run the JS tests for plotly-express"
  • It took a while, but did eventually run them correctly without the skill
  • With the skills it ran them faster

@mofojed mofojed requested a review from dsmmcken February 12, 2026 18:10
@mofojed mofojed self-assigned this Feb 12, 2026
mofojed added a commit to mofojed/deephaven-plugins that referenced this pull request Feb 12, 2026
- Separated into its own PR: deephaven#1303
Copy link
Contributor

@dsmmcken dsmmcken left a comment

Choose a reason for hiding this comment

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

  1. What testing did you do? I assume you tried asking for various things and the skills helped?

  2. I think copilot-instructions.md should be an AGENTS.md, that seems to be the preferred approach now no?

- Just use an AGENTS.md file
- Update the skills so that the source .venv is a general rule for launching a terminal
- Update the settings.json so that we allow tox to be run automatically
@mofojed mofojed requested a review from dsmmcken February 13, 2026 19:23
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.

2 participants