Pre-launch — Gluecron is in final validation. Public signups and git hosting for non-owner users open after launch review.
CodeIssuesPull RequestsActionsSecurityInsightsSettings
✨ AI
More
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-sweepfix/audit-sweep-2026-07-26gatetest/auto-fix-1776586424172gatetest/auto-fix-1776586534814gatetest/auto-fix-1776590685143gatetest/auto-fix-1776590808199mainops/redeploy-retriggerstyle/dxt-cta-themeworktree-agent-a3377aad30d55da26worktree-agent-a7ef607b7ee1d6c74
private-repo-subpage-gate.test.ts3.4 KB · 87 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
/**
 * Platform-wide private-repo gate for /:owner/:repo/... subpages.
 *
 * Every repo subpage owned its own privacy check and it drifted. A live probe
 * of a private repo on 2026-07-28 found 24 subpages returning 200 to an
 * anonymous visitor while the repo root correctly 404'd:
 *
 *   /actions /archaeology /cloud-deployments /contributors /coupling
 *   /dependencies /deployments /discussions /explain /gates /health
 *   /insights /packages /previews /projects /pushes /queue /wiki
 *   /insights/engineering /ai/changelog /ai/tests /docs/tracking
 *   /search/nl /search/semantic
 *
 * These leaked branch and workflow names, CI logs, deployment history,
 * dependency manifests and wiki content for repositories the caller could not
 * open.
 *
 * The gate is registered once in app.tsx. What must not regress: it runs
 * before the repo routers, after softAuth, denies with 404 rather than 403,
 * and falls through for anything that is not a resolvable private repo — so
 * /settings/tokens, /orgs/new and the git Smart HTTP paths are untouched.
 */

import { describe, expect, it } from "bun:test";
import { readFileSync } from "fs";

const SRC = readFileSync("src/app.tsx", "utf8");
const gate = SRC.slice(
  SRC.indexOf('app.use("/:owner/:repo/*"'),
  SRC.indexOf('app.route("/", gitRoutes)')
);

describe("gate placement", () => {
  it("is registered before every repo router", () => {
    const gateIdx = SRC.indexOf('app.use("/:owner/:repo/*"');
    expect(gateIdx).toBeGreaterThan(-1);
    // gitRoutes is the first router mounted; webRoutes the last.
    expect(gateIdx).toBeLessThan(SRC.indexOf('app.route("/", gitRoutes)'));
    expect(gateIdx).toBeLessThan(SRC.indexOf('app.route("/", webRoutes)'));
  });

  it("runs after softAuth so the viewer is resolved", () => {
    // Registered earlier and c.get("user") would always be undefined, which
    // would 404 private repos even for their own owner.
    expect(SRC.indexOf('app.use("*", softAuth)')).toBeLessThan(
      SRC.indexOf('app.use("/:owner/:repo/*"')
    );
  });
});

describe("deny behaviour", () => {
  it("denies with 404, never 403", () => {
    expect(gate).toContain('access !== "none"');
    expect(gate).toContain("404");
    expect(gate).not.toMatch(/,\s*403\s*\)/);
  });

  it("uses resolveRepoAccess so collaborators and org members still pass", () => {
    expect(gate).toContain("resolveRepoAccess");
    // An ownerId equality check would lock out collaborators.
    expect(gate).not.toMatch(/user\.id\s*!==/);
  });

  it("answers JSON callers with JSON", () => {
    expect(gate).toContain("application/json");
  });
});

describe("false positives fall through", () => {
  it("passes through when the repo does not resolve", () => {
    // /settings/tokens, /orgs/new, /:owner/:repo.git/* all resolve to null.
    expect(gate).toContain("if (!row || !row.isPrivate) return next()");
  });

  it("passes through public repos untouched", () => {
    // Same line: a non-private row short-circuits before any access lookup,
    // so public browsing costs nothing extra beyond the namespace read.
    expect(gate).toContain("!row.isPrivate");
  });

  it("fails OPEN on a namespace lookup error", () => {
    // A DB blip must not 404 the entire public site. The handler downstream
    // surfaces its own error instead.
    const cat = gate.slice(gate.indexOf("} catch {"), gate.indexOf("if (!row"));
    expect(cat).toContain("return next()");
  });
});