fix(editor): Use display option's @Version specifier

[tomi]

Nodes can have properties that have a displayOption which specifies a version
for which node versions that property applies to. We should take this into
account when forming the action types for a Node in the NodeList.

For example Notion node has 2 version which have different Page operations:

[
  {
    displayName: 'Operation',
    name: 'operation',
    type: 'options',
    noDataExpression: true,
    displayOptions: {
      show: {
        '@version': [1],
        resource: ['page'],
      },
    },
    options: [
      {
        name: 'Create',
        value: 'create',
        description: 'Create a page',
        action: 'Create a page',
      },
      {
        name: 'Get',
        value: 'get',
        description: 'Get a page',
        action: 'Get a page',
      },
      {
        name: 'Search',
        value: 'search',
        description: 'Text search of pages',
        action: 'Search a page',
      },
    ],
    default: 'create',
  },
  {
    displayName: 'Operation',
    name: 'operation',
    type: 'options',
    noDataExpression: true,
    displayOptions: {
      show: {
        '@version': [2],
        resource: ['page'],
      },
    },
    options: [
      {
        name: 'Archive',
        value: 'archive',
        description: 'Archive a page',
        action: 'Archive a page',
      },
      {
        name: 'Create',
        value: 'create',
        description: 'Create a page',
        action: 'Create a page',
      },
      {
        name: 'Search',
        value: 'search',
        description: 'Text search of pages',
        action: 'Search a page',
      },
    ],
    default: 'create',
  },
]

Before:

After:

[github-actions]

Great PR! Please pay attention to the following items before merging:

Files matching packages/**:

• If fixing bug, added test to cover scenario.
• If addressing forum or Github issue, added link to description.

Files matching packages/**/*.ts:

• Added unit tests to cover new or updated functionality.

Make sure to check off this list before asking for review.

[codecov]

Codecov Report

Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (4bc9164) 33.33% compared to head (503ba9d) 33.41%.
Report is 3 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #7351      +/-   ##
==========================================
+ Coverage   33.33%   33.41%   +0.08%     
==========================================
  Files        3378     3378              
  Lines      205919   205934      +15     
  Branches    22176    22215      +39     
==========================================
+ Hits        68644    68822     +178     
+ Misses     136160   135996     -164     
- Partials     1115     1116       +1     

Files	Coverage Δ
packages/workflow/src/Interfaces.ts	77.77% <ø> (ø)
...de/NodeCreator/composables/useActionsGeneration.ts	59.60% <90.47%> (+59.60%)	⬆️

... and 6 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[OlegIvaniv]

Looks good to me, thanks for adding the tests!

[OlegIvaniv]

LOVE IT 👏

[cypress]

1 flaky test on run #2382 ↗︎

0

242

3

0

1

Details:

🌳 ado-884-not-every-operations-show-up-in-the-sidebar-node-search 🖥️ browsers:...
Project: n8n	Commit: 503ba9d47e
Status: Passed	Duration: 08:34 💡
Started: Oct 5, 2023 12:28 PM	Ended: Oct 5, 2023 12:37 PM

  cypress/e2e/5-ndv.cy.ts • 1 flaky test

View Output Video

Test		Artifacts
NDV > should not retrieve remote options when required params throw errors		Output Screenshots Video

Review all test suite changes for PR #7351 ↗︎

[github-actions]

✅ All Cypress E2E specs passed

[janober]

Got released with n8n@1.9.3