oh-my-hi 0.4.2 → 0.4.4

package/.claude-plugin/marketplace.json

@@ -10,7 +10,7 @@
     {
       "name": "oh-my-hi",
       "description": "Visual dashboard for Claude Code harness — usage/token analysis of skills, agents, plugins, hooks, memory, MCP servers, rules, and principles",
-      "version": "0.4.2",
+      "version": "0.4.4",
       "author": {
         "name": "Jae Sung Park",
         "email": "alberto.park@gmail.com"

package/CHANGELOG.md

@@ -1,5 +1,23 @@
 # Changelog
 
+## [0.4.4] - 2026-04-04
+
+### Fixed
+- SKILL.md: show explicit error message (`ERROR — generate-dashboard.mjs not found`) and exit 1 when script is not found, instead of silently failing
+
+### Added
+- Tests: `test/firstrun.test.mjs` — 10 tests covering `computeDateRange` logic (empty data, null timestamps, multi-scope aggregation) and SKILL.md bash command correctness
+- Tests: `web-ui.test.mjs` — `showFirstRunBanner` function, `_firstRun`/`_dateRange` references, CSS classes, locale keys
+- Tests: `build.test.mjs` — normal builds must not include `_firstRun` or `_partial` flags
+
+## [0.4.3] - 2026-04-04
+
+### Fixed
+- Plugin path resolution: SKILL.md `find` command now searches `plugins/` cache directory instead of relying on `CLAUDE_PLUGIN_ROOT` (which points to the marketplace mirror, not the versioned install cache with `scripts/`)
+
+### Added
+- First-run completion banner: after full data loads on first run, a green banner shows the parsed date range (e.g. "✅ Full data loaded · Jan 1, 2025 – Apr 4, 2026") with an × close button that only disappears on user click
+
 ## [0.4.2] - 2026-04-03
 
 ### Changed

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "oh-my-hi",
-  "version": "0.4.2",
+  "version": "0.4.4",
   "description": "Claude Code harness insights dashboard",
   "repository": {
     "type": "git",

package/scripts/generate-dashboard.mjs

@@ -346,7 +346,7 @@ async function main() {
 
     console.log('  [4/4] loading full history (this may take a moment)...');
     const phase2ScopeData = await collectAllScopes(scopes, { days: 0, cache, cachePath: CACHE_PATH, progress: true });
-    const phase2Data = buildDataObject(scopes, phase2ScopeData, systemLocale);
+    const phase2Data = buildDataObject(scopes, phase2ScopeData, systemLocale, { _firstRun: true, _dateRange: computeDateRange(phase2ScopeData) });
     writeDataJs(phase2Data, dataPath);
     openOrRefreshBrowser(indexPath);
   } else {
@@ -523,6 +523,20 @@ async function collectAllScopes(scopes, { days = 0, cache, cachePath, progress =
 }
 
 /** Build data object from scope data (strips internal _cacheStats) */
+/** Compute min/max date range from all tokenEntries across all scopes */
+function computeDateRange(scopeData) {
+  let minTs = null, maxTs = null;
+  for (const sdata of Object.values(scopeData)) {
+    for (const entry of (sdata?.usage?.tokenEntries || [])) {
+      const ts = entry.timestamp;
+      if (!ts) continue;
+      if (!minTs || ts < minTs) minTs = ts;
+      if (!maxTs || ts > maxTs) maxTs = ts;
+    }
+  }
+  return minTs && maxTs ? { from: minTs, to: maxTs } : null;
+}
+
 function buildDataObject(scopes, scopeData, systemLocale, extra = {}) {
   // Strip _cacheStats from usage data before building output
   const cleanScopeData = {};

package/skills/omh/SKILL.md

@@ -42,14 +42,13 @@ If the user chooses **enable**, run `--enable-auto`. If they choose **disable**,
 - **Auto-refresh**: `--enable-auto` registers a Stop hook — rebuilds on every session end
 - **Browser reuse**: macOS AppleScript tab detection; Windows/Linux fallback to system open
 
-Find the oh-my-hi plugin installation path and run the script:
+Find and run the script (searches plugin cache first, then marketplaces):
 
 ```bash
-OMH_ROOT=$(dirname "$(find "${CLAUDE_CONFIG_DIR:-$HOME/.claude}" -path "*/oh-my-hi/*/scripts/generate-dashboard.mjs" -print -quit 2>/dev/null)") && node "$OMH_ROOT/generate-dashboard.mjs" $ARGUMENTS
-```
-
-If `CLAUDE_PLUGIN_ROOT` is available, use it directly:
-
-```bash
-node "${CLAUDE_PLUGIN_ROOT}/scripts/generate-dashboard.mjs" $ARGUMENTS
+SCRIPT=$(find "${CLAUDE_CONFIG_DIR:-$HOME/.claude}/plugins" -name "generate-dashboard.mjs" -path "*/scripts/generate-dashboard.mjs" -print -quit 2>/dev/null)
+if [ -z "$SCRIPT" ]; then
+  echo "oh-my-hi: ERROR — generate-dashboard.mjs not found. Try: /omh --update"
+  exit 1
+fi
+node "$SCRIPT" $ARGUMENTS
 ```

package/templates/app.js

@@ -4034,7 +4034,24 @@
     document.body.prepend(banner);
   }
 
+  // ── First-run completion banner ──
+  function showFirstRunBanner() {
+    if (!DATA._firstRun) return;
+    const dr = DATA._dateRange;
+    let dateStr = '';
+    if (dr && dr.from && dr.to) {
+      const fmt = d => new Date(d).toLocaleDateString(undefined, { year: 'numeric', month: 'short', day: 'numeric' });
+      dateStr = ' · ' + fmt(dr.from) + ' – ' + fmt(dr.to);
+    }
+    const banner = document.createElement('div');
+    banner.className = 'firstrun-banner';
+    banner.innerHTML = '<span>' + t('firstRunBannerMsg') + dateStr + '</span>'
+      + '<button class="firstrun-close" onclick="this.parentElement.remove()" title="' + t('close') + '">✕</button>';
+    document.body.prepend(banner);
+  }
+
   // ── Boot ──
   init();
   showPartialBanner();
+  showFirstRunBanner();
   checkDataVersion();

package/templates/locales/en.json

@@ -339,6 +339,8 @@
   "updateBannerMsg": "⚡ New data detected. Refresh the page to see the latest data.",
   "updateBannerRefresh": "Refresh",
   "partialBannerMsg": "Showing last 7 days. Full data is loading in the background\u2026",
+  "firstRunBannerMsg": "✅ Full data loaded",
+  "close": "Close",
   "catDescSkills": "Skills are reusable prompt templates invoked via /commands. Provided from the skills/ directory or plugins, each SKILL.md defines name, description, and argument hints in its frontmatter. They can also be auto-invoked via the Skill tool during conversations.",
   "catDescAgents": "Agents are subprocesses that autonomously handle complex tasks. Defined as markdown in the agents/ directory, they specify model (haiku/sonnet/opus), available tools, and domain expertise. Dispatched via the Agent tool, they run in independent contexts.",
   "catDescRules": "Rules are instruction files that control Claude's behavior. Stored as markdown files in {{configDir}}/rules/, they are automatically loaded into context in every session. Unlike CLAUDE.md, they allow separation of concerns into individual files.",

package/templates/locales/ko.json

@@ -339,6 +339,8 @@
   "updateBannerMsg": "⚡ 새로운 데이터가 감지되었습니다. 페이지를 새로고침하면 최신 데이터를 확인할 수 있습니다.",
   "updateBannerRefresh": "새로고침",
   "partialBannerMsg": "최근 7일 데이터로 표시 중입니다. 전체 데이터를 백그라운드에서 로딩하고 있습니다\u2026",
+  "firstRunBannerMsg": "✅ 전체 데이터 파싱이 완료되었습니다",
+  "close": "닫기",
   "catDescSkills": "Skills는 사용자가 /명령어로 호출하는 재사용 가능한 프롬프트 템플릿입니다. skills/ 디렉토리 또는 플러그인을 통해 제공되며, 각 SKILL.md 파일의 frontmatter에 이름·설명·인자 힌트를 정의합니다. Skill 도구를 통해 대화 중 자동으로 호출될 수도 있습니다.",
   "catDescAgents": "Agents는 복잡한 작업을 자율적으로 수행하는 서브프로세스입니다. agents/ 디렉토리에 마크다운으로 정의하며, 모델(haiku/sonnet/opus), 사용 가능 도구, 전문 영역을 설정할 수 있습니다. Agent 도구로 디스패치되어 독립적인 컨텍스트에서 실행됩니다.",
   "catDescRules": "Rules는 Claude의 동작을 제어하는 지시 파일입니다. {{configDir}}/rules/ 디렉토리에 마크다운 파일로 저장되며, 모든 세션에서 자동으로 컨텍스트에 로드됩니다. CLAUDE.md와 달리 개별 파일로 분리하여 관심사를 구분할 수 있습니다.",

package/templates/styles.css

@@ -1829,7 +1829,8 @@ body.dark .bb-tooltip .value {
 
 /* ── Top Banners (shared base) ── */
 .update-banner,
-.partial-banner {
+.partial-banner,
+.firstrun-banner {
   position: fixed;
   top: 0;
   left: var(--sidebar-width);
@@ -2055,6 +2056,31 @@ body.dark .bb-tooltip .value {
   background: #e67700;
 }
 
+.firstrun-banner {
+  gap: 12px;
+  background: #2f9e44;
+}
+
+.firstrun-banner span {
+  flex: 1;
+}
+
+.firstrun-close {
+  background: rgba(255,255,255,0.2);
+  border: 1px solid rgba(255,255,255,0.3);
+  color: #fff;
+  padding: 2px 8px;
+  border-radius: var(--radius-sm);
+  cursor: pointer;
+  font-size: 13px;
+  line-height: 1.5;
+  flex-shrink: 0;
+}
+
+.firstrun-close:hover {
+  background: rgba(255,255,255,0.35);
+}
+
 .partial-spinner {
   display: inline-block;
   width: 14px;

package/test/build.test.mjs

@@ -78,6 +78,16 @@ describe('Build', () => {
       // Test runs from the git repo, so _devBuild should be true
       assert.equal(data._devBuild, true);
     });
+
+    it('should NOT have _firstRun flag in normal --data-only build', () => {
+      // _firstRun is only set during the first full run (no cache)
+      // After that, subsequent builds must not carry the flag
+      assert.equal(data._firstRun, undefined, '_firstRun must be absent in cached builds');
+    });
+
+    it('should NOT have _partial flag in normal --data-only build', () => {
+      assert.equal(data._partial, undefined, '_partial must be absent in full builds');
+    });
   });
 
   describe('index.html', () => {

package/test/firstrun.test.mjs

@@ -0,0 +1,131 @@
+import { describe, it, before } from 'node:test';
+import assert from 'node:assert/strict';
+import fs from 'fs';
+import path from 'path';
+import { fileURLToPath } from 'url';
+
+const __dirname = path.dirname(fileURLToPath(import.meta.url));
+const ROOT = path.resolve(__dirname, '..');
+
+// ── computeDateRange (inline — not exported, so logic is replicated here) ──
+// Keep in sync with scripts/generate-dashboard.mjs:computeDateRange
+function computeDateRange(scopeData) {
+  let minTs = null, maxTs = null;
+  for (const sdata of Object.values(scopeData)) {
+    for (const entry of (sdata?.usage?.tokenEntries || [])) {
+      const ts = entry.timestamp;
+      if (!ts) continue;
+      if (!minTs || ts < minTs) minTs = ts;
+      if (!maxTs || ts > maxTs) maxTs = ts;
+    }
+  }
+  return minTs && maxTs ? { from: minTs, to: maxTs } : null;
+}
+
+describe('computeDateRange', () => {
+  it('returns null for empty scopeData', () => {
+    assert.equal(computeDateRange({}), null);
+  });
+
+  it('returns null when no tokenEntries exist', () => {
+    const scopeData = { global: { usage: { tokenEntries: [] } } };
+    assert.equal(computeDateRange(scopeData), null);
+  });
+
+  it('returns null when tokenEntries have no timestamp', () => {
+    const scopeData = { global: { usage: { tokenEntries: [{ model: 'haiku' }, { model: 'sonnet' }] } } };
+    assert.equal(computeDateRange(scopeData), null);
+  });
+
+  it('returns from === to for a single entry', () => {
+    const ts = '2025-06-15T10:00:00.000Z';
+    const scopeData = { global: { usage: { tokenEntries: [{ timestamp: ts }] } } };
+    const result = computeDateRange(scopeData);
+    assert.deepEqual(result, { from: ts, to: ts });
+  });
+
+  it('returns correct min/max across multiple entries', () => {
+    const scopeData = {
+      global: {
+        usage: {
+          tokenEntries: [
+            { timestamp: '2025-03-01T00:00:00.000Z' },
+            { timestamp: '2025-01-15T00:00:00.000Z' },
+            { timestamp: '2025-06-30T00:00:00.000Z' },
+          ],
+        },
+      },
+    };
+    const result = computeDateRange(scopeData);
+    assert.equal(result.from, '2025-01-15T00:00:00.000Z');
+    assert.equal(result.to, '2025-06-30T00:00:00.000Z');
+  });
+
+  it('aggregates across multiple scopes', () => {
+    const scopeData = {
+      global: { usage: { tokenEntries: [{ timestamp: '2025-03-01T00:00:00.000Z' }] } },
+      proj1:  { usage: { tokenEntries: [{ timestamp: '2024-11-01T00:00:00.000Z' }] } },
+      proj2:  { usage: { tokenEntries: [{ timestamp: '2025-08-20T00:00:00.000Z' }] } },
+    };
+    const result = computeDateRange(scopeData);
+    assert.equal(result.from, '2024-11-01T00:00:00.000Z');
+    assert.equal(result.to, '2025-08-20T00:00:00.000Z');
+  });
+
+  it('skips scopes with missing usage', () => {
+    const scopeData = {
+      global: { usage: { tokenEntries: [{ timestamp: '2025-05-10T00:00:00.000Z' }] } },
+      noUsage: {},
+      nullUsage: { usage: null },
+    };
+    const result = computeDateRange(scopeData);
+    assert.deepEqual(result, { from: '2025-05-10T00:00:00.000Z', to: '2025-05-10T00:00:00.000Z' });
+  });
+
+  it('skips entries with falsy timestamp', () => {
+    const scopeData = {
+      global: {
+        usage: {
+          tokenEntries: [
+            { timestamp: null },
+            { timestamp: '' },
+            { timestamp: '2025-04-04T00:00:00.000Z' },
+            { timestamp: undefined },
+          ],
+        },
+      },
+    };
+    const result = computeDateRange(scopeData);
+    assert.deepEqual(result, { from: '2025-04-04T00:00:00.000Z', to: '2025-04-04T00:00:00.000Z' });
+  });
+});
+
+// ── SKILL.md bash command validation ──
+
+describe('SKILL.md', () => {
+  let skillContent;
+  before(() => {
+    skillContent = fs.readFileSync(path.join(ROOT, 'skills', 'omh', 'SKILL.md'), 'utf-8');
+  });
+
+  it('should search plugins directory broadly (not rely on CLAUDE_PLUGIN_ROOT)', () => {
+    assert.ok(
+      skillContent.includes('plugins') && skillContent.includes('generate-dashboard.mjs'),
+      'bash command searches plugins directory for the script',
+    );
+    assert.ok(
+      !skillContent.includes('CLAUDE_PLUGIN_ROOT'),
+      'CLAUDE_PLUGIN_ROOT removed — it points to marketplace mirror, not install cache',
+    );
+  });
+
+  it('should output an error message when script is not found', () => {
+    assert.ok(skillContent.includes('ERROR'), 'error message present when script not found');
+    assert.ok(skillContent.includes('exit 1'), 'exits with non-zero on failure');
+  });
+
+  it('should not use the old broken find pattern with extra wildcard level', () => {
+    // old pattern: */oh-my-hi/*/scripts/ — extra * meant it never matched the actual install path
+    assert.ok(!skillContent.includes('oh-my-hi/*/scripts'), 'old broken pattern removed');
+  });
+});

package/test/web-ui.test.mjs

@@ -129,6 +129,21 @@ describe('Web UI — Templates', () => {
       assert.ok(js.includes('dayNames'));
       assert.ok(js.includes("dow"));
     });
+
+    it('should have showFirstRunBanner function called at boot', () => {
+      assert.ok(js.includes('function showFirstRunBanner'), 'showFirstRunBanner function defined');
+      assert.ok(js.includes('showFirstRunBanner()'), 'showFirstRunBanner called at boot');
+    });
+
+    it('should reference _firstRun and _dateRange from DATA', () => {
+      assert.ok(js.includes('DATA._firstRun'), '_firstRun flag checked');
+      assert.ok(js.includes('DATA._dateRange'), '_dateRange used for date range display');
+    });
+
+    it('should have firstrun close button inline handler', () => {
+      assert.ok(js.includes('firstrun-banner'), 'firstrun-banner class used');
+      assert.ok(js.includes('firstrun-close'), 'firstrun-close class used');
+    });
   });
 
   describe('styles.css', () => {
@@ -175,6 +190,17 @@ describe('Web UI — Templates', () => {
       assert.ok(css.includes('.cost-trend-label'));
     });
 
+    it('should have first-run completion banner styles', () => {
+      assert.ok(css.includes('.firstrun-banner'), '.firstrun-banner class defined');
+      assert.ok(css.includes('.firstrun-close'), '.firstrun-close button class defined');
+    });
+
+    it('should include firstrun-banner in shared banner base rule', () => {
+      // .firstrun-banner must share position/layout base with .update-banner and .partial-banner
+      const sharedRuleMatch = css.match(/\.update-banner[^{]*\.partial-banner[^{]*\.firstrun-banner|\.firstrun-banner[^{]*\.update-banner|\.update-banner[^{]*\.firstrun-banner/);
+      assert.ok(sharedRuleMatch, '.firstrun-banner included in shared banner base selector');
+    });
+
     it('should have responsive styles', () => {
       assert.ok(css.includes('@media'));
       assert.ok(css.includes('max-width: 768px'));
@@ -207,6 +233,7 @@ describe('Web UI — Templates', () => {
         'compareToggle', 'comparePrev',
         'unusedCleanupTipTitle', 'unusedCleanupTipDetail',
         'cacheTipLowHitTitle', 'cacheTipGoodTitle', 'cacheTipHighCreationTitle', 'cacheTipNoSessionTitle',
+        'firstRunBannerMsg', 'close',
       ];
       for (const key of requiredKeys) {
         assert.ok(en[key], `missing en key: ${key}`);