Ignore files in deps dir when resolving wit packages (#1061)

* fix: ignore files in deps dir

* run cargo fmt

* improve comment and revert .gitignore

* test: files in deps folder are ignored

Author: cardoso

.gitignore

@@ -6,3 +6,4 @@ publish
 *.wasm
 *.wat
 test.config
+

crates/wit-parser/src/resolve.rs

@@ -112,6 +112,14 @@ impl Resolve {
             for dep in path.read_dir().context("failed to read directory")? {
                 let dep = dep.context("failed to read directory iterator")?;
                 let path = dep.path();
+
+                // Files in deps dir are ignored for now to avoid accidentally
+                // including things like `.DS_Store` files in the call below to
+                // `parse_dir`.
+                if path.is_file() {
+                    continue;
+                }
+
                 let pkg = UnresolvedPackage::parse_dir(&path)
                     .with_context(|| format!("failed to parse package: {}", path.display()))?;
                 let prev = ret.insert(pkg.name.clone(), pkg);

crates/wit-parser/tests/ui/ignore-files-deps/deps/bar/types.wit

@@ -0,0 +1,2 @@
+package foo:bar
+interface types {}

crates/wit-parser/tests/ui/ignore-files-deps/deps/ignore-me.txt

@@ -0,0 +1 @@
+this file should be ignored by the parser

crates/wit-parser/tests/ui/ignore-files-deps/world.wit

@@ -0,0 +1,5 @@
+package foo:foo
+
+world foo {
+  import foo:bar/types
+}