groundskeeping 0.2.0__tar.gz → 0.2.2__tar.gz

{groundskeeping-0.2.0 → groundskeeping-0.2.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: groundskeeping
-Version: 0.2.0
+Version: 0.2.2
 Summary: Reusable Textual operator shell, workbench, configuration browser, and telemetry primitives.
 Author: UNSW CAVA platform tooling
 Requires-Python: >=3.12
@@ -75,6 +75,9 @@ OperatorApp(spec).run()
 
 Groundskeeping owns the frame around those pages. `DatabaseSetupPage`,
 `EmbeddingsSetupPage`, and the services behind them stay in the application.
+Use `workbench_labels` on `OperatorAppSpec` when the shared pane chrome needs different
+language, such as naming the upper-right pane **Checks** instead of **Rows**. Page navigation
+and view contracts still supply their own content titles.
 
 The default page surface is the workbench:
 
@@ -91,7 +94,7 @@ Read the screenshot from left to right:
 | Top tabs | The registered pages in the app | `PageRoute`, `PageRegistration` |
 | Left pane | Things the operator can move between inside the current page | `SectionNavigation` + `SectionItem`, or `CatalogueNavigation` + `CatalogueItem` |
 | Upper-right pane | The main content for the selected section or catalogue item | `TableView`, `TreeView`, `EmptyView`, `LoadingView` |
-| Lower-right pane | Extra context for the highlighted row or item | `KeyValueView`, `TextView` |
+| Lower-right pane | Extra context for the highlighted row or item | `KeyValueView`, `TextView`, `TableView` |
 | Buttons above the view | Commands for the current view only | `ViewAction`, routed to `action_selected` |
 
 Pages choose `SectionNavigation` for peer areas or `CatalogueNavigation` for hierarchical

{groundskeeping-0.2.0 → groundskeeping-0.2.2}/README.md

@@ -58,6 +58,9 @@ OperatorApp(spec).run()
 
 Groundskeeping owns the frame around those pages. `DatabaseSetupPage`,
 `EmbeddingsSetupPage`, and the services behind them stay in the application.
+Use `workbench_labels` on `OperatorAppSpec` when the shared pane chrome needs different
+language, such as naming the upper-right pane **Checks** instead of **Rows**. Page navigation
+and view contracts still supply their own content titles.
 
 The default page surface is the workbench:
 
@@ -74,7 +77,7 @@ Read the screenshot from left to right:
 | Top tabs | The registered pages in the app | `PageRoute`, `PageRegistration` |
 | Left pane | Things the operator can move between inside the current page | `SectionNavigation` + `SectionItem`, or `CatalogueNavigation` + `CatalogueItem` |
 | Upper-right pane | The main content for the selected section or catalogue item | `TableView`, `TreeView`, `EmptyView`, `LoadingView` |
-| Lower-right pane | Extra context for the highlighted row or item | `KeyValueView`, `TextView` |
+| Lower-right pane | Extra context for the highlighted row or item | `KeyValueView`, `TextView`, `TableView` |
 | Buttons above the view | Commands for the current view only | `ViewAction`, routed to `action_selected` |
 
 Pages choose `SectionNavigation` for peer areas or `CatalogueNavigation` for hierarchical

{groundskeeping-0.2.0 → groundskeeping-0.2.2}/docs/pages.md

@@ -23,6 +23,11 @@ Translate domain objects before they reach the workbench. The workbench understa
 `SectionItem`, `CatalogueItem`, `TableView`, `TreeView`, and friends; it does not need to know
 what a Groundworkers resource or `cava-nlp-shard` evaluation object is.
 
+Use `OperatorAppSpec.workbench_labels` to rename shared pane chrome such as the result and
+detail panel labels. Page-owned titles still live on navigation and view contracts: for
+example, `SectionNavigation.title` names the left pane for a specific page, and
+`TableView.title` names the current result content.
+
 ## Routing
 
 `PageRoute` is the navigation identity for one page: a `key`, an operator-facing `label`, and
@@ -49,6 +54,9 @@ startup rather than when an operator clicks a tab.
 Row events return to the active page. The workbench renders generic models; the page decides
 what a highlighted row means.
 
+Detail panes can render `TextView`, `KeyValueView`, or `TableView`. Use a detail `TableView`
+when the selected item has its own repeated data, such as available LLM models for a provider.
+
 If `landing_view` raises, the shell catches the exception and renders an `EmptyView`
 explaining that the page could not be rendered. A page that cannot build its landing content
 degrades to a message instead of taking down the app.

{groundskeeping-0.2.0 → groundskeeping-0.2.2}/src/groundskeeping/app.py

@@ -4,7 +4,7 @@ from __future__ import annotations
 
 from collections.abc import Mapping
 from dataclasses import dataclass, field
-from typing import ClassVar
+from typing import ClassVar, cast
 
 from rich.text import Text
 from textual.app import App, ComposeResult
@@ -52,6 +52,7 @@ from groundskeeping.contracts.views import (
     PageNavigation,
     SectionItem,
     SurfaceView,
+    WorkbenchLabels,
 )
 from groundskeeping.contracts.wizards import (
     WizardController,
@@ -76,6 +77,7 @@ class OperatorAppSpec:
     operation_policy: OperationPolicy = field(default_factory=AllowAllOperationPolicy)
     job_policy: JobPolicy = field(default_factory=SingleForegroundJobPolicy)
     result_presenter: ResultPresenter = field(default_factory=DefaultResultPresenter)
+    workbench_labels: WorkbenchLabels = field(default_factory=WorkbenchLabels)
     default_page: str | None = None
     metadata: Mapping[str, object] | None = None
 
@@ -157,7 +159,7 @@ class OperatorApp(App[None]):
         self.sub_title = spec.subtitle or ""
         self.jobs = JobManager(spec.job_policy)
         self._active_page = spec.default_page or self.registry[0].key
-        self._workbench = Workbench()
+        self._workbench = Workbench(spec.workbench_labels)
         self._surface = _WorkbenchSurface(self._workbench)
         # Textual's App already owns an internal `_context()` method used during
         # startup. Keep the page-facing context under a distinct name so the shell does
@@ -180,13 +182,11 @@ class OperatorApp(App[None]):
             with Vertical(id="workspace"):
                 yield Label("", id="workspace-title")
                 for route in self.registry:
-                    page = self._pages[route.key]
-                    if not isinstance(page, Widget):
-                        raise TypeError(f"Page {route.key!r} must be a Textual Widget.")
-                    page.add_class("operator-page")
-                    page.styles.height = 0
-                    page.styles.max_height = 0
-                    yield page
+                    page_widget = self._page_widget(route.key)
+                    page_widget.add_class("operator-page")
+                    page_widget.styles.height = 0
+                    page_widget.styles.max_height = 0
+                    yield page_widget
                 yield self._workbench
         yield Footer()
 
@@ -254,16 +254,13 @@ class OperatorApp(App[None]):
         next_page = self._pages[page_key]
         if previous is not None and previous is not next_page:
             previous.deactivate(self._page_context)
-            if isinstance(previous, Widget):
-                previous.remove_class("-active")
+            self._page_widget(self._active_page).remove_class("-active")
         self._active_page = page_key
-        page = next_page
-        if isinstance(page, Widget):
-            page.add_class("-active")
+        self._page_widget(page_key).add_class("-active")
         tabs = self.query_one("#page-tabs", Tabs)
         if tabs.active != page_key:
             tabs.active = page_key
-        page.activate(self._page_context)
+        next_page.activate(self._page_context)
         self._render_page(route)
 
     def _render_page(self, route: PageRoute) -> None:
@@ -286,6 +283,12 @@ class OperatorApp(App[None]):
     def _active_widget(self) -> OperatorPage:
         return self._pages[self._active_page]
 
+    def _page_widget(self, page_key: str) -> Widget:
+        page = self._pages[page_key]
+        if not isinstance(page, Widget):
+            raise TypeError(f"Page {page_key!r} must be a Textual Widget.")
+        return cast(Widget, page)
+
     def _select_navigation_item(self, item: NavigationItem) -> None:
         self._active_widget().navigation_selected(item, self._page_context)
 

{groundskeeping-0.2.0 → groundskeeping-0.2.2}/src/groundskeeping/contracts/__init__.py

@@ -69,6 +69,7 @@ from groundskeeping.contracts.views import (
     TreeView,
     ViewAction,
     ViewActionVariant,
+    WorkbenchLabels,
 )
 from groundskeeping.contracts.wizards import (
     Choice,

{groundskeeping-0.2.0 → groundskeeping-0.2.2}/src/groundskeeping/contracts/views.py

@@ -12,6 +12,21 @@ type ViewActionVariant = Literal["default", "primary", "success", "warning", "er
 MAX_VIEW_ACTIONS = 6
 
 
+@dataclass(frozen=True)
+class WorkbenchLabels:
+    """Operator-facing labels for reusable workbench chrome.
+
+    Page-owned content still supplies its own titles through navigation and view contracts.
+    These labels only cover structural defaults in the shared shell.
+    """
+
+    navigation_panel: str = "Sections"
+    result_panel: str = "Rows"
+    detail_panel: str = "Detail"
+    initial_result_summary: str = "Select a section to inspect it."
+    key_value_columns: tuple[str, str] = ("Field", "Value")
+
+
 class SemanticStatus(StrEnum):
     """Small health/status vocabulary understood by shared widgets."""
 
@@ -160,5 +175,5 @@ class KeyValueView:
     title: str = "Detail"
 
 
-type DetailView = TextView | KeyValueView
+type DetailView = TextView | KeyValueView | TableView
 type SurfaceView = TableView | TreeView | EmptyView | LoadingView

groundskeeping-0.2.2/src/groundskeeping/py.typed

@@ -0,0 +1 @@
+

{groundskeeping-0.2.0 → groundskeeping-0.2.2}/src/groundskeeping/widgets/wizard.py

@@ -14,10 +14,12 @@ from textual.widgets import (
     DataTable,
     Input,
     Label,
+    OptionList,
     Select,
     Static,
     TextArea,
 )
+from textual.widgets.option_list import Option
 
 from groundskeeping.contracts.actions import FieldKind, FieldSpec
 from groundskeeping.contracts.wizards import (
@@ -42,6 +44,7 @@ class WizardScreen(ModalScreen[WizardResult]):
         self._snapshot: WizardSnapshot | None = None
         self._field_by_widget_id: dict[str, FieldSpec] = {}
         self._choice_widget_id = "wizard-choice"
+        self._blank_choice_id = "__groundskeeping_blank_choice__"
 
     def compose(self) -> ComposeResult:
         with Vertical(id="wizard-frame"):
@@ -149,21 +152,45 @@ class WizardScreen(ModalScreen[WizardResult]):
             for warning in step.review.warnings:
                 await body.mount(Static(f"Warning: {warning}", classes="wizard-warning"))
 
-    def _field_widget(self, widget_id: str, field: FieldSpec, value: object) -> Input | TextArea | Checkbox | Select:
+    def _field_widget(
+        self, widget_id: str, field: FieldSpec, value: object
+    ) -> Input | TextArea | Checkbox | OptionList:
         disabled = field.disabled or field.read_only
         if field.kind is FieldKind.BOOLEAN:
             return Checkbox(value=bool(value if value is not None else field.default), id=widget_id, disabled=disabled)
         if field.kind is FieldKind.CHOICE:
-            options = tuple((choice.label, choice.value) for choice in field.choices)
             current = value if value is not None else field.default
-            return Select(
-                options,
-                prompt=field.placeholder or "Choose",
-                value=current if current is not None else Select.BLANK,
+            options = []
+            if not field.required:
+                options.append(
+                    Option(field.placeholder or "No selection", id=self._blank_choice_id)
+                )
+            options.extend(
+                Option(
+                    choice.label
+                    if choice.description is None
+                    else f"{choice.label} — {choice.description}",
+                    id=choice.value,
+                )
+                for choice in field.choices
+            )
+            if not options:
+                options.append(Option(field.placeholder or "No choices", disabled=True))
+            widget = OptionList(
+                *options,
                 id=widget_id,
-                allow_blank=not field.required,
                 disabled=disabled,
+                classes="wizard-choice-list",
             )
+            widget.styles.height = min(max(len(options), 2), 8)
+            choice_ids = tuple(option.id for option in options)
+            if current is None and not field.required:
+                widget.highlighted = 0
+            elif current in choice_ids:
+                widget.highlighted = choice_ids.index(current)
+            elif field.required and field.choices:
+                widget.highlighted = 0
+            return widget
         if field.kind is FieldKind.MULTILINE:
             widget = TextArea("" if value is None else str(value), id=widget_id)
             widget.read_only = disabled
@@ -238,6 +265,11 @@ def _widget_value(widget: object) -> object:
         return widget.text
     if isinstance(widget, Checkbox):
         return widget.value
+    if isinstance(widget, OptionList):
+        if widget.highlighted is None:
+            return None
+        selected = widget.get_option_at_index(widget.highlighted)
+        return None if selected.id == "__groundskeeping_blank_choice__" else selected.id
     if isinstance(widget, Select):
         selected = widget.value
         return None if selected is Select.BLANK else selected

{groundskeeping-0.2.0 → groundskeeping-0.2.2}/src/groundskeeping/widgets/workbench.py

@@ -29,6 +29,7 @@ from groundskeeping.contracts.views import (
     TreeNode,
     TreeView,
     ViewAction,
+    WorkbenchLabels,
 )
 from groundskeeping.theme import node_label, status_style
 from groundskeeping.widgets.primitives import EmptyState, LoadingState
@@ -37,8 +38,9 @@ from groundskeeping.widgets.primitives import EmptyState, LoadingState
 class Workbench(Widget):
     """Shared page surface with navigation left and rows/detail right."""
 
-    def __init__(self) -> None:
+    def __init__(self, labels: WorkbenchLabels | None = None) -> None:
         super().__init__(id="workbench")
+        self.labels = labels or WorkbenchLabels()
         self._loading_timer: Timer | None = None
         self._loading_frame = 0
         self._loading_view: LoadingView | None = None
@@ -54,7 +56,7 @@ class Workbench(Widget):
                     with Horizontal(id="result-header"):
                         yield Static("", id="result-status")
                         yield Static(
-                            "Select a section to inspect it.", id="result-summary"
+                            self.labels.initial_result_summary, id="result-summary"
                         )
                     with Horizontal(id="result-actions"):
                         for index in range(MAX_VIEW_ACTIONS):
@@ -68,9 +70,9 @@ class Workbench(Widget):
                     yield DataTable(id="context-table")
 
     def on_mount(self) -> None:
-        self.query_one("#catalogue-panel").border_title = "Sections"
-        self.query_one("#result-panel").border_title = "Rows"
-        self.query_one("#context-panel").border_title = "Detail"
+        self.query_one("#catalogue-panel").border_title = self.labels.navigation_panel
+        self.query_one("#result-panel").border_title = self.labels.result_panel
+        self.query_one("#context-panel").border_title = self.labels.detail_panel
         self.catalogue.show_root = False
         self.catalogue.root.expand()
         self.catalogue.styles.display = "none"
@@ -284,27 +286,47 @@ class Workbench(Widget):
         self.query_one("#result-loading", LoadingState).hide_loading()
 
     def show_detail(self, detail: DetailView) -> None:
+        panel = self.query_one("#context-panel")
+        panel.border_subtitle = ""
         if isinstance(detail, TextView):
             self.query_one("#context-table", DataTable).styles.display = "none"
             context = self.query_one("#context", TextArea)
             context.styles.display = "block"
             context.load_text(detail.body)
+            panel.border_title = detail.title
             return
         if isinstance(detail, KeyValueView):
+            panel.border_title = detail.title
             self.show_context_table(detail.rows)
             return
+        if isinstance(detail, TableView):
+            self.show_detail_table(detail)
+            return
         raise TypeError(f"Unsupported detail view: {type(detail).__name__}")
 
+    def show_detail_table(self, detail: TableView) -> None:
+        self.query_one("#context", TextArea).styles.display = "none"
+        table = self.query_one("#context-table", DataTable)
+        table.styles.display = "block"
+        table.clear(columns=True)
+        table.add_columns(*detail.columns)
+        for row in detail.rows:
+            table.add_row(*row.cells, key=row.key)
+        self.query_one("#context-panel").border_title = detail.title
+        self.query_one("#context-panel").border_subtitle = f"{len(detail.rows)} rows"
+
     def show_context_table(
         self,
         rows: Iterable[tuple[str, str]],
         *,
-        columns: tuple[str, str] = ("Field", "Value"),
+        columns: tuple[str, ...] | None = None,
     ) -> None:
         self.query_one("#context", TextArea).styles.display = "none"
         table = self.query_one("#context-table", DataTable)
         table.styles.display = "block"
         table.clear(columns=True)
-        table.add_columns(*columns)
+        table.add_columns(
+            *(self.labels.key_value_columns if columns is None else columns)
+        )
         for label, value in rows:
             table.add_row(label, value)

{groundskeeping-0.2.0 → groundskeeping-0.2.2}/tests/test_app_smoke.py

@@ -3,22 +3,28 @@ from __future__ import annotations
 import asyncio
 from dataclasses import replace
 
+import pytest
 from textual.widget import Widget
-from textual.widgets import Button
+from textual.widgets import Button, DataTable
 
 from groundskeeping.app import OperatorApp, OperatorAppSpec
 from groundskeeping.contracts import (
     MAX_VIEW_ACTIONS,
     EmptyView,
+    KeyValueView,
     NavigationItem,
     PageContext,
     PageRegistration,
     PageRoute,
     SectionNavigation,
     SurfaceView,
+    TableRow,
+    TableView,
+    TextView,
     ViewAction,
     WizardResult,
     WizardResultStatus,
+    WorkbenchLabels,
 )
 from groundskeeping.demo import build_demo_spec
 
@@ -60,6 +66,35 @@ class _CountingPage(Widget):
         return None
 
 
+class _NonWidgetPage:
+    def __init__(self, route: PageRoute) -> None:
+        self.route = route
+
+    def activate(self, context: PageContext) -> None:
+        return None
+
+    def deactivate(self, context: PageContext) -> None:
+        return None
+
+    def build_navigation(self, context: PageContext) -> SectionNavigation:
+        return SectionNavigation(items=())
+
+    def landing_view(self, context: PageContext) -> SurfaceView:
+        return EmptyView(title=self.route.label, message="render")
+
+    def navigation_selected(self, item: NavigationItem, context: PageContext) -> None:
+        return None
+
+    def action_selected(self, action_key: str, context: PageContext) -> None:
+        return None
+
+    def row_highlighted(self, row_key: str, context: PageContext) -> None:
+        return None
+
+    def row_selected(self, row_key: str, context: PageContext) -> None:
+        return None
+
+
 def test_demo_app_enters_textual_startup_context() -> None:
     async def run() -> None:
         app = OperatorApp(build_demo_spec())
@@ -91,6 +126,25 @@ def test_demo_app_enters_textual_startup_context() -> None:
     asyncio.run(run())
 
 
+def test_operator_app_rejects_non_widget_pages() -> None:
+    async def run() -> None:
+        route = PageRoute("setup", "Setup", "Setup page")
+        app = OperatorApp(
+            OperatorAppSpec(
+                app_id="non-widget-test",
+                title="Non Widget Test",
+                subtitle=None,
+                pages=(PageRegistration(route, lambda context: _NonWidgetPage(route)),),
+            )
+        )
+
+        with pytest.raises(TypeError, match="must be a Textual Widget"):
+            async with app.run_test():
+                pass
+
+    asyncio.run(run())
+
+
 def test_single_page_hides_tabs_without_removing_page_registry() -> None:
     async def run() -> None:
         demo = build_demo_spec()
@@ -109,6 +163,46 @@ def test_single_page_hides_tabs_without_removing_page_registry() -> None:
     asyncio.run(run())
 
 
+def test_app_spec_can_override_workbench_chrome_labels() -> None:
+    async def run() -> None:
+        demo = build_demo_spec()
+        spec = replace(
+            demo,
+            workbench_labels=WorkbenchLabels(
+                navigation_panel="Setup areas",
+                result_panel="Checks",
+                detail_panel="Inspector",
+                initial_result_summary="Choose a setup area.",
+                key_value_columns=("Setting", "Value"),
+            ),
+        )
+        app = OperatorApp(spec)
+
+        async with app.run_test() as pilot:
+            assert app.query_one("#result-panel").border_title == "Checks"
+            assert app.query_one("#context-panel").border_title == "Inspector"
+
+            app._workbench.show_navigation(SectionNavigation(items=()))
+            assert app.query_one("#catalogue-panel").border_title == "Sections"
+
+            app._workbench.show_context_table((("provider", "ollama"),))
+            await pilot.pause()
+
+            table = app.query_one("#context-table", DataTable)
+            assert tuple(str(column.label) for column in table.ordered_columns) == (
+                "Setting",
+                "Value",
+            )
+
+            app._workbench.show_context_table((), columns=())
+            await pilot.pause()
+
+            assert len(table.ordered_columns) == 0
+            await pilot.press("q")
+
+    asyncio.run(run())
+
+
 def test_wizard_result_refreshes_active_page_when_requested() -> None:
     async def run() -> None:
         active_route = PageRoute("setup", "Setup", "Setup page")
@@ -202,3 +296,58 @@ def test_workbench_clamps_extra_view_actions_without_crashing() -> None:
             await pilot.press("q")
 
     asyncio.run(run())
+
+
+def test_workbench_renders_table_view_in_detail_pane() -> None:
+    async def run() -> None:
+        app = OperatorApp(build_demo_spec())
+
+        async with app.run_test(size=(120, 40)) as pilot:
+            detail = TableView(
+                title="Model inventory",
+                columns=("Model", "Context", "Status"),
+                rows=tuple(
+                    TableRow(
+                        key=f"model-{index}",
+                        cells=(f"model-{index}", f"{index * 1000}", "available"),
+                    )
+                    for index in range(12)
+                ),
+            )
+
+            app._workbench.show_detail(detail)
+            await pilot.pause()
+
+            context = app.query_one("#context")
+            table = app.query_one("#context-table", DataTable)
+
+            assert context.styles.display == "none"
+            assert table.styles.display == "block"
+            assert table.row_count == 12
+            assert len(table.ordered_columns) == 3
+            assert app.query_one("#context-panel").border_title == "Model inventory"
+            assert app.query_one("#context-panel").border_subtitle == "12 rows"
+
+            app._workbench.show_detail(TextView("Model notes", "Use qwen locally."))
+            await pilot.pause()
+
+            assert context.styles.display == "block"
+            assert table.styles.display == "none"
+            assert app.query_one("#context-panel").border_title == "Model notes"
+            assert app.query_one("#context-panel").border_subtitle == ""
+
+            app._workbench.show_detail(
+                KeyValueView(
+                    rows=(("Provider", "Ollama"),),
+                    title="Model facts",
+                )
+            )
+            await pilot.pause()
+
+            assert context.styles.display == "none"
+            assert table.styles.display == "block"
+            assert app.query_one("#context-panel").border_title == "Model facts"
+            assert app.query_one("#context-panel").border_subtitle == ""
+            await pilot.press("q")
+
+    asyncio.run(run())

{groundskeeping-0.2.0 → groundskeeping-0.2.2}/tests/test_wizard_contracts.py

@@ -1,9 +1,10 @@
 from __future__ import annotations
 
 import asyncio
+from collections.abc import Mapping
 
 import pytest
-from textual.widgets import Button
+from textual.widgets import Button, OptionList
 
 from groundskeeping.app import OperatorApp
 from groundskeeping.configurator import (
@@ -14,19 +15,117 @@ from groundskeeping.configurator import (
 )
 from groundskeeping.contracts import (
     Choice,
+    ChoiceOption,
     ChoiceStep,
+    FieldKind,
     FieldSpec,
     FormStep,
     ReviewChange,
     ReviewStep,
     WizardDefinitionError,
+    WizardResult,
     WizardResultStatus,
     WizardReview,
     WizardSnapshot,
     WizardSpec,
+    WizardTransition,
     validate_wizard_steps,
 )
 from groundskeeping.demo import ConfigPage, _DemoConfigWizardController, build_demo_spec
+from groundskeeping.widgets.wizard import WizardScreen
+
+
+class _DynamicChoiceWizard:
+    spec = WizardSpec(
+        key="model-picker",
+        title="Pick a model",
+        purpose="Choose one model from inventory.",
+        apply_label="Apply",
+    )
+
+    def __init__(self) -> None:
+        self.submitted: dict[str, object] = {}
+        self.step = FormStep(
+            key="model",
+            title="Model inventory",
+            fields=(
+                FieldSpec(
+                    key="model",
+                    label="Model",
+                    kind=FieldKind.CHOICE,
+                    choices=tuple(
+                        ChoiceOption(
+                            value=f"model-{index:02d}",
+                            label=f"Model {index:02d}",
+                            description=f"{index * 1000} context",
+                        )
+                        for index in range(12)
+                    ),
+                ),
+            ),
+        )
+        self.review_step = ReviewStep("review", "Review", WizardReview())
+
+    def start(self) -> WizardSnapshot:
+        return WizardSnapshot(
+            spec=self.spec,
+            step=self.step,
+            step_index=0,
+            step_count=2,
+            values={"model": "model-04"},
+        )
+
+    def submit(self, values: Mapping[str, object]) -> WizardTransition:
+        self.submitted = dict(values)
+        return WizardTransition(
+            WizardSnapshot(
+                spec=self.spec,
+                step=self.review_step,
+                step_index=1,
+                step_count=2,
+                can_back=True,
+                can_next=False,
+                can_apply=True,
+            )
+        )
+
+    def back(self) -> WizardSnapshot:
+        return self.start()
+
+    def review(self) -> WizardTransition:
+        return self.submit(self.submitted)
+
+    def apply(self) -> WizardResult:
+        return WizardResult(WizardResultStatus.APPLIED, "Applied.")
+
+    def cancel(self) -> WizardResult:
+        return WizardResult(WizardResultStatus.CANCELLED, "Cancelled.")
+
+
+class _NoChoiceWizard(_DynamicChoiceWizard):
+    def __init__(self) -> None:
+        super().__init__()
+        self.step = FormStep(
+            key="model",
+            title="Model inventory",
+            fields=(
+                FieldSpec(
+                    key="model",
+                    label="Model",
+                    kind=FieldKind.CHOICE,
+                    choices=(),
+                ),
+            ),
+        )
+
+    def start(self) -> WizardSnapshot:
+        return WizardSnapshot(
+            spec=self.spec,
+            step=self.step,
+            step_index=0,
+            step_count=2,
+            values={"model": None},
+        )
 
 
 def test_wizard_definitions_require_unique_keys() -> None:
@@ -166,6 +265,53 @@ def test_demo_config_wizard_reports_stale_revision_conflict() -> None:
     assert result.refresh_pages == frozenset({"config"})
 
 
+def test_wizard_choice_field_renders_dynamic_options_inline() -> None:
+    async def run() -> None:
+        controller = _DynamicChoiceWizard()
+        app = OperatorApp(build_demo_spec())
+
+        async with app.run_test(size=(120, 40)) as pilot:
+            app.push_screen(WizardScreen(controller))
+            await pilot.pause()
+
+            picker = app.screen.query_one("#wizard-field-0", OptionList)
+            assert picker.option_count == 12
+            assert picker.region.height > 1
+            assert picker.highlighted == 4
+
+            picker.focus()
+            await pilot.press("down")
+            await pilot.pause()
+
+            assert picker.highlighted == 5
+
+            await pilot.click("#wizard-next")
+            await pilot.pause()
+
+            assert controller.submitted["model"] == "model-05"
+            await pilot.press("escape")
+            await pilot.press("q")
+
+    asyncio.run(run())
+
+
+def test_wizard_choice_field_handles_empty_required_choices() -> None:
+    async def run() -> None:
+        app = OperatorApp(build_demo_spec())
+
+        async with app.run_test(size=(120, 40)) as pilot:
+            app.push_screen(WizardScreen(_NoChoiceWizard()))
+            await pilot.pause()
+
+            picker = app.screen.query_one("#wizard-field-0", OptionList)
+            assert picker.option_count == 1
+            assert picker.highlighted == 0
+            await pilot.press("escape")
+            await pilot.press("q")
+
+    asyncio.run(run())
+
+
 def test_config_draft_and_apply_intent_are_safe_and_revision_aware() -> None:
     target = ConfigTarget(kind="database", key="metadata", title="metadata")
     draft = ConfigDraft(