checkJs error for indirect JSDoc param tag

[DanielRosenwasser]

TypeScript Version: 2.6.0-dev.20171015

// @ts-check

function Foo() {
    /**
     * @param {string} wat
     */
    this.bar = function(wat) {

    }
}

Expected: No error
Actual: JSDoc '@param' tag has name 'wat', but there is no parameter with that name.

[DanielRosenwasser]

Probably due to #18777.

[OliverJAsh]

Any chance this will be fixed in TS 2.6? This is a big blocker for us upgrading to 2.6, as we use JSDoc in many places.

As a workaround, I'm using @ts-ignore.

[sandersn]

Yeah, it's certainly #18777 -- the type is correct, it just issues a spurious error.

[sandersn]

Fix is up at #19355

[OliverJAsh]

@sandersn Thanks for getting this fixed so quickly!

[KtorZ]

Hey!
I ran into a similar issue but from the other perspective. Using a destructuring assignment on the function argument, I got the following error:

/**
 * @param {Object} args 
 * @param {string} args.prop1
 * @param {string} args.prop2
 */
function patate({ prop1, prop2 }) {

}

// JSDoc '@param' tag has name 'args', but there is no parameter with that name.

The problem is the same using an enum syntax:

/**
 * @param {{prop1: string, prop2: string}} args 
 */
function patate({ prop1, prop2 }) {

}

Wouldn't it make sense not to raise this error in this case, but still be able to annotate the properties with the right types ?

[mhegazy]

@KtorZ this is tracked by #7429

[KtorZ]

Sweet, thanks 👍