@hover-dev/core 0.10.0 → 0.11.0

package/dist/service.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"service.d.ts","sourceRoot":"","sources":["../src/service.ts"],"names":[],"mappings":"AAqEA,OAAO,EAEL,KAAK,mBAAmB,EAEzB,MAAM,iBAAiB,CAAC;AAEzB,MAAM,WAAW,cAAc;IAC7B,IAAI,EAAE,MAAM,CAAC;IACb,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,gFAAgF;IAChF,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB;;;6EAGyE;IACzE,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB;;;;uBAImB;IACnB,OAAO,CAAC,EAAE,mBAAmB,EAAE,CAAC;CACjC;AAED,MAAM,WAAW,aAAa;IAC5B;4EACwE;IACxE,IAAI,EAAE,MAAM,CAAC;IACb,KAAK,IAAI,OAAO,CAAC,IAAI,CAAC,CAAC;CACxB;AAiDD,wBAAsB,YAAY,CAAC,IAAI,EAAE,cAAc,GAAG,OAAO,CAAC,aAAa,CAAC,CAmrB/E"}
+{"version":3,"file":"service.d.ts","sourceRoot":"","sources":["../src/service.ts"],"names":[],"mappings":"AA6EA,OAAO,EAEL,KAAK,mBAAmB,EAEzB,MAAM,iBAAiB,CAAC;AAEzB,MAAM,WAAW,cAAc;IAC7B,IAAI,EAAE,MAAM,CAAC;IACb,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,gFAAgF;IAChF,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB;;;6EAGyE;IACzE,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB;;;;uBAImB;IACnB,OAAO,CAAC,EAAE,mBAAmB,EAAE,CAAC;CACjC;AAED,MAAM,WAAW,aAAa;IAC5B;4EACwE;IACxE,IAAI,EAAE,MAAM,CAAC;IACb,KAAK,IAAI,OAAO,CAAC,IAAI,CAAC,CAAC;CACxB;AAiDD,wBAAsB,YAAY,CAAC,IAAI,EAAE,cAAc,GAAG,OAAO,CAAC,aAAa,CAAC,CAswB/E"}

package/dist/service.js

@@ -13,6 +13,7 @@
  *     { type: 'skill-saved',     payload: { name, path } }
  *     { type: 'skill-exists',    payload: { slug, existingPath } }
  *     { type: 'skills-list',     payload: { skills: SkillSummary[] } }
+ *     { type: 'specs-list',      payload: { specs: SpecSummary[] } }
  *     { type: 'spec-saved',      payload: { name, path } }
  *     { type: 'spec-exists',     payload: { slug, existingPath } }
  *     { type: 'case-csv-saved',  payload: { name, path } }
@@ -20,7 +21,12 @@
  *     { type: 'error',           payload: { message } }
  *
  *   client → server
- *     { type: 'command',       payload: { text, sessionId? } }
+ *     { type: 'command',       payload: { text, sessionId?, reRecord?: { slug } } }
+ *                                                  // when reRecord.slug is set, the
+ *                                                  // service collects tool_use events
+ *                                                  // into a step list and on a clean
+ *                                                  // session_end overwrites
+ *                                                  // __vibe_tests__/<slug>.spec.ts
  *     { type: 'cancel' }
  *     { type: 'check-cdp',     payload: { pageUrl } }                 // "is this widget in the debug Chrome?"
  *     { type: 'launch-chrome', payload: { pageUrl } }                 // start debug Chrome, navigate to pageUrl
@@ -29,6 +35,7 @@
  *     { type: 'save-spec',     payload: { name, description, steps, assertions?, overwrite? } }
  *     { type: 'save-case-csv', payload: { name, description, steps, assertions?, jiraProjectKey?, labels?, overwrite? } }
  *     { type: 'list-skills' }
+ *     { type: 'list-specs' }                                            // ask for every spec under __vibe_tests__/, with parsed JSDoc headers
  *     { type: 'list-agents' }                                          // ask for the full agent registry + install status
  *     { type: 'switch-agent',  payload: { agentId } }                  // set the service's current agent; broadcasts to all connections
  *
@@ -48,6 +55,7 @@ import { getAgent } from './agents/registry.js';
 import { getPreflight, invalidatePreflight } from './playwright/preflightCache.js';
 import { resolveMcpConfig } from './playwright/resolveMcpConfig.js';
 import { listSkills } from './skills/writeSkill.js';
+import { listSpecs } from './specs/listSpecs.js';
 import { send, sendIfOpen } from './service/types.js';
 import { buildCdpHint, buildCdpHintResume } from './service/cdpHint.js';
 import { handleCheckCdp, handleLaunchChrome, handleFocusDebug, } from './service/cdpHandlers.js';
@@ -525,6 +533,15 @@ export async function startService(opts) {
                 send(ws, { type: 'skills-list', payload: { skills } });
                 return;
             }
+            if (msg.type === 'list-specs') {
+                // Widget asks for every spec under <devRoot>/__vibe_tests__/ so it
+                // can render the Specs tab in the Saved-sessions overlay. Each
+                // summary carries `originalPrompt` (parsed from the JSDoc header)
+                // so the Re-record button can resubmit it as a normal command.
+                const specs = await listSpecs(devRoot);
+                send(ws, { type: 'specs-list', payload: { specs } });
+                return;
+            }
             if (msg.type === 'save-spec') {
                 await handleSaveArtifact(ws, msg, devRoot, SPEC_CONFIG);
                 return;
@@ -551,6 +568,15 @@ export async function startService(opts) {
             const resumeSessionId = typeof msg.payload?.sessionId === 'string' && msg.payload.sessionId.length > 0
                 ? msg.payload.sessionId
                 : undefined;
+            // Re-record mode: when the client (widget Specs tab or hover CLI)
+            // passes `reRecord: { slug }`, we collect tool_use events server-side
+            // into a SkillStep[] and, on session_end with no error, overwrite the
+            // existing __vibe_tests__/<slug>.spec.ts. This is the same flow the
+            // widget uses for "Save as Spec", but the spec already exists and is
+            // being regenerated for the current UI.
+            const reRecordSlug = msg.payload && typeof msg.payload === 'object' && 'reRecord' in msg.payload
+                ? msg.payload.reRecord?.slug
+                : undefined;
             if (typeof text !== 'string' || !text.trim())
                 return;
             if (busy) {
@@ -563,6 +589,16 @@ export async function startService(opts) {
             busy = true;
             cancelled = false;
             inflight = new AbortController();
+            // Re-record step collector — populated as tool_use events stream by,
+            // consumed at session_end to overwrite the original spec. Empty unless
+            // reRecordSlug is set on this command. We seed with a synthetic
+            // `user` step so writeSpec's JSDoc Original-prompt: line carries the
+            // text the agent was actually given (which is the prompt we read out
+            // of the existing spec — the same one we're regenerating).
+            const reRecordSteps = [];
+            if (reRecordSlug) {
+                reRecordSteps.push({ kind: 'user', text });
+            }
             try {
                 // Build the MCP config first — it's pure local file IO and lets
                 // us assert plugin-contributed servers landed in the config even
@@ -684,6 +720,61 @@ export async function startService(opts) {
                     if (cancelled || ws.readyState !== WebSocket.OPEN)
                         return;
                     send(ws, { type: 'event', payload: ev });
+                    // Re-record collection. Mirror what widget client.js does on the
+                    // way past tool_use events: accumulate into a SkillStep[] so we
+                    // can write a fresh spec when the session ends. We do this only
+                    // when this command was launched in re-record mode; ordinary
+                    // commands don't need server-side step retention (widget owns
+                    // that for normal saves).
+                    if (reRecordSlug && ev.kind === 'tool_use') {
+                        reRecordSteps.push({
+                            kind: 'step',
+                            tool: ev.tool,
+                            input: ev.input,
+                        });
+                    }
+                    if (reRecordSlug && ev.kind === 'session_end') {
+                        // Cancelled or errored runs: don't overwrite — the existing
+                        // spec is still valid. Tell the client what happened.
+                        if (ev.isError) {
+                            sendIfOpen(ws, {
+                                type: 'error',
+                                payload: {
+                                    message: `Re-record failed: ${ev.summary ?? 'agent reported an error'}. ` +
+                                        `Original spec left unchanged.`,
+                                },
+                            });
+                        }
+                        else {
+                            // Snapshot the agent's final summary into a synthetic `done`
+                            // step so writeSpec's `Outcome:` header reflects the new run.
+                            if (ev.summary) {
+                                reRecordSteps.push({ kind: 'done', summary: ev.summary });
+                            }
+                            // Overwrite. writeSpec uses the slug to name the file; we
+                            // pass the original slug verbatim so the path is stable.
+                            try {
+                                const { writeSpec } = await import('./specs/writeSpec.js');
+                                const result = await writeSpec({
+                                    devRoot,
+                                    name: reRecordSlug,
+                                    steps: reRecordSteps,
+                                    overwrite: true,
+                                });
+                                sendIfOpen(ws, {
+                                    type: 'spec-saved',
+                                    payload: { name: reRecordSlug, path: result.path },
+                                });
+                            }
+                            catch (e) {
+                                const m = e instanceof Error ? e.message : String(e);
+                                sendIfOpen(ws, {
+                                    type: 'error',
+                                    payload: { message: `Re-record could not write spec: ${m}` },
+                                });
+                            }
+                        }
+                    }
                 }
             }
             catch (err) {

package/dist/specs/listSpecs.d.ts

@@ -0,0 +1,40 @@
+export interface SpecSummary {
+    /** Path-relative slug, e.g. `login-and-counter`. Identifies the spec. */
+    slug: string;
+    /** Absolute path to the .spec.ts file. */
+    path: string;
+    /** `Original prompt:` parsed from the JSDoc header. `null` for
+     *  hand-authored specs that have no header — they list but can't be
+     *  re-recorded automatically. */
+    originalPrompt: string | null;
+    /** First line of `Outcome:` from the JSDoc header, if present. */
+    outcome: string | null;
+    /** Number of `Steps:` lines parsed (informational only). */
+    stepCount: number;
+    /** File mtime in ms — used to show "saved 2 hours ago" in the UI. */
+    mtimeMs: number;
+}
+export interface SpecHeader {
+    /** Raw text of `Original prompt:` line, or null when absent. */
+    originalPrompt: string | null;
+    /** First line of `Outcome:`. */
+    outcome: string | null;
+    /** Step lines from the `Steps:` block, in order. */
+    steps: string[];
+    /** Lines from the `Expected:` block, in order. */
+    expected: string[];
+}
+/**
+ * Parse the JSDoc header that `writeSpec.ts` emits. Tolerant of:
+ *   - Specs without any JSDoc (returns all-null).
+ *   - Hand-edited specs where users reordered or trimmed sections.
+ *   - Long prompts that wrap across lines (we take only the first line).
+ */
+export declare function parseSpecHeader(source: string): SpecHeader;
+/**
+ * List every `*.spec.ts` file under `<devRoot>/__vibe_tests__/` with its
+ * parsed header. Returns newest-first by mtime so the widget overlay shows
+ * recently-saved specs at the top.
+ */
+export declare function listSpecs(devRoot: string): Promise<SpecSummary[]>;
+//# sourceMappingURL=listSpecs.d.ts.map

package/dist/specs/listSpecs.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"listSpecs.d.ts","sourceRoot":"","sources":["../../src/specs/listSpecs.ts"],"names":[],"mappings":"AAkBA,MAAM,WAAW,WAAW;IAC1B,yEAAyE;IACzE,IAAI,EAAE,MAAM,CAAC;IACb,0CAA0C;IAC1C,IAAI,EAAE,MAAM,CAAC;IACb;;qCAEiC;IACjC,cAAc,EAAE,MAAM,GAAG,IAAI,CAAC;IAC9B,kEAAkE;IAClE,OAAO,EAAE,MAAM,GAAG,IAAI,CAAC;IACvB,4DAA4D;IAC5D,SAAS,EAAE,MAAM,CAAC;IAClB,qEAAqE;IACrE,OAAO,EAAE,MAAM,CAAC;CACjB;AAED,MAAM,WAAW,UAAU;IACzB,gEAAgE;IAChE,cAAc,EAAE,MAAM,GAAG,IAAI,CAAC;IAC9B,gCAAgC;IAChC,OAAO,EAAE,MAAM,GAAG,IAAI,CAAC;IACvB,oDAAoD;IACpD,KAAK,EAAE,MAAM,EAAE,CAAC;IAChB,kDAAkD;IAClD,QAAQ,EAAE,MAAM,EAAE,CAAC;CACpB;AAED;;;;;GAKG;AACH,wBAAgB,eAAe,CAAC,MAAM,EAAE,MAAM,GAAG,UAAU,CAsB1D;AA4BD;;;;GAIG;AACH,wBAAsB,SAAS,CAAC,OAAO,EAAE,MAAM,GAAG,OAAO,CAAC,WAAW,EAAE,CAAC,CAkCvE"}

package/dist/specs/listSpecs.js

@@ -0,0 +1,114 @@
+/**
+ * List + parse Hover-generated Playwright specs under `<devRoot>/__vibe_tests__/`.
+ *
+ * Used by:
+ *   - The widget's "Specs" overlay tab (server pushes a SpecSummary[] list).
+ *   - The CLI's `hover re-record <spec>` subcommand (parses one spec for its
+ *     `Original prompt:` JSDoc header).
+ *
+ * Hand-authored specs (no Hover JSDoc header) are listed but reported with
+ * `originalPrompt: null` — the UI / CLI surfaces that "this spec can't be
+ * re-recorded automatically; the natural-language intent isn't recorded."
+ *
+ * Mirrors the listSkills shape so widget UI can use the same row renderer.
+ */
+import { readdir, readFile } from 'node:fs/promises';
+import { stat } from 'node:fs/promises';
+import { join } from 'node:path';
+/**
+ * Parse the JSDoc header that `writeSpec.ts` emits. Tolerant of:
+ *   - Specs without any JSDoc (returns all-null).
+ *   - Hand-edited specs where users reordered or trimmed sections.
+ *   - Long prompts that wrap across lines (we take only the first line).
+ */
+export function parseSpecHeader(source) {
+    // JSDoc block right after the @playwright/test import (or at file top).
+    // We don't require it to be the very first JSDoc — there could be a
+    // banner comment from a linter. We DO require it to appear before the
+    // first `test(` / `test.describe(` so that long file footers can't
+    // confuse the parser.
+    const beforeFirstTest = source.split(/^\s*(?:test|test\.describe)\s*\(/m)[0] ?? source;
+    const blockMatch = beforeFirstTest.match(/\/\*\*([\s\S]*?)\*\//);
+    if (!blockMatch) {
+        return { originalPrompt: null, outcome: null, steps: [], expected: [] };
+    }
+    const block = blockMatch[1];
+    const originalPrompt = extractScalar(block, /^\s*\*\s*Original prompt:\s*(.+?)\s*$/m);
+    const outcome = extractScalar(block, /^\s*\*\s*Outcome:\s*(.+?)\s*$/m);
+    return {
+        originalPrompt,
+        outcome,
+        steps: extractList(block, /^\s*\*\s*Steps:\s*$/m),
+        expected: extractList(block, /^\s*\*\s*Expected:\s*$/m),
+    };
+}
+function extractScalar(block, re) {
+    const m = block.match(re);
+    return m ? m[1].trim() : null;
+}
+/**
+ * Extract a JSDoc list-style block. Given a header regex matching "Steps:"
+ * or "Expected:", read subsequent ` * <indented line>` lines until the next
+ * top-level marker (blank ` *` line or another `Section:` header).
+ */
+function extractList(block, headerRe) {
+    const match = block.match(headerRe);
+    if (!match)
+        return [];
+    const start = (match.index ?? 0) + match[0].length;
+    const tail = block.slice(start);
+    const lines = [];
+    for (const raw of tail.split('\n')) {
+        // Stop at a blank JSDoc line (` *` only) or another `Section:` header.
+        if (/^\s*\*\s*$/.test(raw))
+            break;
+        if (/^\s*\*\s*\w[\w ]*:\s*$/.test(raw) || /^\s*\*\s*\w[\w ]*:\s/.test(raw))
+            break;
+        const m = raw.match(/^\s*\*\s*(?:[•\-\*\d.]\s*)*(.+?)\s*$/);
+        if (m && m[1])
+            lines.push(m[1]);
+    }
+    return lines;
+}
+/**
+ * List every `*.spec.ts` file under `<devRoot>/__vibe_tests__/` with its
+ * parsed header. Returns newest-first by mtime so the widget overlay shows
+ * recently-saved specs at the top.
+ */
+export async function listSpecs(devRoot) {
+    const root = join(devRoot, '__vibe_tests__');
+    let entries;
+    try {
+        entries = await readdir(root);
+    }
+    catch {
+        return [];
+    }
+    const summaries = [];
+    for (const entry of entries) {
+        if (!entry.endsWith('.spec.ts'))
+            continue;
+        const path = join(root, entry);
+        let content;
+        let mtimeMs = 0;
+        try {
+            content = await readFile(path, 'utf-8');
+            const st = await stat(path);
+            mtimeMs = st.mtimeMs;
+        }
+        catch {
+            continue;
+        }
+        const header = parseSpecHeader(content);
+        summaries.push({
+            slug: entry.replace(/\.spec\.ts$/, ''),
+            path,
+            originalPrompt: header.originalPrompt,
+            outcome: header.outcome,
+            stepCount: header.steps.length,
+            mtimeMs,
+        });
+    }
+    summaries.sort((a, b) => b.mtimeMs - a.mtimeMs);
+    return summaries;
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hover-dev/core",
-  "version": "0.10.0",
+  "version": "0.11.0",
   "description": "Hover's local Node service: agent invocation, Playwright CDP preflight, WebSocket bridge.",
   "license": "Apache-2.0",
   "author": "Hyperyond",