Skip to content

chore(deps): re-land dependency upgrade (bullmq 6 schedulers, boot-tested) - #244

Merged
ralyodio merged 1 commit into
mainfrom
chore/deps-2026-10-26-v2
Oct 26, 2026
Merged

ralyodio merged 1 commit into
mainfrom
chore/deps-2026-10-26-v2

Conversation

@ralyodio

Copy link
Copy Markdown
Contributor

Re-lands #241 (reverted in #243 after nichedb.dev crash-looped on queue.getRepeatableJobs is not a function) on top of #242, which ported installSchedules to BullMQ 6 job schedulers.

New in this PR vs #241:

  • packages/queue/src/install-schedules.test.js: runs the real installSchedules twice against Redis (REDIS_TEST_URL) in a child process and checks exactly one scheduler per spec. Verified it FAILS on the chore(deps): upgrade all dependencies (bullmq 6, ioredis 6, simplewebauthn 14) #241 code with the production error.
  • Booted the production image locally (docker build + throwaway postgres:18 and redis:8, ROLES=web,worker): [queue] schedules installed, / 200, /healthz 200, /signup 200, /api/v1/me/profile 401, running 70 s+ with no errors, schedulers present in Redis.

Upgrade content (same as #241):

  • Majors: bullmq 5 -> 6 (retention.js reads queue.getBackend().client), ioredis 5 -> 6, @simplewebauthn/browser + server 13 -> 14, @profullstack/coinpay 0.8 -> 0.25 (declared, not imported)
  • hono 4.13.14, profile 0.1.6, notifications 0.1.4, throttle 0.3.4, leaderboard 0.3.1, openaccess 0.3.1, footer 0.2.6 (spec latest), biome 2.6.3 (md/yaml excluded), pglite 0.5.11
  • Overrides: sharp ^0.35.5, tmp ^0.2.6 (dev-only)
  • Bun 1.4.4 (image, packageManager, CI). No migrations added.
  • Audit: 0 critical / 5 high -> 0 / 0

Gate: biome check, bun test 2573 pass / 0 fail, queue suite vs Redis 14/14, image boot as above.

🤖 Generated with Claude Code

Re-lands #241 (reverted in #243) on top of #242, which ported installSchedules
to upsertJobScheduler/getJobSchedulers/removeJobScheduler. Adds a Redis-backed
test that runs installSchedules twice in a child process (fails on the old
getRepeatableJobs code with 'is not a function').

bullmq 6, ioredis 6, simplewebauthn 14, coinpay 0.25; retention.js reads the raw
client from queue.getBackend().client. Overrides sharp ^0.35.5, tmp ^0.2.6.
biome 2.6 migrate; md/yaml left out of biome. Bun 1.4.4.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@socket-security

Copy link
Copy Markdown

Warning

Review the following alerts detected in dependencies.

According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.

Priority Alert  (click "▶" to expand/collapse) Action
Low priority
Low adoption: npm @logicsrc/openaccess

Location: Package overview

From: apps/web/package.json → npm/@logicsrc/openaccess@0.3.1

ℹ Read more on: This package | This alert | What are unpopular packages?

Next steps: Take a moment to review the security alert above. Review the linked package source code to understand the potential risk. Ensure the package is not malicious before proceeding. If you're unsure how to proceed, reach out to your security team or ask the Socket team for help at support@socket.dev.

Suggestion: Unpopular packages may have less maintenance and contain other problems.

Mark the package as acceptable risk. To ignore this alert only in this pull request, reply with the comment @SocketSecurity ignore npm/@logicsrc/openaccess@0.3.1. You can also ignore all packages with @SocketSecurity ignore-all. To ignore an alert for all future pull requests, use Socket's Dashboard to change the triage state of this alert.

Warn

View full report

@ralyodio
ralyodio merged commit ea756dc into main Oct 26, 2026
3 checks passed
@ralyodio
ralyodio deleted the chore/deps-2026-10-26-v2 branch October 26, 2026 13:34
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