Skip to content

Refactor: migrate all providers to BaseProvider ABC - #32

Merged
C0MPL3XDEV merged 7 commits into
mainfrom
refactor/providers-to-base
Jun 28, 2026
Merged

Refactor: migrate all providers to BaseProvider ABC#32
C0MPL3XDEV merged 7 commits into
mainfrom
refactor/providers-to-base

Conversation

@C0MPL3XDEV

Copy link
Copy Markdown
Owner

Summary

  • Migrated 8 providers to BaseProvider ABC: GitHubProvider, BitlyProvider, PhoneInfoProvider, IPLocationProvider, DomainInfoProvider, MailFinderProvider, UserReconProvider, GoDorkerProvider
  • Each provider implements execute(query) -> list[ProviderResult] with business logic separated from CLI I/O
  • Interactive CLI wrappers preserved as thin functions calling provider classes
  • Errors logged via logging instead of printed in provider logic
  • Added comprehensive unit tests for all migrated providers
  • Facebook and TempMail left unmigrated (deprecated API / polling pattern)

Test plan

  • All tests pass (python -m pytest tests/ -v)
  • Each provider tested: valid input, invalid input, empty query, network errors
  • No changes to cli.py imports — backward compatible

… class, add extensive test coverage, and improve error handling.
…ved modularity and extendibility of network lookups. Update `iplocation` and `infoga` CLI wrappers to utilize the new providers. Add corresponding unit tests for enhanced coverage and reliability.
…r core logic for improved readability and maintainability
… add unit tests for improved maintainability.
@C0MPL3XDEV
C0MPL3XDEV merged commit 97f973b into main Jun 28, 2026
3 checks passed
@C0MPL3XDEV
C0MPL3XDEV deleted the refactor/providers-to-base branch June 29, 2026 13:08
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.

1 participant