aboutsummaryrefslogtreecommitdiff
path: root/modules/custom-misc.el
Commit message (Collapse)AuthorAgeFilesLines
* refactor(load-graph): route C-; registration through the keymap APICraig Jennings12 days1-8/+8
| | | | | | Migrated all 31 cj/custom-keymap registration sites across 24 modules from direct (keymap-set cj/custom-keymap ...) calls to cj/register-prefix-map and cj/register-command. Consumers no longer reference cj/custom-keymap directly, so keybindings.el is the sole owner of the C-; prefix and modules reach it only through the API (each already requires keybindings from Phase 2). Behavior-preserving: I dumped every C-; binding before and after the migration and they're identical: 279 bindings, each resolving to the same command. The which-key label blocks are untouched, since they use string key descriptions and never assumed the keymap existed. I byte-compiled all 24 files (no new free-variable warnings, because the cj/custom-keymap references are gone), and make test, validate-modules, and an init load all pass.
* docs(load-graph): classify text/editing command modulesCraig Jennings12 days1-0/+10
| | | | | | Second classification batch: the nine custom-* text/editing command helpers (case, comments, datetime, buffer-file, line-paragraph, misc, ordering, text-enclose, whitespace). I annotated each with the load-graph header contract and added a Batch 2 table to the inventory. They're all Layer 2, eager only to register a C-; submap at load, with no necessary eager reason, so all are Phase 3/4 deferral candidates. The inventory records a second hidden dependency for Phase 2: custom-buffer-file guards its C-; b registration with (when (boundp 'cj/custom-keymap) ...) and declares the keymap only via eval-when-compile, so the binding silently drops when the module loads without keybindings.
* chore(modules): pass validate-modules in batch by adding requiresCraig Jennings2026-05-071-2/+1
| | | | | | | | `make validate-modules` had 19 module-load failures, all the same shape: a module references a symbol or feature owned by another module without saying so. Production was fine because init.el orders requires correctly. The batch target loads each module in isolation, though, and surfaces the gap. I added explicit `(require 'keybindings)` or `(require 'user-constants)` to each affected module. The requires are idempotent at runtime, so production load order is unchanged. For three optional packages (elpa-mirror, mu4e, org-contacts), I switched to `(require 'X nil t)` so the modules load cleanly when those packages aren't installed. The activation calls become no-ops in that case. `make validate-modules` now reports 0 failures.
* chore:custom-misc: update keybindings for word/char countCraig Jennings2025-10-311-4/+4
| | | | | | Change keybindings for counting words and characters to "# w" and "# c" to improve mnemonic efficiency. Update the corresponding which-key descriptions accordingly.
* feat:count-characters: Add character counting functionalityCraig Jennings2025-10-311-0/+21
| | | | | Introduce functions to count characters in a buffer or region. Bind new character counting function to "C-; C" in custom keymap.
* feat:which-key: Add descriptive labels for custom keymapsCraig Jennings2025-10-271-0/+10
| | | | | | | | | | | | Enhance which-key integration by providing detailed descriptions for new key bindings across multiple modules. This improves the usability of custom keymaps by clarifying the purpose of each keybinding, making it easier for users to navigate and understand different menus and options available within the configuration. This update ensures that all custom keymaps now display a descriptive label in the which-key popup to explain their functionality, aiding users in identifying keymap purposes promptly.
* test+fix:custom-misc: add tests and fix fraction glyph bugCraig Jennings2025-10-261-31/+59
| | | | | | | | | | | | | | | | | | | | | Add test coverage for 4 functions in custom-misc.el: - cj/replace-fraction-glyphs (24 tests) - cj/format-region-or-buffer (17 tests) - cj/count-words-buffer-or-region (20 tests) - cj/jump-to-matching-paren (18 tests) Refactored functions using internal/interactive split pattern: - Internal functions (cj/--function-name) contain business logic with explicit parameters and validation - Interactive wrappers handle UI concerns (region detection, messages) - Tests call internal functions directly (no mocking required) Bug Fix: cj/--replace-fraction-glyphs Fixed "Invalid search bound" error when converting glyphs to text. Original code used fixed end position which became invalid when replacements changed buffer size. Fixed by using copy-marker for dynamic end position tracking.
* feat:custom-misc: added switch to previous buffer C-; SPCCraig Jennings2025-10-211-0/+6
|
* moved toggle-debug-on error keybindign to config-utilsCraig Jennings2025-10-211-1/+0
|
* fix: custom-misc: correct custom-keymap evaluationCraig Jennings2025-10-201-2/+2
| | | | | | Changed the evaluation of `cj/custom-keymap` to use `defvar` for proper reference, fixing potential issues in compilation without requiring 'keybindings'.
* refactor: custom-misc: Improve utility functions and modernize codeCraig Jennings2025-10-181-39/+72
| | | | | | | | | - Add comprehensive module documentation - Enhance jump-to-matching-paren with better delimiter detection and error handling - Add user feedback messages to format and fraction replacement functions - Modernize keybindings to use keymap-set instead of define-key - Fix potential advice stacking issue on file reload - Improve code formatting and consistency throughout
* maint: remaining custom functions grouped into separate modulesCraig Jennings2025-10-121-0/+90