Include HTTP status code in error messages

Author: mattpodwysocki

src/tools/MapboxApiBasedTool.ts

@@ -60,19 +60,19 @@ export abstract class MapboxApiBasedTool<
    * If the response contains a JSON body with a 'message' field, returns that.
    * Otherwise falls back to the response status text.
    * @param response The HTTP response object
-   * @returns The error message string
+   * @returns The error message string in format "statusCode: message"
    */
   protected async getErrorMessage(response: Response): Promise<string> {
     try {
       const errorBody = await response.text();
       const errorJson = JSON.parse(errorBody);
       if (errorJson.message) {
-        return errorJson.message;
+        return `${response.status}: ${errorJson.message}`;
       }
     } catch {
       // If parsing fails, fall back to status text
     }
-    return response.statusText;
+    return `${response.status}: ${response.statusText}`;
   }
 
   /**

test/tools/category-search-tool/CategorySearchTool.test.ts

@@ -114,7 +114,7 @@ describe('CategorySearchTool', () => {
     expect(result.isError).toBe(true);
     expect(result.content[0]).toMatchObject({
       type: 'text',
-      text: 'Category Search API error: Not Found'
+      text: 'Category Search API error: 404: Not Found'
     });
   });
 

test/tools/directions-tool/DirectionsTool.test.ts

@@ -160,7 +160,7 @@ describe('DirectionsTool', () => {
     expect(result.isError).toBe(true);
     expect(result.content[0]).toMatchObject({
       type: 'text',
-      text: 'Directions API error: Not Found'
+      text: 'Directions API error: 404: Not Found'
     });
     assertHeadersSent(mockHttpRequest);
   });

test/tools/matrix-tool/MatrixTool.test.ts

@@ -256,7 +256,7 @@ describe('MatrixTool', () => {
     expect(result.isError).toBe(true);
     expect(result.content[0]).toMatchObject({
       type: 'text',
-      text: 'Matrix API error: Not Found'
+      text: 'Matrix API error: 404: Not Found'
     });
 
     assertHeadersSent(mockHttpRequest);

test/tools/reverse-geocode-tool/ReverseGeocodeTool.test.ts

@@ -224,7 +224,7 @@ describe('ReverseGeocodeTool', () => {
     expect(result.isError).toBe(true);
     expect(result.content[0]).toMatchObject({
       type: 'text',
-      text: 'Reverse Geocode API error: Not Found'
+      text: 'Reverse Geocode API error: 404: Not Found'
     });
   });
 

test/tools/search-and-geocode-tool/SearchAndGeocodeTool.test.ts

@@ -123,7 +123,7 @@ describe('SearchAndGeocodeTool', () => {
     expect(result.isError).toBe(true);
     expect(result.content[0]).toMatchObject({
       type: 'text',
-      text: 'Search API error: Not Found'
+      text: 'Search API error: 404: Not Found'
     });
   });
 

test/tools/static-map-image-tool/StaticMapImageTool.test.ts

@@ -116,7 +116,7 @@ describe('StaticMapImageTool', () => {
     expect(result.isError).toBe(true);
     expect(result.content[0]).toMatchObject({
       type: 'text',
-      text: 'Static Map API error: Not Found'
+      text: 'Static Map API error: 404: Not Found'
     });
   });