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
workflow-secrets-substitute.test.ts4.0 KB · 113 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
/**
 * Pure-helper tests for src/lib/workflow-secrets.ts substituteSecrets.
 *
 * loadSecretsContext / upsert / delete are DB-coupled so they're not
 * exercised here — the crypto round-trip lives in
 * workflow-secrets-crypto.test.ts. This file pins the pure substitution
 * grammar so the runner's secret-injection contract can be relied on
 * without instantiating Postgres.
 */

import { describe, it, expect } from "bun:test";
import { substituteSecrets } from "../lib/workflow-secrets";

describe("substituteSecrets — happy path", () => {
  it("replaces a single token with the matching plaintext", () => {
    const out = substituteSecrets(
      'echo "$TOKEN_${{ secrets.TOKEN }}"',
      { TOKEN: "abc123" }
    );
    expect(out).toBe('echo "$TOKEN_abc123"');
  });

  it("replaces multiple tokens in one template", () => {
    const out = substituteSecrets(
      "DEPLOY_KEY=${{ secrets.DEPLOY_KEY }} REGION=${{ secrets.REGION }}",
      { DEPLOY_KEY: "k1", REGION: "us-east-1" }
    );
    expect(out).toBe("DEPLOY_KEY=k1 REGION=us-east-1");
  });

  it("tolerates whitespace variants inside the braces", () => {
    const map = { X: "v" };
    expect(substituteSecrets("${{secrets.X}}", map)).toBe("v");
    expect(substituteSecrets("${{ secrets.X }}", map)).toBe("v");
    expect(substituteSecrets("${{   secrets   .   X   }}", map)).toBe("v");
  });

  it("repeats substitution when a name appears multiple times", () => {
    const out = substituteSecrets(
      "${{ secrets.X }} and again ${{ secrets.X }}",
      { X: "yes" }
    );
    expect(out).toBe("yes and again yes");
  });
});

describe("substituteSecrets — leaves tokens intact when secret is missing", () => {
  it("missing name → token unchanged (loud failure signal)", () => {
    const tpl = "echo ${{ secrets.MISSING }}";
    expect(substituteSecrets(tpl, {})).toBe(tpl);
    expect(substituteSecrets(tpl, { OTHER: "x" })).toBe(tpl);
  });

  it("substitutes the matching tokens and leaves the missing ones", () => {
    const out = substituteSecrets(
      "${{ secrets.A }} / ${{ secrets.B }} / ${{ secrets.C }}",
      { A: "a", C: "c" }
    );
    expect(out).toBe("a / ${{ secrets.B }} / c");
  });
});

describe("substituteSecrets — strict name grammar", () => {
  it("rejects lowercase names (matches GitHub Actions grammar)", () => {
    const tpl = "echo ${{ secrets.lower }}";
    expect(substituteSecrets(tpl, { lower: "x" })).toBe(tpl);
  });

  it("rejects names starting with a digit", () => {
    const tpl = "echo ${{ secrets.1ABC }}";
    expect(substituteSecrets(tpl, { "1ABC": "x" })).toBe(tpl);
  });

  it("accepts underscore-only names + names with digits", () => {
    expect(
      substituteSecrets("${{ secrets.A_B_C }}", { A_B_C: "v" })
    ).toBe("v");
    expect(
      substituteSecrets("${{ secrets._UNDER }}", { _UNDER: "v" })
    ).toBe("v");
    expect(
      substituteSecrets("${{ secrets.X1Y2 }}", { X1Y2: "v" })
    ).toBe("v");
  });
});

describe("substituteSecrets — defensive on bad input", () => {
  it("returns '' for non-string template", () => {
    expect(substituteSecrets(undefined as any, {})).toBe("");
    expect(substituteSecrets(null as any, {})).toBe("");
    expect(substituteSecrets(42 as any, {})).toBe("");
  });

  it("returns the template untouched when secrets is null/undefined", () => {
    expect(substituteSecrets("hello", null as any)).toBe("hello");
    expect(substituteSecrets("hello", undefined as any)).toBe("hello");
  });

  it("returns '' when both inputs are empty", () => {
    expect(substituteSecrets("", {})).toBe("");
  });

  it("ignores prototype-pollution probes (uses hasOwnProperty)", () => {
    const tpl = "${{ secrets.TO_STRING }}";
    // {}.toString exists on the prototype; substitution must NOT pick it up.
    expect(substituteSecrets(tpl, {} as any)).toBe(tpl);
  });

  it("does not alter unrelated `${{ ... }}` syntax (env, vars)", () => {
    const tpl = "${{ env.FOO }} ${{ vars.BAR }}";
    expect(substituteSecrets(tpl, { FOO: "v" })).toBe(tpl);
  });
});