Fix auto-fix build workflow by moving logic to scripts and improving error handling

[backup-bdg-2]

No description provided.

[mentatbot]

I'm getting to work. See my steps and track my cost here.

• Wake on any new activity.
  _{I always see new activity while actively working. When paused, I'll only respond when tagged unless this box is checked.}

[mentatbot]

The multiline regex pattern might not work correctly in all environments. It looks like this is trying to match content between "error_types":{ and a closing }, but the newline and caret characters might cause issues. A simpler single-line pattern would be more reliable.

Suggested change

	ERROR_TYPES_JSON=$(grep -o '"error_types":{[
	^
	}]*}' build_error_report.json)
	ERROR_TYPES_JSON=$(grep -o '"error_types":{[^}]*}' build_error_report.json)

[mentatbot]

Minor optimization: Consider moving the npm installation to the workflow level and reusing it across both scripts that need @octokit/rest. This would avoid installing the same package twice.

[mentatbot]

I've reviewed the changes to the auto-fix build workflow and they look great! The refactoring significantly improves maintainability and error handling.

Strengths

• ✅ Improved organization by moving inline code into dedicated scripts
• ✅ Better error handling with set -eo pipefail and proper exit codes
• ✅ More robust artifact detection with multiple fallbacks
• ✅ Proper environment variable handling and input validation
• ✅ Better logging throughout the process

Suggestions for improvement

There's only one potential issue I found that should be addressed:

[mentatbot]

With these minor adjustments, the PR will be even more robust. The refactoring is a great improvement to the codebase - it makes the workflow much more maintainable and easier to understand.