Add support for local imports in hooks #2031
Draft
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Render the whole hook directory to a temporary folder before executing hooks so that it's possible to use local imports in hooks.
I kindly request comments regarding this implementation before finalizing this PR by adding tests and documentation.
A few notes about the implementation:
Pre_prompt_hook
is still executed from the original location.A few questions that I have:
run_hook_from_repo_dir()
run_hook()
run_script_with_context()
post_gen_hook
by thepre_gen_hook
?PYTHONPATH
during hook execution so that hooks can access packages from the repo directory?Resolves #824