From 898cb345ec462f2b322a96b3cf5cbc830e4cc079 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=D0=90=D0=BB=D0=B5=D0=BA=D1=81=D0=B0=D0=BD=D0=B4=D1=80=20?=
 =?UTF-8?q?=D0=97=D0=B8=D0=BC=D0=B8=D0=BD?= <zimin.an@nstart.space>
Date: Thu, 14 May 2026 12:33:43 +0000
Subject: [PATCH] fix(ui): DictionaryEditorDialog disables Save when draft
 workflow needed

---
 .../schema/DictionaryEditorDialog.tsx         | 31 +++++++++++-
 .../service/DictionaryDefinitionService.java  | 21 ++++++--
 .../reference/CascadeCloseService.java        | 50 ++++++++++++++++++-
 3 files changed, 95 insertions(+), 7 deletions(-)

diff --git a/ordinis-admin-ui/src/components/schema/DictionaryEditorDialog.tsx b/ordinis-admin-ui/src/components/schema/DictionaryEditorDialog.tsx
index f58a0d3..79d09d5 100644
--- a/ordinis-admin-ui/src/components/schema/DictionaryEditorDialog.tsx
+++ b/ordinis-admin-ui/src/components/schema/DictionaryEditorDialog.tsx
@@ -138,6 +138,24 @@ export const DictionaryEditorDialog = ({ open, mode, onClose, onSuccess }: Props
     [isEdit, initial, changes, schemaVersion],
   )
 
+  // Approval-aware Save: when editing an approval-required dict, any schema
+  // change OR toggling approvalRequired off OR changing approvalMinRole has
+  // to route through the schema-draft workflow. The backend will reject with
+  // 422 approval_required, but it's much better UX to disable Save up front
+  // and tell the user to open a draft instead. Mirror the three backend
+  // gate clauses exactly so the UX matches the server contract.
+  const wasApprovalRequired =
+    Boolean((initial as { approvalRequired?: boolean } | null)?.approvalRequired)
+  const initialMinRole =
+    ((initial as { approvalMinRole?: string } | null)?.approvalMinRole ?? '') || null
+  const currentMinRole = approvalMinRole.trim() === '' ? null : approvalMinRole.trim()
+  const schemaChangedFromInitial = isEdit && changes.length > 0
+  const approvalToggledOff = isEdit && wasApprovalRequired && !approvalRequired
+  const minRoleChanged = isEdit && wasApprovalRequired && initialMinRole !== currentMinRole
+  const requiresDraftFlow =
+    isEdit && wasApprovalRequired
+      && (schemaChangedFromInitial || approvalToggledOff || minRoleChanged)
+
   const activeMutation = isEdit ? updateMut : createMut
   const serverError = serverErrorMessage(activeMutation.error)
 
@@ -412,6 +430,17 @@ export const DictionaryEditorDialog = ({ open, mode, onClose, onSuccess }: Props
 
         {serverError && <Alert variant="error">{serverError}</Alert>}
 
+        {requiresDraftFlow && (
+          <Alert variant="warning">
+            {t('schema.approvalRequired.editorBlock', {
+              defaultValue:
+                'Этот словарь требует ревью изменений. Закройте редактор и нажмите ' +
+                '«Создать черновик схемы» на странице справочника — там вы сможете ' +
+                'отправить эти изменения на согласование.',
+            })}
+          </Alert>
+        )}
+
         <FormActions align="end">
           <Button type="button" variant="ghost" onClick={onClose} disabled={activeMutation.isPending}>
             {t('form.cancel')}
@@ -420,7 +449,7 @@ export const DictionaryEditorDialog = ({ open, mode, onClose, onSuccess }: Props
             type="button"
             variant="primary"
             loading={activeMutation.isPending}
-            disabled={isEdit && breaking}
+            disabled={(isEdit && breaking) || requiresDraftFlow}
             onClick={handleSubmit}
           >
             {activeMutation.isPending ? t('form.saving') : t('form.save')}
diff --git a/ordinis-rest-api/src/main/java/cloud/nstart/terravault/ordinis/restapi/service/DictionaryDefinitionService.java b/ordinis-rest-api/src/main/java/cloud/nstart/terravault/ordinis/restapi/service/DictionaryDefinitionService.java
index 3c5e35d..2cc34ce 100644
--- a/ordinis-rest-api/src/main/java/cloud/nstart/terravault/ordinis/restapi/service/DictionaryDefinitionService.java
+++ b/ordinis-rest-api/src/main/java/cloud/nstart/terravault/ordinis/restapi/service/DictionaryDefinitionService.java
@@ -141,13 +141,24 @@ public class DictionaryDefinitionService {
     boolean schemaChanged = !java.util.Objects.equals(prevSchema, req.schemaJson());
     boolean tryingToDisableApproval =
         req.approvalRequired() != null && !req.approvalRequired();
-    if (existing.isApprovalRequired() && (schemaChanged || tryingToDisableApproval)) {
+    // Approval-min-role flip is a third backdoor: a maker with PUT permission
+    // could clear approvalMinRole (send "" → stored as null) and then approve
+    // their own restricted-dict drafts using a generic RECORD_REVIEWER role.
+    // Treat any change to approvalMinRole the same as a schema rewrite —
+    // route through the draft workflow (or eventually a dedicated admin
+    // endpoint that requires a higher role than `approvalMinRole` itself).
+    boolean tryingToChangeMinRole =
+        req.approvalMinRole() != null
+            && !java.util.Objects.equals(
+                req.approvalMinRole().isBlank() ? null : req.approvalMinRole(),
+                existing.getApprovalMinRole());
+    if (existing.isApprovalRequired()
+        && (schemaChanged || tryingToDisableApproval || tryingToChangeMinRole)) {
       throw OrdinisException.unprocessableEntity(
           "approval_required",
-          "Schema changes require the draft workflow on this dictionary. "
-              + "Use POST /api/v1/admin/dictionaries/" + name
-              + "/schema-drafts. Toggling approvalRequired off is also "
-              + "blocked here — needs an explicit admin endpoint.");
+          "Schema changes, approvalRequired toggle-off, and approvalMinRole "
+              + "edits all require the draft workflow on this dictionary. "
+              + "Use POST /api/v1/admin/dictionaries/" + name + "/schema-drafts.");
     }
 
     existing.setDisplayName(req.displayName());
diff --git a/ordinis-rest-api/src/main/java/cloud/nstart/terravault/ordinis/restapi/service/reference/CascadeCloseService.java b/ordinis-rest-api/src/main/java/cloud/nstart/terravault/ordinis/restapi/service/reference/CascadeCloseService.java
index 0b129f1..b7a2149 100644
--- a/ordinis-rest-api/src/main/java/cloud/nstart/terravault/ordinis/restapi/service/reference/CascadeCloseService.java
+++ b/ordinis-rest-api/src/main/java/cloud/nstart/terravault/ordinis/restapi/service/reference/CascadeCloseService.java
@@ -64,6 +64,10 @@ public class CascadeCloseService {
   private final LineageIndexService lineage;
   private final RecordCloseSink closeSink;
   private final ScopeContext scopeContext;
+  /** Used to enforce the approvalRequired gate on the target + every cascaded
+   *  dependent dict. Optional: null in legacy test ctors that don't exercise
+   *  the gate. Production wiring always provides it. */
+  private final DictionaryDefinitionService defService;
   /** Phase 4 SLO. Optional — null в test ctor'е. */
   private final Optional<MeterRegistry> meterRegistry;
 
@@ -79,6 +83,7 @@ public class CascadeCloseService {
     this(lineage,
         new DefaultRecordCloseSink(recordRepo, defService, auditLogger, outbox),
         scopeContext,
+        defService,
         meterRegistry);
   }
 
@@ -87,17 +92,19 @@ public class CascadeCloseService {
       LineageIndexService lineage,
       RecordCloseSink closeSink,
       ScopeContext scopeContext) {
-    this(lineage, closeSink, scopeContext, Optional.empty());
+    this(lineage, closeSink, scopeContext, null, Optional.empty());
   }
 
   CascadeCloseService(
       LineageIndexService lineage,
       RecordCloseSink closeSink,
       ScopeContext scopeContext,
+      DictionaryDefinitionService defService,
       Optional<MeterRegistry> meterRegistry) {
     this.lineage = lineage;
     this.closeSink = closeSink;
     this.scopeContext = scopeContext;
+    this.defService = defService;
     this.meterRegistry = meterRegistry;
   }
 
@@ -150,6 +157,22 @@ public class CascadeCloseService {
     OffsetDateTime when = closeAt == null ? OffsetDateTime.now() : closeAt;
     CascadePlan plan = evaluatePlan(targetDict, targetKey, when);
 
+    // Approval gate: cascade-close goes through closeSink which writes
+    // straight to the repo + outbox + audit, completely skipping
+    // DictionaryRecordService.close → requireApprovalGate. So we have to
+    // enforce here, against BOTH the target dict and every cascaded
+    // dependent. If any of them is approval-required, refuse the whole
+    // cascade — it cannot atomically transition to drafts mid-cascade,
+    // and partial cascade would leave dangling references.
+    // (Future work: auto-create one parent draft + N dependent drafts in
+    // a single submit-for-review bundle. Out of scope for the gap fix.)
+    if (defService != null) {
+      assertNotApprovalRequired(targetDict);
+      for (var dep : plan.cascade()) {
+        assertNotApprovalRequired(dep.sourceDict());
+      }
+    }
+
     if (!plan.blockers().isEmpty()) {
       stopTimerAndCount(sample, targetDict, "blocked");
       throw OrdinisException.conflict(
@@ -202,6 +225,31 @@ public class CascadeCloseService {
     }
   }
 
+  /**
+   * Block cascade-close from bypassing the maker-checker draft workflow on
+   * approval-required dictionaries. Throws 422 {@code approval_required}
+   * pointing the caller at the drafts API.
+   */
+  private void assertNotApprovalRequired(String dict) {
+    DictionaryDefinition d;
+    try {
+      d = defService.findByName(dict);
+    } catch (RuntimeException e) {
+      // If we can't resolve the dict we definitely shouldn't proceed.
+      throw OrdinisException.badRequest(
+          "dictionary_not_found",
+          "Cascade close: cannot resolve dictionary " + dict);
+    }
+    if (d.isApprovalRequired()) {
+      throw OrdinisException.unprocessableEntity(
+          "approval_required",
+          "Cascade close cannot proceed: dictionary " + dict
+              + " requires the draft workflow. Open a record draft via "
+              + "POST /api/v1/dictionaries/" + dict + "/records/{businessKey}/drafts "
+              + "with operation=CLOSE for each affected record.");
+    }
+  }
+
   // === Metrics helpers ===
 
   private Timer.Sample startTimer() {