Skip to content

fix(marketing): resolve Astro type warnings and Node indexnow error#809

Open
BlueHotDog wants to merge 1 commit intomainfrom
fix/marketing-astro-warnings
Open

fix(marketing): resolve Astro type warnings and Node indexnow error#809
BlueHotDog wants to merge 1 commit intomainfrom
fix/marketing-astro-warnings

Conversation

@BlueHotDog
Copy link
Copy Markdown
Collaborator

@BlueHotDog BlueHotDog commented Apr 8, 2026

Summary

  • Extend Window interface in env.d.ts to declare trackEvent, fixing two ts(2568) warnings in googleAnalytics.astro
  • Replace deprecated Props import in Head.astro with StarlightRouteData from @astrojs/starlight/route-data
  • Add is:inline directive to JSON-LD script tag in Head.astro to silence astro(4000) hint
  • Remove astro.config.mjs import from indexnow.mjs — Node 24 refuses to strip types from hc-starlight's raw .ts exports in node_modules; the site URL is inlined directly instead

Test Plan

  • astro check reports 0 warnings on the 4 previously flagged hints
  • node apps/marketing/scripts/indexnow.mjs runs without the ERR_UNSUPPORTED_NODE_MODULES_TYPE_STRIPPING error

🤖 Generated with Claude Code


Open with Devin

- Extend Window interface in env.d.ts for trackEvent property
- Replace deprecated Props import in Head.astro with StarlightRouteData
- Add is:inline to JSON-LD script tag in Head.astro
- Remove astro.config.mjs import from indexnow.mjs to avoid Node
  failing on hc-starlight's raw .ts node_modules exports

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 2 additional findings.

Open in Devin Review

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