bug: pre-commit's script is installed in migration mode

[joscha]

I randomly started getting:

bug: pre-commit's script is installed in migration mode
run `pre-commit install -f --hook-type commit-msg` to fix this

unsure where the issue is.
I did find pre-commit/pre-commit#1419 and commented there, however, it was locked by the project owner there unfortunately, suggesting we might be the culprit.

Any ideas?

git config --unset-all core.hooksPath
pre-commit install -f --hook-type commit-msg

fixed it, but I am unsure if I have now bypassed some of the logic from git-hooks.nix.

[sandydoo]

What even is migration mode?

[joscha]

What even is migration mode?

I think it refers to this: https://github.com/pre-commit/pre-commit.com/blob/main/sections/advanced.md#running-in-migration-mode

But I haven't done anything to the install other than updating my flake.lock

[joscha]

I do sometimes run pre-commit run --all-files directly from my cmd, could that be an issue?

[sandydoo]

I do sometimes run pre-commit run --all-files directly from my cmd, could that be an issue?

That's completely fine. We don't patch pre-commit, despite what the maintainers of pre-commit insist on saying...

It looks like they're searching for a hash in .git/hooks/<hook_stage>, and if they don't find it, they error out. We call pre-commit install internally, so I'm not really sure why this hash would be missing.

[sandydoo]

If anyone encounters this, please save a copy of .git/hooks before trying to fix the error. The files in there would really help debug this issue.

[joscha]

I do sometimes run pre-commit run --all-files directly from my cmd, could that be an issue?

That's completely fine. We don't patch pre-commit, despite what the maintainers of pre-commit insist on saying...

@asottile this may have been outdated information. Might be worth unlocking that issue, so at least the two issues get cross-referenced for others to find.

[joscha]

If anyone encounters this, please save a copy of .git/hooks before trying to fix the error. The files in there would really help debug this issue.

I had that as an afterthought. I also didn't look at core.hooksPath either before blowing it away 🤦 . If/when it happens again I'll report here.