commitlint.sh (skip) not specified in config file

[sbsrnt]

I created a package that has shared lefthook config: https://github.com/exile-watch/splinters/tree/main/packages/lefthook-config
The file tree of above package:

root // @exile-watch/splinters lerna monorepo
| - .lefthook/commit-msg/commitlint.sh
| - lefthook.yml
| - packages
    | - lefthook-config
        | - lefthook.yml # shareable config
        | - scripts
            | - commitlint.sh

The root lefthook.yml looks like this:

remotes:
  - git_url: https://github.com/exile-watch/splinters
    configs:
      - packages/lefthook-config/lefthook.yml

Now, for some reason commit-msg script is working here.

But when I try to apply the same settings (literally copy-paste) to another repo it doesn't seem to find commitlint.sh: https://github.com/exile-watch/nucleus

On top of that, I'm not sure why the output lists commitlint.sh (skip) twice:

╭──────────────────────────────────────╮
│ 🥊 lefthook v1.6.1  hook: commit-msg │
╰──────────────────────────────────────╯
│  commitlint.sh (skip) not specified in config file
│  commitlint.sh (skip) not specified in config file

---

I thought I understood how to create a shareable solution but now I'm confused why commitlint.sh is not specified ._.
And why the script is specified in @exile-watch/splinters and not in @exile-watch/nucleus even though both root lefthook.yml rely on remotes config

[mrexox]

Hey! I the problem is that the script file name is commitlint.sh but the config has commitlint instead. To fix this the config should be changed to the following:

commit-msg:
  scripts:
    "commitlint.sh":
      runner: bash

Or the script name should be changed to just commitlint.

[sbsrnt]

Removing .lefthook/commit-msg/commitlint.sh removed the duplicated output 🤔

[sbsrnt]

I went with:

1. Remove .lefthook/commit-msg/commitlint.sh
2. `git commit -a -m "bricked commit" -> received only 1 errored output (correct)
3. npx lefthook uninstall
4. npx lefthook install
5. `git commit -a -m "bricked commit" -> received only 1 errored output (correct)

Now I'm double confused if .lefthook dir is required lol

[sbsrnt]

Now I also:

1. removed .git/hooks dir
2. npx lefthook uninstall
   Error: open .git/hooks: The system cannot find the file specified. (makes sense)
3. npx lefthook install
4. `git commit -a -m "bricked commit" -> received only 1 errored output (correct)

All the steps above were done without creating .lefthook/* dir - I guess that's... alright? @mrexox

Even though it's slightly contradictory to the script docs 🤔not sure if remotes and scripts defined in repo we are importing from are playing any role into that but it works - question is if it's a good thing or not

[mrexox]

So, you see the error twice because you have .lefthook/commit-msg/commitlint.sh both locally and in one of the remotes. I am not sure about how to handle this case 🤔 It's probably a very unusual case, but I will put it on my radar to think this out. Thank you for explaining the steps to reproduce it.

[sbsrnt]

I mean to be fair... wouldn't consuming remote .lefthook dir (like it's doing now) be a cleaner approach then 🤔?
(assumig we utilize remotes part in local lefthook.yml

Creating only lefthook.yml in a repo that consumes shared lefthook-config package with this content

remotes:
  - git_url: <git url>
    configs:
      - packages/lefthook-config/lefthook.yml

sounds pretty straight forward and is one less directory to maintain

---

And I suppose .lefthook/* dir in consuming repo would overwrite remote .lefthook/* scripts (or whatever is in it) (if it's not doing that already)