Skip to content

Commit 3168167

Browse files
committed
fix: resolve TypeScript errors from Maestro schema removal
- Remove MaestroConfigSchema and MaestroConfig from config exports - Comment out disabled Maestro hooks (tdd-enforcement, maestro-sisyphus-bridge) - Add stub types to disabled hook files for compilation - Remove PendingNotification from background-agent exports - Remove incorrect tool.execute.after call on background-notification hook - All TypeScript checks now pass
1 parent 567ec1a commit 3168167

File tree

5 files changed

+12
-12
lines changed

5 files changed

+12
-12
lines changed

src/config/index.ts

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@ export {
99
SisyphusAgentConfigSchema,
1010
ExperimentalConfigSchema,
1111
RalphLoopConfigSchema,
12-
MaestroConfigSchema,
1312
} from "./schema"
1413

1514
export type {
@@ -24,5 +23,4 @@ export type {
2423
ExperimentalConfig,
2524
DynamicContextPruningConfig,
2625
RalphLoopConfig,
27-
MaestroConfig,
2826
} from "./schema"
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,3 @@
11
export * from "./types"
2-
export { BackgroundManager, type PendingNotification } from "./manager"
2+
export { BackgroundManager } from "./manager"
33
export { ConcurrencyManager } from "./concurrency"

src/hooks/maestro-sisyphus-bridge/index.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,8 @@ import { existsSync, readdirSync, statSync, watch } from "node:fs"
33
import { join, basename } from "node:path"
44
import { HOOK_NAME, PLAN_READY_PROMPT, DESIGN_PHASE_CONTEXT, AUTO_EXECUTE_PROMPT } from "./constants"
55
import { log } from "../../shared/logger"
6-
import type { MaestroConfig } from "../../config/schema"
6+
7+
type MaestroConfig = { autoExecute?: boolean }
78

89
export * from "./constants"
910

src/hooks/tdd-enforcement/index.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,10 +7,11 @@
77

88
import type { PluginInput } from "@opencode-ai/plugin"
99
import { HOOK_NAME, TDD_PHASES, TDD_GATE_PROMPTS, TEST_FILE_PATTERNS, type TddPhase } from "./constants"
10-
import type { MaestroConfig } from "../../config/schema"
1110
import { readUnifiedState, updateTddState } from "../../features/boulder-state"
1211
import { log } from "../../shared/logger"
1312

13+
type MaestroConfig = { enforceTdd?: boolean; tddGates?: Record<string, boolean> }
14+
1415
export * from "./constants"
1516

1617
interface TddHookInput {

src/index.ts

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -202,13 +202,14 @@ const OhMyOpenCodePlugin: Plugin = async (ctx) => {
202202
? createSisyphusOrchestratorHook(ctx)
203203
: null;
204204

205-
const tddEnforcement = isHookEnabled("tdd-enforcement")
206-
? createTddEnforcementHook(ctx, pluginConfig.maestro)
207-
: null;
205+
// Maestro hooks disabled - removed from schema
206+
// const tddEnforcement = isHookEnabled("tdd-enforcement")
207+
// ? createTddEnforcementHook(ctx, pluginConfig.maestro)
208+
// : null;
208209

209-
const maestroSisyphusBridge = isHookEnabled("maestro-sisyphus-bridge")
210-
? createMaestroSisyphusBridgeHook(ctx, pluginConfig.maestro)
211-
: null;
210+
// const maestroSisyphusBridge = isHookEnabled("maestro-sisyphus-bridge")
211+
// ? createMaestroSisyphusBridgeHook(ctx, pluginConfig.maestro)
212+
// : null;
212213

213214
const prometheusMdOnly = isHookEnabled("prometheus-md-only")
214215
? createPrometheusMdOnlyHook(ctx)
@@ -506,7 +507,6 @@ const OhMyOpenCodePlugin: Plugin = async (ctx) => {
506507
await editErrorRecovery?.["tool.execute.after"](input, output);
507508
await sisyphusOrchestrator?.["tool.execute.after"]?.(input, output);
508509
await taskResumeInfo["tool.execute.after"](input, output);
509-
await backgroundNotificationHook?.["tool.execute.after"]?.(input, output);
510510
},
511511
};
512512
};

0 commit comments

Comments
 (0)