fix: make parseQuery parameters optional with defaults

- Change query parameter to optional (query?: Record<string, unknown> | null)
- Add default empty string for idProp parameter (idProp: string = '')
- Allows calling parseQuery() without arguments (returns null)
- Maintains backward compatibility and proper error handling

Author: marshallswain

src/utils/parse-query.ts

@@ -118,8 +118,8 @@ const specialQueryHandlers: Record<string, QueryHandler> = {
  * @returns Parsed Elasticsearch query or null if empty
  */
 export function parseQuery(
-  query: Record<string, unknown>,
-  idProp: string,
+  query?: Record<string, unknown> | null,
+  idProp: string = '',
   maxDepth: number = 50,
   currentDepth: number = 0
 ): ESQuery | null {

test/utils/parse-query.ts

@@ -1,6 +1,5 @@
 import { expect } from 'chai'
 import errors from '@feathersjs/errors'
-
 import { parseQuery } from '../../lib/utils/index.js'
 
 export default function parseQueryTests() {