Try to match both original and compiled path when determining instrumentation eligibility
Maintainer antworten meist innerhalb von 1 Tag
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Anfängerfreundlichkeit
- 48/100
- Issue-Typ
- Feature
- Klarheit
- Größtenteils klar
- Aktivitätsstatus
- Veraltet
- Tech-Stack
- typescript
- Bereich
- devtools
Rechercherichtung
Beginne in src/transform.ts bei der im Issue verknüpften Behandlung von Source Maps und der Ausnahme für den Pfadvergleich und prüfe anschließend den zugehörigen Kontext in #181. Überprüfe, wie Paketpfade aus appmap.yml mit den ursprünglichen und kompilierten Pfaden verglichen werden. Als erledigt gilt die Aufgabe, wenn die Instrumentierung zulässig ist, sobald einer der beiden Pfade übereinstimmt, auch wenn die Behandlung der Source Map problematisch ist.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Currently, in the presence of source maps, only the original source path (with some exceptions) is matched with the package paths configured in appmap.yml.
This can be too restrictive; if trying to trace third-party code, for example, the user usually doesn't care about the original source directory structure but wants to be able to specify the compiled code path in appmap.yml and have it instrumented (even when there is a problem with handling the sourcemap or the map itself, cf. #181).
My intuition is trying to match BOTH the original and transpiled paths and instrumenting if either does match would be more robust and less surprising.
- Vorherrschende Sprache
- TypeScript
- Sterne
- 13
- Forks
- 6
- Ø Merge
- 4 T. 14 Std.
- Gemergte PRs (30 T.)
- 5
Entwicklungsumgebung
Dieses Projekt bietet weder Dev-Container noch Dockerfile noch Beitragsleitfaden – die Einrichtung liegt bei Ihnen. Beginnen Sie mit der README; die allgemeinen Schritte stehen in unserem Leitfaden für den ersten Beitrag.
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 getappmap/appmap-node
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 38/100
getappmap/appmap-node#183 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 25/100
getappmap/appmap-node#181 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 30/100
getappmap/appmap-node#180 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Support better-sqlite3Offen
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 25/100
getappmap/appmap-node#179 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Support TypeORMOffen
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 20/100
getappmap/appmap-node#178 ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in getappmap/appmap-node
Ähnliche Issues
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 74/100
xdjs/MusicNerdWeb#1532 ·
Maintainer antworten meist innerhalb von 1 Tag
-
bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
sebastian-software/ardo#352 ·
Maintainer antworten meist innerhalb von 1 Tag
-
@react95/core good first issue
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 88/100
Maintainer antworten meist innerhalb von 3 Tagen
-
@shikijs/stream: `ShikiStreamTokenizer.clone()` shares its `tokensStable` array with the originalEvtl. vergeben @xyrolle hat das heute übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
Maintainer antworten meist innerhalb von 6 Tagen
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
Maintainer antworten meist innerhalb von 1 Tag