@pellux/goodvibes-tui 2.0.21 → 2.1.1

package/src/{panels → views}/modals/plugins-modal.ts

@@ -1,5 +1,5 @@
 import { MODAL_TONES } from './modal-theme.ts';
-import { infoRow } from './modal-surface-helpers.ts';
+import { infoRow, headerRow } from './modal-surface-helpers.ts';
 import type {
   ConfigModalActionContext,
   ConfigModalRow,
@@ -14,11 +14,11 @@ import type {
 
 // ---------------------------------------------------------------------------
 // Plugins → config-modal surface (group-B port). Mirrors the retired
-// PluginsPanel's read surface (list/capabilities/getTrustRecord/
+// the retired plugins view's read surface (list/capabilities/getTrustRecord/
 // getQuarantineRecord) plus `verify` (a pure read-only signature check).
 // enable/disable/liftQuarantine are DROPPED from the dep shape on purpose:
 // those verbs are destructive/interactive mutations and route to the `/plugin`
-// command path (action-parity charter rule). Selection-blind port: the panel's
+// command path (action-parity charter rule). Selection-blind port: the earlier view's
 // selected-plugin signature/quarantine/verify detail is folded into row labels.
 // ---------------------------------------------------------------------------
 
@@ -87,9 +87,9 @@ class PluginsModalSurface implements ConfigModalSurface {
 
     if (all.length === 0) {
       rows.push(infoRow('empty:0', 'No plugins discovered.'));
-      rows.push(infoRow('empty:title', 'Next steps'));
-      rows.push(infoRow('empty:list', '/plugin list  — inspect plugin discovery paths and current registry state', { dim: true }));
-      rows.push(infoRow('empty:market', '/marketplace  — review curated ecosystem entries and provenance posture', { dim: true }));
+      rows.push(headerRow('empty:title', 'Next steps'));
+      rows.push(infoRow('empty:list', '/plugin list  — inspect plugin discovery paths and current registry state', { fg: MODAL_TONES.dim }));
+      rows.push(infoRow('empty:market', '/marketplace  — review curated ecosystem entries and provenance posture', { fg: MODAL_TONES.dim }));
       return { title: 'Plugins', tabs: [{ id: 'plugins', label: 'Plugins', rows, emptyText: '' }] };
     }
 

package/src/{panels → views}/modals/policy-modal.ts

@@ -9,7 +9,8 @@ import type {
   PolicyPreflightReview,
   PolicySimulationSummary,
 } from '@/runtime/index.ts';
-import type { ModalSectionStyle } from '../../renderer/modal-factory.ts';
+import { registerThemeRefresh } from '../../renderer/theme.ts';
+import type { ModalSectionStyle } from '../../input/config-modal-types.ts';
 import type {
   ConfigModalActionContext,
   ConfigModalRow,
@@ -22,9 +23,9 @@ import type {
 // not a selectable list, current/candidate bundle state, diff, divergence
 // gate, history, permission audit, lint findings, preflight review, and
 // simulation samples render as non-selectable rows (the host windows/scrolls
-// them). EVERY dispatchable key in the original panel (s/f/l/p/b) already
-// routed to `/policy <sub>` via executeCommand, none were in-panel mutations;
-// the panel's promote/rollback confirm step is dropped per charter (never fold
+// them). EVERY dispatchable key in the original view (s/f/l/p/b) already
+// routed to `/policy <sub>` via executeCommand, none were in-view mutations;
+// the earlier view's promote/rollback confirm step is dropped per charter (never fold
 // a confirm into a modal) and the command dispatches directly. Determinism:
 // timestamps are formatted (fmtTime/fmtRate), never read from the clock.
 // ---------------------------------------------------------------------------
@@ -49,7 +50,14 @@ export interface PolicyModalDeps {
 const BAD: ModalSectionStyle = { fg: MODAL_TONES.bad };
 const WARN: ModalSectionStyle = { fg: MODAL_TONES.warn };
 const GOOD: ModalSectionStyle = { fg: MODAL_TONES.good };
-const DIM: ModalSectionStyle = { dim: true };
+// Rebuilt in place on theme change (MODAL_TONES refreshes first: it registers at its own module eval).
+registerThemeRefresh(() => {
+  Object.assign(BAD, { fg: MODAL_TONES.bad });
+  Object.assign(WARN, { fg: MODAL_TONES.warn });
+  Object.assign(GOOD, { fg: MODAL_TONES.good });
+  Object.assign(DIM, { fg: MODAL_TONES.dim });
+});
+const DIM: ModalSectionStyle = { fg: MODAL_TONES.dim };
 const MAX_ROWS = 5;
 
 function fmtTime(value: string | undefined): string {

package/src/{panels → views}/modals/provider-health-modal.ts

@@ -11,17 +11,17 @@ export interface ProviderRuntimeInspect {
 
 /**
  * Provider-health config-modal surface (migrated from the `provider-health`
- * panel, the charter's live-modal exemplar; also the target of the `providers`
+ * view, the charter's live-modal exemplar; also the target of the `providers`
  * and `accounts` redirects). providerRuntime.inspectAll() is async, so its
  * result is cached and refreshed on open, on `r`, and on a 3s live tick (the
- * panel's display-tick cadence); buildView reads the cache synchronously so
+ * view's display-tick cadence); buildView reads the cache synchronously so
  * live status/model-count values update in place with a stable layout.
  *
- * Fidelity boundary (stated divergence): the panel's per-request latency
+ * Fidelity boundary (stated divergence): the earlier view's per-request latency
  * timelines, auth-route table, and 8 repair-domain posture blocks are driven by
  * an event-fed tracker + account-posture snapshots; those deep views remain on
  * the `/health` and `/accounts` commands (which print them in full). The modal
- * ports the live provider status list + the panel's one dispatch action
+ * ports the live provider status list + the earlier view's one dispatch action
  * (Enter → `/accounts repair <provider>`), across a Health and an Accounts tab.
  */
 class ProviderHealthModalSurface implements ConfigModalSurface {

package/src/{panels → views}/modals/remote-modal.ts

@@ -20,11 +20,11 @@ function stateTone(state: string): Tone {
 }
 
 /**
- * Remote config-modal surface (migrated from the `remote` panel). The read-model
+ * Remote config-modal surface (migrated from the `remote` view). The read-model
  * snapshot is synchronous; a subscribe() keeps live transport/heartbeat values
- * refreshing between key presses. The panel's `Tab` connections/contracts browse
+ * refreshing between key presses. The earlier view's `Tab` connections/contracts browse
  * toggle becomes two real tabs. `r`/Enter dispatch `/remote recover` for the
- * selected runner, the panel's only corrective keybinding; the richer
+ * selected runner, the earlier view's only corrective keybinding; the richer
  * `/remote` command set (show/supervisor/cancel/dispatch/artifact/…) stays on
  * the command front-door.
  */

package/src/{panels → views}/modals/sandbox-modal.ts

@@ -6,7 +6,7 @@ import { summarizeError } from '@pellux/goodvibes-sdk/platform/utils';
 import { toneStyle, statusGlyph, pad, postureLine, kv, type Tone } from './modal-surface-helpers.ts';
 
 /** SandboxSessionRegistry has no event subscription, poll at the same 3s
- *  cadence the retired panel used (POLL_INTERVAL_MS in sandbox-panel.ts). */
+ *  cadence the retired view used (POLL_INTERVAL_MS in the sandbox modal's earlier view). */
 const POLL_INTERVAL_MS = 3_000;
 
 function sessionTone(state: SandboxSession['state']): Tone {
@@ -28,13 +28,13 @@ function sessionIdOf(row: ConfigModalRow | null): string | null {
 }
 
 /**
- * Sandbox config-modal surface (migrated from the `sandbox` panel). All reads
+ * Sandbox config-modal surface (migrated from the `sandbox` view). All reads
  * (buildSandboxReview, listSandboxProfiles, sessions.list) are synchronous, so
- * buildView reads them live each tick. The panel's single profile+session
+ * buildView reads them live each tick. The earlier view's single profile+session
  * selectable list becomes two real tabs. s/x/e call the SandboxSessionRegistry
  * directly (these were never slash commands), start/execute are wrapped so a
  * rejection surfaces via ctx.setStatus instead of an unhandled rejection,
- * matching the panel's try/catch + setError behaviour.
+ * matching the earlier view's try/catch + setError behaviour.
  */
 class SandboxModalSurface implements ConfigModalSurface {
   readonly name = 'sandbox-modal';

package/src/{panels → views}/modals/security-modal.ts

@@ -1,7 +1,8 @@
 import { MODAL_TONES } from './modal-theme.ts';
-import { infoRow } from './modal-surface-helpers.ts';
+import { infoRow, headerRow } from './modal-surface-helpers.ts';
 import type { TokenAuditResult } from '@pellux/goodvibes-sdk/platform/security';
-import type { ModalSectionStyle } from '../../renderer/modal-factory.ts';
+import { registerThemeRefresh } from '../../renderer/theme.ts';
+import type { ModalSectionStyle } from '../../input/config-modal-types.ts';
 import type { UiReadModel, UiSecuritySnapshot } from '../../runtime/ui-read-models.ts';
 import type {
   ConfigModalActionContext,
@@ -18,7 +19,7 @@ import type {
 // the MCP attack-path review). Read-model-backed, so refresh() is a no-op,
 // buildView reads getSnapshot() fresh every render. 'f' (preflight) routes to
 // its `/policy preflight` command path; 'i' jumps to the incident surface
-// (fleet) via the command path. Selection-blind port: the panel's
+// (fleet) via the command path. Selection-blind port: the earlier view's
 // selected-token scope/rotation detail is folded into each token row label.
 // Determinism: absolute ISO timestamps (never Date.now()).
 // ---------------------------------------------------------------------------
@@ -29,6 +30,11 @@ export interface SecurityModalDeps {
 
 const BAD: ModalSectionStyle = { fg: MODAL_TONES.bad };
 const WARN: ModalSectionStyle = { fg: MODAL_TONES.warn };
+// Rebuilt in place on theme change (MODAL_TONES refreshes first: it registers at its own module eval).
+registerThemeRefresh(() => {
+  Object.assign(BAD, { fg: MODAL_TONES.bad });
+  Object.assign(WARN, { fg: MODAL_TONES.warn });
+});
 const MAX_FINDINGS_SHOWN = 5;
 
 function resultColor(result: TokenAuditResult): ModalSectionStyle | undefined {
@@ -105,11 +111,11 @@ class SecurityModalSurface implements ConfigModalSurface {
     if (audit.results.length === 0) {
       const quarantinedMcp = snapshot.mcpServers.filter((s) => s.schemaFreshness === 'quarantined');
       rows.push(infoRow('empty:0', 'No API tokens are registered with the security auditor yet.'));
-      rows.push(infoRow('empty:1', 'The security control room can already review policy, MCP, plugin, and incident posture, but token-specific scope and rotation audit data has not been registered.', { dim: true }));
+      rows.push(infoRow('empty:1', 'The security control room can already review policy, MCP, plugin, and incident posture, but token-specific scope and rotation audit data has not been registered.', { fg: MODAL_TONES.dim }));
       if (quarantinedMcp.length > 0) rows.push(infoRow('empty:mcp', 'MCP quarantine still active despite no registered tokens.', WARN));
-      rows.push(infoRow('empty:title', 'Inspect further'));
-      rows.push(infoRow('empty:storage', '/storage review — inspect secure secret storage and environment overrides', { dim: true }));
-      rows.push(infoRow('empty:mcptrust', '/mcp trust      — inspect active MCP trust and quarantine posture', { dim: true }));
+      rows.push(headerRow('empty:title', 'Inspect further'));
+      rows.push(infoRow('empty:storage', '/storage review — inspect secure secret storage and environment overrides', { fg: MODAL_TONES.dim }));
+      rows.push(infoRow('empty:mcptrust', '/mcp trust      — inspect active MCP trust and quarantine posture', { fg: MODAL_TONES.dim }));
       return { id: 'tokens', label: 'Tokens', header, rows, emptyText: '' };
     }
 
@@ -121,7 +127,7 @@ class SecurityModalSurface implements ConfigModalSurface {
         ...(resultColor(result) ? { style: resultColor(result)! } : {}),
       });
     }
-    rows.push(infoRow('audit:when', `Last audit ${fmtIso(audit.lastAuditAt)}: press r to refresh`, { dim: true }));
+    rows.push(infoRow('audit:when', `Last audit ${fmtIso(audit.lastAuditAt)}: press r to refresh`, { fg: MODAL_TONES.dim }));
     return { id: 'tokens', label: 'Tokens', header, rows, hints: ['f preflight'] };
   }
 
@@ -144,18 +150,18 @@ class SecurityModalSurface implements ConfigModalSurface {
     }
 
     const review = snapshot.attackPathReview;
-    rows.push(infoRow('atk:title', 'MCP Attack-Path Review'));
-    rows.push(infoRow('atk:counts', `critical ${review.criticalFindings}  incoherent ${review.incoherentFindings}`, review.criticalFindings > 0 ? BAD : { dim: true }));
-    rows.push(infoRow('atk:summary', review.summary, { dim: true }));
+    rows.push(headerRow('atk:title', 'MCP Attack-Path Review'));
+    rows.push(infoRow('atk:counts', `critical ${review.criticalFindings}  incoherent ${review.incoherentFindings}`, review.criticalFindings > 0 ? BAD : { fg: MODAL_TONES.dim }));
+    rows.push(infoRow('atk:summary', review.summary, { fg: MODAL_TONES.dim }));
     const shown = review.findings.slice(0, MAX_FINDINGS_SHOWN);
     shown.forEach((finding, i) => {
       rows.push(infoRow(`atk:${i}:h`, `${finding.severity.toUpperCase()} ${finding.serverName}: ${finding.route}`, findingColor(finding.severity)));
-      rows.push(infoRow(`atk:${i}:r`, `  ${finding.reason}`, { dim: true }));
-      rows.push(infoRow(`atk:${i}:e`, `  evidence: ${finding.evidence.join(' | ')}`, { dim: true }));
+      rows.push(infoRow(`atk:${i}:r`, `  ${finding.reason}`, { fg: MODAL_TONES.dim }));
+      rows.push(infoRow(`atk:${i}:e`, `  evidence: ${finding.evidence.join(' | ')}`, { fg: MODAL_TONES.dim }));
     });
     if (review.findings.length > shown.length) {
       const extra = review.findings.length - shown.length;
-      rows.push(infoRow('atk:more', `+${extra} more finding${extra === 1 ? '' : 's'} not shown`, { dim: true }));
+      rows.push(infoRow('atk:more', `+${extra} more finding${extra === 1 ? '' : 's'} not shown`, { fg: MODAL_TONES.dim }));
     }
     return { id: 'governance', label: 'Governance', rows, emptyText: 'No governance findings.' };
   }
@@ -169,8 +175,8 @@ class SecurityModalSurface implements ConfigModalSurface {
     if (id === 'refresh') { ctx.setStatus('Security posture is read live.'); ctx.requestRender(); return; }
     if (id === 'preflight') { void ctx.executeCommand?.('policy', ['preflight']); ctx.setStatus('Dispatched /policy preflight.'); return; }
     if (id === 'jumpToIncident' && this.deps.readModel.getSnapshot().latestIncident) {
-      void ctx.executeCommand?.('panel', ['open', 'incident']);
-      ctx.setStatus('Opened the incident surface (fleet).');
+      void ctx.executeCommand?.('agents', []);
+      ctx.setStatus('Opened Agents on the running processes.');
       return;
     }
     if (id === 'manageDevices') {

package/src/{panels → views}/modals/services-modal.ts

@@ -33,7 +33,7 @@ function statusDotTone(e: Entry): Tone {
   return 'dim';
 }
 
-// Ported verbatim from ServicesPanel.authSummary (the retired panel's derivation).
+// Ported verbatim from the retired services view's authSummary derivation.
 function authSummary(config: ServiceConfig, manager: SubscriptionAccessQuery): string {
   const provider = config.providerId ?? config.name;
   const hasOverride = manager.getAccessToken(provider) != null;
@@ -46,10 +46,10 @@ function authSummary(config: ServiceConfig, manager: SubscriptionAccessQuery): s
 }
 
 /**
- * Services config-modal surface (migrated from the `services` panel). Reads the
+ * Services config-modal surface (migrated from the `services` view). Reads the
  * same ServiceInspectionQuery, getAll() is sync but inspect()/testConnection()
  * are async, so an inspection cache is refreshed on open / `r` and read
- * synchronously by buildView (mirrors the panel's own refresh contract).
+ * synchronously by buildView (mirrors the earlier view's own refresh contract).
  * Actions map onto their existing behaviours; `s` jumps to the subscription
  * surface, `t`/`T` run the same live connection tests, `r` re-inspects.
  */

package/src/{panels → views}/modals/settings-sync-modal.ts

@@ -14,13 +14,13 @@ function sourceTone(source: string, conflict: boolean, locked: boolean): Tone {
 }
 
 /**
- * Settings-sync config-modal surface (migrated from the `settings-sync` panel).
- * All reads are synchronous (getSettingsControlPlaneSnapshot). The panel's six
+ * Settings-sync config-modal surface (migrated from the `settings-sync` view).
+ * All reads are synchronous (getSettingsControlPlaneSnapshot). The earlier view's six
  * Tab-cycled browse modes become six real tabs. Conflict resolution keeps the
- * panel's two-step shape: Enter on a conflicted key arms a resolve prompt, then
+ * view's two-step shape: Enter on a conflicted key arms a resolve prompt, then
  * l/s dispatch the existing `/settings-sync resolve <key> <local|synced>`
  * command (the command owns the mutation + live-session refresh). `m` dispatches
- * `/managed review`. push/pull remain CLI-only (they were never panel actions).
+ * `/managed review`. push/pull remain CLI-only (they were never view actions).
  */
 class SettingsSyncModalSurface implements ConfigModalSurface {
   readonly name = 'settings-sync-modal';

package/src/{panels → views}/modals/skills-modal.ts

@@ -1,22 +1,22 @@
 import { MODAL_TONES } from './modal-theme.ts';
-import { infoRow } from './modal-surface-helpers.ts';
+import { infoRow, headerRow } from './modal-surface-helpers.ts';
 import { readFileSync, readdirSync, mkdtempSync, rmSync } from 'node:fs';
 import { tmpdir } from 'node:os';
 import { join, sep } from 'node:path';
 import type { ConfigModalActionContext, ConfigModalRow, ConfigModalSurface, ConfigModalView } from '../../input/config-modal-types.ts';
-import type { SkillOrigin, SkillRecord } from '../skills-panel.ts';
+import type { SkillOrigin, SkillRecord } from '../skills-discovery.ts';
 import { listInstalledEcosystemEntries, type EcosystemCatalogPathOptions, type ShellPathService } from '@/runtime/index.ts';
 
 // ---------------------------------------------------------------------------
 // Skills → config-modal surface (group-B port). Project-local and global
 // skill-pack discovery. Disk scans happen only in refresh() (never in
-// buildView), mirroring the panel's onActivate()-triggered load. Browse-only:
-// the panel never had a command-routable mutation verb (no /skills delete/remove
+// buildView), mirroring the earlier view's onActivate()-triggered load. Browse-only:
+// the earlier view never had a command-routable mutation verb (no /skills delete/remove
 // subcommand exists), so the only action is refresh/navigate. Selection-blind
-// port: the panel's selected-skill path/deps/includes detail is folded into
+// port: the earlier view's selected-skill path/deps/includes detail is folded into
 // each row label.
 //
-// discoverSkills() in the panel was async; the sync re-implementation below
+// discoverSkills() in the earlier view was async; the sync re-implementation below
 // mirrors the same directory/frontmatter/marketplace-provenance conventions
 // (the two are independent implementations of the same on-disk contract).
 // ---------------------------------------------------------------------------
@@ -136,9 +136,9 @@ class SkillsModalSurface implements ConfigModalSurface {
     const rows: ConfigModalRow[] = [];
     if (this.cached.length === 0) {
       rows.push(infoRow('empty:0', 'No skills discovered.'));
-      rows.push(infoRow('empty:title', 'Next steps'));
-      rows.push(infoRow('empty:dir', '.goodvibes/skills       — place skill .md files here (project-local) or ~/.goodvibes/skills (global)', { dim: true }));
-      rows.push(infoRow('empty:registry', '/registry search skills — inspect the same skill directories from the shell', { dim: true }));
+      rows.push(headerRow('empty:title', 'Next steps'));
+      rows.push(infoRow('empty:dir', '.goodvibes/skills       — place skill .md files here (project-local) or ~/.goodvibes/skills (global)', { fg: MODAL_TONES.dim }));
+      rows.push(infoRow('empty:registry', '/registry search skills — inspect the same skill directories from the shell', { fg: MODAL_TONES.dim }));
       return { title: 'Skills', tabs: [{ id: 'skills', label: 'Skills', rows, emptyText: '' }] };
     }
 

package/src/{panels → views}/modals/subscription-modal.ts

@@ -38,12 +38,12 @@ function isExpiringSoon(sub: ProviderSubscription | null): boolean {
 }
 
 /**
- * Subscription config-modal surface (migrated from the `subscription` panel).
+ * Subscription config-modal surface (migrated from the `subscription` view).
  * All reads are synchronous (serviceRegistry.getAll + subscription-manager
  * queries) so buildView reads them live each tick; a 5s timer keeps an
  * externally-completed browser OAuth handshake visible without a keypress, the
- * same cadence the panel used. Enter signs out (surface-managed confirm,
- * matching the panel's ConfirmState) on an active row, or starts login via the
+ * same cadence the earlier view used. Enter signs out (surface-managed confirm,
+ * matching the earlier view's ConfirmState) on an active row, or starts login via the
  * existing `/subscription login <p> start` command on an available row.
  */
 class SubscriptionModalSurface implements ConfigModalSurface {

package/src/{panels → views}/modals/work-plan-modal.ts

@@ -11,10 +11,10 @@ import type { WorkPlanItemStatus } from '@pellux/goodvibes-sdk/platform/workflow
 // persistent plan's checklist items with a progress summary + status counts.
 // All mutations (add/edit item, cycle/set status, remove, clear-completed)
 // route to the existing `/work-plan` command path (charter: no
-// destructive/interactive mutation direct-called from a modal; the panel's
+// destructive/interactive mutation direct-called from a modal; the earlier view's
 // inline add/edit draft form has no place in a read/navigate modal, the bare
 // `/work-plan add` subcommand is the honest stand-in). Selection-blind port:
-// the panel's selected-item owner/source/notes/linked detail is folded into
+// the earlier view's selected-item owner/source/notes/linked detail is folded into
 // each row label.
 // ---------------------------------------------------------------------------
 
@@ -64,11 +64,8 @@ class WorkPlanModalSurface implements ConfigModalSurface {
 
   private readonly hasRow = (row: ConfigModalRow | null): boolean => row !== null;
 
-  /** 'i'/'w' jump to the selected item's linked agent/WRFC chain in Fleet
-   *  (item 4, restores the retired WorkPlanPanel's per-item deep-link,
-   *  lost when migrated this surface to a modal without it). Gated on
-   *  the link actually being present, same as the retired panel gated the key
-   *  on `item?.linked?.agentId`/`wrfcId` before ever reaching the jump. */
+  /** 'i'/'w' open the Agents modal on the selected item's linked agent or
+   *  WRFC chain. Gated on the link actually being present. */
   private readonly hasAgentLink = (row: ConfigModalRow | null): boolean =>
     !!(row && this.itemFrom(row.id)?.linked?.agentId);
   private readonly hasWrfcLink = (row: ConfigModalRow | null): boolean =>
@@ -137,17 +134,17 @@ class WorkPlanModalSurface implements ConfigModalSurface {
     if (id === 'jumpAgent') {
       const agentId = item.linked?.agentId;
       if (!agentId) return;
-      // 'agent' matches the SDK's ProcessKind for a live agent node, see
-      // FleetPanel.receiveDeepLink's id+kind match.
-      void ctx.executeCommand?.('panel', ['open', 'fleet', '--target', `${agentId}:agent`]);
-      ctx.setStatus(`Opened Fleet on agent ${agentId}.`);
+      // 'agent' matches the SDK's ProcessKind for a live agent node, which
+      // AgentsModal.reveal matches on id and kind.
+      void ctx.executeCommand?.('agents', ['--target', `${agentId}:agent`]);
+      ctx.setStatus(`Opened Agents on agent ${agentId}.`);
       return;
     }
     if (id === 'jumpWrfc') {
       const wrfcId = item.linked?.wrfcId;
       if (!wrfcId) return;
-      void ctx.executeCommand?.('panel', ['open', 'fleet', '--target', `${wrfcId}:wrfc-chain`]);
-      ctx.setStatus(`Opened Fleet on WRFC chain ${wrfcId}.`);
+      void ctx.executeCommand?.('agents', ['--target', `${wrfcId}:wrfc-chain`]);
+      ctx.setStatus(`Opened Agents on WRFC chain ${wrfcId}.`);
       return;
     }
     const status: WorkPlanItemStatus | null =