GUI Git Hook fails when env defined in .npmrc #131

chrisblossom · 2017-05-22T18:26:56Z

my .npmrc:

//registry.npmjs.org/:_authToken=${NPM_TOKEN}
save-exact = true

git -c diff.mnemonicprefix=false -c core.quotepath=false -c credential.helper=sourcetree commit -q -F /var/folders/hm/n_tq5p6j7j313n28pjzvk63c0000gn/T/SourceTreeTemp.SaYrXv -a 
Error: Failed to replace env in config: ${NPM_TOKEN}
    at /Users/chris/.nvm/versions/node/v6.10.3/lib/node_modules/npm/lib/config/core.js:418:13
    at RegExp.[Symbol.replace] (native)
    at RegExp.[Symbol.replace] (native)
    at String.replace (native)
    at envReplace (/Users/chris/.nvm/versions/node/v6.10.3/lib/node_modules/npm/lib/config/core.js:414:12)
    at parseField (/Users/chris/.nvm/versions/node/v6.10.3/lib/node_modules/npm/lib/config/core.js:392:7)
    at /Users/chris/.nvm/versions/node/v6.10.3/lib/node_modules/npm/lib/config/core.js:335:17
    at Array.forEach (native)
    at Conf.add (/Users/chris/.nvm/versions/node/v6.10.3/lib/node_modules/npm/lib/config/core.js:334:23)
    at ConfigChain.addString (/Users/chris/.nvm/versions/node/v6.10.3/lib/node_modules/npm/node_modules/config-chain/index.js:244:8)
nvm is not compatible with the npm config "prefix" option: currently set to ""
Run `nvm use --delete-prefix v6.10.3 --silent` to unset it.
Found '/Users/chris/github/app/.nvmrc' with version <6.10.3>
Error: Failed to replace env in config: ${NPM_TOKEN}
    at /Users/chris/.nvm/versions/node/v6.10.3/lib/node_modules/npm/lib/config/core.js:418:13
    at RegExp.[Symbol.replace] (native)
    at RegExp.[Symbol.replace] (native)
    at String.replace (native)
    at envReplace (/Users/chris/.nvm/versions/node/v6.10.3/lib/node_modules/npm/lib/config/core.js:414:12)
    at parseField (/Users/chris/.nvm/versions/node/v6.10.3/lib/node_modules/npm/lib/config/core.js:392:7)
    at /Users/chris/.nvm/versions/node/v6.10.3/lib/node_modules/npm/lib/config/core.js:335:17
    at Array.forEach (native)
    at Conf.add (/Users/chris/.nvm/versions/node/v6.10.3/lib/node_modules/npm/lib/config/core.js:334:23)
    at ConfigChain.addString (/Users/chris/.nvm/versions/node/v6.10.3/lib/node_modules/npm/node_modules/config-chain/index.js:244:8)
nvm is not compatible with the npm config "prefix" option: currently set to ""
Run `nvm use --delete-prefix v6.10.3` to unset it.
> husky - Can't find npm in PATH. Skipping precommit script in package.json
Completed successfully

The text was updated successfully, but these errors were encountered:

MikelArnaiz · 2018-07-16T08:17:59Z

Same here.

In Sourcetree and Fork the precommit is not run allowing you to commit.

In my .npmrc I have:

//rll.jfrog.io/rll/api/npm/my-npm/:_authToken=${NPM_TOKEN}
engine-strict=true

Modifying ./.git/hooks/pre-commit after the message husky > can't find npm in PATH, skipping precommit script in package.json with an exit 1 instead exit 0 will abort the commit and give you the error message:

Pre-commit hook 'pre-commit' has failed:

Error: Failed to replace env in config: ${NPM_TOKEN}
    at /Users/MY-USERNAME/.nvm/versions/node/v10.4.1/lib/node_modules/npm/lib/config/core.js:415:13
    at String.replace (<anonymous>)
    at envReplace (/Users/MY-USERNAME/.nvm/versions/node/v10.4.1/lib/node_modules/npm/lib/config/core.js:411:12)
    at parseField (/Users/MY-USERNAME/.nvm/versions/node/v10.4.1/lib/node_modules/npm/lib/config/core.js:389:7)
    at /Users/MY-USERNAME/.nvm/versions/node/v10.4.1/lib/node_modules/npm/lib/config/core.js:330:24
    at Array.forEach (<anonymous>)
    at Conf.add (/Users/MY-USERNAME/.nvm/versions/node/v10.4.1/lib/node_modules/npm/lib/config/core.js:328:23)
    at ConfigChain.addString (/Users/MY-USERNAME/.nvm/versions/node/v10.4.1/lib/node_modules/npm/node_modules/config-chain/index.js:244:8)
    at Conf.<anonymous> (/Users/MY-USERNAME/.nvm/versions/node/v10.4.1/lib/node_modules/npm/lib/config/core.js:316:10)
    at /Users/MY-USERNAME/.nvm/versions/node/v10.4.1/lib/node_modules/npm/node_modules/graceful-fs/graceful-fs.js:78:16
/Users/MY-USERNAME/.nvm/versions/node/v10.4.1/lib/node_modules/npm/lib/npm.js:61
      throw new Error('npm.load() required')
      ^

Error: npm.load() required
    at Object.get (/Users/MY-USERNAME/.nvm/versions/node/v10.4.1/lib/node_modules/npm/lib/npm.js:61:13)
    at process.errorHandler (/Users/MY-USERNAME/.nvm/versions/node/v10.4.1/lib/node_modules/npm/lib/utils/error-handler.js:205:18)
    at process.emit (events.js:182:13)
    at process._fatalException (internal/bootstrap/node.js:448:27)
nvm is not compatible with the npm config "prefix" option: currently set to ""
Run `nvm use --delete-prefix v10.4.1 --silent` to unset it.
husky > can't find npm in PATH, skipping precommit script in package.json

I see that indeed that npm is not in the PATH, which is:

echo $PATH
/Users/MY-USERNAME/.nvm/versions/node/v10.4.1/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin

but husky sees this PATH

/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/usr/local/bin:/usr/local

As you can see I'm using nvm

manuel-reil · 2019-12-21T10:18:44Z

Have you guys found a solution to this?

lukeselbeck · 2020-06-24T18:31:01Z

I'm still getting this, why was this closed? What's the solution?

lukeselbeck · 2020-06-24T21:40:44Z

Figured out my problem. For any future peoples: I was commiting/pushing with vscode and that was giving me the above error. When I commit/push with the terminal, it works fine!

Ethaan · 2020-07-17T15:36:27Z

@lukeselbeck i'm commiting/pushing using git kraken and visual studio, there should be a way to export variables for husky

youssef-tharwat · 2020-10-13T15:19:59Z

Why is this closed?

malikalimoekhamedov · 2021-01-28T16:33:17Z

Yes, why is this closed?

sexta13 · 2021-03-07T00:54:03Z

I'm facing same issue as well...

paschalidi · 2021-06-07T07:49:52Z

Still having this issue when I am committing with gitkraken. Perhaps we could reopen, @chrisblossom what you think?

chrisblossom · 2021-06-07T18:02:38Z

Still having this issue when I am committing with gitkraken. Perhaps we could reopen, @chrisblossom what you think?

Because this issue was created so long ago and I closed it without commenting why (my mistake, sorry!), I would recommend opening a new issue with a link to this issue in-case it is still relevant.

paschalidi · 2021-06-07T19:55:33Z

Will do thanks

SalahAdDin · 2021-07-12T09:38:14Z

I'm facing this problem again, with the last husky version.

MacRayy · 2021-08-11T14:06:24Z

I'm having the same issue with GitKraken

carpben · 2022-01-03T14:49:00Z

Facing the same issue with Sourcetree.

YousefMMS · 2022-02-03T16:35:05Z

Having the same issue with Sourcetree & vscode

carpben · 2022-02-04T17:46:03Z

I published a solution on StackOverflow. Can you please try it out and let me know if it works for you too?
https://stackoverflow.com/questions/135688/setting-environment-variables-on-os-x/70959181#70959181

michealroberts · 2022-08-22T16:37:25Z

This should't be closed as the above fix does not work for me, using Github Desktop.

tobilen mentioned this issue Sep 2, 2017

Add prettier joshwcomeau/react-flip-move#182

Merged

7 tasks

chrisblossom closed this as completed Mar 20, 2019

carpben mentioned this issue Jan 3, 2022

Git hooks fail to replace environment variable. #1088

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

GUI Git Hook fails when env defined in .npmrc #131

GUI Git Hook fails when env defined in .npmrc #131

chrisblossom commented May 22, 2017

MikelArnaiz commented Jul 16, 2018

manuel-reil commented Dec 21, 2019

lukeselbeck commented Jun 24, 2020

lukeselbeck commented Jun 24, 2020

Ethaan commented Jul 17, 2020 •

edited

youssef-tharwat commented Oct 13, 2020

malikalimoekhamedov commented Jan 28, 2021

sexta13 commented Mar 7, 2021

paschalidi commented Jun 7, 2021 •

edited

chrisblossom commented Jun 7, 2021

paschalidi commented Jun 7, 2021

SalahAdDin commented Jul 12, 2021

MacRayy commented Aug 11, 2021

carpben commented Jan 3, 2022

YousefMMS commented Feb 3, 2022

carpben commented Feb 4, 2022

michealroberts commented Aug 22, 2022

GUI Git Hook fails when env defined in .npmrc #131

GUI Git Hook fails when env defined in .npmrc #131

Comments

chrisblossom commented May 22, 2017

MikelArnaiz commented Jul 16, 2018

manuel-reil commented Dec 21, 2019

lukeselbeck commented Jun 24, 2020

lukeselbeck commented Jun 24, 2020

Ethaan commented Jul 17, 2020 • edited

youssef-tharwat commented Oct 13, 2020

malikalimoekhamedov commented Jan 28, 2021

sexta13 commented Mar 7, 2021

paschalidi commented Jun 7, 2021 • edited

chrisblossom commented Jun 7, 2021

paschalidi commented Jun 7, 2021

SalahAdDin commented Jul 12, 2021

MacRayy commented Aug 11, 2021

carpben commented Jan 3, 2022

YousefMMS commented Feb 3, 2022

carpben commented Feb 4, 2022

michealroberts commented Aug 22, 2022

Ethaan commented Jul 17, 2020 •

edited

paschalidi commented Jun 7, 2021 •

edited