From 6d3f116ff32cd042f016fcad4249252ee0bc3598 Mon Sep 17 00:00:00 2001
From: "github-actions[bot]"
 <41898282+github-actions[bot]@users.noreply.github.com>
Date: Thu, 10 Sep 2026 21:14:33 +0000
Subject: [PATCH] fix(F6.1): add explicit audited inspection cancellation form

---
 .github/workflows/f6-1-apply-cancel-ux.yml    | 31 --------
 scripts/f6-1-apply-cancel-ux.py               | 77 -------------------
 .../src/pages/InspectionVisitEditorF4Page.tsx | 41 +++++++---
 3 files changed, 30 insertions(+), 119 deletions(-)
 delete mode 100644 .github/workflows/f6-1-apply-cancel-ux.yml
 delete mode 100644 scripts/f6-1-apply-cancel-ux.py

diff --git a/.github/workflows/f6-1-apply-cancel-ux.yml b/.github/workflows/f6-1-apply-cancel-ux.yml
deleted file mode 100644
index 96d1802..0000000
--- a/.github/workflows/f6-1-apply-cancel-ux.yml
+++ /dev/null
@@ -1,31 +0,0 @@
-name: F6.1 apply cancellation UX
-
-on:
-  push:
-    branches:
-      - hotfix/f6-1-operator-cancel-ux
-    paths:
-      - .github/workflows/f6-1-apply-cancel-ux.yml
-
-permissions:
-  contents: write
-
-jobs:
-  apply:
-    runs-on: ubuntu-latest
-    steps:
-      - uses: actions/checkout@v4
-        with:
-          ref: hotfix/f6-1-operator-cancel-ux
-          fetch-depth: 0
-      - name: Apply targeted patch
-        run: python scripts/f6-1-apply-cancel-ux.py
-      - name: Commit cancellation UX
-        run: |
-          rm .github/workflows/f6-1-apply-cancel-ux.yml scripts/f6-1-apply-cancel-ux.py
-          git config user.name 'github-actions[bot]'
-          git config user.email '41898282+github-actions[bot]@users.noreply.github.com'
-          git add web-v2/src/pages/InspectionVisitEditorF4Page.tsx .github/workflows/f6-1-apply-cancel-ux.yml scripts/f6-1-apply-cancel-ux.py
-          git diff --cached --check
-          git commit -m 'fix(F6.1): add explicit audited inspection cancellation form'
-          git push origin HEAD:hotfix/f6-1-operator-cancel-ux
diff --git a/scripts/f6-1-apply-cancel-ux.py b/scripts/f6-1-apply-cancel-ux.py
deleted file mode 100644
index c0e6a46..0000000
--- a/scripts/f6-1-apply-cancel-ux.py
+++ /dev/null
@@ -1,77 +0,0 @@
-from pathlib import Path
-import re
-
-path = Path('web-v2/src/pages/InspectionVisitEditorF4Page.tsx')
-text = path.read_text()
-
-state_before = "  const [success, setSuccess] = useState('');\n"
-state_after = """  const [success, setSuccess] = useState('');
-  const [showCancelForm, setShowCancelForm] = useState(false);
-  const [cancelReason, setCancelReason] = useState('');
-"""
-if text.count(state_before) != 1:
-    raise SystemExit('Could not locate success state exactly once')
-text = text.replace(state_before, state_after)
-
-lifecycle_pattern = re.compile(
-    r"  const lifecycleAction = async \(action: 'PLAN' \| 'UNPLAN' \| 'CANCEL'\) => \{.*?\n  \};\n\n  if \(loading\)",
-    re.S,
-)
-lifecycle_replacement = """  const lifecycleAction = async (action: 'PLAN' | 'UNPLAN') => {
-    if (!id) return;
-    setBusy(true); setError(''); setSuccess('');
-    try {
-      let updated: InspectionVisit;
-      if (action === 'PLAN') {
-        updated = await planInspectionVisit(id);
-        setSuccess('Inspección planificada. Ya puede iniciarse desde la APK.');
-      } else {
-        updated = await unplanInspectionVisit(id);
-        setSuccess('La Inspección volvió a borrador para corregir la planificación.');
-      }
-      applyVisit(updated);
-    } catch (requestError) { setError(errorMessage(requestError)); }
-    finally { setBusy(false); }
-  };
-
-  const cancelCurrentVisit = async (event: FormEvent) => {
-    event.preventDefault();
-    if (!id) return;
-    const reason = cancelReason.trim();
-    if (reason.length < 10) {
-      setError('El motivo de cancelación debe tener al menos 10 caracteres.');
-      return;
-    }
-    if (reason.length > 500) {
-      setError('El motivo de cancelación no puede superar los 500 caracteres.');
-      return;
-    }
-    setBusy(true); setError(''); setSuccess('');
-    try {
-      const updated = await cancelInspectionVisit(id, reason);
-      applyVisit(updated);
-      setCancelReason('');
-      setShowCancelForm(false);
-      setSuccess('Inspección cancelada. El motivo quedó registrado y auditado.');
-    } catch (requestError) { setError(errorMessage(requestError)); }
-    finally { setBusy(false); }
-  };
-
-  if (loading)"""
-text, count = lifecycle_pattern.subn(lifecycle_replacement, text, count=1)
-if count != 1:
-    raise SystemExit(f'Lifecycle block replacements: {count}')
-
-actions_pattern = re.compile(
-    r"    \{visit && <article className=\"panel survey-status-actions\">.*?</article>\}\n",
-    re.S,
-)
-actions_replacement = """    {visit && <article className=\"panel survey-status-actions\"><div><strong>Flujo de la Inspección</strong><p>La oficina planifica. El inicio y cierre operativo se realizan desde la APK. Una Inspección no puede cerrarse hasta que todas sus Actas estén selladas.</p></div><div className=\"form-actions\">{canManage && visit.status === 'DRAFT' && <button type=\"button\" className=\"button primary\" disabled={busy} onClick={() => void lifecycleAction('PLAN')}>Confirmar planificación</button>}{canManage && visit.status === 'PLANNED' && <button type=\"button\" className=\"button secondary\" disabled={busy} onClick={() => void lifecycleAction('UNPLAN')}>Volver a borrador</button>}{canManage && ['DRAFT', 'PLANNED', 'IN_PROGRESS'].includes(visit.status) && <button type=\"button\" className=\"button danger-outline\" disabled={busy} onClick={() => { setError(''); setSuccess(''); setShowCancelForm(true); }}>Cancelar inspección</button>}</div></article>}
-
-    {visit && showCancelForm && ['DRAFT', 'PLANNED', 'IN_PROGRESS'].includes(visit.status) && <form className=\"panel form-panel\" onSubmit={cancelCurrentVisit}><div className=\"panel-heading\"><div><span className=\"eyebrow\">CANCELACIÓN</span><h2>Cancelar inspección</h2><p className=\"section-copy\">La cancelación es definitiva y queda auditada. No se permite si la Inspección ya tiene un Acta sellada.</p></div></div><label className=\"field\"><span>Motivo de cancelación</span><textarea value={cancelReason} onChange={(event) => setCancelReason(event.target.value)} minLength={10} maxLength={500} rows={4} placeholder=\"Explicá el motivo de la cancelación…\" required /><small className=\"muted\">{cancelReason.trim().length}/500 · mínimo 10 caracteres</small></label><div className=\"form-actions\"><button type=\"button\" className=\"button secondary\" disabled={busy} onClick={() => { setShowCancelForm(false); setCancelReason(''); }}>Volver</button><button type=\"submit\" className=\"button danger-outline\" disabled={busy || cancelReason.trim().length < 10 || cancelReason.trim().length > 500}>{busy ? 'Cancelando…' : 'Confirmar cancelación'}</button></div></form>}
-"""
-text, count = actions_pattern.subn(actions_replacement, text, count=1)
-if count != 1:
-    raise SystemExit(f'Actions block replacements: {count}')
-
-path.write_text(text)
diff --git a/web-v2/src/pages/InspectionVisitEditorF4Page.tsx b/web-v2/src/pages/InspectionVisitEditorF4Page.tsx
index cf98b67..499dc51 100644
--- a/web-v2/src/pages/InspectionVisitEditorF4Page.tsx
+++ b/web-v2/src/pages/InspectionVisitEditorF4Page.tsx
@@ -109,6 +109,8 @@ export function InspectionVisitEditorF4Page() {
   const [busy, setBusy] = useState(false);
   const [error, setError] = useState('');
   const [success, setSuccess] = useState('');
+  const [showCancelForm, setShowCancelForm] = useState(false);
+  const [cancelReason, setCancelReason] = useState('');
 
   const applyVisit = (value: InspectionVisit, syncForm = true) => {
     setVisit(value);
@@ -287,7 +289,7 @@ export function InspectionVisitEditorF4Page() {
     finally { setBusy(false); }
   };
 
-  const lifecycleAction = async (action: 'PLAN' | 'UNPLAN' | 'CANCEL') => {
+  const lifecycleAction = async (action: 'PLAN' | 'UNPLAN') => {
     if (!id) return;
     setBusy(true); setError(''); setSuccess('');
     try {
@@ -295,23 +297,38 @@ export function InspectionVisitEditorF4Page() {
       if (action === 'PLAN') {
         updated = await planInspectionVisit(id);
         setSuccess('Inspección planificada. Ya puede iniciarse desde la APK.');
-      } else if (action === 'UNPLAN') {
+      } else {
         updated = await unplanInspectionVisit(id);
         setSuccess('La Inspección volvió a borrador para corregir la planificación.');
-      } else {
-        const reason = window.prompt('Indicá el motivo de cancelación (mínimo 10 caracteres):')?.trim() ?? '';
-        if (reason.length < 10) {
-          if (reason) setError('El motivo debe tener al menos 10 caracteres.');
-          return;
-        }
-        updated = await cancelInspectionVisit(id, reason);
-        setSuccess('Inspección cancelada con motivo auditado.');
       }
       applyVisit(updated);
     } catch (requestError) { setError(errorMessage(requestError)); }
     finally { setBusy(false); }
   };
 
+  const cancelCurrentVisit = async (event: FormEvent) => {
+    event.preventDefault();
+    if (!id) return;
+    const reason = cancelReason.trim();
+    if (reason.length < 10) {
+      setError('El motivo de cancelación debe tener al menos 10 caracteres.');
+      return;
+    }
+    if (reason.length > 500) {
+      setError('El motivo de cancelación no puede superar los 500 caracteres.');
+      return;
+    }
+    setBusy(true); setError(''); setSuccess('');
+    try {
+      const updated = await cancelInspectionVisit(id, reason);
+      applyVisit(updated);
+      setCancelReason('');
+      setShowCancelForm(false);
+      setSuccess('Inspección cancelada. El motivo quedó registrado y auditado.');
+    } catch (requestError) { setError(errorMessage(requestError)); }
+    finally { setBusy(false); }
+  };
+
   if (loading) return <LoadingBlock label="Cargando inspección…" />;
 
   return <section className="survey-editor inspection-editor">
@@ -353,7 +370,9 @@ export function InspectionVisitEditorF4Page() {
 
     {visit && <InspectionActsPanel visit={visit} />}
 
-    {visit && <article className="panel survey-status-actions"><div><strong>Flujo de la Inspección</strong><p>La oficina planifica. El inicio y cierre operativo se realizan desde la APK. Una Inspección no puede cerrarse hasta que todas sus Actas estén selladas.</p></div><div className="form-actions">{canManage && visit.status === 'DRAFT' && <button type="button" className="button primary" disabled={busy} onClick={() => void lifecycleAction('PLAN')}>Confirmar planificación</button>}{canManage && visit.status === 'PLANNED' && <button type="button" className="button secondary" disabled={busy} onClick={() => void lifecycleAction('UNPLAN')}>Volver a borrador</button>}{canManage && ['DRAFT', 'PLANNED'].includes(visit.status) && <button type="button" className="button danger-outline" disabled={busy} onClick={() => void lifecycleAction('CANCEL')}>Cancelar</button>}</div></article>}
+    {visit && <article className="panel survey-status-actions"><div><strong>Flujo de la Inspección</strong><p>La oficina planifica. El inicio y cierre operativo se realizan desde la APK. Una Inspección no puede cerrarse hasta que todas sus Actas estén selladas.</p></div><div className="form-actions">{canManage && visit.status === 'DRAFT' && <button type="button" className="button primary" disabled={busy} onClick={() => void lifecycleAction('PLAN')}>Confirmar planificación</button>}{canManage && visit.status === 'PLANNED' && <button type="button" className="button secondary" disabled={busy} onClick={() => void lifecycleAction('UNPLAN')}>Volver a borrador</button>}{canManage && ['DRAFT', 'PLANNED', 'IN_PROGRESS'].includes(visit.status) && <button type="button" className="button danger-outline" disabled={busy} onClick={() => { setError(''); setSuccess(''); setShowCancelForm(true); }}>Cancelar inspección</button>}</div></article>}
+
+    {visit && showCancelForm && ['DRAFT', 'PLANNED', 'IN_PROGRESS'].includes(visit.status) && <form className="panel form-panel" onSubmit={cancelCurrentVisit}><div className="panel-heading"><div><span className="eyebrow">CANCELACIÓN</span><h2>Cancelar inspección</h2><p className="section-copy">La cancelación es definitiva y queda auditada. No se permite si la Inspección ya tiene un Acta sellada.</p></div></div><label className="field"><span>Motivo de cancelación</span><textarea value={cancelReason} onChange={(event) => setCancelReason(event.target.value)} minLength={10} maxLength={500} rows={4} placeholder="Explicá el motivo de la cancelación…" required /><small className="muted">{cancelReason.trim().length}/500 · mínimo 10 caracteres</small></label><div className="form-actions"><button type="button" className="button secondary" disabled={busy} onClick={() => { setShowCancelForm(false); setCancelReason(''); }}>Volver</button><button type="submit" className="button danger-outline" disabled={busy || cancelReason.trim().length < 10 || cancelReason.trim().length > 500}>{busy ? 'Cancelando…' : 'Confirmar cancelación'}</button></div></form>}
 
     {visit?.status === 'PLANNED' && <div className="temporal-notice"><Icon name="clipboard" /><p><strong>Planificación lista.</strong> El Inspector asignado debe iniciar la Inspección desde la APK.</p></div>}
     {visit?.status === 'IN_PROGRESS' && <div className="temporal-notice"><Icon name="clipboard" /><p><strong>Inspección en curso.</strong> Actas, Hallazgos, firmas y cierre se registran desde la APK.</p></div>}