docs: Update AGENTS.md with string completion status and build gotchas

- Mark string resources as completed for MoneroScreen UI port
- Add Known Issues section with Material Icons and TextSpinner paths
- Clarify icon adaptation options (MaterialSymbols vs Extended)
This commit is contained in:
Nick Pulido
2026-05-20 22:44:23 -07:00
parent 53fed8c256
commit defc97803f
+10 -2
View File
@@ -50,13 +50,21 @@
- **Port the UI layer** (~1 single-file MoneroScreen.kt that was ~1958 lines in Garnet, ported to Amethyst patterns)
- Challenge: Garnet used Material3 standard Icons (Icons.Filled.Edit, etc.) but Amethyst uses custom MaterialSymbol system
- Need to adapt icon usage to Amethyst's `commons.icons.symbols` approach
- Need to add missing string resources (daemon*, monero_fund*, backup_seed*, transaction_priority*)
- String resources already added (daemon*, monero_fund*, backup_seed*, transaction_priority) ✓
- Need to adapt icon usage to Amethyst's `commons.icons.symbols` approach or use androidx.compose.material3.icons.Extended
- Need to use Flow-based state collection instead of LiveData
- Add NIP-69 monero address support
- Full build verification (`./gradlew assembleFdroidDebug`)
- Test monero wallet functionality
### Known Issues & Gotchas
- **Material Icons package**: Amethyst doesn't include `compose-material-icons-extended`. Either:
- Add the dependency and use `androidx.compose.material.icons.Extended.*`
- Convert to `MaterialSymbols` from `commons.icons.symbols` (preferred pattern)
- **TransactionDirection** enum is in `com.vitorpamplona.amethyst.model` package
- **TextSpinner** is in `com.vitorpamplona.amethyst.ui.components.TextSpinner`
---
## What Worked ✓