Skip to content

Conversation

@BenoitZugmeyer
Copy link
Member

@BenoitZugmeyer BenoitZugmeyer commented Nov 14, 2025

Motivation

The webpack configuration for the devtools extension is rather tedious to maintain. WXT offers a better DX out of the box, with a single dependency.

We tried going with a basic Vite config file, but it was proven too cumbersome because of the need to use one config per entry point.

Changes

Replace webpack with WXT

Test instructions

In the extension folder, run yarn dev to open a browser with the extension installed.
You can also run yarn build to build it.

Checklist

  • Tested locally
  • Tested on staging
  • Added unit tests for this change.
  • Added e2e/integration tests for this change.

@BenoitZugmeyer BenoitZugmeyer force-pushed the benoit/migrate-devext-to-wxt branch from b375739 to 8ec48a9 Compare November 14, 2025 15:00
@cit-pr-commenter
Copy link

cit-pr-commenter bot commented Nov 14, 2025

Bundles Sizes Evolution

📦 Bundle Name Base Size Local Size 𝚫 𝚫% Status
Rum 164.05 KiB 164.05 KiB 0 B 0.00%
Rum Profiler 4.84 KiB 4.84 KiB 0 B 0.00%
Rum Recorder 19.50 KiB 19.50 KiB 0 B 0.00%
Logs 55.91 KiB 55.91 KiB 0 B 0.00%
Flagging 944 B 944 B 0 B 0.00%
Rum Slim 121.33 KiB 121.33 KiB 0 B 0.00%
Worker 23.63 KiB 23.63 KiB 0 B 0.00%
🚀 CPU Performance
Action Name Base CPU Time (ms) Local CPU Time (ms) 𝚫%
RUM - add global context 0.0038 0.0068 +78.95%
RUM - add action 0.0109 0.0195 +78.90%
RUM - add error 0.0105 0.0204 +94.29%
RUM - add timing 0.0025 0.0053 +112.00%
RUM - start view 0.003 0.0069 +130.00%
RUM - start/stop session replay recording 0.0006 0.0015 +150.00%
Logs - log message 0.0122 0.0231 +89.34%
🧠 Memory Performance
Action Name Base Memory Consumption Local Memory Consumption 𝚫
RUM - add global context 25.75 KiB 25.43 KiB -327 B
RUM - add action 96.59 KiB 109.57 KiB +12.97 KiB
RUM - add timing 24.39 KiB 24.48 KiB +84 B
RUM - add error 82.49 KiB 114.60 KiB +32.11 KiB
RUM - start/stop session replay recording 23.62 KiB 23.74 KiB +130 B
RUM - start view 439.69 KiB 479.96 KiB +40.27 KiB
Logs - log message 44.42 KiB 45.18 KiB +781 B

🔗 RealWorld

@datadog-official
Copy link

datadog-official bot commented Nov 14, 2025

✅ Tests

🎉 All green!

❄️ No new flaky tests detected
🧪 All tests passed

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: ab845f9 | Docs | Datadog PR Page | Was this helpful? Give us feedback!

@BenoitZugmeyer BenoitZugmeyer force-pushed the benoit/migrate-devext-to-wxt branch 4 times, most recently from f9e1b58 to 749be31 Compare November 14, 2025 15:57
@BenoitZugmeyer BenoitZugmeyer marked this pull request as ready for review November 14, 2025 16:03
@BenoitZugmeyer BenoitZugmeyer requested a review from a team as a code owner November 14, 2025 16:03
@BenoitZugmeyer BenoitZugmeyer force-pushed the benoit/migrate-devext-to-wxt branch from 749be31 to 2d15023 Compare November 14, 2025 16:35
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we clarify why we need this? As we didn't have it before.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We did have it before, just not as a separate file. I linked to MDN docs in the README to clarify a bit the architecture.

@BeltranBulbarellaDD
Copy link
Contributor

Screenshot 2025-11-17 at 14 18 25

I think we could remove the suggestions/warnings from the build so we don't have so much noice the terminal?

@BenoitZugmeyer BenoitZugmeyer requested a review from a team as a code owner November 17, 2025 17:52
@BenoitZugmeyer
Copy link
Member Author

BenoitZugmeyer commented Nov 18, 2025

@BeltranBulbarellaDD

I think we could remove the suggestions/warnings from the build so we don't have so much noice the terminal?

I removed the warning. I don't think we can remove the dotenv log / tip unfortunately. I'll see if we can avoid building the extension when running yarn build at the toplevel, to reduce the noise there.

@BenoitZugmeyer BenoitZugmeyer force-pushed the benoit/migrate-devext-to-wxt branch from e4dea2e to ab845f9 Compare November 18, 2025 10:34
@BeltranBulbarellaDD
Copy link
Contributor

@BeltranBulbarellaDD

I think we could remove the suggestions/warnings from the build so we don't have so much noice the terminal?

I removed the warning. I don't think we can remove the dotenv log / tip unfortunately. I'll see if we can avoid building the extension when running yarn build at the toplevel, to reduce the noise there.

Excellent, sounds good!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants