fix(core): Restore old names for pruning config keys

[ivov]

Summary

#11507 renamed the poorly named pruning config keys in the schema object, but it turns out the internal structure of the schema object is being relied on externally via N8N_CONFIG_FILES, so this PR renames back those keys until we can deprecate and drop this env var in v2.

Related Linear tickets, Github issues, and Community forum posts

8gears/n8n-helm-chart#125

Review / Merge checklist

• PR title and summary are descriptive. (conventions)
• Docs updated or follow-up ticket created.
• Tests included.
• PR Labeled with release/backport (if the PR is an urgent fix that needs to be backported)

[codecov]

Codecov Report

Attention: Patch coverage is 89.47368% with 2 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
packages/@n8n/config/src/index.ts	66.66%	0 Missing and 1 partial ⚠️
...src/databases/repositories/execution.repository.ts	75.00%	1 Missing ⚠️

📢 Thoughts on this report? Let us know!

[tomi]

If I have a config.json file like this:

{
	"executions": {
		"pruneData": true,
		"pruneDataMaxAge": 30,
		"pruneDataMaxCount": 100
	}
}

and I start n8n with N8N_CONFIG_FILES=config.json pn start it works on 1.66.0 but not with latest master or this branch. The pruning config needs to be under executions. Also could we add tests for this so it doesn't break a seconds time?

[ivov]

under executions

Good catch! Fixed

tests for this

Let's add tests for this separately for all remaining schema keys? As it's a more general issue and we should include this PR in today's release.

[tomi]

under executions

Good catch! Fixed

tests for this

Let's add tests for this separately for all remaining schema keys? As it's a more general issue and we should include this PR in today's release.

Fair enough. Ideally we should include a test for the case that broke in the first place. But we can merge this to get it on the release. Should add the tests after this so we don't forget

[tomi]

🚀

[ivov]

Tracking here: https://linear.app/n8n/issue/PAY-2265/add-tests-for-all-old-config-schema-keys

[cypress]

n8n    Run #7994

Run Properties:   Passed #7994  •  82aa71dbc3: 🌳 🖥️ browsers:node18.12.0-chrome107 🤖 ivov 🗃️ e2e/*

Project	n8n
Branch Review	restore-old-names-for-pruning-config-keys
Run status	Passed #7994
Run duration	04m 37s
Commit	82aa71dbc3: 🌳 🖥️ browsers:node18.12.0-chrome107 🤖 ivov 🗃️ e2e/*
Committer	Iván Ovejero
View all properties for this run ↗︎

---

Test results
Failures	0
Flaky	1
Pending	0
Skipped	0
Passing	475
View all changes introduced in this branch ↗︎

[github-actions]

✅ All Cypress E2E specs passed

[janober]

Got released with [email protected]