add feedback tools

Author: tilyupo

README.md

@@ -278,7 +278,7 @@ List Mapbox access tokens for the authenticated user with optional filtering and
 
 Access user feedback items from the Mapbox Feedback API. These tools allow you to retrieve and view user-reported issues, suggestions, and feedback about map data, routing, and POI details.
 
-**feedback_list_tool** - List user feedback items with comprehensive filtering, sorting, and pagination options.
+**list_feedback_tool** - List user feedback items with comprehensive filtering, sorting, and pagination options.
 
 **Parameters:**
 
@@ -298,7 +298,7 @@ Access user feedback items from the Mapbox Feedback API. These tools allow you t
 
 **Returns:** Paginated list of feedback items with pagination cursors.
 
-**feedback_get_tool** - Get a single user feedback item by its unique ID.
+**get_feedback_tool** - Get a single user feedback item by its unique ID.
 
 **Parameters:**
 

TOOL_CONFIGURATION.md

@@ -44,8 +44,8 @@ The following tools are available in the Mapbox MCP Devkit Server:
 
 ### Feedback Tools
 
-- `feedback_list_tool` - List user feedback items with filtering and pagination
-- `feedback_get_tool` - Get a single user feedback item by ID
+- `list_feedback_tool` - List user feedback items with filtering and pagination
+- `get_feedback_tool` - Get a single user feedback item by ID
 
 ### Geographic Tools
 

…get-tool/FeedbackGetTool.input.schema.ts …ack-tool/GetFeedbackTool.input.schema.tssrc/tools/feedback-get-tool/FeedbackGetTool.input.schema.ts renamed to src/tools/get-feedback-tool/GetFeedbackTool.input.schema.ts src/tools/feedback-get-tool/FeedbackGetTool.input.schema.ts renamed to src/tools/get-feedback-tool/GetFeedbackTool.input.schema.ts

@@ -4,9 +4,9 @@
 import { z } from 'zod';
 
 /**
- * Input schema for Feedback Get Tool
+ * Input schema for Get Feedback Tool
  */
-export const FeedbackGetInputSchema = z.object({
+export const GetFeedbackInputSchema = z.object({
   feedback_id: z
     .string()
     .uuid()
@@ -20,4 +20,4 @@ export const FeedbackGetInputSchema = z.object({
     )
 });
 
-export type FeedbackGetInput = z.infer<typeof FeedbackGetInputSchema>;
+export type GetFeedbackInput = z.infer<typeof GetFeedbackInputSchema>;

…ols/feedback-get-tool/FeedbackGetTool.ts …ols/get-feedback-tool/GetFeedbackTool.tssrc/tools/feedback-get-tool/FeedbackGetTool.ts renamed to src/tools/get-feedback-tool/GetFeedbackTool.ts src/tools/feedback-get-tool/FeedbackGetTool.ts renamed to src/tools/get-feedback-tool/GetFeedbackTool.ts

@@ -5,20 +5,20 @@ import type { z } from 'zod';
 import { MapboxApiBasedTool } from '../MapboxApiBasedTool.js';
 import type { CallToolResult } from '@modelcontextprotocol/sdk/types.js';
 import type { HttpRequest } from '../../utils/types.js';
-import { FeedbackGetInputSchema } from './FeedbackGetTool.input.schema.js';
+import { GetFeedbackInputSchema } from './GetFeedbackTool.input.schema.js';
 import { FeedbackGetResponseSchema } from '../feedback.schema.js';
 import type { FeedbackItem } from '../feedback.schema.js';
 
 // API Documentation: https://docs.mapbox.com/api/feedback/
 
-export class FeedbackGetTool extends MapboxApiBasedTool<
-  typeof FeedbackGetInputSchema
+export class GetFeedbackTool extends MapboxApiBasedTool<
+  typeof GetFeedbackInputSchema
 > {
-  name = 'feedback_get_tool';
+  name = 'get_feedback_tool';
   description =
     'Get a single user feedback item from the Mapbox Feedback API by its unique ID. Use this tool to retrieve detailed information about a specific user-reported issue, suggestion, or feedback about map data, routing, or POI details. Requires user-feedback:read scope on the access token.';
   annotations = {
-    title: 'Feedback Get Tool',
+    title: 'Get Feedback Tool',
     readOnlyHint: true,
     destructiveHint: false,
     idempotentHint: true,
@@ -27,7 +27,7 @@ export class FeedbackGetTool extends MapboxApiBasedTool<
 
   constructor(params: { httpRequest: HttpRequest }) {
     super({
-      inputSchema: FeedbackGetInputSchema,
+      inputSchema: GetFeedbackInputSchema,
       httpRequest: params.httpRequest
     });
   }
@@ -55,7 +55,7 @@ export class FeedbackGetTool extends MapboxApiBasedTool<
   }
 
   protected async execute(
-    input: z.infer<typeof FeedbackGetInputSchema>,
+    input: z.infer<typeof GetFeedbackInputSchema>,
     accessToken: string
   ): Promise<CallToolResult> {
     const url = new URL(

…st-tool/FeedbackListTool.input.schema.ts …ck-tool/ListFeedbackTool.input.schema.tssrc/tools/feedback-list-tool/FeedbackListTool.input.schema.ts renamed to src/tools/list-feedback-tool/ListFeedbackTool.input.schema.ts src/tools/feedback-list-tool/FeedbackListTool.input.schema.ts renamed to src/tools/list-feedback-tool/ListFeedbackTool.input.schema.ts

@@ -25,9 +25,9 @@ export const feedbackStatusSchema = z.enum([
 ]);
 
 /**
- * Input schema for Feedback List Tool
+ * Input schema for List Feedback Tool
  */
-export const FeedbackListInputSchema = z.object({
+export const ListFeedbackInputSchema = z.object({
   feedback_ids: z
     .array(z.string().uuid())
     .optional()
@@ -121,4 +121,4 @@ export const FeedbackListInputSchema = z.object({
     )
 });
 
-export type FeedbackListInput = z.infer<typeof FeedbackListInputSchema>;
+export type ListFeedbackInput = z.infer<typeof ListFeedbackInputSchema>;

…s/feedback-list-tool/FeedbackListTool.ts …s/list-feedback-tool/ListFeedbackTool.tssrc/tools/feedback-list-tool/FeedbackListTool.ts renamed to src/tools/list-feedback-tool/ListFeedbackTool.ts src/tools/feedback-list-tool/FeedbackListTool.ts renamed to src/tools/list-feedback-tool/ListFeedbackTool.ts

@@ -5,20 +5,20 @@ import type { z } from 'zod';
 import { MapboxApiBasedTool } from '../MapboxApiBasedTool.js';
 import type { CallToolResult } from '@modelcontextprotocol/sdk/types.js';
 import type { HttpRequest } from '../../utils/types.js';
-import { FeedbackListInputSchema } from './FeedbackListTool.input.schema.js';
+import { ListFeedbackInputSchema } from './ListFeedbackTool.input.schema.js';
 import { FeedbackListResponseSchema } from '../feedback.schema.js';
 import type { FeedbackItem, FeedbackListResponse } from '../feedback.schema.js';
 
 // API Documentation: https://docs.mapbox.com/api/feedback/
 
-export class FeedbackListTool extends MapboxApiBasedTool<
-  typeof FeedbackListInputSchema
+export class ListFeedbackTool extends MapboxApiBasedTool<
+  typeof ListFeedbackInputSchema
 > {
-  name = 'feedback_list_tool';
+  name = 'list_feedback_tool';
   description =
     'List user feedback items from the Mapbox Feedback API with filtering, sorting, and pagination. Use this tool to access user-reported issues, suggestions, and feedback about map data, routing, and POI details. Supports comprehensive filtering by status, category, date ranges, trace IDs, and search text. Requires user-feedback:read scope on the access token.';
   annotations = {
-    title: 'Feedback List Tool',
+    title: 'List Feedback Tool',
     readOnlyHint: true,
     destructiveHint: false,
     idempotentHint: true,
@@ -27,7 +27,7 @@ export class FeedbackListTool extends MapboxApiBasedTool<
 
   constructor(params: { httpRequest: HttpRequest }) {
     super({
-      inputSchema: FeedbackListInputSchema,
+      inputSchema: ListFeedbackInputSchema,
       httpRequest: params.httpRequest
     });
   }
@@ -90,7 +90,7 @@ export class FeedbackListTool extends MapboxApiBasedTool<
    * Builds URL parameters for list operation
    */
   private buildListParams(
-    input: z.infer<typeof FeedbackListInputSchema>,
+    input: z.infer<typeof ListFeedbackInputSchema>,
     url: URL
   ): void {
     // Add optional parameters
@@ -164,7 +164,7 @@ export class FeedbackListTool extends MapboxApiBasedTool<
   }
 
   protected async execute(
-    input: z.infer<typeof FeedbackListInputSchema>,
+    input: z.infer<typeof ListFeedbackInputSchema>,
     accessToken: string
   ): Promise<CallToolResult> {
     const url = new URL(

src/tools/toolRegistry.ts

@@ -7,8 +7,8 @@ import { CoordinateConversionTool } from './coordinate-conversion-tool/Coordinat
 import { CreateStyleTool } from './create-style-tool/CreateStyleTool.js';
 import { CreateTokenTool } from './create-token-tool/CreateTokenTool.js';
 import { DeleteStyleTool } from './delete-style-tool/DeleteStyleTool.js';
-import { FeedbackGetTool } from './feedback-get-tool/FeedbackGetTool.js';
-import { FeedbackListTool } from './feedback-list-tool/FeedbackListTool.js';
+import { GetFeedbackTool } from './get-feedback-tool/GetFeedbackTool.js';
+import { ListFeedbackTool } from './list-feedback-tool/ListFeedbackTool.js';
 import { GeojsonPreviewTool } from './geojson-preview-tool/GeojsonPreviewTool.js';
 import { GetMapboxDocSourceTool } from './get-mapbox-doc-source-tool/GetMapboxDocSourceTool.js';
 import { GetReferenceTool } from './get-reference-tool/GetReferenceTool.js';
@@ -37,8 +37,8 @@ export const ALL_TOOLS = [
   new BoundingBoxTool(),
   new CountryBoundingBoxTool(),
   new CoordinateConversionTool(),
-  new FeedbackGetTool({ httpRequest }),
-  new FeedbackListTool({ httpRequest }),
+  new GetFeedbackTool({ httpRequest }),
+  new ListFeedbackTool({ httpRequest }),
   new GetMapboxDocSourceTool({ httpRequest }),
   new GetReferenceTool(),
   new StyleComparisonTool(),

test/tools/__snapshots__/tool-naming-convention.test.ts.snap

@@ -32,21 +32,16 @@ exports[`Tool Naming Convention > should maintain consistent tool list (snapshot
     "description": "Delete a Mapbox style by ID",
     "toolName": "delete_style_tool",
   },
-  {
-    "className": "FeedbackGetTool",
-    "description": "Get a single user feedback item from the Mapbox Feedback API by its unique ID. Use this tool to retrieve detailed information about a specific user-reported issue, suggestion, or feedback about map data, routing, or POI details. Requires user-feedback:read scope on the access token.",
-    "toolName": "feedback_get_tool",
-  },
-  {
-    "className": "FeedbackListTool",
-    "description": "List user feedback items from the Mapbox Feedback API with filtering, sorting, and pagination. Use this tool to access user-reported issues, suggestions, and feedback about map data, routing, and POI details. Supports comprehensive filtering by status, category, date ranges, trace IDs, and search text. Requires user-feedback:read scope on the access token.",
-    "toolName": "feedback_list_tool",
-  },
   {
     "className": "GeojsonPreviewTool",
     "description": "Generate a geojson.io URL to visualize GeoJSON data. Returns only the URL link.",
     "toolName": "geojson_preview_tool",
   },
+  {
+    "className": "GetFeedbackTool",
+    "description": "Get a single user feedback item from the Mapbox Feedback API by its unique ID. Use this tool to retrieve detailed information about a specific user-reported issue, suggestion, or feedback about map data, routing, or POI details. Requires user-feedback:read scope on the access token.",
+    "toolName": "get_feedback_tool",
+  },
   {
     "className": "GetMapboxDocSourceTool",
     "description": "Get the latest official Mapbox documentation, APIs, SDKs, and developer resources directly from Mapbox. Always up-to-date, comprehensive coverage of all current Mapbox services including mapping, navigation, search, geocoding, and mobile SDKs. Use this for accurate, official Mapbox information instead of web search.",
@@ -57,6 +52,11 @@ exports[`Tool Naming Convention > should maintain consistent tool list (snapshot
     "description": "Get Mapbox reference documentation including Streets v8 field definitions, token scopes, layer type mappings, and style specifications. Use this tool to understand what fields, scopes, or layer types are available before creating styles or tokens.",
     "toolName": "get_reference_tool",
   },
+  {
+    "className": "ListFeedbackTool",
+    "description": "List user feedback items from the Mapbox Feedback API with filtering, sorting, and pagination. Use this tool to access user-reported issues, suggestions, and feedback about map data, routing, and POI details. Supports comprehensive filtering by status, category, date ranges, trace IDs, and search text. Requires user-feedback:read scope on the access token.",
+    "toolName": "list_feedback_tool",
+  },
   {
     "className": "ListStylesTool",
     "description": "List styles for a Mapbox account. Use limit parameter to avoid large responses (recommended: limit=5-10). Use start parameter for pagination.",

…eedback-get-tool/FeedbackGetTool.test.ts …et-feedback-tool/GetFeedbackTool.test.tstest/tools/feedback-get-tool/FeedbackGetTool.test.ts renamed to test/tools/get-feedback-tool/GetFeedbackTool.test.ts test/tools/feedback-get-tool/FeedbackGetTool.test.ts renamed to test/tools/get-feedback-tool/GetFeedbackTool.test.ts

@@ -6,15 +6,15 @@ import {
   setupHttpRequest,
   assertHeadersSent
 } from '../../utils/httpPipelineUtils.js';
-import { FeedbackGetTool } from '../../../src/tools/feedback-get-tool/FeedbackGetTool.js';
+import { GetFeedbackTool } from '../../../src/tools/get-feedback-tool/GetFeedbackTool.js';
 
 const mockToken = 'sk.eyJ1IjoidGVzdC11c2VyIiwiYSI6InRlc3QtYXBpIn0.signature';
 
 beforeAll(() => {
   process.env.MAPBOX_ACCESS_TOKEN = mockToken;
 });
 
-describe('FeedbackGetTool', () => {
+describe('GetFeedbackTool', () => {
   afterEach(() => {
     vi.restoreAllMocks();
   });
@@ -37,8 +37,8 @@ describe('FeedbackGetTool', () => {
           updated_at: '2025-07-28T14:10:30.123Z'
         })
       });
-      const tool = new FeedbackGetTool({ httpRequest });
-      expect(tool.name).toBe('feedback_get_tool');
+      const tool = new GetFeedbackTool({ httpRequest });
+      expect(tool.name).toBe('get_feedback_tool');
       expect(tool.description).toContain('Get a single user feedback item');
     });
   });
@@ -62,7 +62,7 @@ describe('FeedbackGetTool', () => {
       })
     });
 
-    await new FeedbackGetTool({ httpRequest }).run({
+    await new GetFeedbackTool({ httpRequest }).run({
       feedback_id: feedbackId
     });
 
@@ -88,7 +88,7 @@ describe('FeedbackGetTool', () => {
       })
     });
 
-    await new FeedbackGetTool({ httpRequest }).run({
+    await new GetFeedbackTool({ httpRequest }).run({
       feedback_id: feedbackId
     });
 
@@ -119,7 +119,7 @@ describe('FeedbackGetTool', () => {
       json: async () => mockItem
     });
 
-    const result = await new FeedbackGetTool({ httpRequest }).run({
+    const result = await new GetFeedbackTool({ httpRequest }).run({
       feedback_id: mockItem.id
     });
 
@@ -152,7 +152,7 @@ describe('FeedbackGetTool', () => {
       json: async () => mockItem
     });
 
-    const result = await new FeedbackGetTool({ httpRequest }).run({
+    const result = await new GetFeedbackTool({ httpRequest }).run({
       feedback_id: mockItem.id,
       format: 'json_string'
     });
@@ -170,7 +170,7 @@ describe('FeedbackGetTool', () => {
       statusText: 'Not Found'
     });
 
-    const result = await new FeedbackGetTool({ httpRequest }).run({
+    const result = await new GetFeedbackTool({ httpRequest }).run({
       feedback_id: '40eae4c7-b157-4b49-a091-7e1099bba77e'
     });
 
@@ -195,7 +195,7 @@ describe('FeedbackGetTool', () => {
       })
     });
 
-    const result = await new FeedbackGetTool({ httpRequest }).run({
+    const result = await new GetFeedbackTool({ httpRequest }).run({
       feedback_id: '40eae4c7-b157-4b49-a091-7e1099bba77e'
     });
 
@@ -216,7 +216,7 @@ describe('FeedbackGetTool', () => {
       })
     });
 
-    const result = await new FeedbackGetTool({ httpRequest }).run({
+    const result = await new GetFeedbackTool({ httpRequest }).run({
       feedback_id: '40eae4c7-b157-4b49-a091-7e1099bba77e'
     });
 
@@ -235,7 +235,7 @@ describe('FeedbackGetTool', () => {
   it('validates feedback_id format', async () => {
     const { httpRequest } = setupHttpRequest();
 
-    const tool = new FeedbackGetTool({ httpRequest });
+    const tool = new GetFeedbackTool({ httpRequest });
 
     // Invalid UUID format
     const result = await tool.run({
@@ -249,7 +249,7 @@ describe('FeedbackGetTool', () => {
   it('requires feedback_id', async () => {
     const { httpRequest } = setupHttpRequest();
 
-    const tool = new FeedbackGetTool({ httpRequest });
+    const tool = new GetFeedbackTool({ httpRequest });
 
     // Missing feedback_id
     const result = await tool.run({});