Skip to content

Commit fadc0de

Browse files
fix(cli): honor HAPI_HOSTNAME in session metadata (#449)
1 parent 7c6a7fa commit fadc0de

2 files changed

Lines changed: 29 additions & 1 deletion

File tree

Lines changed: 28 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,28 @@
1+
import { afterEach, describe, expect, it } from 'vitest'
2+
import { buildSessionMetadata } from './sessionFactory'
3+
4+
describe('buildSessionMetadata', () => {
5+
const originalHostname = process.env.HAPI_HOSTNAME
6+
7+
afterEach(() => {
8+
if (originalHostname === undefined) {
9+
delete process.env.HAPI_HOSTNAME
10+
} else {
11+
process.env.HAPI_HOSTNAME = originalHostname
12+
}
13+
})
14+
15+
it('uses HAPI_HOSTNAME for session metadata host when provided', () => {
16+
process.env.HAPI_HOSTNAME = 'custom-session-host'
17+
18+
const metadata = buildSessionMetadata({
19+
flavor: 'codex',
20+
startedBy: 'terminal',
21+
workingDirectory: '/tmp/project',
22+
machineId: 'machine-1',
23+
now: 123
24+
})
25+
26+
expect(metadata.host).toBe('custom-session-host')
27+
})
28+
})

cli/src/agent/sessionFactory.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,7 @@ export function buildSessionMetadata(options: {
6363

6464
return {
6565
path: options.workingDirectory,
66-
host: os.hostname(),
66+
host: process.env.HAPI_HOSTNAME || os.hostname(),
6767
version: packageJson.version,
6868
os: os.platform(),
6969
machineId: options.machineId,

0 commit comments

Comments
 (0)