6c47ab5 · hyper-identity

hyper-identity commits / 6c47ab5

test harness: run green under bare (not just node)

The suite was node-only — test:bare died on raw 'fs'/'os' imports, a
bare-only 'bare-path' source import, an undefined global `process`, and a
node-only `delete process.env`. None of this was caught because every test
ran on node while production runs on bare. Fixes, all via the #node:* bridge:

- lib/identity-manager.js: bare-path -> #node:path (was bare-only; broke node)
- test files: raw fs/os/path -> #node:*; add `import process from #node:process`;
  helpers.js crypto -> hypercore-crypto (repo convention)
- ssh-scan.test.js: `delete process.env.HOME` -> assign '' (bare-process env is
  a Proxy that rejects delete; source treats falsy HOME as unset either way)
- package.json: declare bare-os, add #node:os to the imports map

Result: 95/95 tests, 261/261 asserts under BOTH node and bare. This also
validates the prior createUser/view-migration commits under bare.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

6c47ab5b4049f2a2325ada2615a7d91099b58f74

by plankatron <fleeky@gmail.com> · 2026-06-04T15:22:20.000Z

Changed files (6) +20 -14

lib/identity-manager.js +1 -1

…	…	@@ -2,7 +2,7 @@ import process from '#node:process'
2	2	import crypto from 'hypercore-crypto'
3	3	import sodium from 'sodium-native'
4	4	import b4a from 'b4a'
5		import { join } from 'bare-path'
	5	import { join } from '#node:path'
6	6	import { SecureStorage } from './secure-storage.js'
7	7
8	8	/**

package.json +2 -0

…	…	@@ -33,6 +33,7 @@
33	33	"b4a": "^1.6.4",
34	34	"bare-fs": "^4.0.0",
35	35	"bare-net": "^1.0.0",
	36	"bare-os": "^3.9.1",
36	37	"bare-path": "^3.0.0",
37	38	"bare-process": "^4.0.0",
38	39	"bip39-mnemonic": "^2.5.0",
…	…	@@ -57,6 +58,7 @@
57	58	"#node:fs": { "bare": "bare-fs", "default": "fs" },
58	59	"#node:fs/promises": { "bare": "bare-fs/promises", "default": "fs/promises" },
59	60	"#node:path": { "bare": "bare-path", "default": "path" },
	61	"#node:os": { "bare": "bare-os", "default": "os" },
60	62	"#node:net": { "bare": "bare-net", "default": "net" },
61	63	"#node:process": { "bare": "bare-process", "default": "process" }
62	64	},

test/helpers.js +3 -3

…	…	@@ -1,9 +1,9 @@
1	1	import Corestore from 'corestore'
2	2	import Hyperbee from 'hyperbee'
3	3	import RAM from 'random-access-memory'
4		import path from 'path'
5		import os from 'os'
6		import crypto from 'crypto'
	4	import path from '#node:path'
	5	import os from '#node:os'
	6	import crypto from 'hypercore-crypto'
7	7
8	8	/**
9	9	* Create an in-memory Corestore + Hyperbee for testing

test/hyperident-vault.test.js +3 -3

…	…	@@ -7,9 +7,9 @@
7	7	*/
8	8
9	9	import test from 'brittle'
10		import fs from 'fs'
11		import path from 'path'
12		import os from 'os'
	10	import fs from '#node:fs'
	11	import path from '#node:path'
	12	import os from '#node:os'
13	13	import b4a from 'b4a'
14	14	import IdentityKey from 'keet-identity-key'
15	15	import crypto from 'hypercore-crypto'

test/ssh-auto-import.test.js +4 -3

…	…	@@ -1,7 +1,8 @@
1	1	import test from 'brittle'
2		import { mkdtempSync, mkdirSync, writeFileSync } from 'fs'
3		import { tmpdir } from 'os'
4		import { join } from 'path'
	2	import { mkdtempSync, mkdirSync, writeFileSync } from '#node:fs'
	3	import { tmpdir } from '#node:os'
	4	import { join } from '#node:path'
	5	import process from '#node:process'
5	6
6	7	import { autoImportLocalSSHKeys } from '../lib/auth/ssh-auto-import.js'
7	8

test/ssh-scan.test.js +7 -4

…	…	@@ -1,7 +1,8 @@
1	1	import test from 'brittle'
2		import { mkdtempSync, writeFileSync, mkdirSync, symlinkSync, chmodSync } from 'fs'
3		import { tmpdir } from 'os'
4		import { join } from 'path'
	2	import { mkdtempSync, writeFileSync, mkdirSync, symlinkSync, chmodSync } from '#node:fs'
	3	import { tmpdir } from '#node:os'
	4	import { join } from '#node:path'
	5	import process from '#node:process'
5	6
6	7	import { scanSSHKeys, scanHomeSSHKeys } from '../lib/auth/ssh-scan.js'
7	8	import { SSHKeyParser } from '../lib/auth/ssh-parser.js'
…	…	@@ -64,8 +65,10 @@ test('SSHKeyParser.fingerprint is deterministic and matches ssh-keygen format',
64	65	})
65	66
66	67	test('scanHomeSSHKeys returns [] without $HOME', async (t) => {
	68	// Cross-runtime: bare-process's env is a Proxy that rejects `delete`,
	69	// so blank it instead (the source treats falsy HOME the same as unset).
67	70	const origHome = process.env.HOME
68		delete process.env.HOME
	71	process.env.HOME = ''
69	72	t.teardown(() => { if (origHome) process.env.HOME = origHome })
70	73
71	74	const result = await scanHomeSSHKeys()