Skip to content

Make hook dry-runs truthful for install and uninstall - #4983

Merged
Widthdom merged 4 commits into
mainfrom
fix-issue4883
Jul 29, 2026
Merged

Make hook dry-runs truthful for install and uninstall#4983
Widthdom merged 4 commits into
mainfrom
fix-issue4883

Conversation

@Widthdom

Copy link
Copy Markdown
Owner

Summary

  • share install and uninstall planning so successful previews return status: dry_run, filesystem_mutation: false, explicit state/action fields, and hash/provenance-bearing file changes
  • add uninstall previews for managed-block removal, chained-hook restoration, forced removal/restoration, blocked conflicts, and idempotent no-op states
  • preserve surrounding hook bytes and existing POSIX modes, including non-UTF-8 content, CR-only line endings, UTF BOM variants, and bounded hashing for oversized hooks

Validation

Documentation and changelog

  • updated the English and Japanese hook sections in USER_GUIDE.md and TESTING_GUIDE.md
  • added changelog.d/unreleased/4883.fixed.md

Follow-ups

Fixes #4883

@Widthdom
Widthdom marked this pull request as ready for review July 29, 2026 13:30
@Widthdom
Widthdom merged commit d2c8d1c into main Jul 29, 2026
12 checks passed
@Widthdom
Widthdom deleted the fix-issue4883 branch July 29, 2026 13:30
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.

Complete truthful dry-run support for hook install and uninstall

1 participant