Skip to content

Bump frankenphp & dependencies#4128

Merged
ildyria merged 3 commits intomasterfrom
bump-frankenphp
Feb 28, 2026
Merged

Bump frankenphp & dependencies#4128
ildyria merged 3 commits intomasterfrom
bump-frankenphp

Conversation

@ildyria
Copy link
Member

@ildyria ildyria commented Feb 28, 2026 •
edited by coderabbitai bot
Loading

Summary by CodeRabbit

  • Chores
    • Updated Docker base image digest for the runtime image.
    • Reorganized two type-definition packages into devDependencies and removed an unused build tool from runtime dependencies.
  • Build
    • Removed production JS minification settings from the build configuration (console/debug stripping no longer applied).

@ildyria ildyria requested a review from a team as a code owner February 28, 2026 09:32
@coderabbitai
Copy link

coderabbitai bot commented Feb 28, 2026 •
edited
Loading

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between b4acba9 and b491428.

📒 Files selected for processing (1)
  • vite.embed.config.ts
💤 Files with no reviewable changes (1)
  • vite.embed.config.ts
📝 Walkthrough

Walkthrough

Updated the Dockerfile's FrankenPHP base image digest in Stage 3, moved two type-definition packages from dependencies to devDependencies and removed ts-loader from dependencies in package.json, and removed production minification (terser) options from the Vite embed build config.

Changes

Cohort / File(s) Summary
Docker base image update
Dockerfile		 Updated Stage 3 FrankenPHP image digest (sha256 updated); no other build-step or control-flow changes.
Dependency reorganization
package.json		 Moved @types/dropbox-chooser and @types/justified-layout to devDependencies; removed ts-loader from dependencies. No runtime API changes.
Vite build config change
vite.embed.config.ts		 Removed production minify: 'terser' and associated terserOptions (console/debugger drops) from build configuration; other build settings unchanged.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

🐰 A tiny digest hopped along the stream,
Types found snug where dev lights gleam,
Terser trimmed its silent snip,
Packages sorted, tidy ship,
I nibble bugs and dream of clean code cream.

🚥 Pre-merge checks | ✅ 1
✅ Passed checks (1 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

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.

@ildyria ildyria merged commit 6ee6e26 into master Feb 28, 2026
44 checks passed
@ildyria ildyria deleted the bump-frankenphp branch February 28, 2026 10:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@coderabbitai coderabbitai[bot] coderabbitai[bot] approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@ildyria