Merge pull request #3933 from NativeScript/vladimirov/fix-project-create-scoped-packages

fix: unable to create project from scoped package template

Author: rosen-vladimirov

lib/definitions/project.d.ts

@@ -238,6 +238,11 @@ interface ITemplateData {
 	 * The whole content of package.json file
 	 */
 	templatePackageJsonContent: ITemplatePackageJsonContent;
+
+	/**
+	 * The version of the package used for creating the application.
+	 */
+	version: string;
 }
 
 interface ITemplatePackageJsonContent extends IBasePluginData {

lib/services/project-templates-service.ts

@@ -19,8 +19,16 @@ export class ProjectTemplatesService implements IProjectTemplatesService {
 			originalTemplateName = constants.RESERVED_TEMPLATE_NAMES["default"];
 		}
 
-		// support <reserved_name>@<version> syntax
-		const [name, version] = originalTemplateName.split("@");
+		// support <reserved_name>@<version> syntax, for example [email protected]
+		// support <scoped_package_name>@<version> syntax, for example @nativescript/[email protected]
+		const lastIndexOfAtSign = originalTemplateName.lastIndexOf("@");
+		let name = originalTemplateName;
+		let version = "";
+		if (lastIndexOfAtSign > 0) {
+			name = originalTemplateName.substr(0, lastIndexOfAtSign);
+			version = originalTemplateName.substr(lastIndexOfAtSign + 1);
+		}
+
 		const templateName = constants.RESERVED_TEMPLATE_NAMES[name.toLowerCase()] || name;
 		const fullTemplateName = version ? `${templateName}@${version}` : templateName;
 		const templatePackageJsonContent = await this.getTemplatePackageJsonContent(fullTemplateName);
@@ -49,7 +57,7 @@ export class ProjectTemplatesService implements IProjectTemplatesService {
 			});
 		}
 
-		return { templateName, templatePath, templateVersion, templatePackageJsonContent };
+		return { templateName, templatePath, templateVersion, templatePackageJsonContent, version };
 	}
 
 	private async getTemplateVersion(templateName: string): Promise<string> {

test/project-templates-service.ts

@@ -220,5 +220,42 @@ describe("project-templates-service", () => {
 			});
 
 		});
+
+		describe("uses correct version", () => {
+			[
+				{
+					name: "is correct when package version is passed",
+					templateName: "[email protected]",
+					expectedVersion: "1.0.0",
+					expectedTemplateName: "some-template"
+				},
+				{
+					name: "is correct when reserved package name with version is passed",
+					templateName: "[email protected]",
+					expectedVersion: "1.0.0",
+					expectedTemplateName: "tns-template-hello-world-ts"
+				},
+				{
+					name: "is correct when scoped package name without version is passed",
+					templateName: "@nativescript/vue-template",
+					expectedVersion: "",
+					expectedTemplateName: "@nativescript/vue-template"
+				},
+				{
+					name: "is correct when scoped package name with version is passed",
+					templateName: "@nativescript/[email protected]",
+					expectedVersion: "1.0.0",
+					expectedTemplateName: "@nativescript/vue-template"
+				}
+			].forEach(testCase => {
+				it(testCase.name, async () => {
+					const testInjector = createTestInjector();
+					const projectTemplatesService = testInjector.resolve<IProjectTemplatesService>("projectTemplatesService");
+					const { version, templateName } = await projectTemplatesService.prepareTemplate(testCase.templateName, "tempFolder");
+					assert.strictEqual(version, testCase.expectedVersion);
+					assert.strictEqual(templateName, testCase.expectedTemplateName);
+				});
+			});
+		});
 	});
 });