diff options
| author | Craig Jennings <c@cjennings.net> | 2026-06-06 15:00:12 -0500 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-06-06 15:00:12 -0500 |
| commit | 0a3794ccfc56045a2e4155538c145021447d09e3 (patch) | |
| tree | 7f4cc85e738bab1471596331aec4cf9abfcba47d /pearl.el | |
| parent | 6a1d2e9085d6b247e72b4c98e776aa9fb128341f (diff) | |
| download | pearl-0a3794ccfc56045a2e4155538c145021447d09e3.tar.gz pearl-0a3794ccfc56045a2e4155538c145021447d09e3.zip | |
feat(views): grouping source helpers and the effective-grouping resolver
Phase 1 of the interactive-grouping spec: the source/resolver plumbing, no command yet. pearl--grouping-choices maps the display labels (status/project/assignee/priority/none) to Linear's issueGrouping values. pearl--grouping-display-to-value coerces and pearl--check-grouping validates, both refusing anything outside that set. Cycle is left out on purpose: it isn't in the issue drawer, so an offline regroup can't read it.
pearl--source-with-grouping records the choice under :client-group for every source type, leaving a view's own server :group intact. Choosing none removes the key, so the source falls back to the server :group or to flat. pearl--effective-grouping returns :client-group when set, else :group. That's the single read every grouping consumer routes through in phase 4, so a refresh reproduces the layout on screen.
16 tests: coercion, validation, the source helper, resolver precedence, and the :client-group header round-trip.
Diffstat (limited to 'pearl.el')
| -rw-r--r-- | pearl.el | 54 |
1 files changed, 54 insertions, 0 deletions
@@ -4498,6 +4498,60 @@ view's server order is honored); every other source uses `:sort'/`:order'." (cons (plist-get source :client-sort) (plist-get source :client-order)) (cons (plist-get source :sort) (plist-get source :order)))) +(defconst pearl--grouping-choices + '(("status" . "workflowState") + ("project" . "project") + ("assignee" . "assignee") + ("priority" . "priority") + ("none" . nil)) + "Interactive grouping dimensions: (DISPLAY . `issueGrouping' VALUE). +DISPLAY is what `pearl-set-grouping' completes; VALUE is the Linear +`issueGrouping' the render path speaks, or nil for \"none\" (ungroup). `cycle' +is intentionally absent: it isn't rendered into the issue drawer (see +`pearl--format-issue-as-org-entry'), so an offline client-side regroup can't +read it -- deferred until cycle drawer fields land.") + +(defun pearl--grouping-display-to-value (display) + "Coerce a DISPLAY grouping label to its `issueGrouping' value (or nil). +Signals a `user-error' for a label outside `pearl--grouping-choices'." + (let ((cell (assoc display pearl--grouping-choices))) + (unless cell + (user-error "Unknown grouping %s; use one of %s" + display (mapconcat #'car pearl--grouping-choices "/"))) + (cdr cell))) + +(defun pearl--check-grouping (value) + "Validate a grouping VALUE; signal a `user-error' on an unknown one. +Returns t when VALUE is nil (none) or one of the interactive grouping values +\(`pearl--grouping-choices'). `cycle' and any other unsupported value are +rejected -- the render path treats them as flat, but the interactive command +refuses them before writing the header." + (let ((allowed (delq nil (mapcar #'cdr pearl--grouping-choices)))) + (when (and value (not (member value allowed))) + (user-error "Unknown grouping %s; use %s or none" + value (mapconcat #'car (butlast pearl--grouping-choices) "/")))) + t) + +(defun pearl--source-with-grouping (source value) + "Return SOURCE with grouping VALUE recorded under `:client-group'. +VALUE is an `issueGrouping' string for all source types (uniform key); a view's +own server `:group' is left intact so clearing the override restores it. A nil +VALUE (none) removes `:client-group' entirely, so `pearl--effective-grouping' +falls back to the server `:group' (a view) or to flat (a filter)." + (if value + (plist-put (copy-sequence source) :client-group value) + (cl-loop for (k v) on source by #'cddr + unless (eq k :client-group) append (list k v)))) + +(defun pearl--effective-grouping (source) + "Return the `issueGrouping' value SOURCE should render by, or nil for flat. +An interactive `:client-group' overrides a view's server `:group'; with no +override, a view uses its `:group' and a filter is flat. Every grouping +consumer (render, merge, append, refresh) reads through this so a refresh +reproduces the layout on screen rather than the server grouping." + (or (plist-get source :client-group) + (plist-get source :group))) + ;;; Sentinel infrastructure for filter / pick-one prompts ;; ;; These sentinel constants and helpers back the prompt UX shared by |
