From 514ee3f4c1d7369ad9b49efee711c143b09d5f0c Mon Sep 17 00:00:00 2001
From: Oleksandr T <oleksandr.tarasiuk@outlook.com>
Date: Fri, 15 Nov 2024 17:06:00 +0200
Subject: [PATCH] fix(60505): skip extension during module specifier conversion
 to a valid identifier

---
 src/services/utilities.ts                      |  2 +-
 ...refactorConvertImport_namedToNamespace11.ts | 18 ++++++++++++++++++
 2 files changed, 19 insertions(+), 1 deletion(-)
 create mode 100644 tests/cases/fourslash/refactorConvertImport_namedToNamespace11.ts

diff --git a/src/services/utilities.ts b/src/services/utilities.ts
index fbbcb67a106b9..0f7b99250111f 100644
--- a/src/services/utilities.ts
+++ b/src/services/utilities.ts
@@ -4061,7 +4061,7 @@ export function moduleSymbolToValidIdentifier(moduleSymbol: Symbol, target: Scri
 
 /** @internal */
 export function moduleSpecifierToValidIdentifier(moduleSpecifier: string, target: ScriptTarget | undefined, forceCapitalize?: boolean): string {
-    const baseName = getBaseFileName(removeSuffix(moduleSpecifier, "/index"));
+    const baseName = getBaseFileName(removeSuffix(removeFileExtension(moduleSpecifier), "/index"));
     let res = "";
     let lastCharWasValid = true;
     const firstCharCode = baseName.charCodeAt(0);
diff --git a/tests/cases/fourslash/refactorConvertImport_namedToNamespace11.ts b/tests/cases/fourslash/refactorConvertImport_namedToNamespace11.ts
new file mode 100644
index 0000000000000..69d7491609ba5
--- /dev/null
+++ b/tests/cases/fourslash/refactorConvertImport_namedToNamespace11.ts
@@ -0,0 +1,18 @@
+/// <reference path='fourslash.ts' />
+
+// @allowJs: true
+// @checkJs: true
+
+// @filename: /a.js
+/////*a*/import { a } from "./foo.js";/*b*/
+////a;
+
+goTo.select("a", "b");
+edit.applyRefactor({
+    refactorName: "Convert import",
+    actionName: "Convert named imports to namespace import",
+    actionDescription: "Convert named imports to namespace import",
+    newContent:
+`import * as foo from "./foo.js";
+foo.a;`,
+});