From cb7d45f21bfb485cc5cf1ca4427721d6a9f898f0 Mon Sep 17 00:00:00 2001
From: Virgil <virgil@lethean.io>
Date: Fri, 3 Apr 2026 22:58:53 +0000
Subject: [PATCH] feat(html): add aria-keyshortcuts helper

Co-Authored-By: Virgil <virgil@lethean.io>
---
 docs/architecture.md |  1 +
 docs/index.md        |  4 ++--
 node.go              | 10 ++++++++++
 node_test.go         | 20 ++++++++++++++++++++
 4 files changed, 33 insertions(+), 2 deletions(-)

diff --git a/docs/architecture.md b/docs/architecture.md
index 5dd64f9..fe56982 100644
--- a/docs/architecture.md
+++ b/docs/architecture.md
@@ -39,6 +39,7 @@ Accessibility-oriented helpers are also provided for common attribute patterns:
 - `AriaLabelledBy(node, ids...)`
 - `AriaControls(node, ids...)`
 - `AriaOwns(node, ids...)`
+- `AriaKeyShortcuts(node, shortcuts...)`
 - `AriaCurrent(node, current)`
 - `AriaBusy(node, busy)`
 - `AriaLive(node, live)`
diff --git a/docs/index.md b/docs/index.md
index a490c39..798eade 100644
--- a/docs/index.md
+++ b/docs/index.md
@@ -39,7 +39,7 @@ This builds a Header-Content-Footer layout with semantic HTML elements (`<header
 
 | Path | Purpose |
 |------|---------|
-| `node.go` | `Node` interface and all node types: `El`, `Text`, `Raw`, `If`, `Unless`, `Each`, `EachSeq`, `Switch`, `Entitled`, plus `AriaLabel`, `AriaControls`, `AriaOwns`, `Alt`/`AltText`, `AriaBusy`, `AriaLive`, `AriaDisabled`, `AriaChecked`, `AriaInvalid`, `AriaRequired`, `AriaPressed`, `AriaSelected`, `Hidden`, `TabIndex`, and `AutoFocus` helpers |
+| `node.go` | `Node` interface and all node types: `El`, `Text`, `Raw`, `If`, `Unless`, `Each`, `EachSeq`, `Switch`, `Entitled`, plus `AriaLabel`, `AriaControls`, `AriaOwns`, `AriaKeyShortcuts`, `Alt`/`AltText`, `AriaBusy`, `AriaLive`, `AriaDisabled`, `AriaChecked`, `AriaInvalid`, `AriaRequired`, `AriaPressed`, `AriaSelected`, `Hidden`, `TabIndex`, and `AutoFocus` helpers |
 | `layout.go` | HLCRF compositor with semantic HTML elements and ARIA roles |
 | `responsive.go` | Multi-variant breakpoint wrapper (`data-variant` containers, CSS scoping helpers) |
 | `context.go` | Rendering context: identity, locale, entitlements, i18n service |
@@ -52,7 +52,7 @@ This builds a Header-Content-Footer layout with semantic HTML elements (`<header
 
 ## Key Concepts
 
-**Node tree** -- All renderable units implement `Node`, a single-method interface: `Render(ctx *Context) string`. The library composes nodes into trees using `El()` for elements, `Text()` for translated text, and control-flow constructors (`If`, `Unless`, `Each`, `Switch`, `Entitled`), plus accessibility and visibility helpers such as `AriaLabel()`, `AriaControls()`, `AriaOwns()`, `AriaCurrent()`, `AriaBusy()`, `AriaLive()`, `AriaHidden()`, `Hidden()`, `AriaDisabled()`, `AriaChecked()`, `AriaInvalid()`, `AriaRequired()`, `AriaReadOnly()`, and `TabIndex()`.
+**Node tree** -- All renderable units implement `Node`, a single-method interface: `Render(ctx *Context) string`. The library composes nodes into trees using `El()` for elements, `Text()` for translated text, and control-flow constructors (`If`, `Unless`, `Each`, `Switch`, `Entitled`), plus accessibility and visibility helpers such as `AriaLabel()`, `AriaControls()`, `AriaOwns()`, `AriaKeyShortcuts()`, `AriaCurrent()`, `AriaBusy()`, `AriaLive()`, `AriaHidden()`, `Hidden()`, `AriaDisabled()`, `AriaChecked()`, `AriaInvalid()`, `AriaRequired()`, `AriaReadOnly()`, and `TabIndex()`.
 
 **HLCRF Layout** -- A five-slot compositor that maps to semantic HTML: `<header>` (H), `<aside>` (L/R), `<main>` (C), `<footer>` (F). The variant string controls which slots render: `"HLCRF"` for all five, `"HCF"` for three, `"C"` for content only. Layouts nest: placing a `Layout` inside another layout's slot produces hierarchical `data-block` paths like `L-0-C-0`.
 
diff --git a/node.go b/node.go
index 1013e4d..1debe8a 100644
--- a/node.go
+++ b/node.go
@@ -240,6 +240,16 @@ func AriaOwns(n Node, ids ...string) Node {
 	return n
 }
 
+// node.go: AriaKeyShortcuts sets the aria-keyshortcuts attribute on an element node.
+// Example: AriaKeyShortcuts(El("button"), "Ctrl+S", "Meta+S").
+// Multiple shortcuts are joined with spaces, matching the HTML attribute format.
+func AriaKeyShortcuts(n Node, shortcuts ...string) Node {
+	if value := joinNonEmpty(shortcuts...); value != "" {
+		return Attr(n, "aria-keyshortcuts", value)
+	}
+	return n
+}
+
 // node.go: AriaCurrent sets the aria-current attribute on an element node.
 // Example: AriaCurrent(El("a"), "page").
 // An empty value leaves the node unchanged so callers can opt out cleanly.
diff --git a/node_test.go b/node_test.go
index 43d32f1..991aeac 100644
--- a/node_test.go
+++ b/node_test.go
@@ -355,6 +355,26 @@ func TestAriaOwnsHelper_IgnoresEmptyIDs(t *testing.T) {
 	}
 }
 
+func TestAriaKeyShortcutsHelper(t *testing.T) {
+	ctx := NewContext()
+	node := AriaKeyShortcuts(El("button", Raw("save")), "Ctrl+S", "Meta+S")
+	got := node.Render(ctx)
+	want := `<button aria-keyshortcuts="Ctrl+S Meta+S">save</button>`
+	if got != want {
+		t.Errorf("AriaKeyShortcuts() = %q, want %q", got, want)
+	}
+}
+
+func TestAriaKeyShortcutsHelper_IgnoresEmptyShortcuts(t *testing.T) {
+	ctx := NewContext()
+	node := AriaKeyShortcuts(El("button", Raw("save")), "", "Ctrl+S", " ", "Meta+S")
+	got := node.Render(ctx)
+	want := `<button aria-keyshortcuts="Ctrl+S Meta+S">save</button>`
+	if got != want {
+		t.Errorf("AriaKeyShortcuts() with empty shortcuts = %q, want %q", got, want)
+	}
+}
+
 func TestAriaCurrentHelper(t *testing.T) {
 	ctx := NewContext()
 	node := AriaCurrent(El("a", Raw("Home")), "page")