feat(sam): "Sync SAM" shows more info if no template.yaml found

Problem:
When trying the "Sync SAM" command, the "No items found" message is
confusing.

Solution:
- Show a more informative message.
- Visit the help page if user hits Enter.

related: #2996

Author: justinmk3

src/shared/constants.ts

@@ -3,6 +3,8 @@
  * SPDX-License-Identifier: Apache-2.0
  */
 
+import * as vscode from 'vscode'
+
 import { isCloud9 } from './extensionUtilities'
 
 export const extensionSettingsPrefix = 'aws'
@@ -38,9 +40,11 @@ export const supportedLambdaRuntimesUrl: string =
     'https://docs.aws.amazon.com/lambda/latest/dg/runtime-support-policy.html'
 export const createUrlForLambdaFunctionUrl = 'https://docs.aws.amazon.com/lambda/latest/dg/urls-configuration.html'
 // URLs for samInitWizard
-export const samInitDocUrl: string = isCloud9()
-    ? 'https://docs.aws.amazon.com/cloud9/latest/user-guide/serverless-apps-toolkit.html#sam-create'
-    : 'https://docs.aws.amazon.com/toolkit-for-vscode/latest/userguide/serverless-apps.html#serverless-apps-create'
+export const samInitDocUrl = vscode.Uri.parse(
+    isCloud9()
+        ? 'https://docs.aws.amazon.com/cloud9/latest/user-guide/serverless-apps-toolkit.html#sam-create'
+        : 'https://docs.aws.amazon.com/toolkit-for-vscode/latest/userguide/serverless-apps.html#serverless-apps-create'
+)
 export const launchConfigDocUrl: string = isCloud9()
     ? 'https://docs.aws.amazon.com/cloud9/latest/user-guide/sam-debug-config-ref.html'
     : 'https://docs.aws.amazon.com/toolkit-for-vscode/latest/userguide/serverless-apps-run-debug-config-ref.html'
@@ -50,8 +54,9 @@ export const samDeployDocUrl: string = isCloud9()
     : 'https://docs.aws.amazon.com/toolkit-for-vscode/latest/userguide/serverless-apps.html#serverless-apps-deploy'
 export const lambdaFunctionUrlConfigUrl: string = 'https://docs.aws.amazon.com/lambda/latest/dg/urls-configuration.html'
 // URLs for "sam sync" wizard.
-export const samSyncUrl: string =
+export const samSyncUrl = vscode.Uri.parse(
     'https://docs.aws.amazon.com/serverless-application-model/latest/developerguide/accelerate-getting-started.html'
+)
 
 // URLs for CDK
 export const cdkProvideFeedbackUrl: string = `${githubUrl}/issues/new/choose`

src/shared/sam/sync.ts

@@ -7,6 +7,7 @@ import globals from '../extensionGlobals'
 
 import * as vscode from 'vscode'
 import * as path from 'path'
+import * as nls from 'vscode-nls'
 import * as localizedText from '../localizedText'
 import { DefaultS3Client } from '../clients/s3Client'
 import { Wizard } from '../wizards/wizard'
@@ -40,7 +41,9 @@ import { parse } from 'semver'
 import { isAutomation } from '../vscode/env'
 import { getOverriddenParameters } from '../../lambda/config/parameterUtils'
 import { addTelemetryEnvVar } from './cli/samCliInvokerUtils'
-import { samSyncUrl } from '../constants'
+import { samSyncUrl, samInitDocUrl } from '../constants'
+
+const localize = nls.loadMessageBundle()
 
 export interface SyncParams {
     readonly region: string
@@ -166,8 +169,14 @@ function createTemplatePrompter() {
 
     const trimmedItems = folders.size === 1 ? items.map(item => ({ ...item, description: undefined })) : items
     return createQuickPick(trimmedItems, {
-        title: 'Select a CloudFormation Template',
+        title: 'Select a SAM CloudFormation Template',
+        placeholder: 'Select a SAM template.yaml file',
         buttons: createCommonButtons(samSyncUrl),
+        noItemsFoundItem: {
+            label: localize('aws.sam.noWorkspace', 'No SAM template.yaml file(s) found. Select for help'),
+            data: undefined,
+            onClick: () => vscode.env.openExternal(samInitDocUrl),
+        },
     })
 }
 

src/test/shared/sam/sync.test.ts

@@ -21,7 +21,7 @@ describe('SyncWizard', function () {
     const createTester = (params?: Partial<SyncParams>) =>
         createWizardTester(new SyncWizard({ deployType: 'code', ...params }))
 
-    it('prompts for region -> template -> stackName -> bucketName', function () {
+    it('shows steps in correct order', function () {
         const tester = createTester()
         tester.region.assertShowFirst()
         tester.template.assertShowSecond()