Skip to content

Add aligned completion annotations via affixation-function#4129

Merged
bbatsov merged 1 commit into
masterfrom
modernize-completion-annotations
Jul 23, 2026
Merged

Add aligned completion annotations via affixation-function#4129
bbatsov merged 1 commit into
masterfrom
modernize-completion-annotations

Conversation

@bbatsov

@bbatsov bbatsov commented Jul 23, 2026

Copy link
Copy Markdown
Member

Modernization pass on cider-completion.el:

  • Add an affixation-function (cider-affixate-symbol) next to the existing annotation-function. UIs that support it (built-in *Completions*, Corfu, Vertico) now render the type/namespace as an aligned column rather than a ragged suffix; company keeps using the annotation function.
  • Fold the capf table and the completing-read symbol table onto a shared cider--completion-table builder — they had duplicated the metadata / complete-with-action skeleton (the second was added in Optionally read symbol prompts via completing-read #4117).
  • Drop a no-op with-current-buffer, and refresh comments that called the capf company-only (corfu consumes the same :company-* properties).

No behavior change to matching or candidate sourcing. First slice of a broader completion modernization.

Provide an `affixation-function' (`cider-affixate-symbol') alongside the
existing `annotation-function', so completion UIs that support it (the
built-in `*Completions*', Corfu, Vertico) render the type/namespace info as
an aligned column instead of a ragged trailing string; company keeps using
the annotation function.

Also fold the capf table and the completing-read symbol table onto a shared
`cider--completion-table' builder (they had duplicated the metadata /
`complete-with-action' skeleton), drop a no-op `with-current-buffer', and
refresh comments that described the capf as company-only (corfu consumes the
same `:company-*' properties).
@bbatsov
bbatsov force-pushed the modernize-completion-annotations branch from 5c1991b to ea99557 Compare July 23, 2026 16:17
@bbatsov
bbatsov merged commit 26cc96a into master Jul 23, 2026
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant