Skip to content

fix(build): regenerate the packaged JavaScript with the locked Vite 8.3.3 dependency update so the asset drift check passes. - #80

Merged
terabytesoftw merged 1 commit into
mainfrom
fix/regenerate-packaged-js
Oct 15, 2026
Merged

terabytesoftw merged 1 commit into
mainfrom
fix/regenerate-packaged-js

Conversation

@terabytesoftw

Copy link
Copy Markdown
Contributor

Pull Request

  • Breaking change (fix or feature that would cause existing functionality to change)
  • Bugfix (non-breaking change that fixes an issue)
  • CI/build configuration
  • Documentation update
  • New feature (non-breaking change that adds functionality)
  • Refactoring (no functional changes)

…8.3.3` dependency update so the asset drift check passes.
@terabytesoftw terabytesoftw added the bug Something isn't working label Oct 15, 2026
@codecov

codecov Bot commented Oct 15, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (fc61b17) to head (903fdaa).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@             Coverage Diff             @@
##                main       #80   +/-   ##
===========================================
  Coverage     100.00%   100.00%           
  Complexity      2443      2443           
===========================================
  Files            174       174           
  Lines           8876      8876           
===========================================
  Hits            8876      8876           

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

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@terabytesoftw

Copy link
Copy Markdown
Contributor Author

@codex review

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 15, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-15T23:47:40.156127Z 903fdaa Manual request
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@coderabbitai

coderabbitai Bot commented Oct 15, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: 29c75ce7-1fd7-4ef2-b4c4-7902ee24d4d1
📥 Commits

Reviewing files that changed from the base of the PR and between fc61b17 and 903fdaa.

⛔ Files ignored due to path filters: 6
  • package-lock.json is excluded by !**/package-lock.json
  • resources/assets/dist/js/db.min.js is excluded by !**/dist/**, !**/*.min.js
  • resources/assets/dist/js/debug.min.js is excluded by !**/dist/**, !**/*.min.js
  • resources/assets/dist/js/phpinfo-search.min.js is excluded by !**/dist/**, !**/*.min.js
  • resources/assets/dist/js/toolbar.min.js is excluded by !**/dist/**, !**/*.min.js
  • resources/assets/dist/js/userswitch.min.js is excluded by !**/dist/**, !**/*.min.js
📒 Files selected for processing: 1
  • package.json

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

⏰ Context from checks skipped due to timeout: 14
  • GitHub Check: phpunit / PHP 8.3-windows-2022
  • GitHub Check: phpunit / PHP 8.3-ubuntu-latest
  • GitHub Check: phpunit / PHP 8.5-windows-2022
  • GitHub Check: phpunit / PHP 8.4-ubuntu-latest
  • GitHub Check: phpunit / PHP 8.5-ubuntu-latest
  • GitHub Check: mutation / PHP 8.5-ubuntu-latest
  • GitHub Check: phpunit / PHP 8.4-windows-2022
  • GitHub Check: Verify Vite build reproduces dist.
  • GitHub Check: Analyze (actions)
  • GitHub Check: mutation / PHP 8.5-ubuntu-latest
  • GitHub Check: phpunit / PHP 8.4-windows-2022
  • GitHub Check: phpunit / PHP 8.3-windows-2022
  • GitHub Check: phpunit / PHP 8.5-windows-2022
  • GitHub Check: Verify Vite build reproduces dist.
🔇 Additional comments: 1
package.json: 1

54-54: 🎯 Functional Correctness

Keep Vitest 5 for this test layout.

The configured Vitest suite contains 45 test files and no describe(...) declarations. It does not contain the nested describe/it structure required by the cited runner failure. The request to keep Vitest 4 is therefore unsupported for this repository.

📝 Summary

Summary by CodeRabbit

  • Chores
    • Updated versions of several development tools and test utilities.

Updated the declared versions of eight development dependencies in package.json. The Stryker packages and stylelint-config-standard-scss remain unchanged.

Changes: Development Dependency Updates

Layer / File(s) Summary
Update development dependency versions
package.json
Updated the declared versions of Playwright, Vitest and its V8 coverage package, ESLint, jsdom, Prettier, Stylelint, and Vite. Stryker packages and stylelint-config-standard-scss remain unchanged.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to 903fd

The dependency lock and packaged JavaScript outputs are updated, and the identified mutation-testing concern does not affect the current tests. No actionable merge-blocking risk is established; the normal asset CI check should validate the generated output.

Pre-merge checks: Passed 5
Check name Status Explanation
Title check Passed The title clearly identifies the build change, the locked Vite dependency update, and the asset drift check objective.
Description check Passed The description identifies the pull request as a CI/build configuration change, which matches the stated objectives.
Docstring Coverage Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
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.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

A rabbit checks the package list
Eight versions hop to something new
Stryker stays right where it is
Stylelint’s config stays steady too
I tuck the manifest in my burrow
Then nibble greens and bid adieu

Comment @coderabbitai help to get the list of available commands.

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. 🚀

Reviewed commit: 903fdaa73d

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@terabytesoftw
terabytesoftw merged commit 444fa22 into main Oct 15, 2026
43 checks passed
@terabytesoftw
terabytesoftw deleted the fix/regenerate-packaged-js branch October 15, 2026 23:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant