Skip to content

feat: node-redis instrumentation; Prisma, Sequelize, Knex, node-redis in the driver matrix - #22

Merged
thelastbackspace merged 1 commit into
mainfrom
feat/more-drivers
Nov 7, 2026
Merged

thelastbackspace merged 1 commit into
mainfrom
feat/more-drivers

Conversation

@thelastbackspace

Copy link
Copy Markdown
Owner
  • instrumentNodeRedis(tp, require('redis')) for node-redis 4 and 5 (redis GET; MULTI and pipeline commands tracked one by one); auto-instrumented as the redis target (also @redis/client). v4 legacyMode is not covered.
  • sqlLabel: CREATE TABLE IF NOT EXISTS "Users" (Sequelize sync()) was CREATE IF, now CREATE Users; same for DROP TABLE IF EXISTS.
  • test/compat: redis 4/5, Prisma 5/6 on SQLite (each in its own dir, installed + generated by a postinstall prisma-setup.js), Sequelize 6 and Knex 2/3 on pg and mysql2, exact-label assertions. Prisma 7 skipped: SQLite needs a driver adapter + TS-only generator.
  • Unit tests for node-redis 4 and 5 against a fake RESP server; sqlLabel cases for Sequelize/Knex SQL.

…dis in the driver matrix

- instrumentNodeRedis for redis / @redis/client 4 and 5 (MULTI and pipeline
  commands tracked one by one), auto target 'redis'
- sqlLabel: CREATE/DROP TABLE IF [NOT] EXISTS are labelled by table
- test/compat: redis 4/5, Prisma 5/6 on SQLite (prisma-setup.js), Sequelize 6
  and Knex 2/3 on pg and mysql2
@thelastbackspace
thelastbackspace merged commit 54dce34 into main Nov 7, 2026
15 checks passed
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