Refactor ParamsScope and Parameters DSL to use named kwargs

[ericproulx]

Summary

• Replace positional/hash-based method signatures with named keyword arguments throughout ParamsScope and Parameters DSL
• ParamsScope#initialize now uses api:, element: nil, element_renamed: nil, parent: nil, optional: false, type: nil, group: nil, dependent_on: nil
• new_scope, new_lateral_scope, push_declared_params, validate_attributes all converted to named kwargs
• new_scope first argument renamed from attrs to element — callers now pass attrs.first directly, removing the double attrs.first lookup inside the method
• new_group_scope first argument renamed from attrs to group — callers now pass the group hash directly instead of wrapping it in an array
• require_required_and_optional_fields and require_optional_fields now declare using:, except: nil explicitly instead of accepting an opaque opts hash — callers pass only the relevant keys
• with converted from a positional hash to **opts keyword arguments — passing an explicit hash literal with({...}) will now raise ArgumentError (see UPGRADING.md)
• Method comments updated to reflect new signatures throughout

Benefits

• Method interfaces are self-documenting — required vs optional params are clear from the signature
• Eliminates implicit hash contracts where callers had to know which keys were consumed
• Prevents accidental passing of unrelated keys into methods

🤖 Generated with Claude Code

[github-actions]

Danger Report

No issues found.

View run