Remove Partial wrapper and if statements from registerRouteHandlersByTag

- Make ServerForTag interface properties required (remove optional marker)
- Remove Partial<> wrapper from overload signatures
- Remove if statements from implementation since all methods are now required

Co-authored-by: jasonblanchard <1238532+jasonblanchard@users.noreply.github.com>

Author: Copilot

examples/docs/gen/server.ts

@@ -76,30 +76,26 @@ export function registerRouteHandlers<Req, Res>(server: Server<Req, Res>): Route
 export type Tag = null;
 
 export interface ServerForUntagged<Req = unknown, Res = unknown> {
-  makePetSpeak?: (args: MakePetSpeakArgs<Req, Res>) => MakePetSpeakResult;
-  uhoh?: (args: UhohArgs<Req, Res>) => UhohResult;
+  makePetSpeak: (args: MakePetSpeakArgs<Req, Res>) => MakePetSpeakResult;
+  uhoh: (args: UhohArgs<Req, Res>) => UhohResult;
 }
 
-export function registerRouteHandlersByTag<Req, Res>(tag: null, server: Partial<ServerForUntagged<Req, Res>>): Route[];
+export function registerRouteHandlersByTag<Req, Res>(tag: null, server: ServerForUntagged<Req, Res>): Route[];
 export function registerRouteHandlersByTag<Req, Res>(tag: Tag, server: Partial<Server<Req, Res>>): Route[] {
   const routes: Route[] = [];
 
   switch (tag) {
     case null:
-      if (server.makePetSpeak) {
-        routes.push({
-          method: "post",
-          path: "/speak/{petId}",
-          handler: server.makePetSpeak as Route["handler"],
-        });
-      }
-      if (server.uhoh) {
-        routes.push({
-          method: "get",
-          path: "/uhoh",
-          handler: server.uhoh as Route["handler"],
-        });
-      }
+      routes.push({
+        method: "post",
+        path: "/speak/{petId}",
+        handler: server.makePetSpeak as Route["handler"],
+      });
+      routes.push({
+        method: "get",
+        path: "/uhoh",
+        handler: server.uhoh as Route["handler"],
+      });
       break;
   }
 

examples/kitchensink/gen/server.ts

@@ -254,62 +254,50 @@ export function registerRouteHandlers<Req, Res>(server: Server<Req, Res>): Route
 export type Tag = null;
 
 export interface ServerForUntagged<Req = unknown, Res = unknown> {
-  listPets?: (args: ListPetsArgs<Req, Res>) => ListPetsResult;
-  getPetById?: (args: GetPetByIdArgs<Req, Res>) => GetPetByIdResult;
-  updatePetWithForm?: (args: UpdatePetWithFormArgs<Req, Res>) => UpdatePetWithFormResult;
-  mixedContentTypes?: (args: MixedContentTypesArgs<Req, Res>) => MixedContentTypesResult;
-  getPetImage?: (args: GetPetImageArgs<Req, Res>) => GetPetImageResult;
-  getPetWebpage?: (args: GetPetWebpageArgs<Req, Res>) => GetPetWebpageResult;
+  listPets: (args: ListPetsArgs<Req, Res>) => ListPetsResult;
+  getPetById: (args: GetPetByIdArgs<Req, Res>) => GetPetByIdResult;
+  updatePetWithForm: (args: UpdatePetWithFormArgs<Req, Res>) => UpdatePetWithFormResult;
+  mixedContentTypes: (args: MixedContentTypesArgs<Req, Res>) => MixedContentTypesResult;
+  getPetImage: (args: GetPetImageArgs<Req, Res>) => GetPetImageResult;
+  getPetWebpage: (args: GetPetWebpageArgs<Req, Res>) => GetPetWebpageResult;
 }
 
-export function registerRouteHandlersByTag<Req, Res>(tag: null, server: Partial<ServerForUntagged<Req, Res>>): Route[];
+export function registerRouteHandlersByTag<Req, Res>(tag: null, server: ServerForUntagged<Req, Res>): Route[];
 export function registerRouteHandlersByTag<Req, Res>(tag: Tag, server: Partial<Server<Req, Res>>): Route[] {
   const routes: Route[] = [];
 
   switch (tag) {
     case null:
-      if (server.listPets) {
-        routes.push({
-          method: "get",
-          path: "/pets",
-          handler: server.listPets as Route["handler"],
-        });
-      }
-      if (server.getPetById) {
-        routes.push({
-          method: "get",
-          path: "/pet/{petId}",
-          handler: server.getPetById as Route["handler"],
-        });
-      }
-      if (server.updatePetWithForm) {
-        routes.push({
-          method: "post",
-          path: "/pet/{petId}",
-          handler: server.updatePetWithForm as Route["handler"],
-        });
-      }
-      if (server.mixedContentTypes) {
-        routes.push({
-          method: "post",
-          path: "/pet/{petId}/mixed-content-types",
-          handler: server.mixedContentTypes as Route["handler"],
-        });
-      }
-      if (server.getPetImage) {
-        routes.push({
-          method: "get",
-          path: "/pet/{petId}/image",
-          handler: server.getPetImage as Route["handler"],
-        });
-      }
-      if (server.getPetWebpage) {
-        routes.push({
-          method: "get",
-          path: "/pet/{petId}/webpage",
-          handler: server.getPetWebpage as Route["handler"],
-        });
-      }
+      routes.push({
+        method: "get",
+        path: "/pets",
+        handler: server.listPets as Route["handler"],
+      });
+      routes.push({
+        method: "get",
+        path: "/pet/{petId}",
+        handler: server.getPetById as Route["handler"],
+      });
+      routes.push({
+        method: "post",
+        path: "/pet/{petId}",
+        handler: server.updatePetWithForm as Route["handler"],
+      });
+      routes.push({
+        method: "post",
+        path: "/pet/{petId}/mixed-content-types",
+        handler: server.mixedContentTypes as Route["handler"],
+      });
+      routes.push({
+        method: "get",
+        path: "/pet/{petId}/image",
+        handler: server.getPetImage as Route["handler"],
+      });
+      routes.push({
+        method: "get",
+        path: "/pet/{petId}/webpage",
+        handler: server.getPetWebpage as Route["handler"],
+      });
       break;
   }