This repository was archived by the owner on Jul 15, 2026. It is now read-only.
Repository navigation
refactor(hooks): move blast radius guidance from PreToolUse to sessio… - #11
Merged
Merged
Conversation
…n start Move the Edit blast radius check from a per-call PreToolUse prompt hook to the session start context. This is more efficient (injected once vs on every Edit call) and the guidance is static behavioral instruction that doesn't need runtime parameter inspection. - Remove PreToolUse Edit matcher and prompt hook - Delete hooks/pre-tool-use-edit-blast-radius/ - Add blast radius guidance to session start base_context Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
ghost
added
the 
|
This is a good refactoring that improves efficiency. Moving the static blast radius guidance from a per-call hook to the session start context is a sensible change that reduces redundant operations without losing functionality. The change is well-described and justified. Attached by run j-32261132-4c0b-4150-bdae-53927cb28fa3 |
Nihasa350
approved these changes
Feb 27, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
4 participants
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.
Overview
This merge request refactors the "blast radius" guidance by moving it from a
PreToolUsehook to the session start context. Previously, this check was triggered on everyEdittool call. By relocating it to the session start, the guidance is injected only once, improving efficiency and centralizing static behavioral instructions.Technical Details
Files Changed:
.claude-plugin/plugin.jsonChange Type: Refactor
Context:
PreToolUsehook prompted the user to consider the impact of changes in critical integration or infrastructure paths by querying OpenTrace before eachEdittool execution.session-startscript. This script proactively queries OpenTrace when a user intends to edit files in shared paths and automatically calls the most useful follow-up tool after receiving the results.Specific Changes:
PreToolUseEdit matcher and its associated prompt hook.hooks/pre-tool-use-edit-blast-radius/directory.session-startbase context.Review Summary
Impact Assessment
Edittool.Attached by run j-32261132-4c0b-4150-bdae-53927cb28fa3