From 12129ec7663137f14908381efc9198dfcaff7022 Mon Sep 17 00:00:00 2001
From: Craig Jennings <c@cjennings.net>
Date: Sat, 11 Jul 2026 21:30:27 -0500
Subject: fix(org-reveal-config): register presentation keys via prefix map

The seven presentation commands were bound with raw global-set-key on "C-; p ..." chains. That only works once keybindings.el has made "C-;" a live prefix. Otherwise each binding errors with "non-prefix key". I replaced them with cj/reveal-map registered under "C-; p" via cj/register-prefix-map, matching every sibling and dropping the load-order dependency.
---
 tests/test-org-reveal-config-keymap.el | 38 ++++++++++++++++++++++++++++++++++
 1 file changed, 38 insertions(+)
 create mode 100644 tests/test-org-reveal-config-keymap.el

(limited to 'tests')

diff --git a/tests/test-org-reveal-config-keymap.el b/tests/test-org-reveal-config-keymap.el
new file mode 100644
index 00000000..17a21b19
--- /dev/null
+++ b/tests/test-org-reveal-config-keymap.el
@@ -0,0 +1,38 @@
+;;; test-org-reveal-config-keymap.el --- Tests for org-reveal-config prefix keymap -*- lexical-binding: t; -*-
+
+;;; Commentary:
+;; The presentation commands are reached through `cj/reveal-map', a prefix
+;; keymap registered under "C-; p" via `cj/register-prefix-map'.  These
+;; tests pin that structure so the module can't silently regress to raw
+;; `global-set-key' calls (which carry a hidden load-order dependency on
+;; keybindings.el establishing "C-;" as a prefix first).
+
+;;; Code:
+
+(require 'ert)
+
+(add-to-list 'load-path (expand-file-name "modules" user-emacs-directory))
+(require 'keybindings)
+(require 'org-reveal-config)
+
+(ert-deftest test-org-reveal-config-reveal-map-is-a-keymap ()
+  "Normal: `cj/reveal-map' is a keymap."
+  (should (keymapp cj/reveal-map)))
+
+(ert-deftest test-org-reveal-config-reveal-map-bindings ()
+  "Normal: each presentation command is reachable under `cj/reveal-map'."
+  (dolist (pair '(("SPC" . cj/reveal-present)
+                  ("e"   . cj/reveal-export)
+                  ("p"   . cj/reveal-preview-start)
+                  ("s"   . cj/reveal-preview-stop)
+                  ("h"   . cj/reveal-insert-header)
+                  ("H"   . cj/reveal-remove-headers)
+                  ("n"   . cj/reveal-new)))
+    (should (eq (keymap-lookup cj/reveal-map (car pair)) (cdr pair)))))
+
+(ert-deftest test-org-reveal-config-reveal-map-registered-under-p ()
+  "Normal: `cj/reveal-map' is registered under \"p\" in `cj/custom-keymap'."
+  (should (eq (keymap-lookup cj/custom-keymap "p") cj/reveal-map)))
+
+(provide 'test-org-reveal-config-keymap)
+;;; test-org-reveal-config-keymap.el ends here
-- 
cgit v1.2.3