Skip to content

Conversation

@harsh20k
Copy link

@harsh20k harsh20k commented Dec 5, 2025

Summary

  • extract search plan entries/executor to reduce duplication and clarify permissions (addresses duplication/complex method smell)
  • add strategy pattern for Jira auth tokens and project/board imports (replaces conditionals with polymorphism)
  • factor shared search serializer fields into a base class; update refactoring_log.md (pull-up fields, push-down specializations)

Smells Addressed

  • Duplication in search orchestration (search* helpers) reduced via SearchPlanEntry + SearchTaskExecutor
  • Conditional logic in importer token handling and project/board selection replaced with strategies
  • Long/duplicated serializer field definitions consolidated into BaseSearchResultsSerializer
  • Documented all refactorings in refactoring_log.md

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