From 45e5317ecd4ecef3fcf0a04f88e3e9f6d160fb69 Mon Sep 17 00:00:00 2001 From: ZiyaZhang <199893125+ZiyaZhang@users.noreply.github.com> Date: Thu, 30 Jul 2026 20:52:35 -0700 Subject: [PATCH] fix(ci): stabilize merged coverage gates --- knip.json | 1 + packages/client/ui-conversation/README.i18n.yaml | 4 ++-- .../tests/workspace-context.spec.ts | 13 +------------ scripts/install-lefthook.spec.ts | 4 ++-- 4 files changed, 6 insertions(+), 16 deletions(-) diff --git a/knip.json b/knip.json index f5379c8c6e..dbd3643d13 100644 --- a/knip.json +++ b/knip.json @@ -27,6 +27,7 @@ "scripts/**/*.mjs" ], "ignoreDependencies": [ + "lefthook", "playwright" ] }, diff --git a/packages/client/ui-conversation/README.i18n.yaml b/packages/client/ui-conversation/README.i18n.yaml index 98386de138..5137e6b08a 100644 --- a/packages/client/ui-conversation/README.i18n.yaml +++ b/packages/client/ui-conversation/README.i18n.yaml @@ -2,5 +2,5 @@ # side as of the last confirmed-consistent state. Both languages carry equal authority; # after editing either side, bring the other along and re-record with: # pnpm run verify-translation-pairing --write packages/client/ui-conversation/README.md -README.md: e165d2856f24231f28f7cd9bb538d75faeeb62c7 -README.zh.md: db1f85e9e9302c37491b395d451bbc2ebf8fdd87 +README.md: 275b08ae891579a6865b8ce2185028ccb88f0a4c +README.zh.md: c8b2540ecfc6966b5b03e80ae45fa320ad1fcc23 diff --git a/packages/context/workspace-context/tests/workspace-context.spec.ts b/packages/context/workspace-context/tests/workspace-context.spec.ts index edbb4b3145..0ea161c4d2 100644 --- a/packages/context/workspace-context/tests/workspace-context.spec.ts +++ b/packages/context/workspace-context/tests/workspace-context.spec.ts @@ -847,18 +847,7 @@ describe('workspace context request injection', () => { it('mounts without requiring a filesystem provider', async () => { const ctx = new Context() try { - const outcome = await Promise.race([ - ctx.plugin(workspaceContext, { maxBytes: 65536 }).then(() => { - return 'settled' as const - }), - new Promise<'pending'>((resolve) => { - setTimeout(() => { - resolve('pending') - }, 50) - }), - ]) - - expect(outcome).toBe('settled') + await ctx.plugin(workspaceContext, { maxBytes: 65536 }) } finally { await ctx.fiber.dispose() } diff --git a/scripts/install-lefthook.spec.ts b/scripts/install-lefthook.spec.ts index d76c16bba5..0a116dea86 100644 --- a/scripts/install-lefthook.spec.ts +++ b/scripts/install-lefthook.spec.ts @@ -196,7 +196,7 @@ function runInstaller( }) } -describe('worktree-local Lefthook installer', () => { +describe('worktree-local Lefthook installer', { timeout: 15_000 }, () => { for (const [label, extraEnv] of [ ['CI', { CI: 'true' }], ['GitHub Actions', { GITHUB_ACTIONS: 'true' }], @@ -305,7 +305,7 @@ describe('worktree-local Lefthook installer', () => { expect(readFileSync(mainHookPath, 'utf8')).toBe(initialHook) expect(existsSync(join(commonDirectory(fixture), 'dsh-lefthook-install.lock'))).toBe(false) expect(existsSync(join(hooksPath(fixture, fixture.main), '.fake-lefthook-running'))).toBe(false) - }, 15_000) + }) it('waits for a concurrent installer to finish publishing its lock record', async () => { const fixture = createFixture()