Skip to content

Commit 38db941

Browse files
committed
fix(core): final formatting and logic fix
1 parent e9abd47 commit 38db941

51 files changed

Lines changed: 208 additions & 289 deletions

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

docs/ide-integration/ide-companion-spec.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -56,6 +56,7 @@ creating a "discovery file."
5656
}
5757
}
5858
```
59+
5960
- `port` (number, required): The port of the MCP server.
6061
- `workspacePath` (string, required): A list of all open workspace root paths,
6162
delimited by the OS-specific path separator (`:` for Linux/macOS, `;` for

evals/auto_memory_modes.eval.ts

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -329,9 +329,8 @@ async function expectSeedSessionEligible(
329329
fixture: Fixture,
330330
sessionId: string,
331331
): Promise<void> {
332-
const { buildSessionIndex } = await import(
333-
'../packages/core/src/services/memoryService.js'
334-
);
332+
const { buildSessionIndex } =
333+
await import('../packages/core/src/services/memoryService.js');
335334
const { newSessionIds } = await buildSessionIndex(
336335
path.join(fixture.projectTempDir, 'chats'),
337336
{ runs: [] },
@@ -386,9 +385,8 @@ describe('Auto Memory inbox routing', () => {
386385
autoMemoryEval(
387386
'every memory patch lands in .inbox/<kind>/ for review and active files stay untouched',
388387
async () => {
389-
const { startMemoryService } = await import(
390-
'../packages/core/src/services/memoryService.js'
391-
);
388+
const { startMemoryService } =
389+
await import('../packages/core/src/services/memoryService.js');
392390
const fixture = await createFixture();
393391
evalState.sessionFilePath = await seedSession(
394392
fixture,

packages/cli/index.ts

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -141,9 +141,8 @@ async function run() {
141141
// --- Heavy Child Process ---
142142
// Now we can safely import everything.
143143
const { main } = await import('./src/gemini.js');
144-
const { FatalError, writeToStderr } = await import(
145-
'@google/gemini-cli-core'
146-
);
144+
const { FatalError, writeToStderr } =
145+
await import('@google/gemini-cli-core');
147146
const { runExitCleanup } = await import('./src/utils/cleanup.js');
148147

149148
main().catch(async (error: unknown) => {

packages/cli/src/acp/acpSession.test.ts

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -566,9 +566,8 @@ describe('Session', () => {
566566
});
567567

568568
it('should send sessionUpdate when approval mode changes', async () => {
569-
const { coreEvents, CoreEvent, ApprovalMode } = await import(
570-
'@google/gemini-cli-core'
571-
);
569+
const { coreEvents, CoreEvent, ApprovalMode } =
570+
await import('@google/gemini-cli-core');
572571

573572
coreEvents.emit(CoreEvent.ApprovalModeChanged, {
574573
sessionId: 'session-1',

packages/cli/src/commands/extensions/link.test.ts

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -20,9 +20,8 @@ import { ExtensionManager } from '../../config/extension-manager.js';
2020
import { loadSettings, type LoadedSettings } from '../../config/settings.js';
2121

2222
vi.mock('@google/gemini-cli-core', async (importOriginal) => {
23-
const { mockCoreDebugLogger } = await import(
24-
'../../test-utils/mockDebugLogger.js'
25-
);
23+
const { mockCoreDebugLogger } =
24+
await import('../../test-utils/mockDebugLogger.js');
2625
const actual =
2726
await importOriginal<typeof import('@google/gemini-cli-core')>();
2827
const mocked = mockCoreDebugLogger(actual, { stripAnsi: true });

packages/cli/src/commands/extensions/list.test.ts

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -11,9 +11,8 @@ import { ExtensionManager } from '../../config/extension-manager.js';
1111
import { loadSettings, type LoadedSettings } from '../../config/settings.js';
1212

1313
vi.mock('@google/gemini-cli-core', async (importOriginal) => {
14-
const { mockCoreDebugLogger } = await import(
15-
'../../test-utils/mockDebugLogger.js'
16-
);
14+
const { mockCoreDebugLogger } =
15+
await import('../../test-utils/mockDebugLogger.js');
1716
const actual =
1817
await importOriginal<typeof import('@google/gemini-cli-core')>();
1918
const mocked = mockCoreDebugLogger(actual, { stripAnsi: false });

packages/cli/src/commands/gemma/logs.test.ts

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -16,9 +16,8 @@ import { getLogFilePath } from './constants.js';
1616
import { logsCommand, readLastLines } from './logs.js';
1717

1818
vi.mock('@google/gemini-cli-core', async (importOriginal) => {
19-
const { mockCoreDebugLogger } = await import(
20-
'../../test-utils/mockDebugLogger.js'
21-
);
19+
const { mockCoreDebugLogger } =
20+
await import('../../test-utils/mockDebugLogger.js');
2221
return mockCoreDebugLogger(
2322
await importOriginal<typeof import('@google/gemini-cli-core')>(),
2423
{

packages/cli/src/commands/gemma/stop.test.ts

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -16,9 +16,8 @@ const mockReadServerProcessInfo = vi.hoisted(() => vi.fn());
1616
const mockResolveGemmaConfig = vi.hoisted(() => vi.fn());
1717

1818
vi.mock('@google/gemini-cli-core', async (importOriginal) => {
19-
const { mockCoreDebugLogger } = await import(
20-
'../../test-utils/mockDebugLogger.js'
21-
);
19+
const { mockCoreDebugLogger } =
20+
await import('../../test-utils/mockDebugLogger.js');
2221
return mockCoreDebugLogger(
2322
await importOriginal<typeof import('@google/gemini-cli-core')>(),
2423
{

packages/cli/src/commands/skills/disable.test.ts

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -14,9 +14,8 @@ import {
1414
} from '../../config/settings.js';
1515

1616
const { emitConsoleLog, debugLogger } = await vi.hoisted(async () => {
17-
const { createMockDebugLogger } = await import(
18-
'../../test-utils/mockDebugLogger.js'
19-
);
17+
const { createMockDebugLogger } =
18+
await import('../../test-utils/mockDebugLogger.js');
2019
return createMockDebugLogger({ stripAnsi: true });
2120
});
2221

packages/cli/src/commands/skills/enable.test.ts

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -13,9 +13,8 @@ import {
1313
} from '../../config/settings.js';
1414

1515
const { emitConsoleLog, debugLogger } = await vi.hoisted(async () => {
16-
const { createMockDebugLogger } = await import(
17-
'../../test-utils/mockDebugLogger.js'
18-
);
16+
const { createMockDebugLogger } =
17+
await import('../../test-utils/mockDebugLogger.js');
1918
return createMockDebugLogger({ stripAnsi: true });
2019
});
2120

0 commit comments

Comments
 (0)