Fix issue where Sandbox watcher would lose a specified port setting; fixes #1395

Author: ryanblock

changelog.md

@@ -2,6 +2,20 @@
 
 ---
 
+## [5.5.1] 2023-02-07
+
+### Added
+
+- Allow ctrl+d keypress to quit Sandbox
+
+
+### Fixed
+
+- Fixed issue where Sandbox watcher would lose a specified port setting; fixes #1395
+- Fixed issue where caller to Arc's CLI interface would unnecessarily execute a callback upon quitting
+
+---
+
 ## [5.5.0] 2023-02-01
 
 ### Added

src/cli/_watcher.js

@@ -36,12 +36,12 @@ module.exports = function runWatcher (args, params) {
     return
   }
 
-  function restart ({ cwd, inventory, update }) {
+  function restart () {
   // Always attempt to close the http server, but only reload if necessary
     let start = Date.now()
     sandbox.end(err => {
       if (err) update.error(err)
-      sandbox.start({ cwd, inventory, restart: true, update }, err => {
+      sandbox.start({ ...params, restart: true }, err => {
         if (err) update.error(err)
         else {
           let end = Date.now()
@@ -160,7 +160,7 @@ module.exports = function runWatcher (args, params) {
             params.inventory = inventory = result
             refreshInventory()
             update.status('Loaded latest Architect project manifest')
-            restart({ cwd, inventory, update })
+            restart()
           }
         })
       }, debounce)
@@ -185,7 +185,7 @@ module.exports = function runWatcher (args, params) {
             params.inventory = inventory = result
             refreshInventory()
             update.status('Loaded latest project preferences')
-            restart({ cwd, inventory, update })
+            restart()
           }
         })
       }, debounce)