@linxin666/dsh-client-ui-task-board 0.4.4 → 0.4.6

package/src/core/tasks.ts

@@ -5,7 +5,9 @@
  * unit-testable in isolation.
  */
 import type { FreezeSnapshot } from './freeze-snapshot.ts'
+import type { ExecutionVerification } from './verification.ts'
 import type { TaskHandover, TaskHandoverInput } from './handover.ts'
+import { normalizeTaskIntegrations, type TaskBoardExtensionPayload } from './extension.ts'
 
 /** Task lifecycle status, one per kanban column. */
 export type TaskStatus = 'backlog' | 'todo' | 'running' | 'done' | 'failed'
@@ -35,8 +37,23 @@ export interface ExecutionRecord {
    * Session id of the DSH session that issued the run/rerun action (issue #6
    * audit origin). Client-asserted, not a trust boundary; absent when the run
    * was triggered by cron (source unknown).
+   *
+   * On an EXTERNAL record (issue #1826) this names the agent that completed the
+   * work outside the Host instead, which is what makes the history readable.
    */
   initiatedBy?: string
+  /**
+   * True when this outcome was recorded from outside the Host by
+   * `record-external-outcome` rather than observed from a DSH session
+   * (issue #1826).
+   *
+   * The board never manufactures one of these itself: every Host-side path
+   * (`run`, `settle`, cascade folding, restart recovery) writes an execution
+   * without it, so `external` is the single field separating work this Host ran
+   * from work an outside agent reported. Such a record never carries a
+   * `sessionId` and never claims a verdict the Host observed.
+   */
+  external?: boolean
   /** Freeze instant captured from the card snapshot when the run opened. */
   frozenAt?: number
   /** Freeze source session captured from the card snapshot when the run opened. */
@@ -55,6 +72,14 @@ export interface ExecutionRecord {
   ownResult?: ExecutionOutcome
   /** Human failure text that arrived with {@link ownResult}. */
   ownError?: string
+  /**
+   * Goal acceptance state of this execution (task-board goal verification).
+   * Fixed when the execution starts and never rewritten from later settings:
+   * the contract is frozen here, the attempts accrue here, and the settlement
+   * rule reads the pass record here. Absent on an execution that started
+   * before the feature existed, which keeps it on the historical verdict.
+   */
+  verification?: ExecutionVerification
 }
 
 /**
@@ -78,17 +103,74 @@ export function retainRecentExecutions(executions: readonly ExecutionRecord[]):
   return [...settled.slice(Math.max(settled.length - keepSettled, 0)), ...open]
 }
 
+/**
+ * How a rule produces its triggers: a recurring cron rule, or one single
+ * planned instant. The distinction is stored rather than inferred so a
+ * one-shot is never represented as a date-restricted cron expression.
+ */
+export type ScheduleMode = 'cron' | 'once'
+
+/**
+ * Why a due occurrence produced no run, or why a rule stopped on its own.
+ *
+ * These are stable codes, not display text: the board renders them through its
+ * locale dictionary, so the ledger never carries a translated string.
+ */
+export type ScheduleStopReason =
+  /** The single planned instant produced its run. */
+  | 'fired'
+  /** The recurring rule reached its run budget. */
+  | 'limit'
+  /** The plan has no reachable instant any more (an impossible cron). */
+  | 'no-target'
+  /** The instant passed while the board was not running through it. */
+  | 'missed'
+  /** The task already had an open execution, so this occurrence was skipped. */
+  | 'busy'
+  /** An above-default permission was still unconfirmed, so this occurrence was skipped. */
+  | 'permission'
+  /**
+   * A scheduled launch failed before any session existed, so the occurrence
+   * created no execution and its run was refunded rather than spent.
+   */
+  | 'launch-failed'
+
+/** Every stop reason this build understands; the repair and wire gates share it. */
+export const SCHEDULE_STOP_REASONS: readonly ScheduleStopReason[] = [
+  'fired', 'limit', 'no-target', 'missed', 'busy', 'permission', 'launch-failed',
+]
+
 /**
  * A scheduled-run rule attached to a task. The Host scheduler triggers the
  * task when `nextRunAt` is due and persists the rule in the Host ledger.
+ *
+ * A rule is either `cron` (the historical shape: a 5-field expression with an
+ * explicit IANA zone, optionally capped by `maxRuns`) or `once` (one planned
+ * instant, `at`, with an implicit budget of one run). Both modes count the
+ * executions the scheduler actually started in `runCount`; a rule that reaches
+ * its budget stops itself in the ledger (`enabled: false` + `endedReason`)
+ * instead of relying on the execution agent to disable it.
  */
 export interface ScheduleRule {
   /** Whether the schedule is armed. */
   enabled: boolean
-  /** 5-field cron expression: `分 时 日 月 周`. */
-  cron: string
   /**
-   * IANA zone the cron wall clock is read in (issue #1722). Absent means the
+   * Which kind of plan this rule carries. A rule loaded from a ledger written
+   * before the field existed normalizes to `cron` (its `cron` is the plan).
+   */
+  mode: ScheduleMode
+  /**
+   * 5-field cron expression: `分 时 日 月 周`. Present on a `cron` rule and
+   * absent on a `once` rule.
+   */
+  cron?: string
+  /**
+   * The instant a `once` rule executes (ms epoch). Kept after the run so the
+   * detail view can still show the planned time once the rule has ended.
+   */
+  at?: number
+  /**
+   * IANA zone the wall clock is read in (issue #1722). Absent means the
    * Host process zone, which is what a rule written before zones were
    * persisted keeps using; the Host stamps its own zone at ledger migration so
    * a later `TZ` change cannot silently move an existing rule.
@@ -98,6 +180,62 @@ export interface ScheduleRule {
   nextRunAt: number | undefined
   /** Instant of the latest scheduled trigger (ms epoch). */
   lastTriggeredAt: number | undefined
+  /**
+   * Scheduled executions the Host has created under this rule. This is the
+   * run budget's counter: it counts executions that were successfully opened
+   * (whatever their later outcome), never completions, and a manual run never
+   * touches it. A skipped occurrence (busy / permission / missed / no target)
+   * creates no execution and therefore consumes nothing.
+   */
+  runCount: number
+  /**
+   * Maximum scheduled executions for a `cron` rule; absent means unlimited
+   * (the historical behavior). A `once` rule has an implicit budget of one and
+   * stores no `maxRuns`.
+   */
+  maxRuns?: number
+  /** Instant the rule stopped itself (ms epoch); absent while it may still fire. */
+  endedAt?: number
+  /** Why the rule stopped itself; present with `endedAt`. */
+  endedReason?: ScheduleStopReason
+  /** Instant of the latest occurrence that produced no run. */
+  skippedAt?: number
+  /** Why that occurrence produced no run; present with `skippedAt`. */
+  skippedReason?: ScheduleStopReason
+}
+
+/**
+ * The rule's total run budget: one for a one-shot, the stored cap for a
+ * recurring rule, and undefined when the rule is unlimited.
+ * @param rule - the schedule rule.
+ * @returns the budget, or undefined for an uncapped rule.
+ */
+export function scheduleRunBudget(rule: ScheduleRule): number | undefined {
+  if (rule.mode === 'once') return 1
+  return rule.maxRuns
+}
+
+/**
+ * Whether a rule has no runs left. The scheduler refuses to open another
+ * execution for an exhausted rule, which is what makes the budget terminal
+ * regardless of restarts or repeated timer fires.
+ * @param rule - the schedule rule.
+ * @returns true when the run budget is spent.
+ */
+export function scheduleExhausted(rule: ScheduleRule): boolean {
+  const budget = scheduleRunBudget(rule)
+  return budget !== undefined && rule.runCount >= budget
+}
+
+/**
+ * The instant a rule plans to fire next, for display, whatever its mode: the
+ * planned `at` of a one-shot, or the stored `nextRunAt` of a recurring rule.
+ * @param rule - the schedule rule.
+ * @returns the planned instant, or undefined when the rule carries none.
+ */
+export function schedulePlannedAt(rule: ScheduleRule): number | undefined {
+  if (rule.mode === 'once') return rule.at
+  return rule.nextRunAt
 }
 
 /**
@@ -290,11 +428,27 @@ export interface TaskRecord {
    * armed with the composed execution prompt as its objective. The session then
    * keeps working automatic continuation rounds until the agent marks the goal
    * complete, and this board settles the execution on the goal's own end
-   * instead of at the first turn end. Absent means ON — the option is checked
-   * by default, so only an explicit `false` runs the task as a single plain
-   * turn. A session whose `/goal` command is refused still runs the prompt.
+   * instead of at the first turn end. Absent means OFF — the option is
+   * unchecked by default, so only an explicit `true` joins the goal run and
+   * every card that never touched it (including every card written before the
+   * field existed) runs one plain turn. A session whose `/goal` command is
+   * refused still runs the prompt.
    */
   goalRun?: boolean
+  /**
+   * Run this task's goal executions WITHOUT the board's acceptance gate.
+   *
+   * Absent means INHERIT: the execution's applicability is decided by the
+   * board-wide `goalVerification` switch frozen at start, exactly as a card
+   * that never touched this option behaves. Only an explicit `true` opts the
+   * card out, and the execution records `applicability: 'disabled'` so the
+   * report says plainly that nothing certified this run.
+   *
+   * The opt-out is per execution, not per run: flipping it never rewrites the
+   * verdict rule of an execution already open, and a rerun or a scheduled
+   * occurrence reads the card's current value when it freezes its contract.
+   */
+  skipVerification?: boolean
   /**
    * Frozen context snapshot for a continuation card; absent on plain tasks.
    * Sanitized before it enters the ledger (redaction, slash-command taint,
@@ -326,6 +480,20 @@ export interface TaskRecord {
    * absent means on-board.
    */
   archivedAt?: number
+  /**
+   * Opaque external-provider metadata, keyed by extension id; optional and
+   * additive so existing tasks require no ledger migration. The board stores
+   * and forwards it without interpreting the keys — each provider owns the
+   * shape and validation of its own entry.
+   */
+  integrations?: Record<string, TaskBoardExtensionPayload>
+  /**
+   * Whether the card is held off the board: it exists in the ledger (and its
+   * execution history is intact) but no column ever renders it. Providers
+   * create hidden cards for work they materialize but do not want displayed;
+   * absent means on-board.
+   */
+  hidden?: boolean
 }
 
 /**
@@ -359,7 +527,7 @@ export interface NewTaskInput {
    * inherits the parent's unset execution targets.
    */
   parentId?: string
-  /** Workspace the execution must run in; empty/absent = the recent workspace. */
+  /** Workspace the execution must run in; empty/absent = the creating session's workspace for a root task, else the recent workspace at execution time. */
   workspaceId?: string
   /** Agent preset the execution session must be composed from; empty/absent = deployment default. */
   mode?: string
@@ -372,17 +540,32 @@ export interface NewTaskInput {
   /** Run the subtree as an Agent Team (Team Lead session plus one teammate per direct subtask). */
   teamRun?: boolean
   /**
-   * Start the execution with dsh's built-in `/goal`. Absent/true keeps the
-   * default (goal run); an explicit false requests a single plain turn.
+   * Start the execution with dsh's built-in `/goal`. Absent means the default
+   * (one plain turn); only an explicit true joins the goal run.
    */
   goalRun?: boolean
   /**
-   * Optional scheduled-run rule requested at creation time (the new-task
-   * dialog): an enable flag, a 5-field cron expression, and the IANA zone its
-   * wall clock is read in (absent means the Host zone). The create use case
-   * arms it only when enabled and the expression is valid.
+   * Run this task's goal executions without the acceptance gate; absent
+   * inherits the board-wide `goalVerification` switch.
+   */
+  skipVerification?: boolean
+  /**
+   * Optional scheduling rule requested at creation time (the new-task
+   * dialog): an enable flag, the kind of plan (a recurring 5-field cron
+   * expression, or one planned instant for a one-shot), an optional run cap,
+   * and the IANA zone its wall clock is read in (absent means the Host zone).
+   * The create use case arms it only when enabled and the plan is valid.
    */
-  schedule?: { enabled: boolean; cron: string; timeZone?: string }
+  schedule?: {
+    enabled: boolean
+    mode?: ScheduleMode
+    cron?: string
+    /** Planned instant of a one-shot rule (ms epoch). */
+    at?: number
+    timeZone?: string
+    /** Run cap of a recurring rule; absent means unlimited. */
+    maxRuns?: number
+  }
   /**
    * Optional frozen context snapshot (goal/progress/next, sanitized by the
    * protocol gate) turning the new task into a continuation card.
@@ -398,6 +581,12 @@ export interface NewTaskInput {
    * ahead of the execution prompt; a bare name changes nothing at run time.
    */
   tags?: TaskTag[]
+  /** Opaque external-provider metadata keyed by extension id. */
+  integrations?: Record<string, TaskBoardExtensionPayload>
+  /** Column the created card opens in; absent means the board default (todo). */
+  status?: TaskStatus
+  /** Create the card off-board (held in the ledger, never rendered). */
+  hidden?: boolean
 }
 
 /** The five kanban columns, in display order. */
@@ -409,8 +598,20 @@ export const COLUMNS: readonly { status: TaskStatus; label: string }[] = [
   { status: 'failed', label: '已失败' },
 ]
 
-/** Statuses a user may move a card to manually (execution states are owned by the runner). */
-export const MANUAL_STATUSES: readonly TaskStatus[] = ['backlog', 'todo']
+/**
+ * Statuses a card may be moved to by hand: every column. The board UI and the
+ * agent tool surface share this one list, so no column can be manual on one
+ * surface and locked on the other.
+ *
+ * The column is a board statement, not a claim about a run. A hand-written
+ * `done`/`failed` declares that the work finished (or failed) outside a
+ * Host-run execution — human hands, an external system — and a hand-written
+ * `running` says the work is under way without a tracked session. None of them
+ * creates an {@link ExecutionRecord}, which is what keeps a declaration
+ * distinguishable from a recorded outcome in the card's execution history; the
+ * next settled run overwrites the column with the real one.
+ */
+export const MANUAL_STATUSES: readonly TaskStatus[] = ['backlog', 'todo', 'running', 'done', 'failed']
 
 /** Statuses the runner may move a card to from 'running'. */
 export const RUNNER_SETTLE_STATUSES: readonly TaskStatus[] = ['done', 'failed']
@@ -425,9 +626,35 @@ export function isTaskStatus(value: unknown): value is TaskStatus {
   return typeof value === 'string' && (ALL_STATUSES as readonly string[]).includes(value)
 }
 
-/** Whether a manual move target is allowed from the given status. */
+/**
+ * Whether the manual move `from` -> `to` is a legal *column* change: any
+ * column, and never the one the card already shows.
+ *
+ * Whether the card may be moved at all is a separate question. The lock an
+ * executing card carries belongs to its open execution, never to the column
+ * text, so it is {@link canMoveTask} — not this predicate — that expresses it.
+ */
 export function canMoveManually(from: TaskStatus, to: TaskStatus): boolean {
-  return from !== 'running' && (MANUAL_STATUSES as readonly TaskStatus[]).includes(to)
+  return from !== to && (MANUAL_STATUSES as readonly TaskStatus[]).includes(to)
+}
+
+/**
+ * Whether the board may move `task` to `to` by hand: the card is on-board,
+ * holds no open execution, and the target is a legal column change. The ledger,
+ * the detail view's status buttons and the board's drop handler all decide
+ * through this one predicate, so no two surfaces can disagree about a card.
+ */
+export function canMoveTask(task: TaskRecord, to: TaskStatus): boolean {
+  return task.archivedAt === undefined && !hasOpenExecution(task) && canMoveManually(task.status, to)
+}
+
+/**
+ * Whether the task still has an execution the runner has not settled. This —
+ * not the `running` column — is what "the runner owns this card" means: a card
+ * parked in `running` by hand has no session and stays fully movable.
+ */
+export function hasOpenExecution(task: TaskRecord): boolean {
+  return task.executions.some(execution => execution.endedAt === undefined)
 }
 
 /** Normalize one optional execution-target string: trim; blank collapses to undefined. */
@@ -457,13 +684,14 @@ export function freezeOf(
 /** Create a task from user input. */
 export function createTask(input: NewTaskInput, now: number, id: string): TaskRecord {
   const tags = normalizeTags(input.tags)
+  const integrations = normalizeTaskIntegrations(input.integrations)
   return {
     id,
     title: input.title.trim(),
     description: input.description.trim(),
     prompt: input.prompt.trim(),
     parentId: normalizeTargetId(input.parentId),
-    status: 'todo',
+    status: input.status ?? 'todo',
     createdAt: now,
     updatedAt: now,
     executions: [],
@@ -473,13 +701,19 @@ export function createTask(input: NewTaskInput, now: number, id: string): TaskRe
     model: normalizeTargetId(input.model),
     reuseSession: input.reuseSession === true ? true : undefined,
     teamRun: input.teamRun === true ? true : undefined,
-    // Default ON: only an explicit false is stored, so a card that never
+    // Default OFF: only an explicit true is stored, so a card that never
     // touched the option (and every card written before the field existed)
-    // keeps starting its runs with /goal.
-    goalRun: input.goalRun === false ? false : undefined,
+    // keeps running one plain turn.
+    goalRun: input.goalRun === true ? true : undefined,
+    // Default OFF: only an explicit true is stored, so a card that never
+    // touched the option inherits the board-wide acceptance switch exactly as
+    // every card written before the field existed does.
+    skipVerification: input.skipVerification === true ? true : undefined,
     ...(input.freeze === undefined ? {} : { freeze: freezeOf(input.freeze, now) }),
     ...(input.handover === undefined ? {} : { handover: { ...input.handover, bundledAt: now } }),
     ...(tags === undefined ? {} : { tags }),
+    ...(integrations === undefined ? {} : { integrations }),
+    ...(input.hidden === true ? { hidden: true } : {}),
   }
 }
 
@@ -488,35 +722,38 @@ export function withStatus(task: TaskRecord, status: TaskStatus, now: number): T
   return { ...task, status, updatedAt: now }
 }
 
+/** Every field a schedule patch may carry; the rest of the rule is carried over. */
+const SCHEDULE_PATCH_KEYS = [
+  'enabled', 'mode', 'cron', 'at', 'timeZone', 'nextRunAt', 'lastTriggeredAt',
+  'runCount', 'maxRuns', 'endedAt', 'endedReason', 'skippedAt', 'skippedReason',
+] as const satisfies ReadonlyArray<keyof ScheduleRule>
+
 /**
  * Merge a schedule patch into a task's schedule rule (creating it when
  * absent), with a fresh updatedAt. Keys present in the patch overwrite the
  * current value — including explicit `undefined`, which clears a field (used
- * to disarm `nextRunAt`); absent keys keep their current value.
+ * to disarm `nextRunAt` or to drop an ended/skip record); absent keys keep
+ * their current value.
  */
 export function withSchedule(
   task: TaskRecord,
   patch: Partial<ScheduleRule>,
   now: number,
 ): TaskRecord {
-  const current = task.schedule
-  const schedule: ScheduleRule = {
-    enabled: current?.enabled ?? false,
-    cron: current?.cron ?? '',
-    ...(current?.timeZone === undefined ? {} : { timeZone: current.timeZone }),
-    nextRunAt: current?.nextRunAt,
-    lastTriggeredAt: current?.lastTriggeredAt,
-  }
-  if ('enabled' in patch) schedule.enabled = patch.enabled ?? false
-  if ('cron' in patch) schedule.cron = patch.cron ?? ''
-  // An explicit `undefined` drops the stored zone (back to the Host zone);
-  // an absent key keeps it.
-  if ('timeZone' in patch) {
-    if (patch.timeZone === undefined) delete schedule.timeZone
-    else schedule.timeZone = patch.timeZone
+  const schedule: ScheduleRule = task.schedule === undefined
+    ? { enabled: false, mode: 'cron', nextRunAt: undefined, lastTriggeredAt: undefined, runCount: 0 }
+    : { ...task.schedule }
+  const fields = schedule as unknown as Record<string, unknown>
+  const next = patch as unknown as Record<string, unknown>
+  for (const key of SCHEDULE_PATCH_KEYS) {
+    if (!(key in patch)) continue
+    if (next[key] === undefined) delete fields[key]
+    else fields[key] = next[key]
   }
-  if ('nextRunAt' in patch) schedule.nextRunAt = patch.nextRunAt
-  if ('lastTriggeredAt' in patch) schedule.lastTriggeredAt = patch.lastTriggeredAt
+  // The three non-optional fields can never be cleared into an unusable rule.
+  if (schedule.enabled !== true) schedule.enabled = false
+  if (schedule.mode !== 'once') schedule.mode = 'cron'
+  if (!Number.isFinite(schedule.runCount) || schedule.runCount < 0) schedule.runCount = 0
   return { ...task, updatedAt: now, schedule }
 }
 
@@ -560,6 +797,21 @@ export function startExecution(
   }
 }
 
+/**
+ * The column a settled outcome puts a card in. The ONE derivation shared by
+ * every settlement path (issue #1826): a Host-run execution and an outcome an
+ * outside agent reported both land in the same column from the same rule, so
+ * the column never becomes a second, independent piece of state.
+ @param task - the card being settled.
+ @param outcome - the settled outcome.
+ @returns the status the card shows afterwards.
+ */
+export function settledStatus(task: TaskRecord, outcome: ExecutionOutcome): TaskStatus {
+  if (outcome === 'succeeded') return task.schedule?.enabled === true ? 'todo' : 'done'
+  if (outcome === 'failed') return 'failed'
+  return task.status === 'running' ? 'todo' : task.status
+}
+
 /**
  * Settle a running execution: record the outcome and move the task into the
  * matching column. No-op (returns the input task) when the execution is not
@@ -579,11 +831,7 @@ export function settleExecution(
   const settled: ExecutionRecord = { ...execution, endedAt: now, result: outcome, error }
   const executions = [...task.executions]
   executions[index] = settled
-  const status: TaskStatus = outcome === 'succeeded'
-    ? (task.schedule?.enabled ? 'todo' : 'done')
-    : outcome === 'failed' ? 'failed'
-      : task.status === 'running' ? 'todo' : task.status
-  return { ...task, status, updatedAt: now, executions }
+  return { ...task, status: settledStatus(task, outcome), updatedAt: now, executions }
 }
 
 /** A settled-execution summary string for the detail view. */

package/src/core/use-cases/task-create.ts

@@ -67,22 +67,43 @@ export function applyCreateTask(
     model: input.model ?? parent.model,
   }
   let task = createTask(effective, now, id)
-  // Arm the requested schedule (new-task dialog): only an enabled rule with
-  // a valid cron is applied; blank, invalid, or disabled requests leave the
-  // task unscheduled.
+  // Arm the requested schedule (new-task dialog). The Host wire gate already
+  // rejects a malformed enabled plan; this arm re-checks it so a direct caller
+  // (a test or an embedder) cannot persist an unusable rule either.
   const requested = input.schedule
-  if (requested?.enabled === true && requested.cron.trim() !== '' && isValidCron(requested.cron)) {
-    const cron = requested.cron.trim()
+  if (requested?.enabled === true) {
     const requestedZone = requested.timeZone
     // An unusable zone is refused rather than silently reinterpreted as the
     // Host zone: the caller asked for a specific wall clock.
     if (requestedZone !== undefined && !isValidTimeZone(requestedZone)) {
       return { task: undefined, tasks, error: 'invalid schedule time zone' }
     }
-    const zone = requestedZone ?? hostTimeZone
-    const nextRunAt = nextRunAtMs(cron, now, zone)
-    if (nextRunAt !== undefined) {
-      task = withSchedule(task, { enabled: true, cron, timeZone: requestedZone, nextRunAt }, now)
+    if (requested.mode === 'once') {
+      const at = requested.at
+      if (at === undefined || !Number.isInteger(at) || at <= now) {
+        return { task: undefined, tasks, error: 'invalid schedule instant' }
+      }
+      task = withSchedule(task, {
+        enabled: true,
+        mode: 'once',
+        at,
+        timeZone: requestedZone,
+        nextRunAt: at,
+        runCount: 0,
+      }, now)
+    } else {
+      const cron = (requested.cron ?? '').trim()
+      if (cron !== '' && isValidCron(cron)) {
+        const maxRuns = requested.maxRuns
+        if (maxRuns !== undefined && (!Number.isInteger(maxRuns) || maxRuns < 1)) {
+          return { task: undefined, tasks, error: 'invalid schedule run cap' }
+        }
+        const zone = requestedZone ?? hostTimeZone
+        const nextRunAt = nextRunAtMs(cron, now, zone)
+        if (nextRunAt !== undefined) {
+          task = withSchedule(task, { enabled: true, mode: 'cron', cron, maxRuns, timeZone: requestedZone, nextRunAt, runCount: 0 }, now)
+        }
+      }
     }
   }
   return { task, tasks: [...tasks, task] }

package/src/core/use-cases/task-parent.ts

@@ -49,8 +49,9 @@ export function applySetParent(
   if (task.archivedAt !== undefined) return { tasks, applied: false, error: 'archived task is read-only' }
   // A task with an open execution may be a participant of a running cascade.
   // Moving it would break that group's parent walk and leave the parent card
-  // in the running column forever, so the link is frozen until it settles.
-  if (task.status === 'running' || task.executions.some(execution => execution.endedAt === undefined)) {
+  // in the running column forever, so the link is frozen until it settles. The
+  // lock is the open execution: a card parked in 'running' by hand links freely.
+  if (task.executions.some(execution => execution.endedAt === undefined)) {
     return { tasks, applied: false, error: 'running task cannot be re-parented' }
   }
   const check = checkParentLink(tasks, id, parentId, normalizeSubtaskDepth(maxSubtaskDepth))