Skip to content
CJS

Migrate tools.go to Go 1.24 tool directives; remove dead gosec config from .golangci.yml #9825

Sign in to view logs

Migrate tools.go to Go 1.24 tool directives; remove dead gosec config from .golangci.yml

Migrate tools.go to Go 1.24 tool directives; remove dead gosec config from .golangci.yml #9825

Re-run triggered July 29, 2026 11:43
@pelikhanpelikhan
#48830
copilot/go-fan-review-gosec
Status Success
Total duration 2m 22s
Artifacts –

cjs.yml

on: pull_request
js-typecheck
36s
js-typecheck
impacted-js-tests
2m 10s
impacted-js-tests
lint-js
1m 8s
lint-js
Artifact API Integration
35s
Artifact API Integration
Matrix: js-tests
Fit to window
Zoom out
Zoom in

Annotations

10 warnings
lint-js: actions/setup/js/build_checkout_manifest.cjs#L116
Use getErrorMessage(err) instead of String(err) — getErrorMessage is already available in this scope and produces a cleaner, sanitized message for caught errors
lint-js: actions/setup/js/artifact_client.cjs#L335
Wrap `await fetch(location)` in try/catch — fetch throws TypeError on network errors and will crash the action if unhandled
lint-js: actions/setup/js/artifact_client.cjs#L319
Wrap `await fetch(apiUrl.toString())` in try/catch — fetch throws TypeError on network errors and will crash the action if unhandled
lint-js: actions/setup/js/artifact_client.cjs#L311
Use getErrorMessage(err) instead of String(err) — getErrorMessage is already available in this scope and produces a cleaner, sanitized message for caught errors
lint-js: actions/setup/js/artifact_client.cjs#L269
Wrap `await fetch(url.toString())` in try/catch — fetch throws TypeError on network errors and will crash the action if unhandled
lint-js: actions/setup/js/artifact_client.cjs#L217
Wrap `await fetch(signedUploadURL)` in try/catch — fetch throws TypeError on network errors and will crash the action if unhandled
lint-js: actions/setup/js/apply_samples.cjs#L714
Prefer getErrorMessage(err) from error_helpers.cjs. The `err.stack` ternary surfaces noisy stack frames; getErrorMessage() returns a clean, consistent message
lint-js: actions/setup/js/apply_samples.cjs#L461
Use getErrorMessage(err) instead of String(err) — getErrorMessage is already available in this scope and produces a cleaner, sanitized message for caught errors
lint-js: actions/setup/js/apply_safe_outputs_replay.cjs#L110
Use getErrorMessage(err) instead of String(err) — getErrorMessage is already available in this scope and produces a cleaner, sanitized message for caught errors
lint-js: actions/setup/js/apply_safe_outputs_replay.cjs#L76
Use getErrorMessage(err) instead of String(err) — getErrorMessage is already available in this scope and produces a cleaner, sanitized message for caught errors