fix: surface Docker image retry instructions in MCP compile error message (#19792)

Copilot · pelikhan · web-flow · commit fac1c3306d98 · 2026-03-05T19:33:14.000-08:00
* Initial plan

* fix: include retry instructions in MCP error message for docker images not ready

Co-authored-by: pelikhan &lt;4175913+pelikhan@users.noreply.github.com&gt;

---------

Co-authored-by: copilot-swe-agent[bot] &lt;198982749+Copilot@users.noreply.github.com&gt;
Co-authored-by: pelikhan &lt;4175913+pelikhan@users.noreply.github.com&gt;
Co-authored-by: Peli de Halleux &lt;pelikhan@users.noreply.github.com&gt;
diff --git a/pkg/cli/mcp_tools_readonly.go b/pkg/cli/mcp_tools_readonly.go
@@ -129,7 +129,7 @@ Returns JSON array with validation results for each workflow:
 		if args.Zizmor || args.Poutine || args.Actionlint {
 			// Check if Docker images are available; if not, start downloading and return retry message
 			if err := CheckAndPrepareDockerImages(ctx, args.Zizmor, args.Poutine, args.Actionlint); err != nil {
-				return nil, nil, newMCPError(jsonrpc.CodeInternalError, "docker images not ready", err.Error())
+				return nil, nil, newMCPError(jsonrpc.CodeInternalError, err.Error(), nil)
 			}
 
 			// Check for cancellation after Docker image preparation

Original file line number	Diff line number	Diff line change
`@@ -129,7 +129,7 @@ Returns JSON array with validation results for each workflow:`
`129`	`129`	`if args.Zizmor \|\| args.Poutine \|\| args.Actionlint {`
`130`	`130`	`// Check if Docker images are available; if not, start downloading and return retry message`
`131`	`131`	`if err := CheckAndPrepareDockerImages(ctx, args.Zizmor, args.Poutine, args.Actionlint); err != nil {`
`132`		`- return nil, nil, newMCPError(jsonrpc.CodeInternalError, "docker images not ready", err.Error())`
	`132`	`+ return nil, nil, newMCPError(jsonrpc.CodeInternalError, err.Error(), nil)`
`133`	`133`	`}`
`134`	`134`
`135`	`135`	`// Check for cancellation after Docker image preparation`