Missing readonly and declare in ClassMethod and TSMethodSignature

[fisker]

Prettier 2.2.1
Playground link

--parser typescript

Input:

class C {
  readonly *e() {}
  declare *f() {}
}

interface Foo {
  declare e();
  abstract f();
}

Output:

class C {
  *e() {}
  *f() {}
}

interface Foo {
  e();
  f();
}

Expected behavior:

class C {
  readonly *e() {}
  declare *f() {}
}

interface Foo {
  declare e();
  abstract f();
}

---

Update

Expected behavior:

Throw SyntaxError.

[brodybits]

Playground link

Same output comes from both typescript and babel-ts parsers (other JavaScript parsers signal a syntax error).

Background is in PR #9748 & typescript-eslint/typescript-eslint#2908.

I gather that the testing with AST_COMPARE=1 does not show an issue because the attributes are not present in the AST coming either from babel-ts or typescript parser.

[fisker]

I gather that the testing with AST_COMPARE=1 does not show an issue because the attributes are not present in the AST

#9768

[fisker]

Invalid declare on accessors not throwing, should be the same problem. babel-ts fixed this in #10750.

Prettier 2.2.1
Playground link

--parser typescript

Input:

class Foo {
  declare get foo()
  declare set foo(v)
}

Output:

class Foo {
  get foo();
  set foo(v);
}