From 34028b1ee79a3bf3c6e09bb92e6a62fa6db97e1a Mon Sep 17 00:00:00 2001 From: Aaron Powell Date: Fri, 24 Oct 2025 03:39:14 +0000 Subject: [PATCH 1/5] Dumping npm logs --- .github/actions/setup-runtimes-caching/action.yml | 8 ++++++++ .github/workflows/tests.yaml | 7 +++++++ 2 files changed, 15 insertions(+) diff --git a/.github/actions/setup-runtimes-caching/action.yml b/.github/actions/setup-runtimes-caching/action.yml index 9d98a7ec0..2620dd0cf 100644 --- a/.github/actions/setup-runtimes-caching/action.yml +++ b/.github/actions/setup-runtimes-caching/action.yml @@ -123,3 +123,11 @@ runs: run: | dapr init --runtime-version=${{ env.DAPR_VERSION }} dapr --version + + - name: Set npm log directory + shell: bash + if: ${{ contains(inputs.name, 'Node') }} + run: | + mkdir -p ./.npm-logs + npm config set loglevel verbose + npm config set prefix './.npm-logs' diff --git a/.github/workflows/tests.yaml b/.github/workflows/tests.yaml index 570b05435..1e635f6f0 100644 --- a/.github/workflows/tests.yaml +++ b/.github/workflows/tests.yaml @@ -131,3 +131,10 @@ jobs: with: name: testresults-${{ matrix.name }}-${{ matrix.os }} path: testresults/** + + - name: Upload npm logs if failed + if: failure() + uses: actions/upload-artifact@v4 + with: + name: npm-logs-${{ matrix.name }}-${{ matrix.os }} + path: ./.npm-logs/** From f6a2f5c0a1cfaab9add6a0a99b52e6d2221baf66 Mon Sep 17 00:00:00 2001 From: Aaron Powell Date: Fri, 24 Oct 2025 03:41:19 +0000 Subject: [PATCH 2/5] Setting the right property --- .github/actions/setup-runtimes-caching/action.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/actions/setup-runtimes-caching/action.yml b/.github/actions/setup-runtimes-caching/action.yml index 2620dd0cf..a2f857170 100644 --- a/.github/actions/setup-runtimes-caching/action.yml +++ b/.github/actions/setup-runtimes-caching/action.yml @@ -130,4 +130,4 @@ runs: run: | mkdir -p ./.npm-logs npm config set loglevel verbose - npm config set prefix './.npm-logs' + npm config set logs-dir './.npm-logs' From b77d1aa74e9ffcb99c5b5e4e727205754b3d541c Mon Sep 17 00:00:00 2001 From: Aaron Powell Date: Fri, 24 Oct 2025 04:16:11 +0000 Subject: [PATCH 3/5] changing where to look for logs --- .github/actions/setup-runtimes-caching/action.yml | 8 -------- .github/workflows/tests.yaml | 2 +- 2 files changed, 1 insertion(+), 9 deletions(-) diff --git a/.github/actions/setup-runtimes-caching/action.yml b/.github/actions/setup-runtimes-caching/action.yml index a2f857170..9d98a7ec0 100644 --- a/.github/actions/setup-runtimes-caching/action.yml +++ b/.github/actions/setup-runtimes-caching/action.yml @@ -123,11 +123,3 @@ runs: run: | dapr init --runtime-version=${{ env.DAPR_VERSION }} dapr --version - - - name: Set npm log directory - shell: bash - if: ${{ contains(inputs.name, 'Node') }} - run: | - mkdir -p ./.npm-logs - npm config set loglevel verbose - npm config set logs-dir './.npm-logs' diff --git a/.github/workflows/tests.yaml b/.github/workflows/tests.yaml index 1e635f6f0..e0fdf4963 100644 --- a/.github/workflows/tests.yaml +++ b/.github/workflows/tests.yaml @@ -137,4 +137,4 @@ jobs: uses: actions/upload-artifact@v4 with: name: npm-logs-${{ matrix.name }}-${{ matrix.os }} - path: ./.npm-logs/** + path: ${{ runner.os == 'Windows' && 'C:\\npm\\cache\\_logs\\**' || '~/.npm/cache/_logs/**' }} From 3cefca58c971e271ca006d3614127a433081ec3d Mon Sep 17 00:00:00 2001 From: Aaron Powell Date: Fri, 24 Oct 2025 05:13:18 +0000 Subject: [PATCH 4/5] Updating inspector version --- .../McpInspectorResource.cs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/CommunityToolkit.Aspire.Hosting.McpInspector/McpInspectorResource.cs b/src/CommunityToolkit.Aspire.Hosting.McpInspector/McpInspectorResource.cs index 89fc6bf94..7ef9ebd6e 100644 --- a/src/CommunityToolkit.Aspire.Hosting.McpInspector/McpInspectorResource.cs +++ b/src/CommunityToolkit.Aspire.Hosting.McpInspector/McpInspectorResource.cs @@ -37,7 +37,7 @@ public class McpInspectorResource(string name) : ExecutableResource(name, "npx", /// /// Gets the version of the MCP Inspector. /// - public const string InspectorVersion = "0.16.5"; + public const string InspectorVersion = "0.17.2"; private readonly List _mcpServers = []; From 5743b0996b8378383a08da8b469bb126a3fcd363 Mon Sep 17 00:00:00 2001 From: Aaron Powell Date: Fri, 24 Oct 2025 05:21:31 +0000 Subject: [PATCH 5/5] skipping npm package lock to try and avoid the conflict on install --- .github/workflows/tests.yaml | 1 + 1 file changed, 1 insertion(+) diff --git a/.github/workflows/tests.yaml b/.github/workflows/tests.yaml index e0fdf4963..f2a985f0a 100644 --- a/.github/workflows/tests.yaml +++ b/.github/workflows/tests.yaml @@ -10,6 +10,7 @@ jobs: DAPR_VERSION: "1.15.0" TEST_PROJECT_PATH: tests/CommunityToolkit.Aspire.${{ matrix.name }}/CommunityToolkit.Aspire.${{ matrix.name }}.csproj CI: true + NPM_CONFIG_PACKAGE_LOCK: false runs-on: "${{ matrix.os }}" timeout-minutes: 60 name: ${{ matrix.name }}-${{ matrix.os }}