fix: repl

Author: load1n9

Cargo.lock

@@ -7,7 +7,7 @@ authors = ["the Andromeda team"]
 edition = "2024"
 license = "Mozilla Public License 2.0"
 repository = "https://github.com/tryandromeda/andromeda"
-version = "0.1.0-draft39"
+version = "0.1.0-draft40"
 
 [workspace.dependencies]
 andromeda-core = { path = "core" }

Cargo.toml

@@ -19,7 +19,10 @@ use nova_vm::{
             Agent, JsResult,
             agent::{GcAgent, Options},
         },
-        scripts_and_modules::script::{parse_script, script_evaluation},
+        scripts_and_modules::{
+            module::module_semantics::source_text_module_records::parse_module,
+            script::{parse_script, script_evaluation},
+        },
         types::{
             self, InternalMethods, IntoValue, Object, OrdinaryObject, PropertyDescriptor,
             PropertyKey, Value,
@@ -287,25 +290,25 @@ fn initialize_global_object(agent: &mut Agent, global_object: Object, mut gc: Gc
 
     let mut extensions = recommended_extensions();
     for extension in &mut extensions {
-        for file in &extension.files {
+        for (idx, file) in extension.files.iter().enumerate() {
+            let specifier = format!("<ext:{}:{}>", extension.name, idx);
             let source_text = types::String::from_str(agent, file, gc.nogc());
-            let script = match parse_script(
+            let module = match parse_module(
                 agent,
                 source_text,
                 agent.current_realm(gc.nogc()),
-                true,
-                None,
+                Some(std::rc::Rc::new(specifier.clone())),
                 gc.nogc(),
             ) {
-                Ok(script) => script,
+                Ok(module) => module,
                 Err(errors) => {
-                    handle_parse_errors(errors, "<extension>", file);
+                    handle_parse_errors(errors, &specifier, file);
                     std::process::exit(1);
                 }
             };
-            if script_evaluation(agent, script.unbind(), gc.reborrow()).is_err() {
-                eprintln!("⚠️  Warning: Error loading extension");
-                handle_runtime_error_with_message("Script evaluation failed".to_string());
+            if agent.run_parsed_module(module.unbind(), None, gc.reborrow()).unbind().is_err() {
+                eprintln!("⚠️  Warning: Error loading extension {}", specifier);
+                handle_runtime_error_with_message("Module evaluation failed".to_string());
             }
         }
         // Register native ops

cli/src/repl.rs

@@ -733,7 +733,7 @@ const Andromeda = {
       // If no more listeners, unregister the native handler
       if (listeners.size === 0) {
         signalListeners.delete(signal);
-        internal_remove_signal_listener(signal, () => {});
+        __andromeda__.internal_remove_signal_listener(signal, () => {});
       }
     }
   },
@@ -762,7 +762,11 @@ const Andromeda = {
    */
   get serve() {
     // @ts-ignore - internal use
-    return globalThis.__andromeda_http_serve;
+    const httpServe = globalThis.__andromeda_http_serve;
+    if (typeof httpServe !== "function") {
+      throw new Error("HTTP extension is not available. Make sure the 'serve' feature is enabled.");
+    }
+    return httpServe;
   },
 };
 
@@ -890,4 +894,9 @@ function decodeURI(input: string): string {
 }
 
 // Export Andromeda globally
-(globalThis as any).Andromeda = Andromeda;
+Object.defineProperty(globalThis, "Andromeda", {
+  value: Andromeda,
+  writable: false,
+  enumerable: true,
+  configurable: false,
+});

namespace/mod.ts

@@ -1,12 +1,6 @@
 // This Source Code Form is subject to the terms of the Mozilla Public
 // License, v. 2.0. If a copy of the MPL was not distributed with this
 // file, You can obtain one at https://mozilla.org/MPL/2.0/.
-
-// deno-lint-ignore-file no-explicit-any
-
-// Import existing Request/Response implementations
-// These are loaded by the fetch extension, so we don't need to import them here
-
 export interface ServeHandler {
   (request: Request): Response | Promise<Response>;
 }
@@ -23,7 +17,7 @@ export interface ServeInit extends ServeOptions {
 
 export interface HttpServer {
   finished: Promise<void>;
-  shutdown(): Promise<void>;
+  shutdown(): void;
   unref(): void;
   ref(): void;
 }
@@ -43,10 +37,10 @@ export function serve(handler: ServeHandler | ServeInit): HttpServer {
   serverHandlers.set(serverId, options.handler);
 
   return {
-    finished: new Promise((resolve) => {
+    finished: new Promise((_resolve) => {
       // TODO: Track server lifecycle
     }),
-    shutdown: async () => {
+    shutdown: () => {
       __andromeda__.internal_http_close(serverId);
       serverHandlers.delete(serverId);
     },
@@ -59,6 +53,8 @@ export function serve(handler: ServeHandler | ServeInit): HttpServer {
   };
 }
 
-// Export serve function globally for namespace to pick up
-// @ts-ignore - internal use
-globalThis.__andromeda_http_serve = serve;
+// Store serve function for namespace access, but use a safer approach
+if (typeof globalThis === "object" && globalThis) {
+  // @ts-ignore - internal use
+  globalThis.__andromeda_http_serve = serve;
+}