fix: allow setting static patterns when creating a user worker (#471)

laktek · web-flow · commit d2c21c16ca07 · 2025-01-09T15:22:06.000+11:00
* fix: allow setting static patterns when creating a user worker

* fix: clippy

* fix: make static_patterns optional

* formatting

* Trigger Build
diff --git a/crates/base/src/worker/pool.rs b/crates/base/src/worker/pool.rs
@@ -365,6 +365,7 @@ impl WorkerPool {
                         maybe_jsx_import_source_config,
                         maybe_s3_fs_config,
                         maybe_tmp_fs_config,
+                        static_patterns,
                         ..
                     } = worker_options;
 
@@ -381,7 +382,7 @@ impl WorkerPool {
                                 maybe_module_code,
                                 maybe_entrypoint,
                                 maybe_decorator,
-                                static_patterns: vec![],
+                                static_patterns,
 
                                 maybe_jsx_import_source_config,
                                 maybe_s3_fs_config,
@@ -714,7 +715,7 @@ pub async fn create_user_worker_pool(
                             None => break,
                             Some(UserWorkerMsgs::Create(worker_options, tx)) => {
                                 worker_pool.create_user_worker(WorkerContextInitOpts {
-                                    static_patterns: static_patterns.clone(),
+                                    static_patterns: [worker_options.static_patterns, static_patterns.clone()].concat(),
                                     maybe_jsx_import_source_config: {
                                         if worker_options.maybe_jsx_import_source_config.is_some() {
                                             worker_options.maybe_jsx_import_source_config
diff --git a/deno.json b/deno.json
@@ -4,7 +4,7 @@
     "./crates/base/test_cases"
   ],
   "fmt": {
-    "useTabs": true,
+    "useTabs": false,
     "lineWidth": 100,
     "indentWidth": 4,
     "singleQuote": true,
@@ -14,4 +14,4 @@
     "npm:@meowmeow/foobar": "npm:is-odd",
     "openai": "npm:openai"
   }
-}
+}
diff --git a/examples/main/index.ts b/examples/main/index.ts
@@ -131,6 +131,9 @@ Deno.serve(async (req: Request) => {
         //
         const cpuTimeSoftLimitMs = 10000;
         const cpuTimeHardLimitMs = 20000;
+        const staticPatterns = [
+            './examples/**/*.html',
+        ];
 
         return await EdgeRuntime.userWorkers.create({
             servicePath,
@@ -143,6 +146,7 @@ Deno.serve(async (req: Request) => {
             netAccessDisabled,
             cpuTimeSoftLimitMs,
             cpuTimeHardLimitMs,
+            staticPatterns,
             // maybeEszip,
             // maybeEntrypoint,
             // maybeModuleCode,
diff --git a/examples/serve-html/bar.html b/examples/serve-html/bar.html
@@ -0,0 +1,3 @@
+<html>
+  <h1>Bar</h1>
+</html>
diff --git a/examples/serve-html/foo.html b/examples/serve-html/foo.html
@@ -0,0 +1,3 @@
+<html>
+  <h1>Foo</h1>
+</html>
diff --git a/examples/serve-html/index.ts b/examples/serve-html/index.ts
@@ -0,0 +1,9 @@
+import { join } from 'https://deno.land/std/path/mod.ts';
+
+Deno.serve(async (req) => {
+	if (req.url.endsWith('/foo')) {
+		return new Response(await Deno.readTextFile(join(import.meta.dirname, 'foo.html')));
+	} else {
+		return new Response(await Deno.readTextFile(join(import.meta.dirname, 'bar.html')));
+	}
+});
diff --git a/ext/workers/lib.rs b/ext/workers/lib.rs
@@ -91,6 +91,8 @@ pub struct UserWorkerCreateOptions {
     tmp_fs_config: Option<TmpFsConfig>,
 
     context: Option<JsonMap>,
+    #[serde(default)]
+    static_patterns: Vec<String>,
 }
 
 #[op2(async)]
@@ -131,6 +133,7 @@ pub async fn op_user_worker_create(
             tmp_fs_config: maybe_tmp_fs_config,
 
             context,
+            static_patterns,
         } = opts;
 
         let user_worker_options = WorkerContextInitOpts {
@@ -165,7 +168,7 @@ pub async fn op_user_worker_create(
                 }
             }),
 
-            static_patterns: vec![],
+            static_patterns,
             import_map_path,
             timing: None,
 

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,3 @@`
	`1`	`+<html>`
	`2`	`+ <h1>Bar</h1>`
	`3`	`+</html>`
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,3 @@`
	`1`	`+<html>`
	`2`	`+ <h1>Foo</h1>`
	`3`	`+</html>`