Skip to content

fix: copy entire deploy/ dir to hooks, not just commands.py#755

Merged
prathamesh0 merged 1 commit into
mainfrom
copy-full-deploy-dir-to-hooks
May 27, 2026
Merged

fix: copy entire deploy/ dir to hooks, not just commands.py#755
prathamesh0 merged 1 commit into
mainfrom
copy-full-deploy-dir-to-hooks

Conversation

@prathamesh0

Copy link
Copy Markdown
Collaborator
  • _copy_hooks was hardcoded to only copy commands.py into the deployment's hooks/ dir
  • Any sibling file read via Path(__file__).parent at start() time would silently not exist
  • Now copies all files from deploy/; the multi-plugin commands_N.py indexing is preserved

…s.py

Sidecar files (e.g. provision.sh) referenced via Path(__file__).parent in
commands.py were silently absent from {deploy_dir}/hooks/ at start() time,
causing FileNotFoundError at runtime.

Replace the commands.py-only copy with a unified loop over all files in each
plugin's deploy/ directory. The commands.py multi-repo indexing convention
(commands_0.py, commands_1.py, ...) is preserved; all other files are copied
as-is (last writer wins on name collision across plugins).

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@prathamesh0 prathamesh0 merged commit fe77540 into main May 27, 2026
7 checks passed
@prathamesh0 prathamesh0 deleted the copy-full-deploy-dir-to-hooks branch May 27, 2026 11:11
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