Skip to content

Commit d44c6a4

Browse files
committed
Merge branch 'fix/fix-plugin-commands' into 'master'
fix(plugin): fix plugin add and remove commands for case where there is no plugins block in config See merge request auto-cloud/cloudgraph/cli!198
2 parents 6f58f5f + 3d7265c commit d44c6a4

2 files changed

Lines changed: 5 additions & 3 deletions

File tree

src/commands/policy/add.ts

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -31,8 +31,7 @@ export default class AddPolicy extends OperationBaseCommand {
3131
const config = this.getCGConfig()
3232
if (config) {
3333
let configuredPolicies =
34-
config.cloudGraph.plugins[PluginType.PolicyPack] || []
35-
34+
config.cloudGraph.plugins?.[PluginType.PolicyPack] || []
3635
if (isEmpty(configuredPolicies)) {
3736
// Set new Policy Pack Plugin array
3837
configuredPolicies = [
@@ -51,6 +50,9 @@ export default class AddPolicy extends OperationBaseCommand {
5150
},
5251
]
5352
}
53+
if (!config.cloudGraph.plugin) {
54+
config.cloudGraph.plugins = {}
55+
}
5456
config.cloudGraph.plugins[PluginType.PolicyPack] = uniqBy(
5557
configuredPolicies,
5658
'name'

src/commands/policy/remove.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ export default class RemovePolicy extends OperationBaseCommand {
4040
const config = this.getCGConfig()
4141
if (config) {
4242
const configuredPolicies =
43-
config.cloudGraph.plugins[PluginType.PolicyPack] || []
43+
config.cloudGraph.plugins?.[PluginType.PolicyPack] || []
4444

4545
if (!isEmpty(configuredPolicies)) {
4646
// Remove policy from Policy Pack Plugin array

0 commit comments

Comments
 (0)