tristan/gptel
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

gptel-transient: Change menu display

Browse source 
* gptel-transient.el (gptel-menu, gptel-lisp-variable,
gptel--infix-num-messages-to-send, gptel--infix-max-tokens,
gptel--infix-provider): Improve the wording and the display of the
default settings.  Define a new class `gptel-lisp-variable` to
faciliate this.
This commit is contained in:
Karthik Chikmagalur 2024-03-13 22:05:11 -07:00
parent f529457bbe
commit 7e6b106516
1 changed files with 19 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

24
gptel-transient.el
View file

@ -294,6 +294,16 @@ documention."
(ignore-errors (ignore-errors
(read-from-minibuffer prompt initial-input read-expression-map t history))) (read-from-minibuffer prompt initial-input read-expression-map t history)))
(defclass gptel-lisp-variable (transient-lisp-variable)
((display-nil :initarg :display-nil))
"Lisp variables that show :display-nil instead of nil.")
(cl-defmethod transient-format-value
((obj gptel-lisp-variable))
(propertize (prin1-to-string (or (oref obj value)
(oref obj display-nil)))
'face 'transient-value))
(transient-define-infix gptel--infix-num-messages-to-send () (transient-define-infix gptel--infix-num-messages-to-send ()
"Number of recent messages to send with each exchange. "Number of recent messages to send with each exchange.
@ -301,9 +311,11 @@ By default, the full conversation history is sent with every new
prompt. This retains the full context of the conversation, but prompt. This retains the full context of the conversation, but
can be expensive in token size. Set how many recent messages to can be expensive in token size. Set how many recent messages to
include." include."
:description "Number of past messages to send" :description "previous responses"
:class 'transient-lisp-variable :class 'gptel-lisp-variable
:variable 'gptel--num-messages-to-send :variable 'gptel--num-messages-to-send
:display-nil 'all
:format " %k %v %d"
:key "-n" :key "-n"
:prompt "Number of past messages to include for context (leave empty for all): " :prompt "Number of past messages to include for context (leave empty for all): "
:reader 'gptel--transient-read-variable) :reader 'gptel--transient-read-variable)
@ -315,8 +327,9 @@ This is roughly the number of words in the response. 100-300 is a
reasonable range for short answers, 400 or more for longer reasonable range for short answers, 400 or more for longer
responses." responses."
:description "Response length (tokens)" :description "Response length (tokens)"
:class 'transient-lisp-variable :class 'gptel-lisp-variable
:variable 'gptel-max-tokens :variable 'gptel-max-tokens
:display-nil 'auto
:key "-c" :key "-c"
:prompt "Response length in tokens (leave empty: default, 80-200: short, 200-500: long): " :prompt "Response length in tokens (leave empty: default, 80-200: short, 200-500: long): "
:reader 'gptel--transient-read-variable) :reader 'gptel--transient-read-variable)
@ -344,7 +357,7 @@ responses."
(transient-define-infix gptel--infix-provider () (transient-define-infix gptel--infix-provider ()
"AI Provider for Chat." "AI Provider for Chat."
:description "GPT Model: " :description "GPT Model"
:class 'gptel-provider-variable :class 'gptel-provider-variable
:prompt "Model provider: " :prompt "Model provider: "
:variable 'gptel-backend :variable 'gptel-backend
@ -412,7 +425,8 @@ responses."
"Send ARGS." "Send ARGS."
:key "RET" :key "RET"
:description "Send prompt" :description "Send prompt"
(interactive (list (transient-args transient-current-command))) (interactive (list (transient-args
(or transient-current-command 'gptel-menu))))
(let ((stream gptel-stream) (let ((stream gptel-stream)
(in-place (and (member "i" args) t)) (in-place (and (member "i" args) t))
(output-to-other-buffer-p) (output-to-other-buffer-p)