Clean

Author: DustinJSilk

packages/qwik-grpc/src/index.ts

@@ -53,7 +53,6 @@ function findBufGenTemplate(protoPath: string, outDir: string): string {
 
   for (const file of candidates) {
     if (fs.existsSync(file)) {
-      console.log(`[qwikGrpc] Using ${path.relative(process.cwd(), file)}`);
       return fs.readFileSync(file, 'utf8');
     }
   }
@@ -74,9 +73,6 @@ function runBufGenerate(protoPath: string, outDir: string, flags: string): strin
     execSync(`buf generate ${protoPath} --template ${tmpPath} ${flags}`, {
       stdio: 'inherit',
     });
-  } catch (err) {
-    console.error('[qwikGrpc] buf generate failed:', err);
-    throw err;
   } finally {
     fs.unlinkSync(tmpPath);
   }
@@ -86,8 +82,11 @@ function runBufGenerate(protoPath: string, outDir: string, flags: string): strin
     for (const entry of fs.readdirSync(dir)) {
       const full = path.join(dir, entry);
       const stat = fs.statSync(full);
-      if (stat.isDirectory()) walk(full);
-      else if (entry.endsWith('_pb.ts')) files.push(full);
+      if (stat.isDirectory()) {
+        walk(full);
+      } else if (entry.endsWith('_pb.ts')) {
+        files.push(full);
+      }
     }
   };
   walk(outDir);
@@ -97,26 +96,22 @@ function runBufGenerate(protoPath: string, outDir: string, flags: string): strin
 
 // Creates a list of Services by reading the generated clients
 function getServices(outDir: string, files: string[]): Service[] {
-  return files
-    .map((filePath) => {
-      const fileContent = fs.readFileSync(filePath, 'utf8');
-
-      // Match pattern: export const FooService: GenService<...
-      const match = fileContent.match(/export\s+const\s+(\w+)Service\s*:\s*GenService\s*</);
-      if (!match) {
-        console.warn(`[qwikGrpc] No service export found in ${filePath}`);
-        return null;
-      }
+  return files.map((filePath) => {
+    const fileContent = fs.readFileSync(filePath, 'utf8');
+
+    // Match pattern: export const FooService: GenService<...
+    const match = fileContent.match(/export\s+const\s+(\w+)Service\s*:\s*GenService\s*</);
+    if (!match) {
+      throw new Error(`[qwikGrpc] No service export found in ${filePath}`);
+    }
 
-      const name = match[1]; // e.g. "Foo"
-      const instanceName = name[0].toLowerCase() + name.slice(1); // e.g. "foo"
-      const serviceName = `${name}Service`; // e.g. "FooService"
-      const relPath =
-        './' + path.relative(outDir, filePath).replace(/\\/g, '/').replace(/\.ts$/, '');
+    const name = match[1]; // e.g. "Foo"
+    const instanceName = name[0].toLowerCase() + name.slice(1); // e.g. "foo"
+    const serviceName = `${name}Service`; // e.g. "FooService"
+    const relPath = './' + path.relative(outDir, filePath).replace(/\\/g, '/').replace(/\.ts$/, '');
 
-      return { serviceName, name, instanceName, path: relPath };
-    })
-    .filter((s): s is Service => !!s);
+    return { serviceName, name, instanceName, path: relPath };
+  });
 }
 
 // Generates the client.ts file which registers the clients