Skip to content

fix: resolve dependency and secret alerts#351

Merged
santoshkumarradha merged 2 commits intomainfrom
codex/dependency-secret-alerts
Apr 7, 2026
Merged

fix: resolve dependency and secret alerts#351
santoshkumarradha merged 2 commits intomainfrom
codex/dependency-secret-alerts

Conversation

@santoshkumarradha
Copy link
Copy Markdown
Member

Summary

  • refresh the web client lockfiles to move vulnerable transitive glob, lodash, and yaml resolutions onto patched versions
  • pin the TypeScript SDK test toolchain to vite@^6.4.2 and vitest@^3.2.4, updating the npm lockfile away from vulnerable vite/esbuild entries while keeping the Node 18-compatible Vite 6 line
  • remove the leaked OpenRouter API key from the simulation engine example notebook and switch it to OPENROUTER_API_KEY

Verification

  • parsed the updated JSON lockfiles, package manifest, and notebook with Node
  • parsed control-plane/web/client/pnpm-lock.yaml with Ruby YAML
  • confirmed the targeted vulnerable version strings and raw sk-or-v1- secrets are absent from the touched files
  • ran git diff --check

Notes

  • full npm/pnpm install-based verification was not possible here because registry access from the sandboxed package managers timed out / was unavailable

@santoshkumarradha santoshkumarradha requested review from a team and AbirAbbas as code owners April 7, 2026 12:04
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 7, 2026

Performance

SDK Memory Δ Latency Δ Tests Status
TS 445 B +27% 3.62 µs +81%

Regression detected:

  • TypeScript memory: 350 B → 445 B (+27%)

@santoshkumarradha santoshkumarradha marked this pull request as draft April 7, 2026 12:09
@santoshkumarradha santoshkumarradha marked this pull request as ready for review April 7, 2026 13:15
@santoshkumarradha santoshkumarradha merged commit 4980751 into main Apr 7, 2026
25 checks passed
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