microsoft / microsoft/TypeScript

JSDoc present in mapped types is not output in generated type literals in declaration files

Ouverte
#62,309 2 commentaires 6 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Domain: Declaration Emit Help Wanted Possible Improvement
Langage dominant
Go
Étoiles
111k
Forks
14.3k
Merge moyen
2 j 4 h
PR mergées (30 j)
132

Description

🔎 Search Terms

generated declaration files mapped types jsdoc missing

🕗 Version & Regression Information
  • This is the behavior in every version I tried, and I reviewed the FAQ for entries about: I don't believe the FAQ contains anything applicable to this.
⏯ Playground Link

https://www.typescriptlang.org/play/?#code/JYWwDg9gTgLgBALzgMyhEcDkCIBNMBQBApgB6SxwDGEAdgM7z1UAWxIAhnALyIB0EAEYArYlRgAKAN4E4cAPQAqRbLmK4AZXTE4tYFR0ApDQBEIVVXEXzVyCBABc-RlGC0A5hICUAGgIBfLyIaBnhSHjhmNk4+MA4oemJpO0csDkxAvhTLOVy8-ILCuXl5OAA9CrgWCAA3YlcPFHtI6oB3ejgYNjhjMwsgA

💻 Code

This is not the same issue as https://github.com/microsoft/typescript/issues/50715, that issue is about type mappings that lose the JSDoc period.

I'm talking about what happens in the .d.ts file generated for source like this where JSDoc does survive type mapping:

import z from 'zod'

export const schema = z.object({
  /**
   * Some nice JSDoc
   */
  foo: z.string(),
})

const x = schema.parse({foo: 'a'}).foo
                                // ^^^ hovering foo shows the JSDoc
🙁 Actual behavior

Hovering the parsed .foo in the source file displays the JSDoc for the property:
Image

But if I consume the same type from the generated .d.ts file, I don't get the JSDoc:

Image
🙂 Expected behavior

We should get the same JSDoc when consuming the .d.ts file as we do when consuming the source file it was generated from.

Additional information about the issue

If we look at the generated .d.ts the problem is clear: tsc isn't outputting JSDoc it has in memory for mapped type properties to the generated type literals:

import z from 'zod';
export declare const schema: z.ZodObject<{
  /**
   * Some nice JSDoc
   */
  foo: z.ZodString;
}, "strip", z.ZodTypeAny, {
  foo: string; // (tsc should have copied the JSDoc to this property)
}, {
  foo: string; // (and to this one)
}>;
//# sourceMappingURL=Schema.d.ts.map

It would at least be nice to have an option to output these copies of JSDoc, even though it would make the declaration file bigger.

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Reproduisez la sortie de la déclaration de mapped-type à l’aide du TypeScript Playground lié et comparez le .d.ts généré avec le JSDoc du code source. Suivez le declaration emitter et la gestion des propriétés de mapped-type, puis ajoutez une couverture montrant que les littéraux de type générés préservent le JSDoc et vérifiez les déclarations attendues.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
typescript
Domaine
compilers
Type d'issue
Bug
Difficulté
4/5
Temps estimé
3-5 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
35/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.