From b9cf4368109040540a021bbf618d18aba0039baa Mon Sep 17 00:00:00 2001 From: Ricardo Espinoza Date: Tue, 11 Aug 2026 05:28:56 -0400 Subject: [PATCH] refactor(core): Add deprecation warning for N8N_WORKFLOW_TAGS_DISABLED (#35673) --- .../src/deprecation/__tests__/deprecation.service.test.ts | 4 ++++ packages/cli/src/deprecation/deprecation.service.ts | 7 +++++++ 2 files changed, 11 insertions(+) diff --git a/packages/cli/src/deprecation/__tests__/deprecation.service.test.ts b/packages/cli/src/deprecation/__tests__/deprecation.service.test.ts index 3f49eb941c1..62afc3d7e4c 100644 --- a/packages/cli/src/deprecation/__tests__/deprecation.service.test.ts +++ b/packages/cli/src/deprecation/__tests__/deprecation.service.test.ts @@ -69,6 +69,10 @@ describe('DeprecationService', () => { ['N8N_DEFAULT_BINARY_DATA_MODE', 'filesystem', false], ['N8N_EXPRESSION_ENGINE', 'legacy', true], ['N8N_EXPRESSION_ENGINE', 'vm', false], + ['N8N_WORKFLOW_TAGS_DISABLED', 'true', true], + ['N8N_WORKFLOW_TAGS_DISABLED', '1', true], + ['N8N_WORKFLOW_TAGS_DISABLED', 'false', false], + ['N8N_WORKFLOW_TAGS_DISABLED', undefined, false], ])('should detect when %s is `%s`', (envVar, value, mustWarn) => { toTest(envVar, value, mustWarn); }); diff --git a/packages/cli/src/deprecation/deprecation.service.ts b/packages/cli/src/deprecation/deprecation.service.ts index 42e6b89727d..22192a24ab5 100644 --- a/packages/cli/src/deprecation/deprecation.service.ts +++ b/packages/cli/src/deprecation/deprecation.service.ts @@ -103,6 +103,13 @@ export class DeprecationService { 'In-memory binary data storage (`default` mode) will be removed in a future version. Switch to `filesystem`, `s3`, or `database`.', checkValue: (value?: string) => value === 'default', }, + { + envVar: 'N8N_WORKFLOW_TAGS_DISABLED', + message: + 'Disabling workflow tags is deprecated. Tags will always be enabled in a future version and this environment variable will be removed, so the tags feature will become visible again after upgrading.', + checkValue: (value?: string) => + value !== undefined && ['true', '1'].includes(value.toLowerCase()), + }, { envVar: 'EXECUTIONS_PROCESS', message: SAFE_TO_REMOVE,