Bug in Argument Parsing for conventional-pre-commit Hook Configuration

[jg197]

Hi,

When using the args field to pass command-line arguments in the pre-commit hook configuration for conventional-pre-commit, the arguments are not being parsed correctly. For example, the configuration:

- repo: https://github.com/compilerla/conventional-pre-commit
  rev: v4.0.0
  hooks:
    - id: conventional-pre-commit
      stages: [commit-msg]
      args: [--verbose, --force-scope, --scopes api]

fails to correctly interpret the arguments. However, directly specifying the CLI command via the entry field works as intended:

- repo: https://github.com/compilerla/conventional-pre-commit
  rev: v4.0.0
  hooks:
    - id: conventional-pre-commit
      stages: [commit-msg]
      entry: conventional-pre-commit --verbose --force-scope --scopes api .git/COMMIT_EDITMSG

This was true for the commit message:
fix(foobar): test commit rejected as expected only when specifying the CLI command via the entry field.

I believe the root cause is how the args are being handled. I don't think there are any tests that cover usage of the args field, and instead existing tests call the module directly.

Thanks!

[maikol-solis]

Today I had the same issue as you, and this format worked for me

repos:
  - repo: https://github.com/compilerla/conventional-pre-commit
    rev: v4.0.0
    hooks:
      - id: conventional-pre-commit
        stages: [commit-msg]
        args:
          [
            "--verbose",
            "--scope",
            "data,src,lib,munge,docs,config",
            "feat",
            "fix",
            "refactor",
            "perf",
            "style",
            "test",
            "docs",
            "build",
            "chore",
            "ci"
          ]

[wkeifenheim]

On v4.3.0, specifying args as in @maikol-solis solution did not work for me. Note: I omitted scope from my args as my repo only has one scope.

@jg197's approach still works.

- repo: https://github.com/compilerla/conventional-pre-commit
    rev: v4.3.0
    hooks:
      - id: conventional-pre-commit
        entry: conventional-pre-commit --verbose feat fix chore test .git/COMMIT_EDITMSG
        stages: [commit-msg]

[MRDGH2821]

Having a config file or --types cli arg would be good. Types can be comma separated just like --scopes