mirror of
https://github.com/NousResearch/hermes-agent.git
synced 2026-07-23 16:36:23 +00:00
The SSH modules predate the stricter lint config that landed on main (curly, no-empty, perfectionist sorting, prettier). Mechanical lint:fix + fmt pass, empty catch blocks filled with the codebase's void-0 convention, and inline no-control-regex disables on the three deliberate control-char patterns (same pattern as lib/ansi.ts).
122 lines
4.1 KiB
TypeScript
122 lines
4.1 KiB
TypeScript
import assert from 'node:assert/strict'
|
|
|
|
import { test } from 'vitest'
|
|
|
|
import {
|
|
buildWindowsInteractiveCommand,
|
|
detectRemotePlatform,
|
|
encodedPowerShell,
|
|
helperCommand,
|
|
powerShellCommand,
|
|
psLiteral,
|
|
validLock
|
|
} from './windows-remote-lifecycle'
|
|
|
|
const ownershipId = '0123456789abcdef0123456789abcdef'
|
|
|
|
function sshWith(exec) {
|
|
return { exec }
|
|
}
|
|
|
|
test('PowerShell transport uses UTF-16LE encoded commands and literal escaping', () => {
|
|
assert.equal(Buffer.from(encodedPowerShell("'ok'"), 'base64').toString('utf16le'), "'ok'")
|
|
assert.equal(psLiteral("a'b"), "'a''b'")
|
|
assert.match(powerShellCommand('Write-Output ok'), /^powershell\.exe -NoProfile -NonInteractive .* -EncodedCommand /)
|
|
})
|
|
|
|
test('platform detection preserves POSIX and falls back to Windows PowerShell', async () => {
|
|
assert.deepEqual(await detectRemotePlatform(sshWith(async () => 'Linux\nx86_64\n')), { os: 'Linux', arch: 'x86_64' })
|
|
const calls: string[] = []
|
|
|
|
const result = await detectRemotePlatform(
|
|
sshWith(async command => {
|
|
calls.push(command)
|
|
|
|
if (command.startsWith('uname ')) {
|
|
throw new Error('PowerShell does not recognize uname')
|
|
}
|
|
|
|
return JSON.stringify({
|
|
os: 'Windows',
|
|
arch: 'ARM64',
|
|
hermesHome: 'C:\\h',
|
|
hermesPath: 'C:\\h\\hermes.exe',
|
|
python: 'C:\\h\\python.exe'
|
|
})
|
|
})
|
|
)
|
|
|
|
assert.equal(result.os, 'Windows')
|
|
assert.match(calls[1], /EncodedCommand/)
|
|
})
|
|
|
|
test('platform detection surfaces transport failures as themselves, not unsupported-platform', async () => {
|
|
// A dead/unauthorized host is a connectivity verdict; only a host that answers
|
|
// neither probe is an unsupported platform.
|
|
const transportErr: any = new Error('SSH connection timed out')
|
|
transportErr.kind = 'timeout'
|
|
await assert.rejects(
|
|
detectRemotePlatform(
|
|
sshWith(async () => {
|
|
throw transportErr
|
|
})
|
|
),
|
|
(err: any) => err.kind === 'timeout'
|
|
)
|
|
// Probe genuinely failing on a reachable host still classifies unsupported,
|
|
// and carries the probe detail for diagnosis.
|
|
await assert.rejects(
|
|
detectRemotePlatform(
|
|
sshWith(async command => {
|
|
if (command.startsWith('uname ')) {
|
|
throw new Error('not recognized')
|
|
}
|
|
|
|
throw new Error('Hermes is not installed on the remote Windows host.')
|
|
})
|
|
),
|
|
(err: any) => err.kind === 'unsupported-platform' && /Hermes is not installed/.test(err.message)
|
|
)
|
|
})
|
|
|
|
test('helper command uses the fixed remote Python entry point and quotes path data', () => {
|
|
const command = helperCommand({ python: "C:\\Program Files\\Hermes's\\python.exe" }, 'inspect', [
|
|
'C:\\x y\\hermes.exe'
|
|
])
|
|
|
|
const encoded = command.split(' ').pop()!
|
|
const script = Buffer.from(encoded, 'base64').toString('utf16le')
|
|
assert.match(script, /-m' 'hermes_cli\.windows_ssh_runtime' 'inspect'/)
|
|
assert.match(script, /Hermes''s/)
|
|
assert.match(script, /C:\\x y\\hermes\.exe/)
|
|
})
|
|
|
|
test('Windows lock validation is scoped and exact', () => {
|
|
const lock = {
|
|
schemaVersion: 2,
|
|
protocolVersion: 1,
|
|
ownershipId,
|
|
spawnNonce: '0123456789abcdef',
|
|
pid: 10,
|
|
creationTimeNs: '1784219690452757504',
|
|
port: 1234,
|
|
tokenFingerprint: 'a'.repeat(32),
|
|
hermesPath: 'C:\\h\\hermes.exe',
|
|
hermesHome: 'C:\\h'
|
|
}
|
|
|
|
assert.equal(validLock(lock, ownershipId), true)
|
|
assert.equal(validLock({ ...lock, ownershipId: 'b'.repeat(32) }, ownershipId), false)
|
|
assert.equal(validLock({ ...lock, creationTimeNs: '0' }, ownershipId), false)
|
|
// port 0 = spawn-in-progress record: valid ownership proof (cleanup can act
|
|
// on it) but the reuse gate must reject it separately.
|
|
assert.equal(validLock({ ...lock, port: 0 }, ownershipId), true)
|
|
assert.equal(validLock({ ...lock, port: -1 }, ownershipId), false)
|
|
})
|
|
|
|
test('Windows integrated terminal uses encoded PowerShell and preserves cwd as literal data', () => {
|
|
const command = buildWindowsInteractiveCommand("C:\\Users\\O'Brien\\repo")
|
|
const script = Buffer.from(command.split(' ').pop()!, 'base64').toString('utf16le')
|
|
assert.match(script, /Set-Location -LiteralPath 'C:\\Users\\O''Brien\\repo'/)
|
|
assert.match(script, /powershell\.exe -NoLogo/)
|
|
})
|