refactor: asset addresses constants as typed dicts

[matias-gonz]

Closes #28

[timbrinded]

lgtm

[Copilot]

Pull Request Overview

This PR refactors asset address management by introducing typed dictionaries for network-specific asset addresses. The changes replace hardcoded asset address constants with a more flexible, network-aware configuration system.

Key changes:

• Introduced NetworkAssets TypedDict and network-specific asset dictionaries (mainnet, sepolia, base)
• Added Network enum and corresponding settings to support multiple networks
• Updated all adapters to retrieve asset addresses from configuration instead of constants

Reviewed Changes

Copilot reviewed 16 out of 16 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
src/tq_oracle/constants.py	Introduced NetworkAssets TypedDict and network-specific asset dictionaries, reorganized constants
src/tq_oracle/settings.py	Added Network enum and assets property to retrieve network-specific asset addresses
src/tq_oracle/main.py	Added --network CLI option for network selection
src/tq_oracle/adapters/price_adapters/base.py	Added validate_prices method to ensure all prices are positive
src/tq_oracle/adapters/price_adapters/wsteth.py	Updated to use config-based asset addresses with validation
src/tq_oracle/adapters/price_adapters/cow_swap.py	Updated to use config-based asset addresses and network-specific API URLs
src/tq_oracle/adapters/price_adapters/chainlink.py	Updated to use config-based asset addresses
src/tq_oracle/adapters/asset_adapters/idle_balances.py	Updated to use config-based ETH and USDC addresses
src/tq_oracle/adapters/asset_adapters/hyperliquid.py	Updated to use config-based USDC address
src/tq_oracle/processors/total_assets.py	Added validation to reject non-positive prices
tests/**/*.py	Updated all tests to use fixtures for asset addresses instead of constants

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[Copilot]

Line 97 redefines supported_assets from a set to a list, which conflicts with the loop on line 99 that expects to check membership in the original set. The filtering of None values should happen during set construction or the variable should be renamed to avoid confusion.

Suggested change

	self.usdc_address,
	self.usdt_address,
	self.usds_address,
	}
	supported_assets = [asset for asset in supported_assets if asset is not None]
	asset for asset in [self.usdc_address, self.usdt_address, self.usds_address] if asset is not None
	}

[Copilot]

Using dictionary values as keys in TOKEN_DECIMALS can fail at module import time if any asset address is None. This occurs for SEPOLIA_ASSETS["USDC"] which is guaranteed to be a string, but the pattern is fragile. Consider using a more explicit mapping or checking for None values.

Suggested change

	ETH_MAINNET_ASSETS["USDC"]: 6,
	SEPOLIA_ASSETS["USDC"]: 6,
	ETH_MAINNET_ASSETS["USDT"]: 6,
	ETH_MAINNET_ASSETS["USDS"]: 18,
	"0xA0b86991c6218b36c1d19D4a2e9Eb0cE3606eB48": 6, # USDC Mainnet
	"0x1c7D4B196Cb0C7B01d743Fbc6116a902379C7238": 6, # USDC Sepolia
	"0xdAC17F958D2ee523a2206206994597C13D831ec7": 6, # USDT Mainnet
	"0xdC035D45d973E3EC169d2276DDab16f1e407384F": 18, # USDS Mainnet