From acf4568ae873ef62e0767574122d65319f2a8b42 Mon Sep 17 00:00:00 2001 From: Jake Bailey <5341706+jakebailey@users.noreply.github.com> Date: Fri, 14 Nov 2025 09:06:57 -0800 Subject: [PATCH 1/4] Set MS_GOTOOLCHAIN_TELEMETRY_ENABLED=0 in copilot steps --- .github/workflows/copilot-setup-steps.yml | 3 +++ 1 file changed, 3 insertions(+) diff --git a/.github/workflows/copilot-setup-steps.yml b/.github/workflows/copilot-setup-steps.yml index 61adbde600..7586291654 100644 --- a/.github/workflows/copilot-setup-steps.yml +++ b/.github/workflows/copilot-setup-steps.yml @@ -13,6 +13,9 @@ jobs: # If you want to clone the repository as part of your setup steps, for example to install dependencies, you'll need the `contents: read` permission. If you don't clone the repository in your setup steps, Copilot will do this for you automatically after the steps complete. contents: read + env: + MS_GOTOOLCHAIN_TELEMETRY_ENABLED: '0' + # You can define any steps you want, and they will run before the agent starts. # If you do not check out your code, Copilot will do this for you. steps: From a634d8688d6b29819508052556c6f95806f23570 Mon Sep 17 00:00:00 2001 From: Jake Bailey <5341706+jakebailey@users.noreply.github.com> Date: Fri, 14 Nov 2025 10:26:56 -0800 Subject: [PATCH 2/4] Install gopls --- .github/workflows/copilot-setup-steps.yml | 1 + 1 file changed, 1 insertion(+) diff --git a/.github/workflows/copilot-setup-steps.yml b/.github/workflows/copilot-setup-steps.yml index 7586291654..24c496d88e 100644 --- a/.github/workflows/copilot-setup-steps.yml +++ b/.github/workflows/copilot-setup-steps.yml @@ -28,6 +28,7 @@ jobs: with: lint-cache: 'true' - run: npm i -g @playwright/mcp@0.0.28 + - run: go install golang.org/x/tools/gopls@latest - run: npm ci # pull dprint caches before network access is blocked - run: npx hereby check:format || true From b54722af16f0ce24a854b10e9e141c97d8c48756 Mon Sep 17 00:00:00 2001 From: Jake Bailey <5341706+jakebailey@users.noreply.github.com> Date: Fri, 14 Nov 2025 10:34:43 -0800 Subject: [PATCH 3/4] new enough node to have strip types --- .github/workflows/copilot-setup-steps.yml | 2 ++ 1 file changed, 2 insertions(+) diff --git a/.github/workflows/copilot-setup-steps.yml b/.github/workflows/copilot-setup-steps.yml index 24c496d88e..2052c753b1 100644 --- a/.github/workflows/copilot-setup-steps.yml +++ b/.github/workflows/copilot-setup-steps.yml @@ -23,6 +23,8 @@ jobs: with: submodules: true - uses: actions/setup-node@2028fbc5c25fe9cf00d9f06a71cc4710d4507903 # v6.0.0 + with: + node-version: '>=22.16.0' - uses: dtolnay/rust-toolchain@fcf085fcb4b4b8f63f96906cd713eb52181b5ea4 # stable - uses: ./.github/actions/setup-go with: From 436aa232c2cb49fb4d849662e519ad4f14d62695 Mon Sep 17 00:00:00 2001 From: Jake Bailey <5341706+jakebailey@users.noreply.github.com> Date: Fri, 14 Nov 2025 10:37:10 -0800 Subject: [PATCH 4/4] Update instructions to say not to use timeout --- .github/copilot-instructions.md | 1 + 1 file changed, 1 insertion(+) diff --git a/.github/copilot-instructions.md b/.github/copilot-instructions.md index d1d78dc6d5..27d34bb431 100644 --- a/.github/copilot-instructions.md +++ b/.github/copilot-instructions.md @@ -110,3 +110,4 @@ The TypeScript submodule serves as the reference implementation for behavior and - Do not add or change existing dependencies unless asked to. - Do not remove any debug assertions or panic calls. Existing assertions are never too strict or incorrect. +- Do not use the `timeout` command when running tests or other commands, unless specifically debugging a hanging issue. Commands should be run directly without timeout wrappers in normal operation.