Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Create gptel define tool macro #1

Open
wants to merge 5 commits into
base: experimental
Choose a base branch
from

Conversation

swflint
Copy link

@swflint swflint commented Mar 8, 2025

Per karthink#685, PRing here.

I'll probably spend a bit of time trying to figure out how to improve argument parsing.

psionic-k and others added 5 commits March 5, 2025 13:16
* gptel.el (gptel--handle-post-insert): Use
`gptel-response-separator'.
* gptel-org.el (gptel-org--create-prompt): The warning and
fallback to parsing the entire buffer can happen in the
conditionals.  This change is primarily structural,
non-behavioral.
* gptel-kagi.el (gptel--parse-buffer): Use this helper instead of
the expanded call to string-trim.
Rename gptel--get-bounds to gptel--get-response-bounds since the
upcoming explicit tool use feature will mean that there will be
more than one kind of bound in the chat buffer.

* gptel.el: (gptel--get-response-bounds, gptel--ediff,
gptel--mark-response, gptel--previous-variant): Fix calls to
gptel--get-bounds.

* gptel-transient.el (gptel--regenerate): Ditto.
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.

2 participants