Skip to content

fix: address low-risk repo cleanup and performance issues#845

Open
ugcodrr wants to merge 1 commit into
mainfrom
fix/low-risk-script-cleanups
Open

fix: address low-risk repo cleanup and performance issues#845
ugcodrr wants to merge 1 commit into
mainfrom
fix/low-risk-script-cleanups

Conversation

@ugcodrr
Copy link
Copy Markdown
Member

@ugcodrr ugcodrr commented May 12, 2026

Summary

  • Update the command validation script for current Node/Zod behavior so it runs successfully instead of failing on JSON import assertions or Zod v4 schema/error shapes.
  • Make maintenance scripts wait for their asynchronous Redis/Mongo work before reporting completion, which makes scheduled/manual runs more reliable and easier to debug.
  • Apply historical Redis leaderboard prefixes consistently when trimming leaderboard sizes, avoiding accidental lifetime-key trimming during historical cleanup.
  • Correct a small API key deletion confirmation bug and refresh README pnpm workspace examples to match the commands used by the repo.

Why / Benefits

  • Keeps the changes localized to developer and maintenance tooling, avoiding public API or bot command behavior changes.
  • Reduces the chance of silent maintenance script failures by replacing fire-and-forget async work with awaited operations.
  • Makes Redis limiter behavior match its existing cleanup path, which is safer for historical leaderboard maintenance.
  • Improves contributor workflow by making validation and README setup commands accurate.

Validation

  • pnpm --filter scripts lint
  • pnpm --filter scripts validate-commands
  • git diff --check
  • pnpm lint
  • pnpm test:types
  • pnpm test
  • pnpm build

@vercel
Copy link
Copy Markdown

vercel Bot commented May 12, 2026

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

Project Deployment Actions Updated (UTC)
statsify Ready Ready Preview, Comment May 12, 2026 11:48am

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant