use "_.._" instead of ".._" for finder (#634)

CHANGELOG.md

@@ -8,7 +8,7 @@
 
     However, this means that the `outbase` directory is expected to contain all entry point files (even implicit entry point files from `import()` expressions). If an entry point isn't under the outbase directory then esbuild will to try to write the output file outside of the output directory, since the path of the entry point relative to `outbase` will start with `../` which is then joined to the output directory. This is unintentional. All output files are supposed to be written inside of the output directory.
 
-    This release fixes the problem by creating a directory with the name `.._` in the output directory for output file paths of entry points that are not inside the `outbase` directory. So if the previous example was bundled with an outbase directory of `temp`, the output directory will contain `out/.._/pages/home/index.js` and `out/.._/pages/about/index.js`. Doing this instead of stripping the leading `../` off the relative path is necessary to avoid collisions between different entry points with the same path suffix.
+    This release fixes the problem by creating a directory with the name `_.._` in the output directory for output file paths of entry points that are not inside the `outbase` directory. So if the previous example was bundled with an outbase directory of `temp`, the output directory will contain `out/_.._/pages/home/index.js` and `out/_.._/pages/about/index.js`. Doing this instead of stripping the leading `../` off the relative path is necessary to avoid collisions between different entry points with the same path suffix.
 
 * Minification improvements
 

internal/bundler/linker.go

@@ -2601,10 +2601,11 @@ func (c *linkerContext) computeChunks() []chunkInfo {
 					dotDotCount++
 				}
 				if dotDotCount > 0 {
-					// The use of ".._" here is somewhat arbitrary but it is unlikely to
+					// The use of "_.._" here is somewhat arbitrary but it is unlikely to
 					// collide with a folder named by a human and it works on Windows
-					// (Windows doesn't like names that end with a ".").
-					relDir = strings.Repeat(".._/", dotDotCount) + relDir[dotDotCount*3:]
+					// (Windows doesn't like names that end with a "."). And not starting
+					// with a "." means that it will not be hidden on Unix.
+					relDir = strings.Repeat("_.._/", dotDotCount) + relDir[dotDotCount*3:]
 				}
 			} else {
 				baseName = c.fs.Base(source.KeyPath.Text)

internal/bundler/snapshots/snapshots_splitting.txt

@@ -1,5 +1,5 @@
 TestBundlingFilesOutsideOfOutbase
----------- /out/.._/.._/.._/src/entry.js ----------
+---------- /out/_.._/_.._/_.._/src/entry.js ----------
 // src/entry.js
 console.log("test");