Side effect of decorator

Offen

#56,000 erstellt am 05.10.2023 9 Kommentare 0 Reaktionen 1 zugewiesene Person Auf GitHub ansehen

@rbuckton arbeitet bereits daran.

Seit 05.10.2023.

Bewertung

Dieses Issue wurde noch nicht bewertet.

Beschreibung

Bug Domain: Decorators Needs Investigation

🔎 Search Terms

"decorator" "field decorator" "side effect" "decorator side effect"

🕗 Version & Regression Information

This changed between versions 5.0 and 5.2
This is the behavior in every version I tried, and I reviewed the FAQ for entries about decorators

⏯ Playground Link

https://www.typescriptlang.org/play?#code/MYewdgzgLgBAYgSwKYBsAmARJoBOBDKEHGAXhgAoo8cBzJKALhgFcw0kAzBMJNAGhigwUJAA9GMAMIo8ECIlSZsRAkUngR4gJSkAfDADeAXwBQJ4DLkwACtSTDDJmIPDQczYIRzlqNAIxMbtw0Ar4ATIFQOMGhtADMkdFgNDoGTs4wAPSZMAACUBAAtAg0YERI6c5QABYIEAB0AA44II1+pDC+fpVZOflFJWU4FRkwNXVNLY1hHeE92XkFxaXlPeMNza1xs-Hpps6bbQCEicFmB1NhJzBByecwh3HXtzQmpuaWEFK16DBiImwvrZhg40hdWn5nlEzulDldTnd0rkFOgsLhVDhYVMngjXu8hNBOh0eAB3b4IdDkADkfj8VIEVLCYXpMCpcTiVK05lcIBQSHqKBANB8XIWeEmENIJDIrHYXB4aF6MAA6gAVOAmMUS6ZSsiM5lKgDyAGlNTlxY9dSw2JxuLwlWqNUA

💻 Code

const FieldDecorator = (target: undefined, context: ClassFieldDecoratorContext) => {} // do nothing

class Parent {
  constructor(arg1: string, arg2: string, arg3: string) {
    // @ts-ignore
    this.prop1 = arg1
    // @ts-ignore
    this.prop2 = arg2
    // @ts-ignore
    this.prop3 = arg3
  }
}

class Child extends Parent {
  prop1!: string

  prop2!: string

  @FieldDecorator
  prop3!: string
}

const a = new Child('11', '22', '33')
console.log(a)
// a.prop1 === undefined // WTF
// a.prop2 === '22' // OK
// a.prop3 === undefined // WTF

🙁 Actual behavior

the decorated property and the very first property are undefined despite the fact that they are given a value in the constructor

🙂 Expected behavior

a object should be equal to the following obj object

const obj = {
    prop1: "11"
    prop2: "22"
    prop3: "33"
}

Additional information about the issue

No response

Vorherrschende Sprache: Go
Sterne: 111k
Forks: 14.4k
Ø Merge: 1 T. 19 Std.
Gemergte PRs (30 T.): 117

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus microsoft/TypeScript

[Bug] PrivateIdentifier nodes are omitted from 2020 semantic classifications Offen

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 88/100

microsoft/TypeScript#64322 · 2 Kommentare · 1 Reaktion · 2 zugewiesene Personen · Erstellt am 18.9.2026
enhance: Add tests for premature caching of contextual parameter types Offen

Possible Improvement

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100

microsoft/TypeScript#64278 · 1 Kommentar · 1 Reaktion · Erstellt am 15.9.2026
dom.generated.d.ts is missing from this repo while still referenced in the docs Offen

Docs

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 70/100

microsoft/TypeScript#64118 · 1 Kommentar · Erstellt am 01.9.2026
typescript-language-server does not work with @typescript/typescript6 Offen

Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 88/100

microsoft/TypeScript#64094 · Erstellt am 29.8.2026
The new `--lsp` flag is not mentioned in `tsc --help`'s output Offen

Docs

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 76/100

microsoft/TypeScript#63959 · 5 Kommentare · Erstellt am 22.8.2026

Alle Issues in microsoft/TypeScript