maintainer review: a source matches no file when the node_modules exclude comes after it
Mantenedores costumam responder em até 1 dia
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 2/5
- Tempo estimado
- 1-3 horas
- Facilidade para iniciantes
- 88/100
- Tipo de issue
- Bug
- Clareza
- Claramente especificada
- Status de atividade
- Ativa
- Stack de tecnologia
- git, typescript
Direção de pesquisa
Comece na implementação de review list(), onde skill sources e dependencyExclude são passados para git ls-files. Execute a reprodução fornecida e a verificação do maintainer; em seguida, verifique se uma fonte em packages/hooks/src corresponde apesar da exclusão de node_modules e se o comportamento de git diff permanece inalterado.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
In review, list() passes the skill sources before dependencyExclude. In that order, git ls-files returns nothing for most sources, so a valid source matches no file and maintainer check cannot pass.
git init -q repro && cd repro
mkdir -p packages/hooks/src packages/hooks/node_modules/dep
echo x > packages/hooks/src/index.ts
echo x > packages/hooks/node_modules/dep/index.js
echo node_modules > .gitignore
git add -A && git commit -qm init
X=':(top,exclude,glob)**/node_modules/**'
git ls-files -- ':(top,glob)packages/hooks/src/**' "$X" # nothing
git ls-files -- "$X" ':(top,glob)packages/hooks/src/**' # packages/hooks/src/index.ts
Same result on Git 2.43 (Ubuntu 24.04), 2.49 and 2.54, with @tanstack/intent 0.5.0. git diff is not affected.
Fix: pass dependencyExclude before the patterns in list().
- Linguagem predominante
- TypeScript
- Estrelas
- 351
- Forks
- 26
- Merge médio
- 9h 5min
- PRs com merge (30d)
- 42
Preparar o ambiente
- Sem Dockerfile nem arquivo Docker Compose
- Tem um modelo de pull request
- Ler o guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de TanStack/intent
-
Dificuldade 5/5 Mais de uma semana Facilidade para iniciantes 45/100
TanStack/intent#238 · 1 comentário ·
Mantenedores costumam responder em até 1 dia
-
Avoid parsing skill metadata rejected by effective policyTalvez já em andamento @itsfaqih assumiu há 3 dias. Aberta
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 48/100
TanStack/intent#237 · 1 comentário ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 5/5 Mais de uma semana Facilidade para iniciantes 35/100
Mantenedores costumam responder em até 1 dia
-
Dificuldade 5/5 Mais de uma semana Facilidade para iniciantes 45/100
Mantenedores costumam responder em até 1 dia
-
enhancement
Dificuldade 5/5 Mais de uma semana Facilidade para iniciantes 35/100
Mantenedores costumam responder em até 1 dia
Todas as issues de TanStack/intent
Issues semelhantes
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 74/100
paperclipai/paperclip#17151 ·
Mantenedores costumam responder em até 1 dia
-
needs-triage
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 83/100
AgoraIO-Extensions/agent-infra#3159 ·
Mantenedores costumam responder em até 1 dia
-
blocklist addition
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 66/100
MetaMask/eth-phishing-detect#304596 ·
Mantenedores costumam responder em até 1 dia
-
friction
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 70/100
kentcdodds/kody#4142 ·
-
[Bug] Statistics | Task statistics are always empty on MikroORM with MySQL or SQLiteTalvez já em andamento @NdekoCode assumiu hoje. Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 76/100
ever-co/ever-gauzy#11214 ·
Mantenedores costumam responder em até 1 dia