The pre-commit `types` hook never runs for a commit of only `.tsx` or `.jsx` files: its glob `"*.{js,ts, jsx, tsx}"` has spaces

Aperta Adatta ai principianti
  • Qualcosa non va?
#554 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 5 giorni

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
1/5
Tempo stimato
Meno di un'ora
Idoneità per principianti
88/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
git, typescript
Ambito
developer-experience

Direzione di ricerca

Il bug è nel glob del comando types sotto pre-commit in lefthook.yml: "*.{js,ts, jsx, tsx}" contiene spazi superflui che fanno sì che le alternative jsx e tsx non corrispondano a nulla. Rimuovi gli spazi in modo che corrisponda al glob di lint, poi verifica aggiungendo allo stage solo un file .tsx con un errore di tipo e confermando che yarn tsc viene eseguito e blocca il commit. È completato quando types viene eseguito per commit che contengono solo .tsx e per commit che contengono solo .jsx.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

Before submitting a new issue
  • I tested using the latest version of the library, as the bug might be already fixed.
  • I tested using a supported version of react native.
  • I checked for possible duplicate issues, with possible answers.
Bug summary

lefthook.yml's pre-commit types command globs "*.{js,ts, jsx, tsx}". The spaces make two of the brace alternatives jsx and tsx, which no file name matches, so a commit that stages only .tsx or .jsx files skips yarn tsc, and a type error in it is committed. The lint command above it globs "*.{js,ts,jsx,tsx}" and runs.

pre-commit:
  parallel: true
  commands:
    lint:
      glob: "*.{js,ts,jsx,tsx}"
      run: yarn eslint {staged_files}
    types:
      glob: "*.{js,ts, jsx, tsx}"
      run: yarn tsc

Most of the repository's tests are .tsx files, so a commit of a test alone is the common case.

Library version

main at a37903f (0.0.0-nightly.a37903f)

Environment info
Windows 11 10.0.26300, Node 26.12.1, Yarn 4.9.2, lefthook 2.1.9 (the repository's dev dependency), Git for Windows.
No app or native build is involved: this is the repository's own pre-commit hook.
Steps to reproduce
  1. Clone main and install, which installs the lefthook hooks: 
    git clone https://github.com/nativewind/react-native-css
cd react-native-css
yarn install
  2. Add a .tsx file with a type error and commit it alone: 
    printf 'export const count: number = "one";\n' > src/type-error.tsx
git add src/type-error.tsx
git commit -m "test: a .tsx file with a type error, committed alone"
  3. Add any .ts file beside it and commit again: 
    printf 'export const control = 1;\n' > src/control.ts
git add src/control.ts
git commit -m "test: a .ts file beside it"

The reproducer branch is main at a37903f with step 2's commit on it, which went in with types skipped.

Expected: types runs in step 2, as it does in step 3, and refuses the commit with the type error.

Actual: step 2 prints types (skip) no matching staged files and commits; step 3 runs types, which reports src/type-error.tsx(1,14): error TS2322 and refuses the commit.

Step 2's output on main at a37903f, Windows 11 
╭──────────────────────────────────────╮
│ lefthook  v2.1.9   hook:  pre-commit │
╰──────────────────────────────────────╯
│  types (skip) no matching staged files
┃  lint ❯ 

  ────────────────────────────────────
summary: (done in 6.78 seconds)
✓ lint (6.62 seconds)
╭──────────────────────────────────────╮
│ lefthook  v2.1.9   hook:  commit-msg │
╰──────────────────────────────────────╯
┃  commitlint ❯ 

  ────────────────────────────────────
summary: (done in 1.86 seconds)
✓ commitlint (1.86 seconds)
[repro/554-types-hook-skips-tsx-commits 07d3993] test: a .tsx file with a type error, committed alone
 1 file changed, 1 insertion(+)
 create mode 100644 src/type-error.tsx
Step 3's output 
╭──────────────────────────────────────╮
│ lefthook  v2.1.9   hook:  pre-commit │
╰──────────────────────────────────────╯
┃  lint ❯ 

┃  types ❯ 
src/type-error.tsx(1,14): error TS2322: Type 'string' is not assignable to type 'number'.

exit status 2
  ────────────────────────────────────
summary: (done in 7.96 seconds)
✓ lint (4.13 seconds)
✗ types (7.89 seconds)
Reproducible example repository

https://github.com/YevheniiKotyrlo/react-native-css/tree/repro/554-types-hook-skips-tsx-commits

Lingua principale
TypeScript
Stelle
166
Fork
45
Merge medio
10g 1h
PR unite (30g)
4

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di nativewind/react-native-css

Tutte le issue di nativewind/react-native-css

Issue simili

Altre issue su TypeScript

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.