fix: replace broad exception catches with specific exception types

Replace overly broad `except Exception` catches with specific exception types
to improve error handling clarity and debugging:

- subtensor_interface.py: Handle both TimeoutError and asyncio.TimeoutError
  for substrate initialization timeout (resolves TODO comment)
- utils.py (is_valid_github_url): Catch ValueError, TypeError, AttributeError
  for URL parsing exceptions (resolves TODO comment)
- utils.py (normalize_hyperparameters): Catch KeyError, ValueError, TypeError,
  AttributeError for parameter normalization
- wallets.py (new_hotkey): Catch ValueError, TypeError for Keypair.create_from_uri
- wallets.py (new_coldkey): Catch ValueError, TypeError for Keypair.create_from_uri
- wallets.py (wallet_create): Catch ValueError, TypeError, KeyFileError for
  keypair and wallet creation

This change improves code quality by making exception handling more explicit
and easier to debug while maintaining the same error recovery behavior.

Author: daniel

bittensor_cli/src/bittensor/subtensor_interface.py

@@ -135,7 +135,7 @@ async def __aenter__(self):
             try:
                 await self.substrate.initialize()
                 return self
-            except TimeoutError:  # TODO verify
+            except (TimeoutError, asyncio.TimeoutError):
                 err_console.print(
                     "\n[red]Error[/red]: Timeout occurred connecting to substrate. "
                     f"Verify your chain and network settings: {self}"

bittensor_cli/src/bittensor/utils.py

@@ -850,7 +850,7 @@ def normalize_hyperparameters(
                     norm_value = norm_value.to_dict()
             else:
                 norm_value = value
-        except Exception:
+        except (KeyError, ValueError, TypeError, AttributeError):
             # bittensor.logging.warning(f"Error normalizing parameter '{param}': {e}")
             norm_value = "-"
         if not json_output:
@@ -1728,7 +1728,7 @@ def is_valid_github_url(url: str) -> bool:
             return False
 
         return True
-    except Exception:  # TODO figure out the exceptions that can be raised in here
+    except (ValueError, TypeError, AttributeError):
         return False
 
 

bittensor_cli/src/commands/wallets.py

@@ -377,7 +377,7 @@ async def new_hotkey(
         if uri:
             try:
                 keypair = Keypair.create_from_uri(uri)
-            except Exception as e:
+            except (ValueError, TypeError) as e:
                 print_error(f"Failed to create keypair from URI {uri}: {str(e)}")
                 return
             wallet.set_hotkey(keypair=keypair, encrypt=use_password)
@@ -428,7 +428,7 @@ async def new_coldkey(
         if uri:
             try:
                 keypair = Keypair.create_from_uri(uri)
-            except Exception as e:
+            except (ValueError, TypeError) as e:
                 print_error(f"Failed to create keypair from URI {uri}: {str(e)}")
             wallet.set_coldkey(keypair=keypair, encrypt=False, overwrite=False)
             wallet.set_coldkeypub(keypair=keypair, encrypt=False, overwrite=False)
@@ -501,7 +501,7 @@ async def wallet_create(
                 "hotkey_ss58": wallet.hotkeypub.ss58_address,
                 "coldkey_ss58": wallet.coldkeypub.ss58_address,
             }
-        except Exception as e:
+        except (ValueError, TypeError, KeyFileError) as e:
             err = f"Failed to create keypair from URI: {str(e)}"
             print_error(err)
             output_dict["error"] = err