Skip to content

Conversation

@area73
Copy link
Collaborator

@area73 area73 commented Jan 6, 2026

  • Upgraded various dependencies in package.json, including @astrojs/mdx to 4.3.13, astro to 5.16.6, and eslint to 9.39.2.
  • Updated pnpm-lock.yaml to reflect the new dependency versions and added an override for mdast-util-to-hast.
  • Enhanced CI workflow by adding validation for PUBLIC_GA_MEASUREMENT_ID to ensure it is configured in GitHub repository variables.

Summary by CodeRabbit

  • Chores
    • Updated development and runtime dependencies including Astro, Playwright, Vitest, ESLint, and Lighthouse to latest versions.
    • Enhanced continuous integration pipeline with improved environment variable configuration and validation for better build reliability.

✏️ Tip: You can customize this high-level summary in your review settings.

- Upgraded various dependencies in package.json, including @astrojs/mdx to 4.3.13, astro to 5.16.6, and eslint to 9.39.2.
- Updated pnpm-lock.yaml to reflect the new dependency versions and added an override for mdast-util-to-hast.
- Enhanced CI workflow by adding validation for PUBLIC_GA_MEASUREMENT_ID to ensure it is configured in GitHub repository variables.
@cursor
Copy link

cursor bot commented Jan 6, 2026

PR Summary

CI updates

  • Sets PUBLIC_GA_MEASUREMENT_ID env in Visual-Regression-Tests and Build jobs and adds a validation step that fails if it’s missing.

Dependency upgrades

  • Bumps @astrojs/mdx to 4.3.13, astro to 5.16.6, @playwright/test to 1.57.0, @vitest/coverage-v8 to 4.0.16, eslint to 9.39.2, happy-dom to 20.0.11, lighthouse to 13.0.1, and vitest to 4.0.16.
  • Adds a pnpm override for mdast-util-to-hast@>=13.2.1.

Written by Cursor Bugbot for commit cae5d65. This will update automatically on new commits. Configure here.

@area73 area73 merged commit 3ec34aa into master Jan 6, 2026
7 of 9 checks passed
@area73 area73 deleted the security-update branch January 6, 2026 13:28
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 6, 2026

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

This pull request updates the CI workflow to enforce environment variable configuration for Google Analytics measurement ID and upgrades multiple runtime and development dependencies to newer versions, including Astro, Playwright, Vitest, and ESLint, while adding a pnpm override for mdast-util-to-hast.

Changes

Cohort / File(s) Summary
CI Workflow Configuration
.github/workflows/ci.yml
Adds PUBLIC_GA_MEASUREMENT_ID environment variable propagation across Visual-Regression-Tests and Build jobs; includes validation steps to error if variable is unconfigured and confirmation echoes when present.
Dependency & Manifest Updates
package.json
Bumps 8 dependencies: Astro 5.13.3→5.16.6, @astrojs/mdx 4.3.4→4.3.13, Playwright 1.55.0→1.57.0, Vitest 3.2.4→4.0.16, ESLint 9.34.0→9.39.2, happy-dom 18.0.1→20.0.11, Lighthouse 12.8.1→13.0.1, @vitest/coverage-v8 3.2.4→4.0.16; adds pnpm override for mdast-util-to-hast >=13.2.1.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Poem

🐰 With GA measurement ID now in place,
Vars propagate through workflows with grace,
Dependencies dance to newer refrains,
Astro and Vitest break old chains,
Our CI pipeline runs ever bright! 🌟


📜 Recent review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between bcfd0ab and cae5d65.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (2)
  • .github/workflows/ci.yml
  • package.json

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

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.

3 participants