Skip to content

test: avoid secret-like fixture keys#2

Merged
flowylife merged 1 commit into
mainfrom
codex/remove-secret-like-test-keys
Jul 1, 2026
Merged

test: avoid secret-like fixture keys#2
flowylife merged 1 commit into
mainfrom
codex/remove-secret-like-test-keys

Conversation

@flowylife

@flowylife flowylife commented Jul 1, 2026

Copy link
Copy Markdown
Owner

Purpose

Avoid secret-scanner false positives before public repository sharing by replacing test fixture strings that looked like real API keys.

Changes

  • Replace sk-shaped fixture values with non-secret-looking redacted placeholders.
  • Preserve the tests that verify API key handling and provider switching behavior.

Verification

  • npm run compile && node --test test/ai-provider.test.mjs test/ui-project-lifecycle.test.mjs
  • Secret-like token grep returned no matches for common token patterns.
  • git diff --check

Risk Areas

  • DB schema changed: No
  • API contract changed: No
  • Auth/permission changed: No
  • UI layout changed: No
  • State management changed: No
  • Data import/export changed: No
  • Build/deployment config changed: No
  • Test coverage changed: No

Merge Note

This PR is safe to merge after CI passes.

@flowylife
flowylife merged commit 4e401b1 into main Jul 1, 2026
2 checks passed
@flowylife
flowylife deleted the codex/remove-secret-like-test-keys branch July 1, 2026 14:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant