rootDirs should merge outputs
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Idoneità per principianti
- 35/100
- Tipo di issue
- Funzionalità
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Ferma
- Stack tecnologico
- typescript
- Ambito
- build-system, compilers
Direzione di ricerca
Iniziare esaminando il comportamento esistente di rootDir e rootDirs del compilatore, insieme alla issue correlata #9875. Il risultato previsto è che ogni directory radice configurata venga rimossa dai percorsi emessi, in modo che i file generati e i file sorgente condividano uno stesso albero di output; verificare il comportamento con le configurazioni e le strutture di directory mostrate qui.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Suggestion
🔍 Search Terms
rootDirs merge output
Related: https://github.com/microsoft/TypeScript/issues/9875
✅ Viability Checklist
My suggestion meets these guidelines:
- This wouldn't be a breaking change in existing TypeScript/JavaScript code
- This wouldn't change the runtime behavior of existing JavaScript code
- This could be implemented without emitting different JS based on the types of the expressions
- This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, new syntax sugar for JS, etc.)
- This feature would agree with the rest of TypeScript's Design Goals.
⭐ Suggestion
rootDirs should merge outputs instead of emitting to the folders listed in rootDirs. This would be consistent with rootDir.
📃 Motivating Example
Starting point: single root
Let's say I start with a project with a src/ folder:
.
└── src
├── index.ts
├── lib
│ └── lib.ts
└── test
└── test.ts
It has one rootDir and this config:
{
"compilerOptions": {
"outDir": "./",
"rootDir": "./src",
},
"include": ["src/**/*.ts"],
"exclude": []
}
When I compile, the output goes to ./ and mirrors the structure of ./src:
.
├── lib
│ └── lib.js
├── src
│ ├── index.ts
│ ├── lib
│ │ └── lib.ts
│ └── test
│ └── test.ts
└── test
└── test.js
All well and good.
Friction point: add generated files
Now I want to add some generated .ts files to my project and keep them outside of src/ for easier source control.
I put a .graphql source file in my src/ tree, and set up the generator to output to gen/:
.
├── gen
│ └── lib
│ └── schema.ts
└── src
├── index.ts
├── lib
│ ├── lib.ts
│ └── schema.graphql
└── test
└── test.ts
Now I want the same output structure as before, but I want to compile the sources in gen/ and output them as if I had "rootDir": "./gen".
The first thing I try is to replace "rootDir": "./src" with "rootDirs": ["./src", "./gen"]:
{
"compilerOptions": {
"outDir": "./",
"rootDirs": ["./src", "./gen"],
},
"include": ["src/**/*.ts", "gen/**/*.ts"],
"exclude": []
}
but this unfortunately defaults rootDir to ./, putting src/ and /gen in the output paths and writing .js files as siblings to their .ts sources:
.
├── gen
│ └── lib
│ ├── schema.js
│ └── schema.ts
└── src
├── index.js
├── index.ts
├── lib
│ ├── lib.js
│ ├── lib.ts
│ └── schema.graphql
└── test
├── test.ts
└── test.ts
This is very much not what I want or expect, and a big difference from the behavior of rootDir.
What I expect is that rootDirs acts like a multi-value rootDir, and for each input the root dir that it's in is stripped from the output path.
One additionally confusing aspect of rootDir vs rootDirs is that "rootDir": "./src" is not equivalent to "rootDirs": ["./src"]. The former will cause output files to be siblings to inputs.
It looks like to work around this we need to set "outDir": "./build", and then add an additional build step just to copy the files from ./build/src and ./build/gen into ./. This obviously can work but it's more of a complication than just a simple step: --watch will no longer work correctly unless you wire up the copy step to automatically run when ./build files change. For a project with simple build scripts, this is a pretty big leap in complexity.
Starting package.json:
"scripts": {
"build": "tsc --build"
},
Desired addition of a generator:
"scripts": {
"build": "npm run build:graphql && npm run build:ts",
"build:ts": "tsc --build",
"build:graphql": "graphql-codegen"
},
Actual addition of a generator, with broken --watch:
"scripts": {
"build": "npm run build:graphql && npm run build:ts && npm run build:copy",
"build:ts": "tsc --build",
"build:graphql": "graphql-codegen",
"build:copy": "cp -r build/{src,gen}/*"
},
Actual addition of a generator, with working --watch:
🤷♂️ I have to go figure this out still. Also, I'm not sure if this will badly interact with a monorepo, composite projects, and --build. The cross-package import paths are going to be different from the compiler output.
Potential workaround
I tried setting "rootDir": "./(src|gen)", hoping to tell tsc to strip those paths from the output. No luck as that path isn't interpreted as a pattern. That could potentially be a non-breaking way to add merging.
💻 Use Cases
Compiling generated files into a common output tree.
- Lingua principale
- Go
- Stelle
- 111k
- Fork
- 14.4k
- Merge medio
- 1g 19h
- PR unite (30g)
- 117
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di microsoft/TypeScript
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
microsoft/TypeScript#64322 · 2 commenti · 1 reazione · 2 assegnatari ·
-
Possible Improvement
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
microsoft/TypeScript#64278 · 1 commento · 1 reazione ·
-
Docs
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
microsoft/TypeScript#64118 · 1 commento ·
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 88/100
microsoft/TypeScript#64094 ·
-
Docs
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
microsoft/TypeScript#63959 · 5 commenti ·
Tutte le issue di microsoft/TypeScript
Issue simili
-
optimization optimization:agents-md-curator
Difficoltà 2/5 1-3 ore Idoneità per principianti 86/100
githubnext/gh-aw-cao#13143 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
blinklabs-io/bursa#904 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
yanet-platform/ipfw-go#129 ·
-
bug confmap/provider/googlesecretmanagerprovider needs triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
open-telemetry/opentelemetry-collector-contrib#51273 · 2 commenti ·
-
bug: AI Gateway client filter lists "Unknown" twice when NULL and literal Unknown clients coexist Apertabug
Difficoltà 2/5 1-3 ore Idoneità per principianti 90/100