Preserve quotes for TSMethodSignature nodes named new (#19621)

changelog_unreleased/typescript/19621.md

@@ -0,0 +1,19 @@
+#### Preserve quotes for methods named `new` (#19621 by @kovsu)
+
+<!-- prettier-ignore -->
+```tsx
+// Input
+interface Container {
+  "new"(id: string): number;
+}
+
+// Prettier stable
+interface Container {
+  new(id: string): number;
+}
+
+// Prettier main
+interface Container {
+  "new"(id: string): number;
+}
+```

src/language-js/print/key.js

@@ -99,6 +99,10 @@ function isKeySafeToUnquote(node, options) {
     return false;
   }
 
+  if (node.type === "TSMethodSignature" && value === "new") {
+    return false;
+  }
+
   // Safe to unquote as identifier
   if (
     // With `--strictPropertyInitialization`, TS treats properties with quoted names differently than unquoted ones.

tests/format/typescript/quote-props/__snapshots__/format.test.js.snap

@@ -240,6 +240,96 @@ interface I2 {
 ================================================================================
 `;
 
+exports[`issue-19618.ts - {"quoteProps":"as-needed"} format 1`] = `
+====================================options=====================================
+parsers: ["typescript"]
+quoteProps: "as-needed"
+                                                      printWidth: 80 (default) |
+=====================================input======================================
+interface Container {
+  "new"(id: string): number;
+  "method"(): void;
+}
+
+type ContainerType = {
+  'new'(id: string): number;
+  'method'(): void;
+};
+
+=====================================output=====================================
+interface Container {
+  "new"(id: string): number;
+  method(): void;
+}
+
+type ContainerType = {
+  "new"(id: string): number;
+  method(): void;
+};
+
+================================================================================
+`;
+
+exports[`issue-19618.ts - {"quoteProps":"consistent"} format 1`] = `
+====================================options=====================================
+parsers: ["typescript"]
+quoteProps: "consistent"
+                                                      printWidth: 80 (default) |
+=====================================input======================================
+interface Container {
+  "new"(id: string): number;
+  "method"(): void;
+}
+
+type ContainerType = {
+  'new'(id: string): number;
+  'method'(): void;
+};
+
+=====================================output=====================================
+interface Container {
+  "new"(id: string): number;
+  "method"(): void;
+}
+
+type ContainerType = {
+  "new"(id: string): number;
+  "method"(): void;
+};
+
+================================================================================
+`;
+
+exports[`issue-19618.ts - {"quoteProps":"preserve"} format 1`] = `
+====================================options=====================================
+parsers: ["typescript"]
+quoteProps: "preserve"
+                                                      printWidth: 80 (default) |
+=====================================input======================================
+interface Container {
+  "new"(id: string): number;
+  "method"(): void;
+}
+
+type ContainerType = {
+  'new'(id: string): number;
+  'method'(): void;
+};
+
+=====================================output=====================================
+interface Container {
+  "new"(id: string): number;
+  "method"(): void;
+}
+
+type ContainerType = {
+  "new"(id: string): number;
+  "method"(): void;
+};
+
+================================================================================
+`;
+
 exports[`type-literal.ts - {"quoteProps":"as-needed"} format 1`] = `
 ====================================options=====================================
 parsers: ["typescript"]

tests/format/typescript/quote-props/issue-19618.ts

@@ -0,0 +1,9 @@
+interface Container {
+  "new"(id: string): number;
+  "method"(): void;
+}
+
+type ContainerType = {
+  'new'(id: string): number;
+  'method'(): void;
+};