summaryrefslogtreecommitdiff
AgeCommit message (Collapse)Author
3 days test+fix:custom-misc: add tests and fix fraction glyph bugCraig Jennings
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.
3 days test+feat:custom-whitespace: add 37 tests + 2 new functionsCraig Jennings
Refactor all 4 existing whitespace functions to use interactive/non-interactive pattern and add comprehensive testing (73 tests). Implement 2 new functions: delete-all-whitespace (removes all whitespace) and ensure-single-blank-line (collapses consecutive blanks to one). Add keybindings for untabify/tabify. Total: 110 tests, 100% pass rate, 0 linter warnings.
3 daysupdated tasksCraig Jennings
3 daysfix:webclipper:fixed org-web-tools not loading when neededCraig Jennings
3 daysbetter commentary for testsCraig Jennings
3 daystest:removing old (now redundant) testsCraig Jennings
3 days test+refactor:custom-whitespace: add comprehensive testing with 73 testsCraig Jennings
Refactor 4 whitespace manipulation functions (remove-leading-trailing, collapse-whitespace, delete-blank-lines, hyphenate-whitespace) to use interactive/non-interactive pattern for testability. Add 73 tests across 4 test files covering normal, boundary, and error cases. Fix linter warnings and add proper input validation.
3 daysprompt: adding learnings from quality-engineer sessionsCraig Jennings
3 daysfix: config-utilities: remove duplicate keymap bindingCraig Jennings
Removed a duplicate keymap binding for 'cj/delete-emacs-home-compiled-files'
3 daysdocs:custom-line-paragraph: Update author info, enhance commentaryCraig Jennings
Add author information and improve commentary to detail line and paragraph manipulation utilities more clearly, enhancing the module's documentation.
3 dayschore: remove old redundant tests.Craig Jennings
3 daysfeat+test:comments: add 7 comment generation functions and testsCraig Jennings
Implement 5 new comment styles (simple-divider, padded-divider, heavy-box, unicode-box, block-banner) and refactor 2 existing functions (inline-border, comment-box) to use interactive/non-interactive pattern for testability. Add 178 tests across 7 test files with validation for length constraints and cross-language support. Fix linter warnings for parameter shadowing and docstring formatting.
3 daystest+fix:comments: add tests for cj/comment-reformat; fix arg issueCraig Jennings
Add 14 unit tests for cj/comment-reformat covering: - Emacs Lisp (12 tests): multiline joining, fill-column restoration, unicode, indentation, empty regions, read-only buffers - Python (1 test): hash-based comments - C (1 test): C-style comments (converted to block style by c-mode Fix issue where cj/comment-reformat was being called with args
3 daystest+docs:custom-comments: add tests for cj/delete-buffer-commentsCraig Jennings
Add 20 unit tests for cj/delete-buffer-comments covering: - Emacs Lisp (14 tests): comprehensive coverage of normal/boundary/error cases - Python (3 tests): hash-based comments (#) - C (3 tests): C-style line (//) and block (/* */) comments
3 days test+fix:custom-file-buffer: add tests, fix linter warningCraig Jennings
Add 58 unit tests for buffer/file utility functions: - cj/copy-link-to-buffer-file (12 tests) - cj/copy-path-to-buffer-file-as-kill (12 tests) - cj/copy-whole-buffer (14 tests) - cj/clear-to-bottom-of-buffer (10 tests) - cj/clear-to-top-of-buffer (10 tests) Tests cover normal cases, boundary cases (unicode, empty buffers, narrowing, read-only buffers), and error cases. Production code change: - Fix linter warning: unused variable 'err' → '_' in cj/move-buffer-and-file
3 days test:custom-file-buffer: add tests for cj/delete-buffer-and-fileCraig Jennings
Add 36 unit tests for cj/delete-buffer-and-file covering: - Normal cases: VC vs non-VC file deletion, trash flag usage - Boundary cases: file content variations, naming edge cases, buffer states - Error cases: permission issues, VC failures, error propagation Key testing decisions documented in Commentary: - Mock delete-file errors for deterministic, portable tests - Test our code's behavior (branching logic, error propagation), not the underlying trash/filesystem implementations - Verify correct calls to vc-delete-file vs delete-file with trash flag All 36 tests passing.
3 daysfeat+test:custom-file-buffer: add tests and safety refactoringCraig Jennings
Add 106 unit tests with full coverage for move-buffer-and-file (51 tests) and rename-buffer-and-file (55 tests). Refactor both functions using interactive/non-interactive split pattern for simpler testing and reusability. Changes: - Split cj/move-buffer-and-file and cj/rename-buffer-and-file into internal implementations (cj/--*) and interactive wrappers - Add ok-if-exists parameter with user confirmation to prevent data loss - Fix bugs: return values, path expansion, string-match arg order, regex - Add test utilities for proper buffer cleanup and isolation - Document interactive/non-interactive pattern in quality-engineer.org - Document error message testing guidelines in quality-engineer.org All 106 tests passing.
4 daysfix:custom-line-paragraph:missing forward declarationsCraig Jennings
4 dayschore:init:change marker identifying test progressionCraig Jennings
4 daysfeat:custom-line-paragraph: add keybinding for duplicate-and-commentCraig Jennings
- Add C-; l c keybinding to duplicate line/region with commenting - Add which-key description "duplicate and comment" for the new binding - Complements existing C-; l d (duplicate without comment)
4 daystest:custom-line-paragraph: add tests for underscore-lineCraig Jennings
- Add 26 tests covering normal cases, boundary cases, and error cases - Tests verify column width calculation with tabs, Unicode support, various underline characters - Mock read-char function for batch mode testing - All 26 tests passing on first run
4 daysfix+test:custom-line-paragraph:infinite loop remove-lines-containingCraig Jennings
- Add guard clause to handle empty string input gracefully (would cause infinite loop) - Add 33 comprehensive tests covering normal cases, boundary cases, and error cases - Tests verify Unicode support, regex character escaping, region vs buffer operations - All 33 tests passing
4 daystest:custom-line-paragraph: remove-duplicate-lines-region-or-bufferCraig Jennings
- Add 35 tests covering normal cases, boundary cases, and error cases - Tests verify duplicate removal with case-insensitive matching, region vs buffer operations, Unicode support - Discovered function requires trailing newlines for duplicate detection and uses case-insensitive matching - All 35 tests passing
4 days test:custom-line-paragraph: tests for duplicate-line-or-regionCraig Jennings
- Add 31 tests covering normal cases, boundary cases, and error cases - Tests verify duplication with/without comments, Unicode, RTL text, undo behavior, and edge cases - All 31 tests passing
4 daystest:custom-line-paragraph: add tests and fix region deactivationCraig Jennings
- Add 11 new tests for cj/join-line-or-region covering boundary cases, Unicode, stress testing, and undo behavior - Fix region deactivation in cj/join-line-or-region function - All 62 tests now passing (41 join-line-or-region + 21 join-paragraph)
4 daysupdated tasksCraig Jennings
4 daysfeat: chime: make chime start immediatelyCraig Jennings
4 daysqa prompt: better direction about when not to mock functionalityCraig Jennings
4 daysfix:org-roam: copy completed tasks to dailies on state transitionsCraig Jennings
Fixed and enhanced the org-roam hook that copies completed tasks to daily notes: - Fixed hook not triggering immediately after Emacs launch by moving it outside the lazy-loaded use-package org-roam :config block and into with-eval-after-load 'org - Changed hook to trigger for ANY org-done-keyword (DONE, CANCELLED, etc.) instead of just "DONE" - Updated hook to only trigger on non-done → done transitions using org-last-state, preventing duplicate copies when changing between done states (e.g., DONE → CANCELLED) - Added docstrings to org-roam helper functions to fix checkdoc linter warnings - Created comprehensive ERT test suite with 10 tests covering: * Hook registration before org-roam loads (lazy-loading fix) * Transitions to done states (nil→DONE, TODO→DONE, IN-PROGRESS→DONE, WAITING→CANCELLED) * Non-triggering cases (done→done, transitions to non-done states)
4 dayschore: ignoring flycheck temp filesCraig Jennings
4 daysrefactor:dirvish: Remove redundant nerd-icons setupCraig Jennings
Move the `nerd-icons` setup from `dirvish-config.el` to `font-config.el` to consolidate icon configurations.
4 daysrefactor:dirvish: remove unused dired sidebar codeCraig Jennings
Remove unused functions and configurations related to dired-sidebar and nerd-icons-dired. This cleans up the code and reduces unnecessary dependencies within the dashboard-config.el and dirvish-config.el modules.
4 daysfeat:erc-config: add image inline support and simplify hooksCraig Jennings
Enable inline image support in ERC by adding 'image' to erc-modules. Simplify hook usage by removing ERC colorization setup, leveraging built-in functionality for better color management.
4 daysrefactor:keymaps: Replace define-prefix-command with defvar-keymapCraig Jennings
Refactor the keymap definitions across multiple modules by replacing `define-prefix-command` with `defvar-keymap`. This simplifies the keymap setup and integrates better with the `which-key` package for enhanced documentation and usability. Updated modules include custom-ordering, custom-text-enclose, custom-whitespace, mail-config, music-config, org-drill-config, test-runner, and vc-config.
4 daysfeat:erc: Enable IRC client in dashboard and update keybindingsCraig Jennings
Uncommented ERC client initialization in `init.el`. Replaced "cj/erc-start-or-switch" with "cj/erc-switch-to-buffer-with-completion" in dashboard. Altered keybindings to use "C-; E" instead of "C-c e" for ERC commands. Defined a new keymap "cj/erc-keymap" for better command organization. Removed deprecated `locate-library` checks and `cj/erc-command-map`. Refined code layout for clarity.
4 daysfeat(dirvish-config): Add sidebar attribute settingCraig Jennings
Add `dirvish-side-attributes` for explicit sidebar configuration.
4 dayschore:gitignore: Add client secret JSON file to ignore listCraig Jennings
Prevent tracking of sensitive client secret file in version control by ignoring it in .gitignore.
4 dayschore:init: Remove obsolete eradio-config require statementCraig Jennings
The eradio-config dependency has been removed from the init file, potentially because it is no longer needed or has been deprecated.
4 daysrefactor:init: adjust startup performance optimizationsCraig Jennings
Enable error notifications by adjusting warning level. Simplify startup performance by removing version control backend adjustments. Restore initial settings after startup.
4 daysrefactor:video-audio-recording: remove unused autoload cookiesCraig Jennings
Remove unnecessary `;;;###autoload` comments from function definitions. This cleanup clarifies the code and has no functional impact as these autoload cookies were not utilized.
4 daysfeat:org-config: Add function to clear org-element cacheCraig Jennings
Introduce `cj/org-clear-element-cache` to clear the org-element cache for the current buffer or all buffers. This is useful for resolving parsing errors encountered during agenda generation, such as 'wrong-type-argument stringp nil'. The function can be triggered with a prefix argument to clear the cache for all org buffers, or without it to clear only the current buffer's cache.
4 daysfix:dwim-shell: ensure dirvish starts dwim-shellCraig Jennings
Update `use-package` to load `dwim-shell-command` after both `dired` and `dirvish`, ensuring proper integration and compatibility with the additional package.
4 daysrefactor: org-agenda: add chimeCraig Jennings
Add chime for managing agenda notifications. Update configuration settings for chime, including notification times and modeline display. Enable chime-mode by default.
4 daysfeat:org-drill: Add cj/drill-start command to autoloadCraig Jennings
Extend the use-package setup to include the new command `cj/drill-start` in addition to `org-drill`.
4 daysfeat:org-gcal: Add function to clear sync lock and bindingCraig Jennings
Add `cj/org-gcal-clear-sync-lock` function to allow clearing the `org-gcal` sync lock. This is useful when a sync fails and leaves the lock in place, preventing future syncs. This function can be invoked with the keybinding "C-; G". Additionally, ensure credentials are retrieved from `authinfo.gpg` before loading `org-gcal`.
4 daysfeat:dirvish: shortcuts for Google Drive, remote git:cjennings.netCraig Jennings
Include shortcuts for Google Drive in documents and add a remote access path for git at cjennings.net in the dirvish configuration.
4 dayschore: music: remove eradio-config and update music-configCraig Jennings
Deleted eradio-config.el file as it is no longer needed. Refactored music-config.el to improve readability and maintainability by removing unused autoload directives, unnecessary eval-when-compile blocks, adjusting indentation, and enhancing the setup function for EMMS. Additionally, removed duplicate code and ensured playlist handling is more robust.
4 daysmaint: localrepo: adding latest packagesCraig Jennings
6 daysignoring browser-choice.el persistence fileCraig Jennings
6 daysupdated tasksCraig Jennings