Skip to content

security: remove exposed credentials and patch dependencies#1

Merged
kingxiaozhe merged 2 commits into
mainfrom
security/remove-exposed-credentials
Jul 23, 2026
Merged

security: remove exposed credentials and patch dependencies#1
kingxiaozhe merged 2 commits into
mainfrom
security/remove-exposed-credentials

Conversation

@kingxiaozhe

Copy link
Copy Markdown
Owner

Summary

  • remove hard-coded DeepSeek/SiliconFlow credentials from the API route and Vercel config
  • stop exposing ARK_API_KEY through Next.js client environment injection
  • remove tracked editor-history snapshots and ignore future .history/ output
  • add fail-closed environment handling, .env.example, and security guidance
  • update Next.js and vulnerable transitive dependencies; production audit now reports zero known vulnerabilities
  • restore strict production type checking, fix the resulting Next.js 15 route/layout issues, and add CI

Verification

  • corepack pnpm@8.15.6 install --frozen-lockfile
  • corepack pnpm@8.15.6 audit --prod — zero known vulnerabilities
  • corepack pnpm@8.15.6 exec tsc --noEmit
  • production next build with placeholder server credentials
  • Gitleaks current-tree scan — zero findings

Required owner action

Previously committed credentials must be revoked and replaced in the SiliconFlow/DeepSeek and Volcengine ARK provider consoles. This PR removes them from the current tree but intentionally does not rewrite Git history.

@vercel

vercel Bot commented Jul 23, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
comic-generator Ready Ready Preview, Comment Jul 23, 2026 10:55am

@kingxiaozhe
kingxiaozhe merged commit 8be2835 into main Jul 23, 2026
4 checks passed
@kingxiaozhe
kingxiaozhe deleted the security/remove-exposed-credentials branch July 23, 2026 10:56
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