@linxin666/dsh-client-ui-task-board 0.1.19 → 0.1.20

package/lib/index.js

@@ -1,5 +1,46 @@
 import { installSettingsSection, settingsNamespace } from "@deepseek-ai/dsh-settings";
 import z from "schemastery";
+//#region src/mount-once.ts
+/**
+* Host single-instance guard shared by the plugin family. The family bundle
+* (dsh-web-ui-all / dsh-skins) namespaces every child row id (web-ui-*), so
+* the loader accepts a standalone install of the same package side by side;
+* without this guard the second instance would still re-register the same
+* webserver routes, tools, settings namespaces, and system-prompt sections
+* and fail the boot. mountOnce makes the second host apply a no-op for the
+* lifetime of the first instance (the browser half is already deduped by
+* package name in the client module host).
+*
+* The registry rides a global symbol so two module instances of the same
+* package (npm copy vs repository link) still share one verdict. cordis
+* `ctx.effect` runs its callback immediately and treats the callback's
+* return value as the fiber disposer, so the unmarker is returned, not run.
+*/
+const MOUNTED = Symbol.for("dsh-web-ui.mounted-plugins");
+function mountedSet() {
+	const registry = globalThis;
+	return registry[MOUNTED] ??= /* @__PURE__ */ new Set();
+}
+/**
+* Wrap a cordis plugin apply so the package runs at most once per process.
+* The first mount registers normally and unmarks when its fiber disposes;
+* any later mount of the same package name is a no-op.
+* @param packageName - npm package identity shared by every install source.
+* @param fn - the original plugin apply.
+* @returns an apply of the same shape.
+*/
+function mountOnce(packageName, fn) {
+	return ((...args) => {
+		const mounted = mountedSet();
+		if (mounted.has(packageName)) return;
+		mounted.add(packageName);
+		args[0]?.effect?.(() => () => {
+			mounted.delete(packageName);
+		});
+		return fn(...args);
+	});
+}
+//#endregion
 //#region src/index.ts
 /** Order of the announcement section within the tool-guidance band. */
 const SECTION_ORDER = 200;
@@ -26,7 +67,8 @@ const DEFAULT_ANNOUNCE = true;
 * @param ctx - the plugin context (systemPrompt injected).
 * @param config - resolved plugin config (schema defaults applied by the loader).
 */
-function apply(ctx, config) {
+const apply = mountOnce("@linxin666/dsh-client-ui-task-board", applyImpl);
+function applyImpl(ctx, config) {
 	let current = () => config ?? {};
 	let disposeSection;
 	const sync = () => {

package/lib/types/client/PluginSettingsCard.d.ts

@@ -9,7 +9,7 @@
 import { type ReactNode } from 'react';
 import type { CardShell } from './settings-form.ts';
 /** Copy keys the card chrome itself reads; every consumer locale carries this shared vocabulary. */
-export declare const CARD_COPY_KEYS: readonly ["settings.collapse", "settings.expand", "settings.notExposed", "settings.unsaved", "settings.readOnly", "settings.saveFailed", "settings.discard", "settings.save", "settings.saving"];
+export declare const CARD_COPY_KEYS: readonly ['settings.collapse', 'settings.expand', 'settings.notExposed', 'settings.unsaved', 'settings.readOnly', 'settings.saveFailed', 'settings.discard', 'settings.save', 'settings.saving'];
 /** Copy key the card chrome itself reads. */
 export type CardCopyKey = (typeof CARD_COPY_KEYS)[number];
 /** Key domain for the plugin's own copy (inferred per consumer). */

package/lib/types/client/board/status-key.d.ts

@@ -0,0 +1,4 @@
+import type { TaskStatus } from '../../core/tasks.ts';
+import type { TaskBoardKey } from '../locales.ts';
+/** Task status → locale key (board column titles and the detail badge). */
+export declare const STATUS_KEY: Record<TaskStatus, TaskBoardKey>;

package/lib/types/core/scheduler.d.ts

@@ -41,6 +41,9 @@ export declare class SchedulerService {
     private environmentListener;
     private disposed;
     private started;
+    /** Cron expressions that matched no instant in the scan window, per task —
+     *  remembered so a never-matching rule is not re-scanned on every tick. */
+    private readonly unmatchableRepair;
     /** @param deps - tasks/clock/trigger/apply faces (see {@link SchedulerDeps}). */
     constructor(deps: SchedulerDeps);
     /**

package/lib/types/core/tasks.d.ts

@@ -84,7 +84,7 @@ export interface TaskRecord {
 /** Statuses a settled task may be archived from. */
 export declare const ARCHIVABLE_STATUSES: readonly TaskStatus[];
 /** Permission presets a task may pin on its execution session (the `/permission <id>` ids). */
-export declare const TASK_PERMISSIONS: readonly ["read-only", "workspace-write", "danger-full-access"];
+export declare const TASK_PERMISSIONS: readonly ['read-only', 'workspace-write', 'danger-full-access'];
 /** One permission preset id. */
 export type TaskPermission = typeof TASK_PERMISSIONS[number];
 /** Whether an unknown value is a known permission preset id. */
@@ -125,6 +125,8 @@ export declare const ALL_STATUSES: readonly TaskStatus[];
 export declare function isTaskStatus(value: unknown): value is TaskStatus;
 /** Whether a manual move target is allowed from the given status. */
 export declare function canMoveManually(_from: TaskStatus, to: TaskStatus): boolean;
+/** Normalize one optional execution-target string: trim; blank collapses to undefined. */
+export declare function normalizeTargetId(value: string | undefined): string | undefined;
 /** Create a task from user input. */
 export declare function createTask(input: NewTaskInput, now: number, id: string): TaskRecord;
 /** Clone a task with an updated status and a fresh updatedAt. */

package/lib/types/index.d.ts

@@ -41,4 +41,6 @@ export declare const Config: z<Config>;
  * @param ctx - the plugin context (systemPrompt injected).
  * @param config - resolved plugin config (schema defaults applied by the loader).
  */
-export declare function apply(ctx: Context, config?: Config): void;
+export declare const apply: typeof applyImpl;
+declare function applyImpl(ctx: Context, config?: Config): void;
+export {};

package/lib/types/mount-once.d.ts

@@ -0,0 +1,24 @@
+/**
+ * Host single-instance guard shared by the plugin family. The family bundle
+ * (dsh-web-ui-all / dsh-skins) namespaces every child row id (web-ui-*), so
+ * the loader accepts a standalone install of the same package side by side;
+ * without this guard the second instance would still re-register the same
+ * webserver routes, tools, settings namespaces, and system-prompt sections
+ * and fail the boot. mountOnce makes the second host apply a no-op for the
+ * lifetime of the first instance (the browser half is already deduped by
+ * package name in the client module host).
+ *
+ * The registry rides a global symbol so two module instances of the same
+ * package (npm copy vs repository link) still share one verdict. cordis
+ * `ctx.effect` runs its callback immediately and treats the callback's
+ * return value as the fiber disposer, so the unmarker is returned, not run.
+ */
+/**
+ * Wrap a cordis plugin apply so the package runs at most once per process.
+ * The first mount registers normally and unmarks when its fiber disposes;
+ * any later mount of the same package name is a no-op.
+ * @param packageName - npm package identity shared by every install source.
+ * @param fn - the original plugin apply.
+ * @returns an apply of the same shape.
+ */
+export declare function mountOnce<T extends (...args: any[]) => unknown>(packageName: string, fn: T): T;

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@linxin666/dsh-client-ui-task-board",
   "description": "Task board for the dsh web GUI: a sidebar entry plus a multi-column kanban view with local persistence, task details, and real execution through dsh sessions (session.prompt). Hot-pluggable — mounted via ~/.dsh/cordis.patch.yml + a profile node_modules symlink, no dsh source changes.",
-  "version": "0.1.19",
+  "version": "0.1.20",
   "type": "module",
   "main": "lib/index.js",
   "types": "lib/types/index.d.ts",

package/src/client/board/TaskBoard.tsx

@@ -9,18 +9,10 @@ import { COLUMNS, type TaskRecord, type TaskStatus } from '../../core/tasks.ts'
 import { t, type TaskBoardKey } from '../locales.ts'
 import css from '../board.module.css'
 import { NewTaskModal } from './NewTaskModal.tsx'
+import { STATUS_KEY } from './status-key.ts'
 import { TaskCard } from './TaskCard.tsx'
 import { TaskDetail } from './TaskDetail.tsx'
 
-/** Column status → locale key. */
-const STATUS_KEY: Record<TaskStatus, TaskBoardKey> = {
-  backlog: 'board.status.backlog',
-  todo: 'board.status.todo',
-  running: 'board.status.running',
-  done: 'board.status.done',
-  failed: 'board.status.failed',
-}
-
 /** Case-insensitive title/description match. */
 function matchesFilter(task: TaskRecord, filter: string): boolean {
   if (filter.trim() === '') return true

package/src/client/board/TaskDetail.tsx

@@ -13,6 +13,7 @@ import { SCHEDULE_PRESETS } from '../schedule-presets.ts'
 import css from '../board.module.css'
 import { ConfirmDialog } from './ConfirmDialog.tsx'
 import { formatTime } from './TaskCard.tsx'
+import { STATUS_KEY } from './status-key.ts'
 
 /** Execution outcome → locale key. */
 const RESULT_KEY: Record<NonNullable<ExecutionRecord['result']>, TaskBoardKey> = {
@@ -21,15 +22,6 @@ const RESULT_KEY: Record<NonNullable<ExecutionRecord['result']>, TaskBoardKey> =
   cancelled: 'detail.result.cancelled',
 }
 
-/** Status → locale key (detail badge). */
-const STATUS_KEY: Record<TaskStatus, TaskBoardKey> = {
-  backlog: 'board.status.backlog',
-  todo: 'board.status.todo',
-  running: 'board.status.running',
-  done: 'board.status.done',
-  failed: 'board.status.failed',
-}
-
 /** One execution-history row. */
 function ExecutionRow({ execution, onOpen }: { execution: ExecutionRecord; onOpen: (sessionId: string) => void }) {
   const result = execution.result

package/src/client/board/status-key.ts

@@ -0,0 +1,11 @@
+import type { TaskStatus } from '../../core/tasks.ts'
+import type { TaskBoardKey } from '../locales.ts'
+
+/** Task status → locale key (board column titles and the detail badge). */
+export const STATUS_KEY: Record<TaskStatus, TaskBoardKey> = {
+  backlog: 'board.status.backlog',
+  todo: 'board.status.todo',
+  running: 'board.status.running',
+  done: 'board.status.done',
+  failed: 'board.status.failed',
+}

package/src/core/scheduler.ts

@@ -68,6 +68,9 @@ export class SchedulerService {
   private environmentListener: (() => void) | undefined
   private disposed = false
   private started = false
+  /** Cron expressions that matched no instant in the scan window, per task —
+   *  remembered so a never-matching rule is not re-scanned on every tick. */
+  private readonly unmatchableRepair = new Map<string, string>()
 
   /** @param deps - tasks/clock/trigger/apply faces (see {@link SchedulerDeps}). */
   constructor(private readonly deps: SchedulerDeps) {}
@@ -133,9 +136,16 @@ export class SchedulerService {
       if (schedule === undefined || !schedule.enabled) continue
       if (schedule.nextRunAt === undefined) {
         // Missing next-run instant (repaired/legacy data): recompute from the
-        // cron expression and wait; an unparseable expression is skipped.
+        // cron expression and wait; an unparseable expression is skipped. A
+        // grammatically valid rule with no match in the window (e.g. Feb 30)
+        // would otherwise re-run the full minute-by-minute scan every tick.
+        if (this.unmatchableRepair.get(task.id) === schedule.cron) continue
         const repaired = nextRunAtMs(schedule.cron, now)
-        if (repaired === undefined) continue
+        if (repaired === undefined) {
+          this.unmatchableRepair.set(task.id, schedule.cron)
+          continue
+        }
+        this.unmatchableRepair.delete(task.id)
         this.deps.applySchedule(task.id, repaired, undefined)
         continue
       }

package/src/core/store.ts

@@ -14,7 +14,7 @@
  * localStorage backend.
  */
 import { isValidCron } from './schedule.ts'
-import { isTaskPermission, isTaskStatus, type ScheduleRule, type TaskRecord, type TaskPermission, type TaskStatus } from './tasks.ts'
+import { isTaskPermission, isTaskStatus, normalizeTargetId, type ScheduleRule, type TaskRecord, type TaskPermission, type TaskStatus } from './tasks.ts'
 
 /** Persistence seam for the task ledger. */
 export interface TaskStore {
@@ -80,11 +80,6 @@ function isTaskRecordShape(value: unknown): value is Omit<TaskRecord, 'status'>
   return true
 }
 
-/** Collapse a blank persisted target string to undefined (clears the pin). */
-function normalizeTargetId(value: string | undefined): string | undefined {
-  return value !== undefined && value.trim() === '' ? undefined : value
-}
-
 /** A task record is structurally valid if it round-trips through the UI. */
 export function isTaskRecord(value: unknown): value is TaskRecord {
   return isTaskRecordShape(value) && isTaskStatus(value.status)

package/src/core/tasks.ts

@@ -151,7 +151,7 @@ export function canMoveManually(_from: TaskStatus, to: TaskStatus): boolean {
 }
 
 /** Normalize one optional execution-target string: trim; blank collapses to undefined. */
-function normalizeTargetId(value: string | undefined): string | undefined {
+export function normalizeTargetId(value: string | undefined): string | undefined {
   const trimmed = value?.trim()
   return trimmed === undefined || trimmed === '' ? undefined : trimmed
 }

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

@@ -8,16 +8,11 @@
  * back to the runtime default); an unknown permission string is ignored so
  * stale UI can never persist a value the execution service rejects.
  */
-import { isTaskPermission, type TaskRecord, type TaskPermission } from '../tasks.ts'
+import { isTaskPermission, normalizeTargetId, type TaskRecord, type TaskPermission } from '../tasks.ts'
 
 /** Editable fields on a task (the update patch surface). */
 export type TaskUpdatePatch = Partial<Pick<TaskRecord, 'title' | 'description' | 'prompt' | 'workspaceId' | 'mode' | 'permission'>>
 
-/** Collapse a blank target string to undefined (clears the pin). */
-function normalizeTargetId(value: string | undefined): string | undefined {
-  return value !== undefined && value.trim() === '' ? undefined : value
-}
-
 /** Keep an unknown permission string from entering the ledger. */
 function normalizePermission(
   current: TaskPermission | undefined,

package/src/index.ts

@@ -15,6 +15,7 @@ import type { Context } from '@deepseek-ai/cordis'
 import { installSettingsSection, settingsNamespace } from '@deepseek-ai/dsh-settings'
 import z from 'schemastery'
 import type {} from '@deepseek-ai/dsh-system-prompt'
+import { mountOnce } from './mount-once.ts'
 
 /** Order of the announcement section within the tool-guidance band. */
 const SECTION_ORDER = 200
@@ -59,7 +60,9 @@ const DEFAULT_ANNOUNCE = true
  * @param ctx - the plugin context (systemPrompt injected).
  * @param config - resolved plugin config (schema defaults applied by the loader).
  */
-export function apply(ctx: Context, config?: Config): void {
+export const apply = mountOnce('@linxin666/dsh-client-ui-task-board', applyImpl)
+
+function applyImpl(ctx: Context, config?: Config): void {
   // The live source the announcement reads: the settings section once the web
   // settings surface is served, the composition entry otherwise
   // (installSettingsSection swaps it when the namespace registers).

package/src/mount-once.ts

@@ -0,0 +1,48 @@
+// Generated by scripts/sync-shared.mjs from shared/host/mount-once.ts. Do not edit this copy; edit the shared source and run "node scripts/sync-shared.mjs".
+/**
+ * Host single-instance guard shared by the plugin family. The family bundle
+ * (dsh-web-ui-all / dsh-skins) namespaces every child row id (web-ui-*), so
+ * the loader accepts a standalone install of the same package side by side;
+ * without this guard the second instance would still re-register the same
+ * webserver routes, tools, settings namespaces, and system-prompt sections
+ * and fail the boot. mountOnce makes the second host apply a no-op for the
+ * lifetime of the first instance (the browser half is already deduped by
+ * package name in the client module host).
+ *
+ * The registry rides a global symbol so two module instances of the same
+ * package (npm copy vs repository link) still share one verdict. cordis
+ * `ctx.effect` runs its callback immediately and treats the callback's
+ * return value as the fiber disposer, so the unmarker is returned, not run.
+ */
+
+const MOUNTED = Symbol.for('dsh-web-ui.mounted-plugins')
+
+interface MountRegistry {
+  [MOUNTED]?: Set<string>
+}
+
+function mountedSet(): Set<string> {
+  const registry = globalThis as MountRegistry
+  return (registry[MOUNTED] ??= new Set())
+}
+
+/**
+ * Wrap a cordis plugin apply so the package runs at most once per process.
+ * The first mount registers normally and unmarks when its fiber disposes;
+ * any later mount of the same package name is a no-op.
+ * @param packageName - npm package identity shared by every install source.
+ * @param fn - the original plugin apply.
+ * @returns an apply of the same shape.
+ */
+export function mountOnce<T extends (...args: any[]) => unknown>(packageName: string, fn: T): T {
+  return ((...args: unknown[]) => {
+    const mounted = mountedSet()
+    if (mounted.has(packageName)) return
+    mounted.add(packageName)
+    const ctx = args[0] as { effect?: (effect: () => unknown) => unknown } | undefined
+    ctx?.effect?.(() => () => {
+      mounted.delete(packageName)
+    })
+    return fn(...args)
+  }) as T
+}