Address review comments

Author: incrypto32

packages/cli/src/commands/init.ts

@@ -594,6 +594,17 @@ async function processInitForm(
       message: `IPFS node to use for fetching subgraph manifest`,
       initial: ipfsUrl,
       skip: () => !isComposedSubgraph,
+      validate: value => {
+        if (!value) {
+          return 'IPFS node URL cannot be empty';
+        }
+        try {
+          new URL(value);
+          return true;
+        } catch {
+          return 'Please enter a valid URL';
+        }
+      },
       result: value => {
         ipfsNode = value;
         initDebugger.extend('processInitForm')('ipfs: %O', value);
@@ -615,9 +626,6 @@ async function processInitForm(
       initial: initContract,
       validate: async (value: string): Promise<string | boolean> => {
         if (isComposedSubgraph) {
-          if (!ipfsNode) {
-            return true;
-          }
           const ipfs = createIpfsClient(ipfsNode);
           const { valid, error } = await validateSubgraphNetworkMatch(ipfs, value, network.id);
           if (!valid) {
@@ -835,7 +843,7 @@ async function processInitForm(
     await promptManager.executeInteractive();
 
     // Validate network matches if loading from IPFS
-    if (ipfsNode && source?.startsWith('Qm')) {
+    if (isComposedSubgraph) {
       const ipfs = createIpfsClient(ipfsNode);
       try {
         const { valid, error } = await validateSubgraphNetworkMatch(ipfs, source!, network.id);

packages/cli/src/utils.ts

@@ -75,14 +75,8 @@ export async function validateSubgraphNetworkMatch(
 
     // Get network from first data source
     const sourceNetwork = allSources[0].network;
-    if (!sourceNetwork) {
-      return { valid: true }; // Network not specified in source, skip validation
-    }
-
-    const normalizedSourceNetwork = sourceNetwork.toLowerCase();
-    const normalizedTargetNetwork = targetNetwork.toLowerCase();
 
-    if (normalizedSourceNetwork !== normalizedTargetNetwork) {
+    if (sourceNetwork !== targetNetwork) {
       return {
         valid: false,
         error: `Network mismatch: The source subgraph is indexing the '${sourceNetwork}' network, but you're creating a subgraph for '${targetNetwork}' network. When composing subgraphs, they must index the same network.`,