Skip to content

[Sync] Update project files from source repository (55c7f74) - #263

Merged
mrz1836 merged 1 commit into
masterfrom
chore/sync-files-go-mail-20260803-192054-55c7f74
Aug 3, 2026
Merged

mrz1836 merged 1 commit into
masterfrom
chore/sync-files-go-mail-20260803-192054-55c7f74

Conversation

@mrz1836

@mrz1836 mrz1836 commented Aug 3, 2026

Copy link
Copy Markdown
Owner

What Changed

  • Updated MAGE_X_VERSION from v1.25.0 to v1.25.1 in .github/env/10-mage-x.env
  • Upgraded CodeQL action versions from e4fba868fa4b1b91e1fdab776edc8cfbe6e9fb81 (v4.37.3) to f205ea1c3313d32999d8d6a48b4f6530d4437b38 (v4.37.4) across init, autobuild, and analyze steps in .github/workflows/codeql-analysis.yml
  • Added null-safe handling for deleted user accounts in PR review processing logic in .github/workflows/auto-merge-on-approval.yml by using optional chaining (review.user?.id) and falling back to a unique key pattern (deleted-user:${review.id})
  • Updated CodeQL action versions from v4.37.3 to v4.37.4 in .github/workflows/dependabot-auto-merge.yml and .github/workflows/scorecard.yml

Why It Was Necessary

  • MAGE-X version bump ensures the latest patch release (v1.25.1) is used with potential bug fixes or improvements
  • CodeQL action updates provide the latest security scanning capabilities and potential bug fixes in the analysis tooling
  • Null-safe review handling prevents workflow failures when processing reviews from deleted GitHub accounts, which would previously throw errors when attempting to access review.user.id

Testing Performed

  • Version bumps for pinned GitHub Actions can be validated by checking that the commit hashes correspond to the specified tag versions
  • The null-safe review handling logic should be tested with PRs that have reviews from deleted user accounts to ensure no JavaScript errors occur
  • CI workflows should be monitored on the next runs to verify CodeQL analysis completes successfully with the updated action versions

Impact / Risk

  • Risk: Low - MAGE-X patch version update and CodeQL minor version updates are generally backward compatible
  • Regression: The null-safe handling fix prevents a potential runtime error; existing functionality with active users remains unchanged
  • Breaking Changes: None - all changes are dependency updates and defensive null checks that maintain existing behavior while fixing an edge case

@mrz1836 mrz1836 self-assigned this Aug 3, 2026
@mrz1836 mrz1836 added automated-sync Automated sync PR, e.g. from a fork or external repo automerge Label to automatically merge pull requests that meet all required conditions chore Simple dependency updates or version bumps labels Aug 3, 2026
@github-actions github-actions Bot added update General updates size/M Medium change (51-200 lines) labels Aug 3, 2026
@mrz1836

mrz1836 commented Aug 3, 2026

Copy link
Copy Markdown
Owner Author

LGTM!

@mrz1836
mrz1836 merged commit fe265ae into master Aug 3, 2026
22 checks passed
@github-actions
github-actions Bot deleted the chore/sync-files-go-mail-20260803-192054-55c7f74 branch August 3, 2026 19:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automated-sync Automated sync PR, e.g. from a fork or external repo automerge Label to automatically merge pull requests that meet all required conditions chore Simple dependency updates or version bumps size/M Medium change (51-200 lines) update General updates

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

⌂ 换网址