deno_core icon indicating copy to clipboard operation
deno_core copied to clipboard

refactor: internal ops are under Deno.core.coreOps

Open bartlomieju opened this issue 5 months ago • 0 comments

This commit changes how ops defined in "deno_core" crate are exposed to JavaScript code.

They are no longer added to "Deno.core.ops", but to "Deno.core.coreOps", to make the distinction between them more obvious.

All ops that are required for embedders are now available under "Deno.core" namespace.

This refactor uncovered a lot of cruft and technical debt accumulated over the years - eg. there are several ops that are added unconditionally that are only ever used in testing code. This poses a question, if "deno_core" should expose some of the ops only when cfg(test) passes.

bartlomieju avatar Jan 31 '24 00:01 bartlomieju