microsoft / microsoft/TypeScript

require() is generated in JS although there's no runtime code

Ouverte
#61,395 5 commentaires 0 réactions 1 personne assignée Voir sur GitHub

@andrewbranch y travaille déjà.

Depuis le 14/3/2025.

Needs Investigation
Langage dominant
Go
Étoiles
111k
Forks
14.4k
Merge moyen
1 j 19 h
PR mergées (30 j)
117

Description

🔎 Search Terms

require generated const enum
namespace require generated
namespace require const enum
namespace require
"export import" require

⏯ Playground Link

Multiple files, so can't use playground

💻 Code
// file1.ts
export namespace MyOtherNamespace {
    export const enum Yay {
        OK,
        KO
    }
}
// file2.ts
import { MyOtherNamespace } from "./file1";
export namespace MyNamespace {
    export import NS2 = MyOtherNamespace;
}
import { MyNamespace } from "./file2.ts";

const val = MyNamespace.NS2.Yay.KO
// Compiled code
const file2_1 = require("./file2"); // notice the variable name which is just "filename_1", as it is not used anywhere
const val = 1;
🙁 Actual behavior

Compiling an unused const file2_1 = require("./file2"); line

🙂 Expected behavior

Do not compile unused line.

Additional information about the issue

The problem is that this file is a types file, so it is installed only in develpment environment (npm install -D xxx) so in runtime it is failing because it can't find the required module.

If I modify the MyNamespace namespace to directly export a namespace instead of doing an export import it works, but we have a lot of namespaces so we have them separated into files, and couldn't find another way to export an imported namespace inside another namespace.

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.

Évaluation

Cette issue n'a pas encore été évaluée.

Recevez les nouvelles issues par e-mail

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