Added file info comments, Broke out common functions, Fixed bugs, improved dependency sort

Author: oezingle

dist/ScriptMerge built.scriptable

@@ -0,0 +1,75 @@
+// Variables used by Scriptable.
+// These must be at the very top of the file. Do not edit.
+// icon-color: deep-gray; icon-glyph: magic;
+/**
+ * FILE: Benchmark.js
+ *
+ * A dead-simple benchmarking suite
+ * 
+ * @author oezingle (oezingle@gmail.com)
+ **/
+
+const Benchmark = class {
+  constructor (title = "Benchmark") {
+    this.title = title
+    
+    this.steps = []
+  }
+  
+  step(name) {
+    const time = Date.now()
+    
+    this.steps.push({name,time})
+  }
+  
+  start () {
+    this.starting = Date.now()
+  }
+  
+  finish () {
+    const end = Date.now()
+    
+    const delta = end - this.starting
+    
+    const seconds = delta / 1000
+    
+    console.log(`${this.title} Finished `.padEnd(46, '='))  
+    console.log(`Time: ${seconds}s`)
+    
+    this.steps.forEach(({name, time}, i) => {
+      const next = this.steps[i+1]?.time ?? end
+      
+      const stepDelta = next - time
+      
+      const stepP = (stepDelta/delta) * 100
+      
+      console.log(` - ${stepP.toFixed(1).toString().padStart(4)}% ${name}`)
+    })
+    
+    console.log("Run Benchmark multiple times for best results")
+    
+    console.log('='.repeat(46))
+  }
+}
+
+importModule("shouldDemo")(module, () => {
+  let benchmark = new Benchmark()
+  
+  benchmark.start()
+  
+  benchmark.step('For')
+  for (let i = 0; i < 1000000; i++) {
+  }
+  
+  benchmark.step("While")
+  let i = 0
+  while (i < 1000000) i++
+  
+  benchmark.step("Fast")
+  
+  for (let q = 0; q < 100000; q ++) {}
+  
+  benchmark.finish()
+})
+
+module.exports = Benchmark

dist/ScriptMerge.js

@@ -2,12 +2,23 @@
 // These must be at the very top of the file. Do not edit.
 // icon-color: gray; icon-glyph: magic;
 /**
- * TODO: edge case w/ code formatted like:
+ * FILE: BracketBuffer.js
+ *
+ * Use brackets, string literals, and regex to 
+ * determine scope at a basic level. Does not
+ * play well witn comments
+ * 
+ * @author oezingle (oezingle@gmail.com)
+ **/
+
+/**
+ * BEWARE: edge case w/ code formatted like:
  * const fn = () =>
  * {
  *   ...
  * }
  * because of BracketBuffer.moveUntilLineEnd()
+ * Minify your code if it looks like this!
  **/
 
 // how on earth is regex detected

dist/ScriptMerge.scriptable

@@ -117,9 +117,11 @@ const HardenedFS = class {
   directory = HardenedFS.directory
 
   path (path) {
-    return path.replace(
-      '~', this.instance.documentsDirectory()
-    )
+    const docs = this.instance.documentsDirectory()
+    
+    return path
+      .replace(/\/[^\/]*\/\.\./g, '')
+      .replace(/^~/m, docs)
   }
 }
 
@@ -134,7 +136,8 @@ importModule("shouldDemo")(module, () => {
     Test.test(() => {fm.readString("@")}, "readString", true),
     Test.test(() => {fm.readImage("@")}, "readImage", true),
     Test.test(() => {fm.read("@")}, "read", true),  
-    Test.assert(fm.directory("/dir/file.ext"), "/dir", "Directory")
+    Test.assert(fm.directory("/dir/file.ext"), "/dir", "Directory"),
+    Test.assert(fm.path("/dir/../"), "/", "Path")
   )
 })
 

src/Benchmark.js

@@ -1,6 +1,18 @@
 // Variables used by Scriptable.
 // These must be at the very top of the file. Do not edit.
 // icon-color: purple; icon-glyph: magic;
+/**
+ * FILE: ObjectiveCProxy.js
+ *
+ * Provide Proxy-like functionality for
+ * Scriptable Objective-C native objects 
+ * (eg, Alert, QuickLook). JS's Proxy with a 
+ * native object with a getter handler (often)  
+ * results in an error.
+ * 
+ * @author oezingle (oezingle@gmail.com)
+ **/
+
 const ObjectiveCProxy = class {
   constructor (target, handler) {
     this.target = target