@goodandready/dsh-cost-meter 0.7.6 → 0.7.7

package/docs/design/DESIGN.md

@@ -82,7 +82,7 @@
   - Не допускать утечек локальных путей и инфраструктурных данных в коде или документации.
 
 ## History & Decisions
-- 2026-09-05 (v0.7.6):
+- 2026-09-05 (v0.7.7):
   - Hotfix: исправлен ReferenceError (t is not defined) в объекте словаря ru (lib/client.js) (#15).
   - Добавлен автоматический тест на чистое исполнение фабрики client loader (`test/public-route.test.mjs`).
 - 2026-09-05 (v0.7.4):

package/docs/plans/findings.md

@@ -0,0 +1,16 @@
+# Technical Findings & Architecture Decisions (v0.7.7)
+
+## 1. Zod Usage Rationale
+- `zod` is used in `lib/index.js` for `usageByModelProjection`:
+  DeepSeek Harness core projection machinery requires `.parse(view)` method on projection schemas for runtime boundary contract validation. Schemastery does not provide `.parse()`, so `zod` is strictly required here and is not overengineering.
+
+## 2. Browser Reactivity & Settings
+- Reactivity in browser UI (`lib/client.js`):
+  Using `React.useSyncExternalStore` connected to `ctx.settingsScope` allows instant updates of `currency` and `usdRate` in the header chip and popover without needing a page refresh or roundtrip state reload.
+
+## 3. Slot Hygiene
+- Cleaned up slot injection in `lib/client.js`:
+  Removed `setTimeout(..., 500)` fallback registering `settings.section`. The standard `settings.plugin.item` slot cleanly renders the settings card inside the Harness Settings view.
+
+## 4. Style Isolation & Deduplication
+- Added `data-dsh-plugin="dsh-cost-meter"` attribute to injected `<style>` tags to adhere to DSH UI design guidelines and prevent duplicate stylesheets.

package/docs/plans/progress.md

@@ -0,0 +1,15 @@
+# Implementation Progress (v0.7.7)
+
+- [x] Create Gitea Issue #17
+- [x] Create isolated worktree `refactor/stability-and-quality`
+- [x] Bump version to 0.7.7 in `package.json`
+- [x] Refactor `lib/client.js` for reactive settings, clean slot registration, and style tagging
+- [x] Verify syntax and run test suite (`npm test`, 9 passed)
+- [ ] Document design contract in `docs/design/DESIGN.md`
+- [ ] Commit changes with conventional commits (`git-antigravity`)
+- [ ] Create Pull Request and merge into `main`
+- [ ] Test package on MiniPC isolated test server (`192.168.1.123:3082`)
+- [ ] Smoke-test package on production candidate (`192.168.1.111:3080`)
+- [ ] Publish v0.7.7 to npm and GitHub Release
+- [ ] Install v0.7.7 from npm on MiniAI and verify live health
+- [ ] Close Gitea Issue #17 and clean up worktree

package/docs/plans/task_plan.md

@@ -0,0 +1,25 @@
+# Task Plan: Refactor dsh-cost-meter for stability, simplification & performance
+
+- **Issue**: Gitea #17
+- **Worktree**: `.worktrees/refactor-stability`
+- **Branch**: `refactor/stability-and-quality`
+
+## Phases
+
+- [ ] **Phase 1: Architecture & Overengineering Cleanup**
+  - Remove `zod` dependency from `lib/index.js`, standardize on `@deepseek-ai/schemastery`.
+  - Remove dynamic fragile `require('@deepseek-ai/dsh-client-ui-primitives')` from `lib/client.js`, standardize on clean native inline SVG chevron.
+  - Remove race-prone `setTimeout(..., 500)` fallback slot registration in `lib/client.js`.
+- [ ] **Phase 2: Fault Tolerance & Safe Caching**
+  - Wrap catalog filesystem operations (`mkdirSync`, `writeFileSync`, `readFileSync`) in safe `try/catch`.
+  - Add safe error catching in `lib/client.js` for `/dsh-cost-meter/state` fetch to prevent console error spam on DSH reload.
+- [ ] **Phase 3: Performance & Settings Reactivity**
+  - Implement in-memory catalog index Map in `lib/index.js` for O(1) model rate lookups.
+  - Subscribe to `settingsScope` updates in `lib/client.js` to dynamically refresh currency and USD rate without reload.
+- [ ] **Phase 4: Verification, Tests & Documentation**
+  - Run regression test suite `npm test` and add tests for schema validation and safe catalog failure.
+  - Update `docs/design/DESIGN.md` and release notes for v0.7.7.
+- [ ] **Phase 5: Release Gate & Production Rollout**
+  - Test on MiniPC test server (`192.168.1.123:3082`).
+  - Merge PR, tag v0.7.7, publish to npm & GitHub release.
+  - Deploy to production web profile on MiniAI (`192.168.1.111:3080`).

package/lib/client.js

@@ -131,6 +131,7 @@ window.__ModuleLoader__.load({
       if (document.getElementById('dsh-cost-meter-card-styles')) return
       const el = document.createElement('style')
       el.id = 'dsh-cost-meter-card-styles'
+      el.dataset.dshPlugin = 'dsh-cost-meter'
       el.textContent = cardCss
       document.head.appendChild(el)
     }
@@ -471,6 +472,14 @@ window.__ModuleLoader__.load({
       const [open, setOpen] = React.useState(false)
       const [now, setNow] = React.useState(() => new Date())
       const box = React.useRef(null)
+      const settingsScope = ctx.settingsScope || (ctx.services && ctx.services.settingsScope)
+      const settingsSnap = React.useSyncExternalStore
+        ? React.useSyncExternalStore(
+            (cb) => (settingsScope && typeof settingsScope.subscribe === 'function' ? settingsScope.subscribe(cb) : () => {}),
+            () => (settingsScope && typeof settingsScope.get === 'function' ? settingsScope.get(NS) : null),
+            () => null,
+          )
+        : null
 
       // Every model this session actually billed, so each part is priced on its
       // own tariff instead of the current model's. Usage arrives split into
@@ -507,8 +516,9 @@ window.__ModuleLoader__.load({
 
       if (!state) return null
 
-      const currency = state.currency || '$'
-      const fx = Number(state.usdRate) || 1
+      const stored = (settingsSnap && settingsSnap.value) || {}
+      const currency = stored.currency !== undefined ? String(stored.currency) : (state.currency || '$')
+      const fx = stored.usdRate !== undefined && !isNaN(Number(stored.usdRate)) ? Number(stored.usdRate) : (Number(state.usdRate) || 1)
       const schedule = state.schedule
       const hasUsage = usage !== undefined && usage !== null
       const minute = now.getUTCHours() * 60 + now.getUTCMinutes()
@@ -734,27 +744,12 @@ window.__ModuleLoader__.load({
         ),
       )
 
-      let placed = false
-      ctx.slots.inject('settings.plugin.item', () => {
-        placed = true
-        return ctx.slots.register(
+      ctx.slots.inject('settings.plugin.item', () =>
+        ctx.slots.register(
           { name: 'settings.plugin.item', key: NS, locale: NS, order: 20, inject: () => ({ ctx }) },
           (p) => React.createElement(CostMeterCard, { ...p, ctx }),
-        )
-      })
-
-      // Fallback to settings.section if settings.plugin.item is not declared
-      setTimeout(() => {
-        if (placed) return
-        try {
-          ctx.slots.inject('settings.section', () =>
-            ctx.slots.register(
-              { name: 'settings.section', id: '@goodandready/dsh-cost-meter', order: 25, locale: NS, label: () => makeT(useActiveLocale(ctx) === 'ru' ? ru : en, en)('title'), inject: () => ({ ctx }) },
-              (p) => React.createElement(CostMeterCard, { ...p, ctx }),
-            ),
-          )
-        } catch (_) {}
-      }, 500)
+        ),
+      )
     }
 
     module.exports = { apply, inject: ['slots', 'locale', 'settingsScope'] }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@goodandready/dsh-cost-meter",
-  "version": "0.7.6",
+  "version": "0.7.7",
   "description": "DeepSeek Harness plugin: a session cost chip in the conversation header - live token usage by bucket, peak/off-peak tariff, and a countdown to the next tariff switch.",
   "type": "module",
   "main": "lib/index.js",