fix(codegen): print override keyword for method and property definitions

Dunqing · claude · Dunqing · commit bf4a1ae6cb89 · 2026-02-26T19:46:34.000+08:00
The codegen was missing the `override` keyword when printing `MethodDefinition`
and `PropertyDefinition` nodes, causing `override` class members to lose the
modifier during code generation.

Co-Authored-By: Claude Opus 4.6 &lt;noreply@anthropic.com&gt;
diff --git a/crates/oxc_codegen/src/gen.rs b/crates/oxc_codegen/src/gen.rs
@@ -2666,6 +2666,11 @@ impl Gen for MethodDefinition<'_> {
             p.print_str("static");
             p.print_soft_space();
         }
+        if self.r#override {
+            p.print_space_before_identifier();
+            p.print_str("override");
+            p.print_soft_space();
+        }
         match &self.kind {
             MethodDefinitionKind::Constructor | MethodDefinitionKind::Method => {}
             MethodDefinitionKind::Get => {
@@ -2748,6 +2753,11 @@ impl Gen for PropertyDefinition<'_> {
             p.print_str("static");
             p.print_soft_space();
         }
+        if self.r#override {
+            p.print_space_before_identifier();
+            p.print_str("override");
+            p.print_soft_space();
+        }
         if self.readonly {
             p.print_space_before_identifier();
             p.print_str("readonly");
diff --git a/crates/oxc_codegen/tests/integration/ts.rs b/crates/oxc_codegen/tests/integration/ts.rs
@@ -15,6 +15,8 @@ fn cases() {
         "class Foo {\n\t#name: string;\n\tf() {\n\t\t#name in other && this.#name === other.#name;\n\t}\n}\n",
     );
     test_same("class B {\n\tconstructor(override readonly a: number) {}\n}\n");
+    test_same("class C extends B {\n\toverride show(): void;\n\toverride hide(): void;\n}\n");
+    test_same("class D extends B {\n\toverride readonly x: number;\n}\n");
     test_same("export { type as as };\n");
 }
 

Original file line number	Diff line number	Diff line change
`@@ -15,6 +15,8 @@ fn cases() {`
`15`	`15`	`"class Foo {\n\t#name: string;\n\tf() {\n\t\t#name in other && this.#name === other.#name;\n\t}\n}\n",`
`16`	`16`	`);`
`17`	`17`	`test_same("class B {\n\tconstructor(override readonly a: number) {}\n}\n");`
	`18`	`+ test_same("class C extends B {\n\toverride show(): void;\n\toverride hide(): void;\n}\n");`
	`19`	`+ test_same("class D extends B {\n\toverride readonly x: number;\n}\n");`
`18`	`20`	`test_same("export { type as as };\n");`
`19`	`21`	`}`
`20`	`22`