improve style preview

Author: zmofei

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

@@ -39,7 +39,7 @@ exports[`Tool Naming Convention should maintain consistent tool list (snapshot t
   },
   {
     "className": "ListStylesTool",
-    "description": "List all styles for a Mapbox account",
+    "description": "List styles for a Mapbox account. Use limit parameter to avoid large responses (recommended: limit=5-10). Use start parameter for pagination.",
     "toolName": "list_styles_tool",
   },
   {

src/tools/list-styles-tool/ListStylesTool.test.ts

@@ -17,7 +17,9 @@ describe('ListStylesTool', () => {
     it('should have correct name and description', () => {
       const tool = new ListStylesTool();
       expect(tool.name).toBe('list_styles_tool');
-      expect(tool.description).toBe('List all styles for a Mapbox account');
+      expect(tool.description).toBe(
+        'List styles for a Mapbox account. Use limit parameter to avoid large responses (recommended: limit=5-10). Use start parameter for pagination.'
+      );
     });
 
     it('should have correct input schema', () => {

src/tools/preview-style-tool/PreviewStyleTool.test.ts

@@ -1,14 +1,32 @@
 // Use a token with valid JWT format for tests
 process.env.MAPBOX_ACCESS_TOKEN =
-  'pk.eyJhbGciOiJIUzI1NiJ9.eyJ1IjoidGVzdC11c2VyIiwiYSI6InRlc3QtYXBpIn0.signature';
+  'eyJhbGciOiJIUzI1NiJ9.eyJ1IjoidGVzdC11c2VyIiwiYSI6InRlc3QtYXBpIn0.signature';
 
+import { MapboxApiBasedTool } from '../MapboxApiBasedTool.js';
 import { PreviewStyleTool } from './PreviewStyleTool.js';
 
 describe('PreviewStyleTool', () => {
   const TEST_ACCESS_TOKEN =
     'pk.eyJhbGciOiJIUzI1NiJ9.eyJ1IjoidGVzdC11c2VyIiwiYSI6InRlc3QtYXBpIn0.signature';
 
-  // No setup needed since we use user-provided tokens directly
+  beforeEach(() => {
+    // Mock getUserNameFromToken to handle pk. prefixed tokens
+    jest
+      .spyOn(MapboxApiBasedTool, 'getUserNameFromToken')
+      .mockImplementation((token) => {
+        // If token starts with pk., we expect it to be a prefixed token
+        if (token && token.startsWith('pk.')) {
+          // For test token, return 'test-user'
+          return 'test-user';
+        }
+        // For non-prefixed tokens, return 'test-user' as well
+        return 'test-user';
+      });
+  });
+
+  afterEach(() => {
+    jest.restoreAllMocks();
+  });
 
   describe('tool metadata', () => {
     it('should have correct name and description', () => {