Next config update

[openint-bot]

Important

Update Next.js configuration to TypeScript, adjust Sentry logs, and remove handlebars dependency.

• Next.js Configuration:
  • Convert next.config.js to next.config.ts with TypeScript imports and type annotations.
  • Update serverExternalPackages and add experimental turbo configuration for .node files.
  • Remove handlebars alias from webpack configuration.
• Sentry Configuration:
  • Update initialization log messages in sentry.client.config.ts and sentry.server.config.ts to include context (e.g., [sentry.client] initialized).
• Dependencies:
  • Remove handlebars from connector-postgres/package.json dependencies.
• Miscellaneous:
  • Update comment link in next-env.d.ts.
  • Simplify turbo.json by removing globalPassThroughEnv entries.

^{This description was created by for 5b07dbb. It will automatically update as commits are pushed.}

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
openint	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Mar 8, 2025 5:04pm

[recurseml]

😱 Found 1 issue. Time to roll up your sleeves! 😱

🗒️ View all ignored comments in this repo

• This export statement is redundant as it appears both in the diff and original source code. Adding it again will cause a duplicate export error. The line should be removed since it already exists in the source code.

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to 5b07dbb in 2 minutes and 14 seconds

More details

• Looked at 279 lines of code in 6 files
• Skipped 1 files when reviewing.
• Skipped posting 12 drafted comments based on config settings.

1. apps/web/next.config.ts:162

• Draft comment:
  Double-check that using process.env['SENTRY_ORG'] for both org and project is intentional. Usually project might come from a separate env var.
• Reason this comment was not posted:
  Comment was on unchanged code.

2. apps/web/sentry.server.config.ts:2

• Draft comment:
  Update header comments to reflect this is the server configuration, not for browser.
• Reason this comment was not posted:
  Comment was not on a location in the diff, so it can't be submitted as a review comment.

3. apps/web/next-env.d.ts:5

• Draft comment:
  Verify that the updated documentation link is correct.
• Reason this comment was not posted:
  Confidence changes required: 33% <= threshold 50%
  None

4. apps/web/next.config.ts:34

• Draft comment:
  Confirm that replacing 'serverComponentsExternalPackages' with 'serverExternalPackages' is supported by your Next.js version.
• Reason this comment was not posted:
  Comment did not seem useful. Confidence is useful = 0% <= threshold 50%
  This comment is asking the PR author to confirm compatibility with their Next.js version, which violates the rule against asking the author to confirm or ensure compatibility. It does not provide a specific code suggestion or point out a specific issue with the code.

5. apps/web/next.config.ts:71

• Draft comment:
  Removal of the swcMinify flag: ensure this change is intentional and its impact on minification and build performance is acceptable.
• Reason this comment was not posted:
  Comment did not seem useful. Confidence is useful = 40% <= threshold 50%
  The comment is asking the PR author to ensure the change is intentional and to consider its impact, which violates the rule against asking for confirmation or ensuring behavior. However, it does point out a specific change (removal of swcMinify) and its potential impact, which could be useful if rephrased to suggest checking the impact or adding tests.

6. apps/web/next.config.ts:81

• Draft comment:
  The alias configuration for 'handlebars' has been removed. Verify that this removal won't break modules relying on 'handlebars/dist/handlebars.js'.
• Reason this comment was not posted:
  Comment did not seem useful. Confidence is useful = 0% <= threshold 50%
  The comment is asking the PR author to verify that the removal of an alias configuration won't break modules. This falls under asking the author to ensure behavior is intended or to double-check things, which is against the rules.

7. apps/web/next.config.ts:73

• Draft comment:
  Review the experimental turbo configuration to ensure compatibility with Next.js and that it integrates well with existing webpack rules.
• Reason this comment was not posted:
  Confidence changes required: 33% <= threshold 50%
  None

8. apps/web/sentry.client.config.ts:76

• Draft comment:
  Updated client-side Sentry log label adds clarity.
• Reason this comment was not posted:
  Confidence changes required: 0% <= threshold 50%
  None

9. apps/web/sentry.server.config.ts:34

• Draft comment:
  Updated server-side Sentry log label enhances clarity.
• Reason this comment was not posted:
  Confidence changes required: 0% <= threshold 50%
  None

10. connectors/connector-postgres/package.json:12

• Draft comment:
  Removal of the 'handlebars' dependency: ensure that no consumers require it directly or that it is provided transitively.
• Reason this comment was not posted:
  Comment did not seem useful. Confidence is useful = 0% <= threshold 50%
  This comment is asking the PR author to ensure that no consumers require the 'handlebars' dependency directly or that it is provided transitively. This falls under the rule of not asking the author to ensure or verify things, which is not allowed. The comment does not provide a specific code suggestion or point out a specific issue with the code.

11. turbo.json:1

• Draft comment:
  The removal of 'globalPassThroughEnv' from turbo.json may affect tasks that rely on those environment variables. Confirm this change is intentional.
• Reason this comment was not posted:
  Decided after close inspection that this draft comment was likely wrong and/or not actionable: usefulness confidence = 10% vs. threshold = 50%
  This comment violates several rules. It's asking for confirmation of intention ("Confirm this change is intentional"). It's speculative ("may affect tasks"). It's not making a clear assertion that something is definitely wrong. The author must have intentionally removed these variables, and if it breaks something, the build/tests would catch it.
  Maybe removing these environment variables could cause silent runtime issues that tests wouldn't catch? Maybe this is a critical configuration change that deserves extra attention?
  Even if there are potential issues, asking for confirmation of intention isn't helpful - the author clearly intended to remove these variables. If there are specific problems, they should be pointed out directly.
  Delete this comment. It violates our rules by asking for confirmation of intention and making speculative statements without pointing out specific issues.

12. apps/web/next.config.ts:72

• Draft comment:
  Typo: In the comment at Line 72, "Wtrubo pack and webpack configs...", it appears that 'Wtrubo' is a typo. Please update it to 'Turbopack' if that is the intended term.
• Reason this comment was not posted:
  Decided after close inspection that this draft comment was likely wrong and/or not actionable: usefulness confidence = 10% vs. threshold = 50%
  While the typo identification is correct, this is just a comment typo that doesn't affect functionality. The code itself uses the correct "turbo" in the configuration. Comments are documentation and while accuracy is good, this is a very minor issue that doesn't warrant a PR comment. Our rules state we should not make purely informative comments or comments about obvious/unimportant issues.
  The typo could potentially confuse future developers looking at this code. Comments are part of code maintenance and clarity.
  While documentation clarity is important, this typo is obvious enough that any developer would understand the intent, and it's not worth the overhead of a PR comment and review cycle to fix such a minor issue.
  Delete this comment as it violates our rule about not making purely informative comments or comments about unimportant issues.

Workflow ID: wflow_uDyhDovdt4Zwnmag

---

You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.