fix(angular): address coderabbit review feedback on paramsFilter

- fix(docs): correct HttpParams behavior — does not throw on undefined,
  stringifies to literal 'undefined'; handle it explicitly
- fix(angular): include paramsFilter in resource mutator imports so
  generated .resource.ts files compile when paramsFilter is configured
- fix(core): check Array.isArray(type) && type.includes('object') before
  array recursion in isSchemaNonPrimitive to handle mixed type unions
- fix(options): tighten usesAngularGenerator guard to require both
  client === ANGULAR_QUERY and httpClient === ANGULAR, not just httpClient

Author: The Ult

docs/content/docs/guides/angular.mdx

@@ -783,7 +783,7 @@ export const filterPetstoreParams = (
 The built-in helper is **not** called alongside your filter. In particular,
 you are responsible for:
 
-- stripping `undefined` (Angular's `HttpParams` constructor will throw)
+- stripping `undefined` (Angular's `HttpParams` does not safely represent it for query encoding; handle it explicitly)
 - handling `null` (Angular treats it as the literal string `"null"`)
 - filtering arrays (`new HttpParams({ fromObject: { tags: [{}] } })` is
   invalid)

packages/angular/src/http-resource.ts

@@ -1553,6 +1553,7 @@ const buildHttpResourceExtraFile = (
         verbOption.formData,
         verbOption.formUrlEncoded,
         verbOption.paramsSerializer,
+        verbOption.paramsFilter,
       ].filter(
         (value): value is NonNullable<typeof value> => value !== undefined,
       );

packages/core/src/getters/query-params.ts

@@ -81,6 +81,9 @@ const isSchemaNonPrimitive = (schema: OpenApiSchemaObject): boolean => {
   if (type === 'object') {
     return true;
   }
+  if (Array.isArray(type) && type.includes('object')) {
+    return true;
+  }
   if (type === 'array' || (Array.isArray(type) && type.includes('array'))) {
     const items = (schema as { items?: unknown }).items;
     if (isOpenApiSchemaObject(items)) {
@@ -90,9 +93,6 @@ const isSchemaNonPrimitive = (schema: OpenApiSchemaObject): boolean => {
     // referenced objects must still be flagged so it survives the filter.
     return isRefObject(items);
   }
-  if (Array.isArray(type) && type.includes('object')) {
-    return true;
-  }
 
   const compositions = [
     ...(Array.isArray(schema.oneOf) ? (schema.oneOf as unknown[]) : []),

packages/orval/src/utils/options.ts

@@ -476,7 +476,8 @@ export async function normalizeOptions(
   // but never called, so fail fast instead of emitting a dead import.
   const usesAngularGenerator =
     normalizedOptions.output.client === OutputClient.ANGULAR ||
-    normalizedOptions.output.httpClient === OutputHttpClient.ANGULAR;
+    (normalizedOptions.output.client === OutputClient.ANGULAR_QUERY &&
+      normalizedOptions.output.httpClient === OutputHttpClient.ANGULAR);
   if (normalizedOptions.output.override.paramsFilter && !usesAngularGenerator) {
     throw new Error(
       styleText(