@chorus-aidlc/chorus-openclaw-plugin 0.2.2 → 0.2.4

package/openclaw.plugin.json

@@ -2,7 +2,6 @@
   "id": "chorus-openclaw-plugin",
   "configSchema": {
     "type": "object",
-    "required": ["chorusUrl", "apiKey"],
     "additionalProperties": false,
     "properties": {
       "chorusUrl": {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@chorus-aidlc/chorus-openclaw-plugin",
-  "version": "0.2.2",
+  "version": "0.2.4",
   "description": "OpenClaw plugin for Chorus AI-DLC collaboration platform — SSE real-time events + MCP tool integration",
   "license": "MIT",
   "type": "module",

package/src/config.ts

@@ -1,13 +1,18 @@
 import { z } from "zod";
 
+export const CONFIG_FILE_PATH = "~/.openclaw/openclaw.json";
+export const CONFIG_KEY_PATH = "plugins.entries.chorus-openclaw-plugin.config";
+
 export const chorusConfigSchema = z.object({
   chorusUrl: z
     .string()
     .url()
+    .optional()
     .describe("Chorus server URL (e.g. https://chorus.example.com)"),
   apiKey: z
     .string()
     .startsWith("cho_")
+    .optional()
     .describe("Chorus API Key (cho_ prefix)"),
   projectUuids: z
     .array(z.string().uuid())
@@ -22,3 +27,30 @@ export const chorusConfigSchema = z.object({
 });
 
 export type ChorusPluginConfig = z.infer<typeof chorusConfigSchema>;
+
+/**
+ * Check required config fields and warn about missing ones.
+ * Returns true if all required fields are present, false otherwise.
+ */
+export function validateConfigWithWarnings(
+  config: ChorusPluginConfig,
+  logger: { warn: (msg: string) => void },
+): boolean {
+  const missing: string[] = [];
+
+  if (!config.chorusUrl) {
+    missing.push(`  - "chorusUrl": set at ${CONFIG_KEY_PATH}.chorusUrl in ${CONFIG_FILE_PATH}`);
+  }
+  if (!config.apiKey) {
+    missing.push(`  - "apiKey": set at ${CONFIG_KEY_PATH}.apiKey in ${CONFIG_FILE_PATH}`);
+  }
+
+  if (missing.length > 0) {
+    logger.warn(
+      `[Chorus] Plugin is missing required configuration. Features will be disabled until configured:\n` +
+      missing.join("\n")
+    );
+    return false;
+  }
+  return true;
+}

package/src/index.ts

@@ -1,7 +1,7 @@
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
 type OpenClawPluginApi = any;
 
-import { chorusConfigSchema, type ChorusPluginConfig } from "./config.js";
+import { chorusConfigSchema, type ChorusPluginConfig, validateConfigWithWarnings } from "./config.js";
 import { ChorusMcpClient } from "./mcp-client.js";
 import { ChorusSseListener } from "./sse-listener.js";
 import { ChorusEventRouter } from "./event-router.js";
@@ -51,31 +51,34 @@ const plugin = {
   register(api: OpenClawPluginApi) {
     const rawConfig = api.pluginConfig ?? {};
     const config: ChorusPluginConfig = {
-      chorusUrl: rawConfig.chorusUrl ?? "",
-      apiKey: rawConfig.apiKey ?? "",
+      chorusUrl: rawConfig.chorusUrl || undefined,
+      apiKey: rawConfig.apiKey || undefined,
       projectUuids: rawConfig.projectUuids ?? [],
       autoStart: rawConfig.autoStart ?? true,
     };
     const logger = api.logger;
 
-    if (!config.chorusUrl || !config.apiKey) {
-      logger.error("Chorus plugin missing required config: chorusUrl and apiKey");
+    if (!validateConfigWithWarnings(config, logger)) {
       return;
     }
 
+    // After validateConfigWithWarnings, chorusUrl and apiKey are guaranteed present
+    const chorusUrl = config.chorusUrl!;
+    const apiKey = config.apiKey!;
+
     // Resolve gateway URL and hooks token from OpenClaw config
     const gatewayPort = api.config?.gateway?.port ?? 18789;
     const gatewayUrl = `http://127.0.0.1:${gatewayPort}`;
     const hooksToken = api.config?.hooks?.token ?? "";
 
     logger.info(
-      `Chorus plugin initializing — ${config.chorusUrl} (${config.projectUuids?.length || "all"} projects)`
+      `Chorus plugin initializing — ${chorusUrl} (${config.projectUuids?.length || "all"} projects)`
     );
 
     // --- MCP Client ---
     const mcpClient = new ChorusMcpClient({
-      chorusUrl: config.chorusUrl,
-      apiKey: config.apiKey,
+      chorusUrl,
+      apiKey,
       logger,
     });
 
@@ -103,8 +106,8 @@ const plugin = {
       id: "chorus-sse",
       async start() {
         sseListener = new ChorusSseListener({
-          chorusUrl: config.chorusUrl,
-          apiKey: config.apiKey,
+          chorusUrl,
+          apiKey,
           logger,
           onEvent: (event) => eventRouter.dispatch(event),
           onReconnect: async () => {

package/src/tools/admin-tools.ts

@@ -44,6 +44,47 @@ export function registerAdminTools(api: any, mcpClient: ChorusMcpClient) {
     },
   });
 
+  api.registerTool({
+    name: "chorus_admin_approve_proposal",
+    description:
+      "Approve a Proposal (Admin exclusive). On approval, documentDrafts and taskDrafts are automatically materialized into real Document and Task entities — materialized Tasks can then be claimed and executed by agents. " +
+      "⚠️ This action is irreversible — unless there is a special reason, you MUST obtain explicit human approval before calling this tool.",
+    parameters: {
+      type: "object",
+      properties: {
+        proposalUuid: { type: "string", description: "Proposal UUID" },
+        reviewNote: { type: "string", description: "Optional review note" },
+      },
+      required: ["proposalUuid"],
+      additionalProperties: false,
+    },
+    async execute(_id: string, { proposalUuid, reviewNote }: { proposalUuid: string; reviewNote?: string }) {
+      const args: Record<string, unknown> = { proposalUuid };
+      if (reviewNote) args.reviewNote = reviewNote;
+      const result = await mcpClient.callTool("chorus_admin_approve_proposal", args);
+      return JSON.stringify(result, null, 2);
+    },
+  });
+
+  api.registerTool({
+    name: "chorus_admin_verify_task",
+    description:
+      "Verify a Task (to_verify → done, Admin exclusive). Marks a task as completed after verification. Downstream tasks that depend on this task will only be unblocked after it is verified. " +
+      "⚠️ This action is irreversible — unless there is a special reason, you MUST obtain explicit human approval before calling this tool.",
+    parameters: {
+      type: "object",
+      properties: {
+        taskUuid: { type: "string", description: "Task UUID" },
+      },
+      required: ["taskUuid"],
+      additionalProperties: false,
+    },
+    async execute(_id: string, { taskUuid }: { taskUuid: string }) {
+      const result = await mcpClient.callTool("chorus_admin_verify_task", { taskUuid });
+      return JSON.stringify(result, null, 2);
+    },
+  });
+
   api.registerTool({
     name: "chorus_mark_acceptance_criteria",
     description: "Mark acceptance criteria as passed or failed (admin verification). Blocked criteria prevent task from being verified (to_verify -> done).",