Invalid declaration with computed property using imported symbol

[dragomirtitian]

🔎 Search Terms

computed property declaration destructure binding

🕗 Version & Regression Information

• This is the behavior in every version I tried, and I reviewed the FAQ

⏯ Playground Link

https://www.typescriptlang.org/dev/bug-workbench/?lib=lib.es2015.d.ts&ts=5.1.6#code/PTAEAEBMFMGMBsCGAnRAXAlgewHYC5Q1kBXaAKBAngwCMDoBnANgrHADMN5odEBbaAVi400AB5oAdGgZlxAByzI0oYTgYqA0tACeoALygAyjr40s8ABQBKANysInbrwEEMOGGOmyMfRctAAb1BtPQBfUHZkLD5QACJJYDVRCTj7BSUVdmIcWExcUAB3DDQACwBhEXE0S2CAbVCAXQIAN0R4UlAwgkQcHWsCDWR3AHMgslBJyeRoNGJkHFA2juh7MLIgA

💻 Code

// @declaration: true
// @lib: es6
// @filename: context.ts
export const Key = Symbol();
// @filename: index.ts
import { Key } from "./context";
export function withContext({ [Key]: value }: any): string {
     return value;
}

🙁 Actual behavior

In index.d.ts the import of Key is removed leading to an invalid declaration file

// index.d.ts
export declare function withContext({ [Key]: value }: any): string;

🙂 Expected behavior

In index.d.ts the import of Key should be kept

// index.d.ts
import { Key } from "./context";
export declare function withContext({ [Key]: value }: any): string;

Additional information about the issue

Seems like filterBindingPatternInitializersAndRenamings in declarations.ts does not call checkEntityNameVisibility to mark the import as used in the declaration.