Skip to content

[TASK] Refactor question handling into domain-specific collections#196

Merged
froemken merged 2 commits intomainfrom
refactorQuestionHandling
Feb 9, 2026
Merged

[TASK] Refactor question handling into domain-specific collections#196
froemken merged 2 commits intomainfrom
refactorQuestionHandling

Conversation

@froemken
Copy link
Copy Markdown
Collaborator

@froemken froemken commented Feb 8, 2026

The previous implementation used a single global QuestionCollection for all commands, leading to an unstructured directory and unclear responsibilities.

This change:

  • Reorganizes question classes into sub-directories by domain.
  • Introduces specific QuestionCollection services for each command.
  • Updates Services.yaml to inject only the required collections into each command.
  • Improves autowiring and tagging by using domain-specific tags for question registration.

This ensures that commands only have access to the questions they actually need and improves the overall project structure.

The previous implementation used a single global QuestionCollection
for all commands, leading to an unstructured directory and unclear
responsibilities.

This change:
- Reorganizes question classes into sub-directories by domain.
- Introduces specific QuestionCollection services for each command.
- Updates Services.yaml to inject only the required collections into
  each command.
- Improves autowiring and tagging by using domain-specific tags
  for question registration.

This ensures that commands only have access to the questions they
actually need and improves the overall project structure.
@froemken
Copy link
Copy Markdown
Collaborator Author

froemken commented Feb 8, 2026

Also a preparation for PR #171

@froemken froemken merged commit 8339573 into main Feb 9, 2026
1 check passed
@froemken froemken deleted the refactorQuestionHandling branch February 9, 2026 19:43
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