fix(webapp): Fix for showing the incident UI panel in the side menu

[samejr]

This fix adds uncrypto package to the remix config

[changeset-bot]

⚠️ No Changeset found

Latest commit: 108aa93

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[coderabbitai]

Walkthrough

A single dependency, "uncrypto", is added to the serverDependenciesToBundle configuration array in the Remix config file. This modification instructs the Remix dev server to bundle the uncrypto package with server-side code during the build process.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

• Simple array configuration addition with no logic modifications
• Single file affected with minimal scope
• No complex interactions or error handling considerations

Pre-merge checks and finishing touches

❌ Failed checks (3 warnings)

Check name	Status	Explanation	Resolution
Title Check	⚠️ Warning	The pull request title states "fix(webapp): Fix for showing the incident UI panel in the side menu," but the actual changeset only adds "uncrypto" to the serverDependenciesToBundle list in remix.config.js. The title describes functionality related to an incident UI panel, which does not appear anywhere in the changes. This is a clear mismatch between what the title claims and what the code actually does, making the title misleading and unrelated to the changeset.
Description Check	⚠️ Warning	The pull request description consists of only a single sentence: "This fix adds uncrypto package to the remix config." While this accurately describes what the change does, it is largely incomplete compared to the required template. The description is missing all the structured sections specified in the template, including the issue closure reference, the checklist, testing documentation, changelog entry, and any screenshots. Though the one-liner is accurate, it fails to provide the comprehensive information expected by the repository's contribution standards.	Expand the description to follow the provided template. Add the issue number closure (Closes #), complete the checklist items confirming you've followed the contributing guide and tested the code, describe the testing steps taken, provide a changelog entry, and include any relevant screenshots. This will ensure the pull request meets the repository's documentation standards.
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	You can run @coderabbitai generate docstrings to improve docstring coverage.

✨ 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 fix(webapp)-incident-panel

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 2283ca6 and 108aa93.

📒 Files selected for processing (1)

• apps/webapp/remix.config.js (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). (18)

• GitHub Check: units / internal / 🧪 Unit Tests: Internal (6, 8)
• GitHub Check: units / internal / 🧪 Unit Tests: Internal (4, 8)
• GitHub Check: units / internal / 🧪 Unit Tests: Internal (5, 8)
• GitHub Check: units / internal / 🧪 Unit Tests: Internal (2, 8)
• GitHub Check: units / webapp / 🧪 Unit Tests: Webapp (8, 8)
• GitHub Check: units / internal / 🧪 Unit Tests: Internal (3, 8)
• GitHub Check: units / internal / 🧪 Unit Tests: Internal (1, 8)
• GitHub Check: units / webapp / 🧪 Unit Tests: Webapp (1, 8)
• GitHub Check: units / internal / 🧪 Unit Tests: Internal (7, 8)
• GitHub Check: units / webapp / 🧪 Unit Tests: Webapp (6, 8)
• GitHub Check: units / webapp / 🧪 Unit Tests: Webapp (4, 8)
• GitHub Check: units / webapp / 🧪 Unit Tests: Webapp (2, 8)
• GitHub Check: units / packages / 🧪 Unit Tests: Packages (1, 1)
• GitHub Check: units / webapp / 🧪 Unit Tests: Webapp (5, 8)
• GitHub Check: e2e / 🧪 CLI v3 tests (ubuntu-latest - npm)
• GitHub Check: e2e / 🧪 CLI v3 tests (windows-latest - npm)
• GitHub Check: e2e / 🧪 CLI v3 tests (windows-latest - pnpm)
• GitHub Check: typecheck / typecheck

🔇 Additional comments (1)

apps/webapp/remix.config.js (1)

29-29: The change is correct and necessary.

The bundling of "uncrypto" directly addresses the incident UI panel issue. The trigger-sdk package imports "subtle" from "uncrypto", which is used for Web Crypto operations in webhook processing. Since the Remix server runs in CJS mode (serverModuleFormat: "cjs"), the ESM-only "uncrypto" package must be bundled to ensure proper module resolution at runtime. The incident panel functionality depends on webhook operations, making this bundling necessary to fix the display issue.

---

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.}