orcheo-studio 0.24.0 → 0.24.1

package/src/features/workflow/lib/workflow-storage-api.ts

@@ -14,6 +14,7 @@ import type {
   WorkflowListenerMetricsResponse,
   WorkflowCredentialReadinessResponse,
   WorkflowPublishResponse,
+  WorkflowRunnableConfig,
   CandidateUpdateNote,
 } from "./workflow-storage.types";
 
@@ -43,6 +44,46 @@ const readText = async (response: Response): Promise<string> => {
   }
 };
 
+/**
+ * Extract a human-readable message from an error response body.
+ *
+ * FastAPI serializes errors as `{ "detail": ... }`, where `detail` may be a
+ * plain string, our structured `{ message, code }` object, or a list of
+ * validation errors. Surface the readable message instead of the raw JSON so
+ * error toasts stay legible; fall back to the original text otherwise.
+ */
+export const extractErrorMessage = (body: string): string => {
+  try {
+    const parsed = JSON.parse(body) as unknown;
+    const detail = (parsed as { detail?: unknown })?.detail ?? parsed;
+
+    if (typeof detail === "string") {
+      return detail;
+    }
+    if (detail && typeof detail === "object") {
+      const message = (detail as { message?: unknown }).message;
+      if (typeof message === "string") {
+        return message;
+      }
+      if (Array.isArray(detail)) {
+        const messages = detail
+          .map((item) =>
+            item && typeof item === "object"
+              ? (item as { msg?: unknown }).msg
+              : undefined,
+          )
+          .filter((msg): msg is string => typeof msg === "string");
+        if (messages.length > 0) {
+          return messages.join("; ");
+        }
+      }
+    }
+  } catch {
+    // Body was not JSON; fall through to the raw text.
+  }
+  return body;
+};
+
 export const request = async <T>(
   path: string,
   options: RequestOptions = {},
@@ -56,7 +97,8 @@ export const request = async <T>(
   });
 
   if (!response.ok) {
-    const detail = (await readText(response)) || response.statusText;
+    const body = await readText(response);
+    const detail = body ? extractErrorMessage(body) : response.statusText;
     throw new ApiRequestError(detail, response.status);
   }
 
@@ -454,6 +496,62 @@ export const extractCronConfigFromVersionGraph = (
   return extractCronConfigFromGraphNodes(graph);
 };
 
+// Matches a value that is exactly a single `{{config.configurable.<name>}}`
+// placeholder, optionally padded with whitespace inside the braces.
+const CONFIGURABLE_TEMPLATE = /^\{\{\s*config\.configurable\.([^}\s.]+)\s*\}\}$/;
+
+const resolveConfigurableTemplate = (
+  value: string | null | undefined,
+  configurable: Record<string, unknown> | undefined,
+): string | null | undefined => {
+  if (typeof value !== "string") {
+    return value;
+  }
+  const match = CONFIGURABLE_TEMPLATE.exec(value.trim());
+  if (!match) {
+    return value;
+  }
+  const name = match[1];
+  const resolved = configurable?.[name];
+  if (typeof resolved !== "string") {
+    throw new Error(
+      `Cron trigger references '{{config.configurable.${name}}}' but the ` +
+        `workflow has no configurable value named '${name}'. Set a default ` +
+        `for it in the workflow config.`,
+    );
+  }
+  return resolved;
+};
+
+// Cron triggers may parametrize fields (e.g. `expression`) with
+// `{{config.configurable.X}}` placeholders that the workflow runtime resolves
+// from its `configurable` config. Scheduling needs the concrete value, so
+// resolve any such placeholder against the version's configurable values
+// before the payload is validated as a cron expression by the backend.
+const resolveConfigurableCronConfig = (
+  config: CronTriggerConfig,
+  runnableConfig: WorkflowRunnableConfig | null | undefined,
+): CronTriggerConfig => {
+  const configurable = runnableConfig?.configurable;
+  return {
+    ...config,
+    expression:
+      resolveConfigurableTemplate(config.expression, configurable) ??
+      config.expression,
+    timezone: resolveConfigurableTemplate(config.timezone, configurable) as
+      | string
+      | undefined,
+    start_at: resolveConfigurableTemplate(config.start_at, configurable) as
+      | string
+      | null
+      | undefined,
+    end_at: resolveConfigurableTemplate(config.end_at, configurable) as
+      | string
+      | null
+      | undefined,
+  };
+};
+
 const normalizeTemplateCronConfig = (
   config: CronTriggerConfig,
   metadata: unknown,
@@ -562,7 +660,10 @@ export const scheduleWorkflowFromLatestVersion = async (
 
   const cronConfigRaw = extractCronConfigFromVersionGraph(latest.graph);
   const cronConfig = cronConfigRaw
-    ? normalizeTemplateCronConfig(cronConfigRaw, latest.metadata)
+    ? normalizeTemplateCronConfig(
+        resolveConfigurableCronConfig(cronConfigRaw, latest.runnable_config),
+        latest.metadata,
+      )
     : null;
   if (!cronConfig) {
     return {

package/src/features/workflow/lib/workflow-storage-helpers.ts

@@ -216,6 +216,7 @@ const parseWorkflowMetadata = (
 ): WorkflowVersionMetadata => {
   const configurableSchemas = parseConfigurableSchemas(
     isRecord(metadata) ? metadata.configurable_schema : undefined,
+    isRecord(metadata) ? metadata.configurable_schema_order : undefined,
   );
   const avatarEmoji = extractAvatarEmoji(metadata);
   const candidateSource = extractCandidateSource(metadata);
@@ -330,6 +331,7 @@ const parseWorkflowMetadata = (
 
 const parseConfigurableSchemas = (
   value: unknown,
+  order: unknown,
 ): Record<string, RJSFSchema> | undefined => {
   if (!value || typeof value !== "object" || Array.isArray(value)) {
     return undefined;
@@ -344,6 +346,23 @@ const parseConfigurableSchemas = (
     return undefined;
   }
 
+  // The schema object is persisted in a JSONB column that does not preserve key
+  // order, so reorder by the authored `configurable_schema_order` array when
+  // present. Keys missing from the order list keep their existing position at
+  // the end, keeping output stable for legacy versions without the array.
+  if (Array.isArray(order) && order.length > 0) {
+    const byKey = new Map(entries);
+    const orderedKeys = order.filter(
+      (key): key is string => typeof key === "string" && byKey.has(key),
+    );
+    const seen = new Set(orderedKeys);
+    const orderedEntries = [
+      ...orderedKeys.map((key) => [key, byKey.get(key)] as const),
+      ...entries.filter(([key]) => !seen.has(key)),
+    ];
+    return Object.fromEntries(orderedEntries) as Record<string, RJSFSchema>;
+  }
+
   return Object.fromEntries(entries) as Record<string, RJSFSchema>;
 };
 

package/src/features/workflow/pages/workflow/components/workflow-config-sheet.tsx

@@ -44,12 +44,28 @@ const baseWorkflowConfigSchema: RJSFSchema = {
   },
 };
 
-const workflowConfigUiSchema: UiSchema = {
-  configurable: {
-    fields: {
-      "ui:field": "jsonObject",
+const buildWorkflowConfigUiSchema = (
+  configurableSchemas?: Record<string, RJSFSchema>,
+): UiSchema => {
+  // Opt any key declared with `"format": "cron"` into the visual cron widget.
+  const cronEntries: UiSchema = {};
+  for (const [key, schema] of Object.entries(configurableSchemas ?? {})) {
+    if (schema?.format === "cron") {
+      cronEntries[key] = { "ui:widget": "cron" };
+    }
+  }
+
+  return {
+    configurable: {
+      // Render configurable fields in the order declared by config.json,
+      // with any data-only keys appended afterwards via the "*" wildcard.
+      "ui:order": [...Object.keys(configurableSchemas ?? {}), "*"],
+      fields: {
+        "ui:field": "jsonObject",
+      },
+      ...cronEntries,
     },
-  },
+  };
 };
 
 interface WorkflowConfigSheetProps {
@@ -91,6 +107,11 @@ export function WorkflowConfigSheet({
     };
   }, [formData.configurable, configurableSchemas]);
 
+  const workflowConfigUiSchema = useMemo<UiSchema>(
+    () => buildWorkflowConfigUiSchema(configurableSchemas),
+    [configurableSchemas],
+  );
+
   useEffect(() => {
     if (open) {
       setFormData(toFormData(initialConfig));

package/src/features/workflow/pages/workflow/components/workflow-tab-content.tsx

@@ -2,12 +2,14 @@ import { useEffect, useMemo, useState } from "react";
 import { useUploadsAllowed } from "@/hooks/use-uploads-allowed";
 import {
   AlertTriangle,
+  CheckCircle2,
   Copy,
   ExternalLink,
   LoaderCircle,
   Play,
   RefreshCw,
   UserMinus,
+  XCircle,
 } from "lucide-react";
 import { useNavigate } from "react-router-dom";
 import { Controls, ReactFlow, type Node, type NodeProps } from "@xyflow/react";
@@ -56,6 +58,7 @@ import {
 } from "@features/workflow/lib/mermaid-renderer";
 import { resolveWorkflowVersionMermaidSource } from "@features/workflow/lib/workflow-storage-helpers";
 import { WorkflowConfigSheet } from "@features/workflow/pages/workflow/components/workflow-config-sheet";
+import type { WorkflowExecutionStatus } from "@features/workflow/pages/workflow/helpers/types";
 
 export interface WorkflowTabContentProps {
   workflowId: string | null;
@@ -66,6 +69,7 @@ export interface WorkflowTabContentProps {
   loadError: string | null;
   isRunPending: boolean;
   isRunning: boolean;
+  lastRunStatus?: WorkflowExecutionStatus | null;
   onRunWorkflow: () => Promise<void>;
   onSaveConfig: (nextConfig: WorkflowRunnableConfig | null) => Promise<void>;
   hasCronTriggerNode: boolean;
@@ -189,6 +193,49 @@ const getErrorMessage = (error: unknown, fallback: string): string => {
   return rawMessage;
 };
 
+interface RunResultBanner {
+  Icon: typeof CheckCircle2;
+  className: string;
+  message: string;
+}
+
+const resolveRunResultBanner = (
+  status: WorkflowExecutionStatus,
+): RunResultBanner => {
+  switch (status) {
+    case "success":
+      return {
+        Icon: CheckCircle2,
+        className:
+          "border-emerald-500/40 bg-emerald-50 text-emerald-700 dark:bg-emerald-950/40 dark:text-emerald-300",
+        message: "Workflow run succeeded. See the full trace on the Trace tab.",
+      };
+    case "failed":
+      return {
+        Icon: XCircle,
+        className: "border-destructive/40 bg-destructive/5 text-destructive",
+        message: "Workflow run failed. Check the Trace tab for details.",
+      };
+    case "partial":
+      return {
+        Icon: AlertTriangle,
+        className:
+          "border-amber-500/40 bg-amber-50 text-amber-700 dark:bg-amber-950/40 dark:text-amber-300",
+        message:
+          "Workflow run finished with partial results. Check the Trace tab for details.",
+      };
+    default:
+      // "running" reaching here means the stream ended without a final status.
+      return {
+        Icon: AlertTriangle,
+        className:
+          "border-amber-500/40 bg-amber-50 text-amber-700 dark:bg-amber-950/40 dark:text-amber-300",
+        message:
+          "Workflow run ended without a final status. Check the Trace tab for the latest status.",
+      };
+  }
+};
+
 export function WorkflowTabContent({
   workflowId,
   workflowName,
@@ -197,6 +244,7 @@ export function WorkflowTabContent({
   loadError,
   isRunPending,
   isRunning,
+  lastRunStatus = null,
   onRunWorkflow,
   onSaveConfig,
   hasCronTriggerNode,
@@ -390,6 +438,11 @@ export function WorkflowTabContent({
   }
 
   const canConfigure = Boolean(workflowId);
+  const isRunActive = isRunPending || isRunning;
+  const runResultBanner =
+    !isRunActive && lastRunStatus
+      ? resolveRunResultBanner(lastRunStatus)
+      : null;
   const canRun = Boolean(workflowId && latestVersion);
   const runButtonDisabled = !canRun || isRunPending || isRunning;
   const runButtonLabel = isRunPending || isRunning ? "Running..." : "Run";
@@ -667,7 +720,7 @@ export function WorkflowTabContent({
           </div>
         </div>
 
-        {(isRunPending || isRunning) && (
+        {isRunActive && (
           <div
             role="status"
             className="rounded-md border border-primary/30 bg-primary/5 px-3 py-2 text-sm text-primary"
@@ -677,6 +730,16 @@ export function WorkflowTabContent({
           </div>
         )}
 
+        {runResultBanner && (
+          <div
+            role="status"
+            className={`flex items-center gap-2 rounded-md border px-3 py-2 text-sm ${runResultBanner.className}`}
+          >
+            <runResultBanner.Icon className="h-4 w-4 shrink-0" />
+            <span>{runResultBanner.message}</span>
+          </div>
+        )}
+
         {isPublished && shareUrl && (
           <div className="flex items-center justify-between rounded-md border border-border/60 bg-muted/20 px-3 py-2">
             <div className="min-w-0">

package/src/features/workflow/pages/workflow/hooks/controller/build-layout-props.ts

@@ -75,6 +75,10 @@ export function buildWorkflowLayoutProps(
   resources: WorkflowResources,
   execution: WorkflowExecutionController,
 ): WorkflowLayoutProps {
+  const activeExecution = core.execution.executions.find(
+    (execution) => execution.id === core.execution.activeExecutionId,
+  );
+
   const workflowProps: WorkflowTabContentProps = {
     workflowId: core.metadata.currentWorkflowId,
     workflowRouteRef: core.routeWorkflowRef,
@@ -84,6 +88,7 @@ export function buildWorkflowLayoutProps(
     loadError: core.metadata.workflowLoadError,
     isRunPending: execution.isRunPending,
     isRunning: core.execution.isRunning,
+    lastRunStatus: activeExecution?.status ?? null,
     onRunWorkflow: execution.handleRunPersistedWorkflow,
     onSaveConfig: resources.saver.handleSaveWorkflowConfig,
     hasCronTriggerNode: hasSchedulableCronTrigger(

package/src/lib/api.ts

@@ -405,6 +405,7 @@ export async function acceptWorkspaceInvitation(
 
 export interface ServiceToken {
   identifier: string;
+  name?: string | null;
   secret?: string | null;
   secret_preview?: string | null;
   scopes: string[];
@@ -425,7 +426,7 @@ export interface ServiceTokenListResponse {
 }
 
 export interface CreateServiceTokenRequest {
-  identifier?: string;
+  name?: string;
   scopes?: string[];
   expires_in_seconds?: number | null;
 }
@@ -504,22 +505,6 @@ export async function createServiceToken(
   );
 }
 
-export async function rotateServiceToken(
-  tokenId: string,
-  overlapSeconds: number,
-  baseUrl?: string,
-): Promise<ServiceToken> {
-  return serviceTokenRequest<ServiceToken>(
-    `/api/admin/service-tokens/${encodeURIComponent(tokenId)}/rotate`,
-    {
-      method: "POST",
-      body: JSON.stringify({ overlap_seconds: overlapSeconds }),
-    },
-    "Failed to rotate service token",
-    baseUrl,
-  );
-}
-
 export async function revokeServiceToken(
   tokenId: string,
   reason: string,

package/dist/assets/channel-C3cH7p5A.js

@@ -1 +0,0 @@
-import{ap as o,aq as n}from"./index-DlptXU2u.js";const t=(a,r)=>o.lang.round(n.parse(a)[r]);export{t as c};

package/dist/assets/classDiagram-2ON5EDUG-o6x5AEsV.js

@@ -1 +0,0 @@
-import{s as a,c as s,a as e,C as t}from"./chunk-B4BG7PRW-BI7iPFRv.js";import{_ as i}from"./index-DlptXU2u.js";import"./chunk-FMBD7UC4-BFVTHaU-.js";import"./chunk-55IACEB6-CwinXV4-.js";import"./chunk-QN33PNHL-xWy7sSv1.js";var p={parser:e,get db(){return new t},renderer:s,styles:a,init:i(r=>{r.class||(r.class={}),r.class.arrowMarkerAbsolute=r.arrowMarkerAbsolute},"init")};export{p as diagram};

package/dist/assets/classDiagram-v2-WZHVMYZB-o6x5AEsV.js

@@ -1 +0,0 @@
-import{s as a,c as s,a as e,C as t}from"./chunk-B4BG7PRW-BI7iPFRv.js";import{_ as i}from"./index-DlptXU2u.js";import"./chunk-FMBD7UC4-BFVTHaU-.js";import"./chunk-55IACEB6-CwinXV4-.js";import"./chunk-QN33PNHL-xWy7sSv1.js";var p={parser:e,get db(){return new t},renderer:s,styles:a,init:i(r=>{r.class||(r.class={}),r.class.arrowMarkerAbsolute=r.arrowMarkerAbsolute},"init")};export{p as diagram};

package/dist/assets/clone-CnJR7M_G.js

@@ -1 +0,0 @@
-import{ay as r}from"./index-DlptXU2u.js";var a=4;function n(o){return r(o,a)}export{n as c};