From 996a239bd78a7037dade3d972ae57ad55305a13c Mon Sep 17 00:00:00 2001 From: "Aaron K. Clark" Date: Tue, 19 May 2026 08:21:17 -0500 Subject: [PATCH] chore(env): drop dead `dialect` + `define` fields from env.js exports MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit `app/config/env.js` exported two fields neither consumer reads: - `dialect: 'postgres'` — `db.config.js` (line 28) and `sequelize-cli.config.js` (line 20) both hardcode `'postgres'` inline; neither references `env.dialect`. - `define: { timestamps: false }` — both consumers override with `define: { schema: 'dbo', timestamps: true }`, so the `false` in env.js was both dead AND misleading (a reader scanning env.js could conclude the runtime has timestamps disabled when in fact PR #148 flipped both consumers to `true`). Verified via grep — nothing in app/, server.js, or tests/ touches `env.dialect` or `env.define`. Drop both. No behavior change; just stops env.js from misrepresenting the runtime. 688 tests still pass. Co-Authored-By: Claude Opus 4.7 (1M context) --- app/config/env.js | 4 ---- 1 file changed, 4 deletions(-) diff --git a/app/config/env.js b/app/config/env.js index 2f5136e..11f4ba9 100644 --- a/app/config/env.js +++ b/app/config/env.js @@ -15,10 +15,6 @@ const env = { password: process.env.DB_PASSWORD || '', host: process.env.DB_HOST || 'localhost', port: parseInt(process.env.DB_PORT, 10) || 5432, - dialect: 'postgres', - define: { - timestamps: false, - }, }; if (!env.password) {