Skip to content

Commit a30be1d

Browse files
data-douserCopilot
andauthored
Apply suggestions from code review
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Nathan Randall <70299490+data-douser@users.noreply.github.com>
1 parent ca534c0 commit a30be1d

4 files changed

Lines changed: 0 additions & 16 deletions

File tree

.github/workflows/build-server.yml

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -40,11 +40,6 @@ jobs:
4040
run: npm ci --include=optional --ignore-scripts
4141
working-directory: .
4242

43-
## esbuild's optional platform binary is normally wired up by its postinstall
44-
## script; .npmrc 'ignore-scripts=true' suppresses that, so rebuild here.
45-
- name: Build Server - Rebuild esbuild platform binary
46-
run: npm rebuild esbuild --ignore-scripts=false
47-
4843
- name: Build Server - Clean previous build
4944
run: npm run clean
5045

.github/workflows/client-integration-tests.yml

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -62,11 +62,6 @@ jobs:
6262
- name: MCP Integration Tests - Install node dependencies for client and server workspaces
6363
run: npm ci --workspace=client --ignore-scripts && npm ci --workspace=server --ignore-scripts
6464

65-
## esbuild's optional platform binary is normally wired up by its postinstall
66-
## script; .npmrc 'ignore-scripts=true' suppresses that, so rebuild here.
67-
- name: MCP Integration Tests - Rebuild esbuild platform binary
68-
run: npm rebuild esbuild --ignore-scripts=false
69-
7065
- name: MCP Integration Tests - Setup CodeQL environment
7166
uses: ./.github/actions/setup-codeql-environment
7267
with:

.github/workflows/copilot-setup-steps.yml

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -48,11 +48,6 @@ jobs:
4848
- name: Copilot Setup - Install dependencies
4949
run: npm ci --include=optional --ignore-scripts
5050

51-
## esbuild's optional platform binary is normally wired up by its postinstall
52-
## script; .npmrc 'ignore-scripts=true' suppresses that, so rebuild here.
53-
- name: Copilot Setup - Rebuild esbuild platform binary
54-
run: npm rebuild esbuild --ignore-scripts=false
55-
5651
- name: Copilot Setup - Setup CodeQL environment
5752
uses: ./.github/actions/setup-codeql-environment
5853
with:

client/scripts/run-integration-tests.sh

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -86,7 +86,6 @@ fi
8686
# Step 1: Build and bundle the server code
8787
echo "📦 Building CodeQL MCP server bundle..."
8888
cd "$SERVER_DIR"
89-
npm run rebuild:esbuild
9089
npm run bundle
9190

9291
# Step 2: Install CodeQL packs (only once for both modes, skip if --no-install-packs)

0 commit comments

Comments
 (0)