From 5c79032637a9593f1530599726842f49ac904a13 Mon Sep 17 00:00:00 2001 From: soryu Date: Tue, 20 Jan 2026 00:34:39 +0000 Subject: Fix TypeScript build error: remove unused useMemo import Remove unused useMemo import from contracts.tsx that was causing the TypeScript build to fail with TS6133 error. Co-Authored-By: Claude Opus 4.5 --- makima/frontend/src/routes/contracts.tsx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'makima/frontend/src') diff --git a/makima/frontend/src/routes/contracts.tsx b/makima/frontend/src/routes/contracts.tsx index 0893ff6..6acda29 100644 --- a/makima/frontend/src/routes/contracts.tsx +++ b/makima/frontend/src/routes/contracts.tsx @@ -1,4 +1,4 @@ -import { useState, useCallback, useEffect, useMemo } from "react"; +import { useState, useCallback, useEffect } from "react"; import { useParams, useNavigate } from "react-router"; import { Masthead } from "../components/Masthead"; import { ContractList } from "../components/contracts/ContractList"; -- cgit v1.2.3 From 94e14814573c8f1ba31f940c0479334a16b32f39 Mon Sep 17 00:00:00 2001 From: soryu Date: Tue, 20 Jan 2026 16:39:04 +0000 Subject: Fix: prevent supervisor auto-start for completed contracts When viewing a completed contract, the supervisor was auto-starting on component mount. This would resurrect completed contracts and cause them to continue running. Changes: - Add contract.status !== 'completed' check to the auto-start condition - Add contract.status to the useEffect dependency array Co-Authored-By: Claude Opus 4.5 --- makima/frontend/src/components/contracts/ContractCliInput.tsx | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'makima/frontend/src') diff --git a/makima/frontend/src/components/contracts/ContractCliInput.tsx b/makima/frontend/src/components/contracts/ContractCliInput.tsx index 821d03c..54d9f3a 100644 --- a/makima/frontend/src/components/contracts/ContractCliInput.tsx +++ b/makima/frontend/src/components/contracts/ContractCliInput.tsx @@ -279,9 +279,9 @@ export function ContractCliInput({ contractId, contract, onUpdate }: ContractCli } }, [messages]); - // Auto-start supervisor when component mounts if it's pending + // Auto-start supervisor when component mounts if it's pending (but not for completed contracts) useEffect(() => { - if (supervisorTask && isSupervisorPending && !supervisorStarting) { + if (supervisorTask && isSupervisorPending && !supervisorStarting && contract.status !== 'completed') { console.log("Auto-starting supervisor task on mount..."); ensureSupervisorStarted().then((started) => { if (started) { @@ -289,7 +289,7 @@ export function ContractCliInput({ contractId, contract, onUpdate }: ContractCli } }); } - }, [supervisorTask?.id]); // Only run when task ID changes, not on every render + }, [supervisorTask?.id, contract.status]); // Only run when task ID or contract status changes // Convert supervisor output events to messages useEffect(() => { -- cgit v1.2.3