Skip to content
CJS

feat: add commit-id config option to pin review attribution to the reviewed commit #9809

Sign in to view logs

feat: add commit-id config option to pin review attribution to the reviewed commit

feat: add commit-id config option to pin review attribution to the reviewed commit #9809

Re-run triggered July 29, 2026 08:05
@pelikhanpelikhan
#48738
copilot/fix-pr-review-commit-id-attribution
Status Success
Total duration 3m 45s
Artifacts –

cjs.yml

on: pull_request
js-typecheck
35s
js-typecheck
impacted-js-tests
3m 37s
impacted-js-tests
lint-js
1m 20s
lint-js
Artifact API Integration
46s
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