Request: Lenient shell command parser

Description

It’d be great if normal bash command modifiers could be accepted (see below for example). I know same can be accomplished with an external script or an npm script – but a one-liner would be super useful and less clutter.

Steps to reproduce

This example will fail the hook if there are any lingering .only()s left in staged test files. The ! is used to negate the exit code of grep.

  "lint-staged": {
    "src/**/*.test.js": [
      "!grep -q '\\.only\\W' "
    ]
  }

Debug Logs

...
Running tasks for src/**/*.test.js [started]
  lint-staged:make-cmd-tasks Creating listr tasks for commands [ "!grep -q '\\.only\\W' " ] +0ms
  lint-staged:find-bin Resolving binary for command `!grep -q '\.only\W' ` +349ms
Running tasks for src/**/*.test.js [failed]
→ !grep could not be found. Try `npm install !grep`.
Running linters... [failed]
!grep could not be found. Try `npm install !grep`.

This and not being able to position the file in the command, (e.g. grep -q '\.only\W' {}; test $? -eq 1 where {} is the (supposed!) file placeholder), is a real handicap in this otherwise very useful tool, IMHO.

Environment

• OS: macOS High Sierra
• Node.js: v11.13.0
• lint-staged: v8.2.1

[moos]

@iiroj Yes that worked – I don’t know how, but it did. Brilliant solution to both complex commands and filename placeholder!

// lint-staged.config.js:
module.exports = {
  "*.js": [
    () => "! grep foo bar.j"
  ]
};
// bar.js:
// I haz foo

Output:

  ❯ Running linters...
    ❯ Running tasks for *.js
      ✖ () => "! grep foo bar.js"

✖ () => "! grep foo bar.js" found some errors. Please fix them and try committing again.
// I haz foo

Removing the ! is successful. Compound commands also works:

  "*.js": [
    () => "grep foo bar.js; test $? -eq 1"
  ]

[lkraav]

The ! is used to negate the exit code of grep.

PS nice idea @moos point_up

Not so simple with ! after all - obviously it also negates exit code 0. Had to convert to lint-staged.config.js and build my own command-line:

const path = require('path');

module.exports = {
    '*.php': (filenames) => {
        // @see https://github.com/okonet/lint-staged#example-use-relative-paths-for-commands
        const cwd = process.cwd();
        const relativeFilenames = filenames.map((file) => path.relative(cwd, file)).join(' ');

        return [
            `./vendor/bin/phpcbf ${relativeFilenames}; if [ $? -eq 1 ]; then exit 0; fi`,
            `./vendor/bin/phpcs -s ${relativeFilenames}`,
            `./vendor/bin/psalm ${relativeFilenames}`,
        ]
    }
}