Declaration emit is broken for parameters marked /* @internal */ #33350

Activity

1. Andrew Branch (andrewbranch)

   added

   BugA bug in TypeScriptA bug in TypeScript

   Domain: Declaration EmitThe issue relates to the emission of d.ts filesThe issue relates to the emission of d.ts files

   on Sep 10, 2019
2. sheetalkamat commented on Sep 10, 2019

   

   Sheetal Nandi (sheetalkamat)

   on Sep 10, 2019

   Member

   More actions

   Seems like multiple issues.. This is d.ts emit through transform. The one you see is because of copying and updating d.ts as per changes in prepends through tsbuild info.

   c:\TypeScript\built\local>type a.ts
   interface TypeChecker {
       getContextualType(node: string, /* @internal */ contextFlags?: number): string | undefined;
   }
   c:\TypeScript\built\local>node tsc.js --stripInternal a.ts --d
   
   c:\TypeScript\built\local>type a.d.ts
   interface TypeChecker {
       getContextualType(node: string, /* @internal */ contextFlags?: number): string | undefined;
   }
   
   
3. Ryan Cavanaugh (RyanCavanaugh)

   added this to the Backlog milestone on Sep 25, 2019
4. juanrgm commented on Sep 15, 2022

   

   Juanra GM (juanrgm)

   on Sep 15, 2022

   More actions

   Same case here with TypeScript 4.8.3.

   • Error

    '*/' expected.
   

   • index.d.ts

   // ...
               lastIndexOf(searchElement: import("@anymodel/docker-compose").Volume, fromIndex?: number | undefined): number;
               every<S extends import("@anymodel/docker-compose").Volume>(predicate: (value: import("@anymodel/docker-compose").Volume, index: number, array: import("@anymodel/docker-compose").Volume[]) => value is S, thisArg?: any): this is S[]; /**"],
             },
   // ...

   • A portion of index.ts

   // ...
   {
     includeIgnored: ["**/.history/**"],
   }
   // ...

   Unfortunately I can't create a POC because the code is very complex and when I delete some parts the problem fixes. But basically the generation stopped after of this is S[]; /**"], and then the rest of the file contains the original source code.

   Workaround?

   "*" + "*/.history/**" // 😅
5. RyanCavanaugh commented on Sep 2, 2026

   

   Ryan Cavanaugh (RyanCavanaugh)

   on Sep 2, 2026

   Member

   More actions

   This is fixed by retirement of the affected build path.

   With TypeScript 5.4.0-dev.20240220, a composite project reference using prepend, outFile, and stripInternal emits malformed declaration text after the referenced source changes:

   getContextualType(node: string, 
   * @internal */ contextFlags?: number
   : number | undefined;

   A consumer of that declaration reports TS1003, TS1138, TS1161, and TS1128. TypeScript 5.5.0-dev.20240221 is the first removal build and reports TS5102: Option 'prepend' has been removed. Current native TypeScript likewise rejects the retired bundled-output configuration.

   The removal shipped in #57452; #57472 removed the remaining implementation code.
6. Ryan Cavanaugh (RyanCavanaugh)

   added

   FixedA PR has been merged for this issueA PR has been merged for this issue

   Needs Human ReviewThis issue has a backlog check awaiting maintainer review.This issue has a backlog check awaiting maintainer review.

   on Sep 2, 2026
7. Ryan Cavanaugh (RyanCavanaugh)

   closed this as completedon Sep 2, 2026
8. Ryan Cavanaugh (RyanCavanaugh)

   removed

   Needs Human ReviewThis issue has a backlog check awaiting maintainer review.This issue has a backlog check awaiting maintainer review.

   on Sep 23, 2026