Fix engine registry test failures after custom engine re-addition#16540
Merged
Fix engine registry test failures after custom engine re-addition#16540
Conversation
Copilot
AI
changed the title
[WIP] Fix failing GitHub Actions workflow test
Fix engine registry test failures after custom engine re-addition
Feb 18, 2026
Contributor
There was a problem hiding this comment.
Copilot wasn't able to review any files in this pull request.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Tests failed after commit 0aab1fb re-added the custom engine to fix
TestValidateEngine, but test expectations still assumed only 3 engines (claude, codex, copilot).Changes
Test expectations updated (
agentic_engine_test.go)TestEngineRegistry: 3 → 4 enginesTestEngineRegistryCustomEngine: 4 → 5 engines (after adding test-custom)Nil slice initialization fixed (
engine_helpers.go)InjectCustomEngineStepsnow returns empty slice[]GitHubActionStep{}instead of nilassert.NotNilfailures in interface tests when no custom steps existFixes 4 failing tests:
TestEngineRegistryTestEngineRegistryCustomEngineTestInterfaceSegregationTestSpecificInterfaceUsageWarning
Firewall rules blocked me from connecting to one or more addresses (expand for details)
I tried to connect to the following addresses, but was blocked by firewall rules:
https://api.github.com/graphql/usr/bin/gh /usr/bin/gh api graphql -f query=query($owner: String!, $name: String!) { repository(owner: $owner, name: $name) { hasDiscussionsEnabled } } -f owner=github -f name=gh-aw /tmp/go-build218^From [0-9a-f]\{40\} 64/pkg/tool/linu/tmp/gh-aw/aw.patch git rev-�� --show-toplevel 64/pkg/tool/linux_amd64/vet /opt/hostedtoolcache/go/1.25.0/x64/pkg/tool/linux_amd64/vet -unreachable=falgit /tmp/go-build218rev-parse k/_temp/ghcca-no--show-toplevel /opt/hostedtoolcache/go/1.25.0/x64/pkg/tool/linux_amd64/vet(http block)/usr/bin/gh /usr/bin/gh api graphql -f query=query($owner: String!, $name: String!) { repository(owner: $owner, name: $name) { hasDiscussionsEnabled } } -f owner=github -f name=gh-aw(http block)https://api.github.com/repos/actions/checkout/git/ref/tags/v3/usr/bin/gh gh api /repos/actions/checkout/git/ref/tags/v3 --jq .object.sha 6605/001/stability-test.md HEAD e_modules/.bin/sh INVALID,NEW ole DROP git diff�� VAyE7WnBC HEAD ache/node/24.13.0/x64/bin/git om/henvic/httpregit .cfg 64/pkg/tool/linu--show-toplevel ortcfg(http block)/usr/bin/gh gh api /repos/actions/checkout/git/ref/tags/v3 --jq .object.sha --show-toplevel git /usr/bin/git 3715881186 HEAD 64/pkg/tool/linu--show-toplevel git rev-�� --show-toplevel 64/pkg/tool/linux_amd64/asm /usr/bin/git a HEAD /usr/sbin/git git(http block)https://api.github.com/repos/actions/checkout/git/ref/tags/v4/usr/bin/gh gh api /repos/actions/checkout/git/ref/tags/v4 --jq .object.sha vaScript1647937954/001/test-simple-frontmatter.md stmain.go /home/REDACTED/.config/composer/vendor/bin/git se 7375530/b013/vetrev-parse .cfg git diff�� 999 HEAD 5909070/b274/vet.cfg se 7375530/b080/vetrev-parse run-script/lib/n--show-toplevel sh(http block)/usr/bin/gh gh api /repos/actions/checkout/git/ref/tags/v4 --jq .object.sha /ref/tags/v8 HEAD /opt/hostedtoolcache/node/24.13.0/x64/bin/bash ACCEPT 7375530/b023/vetrev-parse 64/pkg/tool/linu--show-toplevel bash --no�� 0520-14777/test-1138688520 64/pkg/tool/linuHEAD 5909070/b335/vet.cfg --noprofile(http block)/usr/bin/gh gh api /repos/actions/checkout/git/ref/tags/v4 --jq .object.sha -pack /tmp/go-build3595909070/b390/_testmain.go /usr/bin/git --noprofile -tests a613b164 git bran�� --show-current bash /opt/hostedtoolcache/node/24.13.0/x64/bin/node --noprofile(http block)https://api.github.com/repos/actions/checkout/git/ref/tags/v5/usr/bin/gh gh api /repos/actions/checkout/git/ref/tags/v5 --jq .object.sha --noprofile x_amd64/vet iptables(http block)/usr/bin/gh gh api /repos/actions/checkout/git/ref/tags/v5 --jq .object.sha actions/setup-cli/install.sh(http block)/usr/bin/gh gh api /repos/actions/checkout/git/ref/tags/v5 --jq .object.sha user.name resolved$ /usr/bin/git at.Core.RunCommagit security rgo/bin/git git conf�� user.email test@example.com /usr/bin/git TOKEN"; }; f getgit TOKEN"; }; f getrev-parse 64/pkg/tool/linu--show-toplevel git(http block)https://api.github.com/repos/actions/github-script/git/ref/tags/v8/usr/bin/gh gh api /repos/actions/github-script/git/ref/tags/v8 --jq .object.sha axlifetime=\(.*\)$/\1/p /tmp/go-build2187375530/b116/vet.cfg rgo/bin/bash -c=4 -nolocalimports -importcfg /opt/hostedtoolcache/go/1.25.0/xHEAD -ato�� -bool -buildtags .cfg -errorsas -ifaceassert -nilfunc bash(http block)/usr/bin/gh gh api /repos/actions/github-script/git/ref/tags/v8 --jq .object.sha -unreachable=false /tmp/go-build2187375530/b217/vetpkg/workflow/action_reference_test.go ndor/bin/bash(http block)/usr/bin/gh gh api /repos/actions/github-script/git/ref/tags/v8 --jq .object.sha -unreachable=false /tmp/go-build2187375530/b199/vet.cfg x_amd64/compile(http block)https://api.github.com/repos/actions/setup-go/git/ref/tags/v4/usr/bin/gh gh api /repos/actions/setup-go/git/ref/tags/v4 --jq .object.sha --name-only HEAD ache/go/1.25.0/x64/pkg/tool/linustatus,conclusion se 7375530/b017/vetrev-parse 64/pkg/tool/linu--show-toplevel ache/go/1.25.0/x64/pkg/tool/linux_amd64/vet diff�� --name-only HEAD 5909070/b293/vet.cfg se 7375530/b154/vetrev-parse ache/go/1.25.0/x--show-toplevel sh(http block)/usr/bin/gh gh api /repos/actions/setup-go/git/ref/tags/v4 --jq .object.sha --show-toplevel bash /usr/bin/git --noprofile 64/pkg/tool/linurev-parse .cfg git rev-�� --show-toplevel(http block)https://api.github.com/repos/actions/setup-node/git/ref/tags/v4/usr/bin/gh gh api /repos/actions/setup-node/git/ref/tags/v4 --jq .object.sha --name-only HEAD /home/REDACTED/.dotnet/tools/git se 7375530/b015/vetrev-parse 64/pkg/tool/linu--show-toplevel git diff�� 0520-14777/test-1138688520 HEAD 0/x64/bin/node g_.a(http block)/usr/bin/gh gh api /repos/actions/setup-node/git/ref/tags/v4 --jq .object.sha --show-toplevel bash /usr/bin/git --noprofile 64/pkg/tool/linurev-parse .cfg git rev-�� tch git /usr/bin/git blog-auditor.md HEAD ache/go/1.25.0/x--show-toplevel git(http block)https://api.github.com/repos/github/gh-aw/actions/runs/1/artifacts/usr/bin/gh gh run download 1 --dir test-logs/run-1 -buildtags cal/bin/bash -errorsas -ifaceassert -nilfunc git diff�� --name-only HEAD bin/node(http block)https://api.github.com/repos/github/gh-aw/actions/runs/12345/artifacts/usr/bin/gh gh run download 12345 --dir test-logs/run-12345 x_amd64/compile x_amd64/vet(http block)https://api.github.com/repos/github/gh-aw/actions/runs/12346/artifacts/usr/bin/gh gh run download 12346 --dir test-logs/run-12346 x_amd64/compile git(http block)https://api.github.com/repos/github/gh-aw/actions/runs/2/artifacts/usr/bin/gh gh run download 2 --dir test-logs/run-2 --log-level 64/bin/bash --log-target journal-or-kmsg x_amd64/vet git diff�� --name-only HEAD 0/x64/lib/node_modules/npm/node_modules/@npmcli/run-script/lib/node-gyp-bin/node(http block)https://api.github.com/repos/github/gh-aw/actions/runs/3/artifacts/usr/bin/gh gh run download 3 --dir test-logs/run-3 -test.timeout=10m0s x_amd64/compile -test.run=TestEngit(http block)https://api.github.com/repos/github/gh-aw/actions/runs/4/artifacts/usr/bin/gh gh run download 4 --dir test-logs/run-4 -importcfg sh -s -w -buildmode=exe git diff�� --name-only HEAD ode(http block)https://api.github.com/repos/github/gh-aw/actions/runs/5/artifacts/usr/bin/gh gh run download 5 --dir test-logs/run-5 -buildtags rgo/bin/bash -errorsas -ifaceassert -nilfunc git estl�� --name-only HEAD p/bin/bash(http block)https://api.github.com/repos/github/gh-aw/actions/workflows/usr/bin/gh gh workflow list --json name,state,path -c=4 -nolocalimports -importcfg /tmp/go-build1545120341/b332/importcfg -pack /home/REDACTED/go/pkg/mod/github.com/sourcegraph/conc@v0.3.0/internal/multierror/multierror_go120.go 64/pkg/tool/linux_amd64/asm -uns�� -unreachable=false /tmp/go-build2187375530/b259/vet.cfg ache/go/1.25.0/x64/bin/bash(http block)/usr/bin/gh gh run list --json databaseId,number,url,status,conclusion,workflowName,createdAt,startedAt,updatedAt,event,headBranch,headSha,displayTitle --workflow nonexistent-workflow-12345 --limit 100 -ifaceassert -nilfunc bash --no�� --noprofile -tests a613b164(http block)/usr/bin/gh gh run list --json databaseId,number,url,status,conclusion,workflowName,createdAt,startedAt,updatedAt,event,headBranch,headSha,displayTitle --workflow nonexistent-workflow-12345 --limit 6(http block)https://api.github.com/repos/github/gh-aw/git/ref/tags/v1.0.0/usr/bin/gh gh api /repos/github/gh-aw/git/ref/tags/v1.0.0 --jq .object.sha --name-only HEAD a613b164(http block)/usr/bin/gh gh api /repos/github/gh-aw/git/ref/tags/v1.0.0 --jq .object.sha user.email test@example.com /usr/bin/infocmp a/salsa20_amd64.git -buildtags x_amd64/compile infocmp -1 xterm-color x_amd64/compile /usr/bin/git -stringintconv -tests 2208fca855573da4--show-toplevel git(http block)https://api.github.com/repos/nonexistent/action/git/ref/tags/v999.999.999/usr/bin/gh gh api /repos/nonexistent/action/git/ref/tags/v999.999.999 --jq .object.sha --name-only HEAD 0/x64/lib/node_modules/npm/node_modules/@npmcli/run-script/lib/node-gyp-bin/sh(http block)/usr/bin/gh gh api /repos/nonexistent/action/git/ref/tags/v999.999.999 --jq .object.sha --show-toplevel /opt/hostedtoolcremote.origin.url /usr/bin/git -bool -buildtags x_amd64/compile /usr/bin/git conf�� --get-regexp ^remote\..*\.gh-resolved$ /usr/bin/git -stringintconv -tests ache/go/1.25.0/x--show-toplevel git(http block)https://api.github.com/repos/nonexistent/repo/actions/runs/12345/usr/bin/gh gh run view 12345 --repo nonexistent/repo --json status,conclusion -errorsas -ifaceassert -nilfunc x_amd64/vet --no�� --noprofile -tests a613b164(http block)https://api.github.com/repos/owner/repo/actions/workflows/usr/bin/gh gh workflow list --json name,state,path --repo owner/repo -importcfg /tmp/go-build1545120341/b333/importcfg -pack /home/REDACTED/go/pkg/mod/github.com/spf13/cobra@v1.10.2/active_help.go /home/REDACTED/go/pkg/mod/github.com/spf13/cobra@v1.10.2/args.go -ato�� -bool -buildtags a613b164 -errorsas -ifaceassert -nilfunc bash(http block)/usr/bin/gh gh workflow list --json name,state,path --repo owner/repo a613b164 -c=4 -nolocalimports -importcfg /opt/hostedtoolcache/go/1.25.0/x-importcfg -ato�� -bool -buildtags a613b164 -errorsas -ifaceassert -nilfunc bash(http block)https://api.github.com/repos/owner/repo/contents/file.md/tmp/go-build3595909070/b368/cli.test /tmp/go-build3595909070/b368/cli.test -test.testlogfile=/tmp/go-build3595909070/b368/testlog.txt -test.paniconexit0 -test.v=true -test.parallel=4 -test.timeout=10m0s -test.run=^Test -test.short=true -c=4 -nolocalimports -importcfg /opt/hostedtoolcache/go/1.25.0/x64/pkg/tool/linu-importcfg -uns�� -unreachable=false /tmp/go-build2187375530/b221/vet.cfg _modules/.bin/sh(http block)https://api.github.com/repos/test-owner/test-repo/actions/secrets/usr/bin/gh gh api /repos/test-owner/test-repo/actions/secrets --jq .secrets[].name -c=4 -nolocalimports -importcfg /tmp/go-build1545120341/b277/importcfg -pack /home/REDACTED/work/gh-aw/gh-aw/pkg/envutil/envutil.go /opt/hostedtoolcconntrack -uns�� rkflow/agentic_eINVALID,NEW /tmp/go-build218-j ash go(http block)https://api.github.com/user/usr/bin/gh gh api user --jq .login -bool -buildtags it -errorsas -ifaceassert -nilfunc /opt/hostedtoolcache/go/1.25.0/xHEAD -uns�� ub-actions-workflow-test-again /tmp/go-build2187375530/b195/vet.cfg ache/go/1.25.0/x64/pkg/tool/linux_amd64/asm(http block)/usr/bin/gh gh api user --jq .login -unreachable=false /tmp/go-build2187375530/b189/vet.cfg 7375530/b317/vet.cfg -c=4 -nolocalimports -importcfg /opt/hostedtoolcache/go/1.25.0/xHEAD -uns�� ub-actions-workflow-test-again /tmp/go-build2187375530/b196/vet.cfg /opt/hostedtoolcache/go/1.25.0/x64/pkg/tool/linux_amd64/vet(http block)/usr/bin/gh gh api user --jq .login -unreachable=false /tmp/go-build2187375530/b218/vet.cfg ache/go/1.25.0/x64/bin/sh -c=4 -nolocalimports -importcfg /opt/hostedtoolcache/go/1.25.0/xHEAD -uns�� celain --ignore-submodules | head -n 10 /tmp/go-build2187375530/b198/vet.cfg ndor/bin/git(http block)If you need me to access, download, or install something from one of these locations, you can either:
Original prompt
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.