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
issue-dependencies.test.ts4.1 KB · 125 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
/**
 * Block J14 — Issue dependencies. Pure helpers + route-auth smokes.
 */

import { describe, it, expect } from "bun:test";
import app from "../app";
import { __internal } from "../lib/issue-dependencies";

const { wouldCreateCycle, summariseBlockers } = __internal;

describe("issue-dependencies — wouldCreateCycle", () => {
  it("rejects self-references", () => {
    expect(wouldCreateCycle([], "a", "a")).toBe(true);
  });

  it("empty graph — no cycle", () => {
    expect(wouldCreateCycle([], "a", "b")).toBe(false);
  });

  it("detects a direct back-edge", () => {
    // b already blocks a → adding (a blocks b) would close the loop.
    const edges = [{ blockerIssueId: "b", blockedIssueId: "a" }];
    expect(wouldCreateCycle(edges, "a", "b")).toBe(true);
  });

  it("detects a transitive cycle (a → b → c, then c blocks a?)", () => {
    // Existing: a blocks b, b blocks c. Proposed: c blocks a → cycle.
    const edges = [
      { blockerIssueId: "a", blockedIssueId: "b" },
      { blockerIssueId: "b", blockedIssueId: "c" },
    ];
    expect(wouldCreateCycle(edges, "c", "a")).toBe(true);
  });

  it("allows unrelated edges", () => {
    const edges = [
      { blockerIssueId: "a", blockedIssueId: "b" },
      { blockerIssueId: "c", blockedIssueId: "d" },
    ];
    expect(wouldCreateCycle(edges, "e", "f")).toBe(false);
    expect(wouldCreateCycle(edges, "a", "c")).toBe(false);
  });

  it("allows adding an edge that does not close any path", () => {
    // a blocks b. Adding b blocks c is fine — no cycle.
    const edges = [{ blockerIssueId: "a", blockedIssueId: "b" }];
    expect(wouldCreateCycle(edges, "b", "c")).toBe(false);
  });

  it("detects deeply transitive cycle (length 4)", () => {
    // a → b → c → d, then proposing d blocks a → cycle.
    const edges = [
      { blockerIssueId: "a", blockedIssueId: "b" },
      { blockerIssueId: "b", blockedIssueId: "c" },
      { blockerIssueId: "c", blockedIssueId: "d" },
    ];
    expect(wouldCreateCycle(edges, "d", "a")).toBe(true);
  });

  it("diamond shapes do not count as cycles", () => {
    // a blocks b, a blocks c, b blocks d, c blocks d. Adding e blocks a is fine.
    const edges = [
      { blockerIssueId: "a", blockedIssueId: "b" },
      { blockerIssueId: "a", blockedIssueId: "c" },
      { blockerIssueId: "b", blockedIssueId: "d" },
      { blockerIssueId: "c", blockedIssueId: "d" },
    ];
    expect(wouldCreateCycle(edges, "e", "a")).toBe(false);
  });
});

describe("issue-dependencies — summariseBlockers", () => {
  it("returns zeros for empty input", () => {
    expect(summariseBlockers([])).toEqual({ open: 0, closed: 0, total: 0 });
  });

  it("counts open and closed blockers", () => {
    expect(
      summariseBlockers([
        { blockerIssueId: "a", blockerState: "open" },
        { blockerIssueId: "b", blockerState: "closed" },
        { blockerIssueId: "c", blockerState: "open" },
      ])
    ).toEqual({ open: 2, closed: 1, total: 3 });
  });

  it("treats any non-open state as closed (defensive)", () => {
    expect(
      summariseBlockers([
        { blockerIssueId: "a", blockerState: "merged" },
        { blockerIssueId: "b", blockerState: "open" },
      ])
    ).toEqual({ open: 1, closed: 1, total: 2 });
  });
});

describe("issue-dependencies — routes", () => {
  it("POST /:o/:r/issues/:n/dependencies requires auth", async () => {
    const res = await app.request(
      "/alice/nope/issues/1/dependencies",
      { method: "POST", body: "blockerNumber=2" }
    );
    expect([302, 401].includes(res.status)).toBe(true);
  });

  it("POST remove route requires auth", async () => {
    const res = await app.request(
      "/alice/nope/issues/1/dependencies/blockers/xyz/remove",
      { method: "POST" }
    );
    expect([302, 401].includes(res.status)).toBe(true);
  });

  it("POST with invalid bearer → 401 JSON", async () => {
    const res = await app.request(
      "/alice/nope/issues/1/dependencies",
      {
        method: "POST",
        headers: { authorization: "Bearer glc_garbage" },
        body: "blockerNumber=2",
      }
    );
    expect(res.status).toBe(401);
  });
});