Bitcoin UI
-
AI DESIGN REVIEW & LIBRARY FOR BITCOIN IOS APPS
+
DESIGN REVIEW & LIBRARY FOR BITCOIN IOS APPS
@@ -61,15 +61,21 @@
- INSTALL -> RUN BITCOINUI -> GET A DESIGN REVIEW
- BITCOINUI REVIEW: SendFeeView.swift
-===================================
-MEDIUM (1 issue)
----------------
-[UX] Line 88: Fee picker lacks a high-fee warning
-Fix: Add a warning when fee >= 50% of amount.
-Reference: Bitcoin Design Guide - Send fees https://bitcoin.design/guide/daily-spending-wallet/sending/send-fees/
-SUMMARY: 0 high, 1 medium, 0 low
+ install -> run bitcoinui -> get a design review
+ BITCOINUI
+
+SendFeeView.swift
+Findings: 0 high, 2 medium, 0 low
+
+Medium
+
+1) [UX L88] Fee picker lacks a high-fee warning
+ Fix: Add a warning when fee >= 50% of amount.
+ Ref: Bitcoin Design Guide - Send fees https://bitcoin.design/guide/daily-spending-wallet/sending/send-fees/
+2) [A11Y L142] Icon-only button has no label
+ Fix: Add an accessibilityLabel("Close") and include visible text.
+ Ref: iOS HIG - Accessibility https://developer.apple.com/design/human-interface-guidelines/accessibility
+
diff --git a/site/skills/bitcoinui/SKILL.md b/site/skills/bitcoinui/SKILL.md
index 5d8d278..a2c0efc 100644
--- a/site/skills/bitcoinui/SKILL.md
+++ b/site/skills/bitcoinui/SKILL.md
@@ -19,42 +19,36 @@ Review iOS SwiftUI code and screens for Bitcoin-specific UX correctness, iOS HIG
1. Read the target file(s) before commenting.
2. Apply the rubric in `references/bitcoinui-rubric.md`.
-3. Report issues by severity with line numbers, a short snippet, and a concrete fix. Include a direct URL in each Reference line.
-4. For Bitcoin-specific issues, include at least one Bitcoin Design Guide or BIP URL in the Reference line.
+3. Report issues by severity with line numbers, a short snippet, and a concrete fix. Include a direct URL in each Ref line.
+4. For Bitcoin-specific issues, include at least one Bitcoin Design Guide or BIP URL in the Ref line.
5. Suggest BitcoinUI components when they would reduce errors or improve consistency.
-6. Omit any severity section with 0 issues (including LOW).
-7. LOW is for optional polish/consistency notes; skip it if there are no meaningful suggestions.
+6. Omit any severity section with 0 findings (including Low).
+7. Low is for optional polish/consistency notes; skip it if there are no meaningful suggestions.
8. Do not modify code unless the user asks; offer to fix if appropriate.
## Output Format
-Start directly with the `BITCOINUI REVIEW` block; do not add any preamble text before it.
-Omit any severity section with 0 issues (including LOW).
+Start directly with the `BITCOINUI` block; do not add any preamble text before it.
+Omit any severity section with 0 findings (including Low).
```
-===============================================
-BITCOINUI REVIEW: [filename]
-===============================================
-
-HIGH (X issues)
-------------------
-[UX] Line 42: Missing warning for fee > amount
- Fix: Add a warning state when fee >= 50% of amount.
- Reference: Bitcoin Design Guide — Send fees https://bitcoin.design/guide/daily-spending-wallet/sending/send-fees/
-
-MEDIUM (X issues)
-------------------
-[A11Y] Line 18: Icon-only button has no label
- Fix: Add accessibilityLabel("Close")
- Reference: iOS HIG — Accessibility https://developer.apple.com/design/human-interface-guidelines/accessibility
-
-LOW (X issues)
-------------------
-...
-
-===============================================
-SUMMARY: X high, X medium, X low
-===============================================
+BITCOINUI
+
+[file-or-folder]
+Findings: X high, X medium, X low
+
+High
+1) [UX L42] Missing warning for fee > amount
+ Fix: Add a warning state when fee >= 50% of amount.
+ Ref: Bitcoin Design Guide — Send fees https://bitcoin.design/guide/daily-spending-wallet/sending/send-fees/
+
+Medium
+1) [A11Y L18] Icon-only button has no label
+ Fix: Add accessibilityLabel("Close")
+ Ref: iOS HIG — Accessibility https://developer.apple.com/design/human-interface-guidelines/accessibility
+
+Low
+1) ...
```
## References
diff --git a/site/style.css b/site/style.css
index f9659af..d20d555 100644
--- a/site/style.css
+++ b/site/style.css
@@ -109,7 +109,7 @@ h2 a {
}
h2 a:hover {
- color: var(--orange);
+ color: var(--green);
}
.section-title {
diff --git a/skills/bitcoinui/SKILL.md b/skills/bitcoinui/SKILL.md
index 5d8d278..a2c0efc 100644
--- a/skills/bitcoinui/SKILL.md
+++ b/skills/bitcoinui/SKILL.md
@@ -19,42 +19,36 @@ Review iOS SwiftUI code and screens for Bitcoin-specific UX correctness, iOS HIG
1. Read the target file(s) before commenting.
2. Apply the rubric in `references/bitcoinui-rubric.md`.
-3. Report issues by severity with line numbers, a short snippet, and a concrete fix. Include a direct URL in each Reference line.
-4. For Bitcoin-specific issues, include at least one Bitcoin Design Guide or BIP URL in the Reference line.
+3. Report issues by severity with line numbers, a short snippet, and a concrete fix. Include a direct URL in each Ref line.
+4. For Bitcoin-specific issues, include at least one Bitcoin Design Guide or BIP URL in the Ref line.
5. Suggest BitcoinUI components when they would reduce errors or improve consistency.
-6. Omit any severity section with 0 issues (including LOW).
-7. LOW is for optional polish/consistency notes; skip it if there are no meaningful suggestions.
+6. Omit any severity section with 0 findings (including Low).
+7. Low is for optional polish/consistency notes; skip it if there are no meaningful suggestions.
8. Do not modify code unless the user asks; offer to fix if appropriate.
## Output Format
-Start directly with the `BITCOINUI REVIEW` block; do not add any preamble text before it.
-Omit any severity section with 0 issues (including LOW).
+Start directly with the `BITCOINUI` block; do not add any preamble text before it.
+Omit any severity section with 0 findings (including Low).
```
-===============================================
-BITCOINUI REVIEW: [filename]
-===============================================
-
-HIGH (X issues)
-------------------
-[UX] Line 42: Missing warning for fee > amount
- Fix: Add a warning state when fee >= 50% of amount.
- Reference: Bitcoin Design Guide — Send fees https://bitcoin.design/guide/daily-spending-wallet/sending/send-fees/
-
-MEDIUM (X issues)
-------------------
-[A11Y] Line 18: Icon-only button has no label
- Fix: Add accessibilityLabel("Close")
- Reference: iOS HIG — Accessibility https://developer.apple.com/design/human-interface-guidelines/accessibility
-
-LOW (X issues)
-------------------
-...
-
-===============================================
-SUMMARY: X high, X medium, X low
-===============================================
+BITCOINUI
+
+[file-or-folder]
+Findings: X high, X medium, X low
+
+High
+1) [UX L42] Missing warning for fee > amount
+ Fix: Add a warning state when fee >= 50% of amount.
+ Ref: Bitcoin Design Guide — Send fees https://bitcoin.design/guide/daily-spending-wallet/sending/send-fees/
+
+Medium
+1) [A11Y L18] Icon-only button has no label
+ Fix: Add accessibilityLabel("Close")
+ Ref: iOS HIG — Accessibility https://developer.apple.com/design/human-interface-guidelines/accessibility
+
+Low
+1) ...
```
## References