Skip to content

refactor: remove unused WPVDB\WPVDB bootstrap class#12

Merged
rbcorrales merged 1 commit into
mainfrom
refactor/remove-unused-wpvdb-class
May 27, 2026
Merged

refactor: remove unused WPVDB\WPVDB bootstrap class#12
rbcorrales merged 1 commit into
mainfrom
refactor/remove-unused-wpvdb-class

Conversation

@rbcorrales
Copy link
Copy Markdown
Member

@rbcorrales rbcorrales commented May 27, 2026

Summary

Removes includes/class-wpvdb.php, which defined the WPVDB\WPVDB class.

Background

It's dead code. The active bootstrap in wpvdb.php uses \WPVDB\Plugin::get_instance(), and Plugin already registers the hooks this class duplicated (rest_api_init, wpvdb_process_embedding, wpvdb_process_embedding_batch, wpvdb_run_queue_now). Verified before removal:

  • No references to \WPVDB\WPVDB anywhere in the codebase (excluding vendor).
  • The file is never required/included.
  • It can't autoload: PSR-4 maps WPVDB\ to includes/, so WPVDB\WPVDB would resolve to includes/WPVDB.php, not the WordPress-named includes/class-wpvdb.php.

So nothing loads or calls it; removing it has no runtime effect.

Testing

Deletion-only change. No behavior to exercise; CI lint/test cover the rest.

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Deletes the unused includes/class-wpvdb.php file containing the dead WPVDB\WPVDB bootstrap class. The active bootstrap path uses \WPVDB\Plugin::get_instance() which already registers the same hooks, and the removed file is never required or autoloaded (PSR-4 mapping would not resolve to this WordPress-style filename).

Changes:

  • Remove includes/class-wpvdb.php entirely.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@rbcorrales rbcorrales merged commit 4e107b2 into main May 27, 2026
1 check passed
@rbcorrales rbcorrales deleted the refactor/remove-unused-wpvdb-class branch May 27, 2026 23:09
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