Skip to content

Commit

Permalink
Sustainable Kibana Architecture: Fix group inference logic: add missi…
Browse files Browse the repository at this point in the history
…ng packages folders (#201758)

## Summary

After relocating modules (plugins and packages) to their new folders,
according to the _Sustainable Kibana Architecture_ design, their groups
will be inferred based on their path.

The logic was taking into account plugins' folders, but not packages'
folders. This PR fixes that.

(cherry picked from commit 81993a1)
  • Loading branch information
gsoldevila committed Dec 9, 2024
1 parent 859c567 commit 1d624cd
Showing 1 changed file with 32 additions and 30 deletions.
62 changes: 32 additions & 30 deletions packages/kbn-repo-source-classifier/src/group.ts
Original file line number Diff line number Diff line change
Expand Up @@ -19,36 +19,38 @@ const DEFAULT_MODULE_ATTRS: ModuleAttrs = {
visibility: 'shared',
};

const MODULE_GROUPING_BY_PATH: Record<string, ModuleAttrs> = {
'src/platform/plugins/shared': {
group: 'platform',
visibility: 'shared',
},
'src/platform/plugins/internal': {
group: 'platform',
visibility: 'private',
},
'x-pack/platform/plugins/shared': {
group: 'platform',
visibility: 'shared',
},
'x-pack/platform/plugins/internal': {
group: 'platform',
visibility: 'private',
},
'x-pack/solutions/observability/plugins': {
group: 'observability',
visibility: 'private',
},
'x-pack/solutions/security/plugins': {
group: 'security',
visibility: 'private',
},
'x-pack/solutions/search/plugins': {
group: 'search',
visibility: 'private',
},
};
const MODULE_GROUPING_BY_PATH: Record<string, ModuleAttrs> = ['packages', 'plugins']
.map<Record<string, ModuleAttrs>>((type) => ({
[`src/platform/${type}/shared`]: {
group: 'platform',
visibility: 'shared',
},
[`src/platform/${type}/private`]: {
group: 'platform',
visibility: 'private',
},
[`x-pack/platform/${type}/shared`]: {
group: 'platform',
visibility: 'shared',
},
[`x-pack/platform/${type}/private`]: {
group: 'platform',
visibility: 'private',
},
[`x-pack/solutions/observability/${type}`]: {
group: 'observability',
visibility: 'private',
},
[`x-pack/solutions/security/${type}`]: {
group: 'security',
visibility: 'private',
},
[`x-pack/solutions/search/${type}`]: {
group: 'search',
visibility: 'private',
},
}))
.reduce((acc, current) => ({ ...acc, ...current }), {});

/**
* Determine a plugin's grouping information based on the path where it is defined
Expand Down

0 comments on commit 1d624cd

Please sign in to comment.