Merge pull request #1175 from microsoft/benibenj/assistant-toad

Allow dots in tags and update extension tag handling

Author: benibenj

src/package.ts

@@ -298,12 +298,13 @@ function escapeRegExp(value: string) {
 }
 
 function sanitizeTag(tag: string): string {
-	return tag.replace(/[^\w-]/g, '');
+	return tag.replace(/[^\w.-]/g, '');
 }
 
 function toExtensionTags(extensions: string[]): string[] {
 	return extensions
 		.map(sanitizeTag)
+		.map(s => s.replace('.', ''))
 		.filter(s => !!s)
 		.map(s => `__ext_${s}`);
 }

src/test/package.test.ts

@@ -1643,7 +1643,7 @@ describe('toVsixManifest', () => {
 				languages: [
 					{
 						id: 'go',
-						aliases: ['golang', 'google-go'],
+						aliases: ['golang', 'google-go', '@sql'],
 					},
 				],
 			},
@@ -1656,6 +1656,7 @@ describe('toVsixManifest', () => {
 				assert.ok(tags.some(tag => tag === 'go'));
 				assert.ok(tags.some(tag => tag === 'golang'));
 				assert.ok(tags.some(tag => tag === 'google-go'));
+				assert.ok(tags.some(tag => tag === 'sql'));
 			});
 	});