From 95b40420cda0ac46421a45f5dad5a0b9d9380415 Mon Sep 17 00:00:00 2001 From: Matthew Date: Tue, 13 Jan 2026 09:37:01 -0600 Subject: [PATCH] docs: site copy --- README.md | 31 ++++++++------------ commands/bitcoinui.md | 50 ++++++++++++++------------------ site/bitcoinui.md | 50 ++++++++++++++------------------ site/index.html | 28 +++++++++++------- site/skills/bitcoinui/SKILL.md | 52 +++++++++++++++------------------- site/style.css | 2 +- skills/bitcoinui/SKILL.md | 52 +++++++++++++++------------------- 7 files changed, 120 insertions(+), 145 deletions(-) diff --git a/README.md b/README.md index 498309e..4132f8b 100644 --- a/README.md +++ b/README.md @@ -67,23 +67,16 @@ opencode Example output: ``` -=============================================== -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/ - -LOW (1 issue) ------------------- -[A11Y] Line 42: Icon-only close button has no label - Fix: Add accessibilityLabel("Close") - Reference: iOS HIG — Accessibility https://developer.apple.com/design/human-interface-guidelines/accessibility - -=============================================== -SUMMARY: 0 high, 1 medium, 1 low -=============================================== +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 L42] Icon-only close button has no label + Fix: Add accessibilityLabel("Close") + Ref: iOS HIG — Accessibility https://developer.apple.com/design/human-interface-guidelines/accessibility ``` diff --git a/commands/bitcoinui.md b/commands/bitcoinui.md index 2bc52a8..2109ac2 100644 --- a/commands/bitcoinui.md +++ b/commands/bitcoinui.md @@ -30,33 +30,27 @@ If `$ARGUMENTS` is empty, ask: "Using the bitcoinui command for an iOS review. W ## 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) ... ``` ## Guidance @@ -64,8 +58,8 @@ SUMMARY: X high, X medium, X low 1. Read the file(s) first before making assessments. 2. Be specific with line numbers and code snippets. 3. Provide fixes, not just problems. -4. Include a direct URL in each Reference line. -5. For Bitcoin-specific issues, include at least one Bitcoin Design Guide or BIP URL in the Reference line. +4. Include a direct URL in each Ref line. +5. For Bitcoin-specific issues, include at least one Bitcoin Design Guide or BIP URL in the Ref line. 6. Suggest BitcoinUI components when they fit the pattern. -7. LOW is for optional polish/consistency notes; skip it if there are no meaningful suggestions. +7. Low is for optional polish/consistency notes; skip it if there are no meaningful suggestions. 8. Do not modify code unless asked; offer to fix if appropriate. diff --git a/site/bitcoinui.md b/site/bitcoinui.md index 2bc52a8..2109ac2 100644 --- a/site/bitcoinui.md +++ b/site/bitcoinui.md @@ -30,33 +30,27 @@ If `$ARGUMENTS` is empty, ask: "Using the bitcoinui command for an iOS review. W ## 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) ... ``` ## Guidance @@ -64,8 +58,8 @@ SUMMARY: X high, X medium, X low 1. Read the file(s) first before making assessments. 2. Be specific with line numbers and code snippets. 3. Provide fixes, not just problems. -4. Include a direct URL in each Reference line. -5. For Bitcoin-specific issues, include at least one Bitcoin Design Guide or BIP URL in the Reference line. +4. Include a direct URL in each Ref line. +5. For Bitcoin-specific issues, include at least one Bitcoin Design Guide or BIP URL in the Ref line. 6. Suggest BitcoinUI components when they fit the pattern. -7. LOW is for optional polish/consistency notes; skip it if there are no meaningful suggestions. +7. Low is for optional polish/consistency notes; skip it if there are no meaningful suggestions. 8. Do not modify code unless asked; offer to fix if appropriate. diff --git a/site/index.html b/site/index.html index 9d6a421..e118f2d 100644 --- a/site/index.html +++ b/site/index.html @@ -20,14 +20,14 @@ href="https://fonts.googleapis.com/css2?family=DM+Mono:wght@300;400;500&display=swap" rel="stylesheet" > - +

Bitcoin UI

-

AI DESIGN REVIEW & LIBRARY FOR BITCOIN IOS APPS

+

DESIGN REVIEW & LIBRARY FOR BITCOIN IOS APPS

@@ -61,15 +61,21 @@

OpenCode

-

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