feat: implement ValidationMiddleware at priority 15 for action pipeline #325

Merged
freemo merged 1 commit from feature/m1-validation-middleware into master 2026-03-16 20:08:42 +00:00
Owner

Summary

  • Implements ValidationMiddleware that integrates input validation into the action pipeline at priority 15
  • Provides pass-through for commands without declared parameters (backward compatibility)
  • Runs ValidationPipeline for commands using ParameterDSL, attaching ValidatedParams to context on success or short-circuiting with player error output on failure

Files Added

  • lib/aethyr/core/input_handlers/validation/validation_middleware.rb
  • tests/unit/validation_middleware.feature (15 scenarios)
  • tests/unit/step_definitions/validation_middleware_steps.rb
  • tests/integration/validation_middleware_integration.feature (14 scenarios)
  • tests/integration/step_definitions/validation_middleware_integration_steps.rb

Test Results

  • Unit tests: 15 scenarios passed
  • Integration tests: 14 scenarios passed
  • RuboCop: No offenses detected

Closes #297

## Summary - Implements ValidationMiddleware that integrates input validation into the action pipeline at priority 15 - Provides pass-through for commands without declared parameters (backward compatibility) - Runs ValidationPipeline for commands using ParameterDSL, attaching ValidatedParams to context on success or short-circuiting with player error output on failure ## Files Added - `lib/aethyr/core/input_handlers/validation/validation_middleware.rb` - `tests/unit/validation_middleware.feature` (15 scenarios) - `tests/unit/step_definitions/validation_middleware_steps.rb` - `tests/integration/validation_middleware_integration.feature` (14 scenarios) - `tests/integration/step_definitions/validation_middleware_integration_steps.rb` ## Test Results - Unit tests: 15 scenarios passed - Integration tests: 14 scenarios passed - RuboCop: No offenses detected Closes #297
freemo force-pushed feature/m1-validation-middleware from dd814ccf49 to a9efef47a6 2026-03-16 16:58:06 +00:00 Compare
freemo force-pushed feature/m1-validation-middleware from a9efef47a6 to c7050bf668 2026-03-16 17:32:48 +00:00 Compare
freemo force-pushed feature/m1-validation-middleware from c7050bf668 to b6c8d411bb 2026-03-16 20:08:33 +00:00 Compare
freemo merged commit b6c8d411bb into master 2026-03-16 20:08:42 +00:00
freemo deleted branch feature/m1-validation-middleware 2026-03-16 20:08:43 +00:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: aethyr/Aethyr#325
No description provided.