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
pr-triage.test.ts5.5 KB · 172 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
/**
 * Tests for src/lib/pr-triage.ts.
 *
 * Pure helpers (renderTriageComment) covered exhaustively. The
 * DB-touching paths (alreadyTriaged, loadAvailableLabels,
 * loadCandidateReviewers, buildDiffSummary) are exercised via the
 * fail-open contracts — they must return reasonable values when no
 * DB / no repo on disk, never throw.
 *
 * triggerPrTriage itself is verified to be a no-throw fire-and-forget
 * even when called with garbage inputs and no API key.
 */

import { describe, it, expect } from "bun:test";
import {
  PR_TRIAGE_MARKER,
  triggerPrTriage,
  __test,
} from "../lib/pr-triage";
import type { PrTriage } from "../lib/ai-generators";

const triageFixture = (overrides: Partial<PrTriage> = {}): PrTriage => ({
  suggestedLabels: ["bug", "ui"],
  suggestedReviewerUsernames: ["alice", "bob"],
  priority: "medium",
  riskArea: "frontend",
  summary: "Fix login form colour contrast.",
  ...overrides,
});

describe("PR_TRIAGE_MARKER", () => {
  it("is a stable HTML comment so future searches keep working", () => {
    expect(PR_TRIAGE_MARKER).toBe("<!-- gluecron-pr-triage:summary -->");
    expect(PR_TRIAGE_MARKER.startsWith("<!--")).toBe(true);
    expect(PR_TRIAGE_MARKER.endsWith("-->")).toBe(true);
  });
});

describe("renderTriageComment", () => {
  it("includes the marker, summary, priority, risk area, labels, reviewers", () => {
    const out = __test.renderTriageComment(triageFixture());
    expect(out).toContain(PR_TRIAGE_MARKER);
    expect(out).toContain("## AI Triage");
    expect(out).toContain("Fix login form colour contrast.");
    expect(out).toContain("**Priority:** medium");
    expect(out).toContain("**Risk area:** frontend");
    expect(out).toContain("`bug`");
    expect(out).toContain("`ui`");
    expect(out).toContain("@alice");
    expect(out).toContain("@bob");
  });

  it("uses an italic placeholder when there are no label suggestions", () => {
    const out = __test.renderTriageComment(triageFixture({ suggestedLabels: [] }));
    expect(out).toContain("_(no label suggestions)_");
  });

  it("uses an italic placeholder when there are no reviewer suggestions", () => {
    const out = __test.renderTriageComment(
      triageFixture({ suggestedReviewerUsernames: [] })
    );
    expect(out).toContain("_(no reviewer suggestions)_");
  });

  it("renders an italic placeholder when summary is missing/whitespace", () => {
    const a = __test.renderTriageComment(triageFixture({ summary: "" }));
    const b = __test.renderTriageComment(triageFixture({ summary: "   " }));
    expect(a).toContain("_(no summary)_");
    expect(b).toContain("_(no summary)_");
  });

  it("ends with the suggestions-only disclaimer", () => {
    const out = __test.renderTriageComment(triageFixture());
    expect(out.trimEnd().endsWith(
      "_Suggestions only — nothing has been applied. The PR author stays in control._"
    )).toBe(true);
  });

  it("formats critical priority literally (no pictographic emoji)", () => {
    // Reminder: tests guard the "no emoji" rule. We use the tighter
    // Extended_Pictographic class so legitimate punctuation (em-dash,
    // asterisks) doesn't match.
    const out = __test.renderTriageComment(
      triageFixture({ priority: "critical" })
    );
    expect(out).toContain("**Priority:** critical");
    expect(/\p{Extended_Pictographic}/u.test(out)).toBe(false);
  });
});

describe("__test fail-open contracts", () => {
  it("alreadyTriaged returns false when no DB / unknown PR", async () => {
    const out = await __test.alreadyTriaged(
      "00000000-0000-0000-0000-000000000000"
    );
    expect(out).toBe(false);
  });

  it("loadAvailableLabels returns [] for an unknown repo", async () => {
    const out = await __test.loadAvailableLabels(
      "00000000-0000-0000-0000-000000000000"
    );
    expect(Array.isArray(out)).toBe(true);
  });

  it("loadCandidateReviewers returns [] for an unknown repo", async () => {
    const out = await __test.loadCandidateReviewers(
      "00000000-0000-0000-0000-000000000000",
      "00000000-0000-0000-0000-000000000000"
    );
    expect(Array.isArray(out)).toBe(true);
  });

  it("buildDiffSummary returns '' for an unknown repo", async () => {
    const out = await __test.buildDiffSummary(
      "definitely-not-a-real-owner",
      "definitely-not-a-real-repo",
      "main",
      "feature"
    );
    expect(out).toBe("");
  });
});

describe("triggerPrTriage — fire-and-forget never throws", () => {
  const before = process.env.ANTHROPIC_API_KEY;

  it("returns cleanly when API key is absent", async () => {
    delete process.env.ANTHROPIC_API_KEY;
    try {
      let threw = false;
      try {
        await triggerPrTriage({
          ownerName: "alice",
          repoName: "demo",
          repositoryId: "00000000-0000-0000-0000-000000000000",
          prId: "00000000-0000-0000-0000-000000000000",
          prAuthorId: "00000000-0000-0000-0000-000000000000",
          title: "Test",
          body: "",
          baseBranch: "main",
          headBranch: "feature",
        });
      } catch {
        threw = true;
      }
      expect(threw).toBe(false);
    } finally {
      if (before) process.env.ANTHROPIC_API_KEY = before;
    }
  });

  it("never throws on empty/garbage inputs", async () => {
    let threw = false;
    try {
      await triggerPrTriage({
        ownerName: "",
        repoName: "",
        repositoryId: "not-a-uuid",
        prId: "not-a-uuid",
        prAuthorId: "not-a-uuid",
        title: "",
        body: "",
        baseBranch: "",
        headBranch: "",
      });
    } catch {
      threw = true;
    }
    expect(threw).toBe(false);
  });
});