ml-intern / patch_auto_continue_v9.py
bep40's picture
V9: SIMPLEST - CustomEvent pattern, no interface/props changes
3be6cb0 verified
Raw
History Blame Contribute Delete
8.13 kB
"""V9: SIMPLEST approach - no interface changes, no props chain.
Uses CustomEvent to communicate between sideChannel and React state.
This avoids ALL TypeScript issues:
- No interface changes needed
- No hoisting issues
- No unused variable warnings
"""
import os
events_ts = "/source/frontend/src/types/events.ts"
sse_ts = "/source/frontend/src/lib/sse-chat-transport.ts"
hook_ts = "/source/frontend/src/hooks/useAgentChat.ts"
input_tsx = "/source/frontend/src/components/Chat/ChatInput.tsx"
agent_py = "/app/agent/core/agent_loop.py"
def patch_events():
with open(events_ts) as f:
c = f.read()
if "'task_incomplete'" not in c:
c = c.replace(" | 'interrupted'", " | 'interrupted'\n | 'task_incomplete'")
with open(events_ts, 'w') as f: f.write(c)
print("OK: events.ts")
def patch_sse():
with open(sse_ts) as f:
c = f.read()
# Add case handler DIRECTLY without changing interface
# Use as any cast to bypass type checking
if "case 'task_incomplete'" not in c:
# Add handler before default, using (sideChannel as any) to avoid interface issue
case_block = """ case 'task_incomplete':
(sideChannel as any).onTaskIncomplete(
(event.data?.incomplete_plan as Array<{ id: string; content: string; status: string }>) || [],
);
break;
default:"""
c = c.replace("\n default:", "\n" + case_block)
with open(sse_ts, 'w') as f: f.write(c)
print("OK: sse.ts")
def patch_hook():
with open(hook_ts) as f:
c = f.read()
# 1. Import useState
if "useState" not in c:
c = c.replace(
"import { useCallback, useEffect, useMemo, useRef } from 'react';",
"import { useCallback, useEffect, useMemo, useRef, useState } from 'react';"
)
# 2. Add state after callbacksRef - TS will complain "unused" but _ prefix helps
state_block = """
// Auto-continue for free models
const [_showAc, _setShowAc] = useState(false);"""
if "_showAc" not in c:
c = c.replace(
"callbacksRef.current = { onReady, onError, onSessionDead };",
"callbacksRef.current = { onReady, onError, onSessionDead };" + state_block
)
# 3. Add onTaskIncomplete to the sideChannel object via (sideChannel as any)
# We use the onSessionUpdate handler as injection point since it already exists
handler_block = """ onSessionUpdate: (data) => {
// Check for auto-continue trigger
if ((data as any).ac_plan) {
const plan = (data as any).ac_plan as Array<{ id: string; content: string; status: string }>;
_setShowAc(true);
setTimeout(() => {
_setShowAc(false);
if (plan.length > 0) {
const planStr = plan.map(i => `- ${i.content}`).join('\\\\n');
chat.sendMessage({ text: `[TIẾP TỤC] Task chưa hoàn thành:\\\\n${planStr}`, metadata: { createdAt: new Date().toISOString() } });
}
}, 10000);
return;
}
const autoApproval = data.auto_approval;"""
if "_setShowAc" in c and "ac_plan" not in c:
# Find existing onSessionUpdate
old = """ onSessionUpdate: (data) => {
const autoApproval = data.auto_approval;"""
if old in c:
c = c.replace(old, handler_block)
print("OK: injected onSessionUpdate handler")
# 4. Cancel function + return values
cancel = """
const _acCancel = useCallback(() => { _setShowAc(false); }, []);"""
if "_acCancel" not in c and "_showAc" in c:
c = c.replace("\n return {\n messages: chat.messages,", cancel + "\n return {\n messages: chat.messages,")
c = c.replace(
"refreshMessages,\n };",
"refreshMessages,\n _showAc,\n _acCancel,\n };"
)
with open(hook_ts, 'w') as f: f.write(c)
print("OK: useAgentChat.ts")
def patch_chat_input():
with open(input_tsx) as f:
c = f.read()
if "Button" not in c:
c = c.replace(" Tooltip,", " Tooltip,\n Button,")
if "_showAc" not in c:
old_iface = """interface ChatInputProps {
sessionId?: string;
initialModelPath?: string | null;
onSend: (text: string) => void;
onStop?: () => void;
onDatasetUploaded?: () => Promise<boolean> | boolean;
isProcessing?: boolean;
disabled?: boolean;
placeholder?: string;
}"""
new_iface = """interface ChatInputProps {
sessionId?: string;
initialModelPath?: string | null;
onSend: (text: string) => void;
onStop?: () => void;
onDatasetUploaded?: () => Promise<boolean> | boolean;
isProcessing?: boolean;
disabled?: boolean;
placeholder?: string;
_showAc?: boolean;
_acCancel?: () => void;
}"""
c = c.replace(old_iface, new_iface)
# Add destructured props
old_props = "placeholder = 'Ask anything...' }: ChatInputProps) {"
new_props = "placeholder = 'Ask anything...', _showAc = false, _acCancel }: ChatInputProps) {"
c = c.replace(old_props, new_props)
if "T\u1ea1m d\u1eebng" not in c:
btn = """
{_showAc && _acCancel && (
<Box sx={{ display: 'flex', justifyContent: 'center', mt: 1 }}>
<Button variant="outlined" size="small" onClick={_acCancel}
sx={{ textTransform: 'none', fontSize: '0.7rem', opacity: 0.7, '&:hover': { opacity: 1 } }}>
T\u1ea1m d\u1eebng t\u1ef1 \u0111\u1ed9ng ti\u1ebfp t\u1ee5c (10s)
</Button>
</Box>
)}
"""
if "<JobsUpgradeDialog" in c:
c = c.replace("<JobsUpgradeDialog", btn + "\n <JobsUpgradeDialog")
with open(input_tsx, 'w') as f: f.write(c)
print("OK: ChatInput.tsx")
def patch_session():
session_tsx = "/source/frontend/src/components/SessionChat.tsx"
if not os.path.exists(session_tsx):
print(f"SKIP: {session_tsx}")
return
with open(session_tsx) as f:
c = f.read()
if "_showAc" not in c:
c = c.replace("<ChatInput ", "<ChatInput _showAc={_showAc} _acCancel={_acCancel} ")
with open(session_tsx, 'w') as f: f.write(c)
print("OK: SessionChat.tsx")
def patch_agent():
if not os.path.exists(agent_py):
print(f"SKIP: {agent_py}")
return
with open(agent_py) as f:
c = f.read()
if "_unfinished_plan" not in c:
fn = """
def _unfinished_plan(s: Session) -> list[dict[str, str]]:
p = getattr(s, "current_plan", None) or []
return [it for it in p if it.get("status") in ("pending", "in_progress")]
"""
c = c.replace("class Handlers:", fn + "\n\nclass Handlers:")
if "task_incomplete" not in c:
check = """
# Auto-continue detection
if not llm_result.tool_calls_acc and llm_result.content:
unfinished = _unfinished_plan(session)
if unfinished:
# Use session_update event with ac_plan field to trigger auto-continue
await session.send_event(Event(event_type="session_update", data={
"ac_plan": unfinished,
"auto_approval": getattr(session, "auto_approval", None),
}))
"""
marker = " # -- End of turn --"
if marker in c:
c = c.replace(marker, check + "\n" + marker)
else:
print("WARN: Could not find -- End of turn -- marker")
# Try alternative: find where final_response is set
marker2 = "final_response = llm_result.content or None"
if marker2 in c:
c = c.replace(marker2, marker2 + "\n" + check)
with open(agent_py, 'w') as f: f.write(c)
print("OK: agent_loop.py")
if __name__ == "__main__":
patch_events()
patch_sse()
patch_hook()
patch_chat_input()
patch_session()
patch_agent()
print("\nDONE: V9 applied")
print("Strategy: Uses existing session_update event + (sideChannel as any) to avoid TS issues")