From 1cb7318bb910d787f4a36694854974583e679457 Mon Sep 17 00:00:00 2001 From: GeoffCoxMSFT Date: Mon, 26 Oct 2020 16:44:20 -0700 Subject: [PATCH 1/2] Filter property names, not trigger --- Composer/packages/lib/indexers/src/groupTriggers.ts | 12 +++++------- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/Composer/packages/lib/indexers/src/groupTriggers.ts b/Composer/packages/lib/indexers/src/groupTriggers.ts index dc69ca327d..2ed27cbfb8 100644 --- a/Composer/packages/lib/indexers/src/groupTriggers.ts +++ b/Composer/packages/lib/indexers/src/groupTriggers.ts @@ -39,7 +39,7 @@ const getPropertyReferences = (content: any) => { return uniq(foundProperties); }; -const getTriggerPropertyReferences = (trigger: ITrigger) => { +const getTriggerPropertyReferences = (trigger: ITrigger, isValidProperty: (name: string) => boolean) => { const content = trigger.content; // inspect trigger @@ -52,7 +52,7 @@ const getTriggerPropertyReferences = (trigger: ITrigger) => { } } - const result = uniq(foundProperties); + const result = uniq(foundProperties).filter(isValidProperty); if (result.length === 0) { return [NoGroupingTriggerGroupName]; @@ -76,10 +76,8 @@ export const groupTriggersByPropertyReference = ( ): Record => { const result = {} as Record; - const validProperties = options?.validProperties; - const isValidProperty = validProperties - ? (x: string | undefined) => x && (x === NoGroupingTriggerGroupName || validProperties.includes(x)) - : () => true; + const isValidProperty = (name: string) => + !options?.validProperties || name === NoGroupingTriggerGroupName || options?.validProperties.includes(name); const addResult = (property: string, trigger: ITrigger) => { result[property] ? result[property].push(trigger) : (result[property] = [trigger]); @@ -87,7 +85,7 @@ export const groupTriggersByPropertyReference = ( if (dialog?.triggers) { dialog.triggers.forEach((t) => { - const properties = getTriggerPropertyReferences(t).filter(isValidProperty); + const properties = getTriggerPropertyReferences(t, isValidProperty); if (properties.length > 1 && options?.allowMultiParent) { properties.forEach((p) => { addResult(p, t); From feaae5fd475b6f9fdcc115e9827009dfa3a048d0 Mon Sep 17 00:00:00 2001 From: GeoffCoxMSFT Date: Mon, 26 Oct 2020 17:26:16 -0700 Subject: [PATCH 2/2] Also picking up latest generate library --- Composer/packages/server/package.json | 2 +- Composer/yarn.lock | 8 ++++---- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/Composer/packages/server/package.json b/Composer/packages/server/package.json index ac0be88452..b6031f3a69 100644 --- a/Composer/packages/server/package.json +++ b/Composer/packages/server/package.json @@ -63,7 +63,7 @@ "@bfc/lu-languageserver": "*", "@bfc/shared": "*", "@microsoft/bf-dispatcher": "^4.11.0-beta.20201016.393c6b2", - "@microsoft/bf-generate-library": "^4.10.0-daily.20201015.174962", + "@microsoft/bf-generate-library": "^4.10.0-daily.20201026.178528", "@microsoft/bf-lu": "^4.11.0-dev.20201013.7ccb128", "@microsoft/bf-orchestrator": "4.11.0-beta.20201013.20d7917", "archiver": "^5.0.2", diff --git a/Composer/yarn.lock b/Composer/yarn.lock index 4e3781b4dd..843d05b080 100644 --- a/Composer/yarn.lock +++ b/Composer/yarn.lock @@ -3006,10 +3006,10 @@ ts-md5 "^1.2.6" tslib "^1.10.0" -"@microsoft/bf-generate-library@^4.10.0-daily.20201015.174962": - version "4.10.0-daily.20201015.174962" - resolved "https://botbuilder.myget.org/F/botframework-cli/npm/@microsoft/bf-generate-library/-/@microsoft/bf-generate-library-4.10.0-daily.20201015.174962.tgz#42838aa5a44649a9de7bc3a7c987fe91b7ab65e3" - integrity sha1-QoOKpaRGSanee8OnyYf+kberZeM= +"@microsoft/bf-generate-library@^4.10.0-daily.20201026.178528": + version "4.10.0-daily.20201026.178528" + resolved "https://botbuilder.myget.org/F/botframework-cli/npm/@microsoft/bf-generate-library/-/@microsoft/bf-generate-library-4.10.0-daily.20201026.178528.tgz#9dd8acd3e5140776b515b9089f1238ff034352d0" + integrity sha1-ndis0+UUB3a1FbkInxI4/wNDUtA= dependencies: "@microsoft/bf-lu" "^4.11.0-dev.20201015.a41c691" adaptive-expressions "^4.11.0-dev.20201015.e56fdb7"