chore(testing): remove ParanoiaProfileStrategy, a rename leftover - #27
Conversation
ParanoidProfileStrategy is the live strategy with five references; the misspelled twin had none. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
Important Review skippedReview was skipped as selected files did not have any reviewable changes. 💤 Files selected but had no reviewable changes (1)
⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Pull request overview
This PR removes the unused ParanoiaProfileStrategy class, which the description indicates was a misspelled leftover from a prior rename (with ParanoidProfileStrategy being the intended live implementation).
Changes:
- Deleted
src/Strategy/Profile/ParanoiaProfileStrategy.php(dead/unused profile strategy implementation). - Confirmed (by repository-wide search) there are no remaining references to
ParanoiaProfileStrategyor the string selectorparanoia.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Summary
ParanoidProfileStrategyis the live strategy with five references; the misspelled twinParanoiaProfileStrategyhad none. A leftover from a rename that only ever half-landed.Commits added by this work
Changed in those commits: 1 file changed, 56 deletions(-)
Checks
No repo-defined automated checks exist in this package, so verification was run at workspace level
(this gap is tracked as epic
ep-package-test-standard):bin/semitexa ai:verify --files=...on every edited file — passbin/semitexa test:run— PHPUnit 6013 tests, 0 failures, 3 skippedBackwards compatibility
Public classes are removed from a published package. Every symbol was proven unreferenced twice before
deletion: once by a token index over the whole workspace (
.php .neon .twig .yaml .yml .json .md .sh .js .css .xmlplusbin/semitexa), and once by a rawgrep -rlwwith no extension filter. Neither found asingle reference in any of the 42 packages or the local modules. A
#[As*]/#[Satisfies*]attributesweep was applied first so discovery-registered classes were never treated as dead.
Removal was chosen deliberately over a deprecation cycle: the only consumers are this project's own
installs.
What to Review
paranoiaanywhere in consumer config.What to Verify
bin/semitexa test:run packages/semitexa-testing/tests— green.Epics:
ep-dead-code-sweep,ep-orm-delete-query-adoption. Full inventory, method and false-positive analysis:var/docs/dead-code-inventory.md.🤖 Generated with Claude Code