summaryrefslogtreecommitdiffhomepage
diff options
context:
space:
mode:
authorDax Raad <[email protected]>2025-06-01 14:46:04 -0400
committerDax Raad <[email protected]>2025-06-01 14:46:04 -0400
commita1c4f345a89d46dd44e7e208dc7ffd4b9bcd500f (patch)
tree9c0c486212725db98ca76eb9df2adc66da609f9d
parent526a8ea19a64c131fb0b7924f38860984244b0cc (diff)
downloadopencode-a1c4f345a89d46dd44e7e208dc7ffd4b9bcd500f.tar.gz
opencode-a1c4f345a89d46dd44e7e208dc7ffd4b9bcd500f.zip
Fix TypeScript errors and clean up unused imports
- Remove unused fs import from app.ts - Remove unused LLM import and missing module reference from server.ts - Fix parameter naming inconsistency in fetch.ts execute function - Add missing metadata property to fetch.ts return statement - Update test file to use correct API signatures and parameter names - Remove unused parameters from example.ts 🤖 Generated with opencode Co-Authored-By: opencode <[email protected]>
-rw-r--r--package.json1
-rw-r--r--packages/opencode/package.json4
-rw-r--r--packages/opencode/src/app/app.ts1
-rw-r--r--packages/opencode/src/server/server.ts1
-rw-r--r--packages/opencode/src/tool/example.ts2
-rw-r--r--packages/opencode/src/tool/fetch.ts4
-rw-r--r--packages/opencode/test/tool/tool.test.ts44
7 files changed, 20 insertions, 37 deletions
diff --git a/package.json b/package.json
index f661facf2..3a5f966db 100644
--- a/package.json
+++ b/package.json
@@ -5,6 +5,7 @@
"type": "module",
"packageManager": "[email protected]",
"scripts": {
+ "typecheck": "bun run typecheck --filter='*'",
"dev": "sst dev"
},
"workspaces": {
diff --git a/packages/opencode/package.json b/packages/opencode/package.json
index 25ee16a65..62e0adef0 100644
--- a/packages/opencode/package.json
+++ b/packages/opencode/package.json
@@ -4,7 +4,9 @@
"name": "opencode",
"type": "module",
"private": true,
- "scripts": {},
+ "scripts": {
+ "typecheck": "tsc --noEmit"
+ },
"exports": {
"./*": [
"./src/*.ts",
diff --git a/packages/opencode/src/app/app.ts b/packages/opencode/src/app/app.ts
index eae0d03b4..e6243da86 100644
--- a/packages/opencode/src/app/app.ts
+++ b/packages/opencode/src/app/app.ts
@@ -1,4 +1,3 @@
-import fs from "fs/promises"
import { Log } from "../util/log"
import { Context } from "../util/context"
import { Filesystem } from "../util/filesystem"
diff --git a/packages/opencode/src/server/server.ts b/packages/opencode/src/server/server.ts
index 5f6808419..ba940aa7d 100644
--- a/packages/opencode/src/server/server.ts
+++ b/packages/opencode/src/server/server.ts
@@ -6,7 +6,6 @@ import { streamSSE } from "hono/streaming"
import { Session } from "../session/session"
import { resolver, validator as zValidator } from "hono-openapi/zod"
import { z } from "zod"
-import { LLM } from "../llm/llm"
import { Message } from "../session/message"
import { Provider } from "../provider/provider"
diff --git a/packages/opencode/src/tool/example.ts b/packages/opencode/src/tool/example.ts
index c7c16d49d..8f2c2d21a 100644
--- a/packages/opencode/src/tool/example.ts
+++ b/packages/opencode/src/tool/example.ts
@@ -8,7 +8,7 @@ export const ExampleTool = Tool.define({
foo: z.string().describe("The foo parameter"),
bar: z.number().describe("The bar parameter"),
}),
- async execute(params) {
+ async execute() {
return {
metadata: {
lol: "hey",
diff --git a/packages/opencode/src/tool/fetch.ts b/packages/opencode/src/tool/fetch.ts
index a4abc2293..cf56b26aa 100644
--- a/packages/opencode/src/tool/fetch.ts
+++ b/packages/opencode/src/tool/fetch.ts
@@ -53,7 +53,7 @@ export const FetchTool = Tool.define({
.describe("Optional timeout in seconds (max 120)")
.optional(),
}),
- async execute(param) {
+ async execute(params) {
// Validate URL
if (
!params.url.startsWith("http://") &&
@@ -110,7 +110,7 @@ export const FetchTool = Tool.define({
const markdown = convertHTMLToMarkdown(content)
return { output: markdown, metadata: {} }
}
- return { output: "```\n" + content + "\n```" }
+ return { output: "```\n" + content + "\n```", metadata: {} }
case "html":
return { output: content, metadata: {} }
diff --git a/packages/opencode/test/tool/tool.test.ts b/packages/opencode/test/tool/tool.test.ts
index eeb681bb3..77d0b470d 100644
--- a/packages/opencode/test/tool/tool.test.ts
+++ b/packages/opencode/test/tool/tool.test.ts
@@ -1,34 +1,22 @@
import { describe, expect, test } from "bun:test"
import { App } from "../../src/app/app"
import { GlobTool } from "../../src/tool/glob"
-import { ls } from "../../src/tool/ls"
+import { ListTool } from "../../src/tool/ls"
describe("tool.glob", () => {
test("truncate", async () => {
- await App.provide({ directory: process.cwd() }, async () => {
- let result = await GlobTool.execute(
- {
- pattern: "./node_modules/**/*",
- },
- {
- toolCallId: "test",
- messages: [],
- },
- )
+ await App.provide({ cwd: process.cwd(), version: "test" }, async () => {
+ let result = await GlobTool.execute({
+ pattern: "./node_modules/**/*",
+ })
expect(result.metadata.truncated).toBe(true)
})
})
test("basic", async () => {
- await App.provide({ directory: process.cwd() }, async () => {
- let result = await GlobTool.execute(
- {
- pattern: "*.json",
- },
- {
- toolCallId: "test",
- messages: [],
- },
- )
+ await App.provide({ cwd: process.cwd(), version: "test" }, async () => {
+ let result = await GlobTool.execute({
+ pattern: "*.json",
+ })
expect(result.metadata).toMatchObject({
truncated: false,
count: 2,
@@ -39,16 +27,10 @@ describe("tool.glob", () => {
describe("tool.ls", () => {
test("basic", async () => {
- const result = await App.provide({ directory: process.cwd() }, async () => {
- return await ls.execute(
- {
- path: "./example",
- },
- {
- toolCallId: "test",
- messages: [],
- },
- )
+ const result = await App.provide({ cwd: process.cwd(), version: "test" }, async () => {
+ return await ListTool.execute({
+ path: "./example",
+ })
})
expect(result.output).toMatchSnapshot()
})