More cleanup

Author: cte

.roo/tools/__tests__/system-time.spec.ts

@@ -8,21 +8,6 @@ const mockContext: CustomToolContext = {
 }
 
 describe("system-time tool", () => {
-	describe("definition", () => {
-		it("should have a description", () => {
-			expect(systemTime.description).toBe(
-				"Returns the current system date and time in a friendly, human-readable format.",
-			)
-		})
-
-		it("should have optional timezone parameter", () => {
-			expect(systemTime.parameters).toBeDefined()
-			const shape = systemTime.parameters!.shape
-			expect(shape.timezone).toBeDefined()
-			expect(shape.timezone.isOptional()).toBe(true)
-		})
-	})
-
 	describe("execute", () => {
 		it("should return a formatted date/time string", async () => {
 			const result = await systemTime.execute({}, mockContext)
@@ -33,26 +18,5 @@ describe("system-time tool", () => {
 			)
 			expect(result).toMatch(/\d{1,2}:\d{2}:\d{2}/)
 		})
-
-		it("should use system timezone when no timezone provided", async () => {
-			const result = await systemTime.execute({}, mockContext)
-			expect(result).toMatch(/[A-Z]{2,5}$/)
-		})
-
-		it("should format with specified timezone", async () => {
-			const result = await systemTime.execute({ timezone: "UTC" }, mockContext)
-			expect(result).toMatch(/^The current date and time is:/)
-			expect(result).toMatch(/UTC/)
-		})
-
-		it("should work with different timezone formats", async () => {
-			const result = await systemTime.execute({ timezone: "America/New_York" }, mockContext)
-			expect(result).toMatch(/^The current date and time is:/)
-			expect(result).toMatch(/(EST|EDT)/)
-		})
-
-		it("should throw error for invalid timezone", async () => {
-			await expect(systemTime.execute({ timezone: "Invalid/Timezone" }, mockContext)).rejects.toThrow()
-		})
 	})
 })

.roo/tools/system-time.ts

@@ -1,26 +1,11 @@
-import { defineCustomTool, parametersSchema } from "@roo-code/types"
+import { parametersSchema, defineCustomTool } from "@roo-code/types"
 
-/**
- * A simple custom tool that returns the current date and time in a friendly format.
- *
- * To create your own custom tools:
- * 1. Install @roo-code/types: npm install @roo-code/types
- * 2. Create a .ts file in .roo/tools/
- * 3. Export a default tool definition using defineCustomTool()
- *
- * Note that `parametersSchema` is just an alias for `z` (from zod).
- */
 export default defineCustomTool({
-	name: "system-time",
+	name: "system_time",
 	description: "Returns the current system date and time in a friendly, human-readable format.",
-	parameters: parametersSchema.object({
-		timezone: parametersSchema
-			.string()
-			.optional()
-			.describe("Optional timezone to display the time in (e.g., 'America/New_York', 'Europe/London')"),
-	}),
-	async execute(args) {
-		const options: Intl.DateTimeFormatOptions = {
+	parameters: parametersSchema.object({}),
+	async execute() {
+		const systemTime = new Date().toLocaleString("en-US", {
 			weekday: "long",
 			year: "numeric",
 			month: "long",
@@ -29,15 +14,9 @@ export default defineCustomTool({
 			minute: "2-digit",
 			second: "2-digit",
 			timeZoneName: "short",
-		}
+			timeZone: "America/Los_Angeles",
+		})
 
-		if (args.timezone) {
-			options.timeZone = args.timezone
-		}
-
-		const now = new Date()
-		const formatted = now.toLocaleString("en-US", options)
-
-		return `The current date and time is: ${formatted}`
+		return `The current date and time is: ${systemTime}`
 	},
 })

packages/core/src/custom-tools/__tests__/__snapshots__/format-native.spec.ts.snap

@@ -10,7 +10,6 @@ import cachedTool from "./fixtures/cached.js"
 import legacyTool from "./fixtures/legacy.js"
 import { toolA, toolB } from "./fixtures/multi.js"
 import { validTool as mixedValidTool } from "./fixtures/mixed.js"
-import systemTimeTool from "./fixtures/system-time.js"
 
 const fixtureTools = {
 	simple: simpleTool,
@@ -19,7 +18,6 @@ const fixtureTools = {
 	multi_toolA: toolA,
 	multi_toolB: toolB,
 	mixed_validTool: mixedValidTool,
-	systemTime: systemTimeTool,
 }
 
 describe("formatNative", () => {
@@ -244,12 +242,6 @@ describe("Native Protocol snapshots", () => {
 		expect(result).toMatchSnapshot()
 	})
 
-	it("should generate correct native definition for system time tool", () => {
-		const serialized = serializeCustomTool(fixtureTools.systemTime)
-		const result = formatNative(serialized)
-		expect(result).toMatchSnapshot()
-	})
-
 	it("should generate correct native definitions for all fixtures combined", () => {
 		const allSerialized = Object.values(fixtureTools).map(serializeCustomTool)
 		const result = allSerialized.map(formatNative)