module-registry
$
npx mdskill add cloudflare/workerd/module-registryAnalyze, debug, or modify module resolution logic within workerd's module registry.
- Assists developers needing to understand module compilation, evaluation, or registration code.
- Provides necessary context via pointers to documentation on legacy and new registry designs.
- Guides work by directing attention to specific reference documents detailing architecture and flow.
- Delivers structured guidance and links to critical design specifications for implementation.
SKILL.md
.github/skills/module-registryView on GitHub ↗
--- name: module-registry description: Load when working with the module registry in workerd — reading, modifying, debugging, or reviewing module resolution, compilation, evaluation, or registration code. Provides pointers to three reference documents covering the legacy registry, V8 module internals, and the new registry design. --- ## Module Registry Context When working with any code related to module resolution, compilation, evaluation, or registration in workerd, you **must** read the relevant reference documents before making changes or providing analysis. These documents capture the design decisions, data flow, thread safety model, and integration points that are not obvious from reading the source alone. ### Reference Documents Read these documents (using the Read tool) before proceeding with any module registry work: 1. **`docs/reference/detail/new-module-registry.md`** — The new `jsg::modules::ModuleRegistry` implementation. Covers the two-layer architecture (shared `ModuleRegistry` + per-isolate `IsolateModuleRegistry`), bundle types, resolution priority, compile cache, evaluation flow, and thread safety model. **Read this first** for any work on the new registry. 2. **`docs/reference/detail/legacy-module-registry.md`** — The legacy `ModuleRegistryImpl<TypeWrapper>` implementation. Covers the per-isolate design, `ModuleInfo` structure, `DynamicImportHandler`, and how it interfaces with V8. **Read this** when working on code that still uses the legacy path, or when understanding the dispatch between legacy and new. 3. **`docs/reference/detail/v8-module-internals.md`** — How V8 handles modules internally. Covers `v8::Module` status transitions, `InstantiateModule`, `Evaluate`, `SyntheticModule`, resolve callbacks, and `import.meta`. **Read this** when debugging V8 module state issues, understanding callback contracts, or working on the boundary between workerd and V8. ### When to Read Which | Task | Documents to read | | ------------------------------------------------- | ----------------------------------------------- | | Modifying new registry resolution or caching | new-module-registry, v8 | | Adding a new built-in module | new-module-registry, legacy-module-registry | | Debugging module-not-found errors | all three | | Modifying legacy registry code | legacy-module-registry, v8 | | Understanding legacy-to-new dispatch | all three | | Working on `import.meta` behavior | new-module-registry, v8 | | Working on CJS/require() interop | new-module-registry, legacy-module-registry | | Debugging module evaluation failures | new-module-registry, legacy-module-registry, v8 | | Reviewing a PR that touches module code | all three | | Working on compile cache or cross-isolate sharing | new-module-registry | | Working on dynamic import | all three | | Working on source phase imports (Wasm) | new-module-registry, v8 | ### Key Source Files These are the primary files involved in module handling: - `src/workerd/jsg/modules-new.h` / `.c++` — New registry implementation - `src/workerd/jsg/modules.h` / `.c++` — Legacy registry implementation - `src/workerd/jsg/resource.h:1913–1922` — Legacy vs new dispatch at context creation - `src/workerd/jsg/jsg.c++:357–395` — `Lock::resolveModule` dispatch - `src/workerd/jsg/setup.h:291–297` — `isUsingNewModuleRegistry()` flag - `src/workerd/io/worker-modules.h` — `newWorkerModuleRegistry` construction - `src/workerd/api/modules.h` — Built-in module registration (both legacy and new) - `src/workerd/jsg/observer.h` — `ResolveObserver` metrics interface ### Procedure 1. **Read** the relevant reference documents listed above for your task. 2. **Identify** whether the code path uses the new or legacy registry (check `isUsingNewModuleRegistry()` usage in the surrounding code). 3. **Proceed** with your work, using the reference documents to inform your understanding of data flow, thread safety, and integration contracts. Do not skip reading the reference documents. The module registry has subtle invariants around thread safety, V8 module status transitions, and evaluation context (IoContext suppression) that are easy to violate without full context.
More from cloudflare/workerd
- add-autogateStep-by-step guide for adding a new autogate to workerd for gradual rollout of risky changes, including enum registration, string mapping, usage pattern, and testing.
- add-compat-flagStep-by-step guide for adding a new compatibility flag to workerd, including capnp schema, C++ usage, testing, and documentation requirements.
- bazel-test-hygieneMandatory rules for running bazel tests during development. Load this skill before running any bazel test command, especially when validating fixes or verifying regression tests. Prevents false confidence from cached results, filter flags that silently match nothing, and partial test runs that miss breakage.
- commit-categoriesCommit categorization rules for changelogs and "what's new" summaries. MUST be loaded before categorizing commits in changelog or whats-new commands. Provides the canonical path-based category table used to group commits by area.
- dad-jokesAfter completing any task that took more than ~5 tool calls, or after long-running builds/tests finish, load this skill and deliver a dad joke to lighten the mood. Also load before any user-requested joke, pun, or limerick. Never improvise jokes without loading this skill first.
- find-and-run-testsHow to find, build, and run tests in workerd. Covers wd-test, kj_test target naming, bazel query patterns, and common flags. Also covers parent project integration tests if workerd is used as a submodule. Load this skill when you need to locate or run a test and aren't sure of the exact target name or invocation.
- identify-reviewerIdentifies the local user's GitHub account and git identity before performing code reviews. Load this skill at the start of any PR review, code review, or commit log analysis so findings can be framed relative to the user's own prior comments, commits, and approval status.
- investigation-notesStructured scratch tracking document for investigation state during bug hunts - prevents re-reading code, losing context, and rabbit holes; maintains external memory so you don't re-derive conclusions
- kj-styleKJ/workerd C++ style guidelines for code review. Covers naming, type usage, memory management, error handling, inheritance, constness, and formatting conventions. Load this skill when reviewing or writing C++ code in the workerd codebase.
- markdown-draftsUse markdown formatting when drafting content intended for external systems (GitHub issues/PRs, Jira tickets, wiki pages, design docs, etc.) so formatting is preserved when the user copies it. Load this skill before producing any draft the user will paste elsewhere.