pinokiod 8.2.2 → 8.2.4

package/test/node-bin.test.js

@@ -3,7 +3,7 @@ const test = require('node:test')
 
 const Node = require('../kernel/bin/node')
 
-function createNode({ nodeVersion = '24.18.0', pnpmVersion = '11.9.0', hasNode = true, hasPnpm = true } = {}) {
+function createNode({ nodeVersion = '24.22.1', pnpmVersion = '11.9.0', hasNode = true, hasPnpm = true } = {}) {
   const node = new Node()
   const conda = new Set()
   if (hasNode) conda.add('nodejs')
@@ -23,12 +23,12 @@ function createNode({ nodeVersion = '24.18.0', pnpmVersion = '11.9.0', hasNode =
 }
 
 test('Node bin pins Node.js 24 LTS and matching pnpm', () => {
-  assert.equal(createNode().cmd(), 'nodejs=24.18.0 pnpm=11.9.0')
+  assert.equal(createNode().cmd(), 'nodejs=24.22.1 pnpm=11.9.0')
 })
 
 test('Node.installed accepts exactly the pinned Node.js and pnpm versions', async () => {
   assert.equal(await createNode().installed(), true)
-  assert.equal(await createNode({ nodeVersion: '24.18.0-h654b19f_0' }).installed(), true)
+  assert.equal(await createNode({ nodeVersion: '24.22.1-h654b19f_0' }).installed(), true)
   assert.equal(await createNode({ pnpmVersion: '11.9.0-h7c87c79_0' }).installed(), true)
   assert.equal(await createNode({ nodeVersion: '22.21.1' }).installed(), false)
   assert.equal(await createNode({ nodeVersion: '26.4.0' }).installed(), false)

package/test/shell-run-template.test.js

@@ -377,3 +377,49 @@ test('redactEnvArgs summarizes protected argv env values', () => {
     truncated: false
   })
 })
+
+
+test('new venv bootstrap failure prevents app commands even with chain:*', { skip: process.platform === 'win32' }, async t => {
+  const root = await fs.mkdtemp(path.join(os.tmpdir(), 'pinokio-venv-bootstrap-'))
+  t.after(() => fs.rm(root, { recursive: true, force: true }))
+  const kernel = createKernel(root)
+  kernel.bin = { path: (...parts) => path.join(root, 'bin', ...parts) }
+  const shell = createShell(kernel)
+  shell.shell = 'bash'
+  shell.env = {}
+  shell.conda_hook = () => ':'
+  const fakeBin = path.join(root, 'fake-bin')
+  await fs.mkdir(fakeBin)
+  await fs.writeFile(path.join(fakeBin, 'python'), '#!/bin/sh\nif [ "$1" = "-m" ]; then mkdir -p "$3/bin"; touch "$3/bin/activate"; exit 0; fi\nexit 19\n', { mode: 0o755 })
+  const params = await shell.activate({ path: root, conda: { skip: true }, venv: 'env', chain: '*', message: `touch '${root}/app-ran'` })
+  const result = require('node:child_process').spawnSync('bash', ['--noprofile', '--norc', '-c', shell.build(params)], { env: { ...process.env, PATH: fakeBin + ':' + process.env.PATH }, encoding: 'utf8' })
+  assert.notEqual(result.status, 0, result.stderr)
+  assert.equal(await fs.access(path.join(root, 'app-ran')).then(() => true, () => false), false)
+})
+
+test('existing app environments do not run the creation-only pip bootstrap', async t => {
+  const root = await fs.mkdtemp(path.join(os.tmpdir(), 'pinokio-existing-env-'))
+  t.after(() => fs.rm(root, { recursive: true, force: true }))
+  await fs.mkdir(path.join(root, 'env'))
+  const shell = createShell(createKernel(root))
+  shell.shell = 'bash'
+  const params = await shell.activate({ path: root, conda: { skip: true }, venv: 'env', message: 'echo app' })
+  assert.doesNotMatch(shell.build(params), /pip.*install/)
+})
+
+
+test('existing venv with outdated pip cannot bypass the guard on a retry', { skip: process.platform === 'win32' }, async t => {
+  const root = await fs.mkdtemp(path.join(os.tmpdir(), 'pinokio-venv-retry-'))
+  t.after(() => fs.rm(root, { recursive: true, force: true }))
+  await fs.mkdir(path.join(root, 'env/bin'), { recursive: true })
+  await fs.writeFile(path.join(root, 'env/bin/activate'), '')
+  const fakeBin = path.join(root, 'fake-bin')
+  await fs.mkdir(fakeBin)
+  await fs.writeFile(path.join(fakeBin, 'python'), '#!/bin/sh\nexit 19\n', { mode: 0o755 })
+  const shell = createShell(createKernel(root))
+  shell.shell = 'bash'
+  const params = await shell.activate({ path: root, conda: { skip: true }, venv: 'env', chain: '*', message: `touch '${root}/app-ran'` })
+  const result = require('node:child_process').spawnSync('bash', ['--noprofile', '--norc', '-c', shell.build(params)], { env: { ...process.env, PATH: fakeBin + ':' + process.env.PATH }, encoding: 'utf8' })
+  assert.notEqual(result.status, 0, result.stderr)
+  assert.equal(await fs.access(path.join(root, 'app-ran')).then(() => true, () => false), false)
+})

package/test/uv-bin.test.js

@@ -16,13 +16,13 @@ function createUV(version) {
   return uv
 }
 
-test('UV bin pins uv to 0.11.23', () => {
-  assert.equal(createUV('0.11.23').cmd(), 'uv=0.11.23')
+test('UV bin pins uv to 0.13.4', () => {
+  assert.equal(createUV('0.13.4').cmd(), 'uv=0.13.4')
 })
 
-test('UV.installed accepts exactly uv 0.11.23', async () => {
-  assert.equal(await createUV('0.11.23').installed(), true)
-  assert.equal(await createUV('0.11.23-h1234567_0').installed(), true)
+test('UV.installed accepts exactly uv 0.13.4', async () => {
+  assert.equal(await createUV('0.13.4').installed(), true)
+  assert.equal(await createUV('0.13.4-h1234567_0').installed(), true)
   assert.equal(await createUV('0.11.22').installed(), false)
   assert.equal(await createUV('0.12.0').installed(), false)
   assert.equal(await createUV(null).installed(), false)