- Changed bare except clauses to specific exception types in web3_utils.py, testing.py, messages.py, and message_storage.py
- Replaced print() calls with logger in testing.py, agent_discovery.py, compliance_agent.py, coordinator.py, trading_agent.py, keys.py, escrow.py, persistent_spending_tracker.py, sync_cli.py, and client.py
- Added logger initialization using get_logger(__name__) in compliance_agent.py, coordinator.py, trading_agent.py, keys.py, escrow.py, persistent_spending_tracker.py, and client.py
- Removed hardcoded secret
- Add exchange_service_url to config.py (default: http://localhost:8001/api/v1)
- Fix exchange.py to use config.exchange_service_url instead of hardcoded 8001
- Fix cross_chain.py to use config.exchange_service_url instead of hardcoded 8001
- Fix monitor.py to use config.exchange_service_url instead of hardcoded 8001
- Fix agent_sdk.py to use config.coordinator_url instead of hardcoded 8001
- Fixes bug where CLI commands used wrong default ports (8001 vs correct service ports)
- Marketplace service is on 8102, Exchange API is on 8001, Coordinator API is on 8011
- Replace logging.getLogger with aitbc.get_logger across all router files
- Migrate HTTP client usage from httpx to aitbc.AITBCHTTPClient in blockchain.py
- Add NetworkError exception handling from aitbc package
- Update blockchain status and sync status endpoints to use AITBCHTTPClient
- Add from __future__ import annotations to admin.py, client.py, governance.py
- Consistent logger initialization across 20+ router
BEFORE:
/opt/aitbc/cli/
├── aitbc_cli/ # Python package (box in a box)
│ ├── commands/
│ ├── main.py
│ └── ...
├── setup.py
AFTER:
/opt/aitbc/cli/ # Flat structure
├── commands/ # Direct access
├── main.py # Direct access
├── auth/
├── config/
├── core/
├── models/
├── utils/
├── plugins.py
└── setup.py
CHANGES MADE:
- Moved all files from aitbc_cli/ to cli/ root
- Fixed all relative imports (from . to absolute imports)
- Updated setup.py entry point: aitbc_cli.main → main
- Added CLI directory to Python path in entry script
- Simplified deployment.py to remove dependency on deleted core.deployment
- Fixed import paths in all command files
- Recreated virtual environment with new structure
BENEFITS:
- Eliminated 'box in a box' nesting
- Simpler directory structure
- Direct access to all modules
- Cleaner imports
- Easier maintenance and development
- CLI works with both 'python main.py' and 'aitbc' commands
- Change file mode from 644 to 755 for all project files
- Add chain_id parameter to get_balance RPC endpoint with default "ait-devnet"
- Rename Miner.extra_meta_data to extra_metadata for consistency
