really need to tell Amp to run prettier every time it saves...

Author: geelen

.github/workflows/ci.yml

@@ -9,39 +9,39 @@ on:
 jobs:
   test:
     runs-on: ubuntu-latest
-    
+
     steps:
       - name: Checkout code
         uses: actions/checkout@v4
-      
+
       - name: Setup pnpm
         uses: pnpm/action-setup@v4
-      
+
       - name: Setup Node.js
         uses: actions/setup-node@v4
         with:
           node-version: '22'
           cache: 'pnpm'
-      
+
       - name: Install dependencies - root
         run: pnpm install
-      
+
       - name: Install dependencies - inspector example
         run: pnpm install
         working-directory: examples/inspector
-      
+
       - name: Install dependencies - cf-agents example
         run: pnpm install
         working-directory: examples/servers/cf-agents
-      
+
       - name: Install dependencies - hono-mcp example
         run: pnpm install
         working-directory: examples/servers/hono-mcp
-      
+
       - name: Install dependencies - test
         run: pnpm install
         working-directory: test
-      
+
       - name: Run tests
         run: pnpm test
         working-directory: test

examples/inspector/src/components/McpServers.tsx

@@ -10,7 +10,7 @@ function McpConnection({ serverUrl, onConnectionUpdate }: { serverUrl: string; o
     debug: true,
     autoRetry: false,
     popupFeatures: 'width=500,height=600,resizable=yes,scrollbars=yes',
-    transportType: 'auto'
+    transportType: 'auto',
   })
 
   // Update parent component with connection data

src/react/useMcp.ts

@@ -378,7 +378,7 @@ export function useMcp(options: UseMcpOptions): UseMcpResult {
     // --- Orchestrate Connection Attempts ---
     let finalStatus: 'success' | 'auth_redirect' | 'failed' | 'fallback' = 'failed' // Default to failed
 
-    console.log({transportType})
+    console.log({ transportType })
 
     if (transportType === 'sse') {
       // SSE only - skip HTTP entirely

test/integration/mcp-connection.test.ts

@@ -13,11 +13,11 @@ function getMCPServers() {
   try {
     const stateData = readFileSync(testStateFile, 'utf-8')
     const state = JSON.parse(stateData)
-    
+
     if (!state.honoPort) {
       throw new Error('hono-mcp port not found in test state')
     }
-    
+
     return [
       {
         name: 'hono-mcp',
@@ -40,48 +40,48 @@ async function connectToMCPServer(page: Page, serverUrl: string): Promise<{ succ
   // Navigate to the inspector
   const stateData = readFileSync(testStateFile, 'utf-8')
   const state = JSON.parse(stateData)
-  
+
   if (!state.staticPort) {
     throw new Error('Static server port not available - state: ' + JSON.stringify(state))
   }
   const staticPort = state.staticPort
-  
+
   await page.goto(`http://localhost:${staticPort}`)
-  
+
   // Wait for the page to load
   await page.waitForSelector('input[placeholder="Enter MCP server URL"]', { timeout: 10000 })
-  
+
   // Enter the server URL
   const urlInput = page.locator('input[placeholder="Enter MCP server URL"]')
   await urlInput.fill(serverUrl)
-  
+
   // Click connect button
   const connectButton = page.locator('button:has-text("Connect")')
   await connectButton.click()
-  
+
   // Wait for connection attempt to complete (max 10 seconds)
   await page.waitForTimeout(1000) // Initial wait
-  
+
   // Check for connection status
   let attempts = 0
   const maxAttempts = 20 // 10 seconds total (500ms * 20)
   let isConnected = false
-  
+
   while (attempts < maxAttempts && !isConnected) {
     try {
       // Check if status badge shows "Connected"
       const statusBadge = page.locator('.px-2.py-1.rounded-full')
-      if (await statusBadge.count() > 0) {
+      if ((await statusBadge.count()) > 0) {
         const statusText = await statusBadge.textContent({ timeout: 500 })
         if (statusText?.toLowerCase().includes('connected')) {
           isConnected = true
           break
         }
       }
-      
+
       // Also check if tools count is > 0
       const toolsHeader = page.locator('h3:has-text("Available Tools")')
-      if (await toolsHeader.count() > 0) {
+      if ((await toolsHeader.count()) > 0) {
         const toolsText = await toolsHeader.textContent()
         if (toolsText && /\d+/.test(toolsText)) {
           const toolsCount = parseInt(toolsText.match(/\d+/)?.[0] || '0')
@@ -94,18 +94,18 @@ async function connectToMCPServer(page: Page, serverUrl: string): Promise<{ succ
     } catch (e) {
       // Continue waiting
     }
-    
+
     await page.waitForTimeout(500)
     attempts++
   }
-  
+
   // Extract available tools
   const tools: string[] = []
   try {
     // Look for tool cards in the tools container
     const toolCards = page.locator('.bg-white.rounded.border')
     const toolCount = await toolCards.count()
-    
+
     for (let i = 0; i < toolCount; i++) {
       const toolNameElement = toolCards.nth(i).locator('h4.font-bold.text-base.text-black')
       const toolName = await toolNameElement.textContent()
@@ -116,18 +116,18 @@ async function connectToMCPServer(page: Page, serverUrl: string): Promise<{ succ
   } catch (e) {
     console.warn('Could not extract tools list:', e)
   }
-  
+
   // Extract debug log
   let debugLog = ''
   try {
     const debugContainer = page.locator('.h-32.overflow-y-auto.font-mono.text-xs')
-    if (await debugContainer.count() > 0) {
-      debugLog = await debugContainer.first().textContent() || ''
+    if ((await debugContainer.count()) > 0) {
+      debugLog = (await debugContainer.first().textContent()) || ''
     }
   } catch (e) {
     console.warn('Could not extract debug log:', e)
   }
-  
+
   return {
     success: isConnected,
     tools,
@@ -149,7 +149,7 @@ describe('MCP Connection Integration Tests', () => {
     if (browser) {
       await browser.close()
     }
-    
+
     // Force cleanup before Vitest exits - don't throw errors
     const state = globalThis.__INTEGRATION_TEST_STATE__
     try {
@@ -160,7 +160,7 @@ describe('MCP Connection Integration Tests', () => {
     } catch (e) {
       // Ignore errors - process might already be dead
     }
-    
+
     try {
       if (state?.staticServer) {
         state.staticServer.close()
@@ -174,7 +174,7 @@ describe('MCP Connection Integration Tests', () => {
   beforeEach(async () => {
     const context = await browser.newContext()
     page = await context.newPage()
-    
+
     // Enable console logging for debugging
     page.on('console', (msg) => {
       if (msg.type() === 'error') {
@@ -192,19 +192,19 @@ describe('MCP Connection Integration Tests', () => {
 
   test('should connect to all MCP servers and retrieve tools', async () => {
     const servers = getMCPServers()
-    
+
     for (const server of servers) {
       console.log(`\n🔗 Testing connection to ${server.name} at ${server.url}`)
-      
+
       const result = await connectToMCPServer(page, server.url)
-      
+
       if (result.success) {
         console.log(`✅ Successfully connected to ${server.name}`)
         console.log(`📋 Available tools (${result.tools.length}):`)
         result.tools.forEach((tool, index) => {
           console.log(`   ${index + 1}. ${tool}`)
         })
-        
+
         // Verify connection success
         expect(result.success).toBe(true)
         expect(result.tools.length).toBeGreaterThanOrEqual(server.expectedTools)
@@ -214,7 +214,7 @@ describe('MCP Connection Integration Tests', () => {
           console.log(`🐛 Debug log:`)
           console.log(result.debugLog)
         }
-        
+
         // Fail the test with detailed information
         throw new Error(`Failed to connect to ${server.name}. Debug log: ${result.debugLog}`)
       }