Skip to content

fix(dataset-registry-client): pass search criteria via ldkit's where option#205

Merged
ddeboer merged 2 commits intomainfrom
fix/dataset-registry-client-search-criteria
Mar 6, 2026
Merged

fix(dataset-registry-client): pass search criteria via ldkit's where option#205
ddeboer merged 2 commits intomainfrom
fix/dataset-registry-client-search-criteria

Conversation

@ddeboer
Copy link
Member

@ddeboer ddeboer commented Mar 6, 2026

Summary

  • Wrap caller's search criteria in { where } when calling ldkit's find() and count() — previously the criteria object was spread at the top level, causing caller-provided take/skip to be silently overridden
  • Simplify the public API: callers now pass search criteria directly (e.g. { distribution: { mediaType: ... } }) instead of wrapping in { where: { ... } } — the client handles the ldkit-specific structure internally
  • Export a typed SearchCriteria alias (based on ldkit's SchemaSearchInterface) instead of accepting untyped object

ddeboer added 2 commits March 6, 2026 09:19
…` option

- Wrap caller's search criteria in `{ where }` when calling ldkit's
  `find()` and `count()`, which previously spread args at the top level
  where `take`/`skip` from callers would be silently ignored
- Accept search criteria directly instead of requiring callers to wrap
  in `{ where }` — the client handles the ldkit-specific structure
- Export typed `SearchCriteria` instead of using untyped `object`
@ddeboer ddeboer enabled auto-merge (squash) March 6, 2026 08:22
@ddeboer ddeboer merged commit bae8b72 into main Mar 6, 2026
2 checks passed
@ddeboer ddeboer deleted the fix/dataset-registry-client-search-criteria branch March 6, 2026 08:24
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