karthink

We can replace the crowdsourced prompts option with the fabric patterns, as I think the crowdsourced prompts aren't getting much use. A PR for this is welcome.

> The prompts, referred to as "patterns", appear to be intriguing and potentially more useful than the crowd-sourced options we currently utilize. Yes, that appears to be the case. Eventually...

@ParetoOptimalDev I've pushed a lazy fix in 3842d564. By default the mpv window opens immediately. Let me know if this makes the feedback issue better or worse.

> This seems to be the best one can do to get some feedback asap: > > ``` > mpv --osd-level=3 --force-window=immediate https://youtu.be/DYeh7_FdmDg > ``` From reading the man page,...

Could you move all the git-related functions `gptel-context--git-files`, `gptel-context--skip-file-p`, `gptel-context--message-git-skipped` and `gptel-context--build-git-cache` into one place in the file? You can add a `;;; git tracking functions` header over them for...

`locate-dominating-file` can be _very slow_ on remote files, to the point of locking up Emacs for a while. It might be worth using `file-remote-p` and skipping the gitignore check if...

> @karthink, is there an estimated timeline for merging this? I'd be happy to contribute by implementing the hash-table optimisation that was discussed, if that would help move this PR...

> @karthink, shall we make a decision regarding this? If you don’t want to merge it, that’s totally fine by me, but I would prefer to close it so that...

Changed the caching logic, fixed some bugs and merged. Thanks for the PR @benthamite.

> I've played around with it and it integrates very nicely, except for one issue: > since I am using pre-existing overlays (not in any way related to the overlays...