Give user facing error messages when there are bad paths (#35871)

Make ModulePath::components() fallible

Give user facing error messages when there are bad paths

Eg
✖ Error: Unable to start push to http://127.0.0.1:8000
Invalid component "_" in module path mastra/_

GitOrigin-RevId: 1391bcbf35bbdb8b2a210072ad50ae8132b142cf

Author: nipunn1313

crates/convex/sync_types/src/module_path.rs

@@ -68,10 +68,15 @@ impl ModulePath {
         self.path.components().map(|component| match component {
             Component::Normal(c) => c
                 .to_str()
-                .context("Non-unicode data in module path?")?
+                .with_context(|| format!("Non-unicode data in module path {}", self.as_str()))?
                 .parse()
-                .context("Invalid component in module path"),
-            c => anyhow::bail!("Unexpected component {c:?}"),
+                .with_context(|| {
+                    format!("Invalid component {c:?} in module path {}", self.as_str())
+                }),
+            c => anyhow::bail!(
+                "Unexpected component {c:?} in module path {}",
+                self.as_str()
+            ),
         })
     }
 

crates/model/src/components/file_based_routing.rs

@@ -28,7 +28,15 @@ pub fn file_based_exports(
     for (module_path, module) in functions {
         let stripped = module_path.clone().strip();
 
-        let identifiers = stripped.components().collect::<anyhow::Result<Vec<_>>>()?;
+        let identifiers = stripped
+            .components()
+            .collect::<anyhow::Result<Vec<_>>>()
+            .or_else(|e| {
+                anyhow::bail!(ErrorMetadata::bad_request(
+                    "InvalidModulePath",
+                    e.to_string()
+                ))
+            })?;
         for function in &module.functions {
             if function.visibility != Some(Visibility::Public) {
                 continue;