Change iterm escape code

gartnera · gartnera · commit 23ad2ec87612 · 2018-03-21T20:05:51.000-07:00
diff --git a/index.js b/index.js
@@ -245,7 +245,7 @@ module.exports = function(results) {
   }
 
   if (process.env.FORCE_ITERM_HINT === 'true' || (process.stdout.isTTY && !process.env.CI)) {
-    output = '\u001B]50;CurrentDir=' + process.cwd() + '\u0007' + output;
+    output = '\u001B]1337;CurrentDir=' + process.cwd() + '\u0007' + output;
   }
 
   return total > 0 ? output : '';
diff --git a/test/absolute-paths-no-gray/basic/results.txt b/test/absolute-paths-no-gray/basic/results.txt
@@ -1,4 +1,4 @@
-]50;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
+]1337;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
   [33m⚠[39m  [4mhttp://eslint.org/docs/rules/[37mno-unused-vars[39m[24m   path is defined but never used                    
   [4m/home/usr/roy/index.js:8:2[24m
     path = require( 'path' ),
diff --git a/test/absolute-paths-no-gray/fatal/results.txt b/test/absolute-paths-no-gray/fatal/results.txt
@@ -1,4 +1,4 @@
-]50;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
+]1337;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
   [31m✘[39m  [4mhttp://eslint.org/docs/rules/[24m  Unexpected token {  
   [4m/home/usr/roy/index.js:3:13[24m
   
diff --git a/test/absolute-paths-no-gray/missing-line-column/results.txt b/test/absolute-paths-no-gray/missing-line-column/results.txt
@@ -1,4 +1,4 @@
-]50;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
+]1337;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
   [33m⚠[39m  [4mhttp://eslint.org/docs/rules/[24m                 path is defined but never used                    
   [4m/home/usr/roy/index.js:0:0[24m
     path = require( 'path' ),
diff --git a/test/absolute-paths-no-gray/missing-messages/results.txt b/test/absolute-paths-no-gray/missing-messages/results.txt
@@ -1,4 +1,4 @@
-]50;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
+]1337;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
   [31m✘[39m  [4mhttp://eslint.org/docs/rules/[37mcurly[39m[24m            Expected { after 'if' condition                   
   [4m/home/usr/roy/test/runner.js:41:2[24m
     if ( err ) console.log( 'failed tests: ' + err );
diff --git a/test/absolute-paths-no-gray/missing-rule-id/results.txt b/test/absolute-paths-no-gray/missing-rule-id/results.txt
@@ -1,4 +1,4 @@
-]50;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
+]1337;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
   [33m⚠[39m  [4mhttp://eslint.org/docs/rules/[24m                 path is defined but never used                    
   [4m/home/usr/roy/index.js:8:2[24m
     path = require( 'path' ),
diff --git a/test/absolute-paths-no-gray/single-error/results.txt b/test/absolute-paths-no-gray/single-error/results.txt
@@ -1,4 +1,4 @@
-]50;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
+]1337;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
   [33m⚠[39m  [4mhttp://eslint.org/docs/rules/[37mno-unused-vars[39m[24m   path is defined but never used                    
   [4m/home/usr/roy/index.js:8:2[24m
     path = require( 'path' ),
diff --git a/test/absolute-paths/basic/results.txt b/test/absolute-paths/basic/results.txt
@@ -1,4 +1,4 @@
-]50;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
+]1337;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
   [33m⚠[39m  [4m[90mhttp://eslint.org/docs/rules/[37mno-unused-vars[90m[39m[24m   path is defined but never used                    
   [4m[90m/home/usr/roy/index.js:8:2[39m[24m
   [90m  path = require( 'path' ),[39m
diff --git a/test/absolute-paths/fatal/results.txt b/test/absolute-paths/fatal/results.txt
@@ -1,4 +1,4 @@
-]50;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
+]1337;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
   [31m✘[39m  [4m[90mhttp://eslint.org/docs/rules/[39m[24m  Unexpected token {  
   [4m[90m/home/usr/roy/index.js:3:13[39m[24m
   
diff --git a/test/absolute-paths/missing-line-column/results.txt b/test/absolute-paths/missing-line-column/results.txt
@@ -1,4 +1,4 @@
-]50;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
+]1337;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
   [33m⚠[39m  [4m[90mhttp://eslint.org/docs/rules/[39m[24m                 path is defined but never used                    
   [4m[90m/home/usr/roy/index.js:0:0[39m[24m
   [90m  path = require( 'path' ),[39m
diff --git a/test/absolute-paths/missing-messages/results.txt b/test/absolute-paths/missing-messages/results.txt
@@ -1,4 +1,4 @@
-]50;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
+]1337;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
   [31m✘[39m  [4m[90mhttp://eslint.org/docs/rules/[37mcurly[90m[39m[24m            Expected { after 'if' condition                   
   [4m[90m/home/usr/roy/test/runner.js:41:2[39m[24m
   [90m  if ( err ) console.log( 'failed tests: ' + err );[39m
diff --git a/test/absolute-paths/missing-rule-id/results.txt b/test/absolute-paths/missing-rule-id/results.txt
@@ -1,4 +1,4 @@
-]50;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
+]1337;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
   [33m⚠[39m  [4m[90mhttp://eslint.org/docs/rules/[39m[24m                 path is defined but never used                    
   [4m[90m/home/usr/roy/index.js:8:2[39m[24m
   [90m  path = require( 'path' ),[39m
diff --git a/test/absolute-paths/single-error/results.txt b/test/absolute-paths/single-error/results.txt
@@ -1,4 +1,4 @@
-]50;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
+]1337;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
   [33m⚠[39m  [4m[90mhttp://eslint.org/docs/rules/[37mno-unused-vars[90m[39m[24m   path is defined but never used                    
   [4m[90m/home/usr/roy/index.js:8:2[39m[24m
   [90m  path = require( 'path' ),[39m
diff --git a/test/filter/single-error/results.txt b/test/filter/single-error/results.txt
@@ -1,4 +1,4 @@
-]50;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
+]1337;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
   [33m⚠[39m  [4m[90mhttp://eslint.org/docs/rules/[37mno-unused-vars[90m[39m[24m  path is defined but never used  
   [4m[90mindex.js:8:2[39m[24m
   [90m  path = require( 'path' ),[39m
diff --git a/test/fixtures/basic/results.txt b/test/fixtures/basic/results.txt
@@ -1,4 +1,4 @@
-]50;CurrentDir=/path/to/working/directory
+]1337;CurrentDir=/path/to/working/directory
   [33m⚠[39m  [4m[90mhttp://eslint.org/docs/rules/[37mno-unused-vars[90m[39m[24m   path is defined but never used                    
   [4m[90mindex.js:8:2[39m[24m
   [90m  path = require( 'path' ),[39m
diff --git a/test/fixtures/fatal/results.txt b/test/fixtures/fatal/results.txt
@@ -1,4 +1,4 @@
-]50;CurrentDir=/path/to/working/directory
+]1337;CurrentDir=/path/to/working/directory
   [31m✘[39m  [4m[90mhttp://eslint.org/docs/rules/[39m[24m  Unexpected token {  
   [4m[90mindex.js:3:13[39m[24m
   
diff --git a/test/fixtures/missing-line-column/results.txt b/test/fixtures/missing-line-column/results.txt
@@ -1,4 +1,4 @@
-]50;CurrentDir=/path/to/working/directory
+]1337;CurrentDir=/path/to/working/directory
   [33m⚠[39m  [4m[90mhttp://eslint.org/docs/rules/[39m[24m                 path is defined but never used                    
   [4m[90mindex.js:0:0[39m[24m
   [90m  path = require( 'path' ),[39m
diff --git a/test/fixtures/missing-messages/results.txt b/test/fixtures/missing-messages/results.txt
@@ -1,4 +1,4 @@
-]50;CurrentDir=/path/to/working/directory
+]1337;CurrentDir=/path/to/working/directory
   [31m✘[39m  [4m[90mhttp://eslint.org/docs/rules/[37mcurly[90m[39m[24m            Expected { after 'if' condition                   
   [4m[90mtest/runner.js:41:2[39m[24m
   [90m  if ( err ) console.log( 'failed tests: ' + err );[39m
diff --git a/test/fixtures/missing-rule-id/results.txt b/test/fixtures/missing-rule-id/results.txt
@@ -1,4 +1,4 @@
-]50;CurrentDir=/path/to/working/directory
+]1337;CurrentDir=/path/to/working/directory
   [33m⚠[39m  [4m[90mhttp://eslint.org/docs/rules/[39m[24m                 path is defined but never used                    
   [4m[90mindex.js:8:2[39m[24m
   [90m  path = require( 'path' ),[39m
diff --git a/test/fixtures/single-error/results.txt b/test/fixtures/single-error/results.txt
@@ -1,4 +1,4 @@
-]50;CurrentDir=/path/to/working/directory
+]1337;CurrentDir=/path/to/working/directory
   [33m⚠[39m  [4m[90mhttp://eslint.org/docs/rules/[37mno-unused-vars[90m[39m[24m   path is defined but never used                    
   [4m[90mindex.js:8:2[39m[24m
   [90m  path = require( 'path' ),[39m
diff --git a/test/no-gray-fixtures/basic/results.txt b/test/no-gray-fixtures/basic/results.txt
@@ -1,4 +1,4 @@
-]50;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
+]1337;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
   [33m⚠[39m  [4mhttp://eslint.org/docs/rules/[37mno-unused-vars[39m[24m   path is defined but never used                    
   [4mindex.js:8:2[24m
     path = require( 'path' ),
diff --git a/test/no-gray-fixtures/fatal/results.txt b/test/no-gray-fixtures/fatal/results.txt
@@ -1,4 +1,4 @@
-]50;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
+]1337;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
   [31m✘[39m  [4mhttp://eslint.org/docs/rules/[24m  Unexpected token {  
   [4mindex.js:3:13[24m
   
diff --git a/test/no-gray-fixtures/missing-line-column/results.txt b/test/no-gray-fixtures/missing-line-column/results.txt
@@ -1,4 +1,4 @@
-]50;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
+]1337;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
   [33m⚠[39m  [4mhttp://eslint.org/docs/rules/[24m                 path is defined but never used                    
   [4mindex.js:0:0[24m
     path = require( 'path' ),
diff --git a/test/no-gray-fixtures/missing-messages/results.txt b/test/no-gray-fixtures/missing-messages/results.txt
@@ -1,4 +1,4 @@
-]50;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
+]1337;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
   [31m✘[39m  [4mhttp://eslint.org/docs/rules/[37mcurly[39m[24m            Expected { after 'if' condition                   
   [4mtest/runner.js:41:2[24m
     if ( err ) console.log( 'failed tests: ' + err );
diff --git a/test/no-gray-fixtures/missing-rule-id/results.txt b/test/no-gray-fixtures/missing-rule-id/results.txt
@@ -1,4 +1,4 @@
-]50;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
+]1337;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
   [33m⚠[39m  [4mhttp://eslint.org/docs/rules/[24m                 path is defined but never used                    
   [4mindex.js:8:2[24m
     path = require( 'path' ),
diff --git a/test/no-gray-fixtures/single-error/results.txt b/test/no-gray-fixtures/single-error/results.txt
@@ -1,4 +1,4 @@
-]50;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
+]1337;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
   [33m⚠[39m  [4mhttp://eslint.org/docs/rules/[37mno-unused-vars[39m[24m   path is defined but never used                    
   [4mindex.js:8:2[24m
     path = require( 'path' ),
diff --git a/test/no-links/basic/results.txt b/test/no-links/basic/results.txt
@@ -1,4 +1,4 @@
-]50;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
+]1337;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
   [33m⚠[39m  [37mno-unused-vars[39m   path is defined but never used                    
   [4m[90mindex.js:8:2[39m[24m
   [90m  path = require( 'path' ),[39m
diff --git a/test/no-links/fatal/results.txt b/test/no-links/fatal/results.txt
@@ -1,4 +1,4 @@
-]50;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
+]1337;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
   [31m✘[39m    Unexpected token {  
   [4m[90mindex.js:3:13[39m[24m
   
diff --git a/test/no-links/missing-line-column/results.txt b/test/no-links/missing-line-column/results.txt
@@ -1,4 +1,4 @@
-]50;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
+]1337;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
   [33m⚠[39m                   path is defined but never used                    
   [4m[90mindex.js:0:0[39m[24m
   [90m  path = require( 'path' ),[39m
diff --git a/test/no-links/missing-messages/results.txt b/test/no-links/missing-messages/results.txt
@@ -1,4 +1,4 @@
-]50;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
+]1337;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
   [31m✘[39m  [37mcurly[39m            Expected { after 'if' condition                   
   [4m[90mtest/runner.js:41:2[39m[24m
   [90m  if ( err ) console.log( 'failed tests: ' + err );[39m
diff --git a/test/no-links/missing-rule-id/results.txt b/test/no-links/missing-rule-id/results.txt
@@ -1,4 +1,4 @@
-]50;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
+]1337;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
   [33m⚠[39m                   path is defined but never used                    
   [4m[90mindex.js:8:2[39m[24m
   [90m  path = require( 'path' ),[39m
diff --git a/test/no-links/single-error/results.txt b/test/no-links/single-error/results.txt
@@ -1,4 +1,4 @@
-]50;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
+]1337;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
   [33m⚠[39m  [37mno-unused-vars[39m   path is defined but never used                    
   [4m[90mindex.js:8:2[39m[24m
   [90m  path = require( 'path' ),[39m
diff --git a/test/plugin-rules/single-error/results.txt b/test/plugin-rules/single-error/results.txt
@@ -1,4 +1,4 @@
-]50;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
+]1337;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
   [33m⚠[39m  [4m[90mhttp://eslint.org/docs/rules/[37mno-unused-vars[90m[39m[24m    path is defined but never used                    
   [4m[90mindex.js:8:2[39m[24m
   [90m  path = require( 'path' ),[39m
diff --git a/test/scheme/basic/results.txt b/test/scheme/basic/results.txt
@@ -1,4 +1,4 @@
-]50;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
+]1337;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
   [33m⚠[39m  [4m[90mhttp://eslint.org/docs/rules/[37mno-unused-vars[90m[39m[24m   path is defined but never used                    
   [90m/home/usr/roy/index.js:8:2[39m
   [4m[90meditor://open?file=%2Fhome%2Fusr%2Froy%2Findex.js&line=8&column=2[39m[24m
diff --git a/test/scheme/fatal/results.txt b/test/scheme/fatal/results.txt
@@ -1,4 +1,4 @@
-]50;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
+]1337;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
   [31m✘[39m  [4m[90mhttp://eslint.org/docs/rules/[39m[24m  Unexpected token {  
   [90m/home/usr/roy/index.js:3:13[39m
   [4m[90meditor://open?file=%2Fhome%2Fusr%2Froy%2Findex.js&line=3&column=13[39m[24m
diff --git a/test/scheme/missing-line-column/results.txt b/test/scheme/missing-line-column/results.txt
@@ -1,4 +1,4 @@
-]50;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
+]1337;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
   [33m⚠[39m  [4m[90mhttp://eslint.org/docs/rules/[39m[24m                 path is defined but never used                    
   [90m/home/usr/roy/index.js:0:0[39m
   [4m[90meditor://open?file=%2Fhome%2Fusr%2Froy%2Findex.js&line=0&column=0[39m[24m
diff --git a/test/scheme/missing-messages/results.txt b/test/scheme/missing-messages/results.txt
@@ -1,4 +1,4 @@
-]50;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
+]1337;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
   [31m✘[39m  [4m[90mhttp://eslint.org/docs/rules/[37mcurly[90m[39m[24m            Expected { after 'if' condition                   
   [90m/home/usr/roy/test/runner.js:41:2[39m
   [4m[90meditor://open?file=%2Fhome%2Fusr%2Froy%2Ftest%2Frunner.js&line=41&column=2[39m[24m
diff --git a/test/scheme/missing-rule-id/results.txt b/test/scheme/missing-rule-id/results.txt
@@ -1,4 +1,4 @@
-]50;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
+]1337;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
   [33m⚠[39m  [4m[90mhttp://eslint.org/docs/rules/[39m[24m                 path is defined but never used                    
   [90m/home/usr/roy/index.js:8:2[39m
   [4m[90meditor://open?file=%2Fhome%2Fusr%2Froy%2Findex.js&line=8&column=2[39m[24m
diff --git a/test/scheme/single-error/results.txt b/test/scheme/single-error/results.txt
@@ -1,4 +1,4 @@
-]50;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
+]1337;CurrentDir=/Users/roy/royws/eslint-friendly-formatter
   [33m⚠[39m  [4m[90mhttp://eslint.org/docs/rules/[37mno-unused-vars[90m[39m[24m   path is defined but never used                    
   [90m/home/usr/roy/index.js:8:2[39m
   [4m[90meditor://open?file=%2Fhome%2Fusr%2Froy%2Findex.js&line=8&column=2[39m[24m
diff --git a/test/specs/index.js.snap b/test/specs/index.js.snap

Original file line number	Diff line number	Diff line change
`@@ -245,7 +245,7 @@ module.exports = function(results) {`
`245`	`245`	`}`
`246`	`246`
`247`	`247`	`if (process.env.FORCE_ITERM_HINT === 'true' \|\| (process.stdout.isTTY && !process.env.CI)) {`
`248`		`- output = '\u001B]50;CurrentDir=' + process.cwd() + '\u0007' + output;`
	`248`	`+ output = '\u001B]1337;CurrentDir=' + process.cwd() + '\u0007' + output;`
`249`	`249`	`}`
`250`	`250`
`251`	`251`	`return total > 0 ? output : '';`
Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-]50;CurrentDir=/Users/roy/royws/eslint-friendly-formatter`
	`1`	`+]1337;CurrentDir=/Users/roy/royws/eslint-friendly-formatter`
`2`	`2`	`[33m⚠[39m [4mhttp://eslint.org/docs/rules/[37mno-unused-vars[39m[24m path is defined but never used`
`3`	`3`	`[4m/home/usr/roy/index.js:8:2[24m`
`4`	`4`	`path = require( 'path' ),`