Skip to content

chore: bump webpack-cli to v5 - #6700

Merged
ikeike9o9o merged 1 commit into
EC-CUBE:4.3from
dotani1111:chore/bump-webpack-cli-5
Apr 22, 2026
Merged

chore: bump webpack-cli to v5#6700
ikeike9o9o merged 1 commit into
EC-CUBE:4.3from
dotani1111:chore/bump-webpack-cli-5

Conversation

@dotani1111

@dotani1111 dotani1111 commented Apr 1, 2026

Copy link
Copy Markdown
Contributor

概要(Overview・Refs Issue)

DependabotのPRがコンフリクトが多発していたため、別PRとしてあげます。
#6213

方針(Policy)

実装に関する補足(Appendix)

テスト(Test)

相談(Discussion)

マイナーバージョン互換性保持のための制限事項チェックリスト

  • 既存機能の仕様変更はありません
  • フックポイントの呼び出しタイミングの変更はありません
  • フックポイントのパラメータの削除・データ型の変更はありません
  • twigファイルに渡しているパラメータの削除・データ型の変更はありません
  • Serviceクラスの公開関数の、引数の削除・データ型の変更はありません
  • 入出力ファイル(CSVなど)のフォーマット変更はありません

レビュワー確認項目

  • 動作確認
  • コードレビュー
  • E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
  • 互換性が保持されているか
  • セキュリティ上の問題がないか
    • 権限を超えた操作が可能にならないか
    • 不要なファイルアップロードがないか
    • 外部へ公開されるファイルや機能の追加ではないか
    • テンプレートでのエスケープ漏れがないか

Summary by CodeRabbit

リリースノート

  • Chores

    • Bootstrapライブラリがv5.3.8に更新されました
    • webpack-cliが^4.9.2から^5.1.4にアップグレードされました
  • Style

    • 複数のCSSコンポーネントのスタイル定義が最適化されました
    • フォーム要素、ボタングループ、カルーセルなどのスタイルが改善されました

@coderabbitai

coderabbitai Bot commented Apr 1, 2026

Copy link
Copy Markdown
📝 Walkthrough

Walkthrough

Bootstrapライブラリをv5.3.3からv5.3.8に更新し、著作権年を2011–2024から2011–2025に変更。複数のCSSファイルでセレクタ特異性の調整、CSSカスタムプロパティを使用した新しいテーマング機構の導入、フレックスボックス計算の最適化を実施。webpack-cliを4.9.2から5.1.4にアップグレード。

Changes

コホート / ファイル(s) 変更内容
ライセンスヘッダー更新
html/bundle/admin.bundle.js.LICENSE.txt, html/bundle/front.bundle.js.LICENSE.txt, html/bundle/install.bundle.js.LICENSE.txt
BootstrapバージョンをV5.3.3からv5.3.8に更新、著作権年を2011–2024から2011–2025に変更。
Bootstrap CSSアップデート
html/template/admin/assets/css/bootstrap.css, html/template/default/assets/css/style.css
CSSセレクタの特異性調整、負の余白計算式の再構成(calc(var(--bs-border-width) * -1)calc(-1 * var(--bs-border-width)))、フレックスボックスの末尾%削除、CSSカスタムプロパティによるカルーセルおよびボタンクローズのテーマング機構導入、プレースホルダーセレクタの更新。
ビルドツールアップグレード
package.json
webpack-cliを^4.9.2から^5.1.4にアップグレード。

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Suggested reviewers

  • nanasess
  • saori-kakiuchi

Poem

🐰 ウサギが駆け回り、スタイルを磨き上げ
Bootstrap v5.3.8へとアップグレード
CSSは優雅に、変数で輝く
テーマング機構で、明暗を制す
webpack-cliの力で、ビルドはより強く! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed PR タイトル「chore: bump webpack-cli to v5」は、変更セットの主要な内容(webpack-cli を v4.9.2 から v5.1.4 にバージョンアップ)を正確に反映しており、簡潔かつ明確です。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
html/template/admin/assets/css/bootstrap.css (1)

1436-1468: ⚠️ Potential issue | 🟠 Major

生成済み Bootstrap CSS が現行の stylelint ルールと衝突しています。

Line 1436 / Line 1456 / Line 1468 の :-moz-placeholder 系セレクタ、Line 4381 以降の RGBA(...)、Line 5722 以降の宣言前空行で、現行ルールに違反しています。Bootstrap の生成物をそのまま管理する運用なら、このファイルを lint 対象から外すか、生成元/ビルド側でリポジトリのルールに合わせないと、同種の更新のたびに止まります。

Also applies to: 4381-4449, 5722-5754

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@html/template/admin/assets/css/bootstrap.css` around lines 1436 - 1468, The
generated Bootstrap CSS is violating stylelint via deprecated/incorrect
pseudo-element syntax (e.g. selectors like .form-floating >
.form-control:not(:-moz-placeholder) and related rules), uppercase RGBA(...)
uses, and stray blank lines before declarations; either remove this generated
file from stylelint/CI lint targets, or change the generator/output to be
lint-compliant by: replacing single-colon pseudo-elements with standard
double-colon forms (use ::placeholder and vendor ::-moz-placeholder where
appropriate) for selectors referenced (e.g. .form-floating >
.form-control:not(:-moz-placeholder), .form-floating >
.form-control:-webkit-autofill ~ label), normalize function names to lowercase
(rgba(...)) for occurrences around the RGBA block, and eliminate empty lines
before declarations (the blocks flagged around lines 5722+); update the
generator or pre-commit step so future builds produce the corrected selectors
and formatting.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Outside diff comments:
In `@html/template/admin/assets/css/bootstrap.css`:
- Around line 1436-1468: The generated Bootstrap CSS is violating stylelint via
deprecated/incorrect pseudo-element syntax (e.g. selectors like .form-floating >
.form-control:not(:-moz-placeholder) and related rules), uppercase RGBA(...)
uses, and stray blank lines before declarations; either remove this generated
file from stylelint/CI lint targets, or change the generator/output to be
lint-compliant by: replacing single-colon pseudo-elements with standard
double-colon forms (use ::placeholder and vendor ::-moz-placeholder where
appropriate) for selectors referenced (e.g. .form-floating >
.form-control:not(:-moz-placeholder), .form-floating >
.form-control:-webkit-autofill ~ label), normalize function names to lowercase
(rgba(...)) for occurrences around the RGBA block, and eliminate empty lines
before declarations (the blocks flagged around lines 5722+); update the
generator or pre-commit step so future builds produce the corrected selectors
and formatting.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: b51528d9-ea18-473e-8ab9-6517a2fa1e35

📥 Commits

Reviewing files that changed from the base of the PR and between cdb9778 and 63b6c89.

⛔ Files ignored due to path filters (8)
  • html/bundle/admin.bundle.js.map is excluded by !**/*.map
  • html/bundle/front.bundle.js.map is excluded by !**/*.map
  • html/bundle/install.bundle.js.map is excluded by !**/*.map
  • html/template/admin/assets/css/bootstrap.css.map is excluded by !**/*.map
  • html/template/admin/assets/css/bootstrap.min.css.map is excluded by !**/*.map
  • html/template/default/assets/css/style.css.map is excluded by !**/*.map
  • html/template/default/assets/css/style.min.css.map is excluded by !**/*.map
  • package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing (11)
  • html/bundle/admin.bundle.js
  • html/bundle/admin.bundle.js.LICENSE.txt
  • html/bundle/front.bundle.js
  • html/bundle/front.bundle.js.LICENSE.txt
  • html/bundle/install.bundle.js
  • html/bundle/install.bundle.js.LICENSE.txt
  • html/template/admin/assets/css/bootstrap.css
  • html/template/admin/assets/css/bootstrap.min.css
  • html/template/default/assets/css/style.css
  • html/template/default/assets/css/style.min.css
  • package.json

@codecov

codecov Bot commented Apr 1, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 78.62%. Comparing base (3a697eb) to head (63b6c89).
⚠️ Report is 133 commits behind head on 4.3.

Additional details and impacted files
@@             Coverage Diff              @@
##                4.3    #6700      +/-   ##
============================================
- Coverage     78.65%   78.62%   -0.03%     
  Complexity     6824     6824              
============================================
  Files           476      476              
  Lines         27078    27078              
============================================
- Hits          21299    21291       -8     
- Misses         5779     5787       +8     
Flag Coverage Δ
Unit 78.62% <ø> (-0.03%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

@dotani1111 dotani1111 added this to the 4.4.0 milestone Apr 1, 2026
@ikeike9o9o
ikeike9o9o merged commit 0c27653 into EC-CUBE:4.3 Apr 22, 2026
100 checks passed
dotani1111 added a commit to dotani1111/ec-cube that referenced this pull request May 13, 2026
PR EC-CUBE#6735 (2026-04-22) 以降に 4.3 へ merge された 12 ファイルの変更を取り込む.

- EC-CUBE#6752 Dockerbuild 修正 (index.php)
- EC-CUBE#6705 product_class post size 修正 (twig)
- EC-CUBE#6700 webpack-cli ^5.1.4 / chart.js ^4.5.1 等 (package.json)
- 追加テスト, i18n, codeception router fix

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
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.

2 participants