Skip to content

chore: remove unused @vercel/kv (Redis) dependency#424

Merged
tvi merged 1 commit into
mainfrom
t/rm-redis
Jun 17, 2026
Merged

chore: remove unused @vercel/kv (Redis) dependency#424
tvi merged 1 commit into
mainfrom
t/rm-redis

Conversation

@tvi

@tvi tvi commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Redis/KV was only wired into the /api/health probe and a pair of unused get/set helpers — no feature read or wrote data through it. Drop the client, the KV health check, related env vars, config, docs, and CI.

  • delete src/core/shared/clients/kv.ts and its unit test
  • simplify /api/health to only check the dashboard API
  • remove KV_REST_API_* from env schema, env-check script, and CI
  • remove @vercel/kv from package.json and lockfile
  • clean up .env.example and README

Redis/KV was only wired into the /api/health probe and a pair of unused
get/set helpers — no feature read or wrote data through it. Drop the
client, the KV health check, related env vars, config, docs, and CI.

- delete src/core/shared/clients/kv.ts and its unit test
- simplify /api/health to only check the dashboard API
- remove KV_REST_API_* from env schema, env-check script, and CI
- remove @vercel/kv from package.json and lockfile
- clean up .env.example and README
@vercel

vercel Bot commented Jun 17, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
web Ready Ready Preview, Comment Jun 17, 2026 12:20am

Request Review

@cursor

cursor Bot commented Jun 17, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
Dead-code removal with no remaining imports; health semantics change only if operators relied on KV being part of /api/health.

Overview
Removes @vercel/kv, src/core/shared/clients/kv.ts, and all KV_REST_API_* env wiring. /api/health now only reports dashboardApi; CI integration tests no longer set KV env vars. README and .env.example no longer document optional Redis setup.

No remaining references to KV clients or env keys in the repo.

Reviewed by Cursor Bugbot for commit 9716f46. Bugbot is set up for automated code reviews on this repo. Configure here.

@tvi tvi merged commit 3a2d69e into main Jun 17, 2026
14 checks passed
@tvi tvi deleted the t/rm-redis branch June 17, 2026 00:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants