From 3c920236c50a05a1c0e1a4a9a82d092532bd6a42 Mon Sep 17 00:00:00 2001 From: Szymon Chmal Date: Thu, 30 Jul 2026 15:02:47 -0700 Subject: [PATCH 1/3] test(cli): update config-wrapper expectations for enabled option Commit 427c1515 (#108) added the `{ enabled: process.env.WITH_ROZENITE === 'true' }` option to wrapConfigFile's generated code but never updated these tests, leaving 5 assertions stale and failing. --- packages/cli/src/__tests__/config-wrapper.test.ts | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/packages/cli/src/__tests__/config-wrapper.test.ts b/packages/cli/src/__tests__/config-wrapper.test.ts index a912e784..966efdb8 100644 --- a/packages/cli/src/__tests__/config-wrapper.test.ts +++ b/packages/cli/src/__tests__/config-wrapper.test.ts @@ -261,7 +261,7 @@ describe('wrapConfigFile', () => { "const { withRozenite } = require('@rozenite/metro');" ); expect(wrappedContent).toContain( - 'withRozenite(getDefaultConfig(__dirname))' + "withRozenite(getDefaultConfig(__dirname), { enabled: process.env.WITH_ROZENITE === 'true' })" ); expect(validateJavaScript(wrappedContent)).toBe(true); }); @@ -333,7 +333,7 @@ describe('wrapConfigFile', () => { "import { withRozenite } from '@rozenite/metro';" ); expect(wrappedContent).toContain( - 'export default withRozenite(getDefaultConfig(__dirname))' + "export default withRozenite(getDefaultConfig(__dirname), { enabled: process.env.WITH_ROZENITE === 'true' })" ); expect(validateJavaScript(wrappedContent)).toBe(true); }); @@ -405,7 +405,7 @@ describe('wrapConfigFile', () => { "const { withRozenite } = require('@rozenite/repack');" ); expect(wrappedContent).toContain( - 'withRozenite(getDefaultConfig(__dirname))' + "withRozenite(getDefaultConfig(__dirname), { enabled: process.env.WITH_ROZENITE === 'true' })" ); expect(validateJavaScript(wrappedContent)).toBe(true); }); @@ -459,7 +459,7 @@ describe('wrapConfigFile', () => { "import { withRozenite } from '@rozenite/repack';" ); expect(wrappedContent).toContain( - 'export default withRozenite(getDefaultConfig(process.cwd()))' + "export default withRozenite(getDefaultConfig(process.cwd()), { enabled: process.env.WITH_ROZENITE === 'true' })" ); expect(validateJavaScript(wrappedContent)).toBe(true); }); @@ -727,7 +727,7 @@ export default getDefaultConfig(__dirname);`; "import { withRozenite } from '@rozenite/metro';" ); expect(wrappedContent).toContain( - 'export default withRozenite(getDefaultConfig(__dirname))' + "export default withRozenite(getDefaultConfig(__dirname), { enabled: process.env.WITH_ROZENITE === 'true' })" ); expect(validateJavaScript(wrappedContent)).toBe(true); }); From 99372e0a8a0764b8af7f52826fb2c3a4754b3628 Mon Sep 17 00:00:00 2001 From: Szymon Chmal Date: Thu, 30 Jul 2026 15:05:19 -0700 Subject: [PATCH 2/3] ci(cli): wire rozenite's vitest suite into the turbo test task packages/cli had no test script, so `pnpm turbo run typecheck build lint test --affected` silently skipped its suite in CI (turbo reports rozenite#test -> ). Add the same `vitest --run --passWithNoTests` script used by sibling packages; turbo.json's existing generic test task (dependsOn ^build, build) already builds workspace deps first, so no turbo.json changes needed. --- packages/cli/package.json | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/packages/cli/package.json b/packages/cli/package.json index 042f7da5..c93d36a9 100644 --- a/packages/cli/package.json +++ b/packages/cli/package.json @@ -9,7 +9,8 @@ "scripts": { "build": "vite build", "typecheck": "tsc -p tsconfig.lib.json --noEmit", - "lint": "eslint ." + "lint": "eslint .", + "test": "vitest --run --passWithNoTests" }, "files": [ "dist", From 2b0bd1c01e1dcb774c3ead37a860a5b6d7b80cf8 Mon Sep 17 00:00:00 2001 From: Szymon Chmal Date: Thu, 30 Jul 2026 16:27:28 -0700 Subject: [PATCH 3/3] test(cli): update stale agent host expectation DEFAULT_AGENT_HOST changed from localhost to 127.0.0.1 in 7b00844e ("fix: align agent websocket origin (#307)"). That commit updated the middleware tests but not this one, because packages/cli has no test script and so its suite never ran in CI. --- packages/cli/src/__tests__/agent-command-output.test.ts | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/packages/cli/src/__tests__/agent-command-output.test.ts b/packages/cli/src/__tests__/agent-command-output.test.ts index ce09cfe4..2007cf1f 100644 --- a/packages/cli/src/__tests__/agent-command-output.test.ts +++ b/packages/cli/src/__tests__/agent-command-output.test.ts @@ -91,7 +91,7 @@ describe('agent command output', () => { '{"items":[{"id":"device-1","name":"iPhone"}]}\n', ); expect(mocks.createAgentClient).toHaveBeenCalledWith({ - host: 'localhost', + host: '127.0.0.1', port: 8081, }); }); @@ -131,7 +131,7 @@ describe('agent command output', () => { mocks.transport.createSession.mockResolvedValue({ session: { id: 'device-1', - host: 'localhost', + host: '127.0.0.1', port: 8081, deviceId: 'device-1', deviceName: 'iPhone', @@ -172,7 +172,7 @@ describe('agent command output', () => { mocks.transport.createSession.mockResolvedValue({ session: { id: 'device-1', - host: 'localhost', + host: '127.0.0.1', port: 8081, deviceId: 'device-1', deviceName: 'iPhone', @@ -212,7 +212,7 @@ describe('agent command output', () => { mocks.transport.createSession.mockResolvedValue({ session: { id: 'device-1', - host: 'localhost', + host: '127.0.0.1', port: 8081, deviceId: 'device-1', deviceName: 'iPhone',