fix(client): ensure addOutputText is called on responses.retrieve

Author: dtmeadows

src/resources/responses/responses.ts

@@ -132,10 +132,18 @@ export class Responses extends APIResource {
     query: ResponseRetrieveParams | undefined = {},
     options?: RequestOptions,
   ): APIPromise<Response> | APIPromise<Stream<ResponseStreamEvent>> {
-    return this._client.get(path`/responses/${responseID}`, {
-      query,
-      ...options,
-      stream: query?.stream ?? false,
+    return (
+      this._client.get(path`/responses/${responseID}`, {
+        query,
+        ...options,
+        stream: query?.stream ?? false,
+      }) as APIPromise<Response> | APIPromise<Stream<ResponseStreamEvent>>
+    )._thenUnwrap((rsp) => {
+      if ('object' in rsp && rsp.object === 'response') {
+        addOutputText(rsp as Response);
+      }
+
+      return rsp;
     }) as APIPromise<Response> | APIPromise<Stream<ResponseStreamEvent>>;
   }
 

tests/api-resources/responses/responses.test.ts

@@ -17,6 +17,9 @@ describe('resource responses', () => {
     const dataAndResponse = await responsePromise.withResponse();
     expect(dataAndResponse.data).toBe(response);
     expect(dataAndResponse.response).toBe(rawResponse);
+
+    expect(response).toHaveProperty('output_text');
+    expect(typeof response.output_text).toBe('string');
   });
 
   test('retrieve', async () => {
@@ -28,6 +31,9 @@ describe('resource responses', () => {
     const dataAndResponse = await responsePromise.withResponse();
     expect(dataAndResponse.data).toBe(response);
     expect(dataAndResponse.response).toBe(rawResponse);
+
+    expect(response).toHaveProperty('output_text');
+    expect(typeof response.output_text).toBe('string');
   });
 
   test('retrieve: request options and params are passed correctly', async () => {