Skip to content

fix: align integration test naming and tooling config - #556

Merged
ixartz merged 8 commits into
mainfrom
small-improvement11
May 12, 2026
Merged

ixartz merged 8 commits into
mainfrom
small-improvement11

Conversation

@ixartz

@ixartz ixartz commented May 11, 2026

Copy link
Copy Markdown
Owner

Summary by CodeRabbit

  • Documentation

    • Updated testing conventions (integration vs. E2E), Vitest config reference, and clarified Sentry production setup wording.
  • Tests

    • Standardized integration test filename pattern, updated test matching, and made test assertions Unicode-aware.
  • Chores

    • Adjusted tooling/preset and pre-commit analysis configs, updated test-related scripts, and bumped multiple dependency/devDependency versions.
  • Bug Fixes

    • Middleware routing updated to exclude API paths from processing.

Review Change Stack

@vercel

vercel Bot commented May 11, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
next-js-boilerplate Ready Ready Preview, Comment May 12, 2026 2:11pm

@coderabbitai

coderabbitai Bot commented May 11, 2026

Copy link
Copy Markdown

Note

Reviews paused

It looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the reviews.auto_review.auto_pause_after_reviewed_commits setting.

Use the following commands to manage reviews:

  • @coderabbitai resume to resume automatic reviews.
  • @coderabbitai review to trigger a single review.

Use the checkboxes below for quick actions:

  • ▶️ Resume reviews
  • 🔍 Trigger review

Walkthrough

Updates test naming to use *.integ.ts for integration tests, switches Vitest config references from .mts to .ts, extends linting for Vitest, adjusts Knip and lefthook entries, bumps package versions, excludes api from middleware, and tweaks docs/comments.

Changes

Testing & Tooling Configuration Updates

Layer / File(s) Summary
Test Naming Convention Alignment
AGENTS.md, README.md, playwright.config.ts, tsconfig.json, knip.config.ts
Integration test naming changed from *.spec.ts to *.integ.ts across agent guidance, README, Playwright testMatch, TypeScript exclude/include rules, and Knip Playwright entries.
Vitest Configuration File Migration
README.md, package.json, tsconfig.json
Vitest config references migrated from .mts to .ts in docs and the storybook:test script; TypeScript include globs removed **/*.mts.
Linting, Knip & TreatConfig
oxlint.config.ts, knip.config.ts
Oxlint adds the Vitest preset; Knip ignores/ignoreDependencies adjusted, Playwright test entries added, CSS regex flags changed, and treatConfigHintsAsErrors enabled.
Lefthook knip job key reorder
lefthook.yml
Reordered run and glob keys for the knip pre-commit job.
Package.json dependency & devDependency bumps
package.json
Multiple dependency and devDependency version bumps and related tooling updates.
Storybook Preview Regex
.storybook/preview.ts
Storybook controls.matchers regexes updated to include the Unicode (u) flag.
Tests / Regex Updates
src/utils/Helpers.test.ts, src/templates/BaseTemplate.test.tsx, tests/e2e/Sanity.check.e2e.ts
Test regex literals made Unicode-aware (u flag) and a nested describe block renamed to getI18nPath function.
Middleware Matcher Exclusion
src/proxy.ts
Next.js middleware config.matcher updated to exclude api paths from proxy middleware.
Documentation & Code Comment Improvements
src/utils/AppConfig.ts, src/utils/Helpers.ts, src/app/[locale]/(marketing)/page.tsx, README.md
AppConfig header clarified, JSDoc @param formatting adjusted, marketing JSX whitespace/comment inserted, and Sentry README wording/link updated.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

Suggested labels

released

Poem

🐰 A rabbit hops through CI and docs,
From .spec to .integ I dance on logs,
Vitest dons .ts as springtime shows,
Lint and Knip tidy up the rows,
Proxy skips api — onwards we go!

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly summarizes the main change: aligning integration test naming conventions (*.integ.ts) and updating tooling configurations across the project to support the new test naming scheme.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch small-improvement11

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

  • Generate code and open pull requests
  • Plan features and break down work
  • Investigate incidents and troubleshoot customer tickets together
  • Automate recurring tasks and respond to alerts with triggers
  • Summarize progress and report instantly

Built for teams:

  • Shared memory across your entire org—no repeating context
  • Per-thread sandboxes to safely plan and execute work
  • Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started


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.

@codecov

codecov Bot commented May 12, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 8 lines in your changes missing coverage. Please review.
✅ Project coverage is 5.88%. Comparing base (b4daeaf) to head (889a7e3).

Files with missing lines Patch % Lines
src/app/[locale]/(auth)/dashboard/layout.tsx 0.00% 5 Missing ⚠️
src/app/[locale]/(auth)/dashboard/page.tsx 0.00% 1 Missing ⚠️
...ashboard/user-profile/[[...user-profile]]/page.tsx 0.00% 1 Missing ⚠️
src/proxy.ts 0.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@           Coverage Diff            @@
##            main    #556      +/-   ##
========================================
+ Coverage   5.76%   5.88%   +0.11%     
========================================
  Files         43      43              
  Lines        295     289       -6     
  Branches      30      30              
========================================
  Hits          17      17              
+ Misses       256     250       -6     
  Partials      22      22              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ixartz
ixartz merged commit 1174af3 into main May 12, 2026
15 checks passed
@github-actions

Copy link
Copy Markdown

🎉 This PR is included in version 6.3.3 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant