[review] fix(setup-js): wrap all sync fs calls in try/catch (eslint-monster remediation)#46217
[review] fix(setup-js): wrap all sync fs calls in try/catch (eslint-monster remediation)#46217github-actions[bot] wants to merge 12 commits into
Conversation
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Resolve conflict in .github/workflows/release.lock.yml: - Keep v7.0.0 version tag for actions/setup-go from our branch - Incorporate main's format change (manifest header on single line) - Pick up mai-code-1-flash-picker model addition and other main changes Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
…rkflows Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
- upload_artifact.cjs: wrap copyFileSync in try/catch with descriptive error - evaluate_outcomes.cjs: wrap renameSync in try/catch with descriptive error - start_mcp_gateway.cjs: wrap chmodSync(configDir) and chmodSync(outputPath) in try/catch; wrap both copyFileSync fallback/else branches in try/catch - convert_gateway_config_shared.cjs: move chmodSync inside existing try/catch - action_setup_otlp.cjs: move success log inside try so it only fires on success - detect_agent_errors.cjs: emit stderr warning instead of silently ignoring GITHUB_OUTPUT write failures Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: gh-aw-bot <259018956+gh-aw-bot@users.noreply.github.com>
Co-authored-by: gh-aw-bot <259018956+gh-aw-bot@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
…r-wrap-fs-calls Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
|
Thanks for keeping this narrowly scoped to the sync This repo's CONTRIBUTING.md says non-core contributors should not open direct PRs; contributions are expected to come through issues/agentic plans, with core team members creating implementation PRs. This PR also appears to be authored by automation while GitHub reports the PR author association as Additional improvement: the diff only adds a changeset file and does not include test-file changes, so reviewers may need confirmation that coverage already exists or that tests were intentionally unnecessary.
|
|
@copilot please run the
|
Caution
agentic threat detected
Threat detection flagged this output in warn mode. Manual review is REQUIRED before any follow-up automation.
Reason: agent_failure
Review the workflow run logs for details.
This PR contains changes that were originally intended for PR #46102 (
copilot/eslint-monster-wrap-fs-calls).Please review the changes carefully before merging.