Config Updates

[GrantBirki]

No description provided.

[Copilot]

Pull Request Overview

This PR renames the legacy handler_dir configuration to handler_plugin_dir across tests, schema, loader, and documentation.

• Updated specs to use handler_plugin_dir
• Removed handler_dir from the Dry schema and environment mappings
• Revised docs to reflect handler_plugin_dir

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File	Description
spec/unit/lib/hooks/core/config_validator_spec.rb	Updated tests to replace handler_dir with handler_plugin_dir
lib/hooks/core/config_validator.rb	Removed handler_dir from the global config schema
lib/hooks/core/config_loader.rb	Dropped legacy HOOKS_HANDLER_DIR environment mapping
docs/design.md	Updated configuration references to handler_plugin_dir

Comments suppressed due to low confidence (3)

lib/hooks/core/config_loader.rb:129

• Removing the legacy HOOKS_HANDLER_DIR mapping breaks backward compatibility; consider mapping it to handler_plugin_dir with a deprecation warning.

        env_mappings = {

docs/design.md:350

• [nitpick] The term plugin_dir is not defined in the schema—clarify whether this refers to auth_plugin_dir, lifecycle_plugin_dir, or both.

* Plugin classes are loaded from `plugin_dir` and registered based on class inheritance

lib/hooks/core/config_loader.rb:130

• Add a unit test to verify that HOOKS_HANDLER_PLUGIN_DIR is correctly loaded into handler_plugin_dir in the environment config loader.

          "HOOKS_HANDLER_PLUGIN_DIR" => :handler_plugin_dir,