Pre-launch — Gluecron is in final validation. Public signups and git hosting for non-owner users open after launch review.
CodeIssuesDiscussionsWikiPull RequestsProjectsCommitsActionsReleasesContributorsPulse● GatesSecuritySettingsDeploymentsPipelineInsightsAgents✨ Explain✨ Ask AI✨ Workspace✨ Spec✨ Tests▓ Debt Map✨ NL Search🏛 Archaeology
claude/adoring-hopper-5x74bqclaude/affectionate-feynman-ykrf1hclaude/architecture-audit-design-wxprenclaude/build-status-update-3MXsfclaude/charming-meitner-mllb5rclaude/compare-gate-gluecron-s4mFQclaude/confident-faraday-tikcwbclaude/continue-work-XMTlIclaude/crontech-gluecron-deploy-7MIECclaude/crontech-platform-setup-SeKfwclaude/design-2026claude/ecstatic-ptolemy-jMdigclaude/enhance-github-integration-QNHdGclaude/fix-aa-loop-issue-PonMQclaude/fix-actions-and-processclaude/fix-desktop-errors-XqoW8claude/fix-red-workflowsclaude/fix-website-access-6FKJNclaude/gatetest-integration-hardeningclaude/github-audit-improvements-bDFr9claude/gluecron-launch-status-FoMRlclaude/hopeful-lamport-olfCTclaude/issue-to-pr-and-protectionsclaude/jolly-heisenberg-2sg1Qclaude/launch-preparation-QmTb6claude/new-session-xk1l7claude/plan-platform-architecture-kkN4yclaude/platform-analysis-roadmap-1nUGLclaude/platform-launch-assessment-8dWV8claude/polish-platform-release-AeDrUclaude/resume-previous-work-KzyLwclaude/review-crontech-handoff-qYEVqclaude/review-project-completeness-lHhS2claude/review-readme-docs-ulqPKclaude/serene-edison-rj87weclaude/setup-multi-repo-dev-BCwNQclaude/ship-fixes-and-tests-Jvz1cclaude/site-audit-competitive-pctlwgclaude/site-migration-vercel-XstpKclaude/standalone-product-repos-XHFTDcopilot/feat-smart-empty-states-keyboard-first-enhancementcopilot/feat-smart-morning-digest-review-context-restorecopilot/fix-and-process-workflowscopilot/update-ai-powered-code-reviewfeat/debt-mapfeat/push-policy-codeowners-hardeningfeat/smart-digest-contextfeat/stage-impactfeat/t1-secret-migrationfeat/u-polishfeat/w-self-hostfeat/w2-claude-configfix/agent-journey-orphan-sweepgatetest/auto-fix-1776586424172gatetest/auto-fix-1776586534814gatetest/auto-fix-1776590685143gatetest/auto-fix-1776590808199mainops/redeploy-retriggerstyle/dxt-cta-themeworktree-agent-a3377aad30d55da26worktree-agent-a7ef607b7ee1d6c74
voice-to-pr.test.ts13.5 KB · 396 lines
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
341
342
343
344
345
346
347
348
349
350
351
352
353
354
355
356
357
358
359
360
361
362
363
364
365
366
367
368
369
370
371
372
373
374
375
376
377
378
379
380
381
382
383
384
385
386
387
388
389
390
391
392
393
394
395
396
/**
 * Tests for src/lib/voice-to-pr.ts.
 *
 * Three concerns we can cover without standing up a DB or a real Anthropic
 * client:
 *
 *   1. `interpretVoiceTranscript` — the pure pipeline. We exercise it with
 *      a stub `client.call(prompt)` that returns canned JSON for various
 *      transcripts (spec, issue, unclear, malformed). Also covers the
 *      graceful-degrade path when ANTHROPIC_API_KEY is missing AND no stub
 *      is supplied — the function must still return a valid suggestion.
 *
 *   2. `shipAsSpec` — confirms it computes the correct
 *      `.gluecron/specs/voice-<slug>-*.md` path and calls the underlying
 *      git writer with the right shape. We mock the DB + git plumbing via
 *      module override on the global `mockGitWrite` symbol injected by the
 *      test setup. Verifies the spec body carries `status: ready`.
 *
 *   3. `createIssueFromVoice` — confirms it goes through the existing
 *      `issues` table insert and returns the new issue number. Mocked DB
 *      symbol matches the shape used by the real code so the import path
 *      stays stable.
 *
 * The DB-touching tests use module-level mock injection — we wrap the
 * imports with `mock.module` (bun:test) so the real Drizzle/Neon stack is
 * never reached.
 */

import { afterEach, beforeEach, describe, expect, it, mock } from "bun:test";

import {
  interpretVoiceTranscript,
  voiceSlug,
  normaliseInterpretation,
  buildInterpretPrompt,
  __voiceTest,
} from "../lib/voice-to-pr";

// ---------------------------------------------------------------------------
// Pure helpers
// ---------------------------------------------------------------------------

describe("voiceSlug", () => {
  it("kebab-cases the input", () => {
    expect(voiceSlug("Add dark mode toggle")).toBe("add-dark-mode-toggle");
  });
  it("trims leading/trailing hyphens", () => {
    expect(voiceSlug("--hello--")).toBe("hello");
  });
  it("caps at 40 chars", () => {
    const long = "a".repeat(80);
    expect(voiceSlug(long).length).toBeLessThanOrEqual(40);
  });
  it("falls back to 'voice-note' when empty", () => {
    expect(voiceSlug("")).toBe("voice-note");
    expect(voiceSlug("   !!!  ")).toBe("voice-note");
  });
});

describe("normaliseInterpretation", () => {
  it("returns the heuristic when raw is null", () => {
    const out = normaliseInterpretation(null, "Add dark mode");
    expect(out.kind).toBe("spec");
    expect(out.title.length).toBeGreaterThan(0);
    expect(out.body_markdown).toBe("Add dark mode");
  });

  it("rejects unknown kinds and falls back to 'unclear'", () => {
    const out = normaliseInterpretation(
      { kind: "thingy", title: "t", body_markdown: "b" },
      "noop"
    );
    expect(out.kind).toBe("unclear");
  });

  it("trims and preserves valid fields", () => {
    const out = normaliseInterpretation(
      {
        kind: "issue",
        title: "  Bug: header flickers  ",
        body_markdown: "  On Safari only.  ",
        target_repo_id_hint: "repo-123",
      },
      "fallback"
    );
    expect(out.kind).toBe("issue");
    expect(out.title).toBe("Bug: header flickers");
    expect(out.body_markdown).toBe("On Safari only.");
    expect(out.target_repo_id_hint).toBe("repo-123");
  });
});

describe("buildInterpretPrompt", () => {
  it("embeds the transcript verbatim", () => {
    const p = buildInterpretPrompt("Add dark mode", []);
    expect(p.includes("Add dark mode")).toBe(true);
    expect(p.includes('"kind"')).toBe(true);
  });
  it("includes a repo list block when repos are supplied", () => {
    const p = buildInterpretPrompt("x", [
      { id: "id-1", fullName: "alice/dashboard" },
      { id: "id-2", fullName: "alice/landing" },
    ]);
    expect(p.includes("alice/dashboard")).toBe(true);
    expect(p.includes("id-1")).toBe(true);
  });
});

describe("__voiceTest.classifyHeuristically", () => {
  const { classifyHeuristically } = __voiceTest;
  it("classifies feature requests as spec", () => {
    expect(classifyHeuristically("Add a dark mode toggle").kind).toBe("spec");
    expect(classifyHeuristically("Implement export to CSV").kind).toBe("spec");
  });
  it("classifies bug reports as issue", () => {
    expect(classifyHeuristically("Login is broken on Safari").kind).toBe("issue");
    expect(classifyHeuristically("The dashboard crashes when I click").kind).toBe("issue");
  });
  it("falls back to 'unclear' for ambiguous text", () => {
    expect(classifyHeuristically("hmm something").kind).toBe("unclear");
  });
});

// ---------------------------------------------------------------------------
// interpretVoiceTranscript — with mocked Claude
// ---------------------------------------------------------------------------

describe("interpretVoiceTranscript", () => {
  it("returns ok:false on empty transcript", async () => {
    const r = await interpretVoiceTranscript({ transcript: "" });
    expect(r.ok).toBe(false);
  });

  it("uses the injected client.call and parses spec JSON", async () => {
    const r = await interpretVoiceTranscript({
      transcript: "Add a dark mode toggle to settings",
      client: {
        call: async () =>
          JSON.stringify({
            kind: "spec",
            title: "Add dark mode toggle",
            body_markdown: "Users want a moon icon in settings.",
          }),
      },
    });
    expect(r.ok).toBe(true);
    if (!r.ok) throw new Error();
    expect(r.suggestion.kind).toBe("spec");
    expect(r.suggestion.title).toBe("Add dark mode toggle");
    expect(r.suggestion.body_markdown).toContain("moon");
  });

  it("parses issue JSON wrapped in a ```json fence", async () => {
    const r = await interpretVoiceTranscript({
      transcript: "The deploy pill keeps flashing",
      client: {
        call: async () =>
          '```json\n{"kind":"issue","title":"Deploy pill flashes","body_markdown":"Race in SSE reconnect."}\n```',
      },
    });
    expect(r.ok).toBe(true);
    if (!r.ok) throw new Error();
    expect(r.suggestion.kind).toBe("issue");
  });

  it("falls back to 'unclear' on malformed model output", async () => {
    const r = await interpretVoiceTranscript({
      transcript: "Some random utterance",
      client: { call: async () => "not json at all" },
    });
    expect(r.ok).toBe(true);
    if (!r.ok) throw new Error();
    // We don't enforce kind here — heuristic picks it. We *do* enforce
    // the title isn't empty so the UI can render something.
    expect(r.suggestion.title.length).toBeGreaterThan(0);
  });

  it("gracefully degrades to the heuristic when ANTHROPIC_API_KEY is missing", async () => {
    const before = process.env.ANTHROPIC_API_KEY;
    delete process.env.ANTHROPIC_API_KEY;
    try {
      const r = await interpretVoiceTranscript({
        transcript: "Add a settings page",
      });
      expect(r.ok).toBe(true);
      if (!r.ok) throw new Error();
      expect(r.suggestion.kind).toBe("spec");
    } finally {
      if (before) process.env.ANTHROPIC_API_KEY = before;
    }
  });

  it("returns ok:false (not throw) when the injected client throws", async () => {
    const r = await interpretVoiceTranscript({
      transcript: "stub fails",
      client: {
        call: async () => {
          throw new Error("boom");
        },
      },
    });
    expect(r.ok).toBe(false);
    if (r.ok) throw new Error();
    expect(r.error).toContain("boom");
  });
});

// ---------------------------------------------------------------------------
// shipAsSpec + createIssueFromVoice — DB + git plumbing mocks
// ---------------------------------------------------------------------------

// Track captured git writes across the suite.
const gitWrites: Array<any> = [];
const issueInserts: Array<any> = [];

// Mock the git plumbing — return success and stash the call args.
mock.module("../git/repository", () => ({
  createOrUpdateFileOnBranch: async (input: any) => {
    gitWrites.push(input);
    return { commitSha: "deadbeef", blobSha: "cafebabe", parentSha: null };
  },
}));

// Mock the DB layer with a tiny fluent stub. Only the call shapes the
// real code uses are implemented — anything else throws so we catch drift.
mock.module("../db", () => {
  // We need to vary the returned row shape per select call so both
  // `resolveRepoAndAuthor` (which uses `{repoName, defaultBranch, ownerName}`)
  // and `createIssueFromVoice` (which uses `{id, name, issueCount, ownerName}`)
  // get back rows that match the column aliases they asked for. The stub
  // remembers the aliases passed to `select()` and returns a row whose keys
  // include all of them.
  function selectStub(shape: Record<string, unknown>): any {
    const keys = Object.keys(shape || {});
    const row: Record<string, unknown> = {};
    // Fill in a sensible value per known alias.
    const aliasDefaults: Record<string, unknown> = {
      id: "repo-1",
      name: "demo",
      repoName: "demo",
      defaultBranch: "main",
      ownerName: "alice",
      issueCount: 0,
      username: "alice",
      email: "alice@example.com",
    };
    for (const k of keys) {
      row[k] = aliasDefaults[k] ?? null;
    }
    // For the user lookup (`users.username`, `users.email`), no leftJoin.
    return {
      from: (_table: any) => ({
        leftJoin: (_t: any, _on: any) => ({
          where: (_cond: any) => ({
            limit: async (_n: number) => [row],
          }),
        }),
        innerJoin: (_t: any, _on: any) => ({
          where: () => ({ limit: async () => [row] }),
        }),
        where: (_cond: any) => ({
          limit: async (_n: number) => [row],
        }),
      }),
    };
  }
  function insertStub(): any {
    return {
      values: (row: any) => ({
        returning: async () => {
          issueInserts.push(row);
          return [{ id: "issue-id", number: 42 }];
        },
      }),
    };
  }
  function updateStub(): any {
    return {
      set: (_row: any) => ({
        where: async (_cond: any) => undefined,
      }),
    };
  }
  return {
    db: {
      select: (shape?: any) => selectStub(shape || {}),
      insert: () => insertStub(),
      update: () => updateStub(),
    },
  };
});

// Re-import AFTER the mocks so the lib picks them up.
let shipAsSpec: typeof import("../lib/voice-to-pr").shipAsSpec;
let createIssueFromVoice: typeof import("../lib/voice-to-pr").createIssueFromVoice;

beforeEach(async () => {
  gitWrites.length = 0;
  issueInserts.length = 0;
  // Bun's module cache + mock.module work together: require/import returns
  // the patched module after the call above. We re-import here so each
  // suite sees the freshest mocks.
  const mod = await import("../lib/voice-to-pr");
  shipAsSpec = mod.shipAsSpec;
  createIssueFromVoice = mod.createIssueFromVoice;
});

afterEach(() => {
  // No global cleanup needed; mocks persist across tests in the suite
  // which is the desired behaviour for shipAsSpec/createIssueFromVoice.
});

describe("shipAsSpec", () => {
  it("rejects an empty transcript", async () => {
    const r = await shipAsSpec({
      repositoryId: "repo-1",
      transcript: "  ",
      userId: "user-1",
    });
    expect(r.ok).toBe(false);
  });

  it("writes to `.gluecron/specs/voice-<slug>-<ts>.md` on the default branch", async () => {
    const r = await shipAsSpec({
      repositoryId: "repo-1",
      transcript: "Add dark mode toggle to settings",
      userId: "user-1",
      interpretation: {
        kind: "spec",
        title: "Add dark mode toggle",
        body_markdown: "Users want a moon icon.",
      },
    });
    expect(r.ok).toBe(true);
    if (!r.ok) throw new Error();
    expect(r.specPath.startsWith(".gluecron/specs/voice-add-dark-mode-toggle-")).toBe(true);
    expect(r.specPath.endsWith(".md")).toBe(true);
    expect(r.branch).toBe("main");
    expect(gitWrites.length).toBe(1);
    const w = gitWrites[0];
    expect(w.owner).toBe("alice");
    expect(w.name).toBe("demo");
    expect(w.branch).toBe("main");
    // Decode the spec body and assert key invariants.
    const body = new TextDecoder().decode(w.bytes);
    expect(body.includes("status: ready")).toBe(true);
    expect(body.includes("source: voice-to-pr")).toBe(true);
    expect(body.includes("Users want a moon icon.")).toBe(true);
  });

  it("uses a heuristic title when interpretation is omitted", async () => {
    const r = await shipAsSpec({
      repositoryId: "repo-1",
      transcript: "Implement CSV export",
      userId: "user-1",
    });
    expect(r.ok).toBe(true);
    if (!r.ok) throw new Error();
    expect(r.specPath.includes("voice-implement-csv-export-")).toBe(true);
  });
});

describe("createIssueFromVoice", () => {
  it("rejects an empty transcript", async () => {
    const r = await createIssueFromVoice({
      repositoryId: "repo-1",
      transcript: "",
      userId: "user-1",
    });
    expect(r.ok).toBe(false);
  });

  it("inserts an issue row and returns the issue number", async () => {
    const r = await createIssueFromVoice({
      repositoryId: "repo-1",
      transcript: "Header flickers on Safari",
      userId: "user-1",
      interpretation: {
        kind: "issue",
        title: "Header flickers on Safari",
        body_markdown: "Repro: open dashboard on Safari 17.",
      },
    });
    expect(r.ok).toBe(true);
    if (!r.ok) throw new Error();
    expect(r.issueNumber).toBe(42);
    expect(r.ownerName).toBe("alice");
    expect(r.repoName).toBe("demo");
    expect(issueInserts.length).toBe(1);
    expect(issueInserts[0].repositoryId).toBe("repo-1");
    expect(issueInserts[0].authorId).toBe("user-1");
    expect(issueInserts[0].title).toContain("Header flickers");
    expect(issueInserts[0].body).toContain("Repro");
    expect(issueInserts[0].body.toLowerCase()).toContain("voice-to-pr");
  });
});