@climber47/dsh-step-clock 0.1.0 → 0.1.1

package/README.md

@@ -63,7 +63,7 @@ A dsh bundle, mounted as one inserted row:
 - `cordis.patch.yml` inserts the `step-clock` row.
 - `lib/index.js` is the (intentionally empty) host half. A bundle's row resolves the package root, so the package must be importable; this plugin has no host behaviour.
 - `lib/client.js` is the browser half, in the `window.__ModuleLoader__.load({ id, factory })` registration shape a client bundle must have. React is taken from the module loader via `require('react')` rather than bundled.
-- It registers two additive entries — `conversation.input.dock` above the composer and `conversation.composer.dock` below it — both `replaceRisk: none`, so every shipped todo / goal / queue / stats entry is untouched. It owns its stylesheet through `ctx.styles.insert`, disposed with its fiber.
+- It registers one additive entry in `conversation.input.dock` (`replaceRisk: none`, so every shipped todo / goal / queue entry is untouched) and injects its stylesheet by creating a tagged `<style>` element, removed when the plugin unloads.
 - The shipped bundle is generated from `src/client/` by `npm run build`, so the readable source and the artifact cannot drift; `npm test` rebuilds and drives the bundle through its real loader contract.
 
 It reads only facts the engine already publishes — the Chat timeline snapshot and the live running-call list — and polls nothing itself.

package/README.zh.md

@@ -63,7 +63,8 @@ dsh plugin --profile web add @climber47/dsh-step-clock
 - `cordis.patch.yml` 插入 `step-clock` 行。
 - `lib/index.js` 是（有意为空的）宿主半边。bundle 的 insert 行会解析包根，所以包必须可被导入；本插件没有宿主行为。
 - `lib/client.js` 是浏览器半边，采用客户端 bundle 必须的 `window.__ModuleLoader__.load({ id, factory })` 注册形状。React 通过 `require('react')` 从模块加载器取得，不打包进产物。
-- 它注册**两个纯新增**条目——输入框上方的 `conversation.input.dock` 与下方的 `conversation.composer.dock`，两者都是 `replaceRisk: none`，不会动自带的 todo / goal / queue / stats 条目。样式通过 `ctx.styles.insert` 归属自身，随 fiber 一起销毁。
+- 它只注册**一个纯新增**条目：输入框上方的 `conversation.input.dock`（`replaceRisk: none`，不会动自带的 todo / goal / queue 条目）。样式通过创建一个带标记的 `<style>` 元素注入，插件卸载时移除。
+- 它只声明**真实存在**的服务（`slots`、`timer`）。客户端插件没有样式服务——`styles` 只存在于动态插件沙箱里。声明一个无人提供的服务会让 Cordis 无限期等待，插件会显示"已加载"、不报任何错，却永远不渲染。
 - 发布的 bundle 由 `npm run build` 从 `src/client/` 生成，因此可读源码与产物不会漂移；`npm test` 会先重新构建，再把 bundle 喂进它真实的加载器契约里跑行为测试。
 
 它只读取引擎**已经发布**的事实——Chat 时间线快照与运行中调用列表——自身不做任何轮询。

package/lib/client.js

@@ -267,7 +267,7 @@ function StepClock(props) {
       if (typeof began === 'number' && typeof ended === 'number' && ended >= began) {
         if (record.seenTurn !== candidate.turn || record.seenStep !== last.step) {
           record.final = record.pending
-          record.pending = { turn: candidate.turn, step: last.step, ms: ended - began, closedAt: now }
+          record.pending = { turn: candidate.turn, step: last.step, ms: ended - began }
           record.seenTurn = candidate.turn
           record.seenStep = last.step
         }
@@ -300,7 +300,9 @@ function StepClock(props) {
       say = '已提交，正在等待模型响应，已等待 ' + humanDuration(elapsedMs)
       where = '第 ' + stepNumber + ' 步'
     }
-    clock = clockOf(elapsedMs)
+    // An unknown anchor must not render as `0:00`, which would read as
+    // "just started" when in fact the start is not known.
+    clock = anchor === null ? '--:--' : clockOf(elapsedMs)
   } else if (shown !== null) {
     // No step is running (yet). Keep reporting the previous step, so the gap
     // between a Turn closing and the next step starting — and a fresh page load
@@ -346,13 +348,44 @@ function StepClock(props) {
 }
 
 /**
- * Services this plugin waits for before applying.
+ * Cordis services this plugin waits for before applying.
  *
- * Declared rather than assumed: `styles` and `slots` are owned by the renderer
- * module, and Cordis parks this plugin until they exist instead of calling
- * `apply` with a half-built context. `timer` supplies the ticking interval.
+ * Only real services belong here. Cordis parks a plugin whose declared service
+ * is missing and **waits indefinitely** — it does not throw — so adding a name
+ * that no provider offers silently produces a plugin that loads, reports no
+ * error, and never runs. `slots` owns registration; `timer` supplies the
+ * ticking interval.
+ *
+ * Notably absent: there is no `styles` service. Stylesheet insertion is done
+ * directly on the DOM below, which is what the shipped client plugins do.
+ */
+const inject = ['slots', 'timer']
+
+/** Style-tag identity, so re-mounting a second copy reuses one tag. */
+const STYLE_TAG = '@climber47/dsh-step-clock/step-clock.css'
+
+/**
+ * Insert the plugin's stylesheet once, returning its disposer.
+ *
+ * A client plugin has no stylesheet service to hand this to: the style element
+ * is created and owned by this plugin, tagged so a second instance reuses it,
+ * and removed when the plugin unloads.
+ * @returns a disposer removing the tag, or a no-op outside a browser.
  */
-const inject = ['slots', 'styles', 'timer']
+function insertStyles() {
+  if (typeof document === 'undefined') return function () {}
+  if (document.querySelector('style[data-plugin-css="' + STYLE_TAG + '"]') !== null) {
+    return function () {}
+  }
+  const tag = document.createElement('style')
+  tag.dataset.plugin = '@climber47/dsh-step-clock'
+  tag.dataset.pluginCss = STYLE_TAG
+  tag.textContent = CSS
+  document.head.appendChild(tag)
+  return function () {
+    tag.remove()
+  }
+}
 
 /**
  * Register the dock entry.
@@ -362,9 +395,7 @@ const inject = ['slots', 'styles', 'timer']
  * `id` is added beside the shipped entries rather than replacing any of them.
  */
 function apply(ctx) {
-  ctx.effect(function () {
-    return ctx.styles.insert(CSS)
-  })
+  ctx.effect(insertStyles)
   const mount = function (slot, id, order, label) {
     ctx.slots.inject(slot, function () {
       return ctx.slots.register({ name: slot, id: id, order: order }, function (props) {
@@ -382,6 +413,7 @@ function apply(ctx) {
 //#endregion
 		exports.StepClock = StepClock;
 		exports.CSS = CSS;
+		exports.insertStyles = insertStyles;
 		exports.apply = apply;
 		exports.inject = inject;
 		return module.exports;

package/lib/index.js

@@ -13,7 +13,12 @@
  * @module @climber47/dsh-step-clock
  */
 
-/** No services are required: the host half registers nothing. */
+/**
+ * No host services are required.
+ *
+ * The browser half declares its own (`slots`, `timer`); see
+ * `src/client/index.js`. Nothing here should be read as describing it.
+ */
 export const inject = []
 
 /**

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@climber47/dsh-step-clock",
-  "version": "0.1.0",
+  "version": "0.1.1",
   "description": "Live per-step elapsed-time clock for the DeepSeek Harness web GUI: shows which step is running, which tools it is running, and how long it has been going.",
   "keywords": [
     "deepseek",

package/src/client/index.js

@@ -231,7 +231,7 @@ export function StepClock(props) {
       if (typeof began === 'number' && typeof ended === 'number' && ended >= began) {
         if (record.seenTurn !== candidate.turn || record.seenStep !== last.step) {
           record.final = record.pending
-          record.pending = { turn: candidate.turn, step: last.step, ms: ended - began, closedAt: now }
+          record.pending = { turn: candidate.turn, step: last.step, ms: ended - began }
           record.seenTurn = candidate.turn
           record.seenStep = last.step
         }
@@ -264,7 +264,9 @@ export function StepClock(props) {
       say = '已提交，正在等待模型响应，已等待 ' + humanDuration(elapsedMs)
       where = '第 ' + stepNumber + ' 步'
     }
-    clock = clockOf(elapsedMs)
+    // An unknown anchor must not render as `0:00`, which would read as
+    // "just started" when in fact the start is not known.
+    clock = anchor === null ? '--:--' : clockOf(elapsedMs)
   } else if (shown !== null) {
     // No step is running (yet). Keep reporting the previous step, so the gap
     // between a Turn closing and the next step starting — and a fresh page load
@@ -310,13 +312,44 @@ export function StepClock(props) {
 }
 
 /**
- * Services this plugin waits for before applying.
+ * Cordis services this plugin waits for before applying.
  *
- * Declared rather than assumed: `styles` and `slots` are owned by the renderer
- * module, and Cordis parks this plugin until they exist instead of calling
- * `apply` with a half-built context. `timer` supplies the ticking interval.
+ * Only real services belong here. Cordis parks a plugin whose declared service
+ * is missing and **waits indefinitely** — it does not throw — so adding a name
+ * that no provider offers silently produces a plugin that loads, reports no
+ * error, and never runs. `slots` owns registration; `timer` supplies the
+ * ticking interval.
+ *
+ * Notably absent: there is no `styles` service. Stylesheet insertion is done
+ * directly on the DOM below, which is what the shipped client plugins do.
+ */
+export const inject = ['slots', 'timer']
+
+/** Style-tag identity, so re-mounting a second copy reuses one tag. */
+const STYLE_TAG = '@climber47/dsh-step-clock/step-clock.css'
+
+/**
+ * Insert the plugin's stylesheet once, returning its disposer.
+ *
+ * A client plugin has no stylesheet service to hand this to: the style element
+ * is created and owned by this plugin, tagged so a second instance reuses it,
+ * and removed when the plugin unloads.
+ * @returns a disposer removing the tag, or a no-op outside a browser.
  */
-export const inject = ['slots', 'styles', 'timer']
+function insertStyles() {
+  if (typeof document === 'undefined') return function () {}
+  if (document.querySelector('style[data-plugin-css="' + STYLE_TAG + '"]') !== null) {
+    return function () {}
+  }
+  const tag = document.createElement('style')
+  tag.dataset.plugin = '@climber47/dsh-step-clock'
+  tag.dataset.pluginCss = STYLE_TAG
+  tag.textContent = CSS
+  document.head.appendChild(tag)
+  return function () {
+    tag.remove()
+  }
+}
 
 /**
  * Register the dock entry.
@@ -326,9 +359,7 @@ export const inject = ['slots', 'styles', 'timer']
  * `id` is added beside the shipped entries rather than replacing any of them.
  */
 export function apply(ctx) {
-  ctx.effect(function () {
-    return ctx.styles.insert(CSS)
-  })
+  ctx.effect(insertStyles)
   const mount = function (slot, id, order, label) {
     ctx.slots.inject(slot, function () {
       return ctx.slots.register({ name: slot, id: id, order: order }, function (props) {