From 5edd102f3f912a53c7bcad3b0fa4feb672ada323 Mon Sep 17 00:00:00 2001
From: David Sherret <dsherret@users.noreply.github.com>
Date: Fri, 29 Sep 2023 09:26:25 -0400
Subject: refactor(cli): make `CliNpmResolver` a trait (#20732)

This makes `CliNpmResolver` a trait. The terminology used is:

- **managed** - Deno manages the node_modules folder and does an
auto-install (ex. `ManagedCliNpmResolver`)
- **byonm** - "Bring your own node_modules" (ex. `ByonmCliNpmResolver`,
which is in this PR, but unimplemented at the moment)

Part of #18967
---
 cli/ops/mod.rs | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

(limited to 'cli/ops')

diff --git a/cli/ops/mod.rs b/cli/ops/mod.rs
index ab3a55468..eb75dc272 100644
--- a/cli/ops/mod.rs
+++ b/cli/ops/mod.rs
@@ -12,7 +12,7 @@ pub mod bench;
 pub mod jupyter;
 pub mod testing;
 
-pub fn cli_exts(npm_resolver: Arc<CliNpmResolver>) -> Vec<Extension> {
+pub fn cli_exts(npm_resolver: Arc<dyn CliNpmResolver>) -> Vec<Extension> {
   vec![
     #[cfg(not(feature = "__runtime_js_sources"))]
     cli::init_ops(npm_resolver),
@@ -33,7 +33,7 @@ deno_core::extension!(cli,
     "99_main.js"
   ],
   options = {
-    npm_resolver: Arc<CliNpmResolver>,
+    npm_resolver: Arc<dyn CliNpmResolver>,
   },
   state = |state, options| {
     state.put(options.npm_resolver);
@@ -51,6 +51,6 @@ deno_core::extension!(cli,
 #[op2]
 #[string]
 fn op_npm_process_state(state: &mut OpState) -> Result<String, AnyError> {
-  let npm_resolver = state.borrow_mut::<Arc<CliNpmResolver>>();
+  let npm_resolver = state.borrow_mut::<Arc<dyn CliNpmResolver>>();
   Ok(npm_resolver.get_npm_process_state())
 }
-- 
cgit v1.2.3