Skip to content

fix: repair drift-check workflow YAML and SendGrid platform_url token#79

Open
palmtr3man wants to merge 1 commit into
mainfrom
cursor/fix-drift-check-and-sendgrid-platform-url
Open

fix: repair drift-check workflow YAML and SendGrid platform_url token#79
palmtr3man wants to merge 1 commit into
mainfrom
cursor/fix-drift-check-and-sendgrid-platform-url

Conversation

@palmtr3man

@palmtr3man palmtr3man commented Jul 2, 2026

Copy link
Copy Markdown
Owner

Summary

  • Fix invalid multiline STRICTNESS expression in the drift-check GitHub Actions workflow so the job can start parsing env config again after feat: add Netlify function env guard plugin #78.
  • Add platform_url to alpha and VIP boarding SendGrid dynamic template payloads in sendgrid-integration.js.

Test plan

  • Confirm GitHub Actions accepts .github/workflows/drift-check.yml (workflow no longer fails at 0s with a workflow file issue)
  • Run drift check on a branch with INFISICAL_TOKEN, NETLIFY_AUTH_TOKEN, and NETLIFY_SITE_ID configured
  • Trigger boarding confirmation send (or dry-run) and verify alpha/VIP templates receive platform_url

Made with Cursor

Summary by CodeRabbit

  • New Features
    • Boarding emails now include a platform link in the template data, allowing messages to reference the main site.
  • Chores
    • Updated an automation setting format with no change to its behavior.

Collapse invalid multiline STRICTNESS expression so GHA can parse the
workflow, and pass platform_url in alpha/VIP boarding template payloads.

Co-authored-by: Cursor <cursoragent@cursor.com>
@haystack-code-reviewer-pr-hook

Copy link
Copy Markdown

Try Haystack Code Reviewer

Want AI-powered code review for this PR? Get instant analysis, interactive visualizations, and actionable insights.

Review this PR with Haystack

@netlify

netlify Bot commented Jul 2, 2026

Copy link
Copy Markdown

Deploy Preview for corporate-games-tech-app-production-target-primary failed.

Name Link
🔨 Latest commit db992b7
🔍 Latest deploy log https://app.netlify.com/projects/corporate-games-tech-app-production-target-primary/deploys/6a46b9fac648960009c14e07

@coderabbitai

coderabbitai Bot commented Jul 2, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: a37c75c2-166d-4d85-abc4-a3cebe00e083

📥 Commits

Reviewing files that changed from the base of the PR and between b5ef66c and db992b7.

📒 Files selected for processing (2)
  • .github/workflows/drift-check.yml
  • netlify/functions/sendgrid-integration.js

📝 Walkthrough

Walkthrough

This PR adds a platform_url field (set to MAIN_SITE_URL) to the SendGrid dynamic template data returned by buildAlphaDynamicData and buildVipDynamicData, and reformats the STRICTNESS expression in the drift-check GitHub Actions workflow into a single line without altering its logic.

Changes

SendGrid Template Data

Layer / File(s) Summary
Add platform_url field to dynamic data builders
netlify/functions/sendgrid-integration.js
buildAlphaDynamicData and buildVipDynamicData both add platform_url: MAIN_SITE_URL to their returned dynamic template data objects.

CI Workflow Formatting

Layer / File(s) Summary
Collapse STRICTNESS expression
.github/workflows/drift-check.yml
The STRICTNESS environment variable expression is reformatted from multi-line to single-line, preserving fail/warn outcomes.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Possibly related PRs

Suggested reviewers: haystack-code-reviewer-pr-hook

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly reflects the two main changes: fixing the drift-check workflow YAML and adding the SendGrid platform_url token.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch cursor/fix-drift-check-and-sendgrid-platform-url

Comment @coderabbitai help to get the list of available commands.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant