fix(convex): corrigir memory leak com .collect() sem limite e adicionar otimizacoes
Problema: Convex backend consumindo 16GB+ de RAM causando OOM kills Correcoes aplicadas: - Substituido todos os .collect() por .take(LIMIT) em 27+ arquivos - Adicionado indice by_usbPolicyStatus para otimizar query de maquinas - Corrigido N+1 problem em alerts.ts usando Map lookup - Corrigido full table scan em usbPolicy.ts - Corrigido subscription leaks no frontend (tickets-view, use-ticket-categories) - Atualizado versao do Convex backend para precompiled-2025-12-04-cc6af4c Arquivos principais modificados: - convex/*.ts - limites em todas as queries .collect() - convex/schema.ts - novo indice by_usbPolicyStatus - convex/alerts.ts - N+1 fix com Map - convex/usbPolicy.ts - uso do novo indice - src/components/tickets/tickets-view.tsx - skip condicional - src/hooks/use-ticket-categories.ts - skip condicional 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This commit is contained in:
parent
a4b46b08ba
commit
638faeb287
33 changed files with 139 additions and 128 deletions
|
|
@ -81,7 +81,7 @@ async function findActiveMachineToken(ctx: QueryCtx, machineId: Id<"machines">,
|
|||
.withIndex("by_machine_revoked_expires", (q) =>
|
||||
q.eq("machineId", machineId).eq("revoked", false).gt("expiresAt", now),
|
||||
)
|
||||
.collect()
|
||||
.take(100)
|
||||
|
||||
return tokens.length > 0 ? tokens[0]! : null
|
||||
}
|
||||
|
|
@ -550,7 +550,7 @@ export const register = mutation({
|
|||
const candidates = await ctx.db
|
||||
.query("machines")
|
||||
.withIndex("by_tenant_hostname", (q) => q.eq("tenantId", tenantId).eq("hostname", args.hostname))
|
||||
.collect()
|
||||
.take(200)
|
||||
// Procura uma maquina com hostname igual E hardware compativel (MAC ou serial)
|
||||
for (const candidate of candidates) {
|
||||
if (matchesExistingHardware(candidate, identifiers, args.hostname)) {
|
||||
|
|
@ -643,7 +643,7 @@ export const register = mutation({
|
|||
const previousTokens = await ctx.db
|
||||
.query("machineTokens")
|
||||
.withIndex("by_machine", (q) => q.eq("machineId", machineId))
|
||||
.collect()
|
||||
.take(100)
|
||||
|
||||
for (const token of previousTokens) {
|
||||
if (!token.revoked) {
|
||||
|
|
@ -932,7 +932,7 @@ export const listByTenant = query({
|
|||
const tenantCompanies = await ctx.db
|
||||
.query("companies")
|
||||
.withIndex("by_tenant", (q) => q.eq("tenantId", tenantId))
|
||||
.collect()
|
||||
.take(200)
|
||||
|
||||
const companyById = new Map<string, typeof tenantCompanies[number]>()
|
||||
const companyBySlug = new Map<string, typeof tenantCompanies[number]>()
|
||||
|
|
@ -1574,7 +1574,7 @@ export const listMachineRequesters = query({
|
|||
const tickets = await ctx.db
|
||||
.query("tickets")
|
||||
.withIndex("by_tenant_machine", (q) => q.eq("tenantId", machine.tenantId).eq("machineId", args.machineId))
|
||||
.collect()
|
||||
.take(200)
|
||||
|
||||
const requestersMap = new Map<string, { email: string; name: string | null }>()
|
||||
for (const ticket of tickets) {
|
||||
|
|
@ -2131,7 +2131,7 @@ export const resetAgent = mutation({
|
|||
const tokens = await ctx.db
|
||||
.query("machineTokens")
|
||||
.withIndex("by_machine", (q) => q.eq("machineId", machineId))
|
||||
.collect()
|
||||
.take(100)
|
||||
|
||||
const now = Date.now()
|
||||
let revokedCount = 0
|
||||
|
|
@ -2640,7 +2640,7 @@ export const remove = mutation({
|
|||
const tokens = await ctx.db
|
||||
.query("machineTokens")
|
||||
.withIndex("by_machine", (q) => q.eq("machineId", machineId))
|
||||
.collect()
|
||||
.take(100)
|
||||
|
||||
await Promise.all(tokens.map((token) => ctx.db.delete(token._id)))
|
||||
await ctx.db.delete(machineId)
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue