mirror of
https://github.com/AutoMaker-Org/automaker.git
synced 2026-02-03 21:03:08 +00:00
fix: move terminal creation debounce to view level
The per-panel debounce didn't work because each new terminal has its own fresh ref. Move debounce to createTerminal function with: - 500ms cooldown between creations - isCreating flag to prevent concurrent requests 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This commit is contained in:
@@ -1,6 +1,6 @@
|
|||||||
"use client";
|
"use client";
|
||||||
|
|
||||||
import React, { useState, useEffect, useCallback, useMemo } from "react";
|
import React, { useState, useEffect, useCallback, useMemo, useRef } from "react";
|
||||||
import {
|
import {
|
||||||
Terminal as TerminalIcon,
|
Terminal as TerminalIcon,
|
||||||
Plus,
|
Plus,
|
||||||
@@ -141,8 +141,11 @@ export function TerminalView() {
|
|||||||
const [authError, setAuthError] = useState<string | null>(null);
|
const [authError, setAuthError] = useState<string | null>(null);
|
||||||
const [activeDragId, setActiveDragId] = useState<string | null>(null);
|
const [activeDragId, setActiveDragId] = useState<string | null>(null);
|
||||||
const [dragOverTabId, setDragOverTabId] = useState<string | null>(null);
|
const [dragOverTabId, setDragOverTabId] = useState<string | null>(null);
|
||||||
|
const lastCreateTimeRef = useRef<number>(0);
|
||||||
|
const isCreatingRef = useRef<boolean>(false);
|
||||||
|
|
||||||
const serverUrl = process.env.NEXT_PUBLIC_SERVER_URL || "http://localhost:3008";
|
const serverUrl = process.env.NEXT_PUBLIC_SERVER_URL || "http://localhost:3008";
|
||||||
|
const CREATE_COOLDOWN_MS = 500; // Prevent rapid terminal creation
|
||||||
|
|
||||||
// Get active tab
|
// Get active tab
|
||||||
const activeTab = terminalState.tabs.find(t => t.id === terminalState.activeTabId);
|
const activeTab = terminalState.tabs.find(t => t.id === terminalState.activeTabId);
|
||||||
@@ -260,6 +263,15 @@ export function TerminalView() {
|
|||||||
// Create a new terminal session
|
// Create a new terminal session
|
||||||
// targetSessionId: the terminal to split (if splitting an existing terminal)
|
// targetSessionId: the terminal to split (if splitting an existing terminal)
|
||||||
const createTerminal = async (direction?: "horizontal" | "vertical", targetSessionId?: string) => {
|
const createTerminal = async (direction?: "horizontal" | "vertical", targetSessionId?: string) => {
|
||||||
|
// Debounce: prevent rapid terminal creation
|
||||||
|
const now = Date.now();
|
||||||
|
if (now - lastCreateTimeRef.current < CREATE_COOLDOWN_MS || isCreatingRef.current) {
|
||||||
|
console.log("[Terminal] Debounced terminal creation");
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
lastCreateTimeRef.current = now;
|
||||||
|
isCreatingRef.current = true;
|
||||||
|
|
||||||
try {
|
try {
|
||||||
const headers: Record<string, string> = {
|
const headers: Record<string, string> = {
|
||||||
"Content-Type": "application/json",
|
"Content-Type": "application/json",
|
||||||
@@ -286,6 +298,8 @@ export function TerminalView() {
|
|||||||
}
|
}
|
||||||
} catch (err) {
|
} catch (err) {
|
||||||
console.error("[Terminal] Create session error:", err);
|
console.error("[Terminal] Create session error:", err);
|
||||||
|
} finally {
|
||||||
|
isCreatingRef.current = false;
|
||||||
}
|
}
|
||||||
};
|
};
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user