di-sukharev / opencommit

Auto-generate impressive commits with AI in 1 second 🤯🔫
https://www.npmjs.com/package/opencommit
MIT License
5.67k stars 292 forks source link

[Feature]: More customizable commit message template and support for Perforce VCS #356

Open philou opened 3 weeks ago

philou commented 3 weeks ago

Description

💬 Discuss and agree about design before contribution

I am part of the Murex organization, we have a bit of time this summer, and we would like to contribute to your project. We want to extend it with the following features:

To avoid rework, I would like to start a conversation to get your feedback about the proposed features and design before we send pull request.

Suggested Solution

Alternatives

We could start from another tool (ex aicommits) We could build something from scratch ourselves

Additional Context

We also plan to use the tool on our own internal RAG. I'm not sure what impact will this have yet.

di-sukharev commented 3 weeks ago

hi @philou, nice to meet you

will get back to you this week :)

di-sukharev commented 2 weeks ago

@philou hi

could you provide an example of what response you expect to get with commit customization? and any draft of a command or config to achieve this would be great

as for the perforce, i think the best solution here is to support it as a dependency injection, doing it this way you would need to extract git commands (as they are hardcoded now) and pass them (or Perforce) into the (let's call it) module which requests an LLM to generate a commit message as per the config and CLI options

if Perforce is used instead of git then the github action or githook should be disabled, i imagine a one liner if (engine === perforce)log('no no no, you cant set the hook with perforce'); return;`

philou commented 6 days ago

Thanks for your answer.

We were thinking of the same thing for Perforce.

Regarding the commit customization, we can already go a long way with the template that oco currently provides. We might come back to you later when we need more.

We might also submit PRs to allow configuring the Ollama server address and adding a flowiseAI engine.