fix: Validate required method params

codegen/layouts/partials/route-class-endpoint-export.hbs

@@ -1,4 +1,4 @@
-export type {{parametersTypeName}} = {{> request-object parameters=parameters}}
+export type {{parametersTypeName}} = {{#if requiresAtLeastOneParameter}}RequireAtLeastOne<{{/if}}{{> request-object parameters=parameters}}{{#if requiresAtLeastOneParameter}}>{{/if}}
 
 /**
  * @deprecated Use {{requestTypeName}} instead.

codegen/layouts/partials/route-class-endpoint.hbs

@@ -2,6 +2,8 @@
 {{methodName}}
 {{> endpont-method-signature }}
 {
+  assertValidRequestParameters(parameters, '{{path}}', {{hasRequiredParameters}})
+
   return new SeamHttpRequest(this, {
     pathname: '{{path}}',
     method: '{{method}}',

codegen/layouts/route.hbs

@@ -5,6 +5,13 @@
 
 {{> route-imports }}
 
+{{#if endpoints}}
+import { assertValidRequestParameters } from 'lib/request-parameters.js'
+{{/if}}
+{{#if needsRequireAtLeastOneImport}}
+import type { RequireAtLeastOne } from 'lib/request-parameters.js'
+{{/if}}
+
 export class {{className}} {
   {{> route-class-methods }}
 

codegen/lib/layouts/route.ts

@@ -10,6 +10,7 @@ export interface RouteLayoutContext {
   subroutes: SubrouteLayoutContext[]
   skipClientSessionImport: boolean
   needsActionAttemptsImport: boolean
+  needsRequireAtLeastOneImport: boolean
   resourceTypeImports: ResourceTypeImport[]
 }
 
@@ -35,6 +36,8 @@ export interface EndpointLayoutContext {
   returnsActionAttempt: boolean
   returnsVoid: boolean
   isOptionalParamsOk: boolean
+  hasRequiredParameters: boolean
+  requiresAtLeastOneParameter: boolean
   parameters: Parameter[]
   responseIsList: boolean
   responseResourceTypeName: string
@@ -64,6 +67,10 @@ export const setRouteLayoutContext = (
     node.path !== '/action_attempts' &&
     'endpoints' in node &&
     node.endpoints.some(isActionAttemptEndpoint)
+  file.needsRequireAtLeastOneImport =
+    node != null &&
+    'endpoints' in node &&
+    node.endpoints.some(requiresAtLeastOneParameter)
   file.resourceTypeImports =
     node != null && 'endpoints' in node
       ? [
@@ -140,9 +147,9 @@ export const getEndpointLayoutContext = (
     responseTypeName: `${prefix}Response`,
     optionsTypeName: `${prefix}Options`,
     requestTypeName: `${prefix}Request`,
-    isOptionalParamsOk: endpoint.request.parameters.every(
-      (parameter) => !parameter.isRequired,
-    ),
+    isOptionalParamsOk: !endpoint.request.hasRequiredParameters,
+    hasRequiredParameters: endpoint.request.hasRequiredParameters,
+    requiresAtLeastOneParameter: requiresAtLeastOneParameter(endpoint),
     parameters: endpoint.request.parameters,
     responseIsList: endpoint.response.responseType === 'resource_list',
     responseResourceTypeName:
@@ -155,6 +162,10 @@ export const getEndpointLayoutContext = (
   }
 }
 
+const requiresAtLeastOneParameter = (endpoint: Endpoint): boolean =>
+  endpoint.request.hasRequiredParameters &&
+  endpoint.request.parameters.every(({ isRequired }) => !isRequired)
+
 const isActionAttemptEndpoint = (endpoint: Endpoint): boolean =>
   endpoint.response.responseType === 'resource' &&
   endpoint.response.resourceType === 'action_attempt'

src/lib/request-parameters.ts

@@ -0,0 +1,33 @@
+export type RequireAtLeastOne<
+  T,
+  Keys extends keyof T = keyof T,
+> = Keys extends keyof T
+  ? Required<Pick<T, Keys>> & Partial<Omit<T, Keys>>
+  : never
+
+export const assertValidRequestParameters = (
+  parameters: unknown,
+  path: string,
+  hasRequiredParameters: boolean,
+): void => {
+  if (parameters === undefined) {
+    if (hasRequiredParameters) {
+      throw new TypeError(`Parameters are required for ${path}`)
+    }
+    return
+  }
+
+  if (
+    parameters === null ||
+    typeof parameters !== 'object' ||
+    Array.isArray(parameters)
+  ) {
+    throw new TypeError(`Parameters for ${path} must be an object`)
+  }
+
+  if (hasRequiredParameters && Object.keys(parameters).length === 0) {
+    throw new TypeError(
+      `Parameters for ${path} must contain at least one property`,
+    )
+  }
+}