-
Notifications
You must be signed in to change notification settings - Fork 9
Comparing changes
Open a pull request
base repository: replayio/replay-cli
base: replayio@1.4.1
head repository: replayio/replay-cli
compare: main
Commits on Jun 24, 2024
-
Use modern
moduleResolution
setting (#553)* Use modern moduleResolution * revert redundant changes * move patch to resolutions * fixed test expectation * fixed the same expectation in another place
Configuration menu - View commit details
-
Copy full SHA for d208047 - Browse repository at this point
Copy the full SHA d208047View commit details -
Configuration menu - View commit details
-
Copy full SHA for f1e815a - Browse repository at this point
Copy the full SHA f1e815aView commit details -
Configuration menu - View commit details
-
Copy full SHA for 40a1ec8 - Browse repository at this point
Copy the full SHA 40a1ec8View commit details
Commits on Jun 25, 2024
-
Update glob version to fix nested dependency deprecation warning (#556)
* Update glob version * Update glob usage * Update Webpack plugin readme * Create large-cups-grin.md
Configuration menu - View commit details
-
Copy full SHA for 89c5082 - Browse repository at this point
Copy the full SHA 89c5082View commit details -
Configuration menu - View commit details
-
Copy full SHA for d7d654b - Browse repository at this point
Copy the full SHA d7d654bView commit details -
Retry failed GitHub API requests with exponential backoff (#552)
- Expand the existing `fetch` wrapper to also include automatic retries requests (with exponential backoff) - Add unit tests I imagine we can move this code into the `packages/shared` package as part of PRO-676
Configuration menu - View commit details
-
Copy full SHA for 4baf436 - Browse repository at this point
Copy the full SHA 4baf436View commit details -
Configuration menu - View commit details
-
Copy full SHA for c6f2b8c - Browse repository at this point
Copy the full SHA c6f2b8cView commit details -
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Configuration menu - View commit details
-
Copy full SHA for 26eff0a - Browse repository at this point
Copy the full SHA 26eff0aView commit details -
Setup Rollup-based build system for packages (#549)
This PR creates a build system that allows us to inline/bundle some of the internal code into packages that are meant to be published. This reduces the versioning overhead and the result of all of this could look smth like this once we start moving things around: ``` . ├── cypress │ └── import shared, test-utils ├── observability-node │ └── goes away ├── playwright │ └── import shared, test-utils ├── replay │ └── goes away ├── replayio │ └── imports shared ├── shared │ └── inlined └── test-utils └── inlined ```
Configuration menu - View commit details
-
Copy full SHA for ac7aa52 - Browse repository at this point
Copy the full SHA ac7aa52View commit details -
- Move shared "logger" class from test-utils to the new shared package - Move shared GraphQL util methods from test-utils to the new shared package - Delete observability-node package
Configuration menu - View commit details
-
Copy full SHA for fb95bbe - Browse repository at this point
Copy the full SHA fb95bbeView commit details -
Configuration menu - View commit details
-
Copy full SHA for e7bd234 - Browse repository at this point
Copy the full SHA e7bd234View commit details -
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Configuration menu - View commit details
-
Copy full SHA for 31a4fbe - Browse repository at this point
Copy the full SHA 31a4fbeView commit details
Commits on Jun 26, 2024
-
Make build script to be executed per pkg and introduce a watch script (…
…#567) * Make build script to be executed per pkg and introduce a watch script * add missing build script * fixed build * prettierignore turbo files
Configuration menu - View commit details
-
Copy full SHA for 40beb19 - Browse repository at this point
Copy the full SHA 40beb19View commit details -
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Configuration menu - View commit details
-
Copy full SHA for a4fcfd8 - Browse repository at this point
Copy the full SHA a4fcfd8View commit details -
Move legacy @replayio/replay Node APIs to new shared package (#562)
- [x] Remove Node entrypoint from `replayio` package - This should technically require a new major version but since there was only a single, useless API exported up until this point– I think we can safely assume that nothing was using it - [x] Update cypress, playwright, puppeteer, and test-utils packages to depend on the new `@replay-cli/shared` package - [x] Moved some of the code from the legacy `@replayio/replay` package to `shared` or `test-utils` packages - At this point the only package still depending on the legacy `@replayio/replay` package (Node APIs) is `test-utils` which will require follow up work because some of the code is harder to extricate As of this PR the following dependencies have been updated: * `@replayio/cypress` * ✅ `@replay-cli/shared` * ❌ ~~`@replayio/replay`~~ * `@replayio/playwright` * ✅ `@replay-cli/shared` * ❌ ~~`@replayio/replay`~~ * `@replayio/puppeteer` * ✅ `@replay-cli/shared` * ❌ ~~`@replayio/replay`~~ ### Next steps The major remaining work will be to update `@replayio/test-utils` to no longer depend on `@replayio/replay` (so that we can delete that package entirely)
Configuration menu - View commit details
-
Copy full SHA for 61128fe - Browse repository at this point
Copy the full SHA 61128feView commit details
Commits on Jun 27, 2024
-
Configuration menu - View commit details
-
Copy full SHA for 1861a30 - Browse repository at this point
Copy the full SHA 1861a30View commit details -
Fixed turborepo caching on
replayio
changes (#569)* Fixed turborepo caching on `replayio` changes * fixed which pkg we actually install temporarily * do the same thing in playwright tests
Configuration menu - View commit details
-
Copy full SHA for 2185925 - Browse repository at this point
Copy the full SHA 2185925View commit details -
Change logger to singleton (#564)
Closes PRO-722. This PR moves the logger to singleton, we can keep all tags centrally. Will be needed for the sessionId change I'm doing.
Configuration menu - View commit details
-
Copy full SHA for 6ce3100 - Browse repository at this point
Copy the full SHA 6ce3100View commit details
Commits on Jun 28, 2024
-
Moving more code into shared (#571)
Continuation of [PRO-720](https://linear.app/replay/issue/PRO-720); follows up to #562 Moving much of the `replayio` utility-type code into the new `shared` package so that we can use those Node APIs for querying the list of recording and uploading or deleting them. Most of this was rote but I did make a couple of changes worth calling out explicitly: - I changed the value returned by `fetchAuthIdsFromGraphQL` from `{ userId: string | null; workspaceId: string | null }` to `{ id: string; type: "user" | "workspace" }` and updated the callers; this was easier to work with (from a types perspective) - Grafana still gets `userId` _or_ `workspaceId` though, just not both - `Logger` is now eagerly instantiated; the `initialize()` method will accept a name and version and pass them along to the Grafana instance. Logging before initialization (or in Jest tests) will be local-only. - Note that a version parameter is also supported now (so we can associate Grafana logs with specific releases) - Both name and version should probably be imported and passed directly from `package.json` (which is what I am currently doing for Puppeteer and replayio) - Note that I've replaced the previous "replayio" name prefix with whatever the passed package name is - Lots of `debug(...args)` logging has been changed to `logger.debug("...", {...args})` - I've moved `mixpanel` and `launch-darkly` code over to `shared` too, even though (for now) `replayio` is the only thing using this stuff - I also tidied up the `getRecordings` method to not create so many throw-away arrays; (I didn't just do this for performance, but TypeScript was complaining about the return array containing recordings _and_ `undefined` with the previous code structure) ## Next steps - Remove the `test-utils` dependency on `@replayio/replay` (and delete `@replayio/replay`) - Update `test-utils` to be a peer dependency so that it will get inlined as well (to resolve ambiguous logger ownership) - Audit `logger.debug` entries and decide which ones should be `info`, `log`, or even `error` - Address any added `TODO [PRO-720]` comments
Configuration menu - View commit details
-
Copy full SHA for 33656ea - Browse repository at this point
Copy the full SHA 33656eaView commit details -
Configuration menu - View commit details
-
Copy full SHA for 9494138 - Browse repository at this point
Copy the full SHA 9494138View commit details -
Add device and session ID to logger (#565)
* Use singleton logger * Add deviceId and sessionId to logger * logger: warn if already initialized * Resolve cache path * Refactor observability profile path
Configuration menu - View commit details
-
Copy full SHA for 18d4972 - Browse repository at this point
Copy the full SHA 18d4972View commit details -
- [x] Merge replayio/docs#211 to add the new API keys redirect page ### [Loom demo](https://www.loom.com/share/448e43d722e84fae975a2013cbbd73cb) ### Screenshots | Scenario | | | :--- | :--- | | Not authenticated | ![Not authenticated](https://github.com/replayio/replay-cli/assets/29597/aa0c5ab6-feb8-4a16-9987-dbdc4f7538d0) | | Authenticated by email and password | ![Authenticated by email and password](https://github.com/replayio/replay-cli/assets/29597/2caee8ce-891b-4b06-907f-c33f2c721daf) | Team API key | ![Team API key](https://github.com/replayio/replay-cli/assets/29597/08df1946-cf6c-4bda-80a1-a11f124976db) | Personal API key | ![Personal API key](https://github.com/replayio/replay-cli/assets/29597/a39723cc-ad23-4efa-b6bd-a5ad30c0c5a7)
Configuration menu - View commit details
-
Copy full SHA for db3f422 - Browse repository at this point
Copy the full SHA db3f422View commit details -
Forked @replayio/replay Node APIs by copying them into test-utils (#575)
To deprecate `@replayio/replay` we need to remove the following dependencies from `test-utils`: * `listAllRecordings` * `removeRecording` * `uploadRecording` The two options we can explore are: 1. Update `test-utils` to use the new shared variations of these methods (e.g. replace `listAllRecordings` from the legacy CLI with the new shared `getRecordings`). - **Cons**: The in-memory types for recordings are different– the metadata and the status field(s)– and it's unclear how much work would be required to bring them into alignment. - **Pros**: Once it's done, there is no further clean up work. We can delete the legacy CLI code and be done with it. (Pending the outcome of [PRO-730](https://linear.app/replay/issue/PRO-730/take-care-of-github-actions-after-nuking-the-old-cli) possibly.) 1. Fork those three methods from `@replayio/replay` into `test-utils` and leave the cleanup for later. - **Cons**: Those methods refer to a lot of other methods and it would probably end up being a pretty big amount of stuff to fork over. Maybe we could do it as a batch and separate it into its own `test-utils/legacy` directory or something so that it's clear we need to clean it up. (This doesn't really move us any closer toward our longer-term code sharing goals in other words.) - **Pros**: It's probably probably faster and a little less risky in the short term, and most importantly it unblocks the observability and funnel tracking work. After some discussion with Mateusz, this PR implements the second option above. This PR also updates Cypress, Jest, Playwright, and Puppeteer packages so that `@replayio/test-utils` is a _peer dependency_ (so it will be inlined).
Configuration menu - View commit details
-
Copy full SHA for c9b0400 - Browse repository at this point
Copy the full SHA c9b0400View commit details
Commits on Jul 1, 2024
-
Migrate logs in
playwright
package (#574)Move logger init and close from ReplayReporter in test-utils to ReplayPlaywrightReporter.
Configuration menu - View commit details
-
Copy full SHA for 10fc3f2 - Browse repository at this point
Copy the full SHA 10fc3f2View commit details -
Configuration menu - View commit details
-
Copy full SHA for ac23741 - Browse repository at this point
Copy the full SHA ac23741View commit details -
Configuration menu - View commit details
-
Copy full SHA for dd41fa8 - Browse repository at this point
Copy the full SHA dd41fa8View commit details -
Rework the dts generation (#576)
* Switch to using virtual _bundled directory * js bundling works * try to avoid using \0, still unsure what's better * port preconstruct dts plkugin * tweak * fix cwd * working dts emit without replacements * fixed issues * test change * wip * switch to vfs module * make sure that unique ids are generated for bundled modules * use the correct system 😭😭😭 * simplify bundled plugin * fix resolve errors * tweak resolve Errors plugin further * ignore resolve errors for entries * getting closer * fix bundling scenarios * rollback temp changes * bring back dev dep * fix build crashes with multiple bundled dependencies * fix dep * fix user agent thing * remove redundant check from `esbuild` plugin * small tweak * call init functions where appropriate * fix reference
Configuration menu - View commit details
-
Copy full SHA for adb0dff - Browse repository at this point
Copy the full SHA adb0dffView commit details -
Migration logs in
cypress
package (#581)* Migration logs in `cypress` package * Move logger.close to after:run * Add back `onAfterRun` hook * Rename logs
Configuration menu - View commit details
-
Copy full SHA for 431eaad - Browse repository at this point
Copy the full SHA 431eaadView commit details
Commits on Jul 2, 2024
-
Change formatting for browser warning (#584)
If you have not installed the Replay browser we will show the following message: ![Screenshot 2024-07-01 at 12 23 12 PM](https://github.com/replayio/replay-cli/assets/29597/57739d70-79f3-4d08-a5e5-367ab0f2ad16) If you have already installed the Replay browser, we will show: ![Screenshot 2024-07-01 at 12 25 43 PM](https://github.com/replayio/replay-cli/assets/29597/9f84c96d-1406-40b9-b5a6-32929f3adf77)
Configuration menu - View commit details
-
Copy full SHA for 5b80bcb - Browse repository at this point
Copy the full SHA 5b80bcbView commit details -
Add Mixpanel to Playwright (#577)
- [x] Reorganized Mixpanel code to be a bit easier to reason about and added some more unit tests - Most of the lines changed in this PR comes from this reorganization - [x] Add some basic Mixpanel logging to the `playwright` and `test-utils` packages - Playwright specific tracking: - Invalid reporter config - Replay reporter used without Replay browser - Test utils tracking (also applies to Cypress) - Reporter error - Missing API key - Recording metadata - Test end - Test suite begin and end - Test suite results (including number of crashed recordings and uploaded recordings) - [x] Spot test and verify that events are ending up in Mixpanel
Configuration menu - View commit details
-
Copy full SHA for 4624764 - Browse repository at this point
Copy the full SHA 4624764View commit details -
Change the log level and message for
replayio
logs (#582)* Change the log level and message for `replayio` logs * Log sourcemap upload failures * Fix logger.close reference in exitTask * Log unhandled exception * Log browser crash * Log error in `update` command
Configuration menu - View commit details
-
Copy full SHA for 6413d6b - Browse repository at this point
Copy the full SHA 6413d6bView commit details -
Improve the warning about Replay-less executions (#585)
* Improve the warning about Replay-less executions * tweak warning * only check if the browser was installed when a project was executed using replay * Update packages/playwright/src/reporter.ts Co-authored-by: Brian Vaughn <brian.david.vaughn@gmail.com> * Update packages/playwright/src/reporter.ts * Update packages/playwright/src/reporter.ts Co-authored-by: Brian Vaughn <brian.david.vaughn@gmail.com> * fmt * avoid redundant mixpanel calls --------- Co-authored-by: Brian Vaughn <brian.david.vaughn@gmail.com>
Configuration menu - View commit details
-
Copy full SHA for 543dfc9 - Browse repository at this point
Copy the full SHA 543dfc9View commit details -
Configuration menu - View commit details
-
Copy full SHA for 4f1dd73 - Browse repository at this point
Copy the full SHA 4f1dd73View commit details -
migrate log points to the new logger in `packages/test-utils/src/lega…
…cy-cli/main.ts` (#583) * migrate log points to the new logger * add getErrorTags util function * remove unused function * use getErrorTags to keep stacktrace * format errors from the Logger class * assign error tags to object key * pr feedback * truncate stack traces for maximum anonymity
Configuration menu - View commit details
-
Copy full SHA for 951bc7d - Browse repository at this point
Copy the full SHA 951bc7dView commit details
Commits on Jul 3, 2024
-
Configuration menu - View commit details
-
Copy full SHA for f0f488d - Browse repository at this point
Copy the full SHA f0f488dView commit details -
Only save test data for tests run with Replay browser (#586)
Only upload test metadata for tests that were run with the Replay browser. If a given test run has no tests that match this criteria, then we should not even upload any metadata for it.
Configuration menu - View commit details
-
Copy full SHA for 2d78191 - Browse repository at this point
Copy the full SHA 2d78191View commit details -
Initialize Mixpanel events for Cypress reporter (#587)
This will unblock the logger done by test-utils (which previously would not have been sent anywhere, because it waits until Mixpanel has been initialized).
Configuration menu - View commit details
-
Copy full SHA for 78660e4 - Browse repository at this point
Copy the full SHA 78660e4View commit details -
Configuration menu - View commit details
-
Copy full SHA for 09609e9 - Browse repository at this point
Copy the full SHA 09609e9View commit details -
migrate remaining logs in
test-utils/*
(#589)* migrate remaining logs * nit * err -> error
Configuration menu - View commit details
-
Copy full SHA for fe9b225 - Browse repository at this point
Copy the full SHA fe9b225View commit details -
Add missing fetch imports (#593)
* Add missing fetch imports * fixed undici mocking * actually fix it
Configuration menu - View commit details
-
Copy full SHA for 92e7c4b - Browse repository at this point
Copy the full SHA 92e7c4bView commit details -
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Configuration menu - View commit details
-
Copy full SHA for 75a32ec - Browse repository at this point
Copy the full SHA 75a32ecView commit details -
install
stack-utils
to all packages that inline the shared folder (#……594) * add stack-utils to all packages that inline the shared folder * remove stack-utils from root, add to packages/replay package * revert changes to root package.json and yarn.lock files * update yarn.lock * add changeset
Configuration menu - View commit details
-
Copy full SHA for 78f5c72 - Browse repository at this point
Copy the full SHA 78f5c72View commit details -
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Configuration menu - View commit details
-
Copy full SHA for b803eea - Browse repository at this point
Copy the full SHA b803eeaView commit details
Commits on Jul 4, 2024
-
Configuration menu - View commit details
-
Copy full SHA for cfb419b - Browse repository at this point
Copy the full SHA cfb419bView commit details -
Enable
upload.statusThreshold
usage in the Cypress plugin (#598)* Enable `upload.statusThreshold` usage in the Cypress plugin * rename type * Update packages/test-utils/src/types.ts Co-authored-by: Brian Vaughn <brian.david.vaughn@gmail.com> --------- Co-authored-by: Brian Vaughn <brian.david.vaughn@gmail.com>
Configuration menu - View commit details
-
Copy full SHA for 0f9764b - Browse repository at this point
Copy the full SHA 0f9764bView commit details
Commits on Jul 5, 2024
-
Fixed validation in the
resolveErrors
plugin (#597)* Fixed validation in the `resolveErrors` plugin * Use import instead of require to load `stack-utils` (#596) * remove redundant tsconfig options * ok, ok, esModuleInterop is still needed by ts-jest
Configuration menu - View commit details
-
Copy full SHA for 2ad92e5 - Browse repository at this point
Copy the full SHA 2ad92e5View commit details -
Remove unused bits from the test utils (#601)
* Remove unused bits from the test utils * fix things
Configuration menu - View commit details
-
Copy full SHA for ca8e62e - Browse repository at this point
Copy the full SHA ca8e62eView commit details -
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Configuration menu - View commit details
-
Copy full SHA for 5b3f3b1 - Browse repository at this point
Copy the full SHA 5b3f3b1View commit details -
Configuration menu - View commit details
-
Copy full SHA for f2b1c82 - Browse repository at this point
Copy the full SHA f2b1c82View commit details -
Configuration menu - View commit details
-
Copy full SHA for 02a4de3 - Browse repository at this point
Copy the full SHA 02a4de3View commit details
There are no files selected for viewing