Skip to content

[jsweep] Clean validate_secrets.cjs #8835

[jsweep] Clean validate_secrets.cjs

[jsweep] Clean validate_secrets.cjs #8835

Re-run triggered July 13, 2026 21:15
Status Success
Total duration 2m 8s
Artifacts

cjs.yml

on: pull_request
js-typecheck
43s
js-typecheck
impacted-js-tests
2m 0s
impacted-js-tests
lint-js
1m 7s
lint-js
Artifact API Integration
27s
Artifact API Integration
Matrix: js-tests
Fit to window
Zoom out
Zoom in

Annotations

10 warnings
lint-js: actions/setup/js/apply_samples.cjs#L539
Wrap fs.appendFileSync(logPath) in try/catch — synchronous fs methods throw on I/O errors (missing file, permission denied, disk full) and will crash the action if unhandled
lint-js: actions/setup/js/apply_samples.cjs#L443
`new Error(...)` inside catch (err) references err but omits `{ cause: err }` — the original stack trace will be lost. Add `{ cause: err }` as the second argument
lint-js: actions/setup/js/apply_samples.cjs#L403
Wrap fs.writeFileSync(tmpPatch) in try/catch — synchronous fs methods throw on I/O errors (missing file, permission denied, disk full) and will crash the action if unhandled
lint-js: actions/setup/js/apply_samples.cjs#L97
spawnSync result must have its .error property checked. When the child process cannot be spawned (e.g. ENOENT, ETIMEDOUT), result.status is null and result.error contains the cause — checking only result.status produces a misleading diagnostic. Add: if (result.error) { throw result.error; }
lint-js: actions/setup/js/apply_samples.cjs#L67
`new Error(...)` inside catch (err) references err but omits `{ cause: err }` — the original stack trace will be lost. Add `{ cause: err }` as the second argument
lint-js: actions/setup/js/apply_safe_outputs_replay.cjs#L102
Wrap fs.readFileSync(agentOutputFile) in try/catch — synchronous fs methods throw on I/O errors (missing file, permission denied, disk full) and will crash the action if unhandled
lint-js: actions/setup/js/add_workflow_run_comment.cjs#L444
Avoid using a string concatenation expression as the route argument of github.request(). Use the typed placeholder form instead — e.g. github.request("GET /repos/{owner}/{repo}", { owner, repo }) — to preserve typed dispatch and prevent malformed paths
lint-js: actions/setup/js/add_reaction_and_edit_comment.cjs#L266
Avoid using a template literal with interpolations as the route argument of github.request(). Use the typed placeholder form instead — e.g. github.request("GET /repos/{owner}/{repo}", { owner, repo }) — to preserve typed dispatch and prevent malformed paths
lint-js: actions/setup/js/add_reaction.cjs#L193
Avoid using a template literal with interpolations as the route argument of github.request(). Use the typed placeholder form instead — e.g. github.request("GET /repos/{owner}/{repo}", { owner, repo }) — to preserve typed dispatch and prevent malformed paths
lint-js: actions/setup/js/action_setup_otlp.cjs#L43
Wrap fs.appendFileSync(filePath) in try/catch — synchronous fs methods throw on I/O errors (missing file, permission denied, disk full) and will crash the action if unhandled