pinokiod 8.2.11 → 8.2.13

package/kernel/anywhere/cli.js

@@ -59,12 +59,14 @@ class TailscaleCLI {
     }
   }
   setPrefs(hostname) {
-    return this.run(['set', '--hostname=' + hostname, '--accept-dns=false', '--accept-routes=false', '--exit-node='])
+    return this.run(['set', '--hostname=' + hostname, '--accept-dns=false', '--accept-routes=false', '--exit-node=',
+      ...(process.platform === 'win32' ? ['--unattended=true'] : [])])
   }
   up(onUpdate, force = false, hostname = 'pinokio') {
     let emitted = 0
     return this.run(['up', '--json', '--timeout=5m', '--hostname=' + hostname,
-      '--accept-dns=false', '--accept-routes=false', '--exit-node=', ...(force ? ['--force-reauth'] : [])], {
+      '--accept-dns=false', '--accept-routes=false', '--exit-node=',
+      ...(process.platform === 'win32' ? ['--unattended=true'] : []), ...(force ? ['--force-reauth'] : [])], {
       timeout: 305000,
       onOutput(output) {
         // The pinned CLI emits pretty-printed top-level JSON objects.

package/kernel/anywhere/index.js

@@ -686,7 +686,7 @@ class Anywhere {
     if (!enabled) {
       return Object.assign(base, { phase: 'off', running: false })
     }
-    if (this.starting) {
+    if (this.starting || !this.api) {
       return Object.assign(base, { phase: 'starting', running: false })
     }
     const owned = await this.assertOwnership()

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pinokiod",
-  "version": "8.2.11",
+  "version": "8.2.13",
   "description": "",
   "main": "index.js",
   "scripts": {

package/test/anywhere.test.js

@@ -577,6 +577,18 @@ test('unverified daemons cannot receive logout, login, approval or Serve changes
   }
 })
 
+test('status reports starting before autostart begins', async () => {
+  const anywhere = new Anywhere(fakeKernel(os.tmpdir()))
+  anywhere.config = { enabled: true }
+  anywhere.readPid = () => assert.fail('ownership must wait for autostart')
+  assert.equal(anywhere.starting, null)
+  const snapshot = await anywhere.snapshot()
+  assert.equal(snapshot.phase, 'starting')
+  assert.equal(snapshot.running, false)
+  assert.equal(snapshot.error, null)
+  assert.equal(anywhere.api, null, 'polling must not create a client before autostart')
+})
+
 test('status cannot control an unowned daemon while autostart is checking installation', async t => {
   const tmp = fs.mkdtempSync(path.join(os.tmpdir(), 'pk-autostart-owner-'))
   t.after(() => fs.rmSync(tmp, { recursive: true, force: true }))

package/test/tailscale-cli.test.js

@@ -10,6 +10,28 @@ const TailscaleCLI = require('../kernel/anywhere/cli')
 const Anywhere = require('../kernel/anywhere')
 const Tailscaled = require('../kernel/bin/tailscaled')
 
+test('Windows startup and login retain the profile after the CLI exits', async () => {
+  const source = fs.readFileSync(require.resolve('../kernel/anywhere/cli'), 'utf8')
+  const cliRequire = require('node:module').createRequire(require.resolve('../kernel/anywhere/cli'))
+  for (const platform of ['win32', 'darwin', 'linux']) {
+    const module = { exports: {} }
+    vm.runInNewContext(source, { module, require: cliRequire, process: { platform } })
+    const cli = new module.exports({ binPath: 'fixture', socketPath: 'fixture' })
+    const calls = []
+    cli.run = async args => { calls.push(args); return '' }
+    await cli.setPrefs('pinokio')
+    await cli.up(() => {})
+    await cli.up(() => {}, true)
+    for (const args of calls) {
+      assert.equal(args.includes('--unattended=true'), platform === 'win32', `${platform} ${args[0]}`)
+      assert.ok(args.includes('--accept-dns=false'))
+      assert.ok(args.includes('--accept-routes=false'))
+      assert.ok(args.includes('--exit-node='))
+    }
+    assert.ok(calls[2].includes('--force-reauth'))
+  }
+})
+
 test('CLI login delivers each streamed update once and rejects output-handler failures', async () => {
   const source = fs.readFileSync(require.resolve('../kernel/anywhere/cli'), 'utf8')
   const cliRequire = require('node:module').createRequire(require.resolve('../kernel/anywhere/cli'))
@@ -17,7 +39,7 @@ test('CLI login delivers each streamed update once and rejects output-handler fa
     let finish, killed = false
     const child = { stdout: new EventEmitter(), kill() { killed = true } }
     const module = { exports: {} }
-    vm.runInNewContext(source, { module, require(name) {
+    vm.runInNewContext(source, { module, process, require(name) {
       if (name !== 'child_process') return cliRequire(name)
       return { execFile(file, args, options, callback) { finish = callback; return child } }
     } })