Add AccordionExpander to Modules summary section in GenerateTab

[Copilot]

The Modules section in GenerateTab rendered as a flat, non-interactive QLabel inside a plain QGroupBox. The existing AccordionExpander component was unused here despite the EPIC explicitly requiring "Modules summary with expansion."

Changes

src/ui/tabs/generate_tab.py

• Wraps _modules_lbl inside an AccordionExpander (starts expanded by default) within the Modules QGroupBox
• reset() restores the accordion to its expanded default state
• _section_groups["Modules"] remains a QGroupBox — no structural breakage to callers

self._modules_expander = AccordionExpander(
    "",
    subtitle="Expand to review selected optional modules",
    start_expanded=True,
)
self._modules_expander.body_layout.addWidget(self._modules_lbl)

tests/test_generate_tab.py

• Adds TestGenerateTabModulesAccordion (9 tests) covering: attribute presence, type assertion, expand/collapse toggle, body layout containment, text content after config update, QGroupBox wrapping, and reset behaviour

[github-actions]

📊 PR Summary

Changes Overview

• Files Changed: 0
• Python Files: 0
• Test Files: 0
• Documentation: 0
• Added: 0
• Deleted: 0

CI Checks

This PR will trigger the following checks:

Please ensure all checks pass before merging.

[github-actions]

📊 PR Summary

Changes Overview

• Files Changed: 2
• Python Files: 2
• Test Files: 1
• Documentation: 0
• Added: 0
• Deleted: 0

CI Checks

This PR will trigger the following checks:

• ✅ Linting (ruff, isort, black)
• ✅ Type Checking (mypy)
• ✅ Tests (pytest)

Please ensure all checks pass before merging.

[github-actions]

📊 PR Summary

Changes Overview

• Files Changed: 2
• Python Files: 2
• Test Files: 1
• Documentation: 0
• Added: 0
• Deleted: 0

CI Checks

This PR will trigger the following checks:

• ✅ Linting (ruff, isort, black)
• ✅ Type Checking (mypy)
• ✅ Tests (pytest)

Please ensure all checks pass before merging.

[Copilot]

Pull request overview

Adds an interactive expand/collapse UI to the Modules summary in GenerateTab by embedding the modules summary label inside the existing AccordionExpander, and adds tests to validate the new behavior.

Changes:

• Wrap Modules summary (_modules_lbl) in an AccordionExpander (default expanded) inside the Modules QGroupBox.
• Update reset() to restore the Modules accordion to its default expanded state.
• Add a new test suite covering expander presence, behavior, containment, and reset behavior.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File	Description
src/ui/tabs/generate_tab.py	Introduces _modules_expander and nests the modules summary label inside it; ensures reset restores expanded state.
tests/test_generate_tab.py	Adds tests validating the Modules accordion wiring, expand/collapse behavior, and reset behavior.

---

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

[Copilot]

AccordionExpander is constructed with an empty title (""). In the current AccordionExpander implementation the title QLabel is always shown, so an empty title can still reserve vertical space and render as a blank header line. Consider providing a meaningful title (e.g. "Optional modules") or updating the component usage so the title label is hidden when the title is empty, to avoid awkward layout/UX.

Suggested change

	"",
	"Optional modules",

[Copilot]

The Modules section is now created with a custom QGroupBox/QVBoxLayout block instead of reusing _make_section_group, which is used for the other sections. This introduces duplicated layout construction and different margin behavior for Modules vs the rest. Consider generalizing _make_section_group to accept a QWidget (not just QLabel) and optional margins/spacing, then reuse it here to keep section construction consistent and easier to maintain.