bkinnightskytw
/
agent-orchestrator
mirror of https://github.com/ComposioHQ/agent-orchestrator.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

fix: resolve checkBuilt from web node_modules and remove unused imports 

Check ao-core/dist/index.js relative to webDir's node_modules instead
of using require.resolve which fails under npm link. Also removes
unused existsSync/resolve/createRequire imports that caused lint errors.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
suraj-markup 2026-03-02 02:50:56 +05:30
parent b43b1556c0
commit 62cd42b30f
2 changed files with 7 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
packages/cli/src/commands/start.ts
View File

@ -153,7 +153,7 @@ export function registerStart(program: Command): void {
if (!existsSync(resolve(webDir, "package.json"))) {
throw new Error("Could not find @composio/ao-web package. Run: pnpm install");
}
await preflight.checkBuilt();
await preflight.checkBuilt(webDir);
if (opts?.rebuild) {
await cleanNextCache(webDir);

16
packages/cli/src/lib/preflight.ts
View File

@ -9,7 +9,6 @@
import { existsSync } from "node:fs";
import { resolve } from "node:path";
import { createRequire } from "node:module";
import { isPortAvailable } from "./web-dir.js";
import { exec } from "./shell.js";
@ -28,16 +27,13 @@ async function checkPort(port: number): Promise<void> {
/**
* Check that workspace packages have been compiled (TypeScript → JavaScript).
* Verifies @composio/ao-core dist output exists, since a missing dist/ is the
* actual cause of module resolution errors when starting the dashboard.
* Works with both `next dev` and `next build` (unlike .next/BUILD_ID which
* is production-only).
* Verifies @composio/ao-core dist output exists from the web package's
* node_modules, since a missing dist/ causes module resolution errors when
* starting the dashboard. Works with both `next dev` and `next build`.
*/
async function checkBuilt(): Promise<void> {
const require = createRequire(import.meta.url);
try {
require.resolve("@composio/ao-core");
} catch {
async function checkBuilt(webDir: string): Promise<void> {
const coreEntry = resolve(webDir, "node_modules", "@composio", "ao-core", "dist", "index.js");
if (!existsSync(coreEntry)) {
throw new Error("Packages not built. Run: pnpm build");
}
}