From 2e43346f1ad23d153417ebbc9a4c94b5ed571166 Mon Sep 17 00:00:00 2001 From: Andy Jessop Date: Thu, 31 Oct 2024 12:12:28 +0100 Subject: [PATCH] chore: Improve readme for devtools package (#7138) * chore: Improve readme for devtools package * chore: remove internal link --- packages/wrangler-devtools/README.md | 74 +++++++++++++++++++++++++++- 1 file changed, 73 insertions(+), 1 deletion(-) diff --git a/packages/wrangler-devtools/README.md b/packages/wrangler-devtools/README.md index 13c14c908e3a..3f4365f7cda0 100644 --- a/packages/wrangler-devtools/README.md +++ b/packages/wrangler-devtools/README.md @@ -1,6 +1,78 @@ # Wrangler Devtools Pages Project -This package contains a Workers specific version of Chrome Devtools that is used by the Wrangler dev command. +This package contains a Workers specific version of Chrome Devtools that is used by the Wrangler dev command. It is a customized fork of Chrome DevTools specifically tailored for debugging Cloudflare Workers. This package provides Worker-specific functionality through carefully maintained patches on top of Chrome DevTools. + +## Overview + +This package is used across multiple Cloudflare products: + +- Workers Playground (`workers-playground`) +- Quick Editor (`@cloudflare/quick-edit`) +- Wrangler CLI via the `InspectorProxy` + +## Features + +Our customized DevTools implementation provides: + +- Source code viewing and live updates +- Network request inspection +- Worker-specific UI optimizations + +## Development + +We maintain this fork by applying patches on top of Chrome DevTools. These patches need to be periodically rebased as Chrome DevTools evolves. + +**Key Development Tasks:** + +- Generating patches from our customizations +- Rebasing patches onto new Chrome DevTools versions +- Testing functionality across all integration points + +## Updating DevTools + +We perform quarterly updates to stay current with upstream Chrome DevTools. The update process involves: + +1. Cloning the devtools-frontend repo +2. Applying our existing patches +3. Rebasing onto the latest Chrome DevTools +4. Regenerating patches +5. Thorough testing across all integration points + +**For detailed instructions on updating DevTools, please refer to our internal documentation on keeping devtools up-to-date.** + +## Testing + +Two methods are available for testing updates: + +**Local Development:** + +- Build and serve DevTools locally +- Test against local Playground instance +- Make targeted fixes as needed + +**Preview Builds:** + +- Add `preview:wrangler-devtools` label to PRs +- Use deployed preview URL for testing +- Verify functionality in Playground + +## Acceptance Criteria + +Our DevTools implementation must maintain full functionality across: + +- Console operations (logging, errors, filters) +- Source code viewing and debugging +- Network request inspection +- Worker-specific UI customizations + +## Contributing + +When making changes: + +- Keep patches minimal and targeted +- Prefer CSS-based UI modifications +- Test thoroughly across all integration points +- Document any new patches or modifications ## Deployment