From f6533c0250cdbe55a31c223cbe76d7cecf469d38 Mon Sep 17 00:00:00 2001 From: nityam Date: Sat, 28 Feb 2026 02:43:15 +0530 Subject: [PATCH] fix: merge duplicate imports in sdk and test-utils packages (1/4) (#19777) --- packages/sdk/src/agent.integration.test.ts | 3 +-- packages/sdk/src/shell.ts | 7 +++++-- packages/sdk/src/skills.integration.test.ts | 3 +-- packages/sdk/src/tool.integration.test.ts | 3 +-- packages/test-utils/src/test-rig.ts | 3 +-- 5 files changed, 9 insertions(+), 10 deletions(-) diff --git a/packages/sdk/src/agent.integration.test.ts b/packages/sdk/src/agent.integration.test.ts index bd98890c52..6ae5e687f5 100644 --- a/packages/sdk/src/agent.integration.test.ts +++ b/packages/sdk/src/agent.integration.test.ts @@ -8,10 +8,9 @@ import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest'; import { GeminiCliAgent } from './agent.js'; import * as path from 'node:path'; import { fileURLToPath } from 'node:url'; -import { dirname } from 'node:path'; const __filename = fileURLToPath(import.meta.url); -const __dirname = dirname(__filename); +const __dirname = path.dirname(__filename); // Set this to true locally when you need to update snapshots const RECORD_MODE = process.env['RECORD_NEW_RESPONSES'] === 'true'; diff --git a/packages/sdk/src/shell.ts b/packages/sdk/src/shell.ts index 30b9979594..ade12c74dc 100644 --- a/packages/sdk/src/shell.ts +++ b/packages/sdk/src/shell.ts @@ -4,8 +4,11 @@ * SPDX-License-Identifier: Apache-2.0 */ -import type { Config as CoreConfig } from '@google/gemini-cli-core'; -import { ShellExecutionService, ShellTool } from '@google/gemini-cli-core'; +import { + ShellExecutionService, + ShellTool, + type Config as CoreConfig, +} from '@google/gemini-cli-core'; import type { AgentShell, AgentShellResult, diff --git a/packages/sdk/src/skills.integration.test.ts b/packages/sdk/src/skills.integration.test.ts index 48f0e0fb06..aaad9f3676 100644 --- a/packages/sdk/src/skills.integration.test.ts +++ b/packages/sdk/src/skills.integration.test.ts @@ -9,10 +9,9 @@ import { GeminiCliAgent } from './agent.js'; import { skillDir } from './skills.js'; import * as path from 'node:path'; import { fileURLToPath } from 'node:url'; -import { dirname } from 'node:path'; const __filename = fileURLToPath(import.meta.url); -const __dirname = dirname(__filename); +const __dirname = path.dirname(__filename); // Set this to true locally when you need to update snapshots const RECORD_MODE = process.env['RECORD_NEW_RESPONSES'] === 'true'; diff --git a/packages/sdk/src/tool.integration.test.ts b/packages/sdk/src/tool.integration.test.ts index cc0a7c2454..28c01c3ca2 100644 --- a/packages/sdk/src/tool.integration.test.ts +++ b/packages/sdk/src/tool.integration.test.ts @@ -10,10 +10,9 @@ import * as path from 'node:path'; import { z } from 'zod'; import { tool, ModelVisibleError } from './tool.js'; import { fileURLToPath } from 'node:url'; -import { dirname } from 'node:path'; const __filename = fileURLToPath(import.meta.url); -const __dirname = dirname(__filename); +const __dirname = path.dirname(__filename); // Set this to true locally when you need to update snapshots const RECORD_MODE = process.env['RECORD_NEW_RESPONSES'] === 'true'; diff --git a/packages/test-utils/src/test-rig.ts b/packages/test-utils/src/test-rig.ts index 1cd55b84f7..5026a47d7b 100644 --- a/packages/test-utils/src/test-rig.ts +++ b/packages/test-utils/src/test-rig.ts @@ -6,13 +6,12 @@ import { expect } from 'vitest'; import { execSync, spawn, type ChildProcess } from 'node:child_process'; -import { mkdirSync, writeFileSync, readFileSync } from 'node:fs'; +import fs, { mkdirSync, writeFileSync, readFileSync } from 'node:fs'; import { join, dirname } from 'node:path'; import { fileURLToPath } from 'node:url'; import { env } from 'node:process'; import { setTimeout as sleep } from 'node:timers/promises'; import { DEFAULT_GEMINI_MODEL, GEMINI_DIR } from '@google/gemini-cli-core'; -import fs from 'node:fs'; import * as pty from '@lydell/node-pty'; import stripAnsi from 'strip-ansi'; import * as os from 'node:os';