fix: DH-21947: MCP data tool fixes (#314)

DH-21947: MCP data tool fixes
- `getTableData` now supports hierarchical tables. The crux of the fix
is in `getTableOrError` which now finds the variable to use the proper
object type for the `getObject` call
- fixed an output schema error with `connectToServer` tool

Rest of the PR is mostly just updating tool dependencies + cleaning up
tests.

## Testing
Create a rollup table. I used:
```py
from deephaven import new_table
from deephaven.column import string_col, int_col, double_col

# Create hierarchical test data
sales_data = new_table([
    string_col("Region", ["North", "North", "North", "North", "South", "South", "South", "East", "East", "West"]),
    string_col("State", ["NY", "NY", "MA", "MA", "TX", "TX", "FL", "PA", "NJ", "CA"]),
    string_col("City", ["NYC", "Albany", "Boston", "Cambridge", "Houston", "Dallas", "Miami", "Philly", "Newark", "LA"]),
    int_col("Sales", [1000, 500, 800, 600, 1200, 900, 700, 850, 650, 1100]),
    double_col("Revenue", [10000.0, 5000.0, 8000.0, 6000.0, 12000.0, 9000.0, 7000.0, 8500.0, 6500.0, 11000.0])
])
```

Ask the agent to get data from the `sales_data` table. It should return
the data matching the table.

Author: bmingles

AGENTS.md

@@ -30,6 +30,14 @@ For detailed instructions on writing tests (mocking patterns, test structure, MC
 - **Main skill**: `.github/skills/test-writing/SKILL.md`
 - **MCP-specific patterns**: `.github/skills/test-writing/mcp-tools-testing.md`
 
+## Linting
+
+### Running ESLint
+
+- Use `npm run test:lint` to check all TypeScript files for linting errors
+- **IMPORTANT**: The `get_errors` tool may not catch all linting issues, especially in files that aren't currently open in the editor
+- Always run `npm run test:lint` before committing to ensure all files pass linting
+
 ## MCP Tools
 
 ### Writing MCP Tools

package-lock.json

@@ -1066,6 +1066,7 @@
     "@deephaven-enterprise/auth-nodejs": "^1.20250219.134-beta",
     "@deephaven-enterprise/query-utils": "^1.20250219.134-beta",
     "@deephaven/jsapi-nodejs": "^1.15.0",
+    "@deephaven/jsapi-utils": "^1.16.0",
     "@modelcontextprotocol/sdk": "^1.27.1",
     "archiver": "^7.0.1",
     "chai": "^4.5.0",
@@ -1076,10 +1077,12 @@
     "@cfworker/json-schema": "^4.1.1",
     "@deephaven-enterprise/jsapi-types": "^1.20250219.134-beta",
     "@deephaven/jsapi-types": "^41.2.0",
+    "@react-types/shared": "^3.33.1",
     "@types/archiver": "^6.0.3",
     "@types/chai": "^4.3.20",
     "@types/mocha": "^10.0.10",
     "@types/node": "22.19.1",
+    "@types/react": "^19.2.14",
     "@types/vscode": "^1.105.0",
     "@types/vscode-webview": "^1.57.5",
     "@types/ws": "^8.5.10",

package.json

@@ -16,7 +16,10 @@ const spec = {
   inputSchema: {
     url: z.string().describe('Server URL (e.g., "http://localhost:10000")'),
   },
-  outputSchema: createMcpToolOutputSchema(),
+  outputSchema: createMcpToolOutputSchema({
+    type: z.enum(['DHC', 'DHE']).optional().describe('Server type'),
+    url: z.string().optional().describe('Server URL'),
+  }),
 } as const;
 
 type Spec = typeof spec;