Skip to content

Plugin structure fixes#2

Open
romeran wants to merge 1 commit intomainfrom
JML-02-Plugin-structure-fixes
Open

Plugin structure fixes#2
romeran wants to merge 1 commit intomainfrom
JML-02-Plugin-structure-fixes

Conversation

@romeran
Copy link
Copy Markdown
Collaborator

@romeran romeran commented Apr 14, 2026

No description provided.

Copy link
Copy Markdown
Collaborator

@davida-jfrog davida-jfrog left a comment

Choose a reason for hiding this comment

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

Approving since we are anyways still working it all out.

Comment thread plugin/hooks/hooks.json
"windows": "powershell -ExecutionPolicy Bypass -File ${CLAUDE_PLUGIN_ROOT}/scripts/ensure-instructions.ps1"
"version": 1,
"hooks": {
"sessionStart": [
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Are you sure its supposed to be lowercase? According to the docs it was correct before as SessionStart.
https://code.visualstudio.com/docs/copilot/customization/hooks#_sessionstart

Wouldnt be suprised if the vscode docs have errors though.

@davida-jfrog
Copy link
Copy Markdown
Collaborator

see #3

If we want the .github format instead of claude then I think it needs to be under .github/plugin/plugin.json.
However if we want to use the CLAUDE_PLUGIN_ROOT we cant use the .github variant.

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