Skip to content

perf: speed up service startup checks#856

Open
ugcodrr wants to merge 2 commits into
mainfrom
perf/startup-command-check
Open

perf: speed up service startup checks#856
ugcodrr wants to merge 2 commits into
mainfrom
perf/startup-command-check

Conversation

@ugcodrr
Copy link
Copy Markdown
Member

@ugcodrr ugcodrr commented May 30, 2026

Summary

  • avoid deep command cache parsing/comparison on the no-change Discord startup path by using a deterministic serialized cache payload
  • stop rewriting commands.json when the cached command payload is already current
  • remove eager Mongo minPoolSize warming from the API startup path so the API does not open 20 pooled connections before it can finish booting

Verification

  • pnpm --filter @statsify/discord test:types
  • pnpm --filter @statsify/discord lint
  • pnpm --filter api test:types
  • pnpm test packages/discord/src/command/command.poster.ts --run

@vercel
Copy link
Copy Markdown

vercel Bot commented May 30, 2026

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

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
statsify Ignored Ignored Preview Jun 2, 2026 2:52am

@ugcodrr ugcodrr force-pushed the perf/startup-command-check branch from d0d73df to 0366d1a Compare May 30, 2026 11:09
@ugcodrr ugcodrr changed the base branch from guild-text-exports to main May 30, 2026 11:09
@ugcodrr ugcodrr requested a review from Copilot May 31, 2026 05:31
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

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.

3 participants