Skip to content

Commit 9b40ee7

Browse files
fix: address code review comments - clarify HTTP transport vs QueryAST naming in dispatcher
Agent-Logs-Url: https://github.com/objectstack-ai/spec/sessions/d243fae5-51a7-4440-926f-46f040744cb0 Co-authored-by: xuyushun441-sys <255036401+xuyushun441-sys@users.noreply.github.com>
1 parent 9b4212a commit 9b40ee7

1 file changed

Lines changed: 5 additions & 1 deletion

File tree

packages/runtime/src/http-dispatcher.ts

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -595,7 +595,11 @@ export class HttpDispatcher {
595595
// Spec-canonical keys.
596596
const normalized: Record<string, unknown> = { ...query };
597597

598-
// filter/filters → where (@deprecated filter is still the canonical HTTP param)
598+
// filter/filters → where
599+
// Note: `filter` is the canonical HTTP *transport* parameter name
600+
// (see HttpFindQueryParamsSchema). It is normalized here to the
601+
// canonical *QueryAST* field name `where` before broker dispatch.
602+
// `filters` (plural) is a deprecated alias for `filter`.
599603
if (normalized.filter != null || normalized.filters != null) {
600604
normalized.where = normalized.where ?? normalized.filter ?? normalized.filters;
601605
delete normalized.filter;

0 commit comments

Comments
 (0)