From 2bc8a85bfa71944d21854c6c85ecd0d67e5c9af8 Mon Sep 17 00:00:00 2001
From: chihlasm <michael@chihlas.com>
Date: Tue, 24 Feb 2026 21:57:23 -0500
Subject: [PATCH] fix: suppress StepFeedback on custom steps, fix resume
 stepState seeding, functional updater for step index

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 frontend/src/pages/ProceduralNavigationPage.tsx | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/frontend/src/pages/ProceduralNavigationPage.tsx b/frontend/src/pages/ProceduralNavigationPage.tsx
index 26fc7781..3f8afe8b 100644
--- a/frontend/src/pages/ProceduralNavigationPage.tsx
+++ b/frontend/src/pages/ProceduralNavigationPage.tsx
@@ -221,8 +221,10 @@ export function ProceduralNavigationPage() {
       setRuntimeSteps(hydrated)
 
       const initialStates = new Map<string, StepState>()
-      for (const step of allSteps) {
-        initialStates.set(step.id, { notes: '', verificationValue: '', completedAt: null })
+      for (const step of hydrated) {
+        if (step.type === 'procedure_step') {
+          initialStates.set(step.id, { notes: '', verificationValue: '', completedAt: null })
+        }
       }
 
       // Hydrate completed steps from decisions
@@ -406,7 +408,7 @@ export function ProceduralNavigationPage() {
       toast.error('Failed to save custom step')
     }
 
-    setCurrentStepIndex(currentStepIndex + 1)
+    setCurrentStepIndex(prev => prev + 1)
   }
 
   const handleSaveForLater = async () => {
@@ -625,7 +627,7 @@ export function ProceduralNavigationPage() {
             </div>
           )}
 
-          {session && currentStep && (
+          {session && currentStep && !('isCustom' in currentStep && currentStep.isCustom) && (
             <div className="mt-3 flex justify-end">
               <StepFeedback stepId={currentStep.id} sessionId={session.id} />
             </div>