Commit 8ffbbf6
committed
fix(mobula): Address Copilot feedback - asset ID consistency and type safety
1. Fix SOL asset ID inconsistency:
- Correct SOL mapping from 102484775 → 100010811 in includes.json
- 102484775 is BBSOL (Bybit Staked SOL), not SOL (Solana)
- Ensures consistency between base currency and quote currency mappings
2. Improve TypeScript type safety:
- Replace 'any' type with proper MobulaSubscribeMessage interface
- Define structure for WebSocket subscription messages
- Maintains type safety throughout transport logic
All hardcoded quote currencies now match includes.json mappings:
✅ BTC: 100001656, ETH: 100004304, SOL: 1000108111 parent 3501b04 commit 8ffbbf6
File tree
2 files changed
+11
-2
lines changed- packages/sources/mobula-state/src
- config
- transport
2 files changed
+11
-2
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1544 | 1544 | | |
1545 | 1545 | | |
1546 | 1546 | | |
1547 | | - | |
| 1547 | + | |
1548 | 1548 | | |
1549 | 1549 | | |
1550 | 1550 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
14 | 14 | | |
15 | 15 | | |
16 | 16 | | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
17 | 26 | | |
18 | 27 | | |
19 | 28 | | |
| |||
105 | 114 | | |
106 | 115 | | |
107 | 116 | | |
108 | | - | |
| 117 | + | |
109 | 118 | | |
110 | 119 | | |
111 | 120 | | |
| |||
0 commit comments