From 9ae2595bfde85d24ae0330d50f97ce0a8e2dc374 Mon Sep 17 00:00:00 2001 From: Noritaka Kobayashi Date: Mon, 30 Jun 2025 04:06:03 +0900 Subject: refactor: remove unnecessary assertion (#2579) --- packages/cli/src/config/settings.test.ts | 3 +-- packages/cli/src/ui/hooks/slashCommandProcessor.ts | 5 +---- 2 files changed, 2 insertions(+), 6 deletions(-) (limited to 'packages/cli/src') diff --git a/packages/cli/src/config/settings.test.ts b/packages/cli/src/config/settings.test.ts index efaddab3..e7565457 100644 --- a/packages/cli/src/config/settings.test.ts +++ b/packages/cli/src/config/settings.test.ts @@ -43,7 +43,6 @@ import stripJsonComments from 'strip-json-comments'; // Will be mocked separatel // These imports will get the versions from the vi.mock('./settings.js', ...) factory. import { - LoadedSettings, loadSettings, USER_SETTINGS_PATH, // This IS the mocked path. SETTINGS_DIRECTORY_NAME, // This is from the original module, but used by the mock. @@ -595,7 +594,7 @@ describe('Settings Loading and Merging', () => { describe('LoadedSettings class', () => { it('setValue should update the correct scope and recompute merged settings', () => { (mockFsExistsSync as Mock).mockReturnValue(false); - const loadedSettings = loadSettings(MOCK_WORKSPACE_DIR) as LoadedSettings; + const loadedSettings = loadSettings(MOCK_WORKSPACE_DIR); vi.mocked(fs.writeFileSync).mockImplementation(() => {}); // mkdirSync is mocked in beforeEach to return undefined, which is fine for void usage diff --git a/packages/cli/src/ui/hooks/slashCommandProcessor.ts b/packages/cli/src/ui/hooks/slashCommandProcessor.ts index 739a08ba..b7dcbdcb 100644 --- a/packages/cli/src/ui/hooks/slashCommandProcessor.ts +++ b/packages/cli/src/ui/hooks/slashCommandProcessor.ts @@ -127,10 +127,7 @@ export const useSlashCommandProcessor = ( }; } else { historyItemContent = { - type: message.type as - | MessageType.INFO - | MessageType.ERROR - | MessageType.USER, + type: message.type, text: message.content, }; } -- cgit v1.2.3