From 90b3f4ac96b7e961f4f28bdaaa0892f7020afaa4 Mon Sep 17 00:00:00 2001 From: YeonGyu-Kim Date: Mon, 18 May 2026 21:19:48 +0900 Subject: [PATCH] test: harden workspace package assumptions Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent) Co-authored-by: Sisyphus --- src/plugin/normalize-tool-arg-schemas.test.ts | 2 +- src/shared/mock-module-lifecycle-audit.test.ts | 6 ------ 2 files changed, 1 insertion(+), 7 deletions(-) diff --git a/src/plugin/normalize-tool-arg-schemas.test.ts b/src/plugin/normalize-tool-arg-schemas.test.ts index 8a9247dda..53a340727 100644 --- a/src/plugin/normalize-tool-arg-schemas.test.ts +++ b/src/plugin/normalize-tool-arg-schemas.test.ts @@ -21,7 +21,7 @@ function getNestedRecord(record: Record, key: string): Record { const pluginPackageDirectory = dirname(Bun.resolveSync("@opencode-ai/plugin/package.json", import.meta.dir)) - const sourceZodDirectory = join(pluginPackageDirectory, "node_modules", "zod") + const sourceZodDirectory = dirname(Bun.resolveSync("zod/package.json", pluginPackageDirectory)) const tempDirectory = mkdtempSync(join(tmpdir(), "omo-host-zod-")) const copiedZodDirectory = join(tempDirectory, "zod") diff --git a/src/shared/mock-module-lifecycle-audit.test.ts b/src/shared/mock-module-lifecycle-audit.test.ts index 375727a83..3f9e00c76 100644 --- a/src/shared/mock-module-lifecycle-audit.test.ts +++ b/src/shared/mock-module-lifecycle-audit.test.ts @@ -6,11 +6,6 @@ import ts from "typescript" const SOURCE_ROOT = path.resolve(import.meta.dir, "..") const MOCK_MODULE_TOKEN = "mock.module" const MOCK_MODULE_LIFECYCLE_ALLOWLIST = new Map([ - // TODO(MOCK-MODULE-AUDIT): add cleanup for ast-grep tool module mocks. - [ - path.join(SOURCE_ROOT, "tools", "ast-grep", "tools.test.ts"), - "justification: legacy mock.module call predates audit; TODO(MOCK-MODULE-AUDIT): add cleanup", - ], // TODO(MOCK-MODULE-AUDIT): add cleanup for team mailbox inbox module mocks. [ path.join(SOURCE_ROOT, "features", "team-mode", "team-mailbox", "inbox.test.ts"), @@ -195,7 +190,6 @@ describe("mock.module lifecycle hygiene", () => { if (!contents.includes(MOCK_MODULE_TOKEN)) { continue } - const sourceFile = ts.createSourceFile(filePath, contents, ts.ScriptTarget.Latest, true) if (hasMockModuleCall(sourceFile) && !hasCleanupPattern(sourceFile)) { offenders.push(relativeSourcePath(filePath))