JSDoc @implements tag/clause not checked

[DanielRosenwasser]

// @ts-check

/**
 * @typedef Foo
 * @property foo {() => string}
 */

/**
 * @implements {Foo}
 */
class C1 {
    foo = () => {
        return 10;
    }
}

/**
 * @implements {Foo}
 */
class C2 {
    foo() {
        return 10;
    }
}

Expected: At least an error on C1
Actual: Neither of these error

[brendankenny]

also see #17498, though it doesn't have the clearest initial comment describing the problem :)

[TimvdLippe]

FYI this does work with an @interface now, since #35629 has been implemented.

[Paril]

I attempted this with a TS interface but was unable to get it working. ie:

interface Test
{
func(n: number): void
}

/** @implements {Test} */
class MyTest
{
func(n) // `n` is implicit any
{
}
}

EDIT: okay, so it technically does work (in that MyTest can be passed to things expecting a Test, and as long as you don't re-declare it you will see the inherited type), but "func" doesn't auto-inherit the documentation or type annotations. I think that's a different issue though? maybe?

[kevinfiol]

I attempted this with a TS interface but was unable to get it working. ie:

interface Test
{
func(n: number): void
}

/** @implements {Test} */
class MyTest
{
func(n) // `n` is implicit any
{
}
}

EDIT: okay, so it technically does work (in that MyTest can be passed to things expecting a Test, and as long as you don't re-declare it you will see the inherited type), but "func" doesn't auto-inherit the documentation or type annotations. I think that's a different issue though? maybe?

Piggybacking on this. I thought I would get my answer in this thread, but also having this issue. Tried @implements and @interface but no dice. The best workaround I found was this comment where you set the type of the this reference to the interface, but this is a very imperfect solution if you're looking to get good IDE integration since the class methods don't pick up their types from the interface.

[dtinth]

Workaround, based on this comment.

class MyStoreImpl {
  constructor() {
    // Verify that the class implements the interface.
    void (/** @type {import('./types').MyStore} */ (this));
  }
}

[pupa91]

解决方法，基于此注释。

class MyStoreImpl {
  constructor() {
    // Verify that the class implements the interface.
    void (/** @type {import('./types').MyStore} */ (this));
  }
}

Hello, have you solved this problem

class BannerPlugin  {
    constructor(){
    // Verify that the class implements the interface.
    void (/** @type {import('webpack').LoaderOptionsPlugin} */ (this));
    }
}

vscode IntelliSense not working

[MSFTKonnor]

+1. Still hitting this problem in TypeScript 4.9.2 as well as TypeScript 5.0.

[david-sanford-deel]

+1.