Skip to content

Surface stalled Claude CLI failures and bound Design Decision Gate diff input #11212

Surface stalled Claude CLI failures and bound Design Decision Gate diff input

Surface stalled Claude CLI failures and bound Design Decision Gate diff input #11212

Re-run triggered August 18, 2026 12:38
Status Success
Total duration 4m 55s
Artifacts

cjs.yml

on: pull_request
js-typecheck
1m 2s
js-typecheck
impacted-js-tests
46s
impacted-js-tests
lint-js
1m 42s
lint-js
Artifact API Integration
56s
Artifact API Integration
Matrix: js-tests
Fit to window
Zoom out
Zoom in

Annotations

10 warnings
lint-js: actions/setup/js/artifact_client.cjs#L375
Wrap fs.mkdtempSync(path.join(os.tmpdir(), "gh-aw-artifact-download-")) in try/catch — mkdtempSync throws on missing parent directory, permission denied, or filesystem errors and will crash the action if unhandled
lint-js: actions/setup/js/artifact_client.cjs#L299
Wrap response.json() in try/catch — even after explicit HTTP-error handling (for example `if (!response.ok) throw ...`), reading a fetch() Response body can still reject (malformed JSON, truncated/errored stream). Without this call-site try/catch, you lose the original parse error context and get a generic, harder-to-diagnose stack instead of a specific message with `{ cause }`
lint-js: actions/setup/js/artifact_client.cjs#L296
Wrap response.text() in try/catch — even after explicit HTTP-error handling (for example `if (!response.ok) throw ...`), reading a fetch() Response body can still reject (malformed JSON, truncated/errored stream). Without this call-site try/catch, you lose the original parse error context and get a generic, harder-to-diagnose stack instead of a specific message with `{ cause }`
lint-js: actions/setup/js/artifact_client.cjs#L241
Wrap response.text() in try/catch — even after explicit HTTP-error handling (for example `if (!response.ok) throw ...`), reading a fetch() Response body can still reject (malformed JSON, truncated/errored stream). Without this call-site try/catch, you lose the original parse error context and get a generic, harder-to-diagnose stack instead of a specific message with `{ cause }`
lint-js: actions/setup/js/add_workflow_run_comment.cjs#L431
Numeric value 'discussionNumber' parsed from a 'split(...)[index]' expression is never validated with Number.isNaN(), isNaN(), Number.isFinite(), isFinite(), or a truthiness check. A malformed delimited string will silently produce NaN, which can then be passed to downstream API calls
lint-js: actions/setup/js/add_workflow_run_comment.cjs#L422
Numeric value 'discussionNumber' parsed from a 'split(...)[index]' expression is never validated with Number.isNaN(), isNaN(), Number.isFinite(), isFinite(), or a truthiness check. A malformed delimited string will silently produce NaN, which can then be passed to downstream API calls
lint-js: actions/setup/js/add_reaction_and_edit_comment.cjs#L267
Numeric value 'discussionNumber' parsed from a 'split(...)[index]' expression is never validated with Number.isNaN(), isNaN(), Number.isFinite(), isFinite(), or a truthiness check. A malformed delimited string will silently produce NaN, which can then be passed to downstream API calls
lint-js: actions/setup/js/add_reaction_and_edit_comment.cjs#L199
`core.error()` immediately before `core.setFailed()` with the same message is redundant: `core.setFailed()` already logs an error annotation and marks the action failed. Remove the `core.error()` call
lint-js: actions/setup/js/add_reaction.cjs#L184
`core.error()` immediately before `core.setFailed()` with the same message is redundant: `core.setFailed()` already logs an error annotation and marks the action failed. Remove the `core.error()` call
lint-js: actions/setup/js/add_mask_redaction.cjs#L89
Interpolated value `maskedValues.map(escapeRegExp).join("|")` in `new RegExp()` template literal is not passed through a regex-escaping helper. Escape regex metacharacters before interpolating, e.g. `maskedValues.map(escapeRegExp).join("|").replace(/[.*+?^${}()|[\]\\]/g, "\\$&")`