Phase 2 of API improvements - #93

Open

moltude wants to merge 16 commits into

mainfrom

fix/api-phase-2

Commits on Apr 21, 2026

Phase 2 of API improvements

moltude
authored and
DominicBM
committed
e2a4508
Update src/main/scala/dpla/api/v2/search/queryBuilders/QueryBuilder.scala

authored and
DominicBM
committed
31c58ea
Update src/main/scala/dpla/api/v2/search/paramValidators/ParamValidator.scala

authored and
DominicBM
committed
573389a
Fix tests and misspellings

moltude
authored and
DominicBM
committed
490ef38
Fix parameter validation to reject over-limit values instead of clamping (#94)

authored and
DominicBM
committed
ad58d6d
Fix broken test

moltude
authored and
DominicBM
committed
a4c778a
Fix facet guard bypass via empty filter values

DominicBM
and
claude
committed
3c4f02e
Address CodeRabbit: normalize empty facets/fields, fix error message bounds

DominicBM
and
claude
committed
99732bc
Fix test failure: use exists(_.nonEmpty) for facets guard, keep filter for fields

DominicBM
and
claude
committed
d7524b1
fix: address CodeRabbit review — ConcurrencyLimiter, fetch limit, filter, op propagation

DominicBM
and
claude
committed
9bc8091
fix: restore exact hit counts and clean up query builder

DominicBM
and
claude
committed
5dc77df
fix: update track_total_hits test to expect true instead of 10000

DominicBM
and
claude
committed
3a2583d
fix: remove dead pagination depth guard; add OR operator multi_match test

DominicBM
and
claude
committed
e98d96a
fix: remove leftover unused pagination variables

DominicBM
and
claude
committed
214fce2
test: rename outdated pagination test descriptions to reflect rejection behavior

DominicBM
and
claude
committed
9407a71
fix: scope facet-only guard to ebook validator, preserving item endpoint compatibility

DominicBM
and
claude
committed
815dc55

Navigation Menu

Phase 2 of API improvements - #93

Phase 2 of API improvements#93
moltude wants to merge 16 commits into
mainfrom
fix/api-phase-2

Commits on Apr 21, 2026

Phase 2 of API improvements

Update src/main/scala/dpla/api/v2/search/queryBuilders/QueryBuilder.scala

Update src/main/scala/dpla/api/v2/search/paramValidators/ParamValidator.scala

Fix tests and misspellings

Fix parameter validation to reject over-limit values instead of clamping (#94)

Fix broken test

Fix facet guard bypass via empty filter values

Address CodeRabbit: normalize empty facets/fields, fix error message bounds

Fix test failure: use exists(_.nonEmpty) for facets guard, keep filter for fields

fix: address CodeRabbit review — ConcurrencyLimiter, fetch limit, filter, op propagation

fix: restore exact hit counts and clean up query builder

fix: update track_total_hits test to expect true instead of 10000

fix: remove dead pagination depth guard; add OR operator multi_match test

fix: remove leftover unused pagination variables

test: rename outdated pagination test descriptions to reflect rejection behavior

fix: scope facet-only guard to ebook validator, preserving item endpoint compatibility