Skip to content

Commit 4463fcc

Browse files
committed
fix(ci): allowlist dummy test key in gitleaks scan
The dummy value `sk-secret-key-1234567890` in test/tools.test.ts trips gitleaks' default `generic-api-key` rule (entropy 4.02) and fails the Secret Scan CI job on PR #1. This is a test-only fixture explicitly endorsed by AGENTS.md ("Tests use dummy values ... sk-secret-key-…"), not a real key — the value is also asserted by the redaction test at line 183, so it cannot be changed. Suppress the single line with a `gitleaks:allow` inline comment rather than weakening the rule set globally. Verified locally with gitleaks 8.24.3 (matching CI): test/ scans clean, .env (gitignored, real key) is the only remaining finding and is not tracked.
1 parent 17822e7 commit 4463fcc

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

test/tools.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ import { type AppConfig } from "../src/config.js";
88
import { DEFAULT_COMPARE_PROMPT } from "../src/prompts.js";
99
import { createServer } from "../src/server.js";
1010

11-
const SECRET_KEY = "sk-secret-key-1234567890";
11+
const SECRET_KEY = "sk-secret-key-1234567890"; // gitleaks:allow — dummy test fixture, not a real key
1212
const cfg: AppConfig = {
1313
apiKey: SECRET_KEY,
1414
model: "qwen3.7-plus",

0 commit comments

Comments
 (0)