Skip to content

Commit 400b33b

Browse files
authored
feat: viz mcp aggregator (#21)
* feat: support dual proxy aggregation for notebooks and visualization * fix: add cross-spawn dependency to root package.json * fix: update CI workflow for renamed mcp directory --------- Co-authored-by: snehshah22 <snehshah22@users.noreply.github.com>
1 parent c3f2d4d commit 400b33b

19 files changed

Lines changed: 1456 additions & 493 deletions
Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ name: Notebook MCP Bundle Check
1717
on:
1818
pull_request:
1919
paths:
20-
- 'notebook_mcp/**'
20+
- 'mcp/**'
2121

2222
env:
2323
FORCE_JAVASCRIPT_ACTIONS_TO_NODE24: 'true'
@@ -36,11 +36,11 @@ jobs:
3636

3737
- name: Install dependencies
3838
run: npm install
39-
working-directory: notebook_mcp
39+
working-directory: mcp
4040

4141
- name: Build bundle
4242
run: npm run bundle
43-
working-directory: notebook_mcp
43+
working-directory: mcp
4444

4545
- name: Check for uncommitted changes in bundle
46-
run: git diff --exit-code notebook_mcp/dist/index.js
46+
run: git diff --exit-code mcp/dist/index.js
File renamed without changes.
Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,7 @@ esbuild.build({
3838
platform: 'node',
3939
format: 'esm',
4040
outfile: 'dist/index.js',
41+
external: ['child_process', 'fs', 'path', 'url', 'cross-spawn'],
4142
banner: {
4243
js: `#!/usr/bin/env node\n${licenseHeader}\n`,
4344
},

0 commit comments

Comments
 (0)