ReviewOS

also looking at this

stacks/bun-query-builder

chore: drop imports nothing uses

#1156
Merged glennmichael123 wants to merge chore/remove-unused-imports into main
9 files +4 -18
8 days ago

Nineteen imports across nine files that nothing references.

pickier 0.1.65 adds a no-unused-imports rule and reports every one as an error, so #1139's tooling bump fails lint until these go. They are dead on their own terms regardless — the rule only made them visible.

filedropped
src/dynamodb-client.tsDynamoDBBatchGetItemParams, DynamoDBBatchWriteItemParams, DynamoDBDeleteItemParams, DynamoDBGetItemParams, DynamoDBTransactWriteParams
src/dynamodb-single-table.tsDynamoDBConfig, DynamoDBDriver, SingleTableEntityMapping, createDynamoDBDriver
src/dynamodb/migration-driver.tshashTableDefinition
src/model.tsisBrowser
src/client.tsDatabaseSchema (AnyDatabaseSchema stays; the name survives only in a comment)
src/actions/db-wipe.ts, db-optimize.tsnode:process
src/actions/migrate-status.tsdirname, join, node:process
src/actions/migrate-rollback.tsdirname, node:process (join is used)

Each was checked for a second reference in its file before removal, including re-exports; createDynamoDBDriver was the only value import among them, and nothing in dynamodb-single-table.ts calls it.

Locally: full suite 2246 pass / 0 fail / 10 skip, tsc clean, bun run build fine with public-surface-types and source-hygiene passing, pickier . unchanged at its 330-warning baseline, and pickier@0.1.65 . down from 19 errors to 0.

🤖 Generated with Claude Code

9 changed files on the files tab, with 0 review threads.

Sign in to comment on this pull request.