Better monorepo support

[VanTanev]

Hey there!

I have been adopting lefthook in a relatively-large monorepo, but unfortunately it's very unergonomic in terms of configuration. The best I have come up with so far is this:

# /lefthook.yml
__shared__:
  commands:
    tsc: &tsc
      run: pnpm tsc --noEmit
    lint: &lint
      glob: "*.{js,ts,cjs,mjs,d.cts,d.mts,jsx,tsx}"
      run: pnpm eslint --max-warnings=0 --report-unused-disable-directives --no-warn-ignored {staged_files}
   # more shared commands here
   
pre-commit:
  parallel: true
  commands:
    "@monorepo-packages/database:tsc": { root: packages/database, <<: *tsc }
    "@monorepo-packages/database:lint": { root: packages/database, <<: *lint }
    # more commands per-package here

    "@monorepo/frontend-nextjs:tsc": { root: apps/frontend-nextjs, <<: *tsc }
    "@monorepo/frontend-nextjs:lint": { root: apps/frontend-nextjs, <<: *lint }

    "@monorepo/frontend-vite:tsc": { root: apps/frontend-vite, <<: *tsc }
    "@monorepo/frontend-vite:lint": { root: apps/frontend-vite, <<: *lint }  

I am not aware of a more compact way to define scripts here.

The above wouldn't be so bad if not for the fact that every time a new package is added the root configuration must be updated. So, even though we have a template for creating packages, we must also manually edit lefthook.yml

An ideal solution would probably be to be able to define multiple pattern-matching options in extends

# /lefthook.yml
extends
 - "apps/*/lefthook.yml"
 - "packages/*/lefthook.yml"

# /package/database/lefthook.yml
pre-commit:
  parallel: true
  commands:
    tsc:
      run: pnpm tsc --noEmit
    lint:
      glob: "*.{js,ts,cjs,mjs,d.cts,d.mts,jsx,tsx}"
      run: pnpm eslint --max-warnings=0 --report-unused-disable-directives --no-warn-ignored {staged_files}

And then have those configs merged at runtime.

Does an API like that make sense, and is if so, @mrexox would you be open to patches introducing it?

[mrexox]

Hey @VanTanev! So, you suggest to add glob expanding to the extends values? I think this is feasible 👍 I'll put it on my radar and implement later. But if you want to submit a PR, I'll be happy 😄

[VanTanev]

Go is definitely not my main language, but I'll give it a... go ;)

[mrexox]

Can you take a look at the PR? #853

It was easier than I thought. I'll release it this week