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

Aberta Para iniciantes
  • Algo errado?
#554 0 comentários 0 reações 0 responsáveis Ver no GitHub

Mantenedores costumam responder em até 12 dias

Ninguém assumiu esta issue ainda.

Avaliação

Dificuldade
1/5
Tempo estimado
Menos de uma hora
Facilidade para iniciantes
88/100
Tipo de issue
Bug
Clareza
Claramente especificada
Status de atividade
Ativa
Stack de tecnologia
git, typescript
Domínio
developer-experience

Direção de pesquisa

O erro está no glob do comando types em pre-commit no lefthook.yml: "*.{js,ts, jsx, tsx}" tem espaços extras, o que faz com que as alternativas jsx e tsx não correspondam a nada. Remova os espaços para que ele corresponda ao glob de lint e depois verifique adicionando ao stage apenas um arquivo .tsx com um erro de tipo e confirmando que yarn tsc é executado e bloqueia o commit. Está concluído quando types é executado em commits somente com .tsx e em commits somente com .jsx.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

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

Linguagem predominante
TypeScript
Estrelas
166
Forks
45
Merge médio
17d 22h
PRs com merge (30d)
3

Preparar o ambiente

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de nativewind/react-native-css

Todas as issues de nativewind/react-native-css

Issues semelhantes

Mais issues de TypeScript

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.