@revenexx/studio-integrations 1.2.0 → 1.3.0

package/CHANGELOG.md

@@ -1,5 +1,160 @@
 # @revenexx/studio-integrations
 
+## 1.3.0
+
+### Minor Changes
+
+- 593c0d5: A step can be changed onto another version of its node
+
+  The editor could say precisely that a step sat on a node version the platform
+  cannot run, and name the version that would work — and there it stopped. Getting
+  there meant deleting the step and inserting a fresh one, which throws away its
+  settings, its name, its note and its connections.
+
+  The version is now a control wherever there is somewhere to go, and two states
+  reach it. Where the pinned version cannot run, the red notice keeps every word it
+  had and gains the button it was missing. Where the step runs fine and a newer
+  version exists, one grey line in its dialog says so — **and nothing else does**.
+  No mark on the canvas card: nothing is wrong there, and a fourth mark for a
+  non-problem is what teaches an operator to stop reading marks. A node with a
+  single usable version says nothing at all.
+
+  Older versions are offered too, not only newer ones. A version that turns out to
+  behave badly otherwise has no way back.
+
+  **What the change costs is said before it happens.** The two versions are read
+  against the settings the step actually holds, and split three ways: what travels,
+  what the chosen version does not have and will be taken out, and what arrives
+  needing an eye — a setting it requires and the step has no answer for, or one
+  whose type or choices moved. A setting nobody filled in appears nowhere; there is
+  nothing to lose. A change that costs nothing says so in a sentence rather than
+  showing three empty lists, and where the version being left is no longer in the
+  catalogue the answer says the two could not be compared at all.
+
+  Values the chosen version does not declare are removed rather than left behind.
+  The service keeps such a key on save without complaint, so it would otherwise sit
+  in the workflow as invisible dead data.
+
+  Nothing about the step's identity moves: same id, same connections, same name,
+  same note, same place on the canvas. It lands as an unsaved change like editing
+  any setting, so discarding the editor's changes undoes it.
+
+  Promised as `specs/node-configuration.md` AC-24 to AC-26. `specs/node-palette.md`
+  AC-7 gains the clause that the version it names is somewhere the step can
+  actually go, and loses the gap entry that described the dead end.
+
+- bb8a80a: Declare what a workflow may remember between runs (PO-374).
+
+  A new **State** entry in the editor's floating control pane (Shift+T) opens a
+  dialog for the blob's `state` block: each namespace with its role — mapping,
+  cursor, dedupe or digest — and whether it belongs to this workflow alone or to
+  every workflow in the tenant. The engine mints a run's grant from exactly this
+  list, so until now a workflow could not reach the state store from the studio at
+  all.
+
+  The role is chosen when the namespace is added and stated as text afterwards:
+  entries written with one meaning cannot be read back with another. What sharing
+  costs — tenant-wide reads _and_ writes, and a namespace that outlives this
+  workflow — is on screen next to the control that chooses it.
+
+  Also fixes a silent loss in both directions of the workflow mapper, which
+  enumerates blob keys one by one: `state` was neither read from the API nor sent
+  back on save, so a declaration made anywhere else would have disappeared on the
+  next save from the studio.
+
+  A node setting of the new type `state-ref` picks one of the workflow's declared
+  namespaces **of that setting's kind**, and declares a new one from inside the
+  picker — the same shape a `credentials-ref` has, so the author never leaves the
+  node to type a name that nothing would have compared.
+
+  A namespace declared inside the picker waits in the node's draft until the
+  inspector keeps it — Save, or prev/next — so Esc discards the declaration
+  together with the setting that named it, and the workflow does not report
+  unsaved changes for something nobody chose to keep.
+
+  A node's test run now names the workflow being edited, which lets it read that
+  workflow's namespaces. Writes stay refused at author time.
+
+  Promised behaviour: `specs/state-namespaces.md`.
+
+- ddff18a: A setting that cannot apply is no longer put in front of the author
+
+  A node can now say in its manifest when one of its settings applies — another
+  setting's key, an operator, and the value to compare against — and the
+  configuration form draws it only while that holds. Until now the only thing
+  warning an author off a meaningless field was a sentence in small print under
+  it, which they read last, having already filled it in. Date Time's **Source**
+  is the plainest case: under "Now" it still asked which field to read the date
+  out of.
+
+  What changed on screen:
+
+  - A setting whose condition does not hold is not drawn, and is not counted
+    among the optional ones either — a fold that offers four settings and shows
+    three teaches the reader to distrust the number.
+  - What was typed into a setting that stops applying is dropped, the same
+    cascade the studio already runs for a setting whose options another one
+    resolves. A value nowhere on screen is a value nobody can correct.
+  - Settings a node resolves while the author edits are treated the same way.
+
+  The words a condition is written in are the fourteen the condition and filter
+  nodes already offer — `equals`, `isEmpty`, `contains`, `>` and the rest — so an
+  author who has met them in one place does not learn a second set for the other.
+  `specs/node-configuration.md` AC-27 and AC-28 carry the promise.
+
+  Nothing changes for a node that says nothing: every setting is drawn exactly as
+  before. A condition written on a setting _inside a group_ is not honoured here —
+  a group is drawn as one value editor rather than as its own settings — and that
+  is written into the spec's Gaps rather than left to be discovered.
+
+### Patch Changes
+
+- 4dd5b12: The answer that keeps unsaved work is legible, and is the one the question emphasises
+
+  Leaving a workflow with unsaved changes asks whether to discard them, and the
+  question offers two answers. Only one of them looked like an answer: "Keep editing"
+  was drawn in a colour its own background shares, so it read as an empty button beside
+  "Discard and leave" and its label appeared only once the pointer rested on it. Which
+  is the wrong half of the dialog to hide — that button is the one the keyboard lands
+  on, so the answer the question exists to make easy was the answer nobody could see.
+
+  It is now drawn as the filled answer, the way every other dialog in the module draws
+  the answer on the right, so the question also says which answer it falls back to. The
+  cause was two button variants on one element: the dialog's cancel element applies the
+  outline variant itself, and the button inside it was left on the default one, so the
+  merge took the background from the first and the text colour from the second. Nothing
+  else about the question moves — the wording, the keyboard default and what each answer
+  does are unchanged.
+
+  Promised as `specs/workflow-save.md` AC-13 and AC-14: the label is checked as a
+  contrast ratio against the button it sits on, unhovered, and the emphasis as a
+  difference between the two answers' surfaces rather than against a class name or a
+  colour value.
+
+- ab2eb96: A build state stops saying "still building" wherever it is shown
+
+  Saving a workflow queues the platform to build an executable bundle from it, and
+  until that build lands the workflow cannot run. Two surfaces say so: the chip beside
+  Save in the editor, and the build column in the workflows list. Only the editor's
+  ever became true on its own — the list said whatever its last read said, so a
+  workflow that finished building while the list stood open kept reading "Building…"
+  until something reloaded the page. Save in the editor, come back to the list, and the
+  row never moved.
+
+  The list now watches a build out the way the editor does: one read of the tenant per
+  turn, so thirty workflows building at once cost what one costs, and it stops as soon
+  as nothing is waiting. A platform build report still makes it quicker where there is
+  one; nothing depends on one arriving any more, on either surface.
+
+  The two waits now share one interval and one ceiling instead of the editor owning
+  both, so neither surface can drift into asking twice as often as the other.
+
+  Promised as `specs/workflow-build.md`, a new spec for the subject: what becomes of a
+  saved workflow before it can run. AC-1 is the editor's promise, moved out of
+  `specs/workflow-save.md`, where a reader asking what a build state means would not
+  have looked; AC-2 is the list's. `specs/realtime-updates.md` AC-7 keeps the
+  announcement half and now says which of the two makes a build state true.
+
 ## 1.2.0
 
 ### Minor Changes

package/dist/module.json

@@ -1,7 +1,7 @@
 {
   "name": "@revenexx/studio-integrations",
   "configKey": "studioIntegrations",
-  "version": "1.2.0",
+  "version": "1.3.0",
   "builder": {
     "@nuxt/module-builder": "1.0.2",
     "unbuild": "unknown"

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

@@ -3,6 +3,7 @@ import { computed } from "vue";
 import { Button } from "@revenexx/studio";
 import { IntegrationsApiError } from "../../utils/apiError";
 import { resolveLocalized } from "../../utils/localize";
+import { applyingFields, staleKeys } from "../../utils/settingCondition";
 const props = defineProps({
   slug: { type: String, required: true },
   version: { type: String, required: true },
@@ -24,6 +25,13 @@ const q = useNodeConfigResolve({
   dependsOn: () => props.field.dependsOn
 });
 const resolvedFields = computed(() => q.data.value?.fields ?? []);
+const applyingResolvedFields = computed(() => applyingFields(resolvedFields.value, props.config));
+function writeResolved(key, value) {
+  emit("set", key, value);
+  const after = { ...props.config, [key]: value };
+  for (const stale of staleKeys(resolvedFields.value, key, after))
+    emit("set", stale, void 0);
+}
 </script>
 
 <template>
@@ -67,7 +75,7 @@ const resolvedFields = computed(() => q.data.value?.fields ?? []);
       <legend class="px-1 text-mono text-muted-foreground">
         {{ resolveLocalized(field.label) }}
       </legend>
-      <template v-for="f in resolvedFields" :key="f.key">
+      <template v-for="f in applyingResolvedFields" :key="f.key">
         <!-- collision guard: a resolved key that shadows a static config field is skipped -->
         <p v-if="staticKeys.includes(f.key)" class="text-sm text-amber-600">
           {{ t("integration.nodeConfig.dynamicSchema.fieldCollision", { key: f.key }) }}
@@ -89,7 +97,7 @@ const resolvedFields = computed(() => q.data.value?.fields ?? []);
           :static-keys="staticKeys"
           :sibling-fields="resolvedFields"
           :depth="(depth ?? 0) + 1"
-          @set="(k, v) => emit('set', k, v)"
+          @set="writeResolved"
         />
       </template>
 

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

@@ -1,4 +1,4 @@
-import type { ConfigField, WorkflowVariable } from '../../types/integrations.js';
+import type { ConfigField, WorkflowStateNamespace, WorkflowStateRole, WorkflowStateVisibility, WorkflowVariable } from '../../types/integrations.js';
 type __VLS_Props = {
     field: ConfigField;
     config: Record<string, unknown>;
@@ -11,6 +11,8 @@ type __VLS_Props = {
     siblingFields?: ConfigField[];
     depth?: number;
     variables?: Record<string, WorkflowVariable>;
+    /** The workflow's declared state namespaces — the options a `state-ref` offers. */
+    stateNamespaces?: WorkflowStateNamespace[];
     upstreamOutputs?: Array<{
         nodeId: string;
         nodeName?: string;
@@ -22,7 +24,9 @@ type __VLS_Props = {
 };
 declare const _default: import("vue").DefineComponent<__VLS_Props, {}, {}, {}, {}, import("vue").ComponentOptionsMixin, import("vue").ComponentOptionsMixin, {
     set: (key: string, value: unknown) => any;
+    "declare-state": (namespace: string, role: WorkflowStateRole, visibility: WorkflowStateVisibility) => any;
 }, string, import("vue").PublicProps, Readonly<__VLS_Props> & Readonly<{
     onSet?: ((key: string, value: unknown) => any) | undefined;
+    "onDeclare-state"?: ((namespace: string, role: WorkflowStateRole, visibility: WorkflowStateVisibility) => any) | undefined;
 }>, {}, {}, {}, {}, string, import("vue").ComponentProvideOptions, false, {}, any>;
 export default _default;

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

@@ -26,10 +26,11 @@ const props = defineProps({
   siblingFields: { type: Array, required: false },
   depth: { type: Number, required: false },
   variables: { type: Object, required: false },
+  stateNamespaces: { type: Array, required: false },
   upstreamOutputs: { type: Array, required: false },
   refused: { type: String, required: false }
 });
-const emit = defineEmits(["set"]);
+const emit = defineEmits(["set", "declare-state"]);
 const t = useIntegrationsT();
 const value = computed(() => props.config[props.field.key]);
 function write(v) {
@@ -127,6 +128,30 @@ function onCredentialSelect(v) {
   }
   write(v);
 }
+const stateOptions = computed(() => {
+  const role = props.field.stateRole;
+  const declared = props.stateNamespaces ?? [];
+  const list = role ? declared.filter((n) => n.role === role) : declared;
+  const selected = value.value;
+  if (typeof selected === "string" && selected && !list.some((n) => n.namespace === selected)) {
+    return [{ namespace: selected, role: role ?? "mapping", visibility: void 0 }, ...list];
+  }
+  return list;
+});
+const stateTakenNames = computed(() => (props.stateNamespaces ?? []).map((n) => n.namespace));
+const stateCreateOpen = ref(false);
+const STATE_ACTION_NEW = "__action_new_state_namespace__";
+function onStateSelect(v) {
+  if (v === STATE_ACTION_NEW) {
+    stateCreateOpen.value = true;
+    return;
+  }
+  write(v);
+}
+function onStateDeclared(namespace, role, visibility) {
+  emit("declare-state", namespace, role, visibility);
+  write(namespace);
+}
 function acceptsSftp(field) {
   const t2 = field.credentialType;
   return t2 === "revenexx:sftp" || Array.isArray(t2) && t2.includes("revenexx:sftp");
@@ -619,6 +644,45 @@ function setArrayItemField(index, fieldKey, v) {
       />
     </template>
 
+    <!-- state-ref — pick one of the workflow's declared namespaces of this
+         field's role. Declaring lives INSIDE the picker, like creating a
+         credential does: a picker whose list is empty must not be a dead end
+         that sends the author to another dialog and back. -->
+    <template v-else-if="field.type === 'state-ref'">
+      <Select
+        :model-value="value ?? ''"
+        @update:model-value="(v) => onStateSelect(v)"
+      >
+        <SelectTrigger :aria-label="resolveLocalized(field.label)" size="sm" class="h-8 w-full rounded-md border border-black/[0.08] ring-0 shadow-none text-sm bg-white">
+          <SelectValue :placeholder="t('integration.nodeConfig.state.placeholder')" />
+        </SelectTrigger>
+        <SelectContent>
+          <SelectItem v-for="o in stateOptions" :key="o.namespace" :value="o.namespace">
+            <span class="flex items-center gap-1.5">
+              <span class="min-w-0 truncate font-mono">{{ o.namespace }}</span>
+              <span
+                v-if="o.visibility === 'shared'"
+                class="shrink-0 rounded-full border px-1.5 py-px text-[10px] font-medium text-muted-foreground"
+              >{{ t("integration.workflows.state.visibility.shared.label") }}</span>
+            </span>
+          </SelectItem>
+          <div v-if="!stateOptions.length" class="px-2 py-1.5 text-sm text-muted-foreground">
+            {{ t("integration.nodeConfig.state.empty") }}
+          </div>
+          <SelectSeparator />
+          <SelectItem :value="STATE_ACTION_NEW" class="font-medium text-violet-600 data-[highlighted]:text-violet-700">
+            {{ t("integration.nodeConfig.state.action.new") }}
+          </SelectItem>
+        </SelectContent>
+      </Select>
+      <IntegrationsStateNamespaceCreateDialog
+        v-model:open="stateCreateOpen"
+        :role="field.stateRole ?? 'mapping'"
+        :taken="stateTakenNames"
+        @declared="onStateDeclared"
+      />
+    </template>
+
     <!-- array of objects (e.g. headers / mappings) -->
     <div v-else-if="field.type === 'array' && field.items?.type === 'object'" class="space-y-2">
       <div

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

@@ -1,4 +1,4 @@
-import type { ConfigField, WorkflowVariable } from '../../types/integrations.js';
+import type { ConfigField, WorkflowStateNamespace, WorkflowStateRole, WorkflowStateVisibility, WorkflowVariable } from '../../types/integrations.js';
 type __VLS_Props = {
     field: ConfigField;
     config: Record<string, unknown>;
@@ -11,6 +11,8 @@ type __VLS_Props = {
     siblingFields?: ConfigField[];
     depth?: number;
     variables?: Record<string, WorkflowVariable>;
+    /** The workflow's declared state namespaces — the options a `state-ref` offers. */
+    stateNamespaces?: WorkflowStateNamespace[];
     upstreamOutputs?: Array<{
         nodeId: string;
         nodeName?: string;
@@ -22,7 +24,9 @@ type __VLS_Props = {
 };
 declare const _default: import("vue").DefineComponent<__VLS_Props, {}, {}, {}, {}, import("vue").ComponentOptionsMixin, import("vue").ComponentOptionsMixin, {
     set: (key: string, value: unknown) => any;
+    "declare-state": (namespace: string, role: WorkflowStateRole, visibility: WorkflowStateVisibility) => any;
 }, string, import("vue").PublicProps, Readonly<__VLS_Props> & Readonly<{
     onSet?: ((key: string, value: unknown) => any) | undefined;
+    "onDeclare-state"?: ((namespace: string, role: WorkflowStateRole, visibility: WorkflowStateVisibility) => any) | undefined;
 }>, {}, {}, {}, {}, string, import("vue").ComponentProvideOptions, false, {}, any>;
 export default _default;

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

@@ -1,9 +1,11 @@
-import type { NodeManifestOutput, WorkflowEdge, WorkflowNode, WorkflowTrigger, WorkflowVariable } from '../../types/integrations.js';
+import type { NodeManifestOutput, WorkflowEdge, WorkflowNode, WorkflowStateNamespace, WorkflowStateRole, WorkflowStateVisibility, WorkflowTrigger, WorkflowVariable } from '../../types/integrations.js';
 type __VLS_Props = {
     /** Either a workflow node OR a trigger. Distinguished by presence of `nodeSlug`. */
     selection: WorkflowNode | WorkflowTrigger | null;
     secretKeys: string[];
     variables?: Record<string, WorkflowVariable>;
+    /** The workflow's declared state namespaces — options for `state-ref` fields. */
+    stateNamespaces?: WorkflowStateNamespace[];
     upstreamOutputs?: Array<{
         nodeId: string;
         nodeName?: string;
@@ -31,11 +33,13 @@ declare const _default: import("vue").DefineComponent<__VLS_Props, {}, {}, {}, {
     update: (next: WorkflowTrigger | WorkflowNode) => any;
     remove: () => any;
     close: () => any;
+    "declare-state": (namespace: string, role: WorkflowStateRole, visibility: WorkflowStateVisibility) => any;
     navigate: (id: string) => any;
 }, string, import("vue").PublicProps, Readonly<__VLS_Props> & Readonly<{
     onUpdate?: ((next: WorkflowTrigger | WorkflowNode) => any) | undefined;
     onRemove?: (() => any) | undefined;
     onClose?: (() => any) | undefined;
+    "onDeclare-state"?: ((namespace: string, role: WorkflowStateRole, visibility: WorkflowStateVisibility) => any) | undefined;
     onNavigate?: ((id: string) => any) | undefined;
 }>, {}, {}, {}, {}, string, import("vue").ComponentProvideOptions, false, {}, any>;
 export default _default;

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

@@ -28,7 +28,8 @@ import {
   ToggleGroupItem
 } from "@revenexx/studio";
 import EventTypeCombobox from "./EventTypeCombobox.vue";
-import { findManifest, nodeCategoryLabel, isVersionExecutable, newestExecutableVersion } from "../../data/integrationNodeCatalog";
+import { findManifest, nodeCategoryLabel, isVersionExecutable, newestExecutableVersion, executableVersionsOf, newerExecutableVersions } from "../../data/integrationNodeCatalog";
+import NodeVersionMoveDialog from "./NodeVersionMoveDialog.vue";
 import {
   collectUpstreamNodes,
   inputPortsForManifest,
@@ -39,6 +40,7 @@ import {
 } from "../../data/nodeInputData";
 import { portDotClass } from "../../data/portTone";
 import { computeLoopBodies, manifestIterates } from "../../data/loopBody";
+import { applyingFields, staleKeys } from "../../utils/settingCondition";
 import { deepClone } from "../../utils/deepClone";
 import { copyToClipboard } from "../../utils/clipboard";
 import { nodeDisplayLabel } from "../../utils/validationFindings";
@@ -57,6 +59,7 @@ const props = defineProps({
   selection: { type: [Object, null], required: true },
   secretKeys: { type: Array, required: true },
   variables: { type: Object, required: false },
+  stateNamespaces: { type: Array, required: false },
   upstreamOutputs: { type: Array, required: false },
   autofocusName: { type: Boolean, required: false },
   nodes: { type: Array, required: false },
@@ -66,17 +69,35 @@ const props = defineProps({
   workflowId: { type: [Number, String], required: false },
   refusedFields: { type: Object, required: false }
 });
-const emit = defineEmits(["update", "close", "remove", "navigate"]);
+const emit = defineEmits(["update", "close", "remove", "navigate", "declare-state"]);
 const draft = ref(deepClone(props.selection));
 watch(() => props.selection?.id, () => {
-  if (props.selection && props.selection.id !== draft.value.id) draft.value = deepClone(props.selection);
+  if (props.selection && props.selection.id !== draft.value.id) {
+    draft.value = deepClone(props.selection);
+    pendingState.value = [];
+  }
 });
 const isNode = computed(() => "nodeSlug" in draft.value);
 const draftNode = computed(() => isNode.value ? draft.value : null);
 const draftTrigger = computed(() => isNode.value ? null : draft.value);
-const dirty = computed(() => JSON.stringify(draft.value) !== JSON.stringify(props.selection));
+const pendingState = ref([]);
+const offeredStateNamespaces = computed(() => [
+  ...props.stateNamespaces ?? [],
+  ...pendingState.value
+]);
+function stageStateNamespace(namespace, role, visibility) {
+  if (offeredStateNamespaces.value.some((n) => n.namespace === namespace)) return;
+  pendingState.value = [...pendingState.value, { namespace, role, visibility }];
+}
+const draftChanged = computed(() => JSON.stringify(draft.value) !== JSON.stringify(props.selection));
+const dirty = computed(() => draftChanged.value || pendingState.value.length > 0);
+function commit() {
+  for (const ns of pendingState.value) emit("declare-state", ns.namespace, ns.role, ns.visibility);
+  pendingState.value = [];
+  if (draftChanged.value) emit("update", deepClone(draft.value));
+}
 function save() {
-  if (dirty.value) emit("update", deepClone(draft.value));
+  commit();
   emit("close");
 }
 const nameInput = ref(null);
@@ -99,6 +120,21 @@ const readyVersion = computed(() => {
   const wn = draftNode.value;
   return wn ? newestExecutableVersion(wn.nodeSlug) : null;
 });
+const versionMoveOpen = ref(false);
+const newerVersions = computed(() => {
+  const wn = draftNode.value;
+  return wn ? newerExecutableVersions(wn.nodeSlug, wn.nodeVersion) : [];
+});
+const canMoveVersion = computed(() => {
+  const wn = draftNode.value;
+  if (!wn) return false;
+  const runnable = executableVersionsOf(wn.nodeSlug);
+  if (!runnable.length) return false;
+  return runnable.length > 1 || runnable[0]?.version !== wn.nodeVersion;
+});
+function applyVersionMove(payload) {
+  patchNode({ nodeVersion: payload.version, config: payload.config });
+}
 const TRIGGER_LABELS = computed(() => ({
   manual: t("integration.nodeInspector.trigger.type.manual"),
   schedule: t("integration.nodeInspector.trigger.type.schedule"),
@@ -150,8 +186,11 @@ watch(showSettingsTab, (hasThem) => {
   if (!hasThem) tab.value = "parameters";
 });
 const SETTINGS_FIELD_KEY = /(^|_)(timeout|retry|retries|backoff)|(timeout|retry|retries|backoff)(ms|sec|seconds|attempts)?$/i;
-const parameterFields = computed(() => (manifest.value?.config ?? []).filter((f) => !SETTINGS_FIELD_KEY.test(f.key)));
-const settingsFields = computed(() => (manifest.value?.config ?? []).filter((f) => SETTINGS_FIELD_KEY.test(f.key)));
+const applyingConfig = computed(
+  () => applyingFields(manifest.value?.config ?? [], draftNode.value?.config ?? {})
+);
+const parameterFields = computed(() => applyingConfig.value.filter((f) => !SETTINGS_FIELD_KEY.test(f.key)));
+const settingsFields = computed(() => applyingConfig.value.filter((f) => SETTINGS_FIELD_KEY.test(f.key)));
 const requiredFields = computed(() => parameterFields.value.filter((f) => f.required));
 const optionalFields = computed(() => parameterFields.value.filter((f) => !f.required));
 function optionalShouldOpen() {
@@ -162,27 +201,14 @@ const optionalOpen = ref(optionalShouldOpen());
 watch([() => props.selection?.id, manifest], () => {
   optionalOpen.value = optionalShouldOpen();
 });
-function dependentKeys(changedKey) {
-  const fields = manifest.value?.config ?? [];
-  const out = /* @__PURE__ */ new Set();
-  const queue = [changedKey];
-  while (queue.length > 0) {
-    const dep = queue.shift();
-    for (const f of fields) {
-      if ((f.dependsOn ?? []).includes(dep) && !out.has(f.key)) {
-        out.add(f.key);
-        queue.push(f.key);
-      }
-    }
-  }
-  out.delete(changedKey);
-  return out;
+function dependentKeys(changedKey, after) {
+  return staleKeys(manifest.value?.config ?? [], changedKey, after);
 }
 function writeConfig(key, value) {
   const wn = draftNode.value;
   if (!wn) return;
   const next = { ...wn.config, [key]: value };
-  const cleared = dependentKeys(key);
+  const cleared = dependentKeys(key, next);
   for (const k of cleared) delete next[k];
   const clearedDynamicSchema = [...cleared].some(
     (k) => (manifest.value?.config ?? []).some((f) => f.key === k && f.type === "dynamic-schema")
@@ -760,7 +786,7 @@ const hasNext = computed(() => seqIndex.value >= 0 && seqIndex.value < sequence.
 function navigate(offset) {
   const target = sequence.value[seqIndex.value + offset];
   if (!target) return;
-  if (dirty.value) emit("update", deepClone(draft.value));
+  commit();
   emit("navigate", target);
 }
 const outputView = ref("schema");
@@ -866,7 +892,16 @@ async function runTestStep() {
   try {
     const res = await api.post(
       `/nodes/${encodeURIComponent(wn.nodeSlug)}/${encodeURIComponent(wn.nodeVersion)}/execute:test`,
-      { config: wn.config ?? {}, inputs }
+      {
+        config: wn.config ?? {},
+        inputs,
+        // Names the workflow being edited, which is what lets the test run READ
+        // the state namespaces this workflow declares (PO-374). Writes stay
+        // refused: a correlation created by a test click would be
+        // indistinguishable from one a real run made. Omitted for an unsaved
+        // workflow, which has nothing declared yet anyway.
+        ...props.workflowId ? { workflow_id: Number(props.workflowId) } : {}
+      }
     );
     testLogs.value = res.logs ?? [];
     testOutputs.value = {
@@ -959,6 +994,26 @@ async function runTestStep() {
             </PopoverContent>
           </Popover>
         </div>
+        <!-- The version, once it is something the author can change (PO-339).
+             Deliberately a line of its own and deliberately grey: where a newer
+             version merely exists, nothing is wrong, and a coloured mark for a
+             non-problem is what teaches people to stop reading marks. It is
+             absent altogether for a node with one runnable version — there is no
+             choice to offer — and for a node whose pinned version cannot run,
+             where the notice below the tabs carries the same route in the louder
+             tone that state has earned. -->
+        <div v-if="isNode && canMoveVersion && versionExecutable" class="flex items-center gap-1 text-xs text-muted-foreground">
+          <span v-if="newerVersions.length" class="tabular-nums">
+            {{ t("integration.nodeInspector.version.available", { version: newerVersions[0]?.version ?? "" }) }}
+          </span>
+          <button
+            type="button"
+            class="rounded-sm underline decoration-dotted underline-offset-2 hover:text-foreground focus:outline-none focus-visible:ring-2 focus-visible:ring-primary/40"
+            @click="versionMoveOpen = true"
+          >
+            {{ t("integration.nodeInspector.version.change") }}
+          </button>
+        </div>
       </div>
       <span class="shrink-0 rounded bg-slate-100 px-1.5 py-0.5 text-[10px] font-medium uppercase tracking-wider text-slate-600">
         {{ groupLabel }}
@@ -1298,6 +1353,18 @@ async function runTestStep() {
           class="mx-4 mb-1 shrink-0 rounded-md border border-rose-200 bg-rose-50 px-2.5 py-1.5 text-sm text-rose-700"
         >
           {{ readyVersion ? t("integration.nodeInspector.notice.versionNotReady.upgrade", { version: draftNode?.nodeVersion ?? "", ready: readyVersion }) : t("integration.nodeInspector.notice.versionNotReady.none", { version: draftNode?.nodeVersion ?? "" }) }}
+          <!-- The sentence named a version and led nowhere (PO-339). It leads
+               here now; where the node has no runnable version at all there is
+               still nothing to offer, and the notice stays the statement it
+               was. -->
+          <button
+            v-if="readyVersion && canMoveVersion"
+            type="button"
+            class="ml-1 rounded-sm font-medium underline underline-offset-2 hover:text-rose-900 focus:outline-none focus-visible:ring-2 focus-visible:ring-rose-400"
+            @click="versionMoveOpen = true"
+          >
+            {{ t("integration.nodeInspector.version.moveTo", { version: readyVersion }) }}
+          </button>
         </p>
 
         <div
@@ -1750,6 +1817,8 @@ async function runTestStep() {
                 :sibling-fields="manifest.config ?? []"
                 :depth="0"
                 :variables="variables"
+                :state-namespaces="offeredStateNamespaces"
+                @declare-state="stageStateNamespace"
                 :upstream-outputs="upstreamOutputs"
                 :refused="refusedFields?.[field.key]"
                 @set="(k, v) => writeConfig(k, v)"
@@ -1790,6 +1859,8 @@ async function runTestStep() {
                     :sibling-fields="manifest.config ?? []"
                     :depth="0"
                     :variables="variables"
+                    :state-namespaces="offeredStateNamespaces"
+                    @declare-state="stageStateNamespace"
                     :upstream-outputs="upstreamOutputs"
                     :refused="refusedFields?.[field.key]"
                     @set="(k, v) => writeConfig(k, v)"
@@ -1832,6 +1903,8 @@ async function runTestStep() {
                 :sibling-fields="manifest?.config ?? []"
                 :depth="0"
                 :variables="variables"
+                :state-namespaces="offeredStateNamespaces"
+                @declare-state="stageStateNamespace"
                 :upstream-outputs="upstreamOutputs"
                 :refused="refusedFields?.[field.key]"
                 @set="(k, v) => writeConfig(k, v)"
@@ -2121,6 +2194,11 @@ async function runTestStep() {
         </Button>
       </div>
     </div>
+
+    <!-- Asked outside the panes so it is not clipped by either, and mounted
+         unconditionally: the two places that open it sit in different parts of
+         the dialog and neither owns it. -->
+    <NodeVersionMoveDialog v-model:open="versionMoveOpen" :node="draftNode" @move="applyVersionMove" />
   </div>
 </template>
 

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

@@ -1,9 +1,11 @@
-import type { NodeManifestOutput, WorkflowEdge, WorkflowNode, WorkflowTrigger, WorkflowVariable } from '../../types/integrations.js';
+import type { NodeManifestOutput, WorkflowEdge, WorkflowNode, WorkflowStateNamespace, WorkflowStateRole, WorkflowStateVisibility, WorkflowTrigger, WorkflowVariable } from '../../types/integrations.js';
 type __VLS_Props = {
     /** Either a workflow node OR a trigger. Distinguished by presence of `nodeSlug`. */
     selection: WorkflowNode | WorkflowTrigger | null;
     secretKeys: string[];
     variables?: Record<string, WorkflowVariable>;
+    /** The workflow's declared state namespaces — options for `state-ref` fields. */
+    stateNamespaces?: WorkflowStateNamespace[];
     upstreamOutputs?: Array<{
         nodeId: string;
         nodeName?: string;
@@ -31,11 +33,13 @@ declare const _default: import("vue").DefineComponent<__VLS_Props, {}, {}, {}, {
     update: (next: WorkflowTrigger | WorkflowNode) => any;
     remove: () => any;
     close: () => any;
+    "declare-state": (namespace: string, role: WorkflowStateRole, visibility: WorkflowStateVisibility) => any;
     navigate: (id: string) => any;
 }, string, import("vue").PublicProps, Readonly<__VLS_Props> & Readonly<{
     onUpdate?: ((next: WorkflowTrigger | WorkflowNode) => any) | undefined;
     onRemove?: (() => any) | undefined;
     onClose?: (() => any) | undefined;
+    "onDeclare-state"?: ((namespace: string, role: WorkflowStateRole, visibility: WorkflowStateVisibility) => any) | undefined;
     onNavigate?: ((id: string) => any) | undefined;
 }>, {}, {}, {}, {}, string, import("vue").ComponentProvideOptions, false, {}, any>;
 export default _default;