Summary
ESLint factory found a large cluster of gh-aw-custom/prefer-get-error-message-over-string warnings across actions/setup/js. These all point to the same root cause: caught errors are being stringified with String(err) even when getErrorMessage() is already available in scope.
Affected files
actions/setup/js/apply_safe_outputs_replay.cjs
actions/setup/js/apply_samples.cjs
actions/setup/js/artifact_client.cjs
actions/setup/js/build_checkout_manifest.cjs
actions/setup/js/check_workflow_recompile_needed.cjs
actions/setup/js/convert_gateway_config_shared.cjs
actions/setup/js/create_code_scanning_alert.cjs
actions/setup/js/create_pull_request.cjs
actions/setup/js/frontmatter_hash_pure.cjs
actions/setup/js/generate_aw_info.cjs
actions/setup/js/generate_git_bundle.cjs
actions/setup/js/generate_git_patch.cjs
actions/setup/js/generate_safe_outputs_tools.cjs
actions/setup/js/github_rate_limit_logger.cjs
actions/setup/js/handle_agent_failure.cjs
actions/setup/js/handle_detection_runs.cjs
actions/setup/js/handle_noop_message.cjs
actions/setup/js/install_frontmatter_skills.cjs
actions/setup/js/load_experiment_state_from_repo.cjs
actions/setup/js/mcp_scripts_config_loader.cjs
actions/setup/js/merge_remote_agent_github_folder.cjs
actions/setup/js/messages_core.cjs
actions/setup/js/mount_mcp_as_cli.cjs
actions/setup/js/parse_mcp_gateway_log.cjs
actions/setup/js/patch_awf_chroot_config.cjs
actions/setup/js/pick_experiment.cjs
actions/setup/js/push_repo_memory.cjs
actions/setup/js/push_to_pull_request_branch.cjs
actions/setup/js/route_slash_command.cjs
actions/setup/js/run_evals.cjs
actions/setup/js/runtime_import.cjs
actions/setup/js/safe_outputs_config.cjs
actions/setup/js/safe_outputs_handlers.cjs
actions/setup/js/safe_outputs_tools_loader.cjs
actions/setup/js/setup_comment_memory_files.cjs
actions/setup/js/setup_threat_detection.cjs
actions/setup/js/start_mcp_gateway.cjs
actions/setup/js/upload_artifact.cjs
Representative diagnostics
actions/setup/js/apply_safe_outputs_replay.cjs:76:63 — Use getErrorMessage(err) instead of String(err)
actions/setup/js/artifact_client.cjs:311:69 — Use getErrorMessage(err) instead of String(err)
actions/setup/js/create_pull_request.cjs:1217:68 — Use getErrorMessage(err) instead of String(err)
actions/setup/js/route_slash_command.cjs:41:82 — Use getErrorMessage(error) instead of String(error)
Expected outcome
Replace String(err)/String(error) with getErrorMessage(...) anywhere that helper is already imported or otherwise available in scope, while keeping behavior limited to actions/setup/js.
Checklist
Generated by 🧹 ESLint Monster · gpt54 · 13.1 AIC · ⌖ 5.29 AIC · ⊞ 4.8K · ◷
Summary
ESLint factory found a large cluster of
gh-aw-custom/prefer-get-error-message-over-stringwarnings acrossactions/setup/js. These all point to the same root cause: caught errors are being stringified withString(err)even whengetErrorMessage()is already available in scope.Affected files
actions/setup/js/apply_safe_outputs_replay.cjsactions/setup/js/apply_samples.cjsactions/setup/js/artifact_client.cjsactions/setup/js/build_checkout_manifest.cjsactions/setup/js/check_workflow_recompile_needed.cjsactions/setup/js/convert_gateway_config_shared.cjsactions/setup/js/create_code_scanning_alert.cjsactions/setup/js/create_pull_request.cjsactions/setup/js/frontmatter_hash_pure.cjsactions/setup/js/generate_aw_info.cjsactions/setup/js/generate_git_bundle.cjsactions/setup/js/generate_git_patch.cjsactions/setup/js/generate_safe_outputs_tools.cjsactions/setup/js/github_rate_limit_logger.cjsactions/setup/js/handle_agent_failure.cjsactions/setup/js/handle_detection_runs.cjsactions/setup/js/handle_noop_message.cjsactions/setup/js/install_frontmatter_skills.cjsactions/setup/js/load_experiment_state_from_repo.cjsactions/setup/js/mcp_scripts_config_loader.cjsactions/setup/js/merge_remote_agent_github_folder.cjsactions/setup/js/messages_core.cjsactions/setup/js/mount_mcp_as_cli.cjsactions/setup/js/parse_mcp_gateway_log.cjsactions/setup/js/patch_awf_chroot_config.cjsactions/setup/js/pick_experiment.cjsactions/setup/js/push_repo_memory.cjsactions/setup/js/push_to_pull_request_branch.cjsactions/setup/js/route_slash_command.cjsactions/setup/js/run_evals.cjsactions/setup/js/runtime_import.cjsactions/setup/js/safe_outputs_config.cjsactions/setup/js/safe_outputs_handlers.cjsactions/setup/js/safe_outputs_tools_loader.cjsactions/setup/js/setup_comment_memory_files.cjsactions/setup/js/setup_threat_detection.cjsactions/setup/js/start_mcp_gateway.cjsactions/setup/js/upload_artifact.cjsRepresentative diagnostics
actions/setup/js/apply_safe_outputs_replay.cjs:76:63— UsegetErrorMessage(err)instead ofString(err)actions/setup/js/artifact_client.cjs:311:69— UsegetErrorMessage(err)instead ofString(err)actions/setup/js/create_pull_request.cjs:1217:68— UsegetErrorMessage(err)instead ofString(err)actions/setup/js/route_slash_command.cjs:41:82— UsegetErrorMessage(error)instead ofString(error)Expected outcome
Replace
String(err)/String(error)withgetErrorMessage(...)anywhere that helper is already imported or otherwise available in scope, while keeping behavior limited toactions/setup/js.Checklist
gh-aw-custom/prefer-get-error-message-over-stringfindings inactions/setup/jsString(err)/String(error)withgetErrorMessage(...)actions/setup/jsnpm run lint:setup-js