Revert "feat(core): Enable insights for sqlite legacy (#14606)"

This reverts commit 88ed7be.

Author: guillaumejacquart

packages/cli/src/modules/insights/__tests__/insights.pre-init.test.ts

@@ -10,13 +10,21 @@ describe('InsightsModulePreInit', () => {
 	it('should return false if instance type is not "main"', () => {
 		const ctx: ModulePreInitContext = {
 			instance: mock<InstanceSettings>({ instanceType: 'worker' }),
-			database: mock<DatabaseConfig>({ type: 'sqlite' }),
+			database: mock<DatabaseConfig>({ type: 'sqlite', sqlite: { poolSize: 10 } }),
 		};
 		expect(shouldLoadModule(ctx)).toBe(false);
 	});
 
-	it.each(['postgresdb', 'mariadb', 'mysqldb', 'sqlite'])(
-		'should return true if instance type is "main"',
+	it('should return false if database type is "sqlite" and poolSize is < 1', () => {
+		const ctx: ModulePreInitContext = {
+			instance: mock<InstanceSettings>({ instanceType: 'main' }),
+			database: mock<DatabaseConfig>({ type: 'sqlite', sqlite: { poolSize: 0 } }),
+		};
+		expect(shouldLoadModule(ctx)).toBe(false);
+	});
+
+	it.each(['postgresdb', 'mariadb', 'mysqldb'])(
+		'should return true if instance type is "main" and database is not sqlite',
 		(dbType: 'postgresdb' | 'mysqldb' | 'sqlite' | 'mariadb') => {
 			const ctx: ModulePreInitContext = {
 				instance: mock<InstanceSettings>({ instanceType: 'main' }),
@@ -25,4 +33,12 @@ describe('InsightsModulePreInit', () => {
 			expect(shouldLoadModule(ctx)).toBe(true);
 		},
 	);
+
+	it('should return true if instance type is "main" and sqlite poolSize is >= 1', () => {
+		const ctx: ModulePreInitContext = {
+			instance: mock<InstanceSettings>({ instanceType: 'main' }),
+			database: mock<DatabaseConfig>({ type: 'sqlite', sqlite: { poolSize: 1 } }),
+		};
+		expect(shouldLoadModule(ctx)).toBe(true);
+	});
 });

packages/cli/src/modules/insights/insights.pre-init.ts

@@ -3,4 +3,7 @@ import type { ModulePreInitContext } from '../modules.config';
 export const shouldLoadModule = (ctx: ModulePreInitContext) =>
 	// Only main instance(s) should collect insights
 	// Because main instances are informed of all finished workflow executions, whatever the mode
-	ctx.instance.instanceType === 'main';
+	ctx.instance.instanceType === 'main' &&
+	// This is because legacy sqlite (without pool) does not support nested transactions needed for insights
+	// TODO: remove once benchmarks confirm this issue is solved with buffering / flushing mechanism
+	(ctx.database.type !== 'sqlite' || ctx.database.sqlite.poolSize > 0);