@revenexx/studio-integrations 3.3.0 → 3.4.0

package/CHANGELOG.md

@@ -1,5 +1,15 @@
 # @revenexx/studio-integrations
 
+## 3.4.0
+
+### Minor Changes
+
+- e755223: A workflow's own Runs tab can be narrowed by outcome and by what started a run, with the same Filter control as the Runs page. The platform narrows the workflow's whole history before it pages it, so a failure from three pages back is found, and the pager counts the matches. A narrowing that matches nothing says so, instead of saying the workflow has never run.
+
+### Patch Changes
+
+- 57b4430: A workflow's own Runs tab says what started each run. Every row there read "Manual", whatever had fired the run, because the list sent no trigger and the studio filled the gap with that word. The tab now shows the trigger the platform reports, and a dash where it reports none — a deleted trigger, or a run older than the platform recording it — on the tenant-wide Runs page as well. A run that is still going keeps its trigger while the list re-reads it.
+
 ## 3.3.0
 
 ### Minor Changes

package/dist/module.json

@@ -1,7 +1,7 @@
 {
   "name": "@revenexx/studio-integrations",
   "configKey": "studioIntegrations",
-  "version": "3.3.0",
+  "version": "3.4.0",
   "builder": {
     "@nuxt/module-builder": "1.0.3",
     "unbuild": "unknown"

package/dist/runtime/components/Integrations/RunsTable.d.vue.ts

@@ -39,6 +39,14 @@ type __VLS_Props = {
     /** The order the platform is reading the list in. Server mode or orderable only. */
     sort?: RunSort;
     direction?: RunSortDirection;
+    /**
+     * The owning list asks the platform to narrow it even though the table is not
+     * in server mode — the workflow's own Runs tab, which narrows on the server but
+     * keeps its own pager (PO-319). The table then offers the same narrowing control
+     * as the tenant-wide list and reports a choice through `narrow`. Server mode
+     * implies it.
+     */
+    narrowable?: boolean;
 };
 type __VLS_PublicProps = __VLS_Props & {
     /** 1-based, as the platform pages. The shared table counts from zero. */
@@ -81,5 +89,6 @@ declare const _default: import("vue").DefineComponent<__VLS_PublicProps, {}, {},
     showWorkflow: boolean;
     serverMode: boolean;
     orderable: boolean;
+    narrowable: boolean;
 }, {}, {}, {}, string, import("vue").ComponentProvideOptions, false, {}, any>;
 export default _default;

package/dist/runtime/components/Integrations/RunsTable.vue

@@ -27,13 +27,14 @@ const props = defineProps({
   triggerType: { type: String, required: false, default: "" },
   orderable: { type: Boolean, required: false, default: false },
   sort: { type: null, required: false, default: DEFAULT_RUN_SORT },
-  direction: { type: String, required: false, default: DEFAULT_RUN_DIRECTION }
+  direction: { type: String, required: false, default: DEFAULT_RUN_DIRECTION },
+  narrowable: { type: Boolean, required: false, default: false }
 });
 const emit = defineEmits(["changed", "narrow", "search", "page-size", "reorder"]);
 const page = defineModel("page", { type: Number, ...{ default: 1 } });
 const t = useIntegrationsT();
 const router = useRouter();
-const emptyText = computed(() => props.emptyText ?? (anyNarrowing.value || searchText.value ? t("integration.runs.table.empty.narrowed") : t("integration.runs.table.empty")));
+const emptyText = computed(() => anyNarrowing.value || searchText.value ? t("integration.runs.table.empty.narrowed") : props.emptyText ?? t("integration.runs.table.empty"));
 const now = useNow();
 function openRun(r) {
   void router.push({ path: `/integrations/workflows/${r.workflowId}`, query: { run: r.id } });
@@ -45,7 +46,8 @@ function liveDuration(run) {
   const end = run.finishedAt ? Date.parse(run.finishedAt) : now.value;
   return end - start;
 }
-const filterTabs = computed(() => props.pagination && !props.serverMode ? [
+const narrowsOnServer = computed(() => props.serverMode || props.narrowable);
+const filterTabs = computed(() => props.pagination && !narrowsOnServer.value ? [
   {
     key: "status",
     label: t("integration.runs.table.column.status"),
@@ -140,7 +142,7 @@ const columns = computed(() => [
     :row-key="(r) => r.id"
     :search="{ placeholder: t('integration.runs.table.search.placeholder') }"
     :filter-tabs="filterTabs"
-    :filter="serverMode ? { width: 'w-72', active: anyNarrowing } : false"
+    :filter="narrowsOnServer ? { width: 'w-72', active: anyNarrowing } : false"
     :server-mode="serverMode"
     :total="total"
     :loading="loading"
@@ -160,14 +162,15 @@ const columns = computed(() => [
       apply them together; this one picks one value per dimension and applies it on
       the spot, because each pick is a request to the platform and staging them
       would only delay the same round trip. So the shared narrowing helper does not
-      reach this page and `runs.spec.ts` carries its own — said in both places
-      rather than left to be discovered (`e2e/support/studio-table.ts`).
+      reach either run list, which share one of their own in `e2e/support/runFilter.ts`
+      — said in both places rather than left to be discovered
+      (`e2e/support/studio-table.ts`).
 
       What is still NOT copied is the count beside each option: the built-in counts
       the rows in hand, and counting a page here would put a number next to
       "failed" that describes this page rather than the history the option narrows.
     -->
-    <template v-if="serverMode" #filter-menu>
+    <template v-if="narrowsOnServer" #filter-menu>
       <!--
         Two selects rather than the sub-menus this slot used to hold. Since
         `studio-shared` 0.3.2 the filter trigger opens a popover instead of a
@@ -242,9 +245,12 @@ const columns = computed(() => [
     </template>
 
     <template #cell-trigger="{ row: run }">
-      <span class="inline-flex items-center whitespace-nowrap rounded-md border border-black/[0.06] bg-muted/50 px-1.5 py-0.5 text-xs font-medium uppercase text-muted-foreground">
+      <span v-if="run.triggerType" class="inline-flex items-center whitespace-nowrap rounded-md border border-black/[0.06] bg-muted/50 px-1.5 py-0.5 text-xs font-medium uppercase text-muted-foreground">
         {{ triggerTypeLabel(run.triggerType) }}
       </span>
+      <!-- What started the run is not known: nothing is said, rather than a guess
+           (specs/run-row.md AC-14). -->
+      <span v-else class="text-xs text-muted-foreground">—</span>
     </template>
 
     <template #cell-started="{ row: run }">

package/dist/runtime/components/Integrations/RunsTable.vue.d.ts

@@ -39,6 +39,14 @@ type __VLS_Props = {
     /** The order the platform is reading the list in. Server mode or orderable only. */
     sort?: RunSort;
     direction?: RunSortDirection;
+    /**
+     * The owning list asks the platform to narrow it even though the table is not
+     * in server mode — the workflow's own Runs tab, which narrows on the server but
+     * keeps its own pager (PO-319). The table then offers the same narrowing control
+     * as the tenant-wide list and reports a choice through `narrow`. Server mode
+     * implies it.
+     */
+    narrowable?: boolean;
 };
 type __VLS_PublicProps = __VLS_Props & {
     /** 1-based, as the platform pages. The shared table counts from zero. */
@@ -81,5 +89,6 @@ declare const _default: import("vue").DefineComponent<__VLS_PublicProps, {}, {},
     showWorkflow: boolean;
     serverMode: boolean;
     orderable: boolean;
+    narrowable: boolean;
 }, {}, {}, {}, string, import("vue").ComponentProvideOptions, false, {}, any>;
 export default _default;

package/dist/runtime/composables/useIntegrationsRuns.d.ts

@@ -1,5 +1,5 @@
 import type { RunStatus, RunStepRow, WorkflowRun } from '../types/integrations.js';
-import type { RunFeedQuery, RunSort, RunSortDirection } from '../utils/runFeedQuery.js';
+import type { RunFeedQuery, RunSort, RunSortDirection, WorkflowRunsNarrowing } from '../utils/runFeedQuery.js';
 export type { RunFeedQuery };
 interface RunSummary {
     total: number;
@@ -27,6 +27,8 @@ export declare function useIntegrationsRuns(): {
         perPage?: number;
         sort?: RunSort;
         direction?: RunSortDirection;
+        /** Narrowed by the platform, before it pages — so `total` counts what is left. */
+        narrowing?: WorkflowRunsNarrowing;
     }) => Promise<{
         runs: WorkflowRun[];
         total: number;
@@ -38,7 +40,7 @@ export declare function useIntegrationsRuns(): {
         status?: RunStatus;
     }) => WorkflowRun[];
     get: (id: string) => WorkflowRun | null;
-    fetchRun: (workflowId: number | string, runId: number | string, workflowName: string) => Promise<WorkflowRun>;
+    fetchRun: (workflowId: number | string, runId: number | string, workflowName: string, listed?: WorkflowRun) => Promise<WorkflowRun>;
     latestPerWorkflow: any;
     latestLoaded: any;
     latestLoading: any;

package/dist/runtime/composables/useIntegrationsRuns.js

@@ -4,6 +4,7 @@ import { useIntegrationsT } from "./useIntegrationsT.js";
 import { errorReason } from "../utils/apiError.js";
 import { bareRecord } from "../utils/apiEnvelope.js";
 import { RUN_OWN_FIELDS } from "../utils/runEnvelope.js";
+import { keepListTrigger, mapRun } from "../utils/runRecord.js";
 import { useLoadGate } from "./useLoadGate.js";
 import {
   DEFAULT_RUN_DIRECTION,
@@ -42,39 +43,6 @@ export function useIntegrationsRuns() {
   const SYNC_LIMIT = 5;
   const api = useIntegrationsApi();
   const t = useIntegrationsT();
-  function mapRun(r, workflowName = "") {
-    const startedMs = r.started_at ? new Date(r.started_at).getTime() : null;
-    const finishedMs = r.completed_at ? new Date(r.completed_at).getTime() : null;
-    return {
-      id: String(r.id),
-      workflowId: r.workflow_id,
-      workflowName: r.workflow_name ?? workflowName,
-      status: mapRunStatus(r.status),
-      // Kept beside the mapped word, because the mapping folds `terminated` into
-      // `failed` and the platform resumes only the literal `failed` (PO-355).
-      apiStatus: r.status,
-      temporalRunId: r.temporal_run_id ?? null,
-      triggerId: r.trigger_handle ?? "",
-      // The platform reports what started the run. This was hardcoded to
-      // `manual` because the payload carried nothing, which left the trigger
-      // column and its filter with one populated value in production.
-      // Null — the trigger was deleted, or the run predates trigger_handle, or the
-      // row came from a per-workflow read, which sends no trigger at all — is read
-      // as `manual` here although the platform said nothing (IN-9, api-contract
-      // runs.feed F6 and runs.execution F2).
-      triggerType: r.trigger_type ?? "manual",
-      startedAt: r.started_at ?? "",
-      finishedAt: r.completed_at ?? null,
-      // The list carries the reason a run failed; dropping it here was why every
-      // failure read the same wherever the list is shown. An empty string is not
-      // a reason — it has to fall through to whatever the surface says instead,
-      // or it renders as a blank line where the sentence belongs.
-      errorMessage: r.error || void 0,
-      // Derived from the timestamps — the list payload carries no duration.
-      durationMs: startedMs != null && finishedMs != null && finishedMs >= startedMs ? finishedMs - startedMs : null,
-      steps: []
-    };
-  }
   async function load(force = false) {
     if (!await gate.claim(force)) return runs.value;
     error.value = null;
@@ -103,7 +71,7 @@ export function useIntegrationsRuns() {
           const active = runs.value.filter((r) => r.status === "running" || r.status === "queued").sort((a, b) => Number(b.id) - Number(a.id)).slice(0, SYNC_LIMIT);
           await Promise.allSettled(active.map(async (r) => {
             try {
-              const fresh = await fetchRun(r.workflowId, r.id, r.workflowName);
+              const fresh = await fetchRun(r.workflowId, r.id, r.workflowName, r);
               runs.value = runs.value.map((x) => x.id === fresh.id ? fresh : x);
             } catch {
             }
@@ -125,22 +93,26 @@ export function useIntegrationsRuns() {
     const page = opts.page ?? 1;
     const perPage = opts.perPage ?? 25;
     const res = await api.get(
-      `/workflows/${workflowId}/runs?${workflowRunsQueryString(page, perPage, opts.sort, opts.direction)}`
+      `/workflows/${workflowId}/runs?${workflowRunsQueryString(page, perPage, opts.sort, opts.direction, opts.narrowing)}`
     );
     let list2 = (res?.data ?? []).map((r) => mapRun(r, workflowName));
     if (opts.reconcile) {
       const active = list2.filter((r) => r.status === "running" || r.status === "queued").sort((a, b) => Number(b.id) - Number(a.id)).slice(0, SYNC_LIMIT);
       await Promise.allSettled(active.map(async (r) => {
         try {
-          const fresh = await fetchRun(workflowId, r.id, workflowName);
+          const fresh = await fetchRun(workflowId, r.id, workflowName, r);
           list2 = list2.map((x) => x.id === fresh.id ? fresh : x);
         } catch {
         }
       }));
     }
+    const wanted = opts.narrowing?.status ?? [];
+    const before = list2.length;
+    if (wanted.length) list2 = list2.filter((r) => wanted.includes(r.status));
+    const total2 = res?.meta?.total ?? before;
     return {
       runs: list2,
-      total: res?.meta?.total ?? list2.length,
+      total: total2 - (before - list2.length),
       page: res?.meta?.current_page ?? page,
       lastPage: res?.meta?.last_page ?? 1
     };
@@ -202,9 +174,10 @@ export function useIntegrationsRuns() {
   const retryRun = (workflowId, runId) => runAction(workflowId, runId, "retry");
   const resumeRun = (workflowId, runId, reason) => runAction(workflowId, runId, "resume", reason);
   const getRun = (workflowId, runId) => api.get(`/workflows/${workflowId}/runs/${runId}`);
-  async function fetchRun(workflowId, runId, workflowName) {
+  async function fetchRun(workflowId, runId, workflowName, listed) {
     const apiRun = bareRecord(await getRun(workflowId, runId), RUN_OWN_FIELDS);
-    return mapRun(apiRun, workflowName);
+    const fresh = mapRun(apiRun, workflowName);
+    return listed ? keepListTrigger(fresh, listed) : fresh;
   }
   const getDetails = (workflowId, runId) => api.get(`/workflows/${workflowId}/runs/${runId}/details`);
   const getSteps = (workflowId, runId, includePayloads = false) => api.get(

package/dist/runtime/pages/integrations/workflows/[id].vue

@@ -209,6 +209,8 @@ watch(id, async () => {
   if (!w) await leaveTo("/integrations/workflows");
   draft.value = null;
   warningsExpanded.value = false;
+  runsStatus.value = [];
+  runsTrigger.value = "";
   if (tab.value === "runs") void loadRuns(1);
   settleBuildStatus();
 }, { immediate: true });
@@ -530,6 +532,8 @@ const runsLastPage = ref(1);
 const runsWanted = ref(1);
 const runsSort = ref(DEFAULT_RUN_SORT);
 const runsDirection = ref(DEFAULT_RUN_DIRECTION);
+const runsStatus = ref([]);
+const runsTrigger = ref("");
 let runsSeq = 0;
 let runsLoadingSeq = 0;
 const RUNS_POLL_MS = 3e3;
@@ -634,7 +638,8 @@ async function loadRuns(page = runsWanted.value, opts = {}) {
       page,
       perPage: RUNS_PER_PAGE,
       sort: runsSort.value,
-      direction: runsDirection.value
+      direction: runsDirection.value,
+      narrowing: { status: runsStatus.value, triggerType: runsTrigger.value }
     });
     if (seq !== runsSeq) return;
     wfRuns.value = res.runs;
@@ -655,6 +660,11 @@ function reorderRuns(order) {
   runsDirection.value = order.direction;
   void loadRuns(1);
 }
+function narrowRuns(patch) {
+  if (patch.status !== void 0) runsStatus.value = patch.status;
+  if (patch.triggerType !== void 0) runsTrigger.value = patch.triggerType;
+  void loadRuns(1);
+}
 function goRunsPage(p) {
   const next = Math.min(Math.max(1, p), runsLastPage.value);
   if (next !== runsWanted.value) void loadRuns(next);
@@ -1179,9 +1189,13 @@ async function confirmDeactivate() {
             orderable
             :sort="runsSort"
             :direction="runsDirection"
+            narrowable
+            :status="runsStatus"
+            :trigger-type="runsTrigger"
             :empty-text="t('integration.editor.runs.empty')"
             @changed="() => loadRuns()"
             @reorder="reorderRuns"
+            @narrow="narrowRuns"
           />
 
           <!-- Server pagination — the endpoint pages, so the whole history is reachable. -->

package/dist/runtime/types/integrations.d.ts

@@ -822,7 +822,12 @@ export interface WorkflowRun {
     /** Temporal run id for this execution; null when the row predates Temporal. */
     temporalRunId?: string | null;
     triggerId: string;
-    triggerType: TriggerType;
+    /**
+     * What started the run, as the platform reported it. Null where it reported
+     * nothing — the trigger was deleted, or the run predates it being recorded — and
+     * drawn as nothing rather than as a guess (PO-335).
+     */
+    triggerType: TriggerType | null;
     startedAt: string;
     finishedAt: string | null;
     durationMs: number | null;

package/dist/runtime/utils/runFeedQuery.d.ts

@@ -122,13 +122,16 @@ export interface RunFeedQuery {
  * empty `status` as a value and answers 422.
  */
 export declare function runFeedQueryString(query: RunFeedQuery): string;
+/** The narrowing a workflow's own list takes: the feed's, minus the text search. */
+export type WorkflowRunsNarrowing = Pick<RunFeedQuery, 'status' | 'triggerType'>;
 /**
- * The query string for a workflow's own `GET /workflows/{id}/runs`: a page, and
- * the order it is read in. That endpoint narrows by nothing yet (PO-319), so this
- * is the paging and the order alone — the order sent by the same rule as the
- * tenant feed's, so the two lists can never ask for one order two ways.
+ * The query string for a workflow's own `GET /workflows/{id}/runs`: a page, the
+ * order it is read in, and the narrowing by outcome and trigger kind (PO-319) —
+ * each sent by the same rule as the tenant feed's, so the two lists can never ask
+ * for one thing two ways. No text search: that endpoint takes none, and the tab's
+ * search box searches the page in hand.
  */
-export declare function workflowRunsQueryString(page: number, perPage: number, sort?: RunSort, direction?: RunSortDirection): string;
+export declare function workflowRunsQueryString(page: number, perPage: number, sort?: RunSort, direction?: RunSortDirection, narrowing?: WorkflowRunsNarrowing): string;
 /**
  * The narrowing alone, without the paging — what `GET /runs/summary` is read
  * with so the pager's total describes the same rows as the page beside it.

package/dist/runtime/utils/runFeedQuery.js

@@ -29,10 +29,11 @@ export function runFeedQueryString(query) {
   setOrder(params, query.sort, query.direction);
   return params.toString();
 }
-export function workflowRunsQueryString(page, perPage, sort = DEFAULT_RUN_SORT, direction = DEFAULT_RUN_DIRECTION) {
+export function workflowRunsQueryString(page, perPage, sort = DEFAULT_RUN_SORT, direction = DEFAULT_RUN_DIRECTION, narrowing = { status: [], triggerType: "" }) {
   const params = new URLSearchParams();
   params.set("page", String(page));
   params.set("per_page", String(perPage));
+  for (const [key, value] of outcomeParams(narrowing)) params.set(key, value);
   setOrder(params, sort, direction);
   return params.toString();
 }
@@ -45,11 +46,15 @@ function setOrder(params, sort, direction) {
 export function runFeedNarrowingString(query) {
   return narrowingParams(query).toString();
 }
-function narrowingParams(query) {
+function outcomeParams(narrowing) {
   const params = new URLSearchParams();
-  const statuses = apiStatusesFor(query.status);
+  const statuses = apiStatusesFor(narrowing.status);
   if (statuses.length) params.set("status", statuses.join(","));
-  if (query.triggerType) params.set("trigger_type", query.triggerType);
+  if (narrowing.triggerType) params.set("trigger_type", narrowing.triggerType);
+  return params;
+}
+function narrowingParams(query) {
+  const params = outcomeParams(query);
   const search = query.search.trim();
   if (search) params.set("search", search);
   return params;

package/dist/runtime/utils/runRecord.d.ts

@@ -0,0 +1,39 @@
+import type { TriggerType, WorkflowRun } from '../types/integrations.js';
+/**
+ * A run as either list sends it, and as the by-id read does. The three do not
+ * carry the same fields: see `mapRun` and `keepListTrigger` for what fills in.
+ */
+export interface ApiRun {
+    id: number;
+    workflow_id: number;
+    status: string;
+    temporal_workflow_id?: string;
+    temporal_run_id?: string;
+    started_at?: string;
+    completed_at?: string;
+    /** Why the run failed, as the platform recorded it. Present in the list too. */
+    error?: string | null;
+    /**
+     * Present on the tenant feed, absent on the per-workflow subresource — which
+     * has no need for them, since the caller supplied the workflow and the tab
+     * shows one workflow's runs.
+     */
+    workflow_name?: string;
+    trigger_type?: TriggerType | null;
+    trigger_handle?: string | null;
+}
+/**
+ * `workflowName` is a FALLBACK, not the source. The tenant feed carries the
+ * name on the row; the per-workflow subresource does not, and its caller knows
+ * it. Passing it used to be the only way a row had a name at all.
+ */
+export declare function mapRun(r: ApiRun, workflowName?: string): WorkflowRun;
+/**
+ * A run re-read by id, keeping what started it from the row it replaces.
+ *
+ * The by-id read sends no trigger — only the two lists do — so swapping a
+ * reconciled run in whole blanked a schedule's run to no trigger at all, a row
+ * after the list had named it (api-contract runs.execution F2). What started a
+ * run does not change while it runs, so the list's word stays the right one.
+ */
+export declare function keepListTrigger(fresh: WorkflowRun, listed: WorkflowRun): WorkflowRun;

package/dist/runtime/utils/runRecord.js

@@ -0,0 +1,39 @@
+import { mapRunStatus } from "./runFeedQuery.js";
+export function mapRun(r, workflowName = "") {
+  const startedMs = r.started_at ? new Date(r.started_at).getTime() : null;
+  const finishedMs = r.completed_at ? new Date(r.completed_at).getTime() : null;
+  return {
+    id: String(r.id),
+    workflowId: r.workflow_id,
+    workflowName: r.workflow_name ?? workflowName,
+    status: mapRunStatus(r.status),
+    // Kept beside the mapped word, because the mapping folds `terminated` into
+    // `failed` and the platform resumes only the literal `failed` (PO-355).
+    apiStatus: r.status,
+    temporalRunId: r.temporal_run_id ?? null,
+    triggerId: r.trigger_handle ?? "",
+    // The platform reports what started the run, on both lists. Null — the
+    // trigger was deleted, or the run predates trigger_handle — stays null: it
+    // was read as `manual` once, and every row of a workflow's own Runs tab then
+    // claimed a start by hand, because that list sent no trigger at all (PO-335).
+    // A by-id read still sends none; see `keepListTrigger`.
+    triggerType: r.trigger_type ?? null,
+    startedAt: r.started_at ?? "",
+    finishedAt: r.completed_at ?? null,
+    // The list carries the reason a run failed; dropping it here was why every
+    // failure read the same wherever the list is shown. An empty string is not
+    // a reason — it has to fall through to whatever the surface says instead,
+    // or it renders as a blank line where the sentence belongs.
+    errorMessage: r.error || void 0,
+    // Derived from the timestamps — the list payload carries no duration.
+    durationMs: startedMs != null && finishedMs != null && finishedMs >= startedMs ? finishedMs - startedMs : null,
+    steps: []
+  };
+}
+export function keepListTrigger(fresh, listed) {
+  return {
+    ...fresh,
+    triggerId: fresh.triggerId || listed.triggerId,
+    triggerType: fresh.triggerType ?? listed.triggerType
+  };
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@revenexx/studio-integrations",
-  "version": "3.3.0",
+  "version": "3.4.0",
   "type": "module",
   "description": "Integration Studio (core: workflows, node editor, runs, schedules, credentials, secrets, audit) as a standalone, publishable Nuxt module.",
   "repository": {
@@ -63,7 +63,7 @@
     "eslint": "^10.11.0",
     "nuxt": "^4.2.2",
     "typescript": "5.9.3",
-    "vitest": "^3.2.0",
+    "vitest": "^5.0.5",
     "vue": "^3.5.26",
     "vue-tsc": "^2.2.0"
   },