Repository navigation
Clarify GitHub API quota failures and reset diagnostics #15075
Annotations
10 warnings and 1 notice
|
Lint JavaScript files:
actions/setup/js/agy_harness.cjs#L80
Wrap fs.writeFileSync(path.join(settings, "settings.json")) in try/catch — synchronous fs methods throw on I/O errors (missing file, permission denied, disk full); without a call-site try/catch, you lose the original error context and get a generic engine-level stack instead of a specific message with `{ cause }`
|
|
Lint JavaScript files:
actions/setup/js/agy_harness.cjs#L79
Wrap fs.mkdirSync(settings) in try/catch — mkdirSync throws on permission denied, invalid path, or filesystem errors; without a call-site try/catch, you lose the original error context and get a generic engine-level stack instead of a specific message with `{ cause }`
|
|
Lint JavaScript files:
actions/setup/js/agy_harness.cjs#L60
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 JavaScript files:
actions/setup/js/agy_harness.cjs#L56
Wrap fs.readFileSync(archive) in try/catch — synchronous fs methods throw on I/O errors (missing file, permission denied, disk full); without a call-site try/catch, you lose the original error context and get a generic engine-level stack instead of a specific message with `{ cause }`
|
|
Lint JavaScript files:
actions/setup/js/agy_harness.cjs#L29
Wrap fs.mkdtempSync(path.join(os.tmpdir(), "gh-aw-agy-")) in try/catch — mkdtempSync throws on missing parent directory, permission denied, or filesystem errors; without a call-site try/catch, you lose the original error context and get a generic engine-level stack instead of a specific message with `{ cause }`
|
|
Lint JavaScript files:
actions/setup/js/agent_session.cjs#L235
Empty catch block silently swallows the error. Log it (e.g. core.debug/core.warning), assign a fallback value, or add an explicit comment explaining why the error is intentionally ignored
|
|
Lint JavaScript files:
actions/setup/js/agent_session.cjs#L212
Empty catch block silently swallows the error. Log it (e.g. core.debug/core.warning), assign a fallback value, or add an explicit comment explaining why the error is intentionally ignored
|
|
Lint JavaScript files:
actions/setup/js/agent_execution.cjs#L257
Empty catch block silently swallows the error. Log it (e.g. core.debug/core.warning), assign a fallback value, or add an explicit comment explaining why the error is intentionally ignored
|
|
Lint JavaScript files:
actions/setup/js/agent_execution.cjs#L176
Wrap JSON.parse(providerError[1]) in try/catch to avoid uncaught runtime failures in actions/setup/js
|
|
Lint JavaScript files:
actions/setup/js/agent_execution.cjs#L149
Empty catch block silently swallows the error. Log it (e.g. core.debug/core.warning), assign a fallback value, or add an explicit comment explaining why the error is intentionally ignored
|
|
"The ubuntu-latest label will migrate to Ubuntu 26 beginning October 19, 2026. For more information, see https://github.com/actions/runner-images/issues/14748"
|
background
wait
wait-all
cancel
parallel
Loading