Skip to content

feat: validation + tests for S1-02#1

Open
medyll wants to merge 4 commits intomasterfrom
feature/s1-02-validation
Open

feat: validation + tests for S1-02#1
medyll wants to merge 4 commits intomasterfrom
feature/s1-02-validation

Conversation

@medyll
Copy link
Owner

@medyll medyll commented Mar 11, 2026

This PR adds server-side input validation for IDQL/REST requests, unit tests for validation errors (missing scheme, invalid limit/page/where/method), and a safe fallback in the query layer used during unit tests.

Files:

  • web/bin/engine/Idae/Api/IdaeApiRest.php
  • web/bin/engine/Idae/Api/IdaeApiParser.php
  • web/bin/engine/Idae/Query/IdaeQuery.php
  • web/bin/tests/unit/ValidateQueryTest.php
  • bmad/artifacts/stories/S1-02.md
  • bmad/status.yaml
  • bmad/dashboard.md

All unit tests pass in the project Docker environment: OK (9 tests, 41 assertions).

Notes:

  • Parser now accepts key from idql payload so it can be validated;
  • returns raw results when scheme metadata is unavailable (test fallback).

Ready for review.

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.

1 participant