Skip to content

INTER-2335: Update dev dependencies and remove unused - #257

Merged
erayaydin merged 2 commits into
mainfrom
chore/update-dependencies-inter-2335
Jul 29, 2026
Merged

INTER-2335: Update dev dependencies and remove unused#257
erayaydin merged 2 commits into
mainfrom
chore/update-dependencies-inter-2335

Conversation

@erayaydin

@erayaydin erayaydin commented Jul 23, 2026

Copy link
Copy Markdown
Member

Changes

Removed

  • commitizen + cz-conventional-changelog + package.json > config.commitizen - project uses changesets.
  • @fingerprintjs/conventional-changelog-dx-team - not referenced in config files.
  • package.json > engines.node field - this project targets Cloudflare Workers runtime (uses v8, no need to declare Node.js runtime)

Vite 8 Migration

  • esbuild.banner is deprecated in Vite 8 and no longer injects the banner at the top of the output
  • Replaced with a custom generateBundle Vite plugin that preserves the exact old license banner format

Dependency Updates

Package From To Type
vite 7.3.1 8.1.5 Major
vitest 3.2.4 4.1.10 Major
@vitest/coverage-istanbul 3.2.4 4.1.10 Major
typescript 5.4.4 6.0.3 Major
@commitlint/cli 19.8.1 21.2.1 Major
lint-staged 13.2.2 17.1.1 Major
wrangler 4.75.0 4.113.0 Minor
prettier 3.2.5 3.9.6 Minor
@cloudflare/vite-plugin 1.29.1 1.46.0 Minor
@changesets/cli 2.30.0 2.31.1 Minor
@playwright/test 1.60.0 1.61.1 Minor
vite-plugin-checker 0.10.3 0.14.5 Minor

Update all devDependencies. Remove unused packages. Remove unnecessary
`engines.node` field.

Related-Task: INTER-2335
@github-actions

github-actions Bot commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

Coverage report

St.
Category Percentage Covered / Total
🟢 Statements 99.62% 263/264
🟢 Branches 99.12% 112/113
🟢 Functions 100% 57/57
🟢 Lines 99.61% 258/259

Test suite run success

285 tests passing in 95 suites.

Report generated by 🧪jest coverage report action from e740208

Show full coverage report
St File % Stmts % Branch % Funcs % Lines Uncovered Line #s
🟢 All files 99.62 99.11 100 99.61
🟢  src 100 100 100 100
🟢   config.ts 100 100 100 100
🟢   env.ts 100 100 100 100
🟢   handler.ts 100 100 100 100
🟢   index.ts 100 100 100 100
🟢  src/handlers 98.7 96.15 100 98.66
🟢   ...ApiRequest.ts 96.42 87.5 100 96.15 79
🟢   ...StatusPage.ts 100 100 100 100
🟢  src/utils 100 100 100 100
🟢   ...ionHeaders.ts 100 100 100 100
🟢   ...Monitoring.ts 100 100 100 100
🟢   cookie.ts 100 100 100 100
🟢   ...orResponse.ts 100 100 100 100
🟢   ...WithMaxAge.ts 100 100 100 100
🟢   ...AgeIfLower.ts 100 100 100 100
🟢   proxyEndpoint.ts 100 100 100 100
🟢   ...tpResponse.ts 100 100 100 100
🟢   routing.ts 100 100 100 100

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates development tooling dependencies and adjusts the Vite build configuration for Vite 8 by replacing the deprecated esbuild.banner behavior with a custom Rollup/Vite plugin that prepends the license banner to output chunks.

Changes:

  • Removed unused dev tooling/config (Commitizen-related packages/config) and dropped the engines.node declaration.
  • Migrated banner injection from esbuild.banner to a custom generateBundle plugin for Vite 8 compatibility.
  • Updated multiple dev dependencies, including major upgrades (Vite 8, Vitest 4, TypeScript 6).

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.

File Description
vite.config.ts Adds a custom Vite/Rollup plugin to prepend the license banner during bundle generation (replacing esbuild.banner).
package.json Removes unused tooling/config and updates dev dependency versions (including major upgrades).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@erayaydin
erayaydin force-pushed the chore/update-dependencies-inter-2335 branch from ee6267a to e740208 Compare July 23, 2026 20:01
@erayaydin
erayaydin marked this pull request as ready for review July 23, 2026 20:03
TheUnderScorer
TheUnderScorer previously approved these changes Jul 24, 2026
Comment thread package.json
Comment thread package.json
@erayaydin
erayaydin requested a review from necipallef July 24, 2026 11:56
necipallef
necipallef previously approved these changes Jul 24, 2026
Base automatically changed from chore/eslint-flat-config-INTER-2334 to main July 29, 2026 14:22
@erayaydin
erayaydin dismissed stale reviews from necipallef and TheUnderScorer July 29, 2026 14:22

The base branch was changed.

@github-actions

Copy link
Copy Markdown
Contributor

⚠️ This PR doesn't contain any changesets. If there are user-facing changes, don't forget to run:

pnpm exec changeset

to create a changeset.

@erayaydin
erayaydin merged commit 0e70ad6 into main Jul 29, 2026
18 of 19 checks passed
@erayaydin
erayaydin deleted the chore/update-dependencies-inter-2335 branch July 29, 2026 14:28
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.

4 participants