Exports previously legitimately reported as unused are no longer reported about knip HOT 1 OPEN

There's a few things to this one. We're only talking about the combination of:

• ignoreExportsUsedInFile: true
• The export { id } notation

Unfortunately, internally when using the TypeScript AST nodes there's a big difference between these notations:

export const id = 1;

const id = 1;
export { id };

With the latter syntax, I couldn't find a reliable way to detect whether id was used more often or not within the same module. That's why I've opted for the stability of assuming export { id } is referenced (which technically could be argued it is, but it could be only the declaration, not other usage).

So this is the situation now, until we've found a reliable way to do it better.

Additional notes:

• Usually ignoreExportsUsedInFile is used only for types, since values don't need to be exported. E.g. "ignoreExportsUsedInFile": { "interface": true, "type": true }
• The former syntax doesn't have this issue, so purely from a Knip perspective that would be recommended.

from knip.