-
Notifications
You must be signed in to change notification settings - Fork 7
Open
Description
The following things need to be sorted out for production-grade rollout.
P0:
- Need to look into the exact GitHub copilot algo for Git diff. [Currently we truncate after 800 characters]
- We can swing between 3 LLMs from terminal, but need to confirm if exactly those model variations are being called. [Can check with Charles proxy]
- Local and Cloud LLMs produce similar results, (atleast for me). Need to check what model does Apple intelligence runs and its benchmarks to fully trust this observation.
- Longer prompt as written in the Readme hallucinates the LLM giving irrelevant messages, so i have written a simpler one-liner/shot prompt. cc: @eonist need to reconsider this.
- We should rethink the suffixes (--verbose and --minimal), while using these developer should directly relate to which LLM are they selecting through this prefix so I propose (--private and --gpt) makes it easier to remember
6 Should mention the exact installation steps in the READMe, for instance (sudo curl -L "" -o /usr/local/bin/diffsense && sudo chmod +x /usr/local/bin/diffsense) would be needed to run(something like this, based on where we want to save the script). Clarity needed!
P1 [Future versions]
- I have not included the autonomous no pop up workflow for agents in this version, we can add it later.
- In case diff is long, local LLM is not used to prune it further, direct truncation is done, have added TODO in the script.
Extras
- We should try improving the quality of the commit message. I have observed it to give really generic messages on occassions.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels