add react in runtime in og edge function

[yujonglee]

No description provided.

[netlify]

✅ Deploy Preview for hyprnote ready!

Name	Link
🔨 Latest commit	1f39b7d
🔍 Latest deploy log	https://app.netlify.com/projects/hyprnote/deploys/691c4bf94d8c0800088ff023
😎 Deploy Preview	https://deploy-preview-1716--hyprnote.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify project configuration.

[coderabbitai]

📝 Walkthrough

Walkthrough

Two configuration and dependency updates: a React import was added to the OG edge function with lint ignore directives, and the dprint formatter configuration was updated to exclude the netlify edge-functions directory from formatting.

Changes

Cohort / File(s)	Change Summary
Edge Function Import apps/web/netlify/edge-functions/og.tsx	Added React import with deno-lint-ignore directives for imports and unused variables.
Formatter Configuration dprint.json	Added "**/netlify/edge-functions" exclude pattern to the excludes list alongside existing schema.json entry.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

• Both changes are straightforward and non-functional: a simple import addition and a configuration entry.

Possibly related PRs

• OG image with netlify function #1700: Directly modifies the same OG edge-function file (apps/web/netlify/edge-functions/og.tsx).

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Description check	❓ Inconclusive	No description was provided by the author, making it impossible to assess relevance to the changeset.	Add a pull request description explaining why React is needed in the OG edge function and any context about the dprint.json changes.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately summarizes the main change: adding React import to the OG edge function file.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch yl-branch-16

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (1)

apps/web/netlify/edge-functions/og.tsx (1)

1-2: Verify whether React import is necessary or if JSX transform can be updated.

React 18.2.0 supports the new JSX transform (since React 17), which doesn't require explicit React imports for JSX to work. The deno-lint-ignore no-unused-vars suggests React isn't actively used in the code—it's only present for JSX syntax.

Consider verifying: Is your build configured for the new JSX transform? If so, you can remove the React import and the lint-ignore directive entirely.

📜 Review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between bf1b730 and 1f39b7d.

📒 Files selected for processing (2)

• apps/web/netlify/edge-functions/og.tsx (1 hunks)
• dprint.json (1 hunks)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: fmt

🔇 Additional comments (1)

dprint.json (1)

34-34: Good call excluding netlify/edge-functions from dprint formatting.

This makes sense given that Deno edge-functions may have different linting and formatting requirements than the main codebase, and it aligns well with adding React to the runtime in that directory.