error-message: Stop gaslighting our users with "this is probably your fault"

Author: iarna

lib/utils/error-message.js

@@ -46,7 +46,6 @@ function errorMessage (er) {
         [
           '',
           'Failed using git.',
-          'This is most likely not a problem with npm itself.',
           'Please check if you have git installed and in your PATH.'
         ].join('\n')
       ])
@@ -178,8 +177,7 @@ function errorMessage (er) {
       detail.push([
         'network',
         [
-          'This is most likely not a problem with npm itself',
-          'and is related to network connectivity.',
+          'This is a problem related to network connectivity.',
           'In most cases you are behind a proxy or have bad network settings.',
           '\nIf you are behind a proxy, please make sure that the',
           "'proxy' config is set properly.  See: 'npm help config'"
@@ -192,7 +190,6 @@ function errorMessage (er) {
       detail.push([
         'package.json',
         [
-          'This is most likely not a problem with npm itself.',
           "npm can't find a package.json file in your current directory."
         ].join('\n')
       ])
@@ -201,7 +198,6 @@ function errorMessage (er) {
     case 'ETARGET':
       short.push(['notarget', er.message])
       msg = [
-        'This is most likely not a problem with npm itself.',
         'In most cases you or one of your dependencies are requesting',
         "a package version that doesn't exist."
       ]
@@ -235,8 +231,8 @@ function errorMessage (er) {
       detail.push([
         'nospc',
         [
-          'This is most likely not a problem with npm itself',
-          'and is related to insufficient space on your system.'
+          'There appears to be insufficient space on your system to finish.',
+          'Clear up some disk space and try again.'
         ].join('\n')
       ])
       break
@@ -246,9 +242,7 @@ function errorMessage (er) {
       detail.push([
         'rofs',
         [
-          'This is most likely not a problem with npm itself',
-          'and is related to the file system being read-only.',
-          '\nOften virtualized file systems, or other file systems',
+          'Often virtualized file systems, or other file systems',
           "that don't support symlinks, give this error."
         ].join('\n')
       ])
@@ -259,8 +253,7 @@ function errorMessage (er) {
       detail.push([
         'enoent',
         [
-          'This is most likely not a problem with npm itself',
-          'and is related to npm not being able to find a file.',
+          'This is related to npm not being able to find a file.',
           er.file ? "\nCheck if the file '" + er.file + "' is present." : ''
         ].join('\n')
       ])
@@ -280,18 +273,6 @@ function errorMessage (er) {
       ])
       break
 
-    case 'EISDIR':
-      short.push(['eisdir', er.message])
-      detail.push([
-        'eisdir',
-        [
-          'This is most likely not a problem with npm itself',
-          'and is related to npm not being able to find a package.json in',
-          'a package you are trying to install.'
-        ].join('\n')
-      ])
-      break
-
     default:
       short.push(['', er.message || er])
       break