simple-module-settings 0.0.40__tar.gz → 0.0.42__tar.gz

{simple_module_settings-0.0.40 → simple_module_settings-0.0.42}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: simple_module_settings
-Version: 0.0.40
+Version: 0.0.42
 Summary: Runtime settings UI — modules plug their own settings panels into a shared admin view
 Project-URL: Homepage, https://github.com/antosubash/simple_module_python
 Project-URL: Repository, https://github.com/antosubash/simple_module_python
@@ -21,9 +21,9 @@ Classifier: Topic :: Internet :: WWW/HTTP
 Classifier: Topic :: Software Development :: Libraries :: Application Frameworks
 Classifier: Typing :: Typed
 Requires-Python: >=3.12
-Requires-Dist: simple-module-core==0.0.40
-Requires-Dist: simple-module-db==0.0.40
-Requires-Dist: simple-module-hosting==0.0.40
+Requires-Dist: simple-module-core==0.0.42
+Requires-Dist: simple-module-db==0.0.42
+Requires-Dist: simple-module-hosting==0.0.42
 Requires-Dist: typer<0.28,>=0.12
 Description-Content-Type: text/markdown
 

{simple_module_settings-0.0.40 → simple_module_settings-0.0.42}/package.json

@@ -1,13 +1,13 @@
 {
   "name": "@simple-module-py/settings",
-  "version": "0.0.40",
+  "version": "0.0.42",
   "private": true,
   "description": "Frontend assets for the Settings module",
   "peerDependencies": {
+    "@inertiajs/react": "^3.9.1",
+    "@simple-module-py/ui": "*",
     "react": "^19.0.0",
-    "react-dom": "^19.0.0",
-    "@inertiajs/react": "^3.7.0",
-    "@simple-module-py/ui": "*"
+    "react-dom": "^19.3.1"
   },
   "devDependencies": {
     "@simple-module-py/tsconfig": "*"

{simple_module_settings-0.0.40 → simple_module_settings-0.0.42}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "simple_module_settings"
-version = "0.0.40"
+version = "0.0.42"
 description = "Runtime settings UI — modules plug their own settings panels into a shared admin view"
 readme = "README.md"
 license = "MIT"
@@ -21,9 +21,9 @@ classifiers = [
     "Typing :: Typed",
 ]
 dependencies = [
-    "simple_module_core==0.0.40",
-    "simple_module_db==0.0.40",
-    "simple_module_hosting==0.0.40",
+    "simple_module_core==0.0.42",
+    "simple_module_db==0.0.42",
+    "simple_module_hosting==0.0.42",
     "typer>=0.12,<0.28",
 ]
 

{simple_module_settings-0.0.40 → simple_module_settings-0.0.42}/settings/_managed_keys.py

@@ -22,7 +22,7 @@ from __future__ import annotations
 
 from collections.abc import Awaitable, Callable
 
-from settings.constants import ERR_MANAGED_KEY_DELETE
+from settings.constants import ERR_MANAGED_KEY_DELETE, ERR_MODULE_KEY_EDIT
 from settings.contracts.registry import SettingsRegistry, clear_route
 from settings.contracts.schemas import SettingScope
 from settings.exceptions import SettingsError
@@ -58,3 +58,11 @@ async def ensure_deletable(
     if orphaned:
         return
     raise ManagedKeyError(key, clear_route(definition, scope))
+
+
+class ModuleKeyError(SettingsError):
+    """A generic mutation would bypass the module editor's validation."""
+
+    def __init__(self, package: str, edit_via: str) -> None:
+        self.edit_via = edit_via
+        super().__init__(ERR_MODULE_KEY_EDIT.format(package=package, edit_via=edit_via))

{simple_module_settings-0.0.40 → simple_module_settings-0.0.42}/settings/constants.py

@@ -145,6 +145,7 @@ ERR_VALUE_MISMATCH: Final = "value does not parse as declared value_type"
 ERR_UNKNOWN_TENANT: Final = "Unknown tenant"
 ERR_NO_ACTIVE_TENANT: Final = "No active organisation for this request"
 ERR_MANAGED_KEY_DELETE: Final = "This setting is managed elsewhere; clear it at {clear_via}"
+ERR_MODULE_KEY_EDIT: Final = "Module settings must be edited at {edit_via} (package: {package})"
 ERR_NOT_TENANT_OVERRIDABLE: Final = "This setting cannot be changed per organisation"
 
 # ── HTTP ─────────────────────────────────────────────────────────────

{simple_module_settings-0.0.40 → simple_module_settings-0.0.42}/settings/deps.py

@@ -12,6 +12,7 @@ from typing import Annotated
 
 from fastapi import Depends, Request
 from simple_module_core.app_state import get_services
+from simple_module_core.module_settings import get_module_settings_registry
 from simple_module_db.deps import get_db
 from sqlalchemy.ext.asyncio import AsyncSession
 
@@ -36,6 +37,15 @@ async def get_setting_service(
     )
 
 
+async def get_generic_setting_service(
+    request: Request,
+    service: SettingService = Depends(get_setting_service),
+) -> SettingService:
+    """Raw store routes must send module configuration through its own editor."""
+    service.module_registry = get_module_settings_registry(request.app)
+    return service
+
+
 def get_settings_registry(request: Request) -> SettingsRegistry:
     """Return the app-wide settings registry populated during boot."""
     services = getattr(request.app.state, MODULE_PACKAGE)

{simple_module_settings-0.0.40 → simple_module_settings-0.0.42}/settings/endpoints/_create_form.py

@@ -7,6 +7,7 @@ from pydantic import ValidationError
 from simple_module_hosting.inertia_utils import redirect_back_with_errors, validation_errors_to_dict
 from starlette.responses import RedirectResponse
 
+from settings._managed_keys import ModuleKeyError
 from settings._unique_write import DuplicateSettingError
 from settings.constants import ERR_SETTING_EXISTS
 from settings.contracts.schemas import SettingCreate, SettingScope
@@ -31,4 +32,6 @@ async def create_from_form(
         await service.create(data)
     except DuplicateSettingError:
         return redirect_back_with_errors(request, {"key": ERR_SETTING_EXISTS})
+    except ModuleKeyError as exc:
+        return redirect_back_with_errors(request, {"key": str(exc)})
     return RedirectResponse(redirect_to, status_code=303)

{simple_module_settings-0.0.40 → simple_module_settings-0.0.42}/settings/endpoints/api.py

@@ -37,7 +37,7 @@ from settings.contracts.schemas import (
     SettingUpdate,
     SettingUpsert,
 )
-from settings.deps import get_setting_service
+from settings.deps import get_generic_setting_service
 from settings.service import SettingService
 from settings.tenant_scope import (
     is_known_tenant,
@@ -69,7 +69,7 @@ def _not_found() -> HTTPException:
 async def list_settings(
     scope: SettingScope | None = Query(default=None, alias=QP_SCOPE),
     scope_id: str = Query(default=SYSTEM_SCOPE_ID, alias=QP_SCOPE_ID),
-    service: SettingService = Depends(get_setting_service),
+    service: SettingService = Depends(get_generic_setting_service),
 ) -> list[SettingOut]:
     if scope is None:
         return await service.list_all()
@@ -84,7 +84,7 @@ async def resolve_setting(
     key: str,
     user_id: str | None = Query(default=None, alias=QP_USER_ID),
     tenant_id: str | None = Query(default=None, alias=QP_TENANT_ID),
-    service: SettingService = Depends(get_setting_service),
+    service: SettingService = Depends(get_generic_setting_service),
 ) -> SettingOut:
     result = await service.resolve(key, user_id=user_id, tenant_id=tenant_id)
     if result is None:
@@ -97,7 +97,7 @@ async def resolve_setting(
 
 @router.get(API_SYSTEM_PATH, response_model=SettingOut, dependencies=_VIEW)
 async def get_system_setting(
-    key: str, service: SettingService = Depends(get_setting_service)
+    key: str, service: SettingService = Depends(get_generic_setting_service)
 ) -> SettingOut:
     result = await service.get_scoped(SettingScope.SYSTEM, SYSTEM_SCOPE_ID, key)
     if result is None:
@@ -109,14 +109,14 @@ async def get_system_setting(
 async def upsert_system_setting(
     key: str,
     data: SettingUpsert,
-    service: SettingService = Depends(get_setting_service),
+    service: SettingService = Depends(get_generic_setting_service),
 ) -> SettingOut:
     return await service.upsert_scoped(SettingScope.SYSTEM, SYSTEM_SCOPE_ID, key, data)
 
 
 @router.delete(API_SYSTEM_PATH, status_code=STATUS_NO_CONTENT, dependencies=_DELETE)
 async def delete_system_setting(
-    key: str, service: SettingService = Depends(get_setting_service)
+    key: str, service: SettingService = Depends(get_generic_setting_service)
 ) -> None:
     if not await service.delete_scoped(SettingScope.SYSTEM, SYSTEM_SCOPE_ID, key):
         raise _not_found()
@@ -133,7 +133,7 @@ async def get_tenant_setting(
     scope_id: str,
     key: str,
     request: Request,
-    service: SettingService = Depends(get_setting_service),
+    service: SettingService = Depends(get_generic_setting_service),
 ) -> SettingOut:
     await require_known_tenant(request, scope_id)
     result = await service.get_scoped(SettingScope.TENANT, scope_id, key)
@@ -148,7 +148,7 @@ async def upsert_tenant_setting(
     key: str,
     data: SettingUpsert,
     request: Request,
-    service: SettingService = Depends(get_setting_service),
+    service: SettingService = Depends(get_generic_setting_service),
 ) -> SettingOut:
     await require_known_tenant(request, scope_id)
     await run_check(request, scope_id, key, data.value)
@@ -159,7 +159,7 @@ async def upsert_tenant_setting(
 async def delete_tenant_setting(
     scope_id: str,
     key: str,
-    service: SettingService = Depends(get_setting_service),
+    service: SettingService = Depends(get_generic_setting_service),
 ) -> None:
     if not await service.delete_scoped(SettingScope.TENANT, scope_id, key):
         raise _not_found()
@@ -169,7 +169,7 @@ async def delete_tenant_setting(
 async def get_user_setting(
     scope_id: str,
     key: str,
-    service: SettingService = Depends(get_setting_service),
+    service: SettingService = Depends(get_generic_setting_service),
 ) -> SettingOut:
     result = await service.get_scoped(SettingScope.USER, scope_id, key)
     if result is None:
@@ -182,7 +182,7 @@ async def upsert_user_setting(
     scope_id: str,
     key: str,
     data: SettingUpsert,
-    service: SettingService = Depends(get_setting_service),
+    service: SettingService = Depends(get_generic_setting_service),
 ) -> SettingOut:
     return await service.upsert_scoped(SettingScope.USER, scope_id, key, data)
 
@@ -191,7 +191,7 @@ async def upsert_user_setting(
 async def delete_user_setting(
     scope_id: str,
     key: str,
-    service: SettingService = Depends(get_setting_service),
+    service: SettingService = Depends(get_generic_setting_service),
 ) -> None:
     if not await service.delete_scoped(SettingScope.USER, scope_id, key):
         raise _not_found()
@@ -204,7 +204,7 @@ async def delete_user_setting(
 async def create_setting(
     data: SettingCreate,
     request: Request,
-    service: SettingService = Depends(get_setting_service),
+    service: SettingService = Depends(get_generic_setting_service),
 ) -> SettingOut:
     if data.scope is SettingScope.TENANT:
         # A body field, not a path segment: an unknown tenant is invalid input.
@@ -219,7 +219,7 @@ async def create_setting(
 
 @router.get(API_BY_ID_PATH, response_model=SettingOut, dependencies=_VIEW)
 async def get_setting(
-    setting_id: int, service: SettingService = Depends(get_setting_service)
+    setting_id: int, service: SettingService = Depends(get_generic_setting_service)
 ) -> SettingOut:
     result = await service.get_by_id(setting_id)
     if result is None:
@@ -232,7 +232,7 @@ async def update_setting(
     setting_id: int,
     data: SettingUpdate,
     request: Request,
-    service: SettingService = Depends(get_setting_service),
+    service: SettingService = Depends(get_generic_setting_service),
 ) -> SettingOut:
     current = await service.get_by_id(setting_id)
     if current is not None and current.scope is SettingScope.TENANT and data.value is not None:
@@ -245,7 +245,7 @@ async def update_setting(
 
 @router.delete(API_BY_ID_PATH, status_code=STATUS_NO_CONTENT, dependencies=_DELETE)
 async def delete_setting(
-    setting_id: int, service: SettingService = Depends(get_setting_service)
+    setting_id: int, service: SettingService = Depends(get_generic_setting_service)
 ) -> None:
     if not await service.delete(setting_id):
         raise _not_found()

{simple_module_settings-0.0.40 → simple_module_settings-0.0.42}/settings/endpoints/views.py

@@ -22,7 +22,7 @@ from simple_module_inertia import InertiaResponse
 from starlette.responses import RedirectResponse
 
 from settings import browse_query, known_keys
-from settings._managed_keys import ManagedKeyError
+from settings._managed_keys import ManagedKeyError, ModuleKeyError
 from settings._module_settings import (
     _package_of,
     collect_module_settings,
@@ -53,7 +53,7 @@ from settings.constants import (
     VIEW_STORE_PATH,
 )
 from settings.contracts.schemas import SettingUpdate
-from settings.deps import get_setting_service
+from settings.deps import get_generic_setting_service
 from settings.endpoints._create_form import create_from_form
 from settings.service import SettingService
 from settings.tenant_scope import tenant_update_error
@@ -81,7 +81,7 @@ router = APIRouter(dependencies=[Depends(RequiresPermission(PERM_VIEW))])
 @router.get(VIEW_STORE_PATH, response_model=None)
 async def browse(
     inertia: InertiaDep,
-    service: SettingService = Depends(get_setting_service),
+    service: SettingService = Depends(get_generic_setting_service),
     scope: str = SCOPE_ALL,
     q: str = "",
     page: browse_query.PageParam = 1,
@@ -139,7 +139,7 @@ async def create_view(request: Request, inertia: InertiaDep) -> InertiaResponse:
 async def edit_view(
     setting_id: int,
     inertia: InertiaDep,
-    service: SettingService = Depends(get_setting_service),
+    service: SettingService = Depends(get_generic_setting_service),
 ) -> InertiaResponse:
     item = await service.get_by_id(setting_id)
     if item is None:
@@ -159,7 +159,7 @@ async def edit_view(
 )
 async def create_action(
     request: Request,
-    service: SettingService = Depends(get_setting_service),
+    service: SettingService = Depends(get_generic_setting_service),
 ) -> RedirectResponse:
     return await create_from_form(request, service, _REDIRECT_SETTINGS)
 
@@ -172,7 +172,7 @@ async def create_action(
 async def update_action(
     setting_id: int,
     request: Request,
-    service: SettingService = Depends(get_setting_service),
+    service: SettingService = Depends(get_generic_setting_service),
 ) -> RedirectResponse:
     body = await request.json()
     try:
@@ -181,7 +181,10 @@ async def update_action(
         return redirect_back_with_errors(request, validation_errors_to_dict(exc))
     if error := await tenant_update_error(request, await service.get_by_id(setting_id), data.value):
         return redirect_back_with_errors(request, {"value": error})
-    await service.update(setting_id, data)
+    try:
+        await service.update(setting_id, data)
+    except ModuleKeyError as exc:
+        return redirect_back_with_errors(request, {"value": str(exc)})
     return RedirectResponse(_REDIRECT_SETTINGS, status_code=303)
 
 
@@ -194,7 +197,7 @@ async def delete_action(
     setting_id: int,
     request: Request,
     t: TranslatorDep,
-    service: SettingService = Depends(get_setting_service),
+    service: SettingService = Depends(get_generic_setting_service),
 ) -> RedirectResponse:
     try:
         await service.delete(setting_id)
@@ -204,6 +207,8 @@ async def delete_action(
             "settings.browse.delete_managed_error", setting=exc.key, clear_via=exc.clear_via
         )
         return redirect_back_with_errors(request, {"delete": message})
+    except ModuleKeyError as exc:
+        return redirect_back_with_errors(request, {"delete": str(exc)})
     return RedirectResponse(_REDIRECT_SETTINGS, status_code=303)
 
 
@@ -211,7 +216,7 @@ async def delete_action(
 async def modules_view(
     request: Request,
     inertia: InertiaDep,
-    service: SettingService = Depends(get_setting_service),
+    service: SettingService = Depends(get_generic_setting_service),
 ) -> InertiaResponse:
     """Read-only view of every module's pydantic ``BaseSettings`` instance.
 

{simple_module_settings-0.0.40 → simple_module_settings-0.0.42}/settings/errors.py

@@ -5,7 +5,7 @@ from __future__ import annotations
 from fastapi import FastAPI, Request
 from fastapi.responses import JSONResponse, Response
 
-from settings._managed_keys import ManagedKeyError
+from settings._managed_keys import ManagedKeyError, ModuleKeyError
 from settings.constants import STATUS_UNPROCESSABLE
 
 
@@ -16,5 +16,13 @@ async def _managed_key(request: Request, exc: Exception) -> Response:
     )
 
 
+async def _module_key(request: Request, exc: Exception) -> Response:
+    assert isinstance(exc, ModuleKeyError)
+    return JSONResponse(
+        {"detail": str(exc), "edit_via": exc.edit_via}, status_code=STATUS_UNPROCESSABLE
+    )
+
+
 def install_exception_handlers(app: FastAPI) -> None:
     app.add_exception_handler(ManagedKeyError, _managed_key)
+    app.add_exception_handler(ModuleKeyError, _module_key)

simple_module_settings-0.0.42/settings/known_keys.py

@@ -0,0 +1,47 @@
+"""Autocomplete for declared generic keys; module settings use their own editor.
+
+Defaults are masked here because Browse renders them in its resolved-value panel.
+"""
+
+from __future__ import annotations
+
+from typing import Any
+
+from fastapi import FastAPI
+from fastapi.encoders import jsonable_encoder
+from simple_module_core.module_settings import get_module_settings_registry
+from simple_module_core.secret_fields import conceals_secret, mask
+
+
+def _from_definition(definition) -> dict[str, Any]:
+    """Mask declarations directly, as module fields are no longer suggested."""
+    value_type = str(definition.value_type)
+    default = jsonable_encoder(definition.default)
+    secret = conceals_secret(definition.key, default, value_type)
+    return {
+        "key": definition.key,
+        "type": value_type,
+        "env_var": "",
+        "env_readable": False,
+        "env_set": False,
+        "env_value": None,
+        "default": mask(default) if secret else default,
+        "requires_restart": False,
+        "is_secret": secret,
+        "choices": None,
+    }
+
+
+def build(app: FastAPI) -> list[dict[str, Any]]:
+    """Declared generic keys only; registered module packages use their editor."""
+    suggestions: dict[str, dict[str, Any]] = {}
+    modules = get_module_settings_registry(app)
+    registry = getattr(getattr(app.state, "settings", None), "registry", None)
+    if registry is not None:
+        for definition in registry.all_definitions:
+            package, sep, _ = definition.key.partition(".")
+            if sep and modules is not None and modules.get(package) is not None:
+                continue
+            suggestions[definition.key] = _from_definition(definition)
+
+    return [suggestions[key] for key in sorted(suggestions)]

{simple_module_settings-0.0.40 → simple_module_settings-0.0.42}/settings/pages/components/FieldSource.tsx

@@ -55,10 +55,10 @@ export function FieldSource({ field, onReset }: { field: FieldMeta; onReset: ()
   const showDefault = !isBlank(field.default) && String(field.default) !== String(field.value);
   return (
     // Wraps rather than clips: `SM_FILE_STORAGE_QUOTA_BYTES` has no space to
-    // break at, so a 210px column cut it mid-word with nothing to say so.
-    // `break-words` breaks only the tokens that do not fit.
+    // break at, and as an unbreakable token it would set the column's minimum
+    // width; `overflow-wrap:anywhere` lets it break inside the identifier.
     <span
-      className="block break-words font-mono text-[11.5px] leading-snug text-muted-foreground"
+      className="block [overflow-wrap:anywhere] font-mono text-[11.5px] leading-snug text-muted-foreground"
       title={
         field.env_readable
           ? t(keys.settings.modules.env_var_hint)

{simple_module_settings-0.0.40 → simple_module_settings-0.0.42}/settings/pages/components/ModuleFieldRow.tsx

@@ -16,10 +16,11 @@ interface Props {
 /**
  * One settings row: label, control, trailing meta.
  *
- * Three columns rather than a stacked label block, because the trailing column
- * is what makes the screen readable at a glance — every row answers "where is
- * this value from?" in the same place, so the exceptions stand out instead of
- * having to be hunted for.
+ * Three columns when the editor card has room, because the trailing column is
+ * what makes the screen readable at a glance — every row answers "where is
+ * this value from?" in the same place, so the exceptions stand out. The tracks
+ * follow the card's width (container query), not the viewport: two sidebars
+ * can leave a narrow card on a wide screen, and the row stacks instead.
  */
 export function ModuleFieldRow({
   field,
@@ -37,15 +38,14 @@ export function ModuleFieldRow({
   const showDescription = field.description && field.type !== 'bool';
 
   return (
-    // `minmax(170px, auto)`: a fixed 170px track cannot grow, so a long field
-    // name overflowed it and printed straight over the input beside it.
-    <div className="grid gap-3 sm:grid-cols-[minmax(170px,auto)_1fr] lg:grid-cols-[minmax(170px,auto)_1fr_210px] lg:items-center">
+    // Query the editor card: two sidebars can leave little space at a wide viewport.
+    <div className="grid grid-cols-1 gap-3 @md:grid-cols-[minmax(0,140px)_minmax(180px,1fr)] @xl:grid-cols-[minmax(0,140px)_minmax(180px,1fr)_minmax(0,160px)] @xl:items-center">
       <div className="min-w-0">
-        <label htmlFor={id} className="block break-words text-[13px] font-medium">
+        <label htmlFor={id} className="block [overflow-wrap:anywhere] text-[13px] font-medium">
           {field.name}
         </label>
         {showDescription && (
-          <p className="mt-0.5 text-[11.5px] leading-snug text-muted-foreground">
+          <p className="mt-0.5 [overflow-wrap:anywhere] text-[11.5px] leading-snug text-muted-foreground">
             {field.description}
           </p>
         )}
@@ -65,7 +65,7 @@ export function ModuleFieldRow({
         </fieldset>
       </div>
 
-      <div className="sm:col-span-2 lg:col-span-1 lg:justify-self-start">
+      <div className="min-w-0 [overflow-wrap:anywhere] @md:col-span-2 @xl:col-span-1 @xl:justify-self-start">
         <FieldSource field={field} onReset={onReset} />
       </div>
 
@@ -75,7 +75,7 @@ export function ModuleFieldRow({
       {error && (
         <p
           data-testid={`field-error-${field.name}`}
-          className="break-words text-xs text-destructive sm:col-span-1 sm:col-start-2 lg:col-span-2"
+          className="min-w-0 [overflow-wrap:anywhere] text-xs text-destructive @md:col-span-1 @md:col-start-2 @xl:col-span-2"
         >
           {error}
         </p>

{simple_module_settings-0.0.40 → simple_module_settings-0.0.42}/settings/pages/components/ModuleForm.tsx

@@ -46,6 +46,7 @@ export function ModuleForm({ module: m, checks = [] }: Props) {
   const [errors, setErrors] = useState<Record<string, string>>({});
   const [busy, setBusy] = useState(false);
   const [saved, setSaved] = useState(false);
+  const [savedOverrides, setSavedOverrides] = useState<Record<string, boolean>>({});
   const [formErrors, setFormErrors] = useState<string[]>([]);
 
   // Refresh saved values without discarding drafts in other fields (#637).
@@ -61,6 +62,7 @@ export function ModuleForm({ module: m, checks = [] }: Props) {
     }));
     setErrors({});
     setSaved(false);
+    setSavedOverrides({});
   }, [initial]);
 
   const modifiedFields = useMemo(() => {
@@ -113,6 +115,14 @@ export function ModuleForm({ module: m, checks = [] }: Props) {
       // Keeping the form mounted lets the confirmation remain visible instead
       // of being discarded by an immediate Inertia reload.
       setBuffer((prev) => ({ ...prev, baseline: { ...values } }));
+      setSavedOverrides((prev) => ({
+        ...prev,
+        ...Object.fromEntries(
+          m.fields
+            .filter((f) => f.name in changed)
+            .map((f) => [f.name, notEqual(changed[f.name], f.default)]),
+        ),
+      }));
       setSaved(true);
     }
     setBusy(false);
@@ -126,15 +136,32 @@ export function ModuleForm({ module: m, checks = [] }: Props) {
         params: { path: { package: m.package, field: name } },
       }),
     );
+    setSavedOverrides((prev) => ({ ...prev, [name]: false }));
     setBuffer((prev) => ({
       ...prev,
       values: { ...prev.values, [name]: prev.baseline[name] },
     }));
-    router.reload({ only: ['modules'] });
+    // Take the field's live value from the reload itself: when the page never
+    // reloaded after a save, the fresh props equal the old ones and the
+    // [initial] effect does not fire, leaving the reverted value on screen.
+    router.reload({
+      only: ['modules'],
+      onSuccess: (page) => {
+        const modules = (page.props as { modules?: ModuleView[] }).modules ?? [];
+        const fresh = modules
+          .find((other) => other.package === m.package)
+          ?.fields.find((f) => f.name === name);
+        if (!fresh) return;
+        setBuffer((prev) => ({
+          values: { ...prev.values, [name]: fresh.value },
+          baseline: { ...prev.baseline, [name]: fresh.value },
+        }));
+      },
+    });
   }
 
   return (
-    <div className="flex flex-1 flex-col gap-4">
+    <div className="flex min-w-0 flex-1 flex-col gap-4">
       {/* Outside the card: the heading names the pane, and the deck keeps the
           card for the fields alone. */}
       <header className="flex flex-col gap-3 sm:flex-row sm:items-start sm:justify-between">
@@ -178,7 +205,7 @@ export function ModuleForm({ module: m, checks = [] }: Props) {
         </div>
       )}
 
-      <Card className="flex flex-1 flex-col gap-4 border-border p-6">
+      <Card className="@container flex min-w-0 flex-1 flex-col gap-4 border-border p-6">
         {/* No inner scroller: the card grows and the page scrolls, which is
             one scrollbar for the screen instead of three nested ones. */}
         <div className="flex flex-col gap-4">
@@ -188,7 +215,7 @@ export function ModuleForm({ module: m, checks = [] }: Props) {
               {fields.map((f) => (
                 <ModuleFieldRow
                   key={f.name}
-                  field={f}
+                  field={{ ...f, db_override: savedOverrides[f.name] ?? f.db_override }}
                   package={m.package}
                   value={values[f.name]}
                   modified={modifiedFields.has(f.name)}