microsoft / microsoft/TypeScript
require() is generated in JS although there's no runtime code
@andrewbranch y travaille déjà.
Depuis le 14/3/2025.
- 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
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Évaluation
Cette issue n'a pas encore été évaluée.