maintainer review: a source matches no file when the node_modules exclude comes after it
Maintainer antworten meist innerhalb von 1 Tag
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Anfängerfreundlichkeit
- 88/100
- Issue-Typ
- Bug
- Klarheit
- Klar beschrieben
- Aktivitätsstatus
- Aktiv
- Tech-Stack
- git, typescript
Rechercherichtung
Beginnen Sie in der review list()-Implementierung, in der Skill-Quellen und dependencyExclude an git ls-files übergeben werden. Führen Sie die bereitgestellte Reproduktion und die Maintainer-Prüfung aus. Überprüfen Sie anschließend, dass eine Quelle unter packages/hooks/src trotz des Ausschlusses von node_modules gefunden wird und dass das Verhalten von git diff unverändert bleibt.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
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().
- Vorherrschende Sprache
- TypeScript
- Sterne
- 351
- Forks
- 26
- Ø Merge
- 11 Std. 51 Min.
- Gemergte PRs (30 T.)
- 32
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Hat eine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus TanStack/intent
-
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 45/100
TanStack/intent#238 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 48/100
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 35/100
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 45/100
Maintainer antworten meist innerhalb von 1 Tag
-
enhancement
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 35/100
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in TanStack/intent
Ähnliche Issues
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100
Maintainer antworten meist innerhalb von 1 Tag
-
auth: agent-bearer --manager-call is told to use the MCP instance argument on an ambiguous managerOffen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 80/100
Maintainer antworten meist innerhalb von 1 Tag
-
Lyric and ApostrophesOffenbug module: playback
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
chthollyphile/folia-major#739 · 2 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 90/100
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
agilepathway/label-checker#773 · 2 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag