Skip to content

Commit

Permalink
Change our process to group stack trace rewriters and debuggers
Browse files Browse the repository at this point in the history
  • Loading branch information
jkup committed Jun 25, 2024
1 parent 591de22 commit c3ec11c
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 7 deletions.
6 changes: 2 additions & 4 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -27,8 +27,7 @@ Thanks for helping out in the [source map effort](https://ecma-international.org
We've been thinking about _source map constituencies_ in the following terms. Please feel free to suggest better titles or categorization methods!

1. Generators. Bundlers, transpilers, compilers. Tools that have access to the source code and emit (among other things) source maps.
2. Debuggers. Browsers and [standalone debuggers](https://www.replay.io/). These are tools that have access to the source map and the generated code.
3. Error monitoring tools. Tools like Sentry and New Relic. These tools have access to an error stack trace and a source map.
2. Debuggers. Browsers, [standalone debuggers](https://www.replay.io/) and stack trace rewriters like [Node](https://nodejs.org/en) and [Sentry](https://sentry.io/).

When considering changes, this list of tools might be useful: https://github.com/jkup/source-map-users

Expand All @@ -37,5 +36,4 @@ When considering changes, this list of tools might be useful: https://github.com
We've been categorizing our testing efforts into three groups, each matching with a [constituency](#constituencies).

1. Validators. For testing generator tools, we'd like to have the ability to validate a source map on its own as well as a source map being a valid map between a source and a generated file. We did a mini hackathon day on what this might look like: https://github.com/jkup/source-map-validator
2. Debuggers. For browsers, we'd like a suite of tests that can run wherever browsers apply source maps to their debug tools like Chrome DevTools. We'd like generic tests that browsers can use showing they apply source maps correctly. My assumption is they each already have their own suite of tests ([Chrome tests](https://github.com/ChromeDevTools/devtools-frontend/blob/main/test/e2e/sources/sourcemap_test.ts), [Firefox tests](https://github.com/mozilla/source-map/tree/master/test)) for this and we'll need to figure out how to integrate with their test harness and extract a shared base of tests we all could use.
3. Stack trace validators. For tools like Sentry and New Relic, we'd like tests that ensure source maps are correctly applied to the stack trace and that new features like [Debug ID](https://github.com/tc39/source-map-rfc/blob/main/proposals/debug-id.md) are working correctly.
2. Debuggers. For browsers, we'd like a suite of tests that can run wherever browsers apply source maps to their debug tools like Chrome DevTools. We'd like generic tests that browsers can use showing they apply source maps correctly. My assumption is they each already have their own suite of tests ([Chrome tests](https://github.com/ChromeDevTools/devtools-frontend/blob/main/test/e2e/sources/sourcemap_test.ts), [Firefox tests](https://github.com/mozilla/source-map/tree/master/test)) for this and we'll need to figure out how to integrate with their test harness and extract a shared base of tests we all could use. For stack trace rewriters, we'd like tests that can show source maps being correctly applied to error stacks.
6 changes: 3 additions & 3 deletions PROCESS.md
Original file line number Diff line number Diff line change
Expand Up @@ -20,9 +20,9 @@ When a proposal reaches Stage 3, no further improvements are possible without im

## Stage 4 (complete)

- 2 source map generators ready
- 2 interactive debuggers
- 2 stack trace decoder implementations
- 2 generators
- 2 debuggers
- 1 complete end to end workflow
- Test suite is complete

When a proposal reaches Stage 4, it is ready to land in the editor draft. There might still be editorial changes, but all the semantics are final.

0 comments on commit c3ec11c

Please sign in to comment.