fix(sync): remove non-existent ComponentManager dependency

[jordanpartridge]

Summary

Fixes issue #95 by removing the non-existent ComponentManager class dependency from SyncComponentsCommand.php. The sync-components command was failing with a fatal error because it attempted to inject a class that doesn't exist in the codebase.

Changes

• Removed the unused use App\Services\ComponentManager; import statement
• Removed the ComponentManager $componentManager parameter from the handle() method
• The command now only uses JsonComponentRegistrar and ServiceProviderDetector which are the actual required dependencies

Why This Fix

The SyncComponentsCommand was attempting to inject ComponentManager on line 19, but this class does not exist in the codebase. This caused the command to fail at runtime with a service container resolution error.

Related Issues

Closes #95

Testing

• Run php artisan sync-components to verify the command executes without errors
• Confirm component registry is properly synchronized
• No functional changes to the command output (only removed unused dependency)

Summary by CodeRabbit

• Refactor
  • Streamlined internal component synchronization command structure for improved code organization and maintainability.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Walkthrough

The SyncComponentsCommand class had a dependency on a non-existent ComponentManager class that was removed during a previous refactoring. This fix removes the unused ComponentManager parameter from the handle() method signature and deletes its import statement, resolving the fatal error.

Changes

Cohort / File(s)	Summary
Remove non-existent ComponentManager dependency app/Commands/System/SyncComponentsCommand.php	Removed ComponentManager $componentManager parameter from handle() method signature; removed corresponding import statement. Method now accepts only JsonComponentRegistrar $registrar and ServiceProviderDetector $detector.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

No complex logic involved—straightforward removal of an unused dependency injection that resolves a fatal error.

Poem

🐰 A dependency gone, no longer in sight,
ComponentManager vanished—the command's now right!
One line removed, one import less,
The sync command blooms from its broken mess. ✨

Pre-merge checks and finishing touches

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main change: removing the non-existent ComponentManager dependency from SyncComponentsCommand, which directly addresses the PR's primary objective.
Linked Issues check	✅ Passed	The PR successfully implements option 3 from issue #95 by refactoring SyncComponentsCommand to remove the ComponentManager dependency, allowing the command to execute without fatal errors.
Out of Scope Changes check	✅ Passed	All changes are scoped to resolving issue #95; only the ComponentManager dependency and import are removed with no other unrelated modifications.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix/issue-95-component-manager

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between d8ad335 and e6183ce.

📒 Files selected for processing (1)

• app/Commands/System/SyncComponentsCommand.php (0 hunks)

💤 Files with no reviewable changes (1)

• app/Commands/System/SyncComponentsCommand.php

---

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

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