Merge pull request #192 from sumleo/fix/path-resolution-unwrap

Fix panic on missing first segment in package path resolution

Author: fibonacci1729

crates/wac-parser/src/resolution.rs

@@ -2553,7 +2553,8 @@ impl<'a> AstResolver<'a> {
             }
 
             // Otherwise, project into the parent based on the current segment
-            let export = match found.unwrap() {
+            let Some(parent) = found else { break };
+            let export = match parent {
                 // The parent is an interface or instance
                 ItemKind::Type(Type::Interface(id)) | ItemKind::Instance(id) => {
                     state.graph.types()[id].exports.get(segment).copied()

crates/wac-parser/tests/resolution/fail/missing-package-definition.wac

@@ -0,0 +1,5 @@
+package test:comp;
+
+interface i {
+    use foo:bar/nonexistent.{a};
+}

crates/wac-parser/tests/resolution/fail/missing-package-definition.wac.result

@@ -0,0 +1,10 @@
+failed to resolve document
+
+  × package `foo:bar` has no export named `nonexistent`
+   ╭─[tests/resolution/fail/missing-package-definition.wac:4:17]
+ 3 │ interface i {
+ 4 │     use foo:bar/nonexistent.{a};
+   ·                 ─────┬─────
+   ·                      ╰── unknown export `nonexistent`
+ 5 │ }
+   ╰────

crates/wac-parser/tests/resolution/fail/missing-package-definition/foo/bar.wat

@@ -0,0 +1,3 @@
+(component
+  (import "foo" (func))
+)