refactor: use Object.fromEntries for headers conversion

corvid-agent · claude · corvid-agent · commit 0be73a4568cf · 2026-02-12T22:18:19.000-07:00
Address review feedback: simplify headers extraction by using
Object.fromEntries() instead of manual forEach loop.

Co-Authored-By: Claude Opus 4.5 &lt;noreply@anthropic.com&gt;
diff --git a/src/client/transports/rest_transport.ts b/src/client/transports/rest_transport.ts
@@ -340,11 +340,10 @@ export class RestTransport implements Transport {
   }
 
   private static extractHttpDetails(response: Response): A2AErrorHttpDetails {
-    const headers: Record<string, string> = {};
-    response.headers.forEach((value, key) => {
-      headers[key] = value;
-    });
-    return { statusCode: response.status, headers };
+    return {
+      statusCode: response.status,
+      headers: Object.fromEntries(response.headers),
+    };
   }
 
   private static mapToError(error: RestErrorResponse, httpDetails?: A2AErrorHttpDetails): Error {

Original file line number	Diff line number	Diff line change
`@@ -340,11 +340,10 @@ export class RestTransport implements Transport {`
`340`	`340`	`}`
`341`	`341`
`342`	`342`	`private static extractHttpDetails(response: Response): A2AErrorHttpDetails {`
`343`		`- const headers: Record<string, string> = {};`
`344`		`- response.headers.forEach((value, key) => {`
`345`		`- headers[key] = value;`
`346`		`- });`
`347`		`- return { statusCode: response.status, headers };`
	`343`	`+ return {`
	`344`	`+ statusCode: response.status,`
	`345`	`+ headers: Object.fromEntries(response.headers),`
	`346`	`+ };`
`348`	`347`	`}`
`349`	`348`
`350`	`349`	`private static mapToError(error: RestErrorResponse, httpDetails?: A2AErrorHttpDetails): Error {`