From 6f8079f6eef13d80d99121bb5ae57b73787a78d1 Mon Sep 17 00:00:00 2001 From: eriksonssilva <43449566+eriksonssilva@users.noreply.github.com> Date: Wed, 28 May 2025 14:05:24 -0300 Subject: [PATCH] fix: allow non-UUID sessionId by changing column type to varchar (#4457) * fix: allow non-UUID sessionId by changing column type to varchar * update postgres migration * Update pnpm-lock.yaml --------- Co-authored-by: Henry Co-authored-by: Henry Heng --- packages/server/src/database/entities/Execution.ts | 2 +- ...48450230238-ModifyExecutionSessionIdFieldType.ts | 13 +++++++++++++ .../src/database/migrations/postgres/index.ts | 4 +++- 3 files changed, 17 insertions(+), 2 deletions(-) create mode 100644 packages/server/src/database/migrations/postgres/1748450230238-ModifyExecutionSessionIdFieldType.ts diff --git a/packages/server/src/database/entities/Execution.ts b/packages/server/src/database/entities/Execution.ts index 7f745123..87885cf8 100644 --- a/packages/server/src/database/entities/Execution.ts +++ b/packages/server/src/database/entities/Execution.ts @@ -18,7 +18,7 @@ export class Execution implements IExecution { agentflowId: string @Index() - @Column({ type: 'uuid' }) + @Column({ type: 'varchar' }) sessionId: string @Column({ nullable: true, type: 'text' }) diff --git a/packages/server/src/database/migrations/postgres/1748450230238-ModifyExecutionSessionIdFieldType.ts b/packages/server/src/database/migrations/postgres/1748450230238-ModifyExecutionSessionIdFieldType.ts new file mode 100644 index 00000000..43656ff7 --- /dev/null +++ b/packages/server/src/database/migrations/postgres/1748450230238-ModifyExecutionSessionIdFieldType.ts @@ -0,0 +1,13 @@ +import { MigrationInterface, QueryRunner } from 'typeorm' + +export class ModifyExecutionSessionIdFieldType1748450230238 implements MigrationInterface { + name = 'ModifyExecutionSessionIdFieldType1748450230238' + + public async up(queryRunner: QueryRunner): Promise { + await queryRunner.query(`ALTER TABLE "execution" ALTER COLUMN "sessionId" type varchar USING "sessionId"::varchar`) + } + + public async down(queryRunner: QueryRunner): Promise { + await queryRunner.query(`ALTER TABLE "execution" ALTER COLUMN "sessionId" type uuid USING "sessionId"::uuid`) + } +} diff --git a/packages/server/src/database/migrations/postgres/index.ts b/packages/server/src/database/migrations/postgres/index.ts index a711af2e..4da17daa 100644 --- a/packages/server/src/database/migrations/postgres/index.ts +++ b/packages/server/src/database/migrations/postgres/index.ts @@ -35,6 +35,7 @@ import { AddSeqNoToDatasetRow1733752119696 } from './1733752119696-AddSeqNoToDat import { AddExecutionEntity1738090872625 } from './1738090872625-AddExecutionEntity' import { FixOpenSourceAssistantTable1743758056188 } from './1743758056188-FixOpenSourceAssistantTable' import { AddErrorToEvaluationRun1744964560174 } from './1744964560174-AddErrorToEvaluationRun' +import { ModifyExecutionSessionIdFieldType1748450230238 } from './1748450230238-ModifyExecutionSessionIdFieldType' import { AddAuthTables1720230151482 } from '../../../enterprise/database/migrations/postgres/1720230151482-AddAuthTables' import { AddWorkspace1720230151484 } from '../../../enterprise/database/migrations/postgres/1720230151484-AddWorkspace' @@ -96,5 +97,6 @@ export const postgresMigrations = [ AddExecutionEntity1738090872625, FixOpenSourceAssistantTable1743758056188, AddErrorToEvaluationRun1744964560174, - ExecutionLinkWorkspaceId1746862866554 + ExecutionLinkWorkspaceId1746862866554, + ModifyExecutionSessionIdFieldType1748450230238 ]