Add 0.24.1 release notes

Author: AndriiSherman

changelogs/drizzle-kit/0.24.1.md

@@ -0,0 +1,32 @@
+## Bug fixes
+
+> Big thanks to @L-Mario564 for his [PR](https://github.com/drizzle-team/drizzle-orm/pull/2804). It conflicted in most cases with a PR that was merged, but we incorporated some of his logic. Merging it would have caused more problems and taken more time to resolve, so we just took a few things from his PR, like removing "::<type>" mappings in introspect and some array type default handlers
+
+### What was fixed
+
+1. The Drizzle Kit CLI was not working properly for the `introspect` command.
+2. Added the ability to use column names with special characters for all dialects.
+3. Included PostgreSQL sequences in the introspection process.
+4. Reworked array type introspection and added all test cases.
+5. Fixed all (we hope) default issues in PostgreSQL, where `::<type>` was included in the introspected output.
+6. `preserve` casing option was broken
+
+### Tickets that were closed
+
+- [[BUG]: invalid schema generation with drizzle-kit introspect:pg](https://github.com/drizzle-team/drizzle-orm/issues/1210)
+- [[BUG][mysql introspection]: TS error when introspect column including colon](https://github.com/drizzle-team/drizzle-orm/issues/1928)
+- [[BUG]: Unhandled defaults when introspecting postgres db](https://github.com/drizzle-team/drizzle-orm/issues/1625)
+- [[BUG]: PostgreSQL Enum Naming and Schema Typing Issue](https://github.com/drizzle-team/drizzle-orm/issues/2315)
+- [[BUG]: drizzle-kit instrospect command generates syntax error on varchar column types](https://github.com/drizzle-team/drizzle-orm/issues/2714)
+- [[BUG]: Introspecting varchar[] type produces syntactically invalid schema.ts](https://github.com/drizzle-team/drizzle-orm/issues/1633)
+- [[BUG]: introspect:pg column not using generated enum name](https://github.com/drizzle-team/drizzle-orm/issues/1648)
+- [[BUG]: drizzle-kit introspect casing "preserve" config not working](https://github.com/drizzle-team/drizzle-orm/issues/2773)
+- [[BUG]: drizzle-kit introspect fails on required param that is defined](https://github.com/drizzle-team/drizzle-orm/issues/2719)
+- [[BUG]: Error when running npx drizzle-kit introspect: "Expected object, received string"](https://github.com/drizzle-team/drizzle-orm/issues/2657)
+- [[BUG]: Missing index names when running introspect command [MYSQL]](https://github.com/drizzle-team/drizzle-orm/issues/2525)
+- [[BUG]: drizzle-kit introspect TypeError: Cannot read properties of undefined (reading 'toLowerCase')](https://github.com/drizzle-team/drizzle-orm/issues/2338)
+- [[BUG]: Wrong column name when using PgEnum.array()](https://github.com/drizzle-team/drizzle-orm/issues/2100)
+- [[BUG]: Incorrect Schema Generated when introspecting extisting pg database](https://github.com/drizzle-team/drizzle-orm/issues/1985)
+- [[⚠️🐞BUG]: index() missing argument after introspection, causes tsc error that fails the build](https://github.com/drizzle-team/drizzle-orm/issues/1870)
+- [[BUG]: drizzle-kit introspect small errors](https://github.com/drizzle-team/drizzle-orm/issues/1738)
+- [[BUG]: Missing bigint import in drizzle-kit introspect](https://github.com/drizzle-team/drizzle-orm/issues/1020)

drizzle-kit/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "drizzle-kit",
-	"version": "0.24.0",
+	"version": "0.24.1",
 	"homepage": "https://orm.drizzle.team",
 	"keywords": [
 		"drizzle",
@@ -98,7 +98,6 @@
 		"pluralize": "^8.0.0",
 		"postgres": "^3.4.4",
 		"prettier": "^2.8.1",
-		"pure-rand": "^6.1.0",
 		"semver": "^7.5.4",
 		"superjson": "^2.2.1",
 		"tsup": "^8.0.2",

drizzle-kit/src/cli/commands/utils.ts

@@ -408,7 +408,7 @@ export const preparePullConfig = async (
 			dialect: 'postgresql',
 			out: config.out,
 			breakpoints: config.breakpoints,
-			casing: config.introspectCasing,
+			casing: config.casing,
 			credentials: parsed.data,
 			tablesFilter,
 			schemasFilter,
@@ -426,7 +426,7 @@ export const preparePullConfig = async (
 			dialect: 'mysql',
 			out: config.out,
 			breakpoints: config.breakpoints,
-			casing: config.introspectCasing,
+			casing: config.casing,
 			credentials: parsed.data,
 			tablesFilter,
 			schemasFilter,
@@ -444,7 +444,7 @@ export const preparePullConfig = async (
 			dialect: 'sqlite',
 			out: config.out,
 			breakpoints: config.breakpoints,
-			casing: config.introspectCasing,
+			casing: config.casing,
 			credentials: parsed.data,
 			tablesFilter,
 			schemasFilter,

drizzle-kit/src/cli/validations/cli.ts

@@ -38,7 +38,7 @@ export const pullParams = object({
 		.optional()
 		.default(['public']),
 	extensionsFilters: literal('postgis').array().optional(),
-	introspectCasing: casing,
+	casing,
 	breakpoints: boolean().optional().default(true),
 	migrations: object({
 		prefix: prefix.optional().default('index'),

drizzle-kit/src/serializer/pgSerializer.ts

@@ -822,8 +822,8 @@ export const fromDatabase = async (
 					const columnTo: string = fk.foreign_column_name;
 					const schemaTo: string = fk.foreign_table_schema;
 					const foreignKeyName = fk.constraint_name;
-					const onUpdate = fk.update_rule.toLowerCase();
-					const onDelete = fk.delete_rule.toLowerCase();
+					const onUpdate = fk.update_rule?.toLowerCase();
+					const onDelete = fk.delete_rule?.toLowerCase();
 
 					if (typeof foreignKeysToReturn[foreignKeyName] !== 'undefined') {
 						foreignKeysToReturn[foreignKeyName].columnsFrom.push(columnFrom);
@@ -1041,7 +1041,12 @@ export const fromDatabase = async (
 					};
 
 					if (identityName) {
-						delete sequencesToReturn[`${tableSchema}.${identityName}`];
+						// remove "" from sequence name
+						delete sequencesToReturn[
+							`${tableSchema}.${
+								identityName.startsWith('"') && identityName.endsWith('"') ? identityName.slice(1, -1) : identityName
+							}`
+						];
 						delete sequencesToReturn[identityName];
 					}