Editing 3D shapes with hand gestures (threejs / mediapipe computer vision) (twitter.com)

I don't understand the reasoning for persisting LLM output that can be generated at any point. If I want to use an LLM to understand someone else's commits, I can use the LLM best suited for that task at the time I need the information, which will likely be more accurate than what was available at the time of the commit and will have access to more context.

I also believe that commit messages should focus on information the code doesn't already convey. Whatever the LLM can generate from looking at your code is likely not the info I'll seek when I read your commit message.

bee_rider · 48m ago

It looks like it just is based on the git diff and status, at least as far as I can tell in a quick skim…

Hypothetically, a tool like this could ingest the bug report you were fixing, some emails, etc etc. It could also read the whole project (to get more context than just the diff). In principle there’s no reason it couldn’t relay more info than just the diff, in some extreme form…

Also, it could be seen as producing a starting point. When a person picks which AI generated text to keep, that is enough to add a bit of human spark into the system, right?

pvdebbe · 1h ago

Maybe I am a bit old-fashioned but I think the commit message should convey intent and not content of the diffs. Perhaps the real utility of this is to describe existing commits in a repository.

owebmaster · 1h ago

I'm also old-fashioned but I always thought it made much more sense to give a content diff, it makes it easier to find changes.

JimDabell · 1h ago

The commit itself is the content diff. Repeating that in the log message is redundant.

owebmaster · 21m ago

no, it is not redundant, a summary makes it easier to search and find the correct commit to read the full diff.

nickcw · 1h ago

When you are looking through commit messages, "Why?", Is the question you want answered. The diff contains "What?" and "How?".

Assuming that the commits in this repo were generated by this tool it is missing the "Why?".

flysand7 · 1h ago

You are talking about the commit message body, right, not just the header? Because for me it's something similar, but:

Header: Contains "What" and the scope of the changes, as short as possible Body: Contains "Why" and the full explanation of the change

myrmidon · 1h ago

Fully agree. Also, using LLMs for things like this can have bad side-effects, too, simply because it raises the noise-floor:

By spelling out things that are not noteworthy enough for a human, you make it more difficult to find comments that are (and were). Injecting a lot of irrelevant information can hamper understanding even if it is technically completely correct.

alzamixer · 1h ago

I use the following script to allow copilot vim plugin to help me.

```plaintext name=../../bin/assisted-commit

#!/bin/bash

# Run git commit with --verbose --dry-run and save the output git commit --verbose --dry-run > ./commit.message

# Prepend # to every line and add "conventional commit message:" at the end sed -i 's/^/# /' ./commit.message echo "# uncommented conventional commit message using feat, fix or doc flags. !beakingchange iff change breaks backward compatibility:" >> ./commit.message echo "" >> ./commit.message

# Open the file in vim for editing, with cursor on a new line at the end and in insert mode vim +':normal Go' +startinsert ./commit.message

# Filter out commented lines and save to a temporary file grep -v '^#' ./commit.message > ./commit.message.filtered

# Commit using the filtered file git commit -F ./commit.message.filtered

# Delete the files rm ./commit.message ./commit.message.filtered

```

esafak · 58m ago

Don't forget to include committed code in the context when amending.

infocollector · 1h ago

Looks like openrouter api can be self-hosted, which means you should be able to run this locally. If anyone is able to run this with ollama, please do post how you did that? :)

theblazehen · 49m ago

The openrouter api is the same as the openai api, so you should be able to use the openai api compatibility built into ollama after updating the url in /src/acmsg/constants.py