CR: avoiding mutation of the argument in nest() method.

RobinTail · RobinTail · commit ba7951c53424 · 2025-09-17T15:11:32.000+02:00
diff --git a/express-zod-api/src/endpoint.ts b/express-zod-api/src/endpoint.ts
@@ -44,7 +44,7 @@ export type Handler<IN, OUT, OPT> = (params: {
 export abstract class AbstractEndpoint {
   /** @desc Enables nested routes within the path assigned to the subject */
   public nest(routing: Routing): Routing {
-    return Object.assign(routing, { "": this });
+    return { ...routing, "": this };
   }
   /** @desc Marks the route as deprecated (makes a copy of the endpoint) */
   public abstract deprecated(): this;

Original file line number	Diff line number	Diff line change
`@@ -44,7 +44,7 @@ export type Handler<IN, OUT, OPT> = (params: {`
`44`	`44`	`export abstract class AbstractEndpoint {`
`45`	`45`	`/** @desc Enables nested routes within the path assigned to the subject */`
`46`	`46`	`public nest(routing: Routing): Routing {`
`47`		`- return Object.assign(routing, { "": this });`
	`47`	`+ return { ...routing, "": this };`
`48`	`48`	`}`
`49`	`49`	`/** @desc Marks the route as deprecated (makes a copy of the endpoint) */`
`50`	`50`	`public abstract deprecated(): this;`