build!: require php-forge/debug ^0.3.

[terabytesoftw]

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)

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (10ed13e) to head (9be6da2).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files

@@             Coverage Diff             @@
##                main       #13   +/-   ##
===========================================
  Coverage     100.00%   100.00%           
  Complexity       310       310           
===========================================
  Files             22        22           
  Lines            813       813           
===========================================
  Hits             813       813           

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

[coderabbitai]

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: 75d8c7bd-15ef-4eda-8173-6a3ea3c882d7

📥 Commits

Reviewing files that changed from the base of the PR and between 628aa8e and 9be6da2.

📒 Files selected for processing (12)

• CHANGELOG.md
• composer.json
• src/Debug/VitePanel.php
• tests/Debug/ViteCollectorTest.php
• tests/Debug/VitePanelTest.php
• tests/Debug/fixtures/chunks.view.json
• tests/Debug/fixtures/development.view.json
• tests/Debug/fixtures/empty.view.json
• tests/Debug/fixtures/mixed.view.json
• tests/Debug/fixtures/production.view.json
• tests/Debug/fixtures/unavailable.view.json
• tests/Debug/fixtures/unknown.view.json

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

📜 Recent review details

⏰ Context from checks skipped due to timeout. (4)

• GitHub Check: phpunit / PHP 8.4-windows-2022
• GitHub Check: phpunit / PHP 8.3-windows-2022
• GitHub Check: mutation / PHP 8.5-ubuntu-latest
• GitHub Check: mutation / PHP 8.5-ubuntu-latest

🧰 Additional context used

🪛 PHPMD (2.15.0)

src/Debug/VitePanel.php

[warning] 48-96: The method present() has a Cyclomatic Complexity of 11. The configured cyclomatic complexity threshold is 10. (undefined)

(CyclomaticComplexity)

---

[warning] 48-96: The method present() has an NPath complexity of 432. The configured NPath complexity threshold is 200. (undefined)

(NPathComplexity)

tests/Debug/VitePanelTest.php

[error] Avoid using static access to class '\PHPForge\Vite\Tests\Provider\VitePanelProvider' in method 'testChunksAndMixedModesRemainProviderOwned'. (undefined)

(StaticAccess)

---

[error] 39-477: The class VitePanelTest has a coupling between objects value of 14. Consider to reduce the number of dependencies under 13. (undefined)

(CouplingBetweenObjects)

🔇 Additional comments (12)

composer.json (1)

34-34: LGTM!

CHANGELOG.md (1)

8-10: LGTM!

src/Debug/VitePanel.php (1)

84-84: LGTM!

tests/Debug/ViteCollectorTest.php (1)

9-9: LGTM!

Also applies to: 54-54, 95-95, 432-432

tests/Debug/VitePanelTest.php (1)

138-138: LGTM!

Also applies to: 155-155, 250-250

tests/Debug/fixtures/unavailable.view.json (1)

149-149: LGTM!

Also applies to: 162-162

tests/Debug/fixtures/unknown.view.json (1)

138-138: LGTM!

Also applies to: 151-151

tests/Debug/fixtures/chunks.view.json (1)

213-213: LGTM!

Also applies to: 226-226

tests/Debug/fixtures/development.view.json (1)

138-138: LGTM!

Also applies to: 151-151

tests/Debug/fixtures/empty.view.json (1)

31-31: LGTM!

tests/Debug/fixtures/mixed.view.json (1)

138-138: LGTM!

Also applies to: 258-258, 271-271

tests/Debug/fixtures/production.view.json (1)

138-138: LGTM!

Also applies to: 151-151

---

📝 Summary

Summary by CodeRabbit

• Breaking Changes

  • Upgraded the required php-forge/debug dependency from version ^0.2 to ^0.3.
  • Vite debug panel views no longer expose or determine an active state based on captured components.

• Documentation

  • Updated the upcoming release version to 0.5.0 and documented the dependency requirement.

Walkthrough

Changes

Vite panel update

Layer / File(s)	Summary
Remove panel active-state handling src/Debug/VitePanel.php, tests/Debug/ViteCollectorTest.php, tests/Debug/VitePanelTest.php	VitePanel no longer sets active state from the captured component count. Related active-state assertions were removed.
Update view fixtures tests/Debug/fixtures/*.view.json	Component and top-level view fixtures no longer contain active properties.
Update dependency and release metadata composer.json, CHANGELOG.md	The php-forge/debug requirement changes to ^0.3. The changelog version changes to 0.5.0 and records the breaking requirement.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to 9be6d

The change removes the Vite panel active-state output consistently and records the breaking dependency update; no material merge risk remains.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 42.86% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 7 functions across 3 files. (9 skipped: 9…	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly identifies the breaking dependency requirement change to php-forge/debug ^0.3.
Description check	✅ Passed	The description correctly identifies the pull request as a breaking change, which matches the dependency upgrade and related behavior changes.
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.

Full details: Docstring Coverage

Explanation

Docstring coverage is 42.86% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 7 functions across 3 files. (9 skipped: 9 unsupported.)

• Fix all pre-merge checks with AI

✨ Finishing Touches 💡 1

📝 Generate docstrings 💡

• Commit to this branch
• Create a new PR

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{A rabbit checks the panel’s bright display
The active flags now hop away
The fixtures match the calmer view
Debug grows to version point three too
Fresh changelog leaves a trail of hay}

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