vodorod/Dorod-Sky
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

Jon/sky 5981 turn off debugger for oss and self host (#3243)

Browse Source
This commit is contained in:
Jonathan Dobson
2025-08-20 14:28:33 -04:00
committed by GitHub
parent 0913e4f0a0
commit d0296f0e25
3 changed files with 37 additions and 36 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
skyvern-frontend/src/router.tsx
View File

@@ -18,7 +18,6 @@ import { WorkflowRun } from "./routes/workflows/WorkflowRun";
import { WorkflowRunParameters } from "./routes/workflows/WorkflowRunParameters"; import { WorkflowRunParameters } from "./routes/workflows/WorkflowRunParameters";
import { Workflows } from "./routes/workflows/Workflows"; import { Workflows } from "./routes/workflows/Workflows";
import { WorkflowsPageLayout } from "./routes/workflows/WorkflowsPageLayout"; import { WorkflowsPageLayout } from "./routes/workflows/WorkflowsPageLayout";
import { Debugger } from "./routes/workflows/debugger/Debugger";
import { WorkflowEditor } from "./routes/workflows/editor/WorkflowEditor"; import { WorkflowEditor } from "./routes/workflows/editor/WorkflowEditor";
import { WorkflowPostRunParameters } from "./routes/workflows/workflowRun/WorkflowPostRunParameters"; import { WorkflowPostRunParameters } from "./routes/workflows/workflowRun/WorkflowPostRunParameters";
import { WorkflowRunOutput } from "./routes/workflows/workflowRun/WorkflowRunOutput"; import { WorkflowRunOutput } from "./routes/workflows/workflowRun/WorkflowRunOutput";
@@ -111,11 +110,11 @@ const router = createBrowserRouter([
}, },
{ {
path: "debug", path: "debug",
element: <Debugger />, element: <WorkflowEditor />,
}, },
{ {
path: ":workflowRunId/:blockLabel/debug", path: ":workflowRunId/:blockLabel/debug",
element: <Debugger />, element: <WorkflowEditor />,
}, },
{ {
path: "edit", path: "edit",

64
skyvern-frontend/src/routes/workflows/editor/WorkflowHeader.tsx
View File

@@ -15,6 +15,7 @@ import {
ReloadIcon, ReloadIcon,
} from "@radix-ui/react-icons"; } from "@radix-ui/react-icons";
import { useNavigate, useParams } from "react-router-dom"; import { useNavigate, useParams } from "react-router-dom";
import { useUser } from "@/hooks/useUser";
import { statusIsRunningOrQueued } from "@/routes/tasks/types"; import { statusIsRunningOrQueued } from "@/routes/tasks/types";
import { useGlobalWorkflowsQuery } from "../hooks/useGlobalWorkflowsQuery"; import { useGlobalWorkflowsQuery } from "../hooks/useGlobalWorkflowsQuery";
import { EditableNodeTitle } from "./nodes/components/EditableNodeTitle"; import { EditableNodeTitle } from "./nodes/components/EditableNodeTitle";
@@ -51,6 +52,7 @@ function WorkflowHeader({
const debugStore = useDebugStore(); const debugStore = useDebugStore();
const workflowRunIsRunningOrQueued = const workflowRunIsRunningOrQueued =
workflowRun && statusIsRunningOrQueued(workflowRun); workflowRun && statusIsRunningOrQueued(workflowRun);
const user = useUser().get();
if (!globalWorkflows) { if (!globalWorkflows) {
return null; // this should be loaded already by some other components return null; // this should be loaded already by some other components
@@ -103,36 +105,38 @@ function WorkflowHeader({
</Button> </Button>
) : ( ) : (
<> <>
<TooltipProvider> {user && (
<Tooltip> <TooltipProvider>
<TooltipTrigger asChild> <Tooltip>
<Button <TooltipTrigger asChild>
size="icon" <Button
variant={debugStore.isDebugMode ? "default" : "tertiary"} size="icon"
className="size-10" variant={debugStore.isDebugMode ? "default" : "tertiary"}
disabled={workflowRunIsRunningOrQueued} className="size-10"
onClick={() => { disabled={workflowRunIsRunningOrQueued}
if (debugStore.isDebugMode) { onClick={() => {
navigate(`/workflows/${workflowPermanentId}/edit`); if (debugStore.isDebugMode) {
} else { navigate(`/workflows/${workflowPermanentId}/edit`);
navigate(`/workflows/${workflowPermanentId}/debug`); } else {
} navigate(`/workflows/${workflowPermanentId}/debug`);
}} }
> }}
{debugStore.isDebugMode ? ( >
<BrowserIcon className="h-6 w-6" /> {debugStore.isDebugMode ? (
) : ( <BrowserIcon className="h-6 w-6" />
<BrowserIcon className="h-6 w-6" /> ) : (
)} <BrowserIcon className="h-6 w-6" />
</Button> )}
</TooltipTrigger> </Button>
<TooltipContent> </TooltipTrigger>
{debugStore.isDebugMode <TooltipContent>
? "Turn off Browser" {debugStore.isDebugMode
: "Turn on Browser"} ? "Turn off Browser"
</TooltipContent> : "Turn on Browser"}
</Tooltip> </TooltipContent>
</TooltipProvider> </Tooltip>
</TooltipProvider>
)}
<TooltipProvider> <TooltipProvider>
<Tooltip> <Tooltip>
<TooltipTrigger asChild> <TooltipTrigger asChild>

4
skyvern-frontend/src/routes/workflows/editor/Workspace.tsx
View File

@@ -36,7 +36,6 @@ import { DebuggerRun } from "@/routes/workflows/debugger/DebuggerRun";
import { useWorkflowRunQuery } from "@/routes/workflows/hooks/useWorkflowRunQuery"; import { useWorkflowRunQuery } from "@/routes/workflows/hooks/useWorkflowRunQuery";
import { DebuggerRunOutput } from "@/routes/workflows/debugger/DebuggerRunOutput"; import { DebuggerRunOutput } from "@/routes/workflows/debugger/DebuggerRunOutput";
import { DebuggerPostRunParameters } from "@/routes/workflows/debugger/DebuggerPostRunParameters"; import { DebuggerPostRunParameters } from "@/routes/workflows/debugger/DebuggerPostRunParameters";
import { useDebugStore } from "@/store/useDebugStore";
import { useWorkflowPanelStore } from "@/store/WorkflowPanelStore"; import { useWorkflowPanelStore } from "@/store/WorkflowPanelStore";
import { import {
useWorkflowHasChangesStore, useWorkflowHasChangesStore,
@@ -88,7 +87,6 @@ function Workspace({
const [content, setContent] = useState("actions"); const [content, setContent] = useState("actions");
const { workflowPanelState, setWorkflowPanelState, closeWorkflowPanel } = const { workflowPanelState, setWorkflowPanelState, closeWorkflowPanel } =
useWorkflowPanelStore(); useWorkflowPanelStore();
const debugStore = useDebugStore();
const [nodes, setNodes, onNodesChange] = useNodesState(initialNodes); const [nodes, setNodes, onNodesChange] = useNodesState(initialNodes);
const [edges, setEdges, onEdgesChange] = useEdgesState(initialEdges); const [edges, setEdges, onEdgesChange] = useEdgesState(initialEdges);
const saveWorkflow = useWorkflowSave(); const saveWorkflow = useWorkflowSave();
@@ -488,7 +486,7 @@ function Workspace({
</div> </div>
)} )}
{debugStore.isDebugMode && ( {showBrowser && (
<div <div
className="absolute right-6 top-[8.5rem] h-[calc(100vh-9.5rem)]" className="absolute right-6 top-[8.5rem] h-[calc(100vh-9.5rem)]"
style={{ zIndex: rankedItems.history ?? 1 }} style={{ zIndex: rankedItems.history ?? 1 }}