refactor: Use string ids on Credentials, Workflows, Tags, and Executions DB entities

[netroy]

Ticket: N8N-3097
DB Tests: https://github.com/n8n-io/n8n/actions/runs/3799877638
E2E Tests: https://github.com/n8n-io/n8n/actions/runs/3799879311

[ivov]

https://linear.app/n8n/issue/N8N-3097/marshalling-column-values-in-entity-models

[ivov]

More IDs to update:

• IWorkflowMetadata at /workflow/src/Interfaces.ts
• IExecutionBase at packages/editor-ui/src/Interface.ts

(Unless they had to be excluded for some reason.)

Only code review, untested.

[ivov]

Great job :)

A few more I found:

• IExecutionBase['retrySuccessId'] in cli is still string | number but string in editor-ui
• credentialId in CredentialConfig.vue is still [String, Number]

Otherwise code looks good, I'll test tomorrow.

[ivov]

1. Filtering executions glitches out: https://share.cleanshot.com/zT0tYKfb
2. Deleting execution throws error: https://share.cleanshot.com/1JB5vxkh
3. Public API schemas need updating: example
4. ITemplatesCollection has workflows: Array<{ id: number }>
5. Searching for id: .*number in cli shows typings for QueryRunner results with number IDs.
6. executeBatch.ts still has number IDs.
7. Tested all I could think of except sharing. Best for sb who knows sharing well to test that feature.

[netroy]

1. Filtering executions glitches out: https://share.cleanshot.com/zT0tYKfb
2. Deleting execution throws error: https://share.cleanshot.com/1JB5vxkh

These both exist on master, and were likely introduced in #5035.
1 will be fixed in #5071

3. Public API schemas need updating: [example](https://github.com/n8n-io/n8n/blob/99e3eb634bd34fa6791e09f112ce9e49407fed49/packages/cli/src/PublicApi/v1/handlers/workflows/spec/schemas/workflow.yml#L10)

✅

4. `ITemplatesCollection` has `workflows: Array<{ id: number }>`

This represents data coming from api.n8n.io. If we want to change this, we should probably also change it in the website code.

5. Searching for `id: .*number` in cli shows typings for `QueryRunner` results with number IDs.

I looked at those. Those are all migrations, and should not be changed, as they do not use entity transformers.

6. `executeBatch.ts` still has number IDs.

✅

7. Tested all I could think of except sharing. Best for sb who knows sharing well to test that feature.

Will ask Omar 👍🏽

[ivov]

Public API schemas changed, should we update version?

[janober]

Got released with n8n@0.210.0