aboutsummaryrefslogtreecommitdiff
path: root/chime.el
Commit message (Collapse)AuthorAgeFilesLines
* Show modeline icon immediately on chime-mode enableCraig Jennings2026-04-041-11/+47
| | | | | | | | | Previously chime-modeline-string was nil until the first async check completed (~10-15 seconds after startup). Now the icon appears instantly with a "waiting for first event check" tooltip. On validation or async failure, the icon stays visible and the tooltip updates to show the error state instead of going blank.
* Default modeline lighter to empty stringCraig Jennings2026-04-041-4/+5
| | | | | | The bell emoji lighter was redundant alongside the alarm clock event display in global-mode-string. One icon is enough. Users who want a separate mode indicator can set chime-modeline-lighter.
* Improve defaults: 2-hour modeline window, notify at event timeCraig Jennings2026-04-041-2/+2
| | | | | | | | | | Bump chime-modeline-lookahead-minutes from 60 to 120 so events within the next 2 hours show in the modeline. The old 60-minute default hid events most users would expect to see. Add (0 . high) to default chime-alert-intervals so users get a notification at event time in addition to 10 minutes before. The single-notification default meant a missed alert had no backup.
* Add comments for non-obvious code patternsCraig Jennings2026-04-041-7/+15
| | | | | | | | Clarify: dash cartesian product pipeline in chime--notifications, butlast 2 dropping USEC/PSEC from time values, Emacs time type guard, upcoming-events item structure, rx macroexpand for async env injection, backquoted lambda subprocess boundary, and org-end-of-meta-data nil argument meaning.
* Remove dead code and reduce default tooltip lookahead to 1 weekCraig Jennings2026-04-041-7/+6
| | | | | | | | | | | Remove chime--agenda-buffer-name (unused variable) and chime--extract-birthday-year (superseded by chime--parse-birthday). Reduce chime-tooltip-lookahead-hours from 8760 (1 year) to 168 (1 week). The 1-year default caused org-agenda-list to scan a 365-day span every check cycle, which is slow for large org collections. The tooltip only shows 5 events, so a week is sufficient for most users.
* Hoist loop-invariant computations and remove redundant sortCraig Jennings2026-04-041-68/+53
| | | | | | | | | Performance improvements with no behavioral change: - chime--group-events-by-day: hoist decode-time calls before loop - chime-event-has-any-passed-time: compute today-start once, not per timestamp - chime-event-is-today: same pattern - chime--build-upcoming-events-list: remove first sort (discarded by hash dedup) - chime--process-notifications: use -mapcat instead of -map + -flatten
* Nil chime--timer after cancellation in chime--stopCraig Jennings2026-04-041-0/+1
| | | | | The timer variable was left pointing at a cancelled timer object. Now set to nil so state accurately reflects no active timer.
* Extract days-until calculation from chime--day-wide-notification-textCraig Jennings2026-04-041-27/+22
| | | | | | Move the all-day timestamp date arithmetic into chime--days-until-event. The notification text function now reads clearly as a cond dispatch over today/advance-notice/fallback cases.
* Extract validation retry logic from chime-check into chime--maybe-validateCraig Jennings2026-04-041-26/+25
| | | | | | The validation/retry state machine was 30 lines embedded in chime-check. Now chime--maybe-validate returns t if OK to proceed, nil to skip. chime-check is reduced to 12 lines: validate, then fetch-and-process.
* Extract display logic from chime-validate-configurationCraig Jennings2026-04-041-14/+14
| | | | | | Move interactive display (message/display-warning calls) into chime--display-validation-results. The validation function stays focused on collecting issues; display is a separate concern.
* Extract three timestamp strategies from chime--extract-timeCraig Jennings2026-04-041-54/+59
| | | | | | | | | Decompose the 72-line function into focused helpers: - chime--extract-gcal-timestamps: org-gcal drawer extraction - chime--extract-property-timestamps: SCHEDULED/DEADLINE properties - chime--extract-plain-timestamps: plain body timestamps chime--extract-time is now a 12-line dispatcher.
* Add 94 tests across 10 new test files and fix 2 bugsCraig Jennings2026-04-041-7/+8
| | | | | | | | | | | | New test coverage for previously untested functions: - filter-day-wide-events, time utilities, day-wide time matching - make-tooltip, event-is-today, get-tags, done-keywords-predicate - extract-title, propertize-modeline-string, warn-persistent-failures - log-silently Bug fixes discovered by new tests: - Fix pluralization in no-events tooltip ("1 hours" -> "1 hour", etc.) - Fix chime--get-tags returning ("") instead of nil for untagged headings
* Add persistent async failure warnings after consecutive errorsCraig Jennings2026-02-231-2/+30
| | | | | | Tracks consecutive async check failures and displays a warning via display-warning when the threshold is reached (default 5). Resets on any successful check.
* Use locate-library fallback for sound file path resolutionCraig Jennings2026-02-231-1/+4
| | | | | Adds locate-library as a fallback between load-file-name and buffer-file-name for more robust path resolution in MELPA installs.
* Improve org-agenda-files validation to distinguish files from directoriesCraig Jennings2026-02-231-7/+10
|
* Include event title in timestamp parse error messagesCraig Jennings2026-02-231-8/+13
| | | | | | | Add optional context parameter to chime--timestamp-parse and pass the org heading title from chime--extract-time. Error messages now show which event failed, e.g. "Failed to parse timestamp '...' in 'Team Meeting': ..."
* Replace anonymous lambda in modeline keymap with named functionCraig Jennings2026-02-231-8/+4
| | | | | | Use chime--jump-to-first-event instead of a closure capturing soonest-event. Improves debuggability and removes unused parameter from chime--propertize-modeline-string.
* Fix docstring grammar and remove double blank lineCraig Jennings2026-02-231-2/+1
|
* Add missing declare-function for chime-debug functionsCraig Jennings2026-02-231-0/+3
| | | | | Declare chime-debug-enable-async-monitoring, chime--debug-log-async-error, and chime--debug-log-async-complete to suppress byte-compiler warnings.
* Fix defcustom :type specs for list and nullable variablesCraig Jennings2026-02-231-4/+5
| | | | | | | - chime-predicate-whitelist: (function) -> (repeat function) - chime-predicate-blacklist: (function) -> (repeat function) - chime-additional-environment-regexes: (string) -> (repeat string) - chime-notification-icon: string -> choice of nil or file path
* Bump minimum Emacs version to 27.1 and require subr-xCraig Jennings2026-02-231-3/+4
| | | | | | Package uses decoded-time-* accessors, time-equal-p (27.1+), and subr-x functions (when-let*, hash-table-values, string-empty-p). Remove stale "Emacs 26 compatibility" comments.
* Update README and fix repo references after rename to chimeCraig Jennings2026-02-221-3/+3
| | | | | | | | | Fix backronym to "CHIME Heralds Imminent Modeline Events" across all files. Update GitHub URLs from chime.el to chime. Add documentation for missing customization variables (notification icon, startup delay, modeline lighter, no-events text, tooltip header format, advance notice, tooltip all-day events). Fix test count (505) and default for chime-day-wide-alert-times.
* Add section headers to chime.el for imenu/outline navigationCraig Jennings2026-02-191-0/+28
| | | | | | | | | 14 ;;;; headers added for code navigation: Dependencies, Customization Variables, Internal State, Time/Date Utilities, All-Day Event Handling, Event Checking & Navigation, Modeline & Tooltip Display, Whitelist/Blacklist Filtering, Async Event Retrieval, Notification Dispatch, Timestamp Parsing, Event Info Extraction, Configuration Validation, Core Lifecycle. Comments only — zero runtime impact.
* Update copyright years to 2026 across all source filesCraig Jennings2026-02-171-1/+1
| | | | | Updated 41 files: chime.el, chime-debug.el, chime-org-contacts.el, convert-org-contacts-birthdays.el, and all test/utility files.
* Fix time comparison bugs in chime--time= and chime--timestamp-within-interval-pCraig Jennings2026-02-171-3/+3
| | | | | | | chime--time= was using %d:%H:%M (day-of-month only), causing false matches across different months/years. Now uses %Y-%m-%d %H:%M for full date comparison. chime--timestamp-within-interval-p now accepts numeric timestamps (integer/float) in addition to lists.
* changed repositoriesCraig Jennings2025-11-181-0/+1825