agent-orchestrator-mcp-server 0.4.4 → 0.4.6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agent-orchestrator-mcp-server",
-  "version": "0.4.4",
+  "version": "0.4.6",
   "description": "Local implementation of agent-orchestrator MCP server",
   "main": "build/index.js",
   "type": "module",

package/shared/tools/get-configs.js

@@ -124,6 +124,7 @@ function formatResponse(configs, fromCache) {
     lines.push('- Use `name` values from **MCP Servers** in `start_session` `mcp_servers` parameter');
     lines.push('- Use `git_root` from **Agent Roots** to start sessions with preconfigured defaults');
     lines.push('- If an **Agent Root** has a `default_subdirectory`, pass it as `subdirectory` in `start_session` — do not set `subdirectory` to arbitrary internal paths');
+    lines.push("- Pass `default_skills` from **Agent Roots** in the `skills` parameter of `start_session` — sessions won't have skills loaded unless you explicitly pass them");
     lines.push('- Use `id` values from **Stop Conditions** in `start_session` `stop_condition` parameter');
     if (fromCache) {
         lines.push('');

package/shared/tools/start-session.d.ts

@@ -12,6 +12,7 @@ export declare const StartSessionSchema: z.ZodObject<{
     stop_condition: z.ZodOptional<z.ZodString>;
     execution_provider: z.ZodOptional<z.ZodEnum<["local_filesystem", "remote_sandbox"]>>;
     mcp_servers: z.ZodOptional<z.ZodArray<z.ZodString, "many">>;
+    skills: z.ZodOptional<z.ZodArray<z.ZodString, "many">>;
     config: z.ZodOptional<z.ZodRecord<z.ZodString, z.ZodUnknown>>;
     custom_metadata: z.ZodOptional<z.ZodRecord<z.ZodString, z.ZodUnknown>>;
 }, "strip", z.ZodTypeAny, {
@@ -25,6 +26,7 @@ export declare const StartSessionSchema: z.ZodObject<{
     slug?: string | undefined;
     stop_condition?: string | undefined;
     execution_provider?: "local_filesystem" | "remote_sandbox" | undefined;
+    skills?: string[] | undefined;
     config?: Record<string, unknown> | undefined;
     custom_metadata?: Record<string, unknown> | undefined;
 }, {
@@ -38,6 +40,7 @@ export declare const StartSessionSchema: z.ZodObject<{
     slug?: string | undefined;
     stop_condition?: string | undefined;
     execution_provider?: "local_filesystem" | "remote_sandbox" | undefined;
+    skills?: string[] | undefined;
     config?: Record<string, unknown> | undefined;
     custom_metadata?: Record<string, unknown> | undefined;
 }>;
@@ -77,7 +80,7 @@ export declare function startSessionTool(_server: Server, clientFactory: () => I
             };
             stop_condition: {
                 type: string;
-                description: "Condition that determines when the agent should stop. Passed to the agent as context.";
+                description: "Stop condition ID from get_configs (e.g. \"pr_merged\"). The description is automatically resolved and passed to the agent as context.";
             };
             execution_provider: {
                 type: string;
@@ -91,6 +94,13 @@ export declare function startSessionTool(_server: Server, clientFactory: () => I
                 };
                 description: "List of MCP server names to enable for this session. Example: [\"github-development\", \"slack\"]";
             };
+            skills: {
+                type: string;
+                items: {
+                    type: string;
+                };
+                description: "List of skill names to enable for this session. Example: [\"discovery-classify\", \"publish-and-pr\"]";
+            };
             config: {
                 type: string;
                 description: "Additional configuration as a JSON object.";

package/shared/tools/start-session.js

@@ -18,9 +18,10 @@ const PARAM_DESCRIPTIONS = {
         '(e.g. "Fix login redirect loop on mobile Safari", "Add dark mode toggle to settings page"). ' +
         'Only omit if you truly have zero context about the session purpose, which should be extremely rare.',
     slug: 'URL-friendly identifier for the session. Must be unique.',
-    stop_condition: 'Condition that determines when the agent should stop. Passed to the agent as context.',
+    stop_condition: 'Stop condition ID from get_configs (e.g. "pr_merged"). The description is automatically resolved and passed to the agent as context.',
     execution_provider: 'Execution environment. Options: "local_filesystem" (runs locally), "remote_sandbox" (runs in isolated sandbox). Default: "local_filesystem"',
     mcp_servers: 'List of MCP server names to enable for this session. Example: ["github-development", "slack"]',
+    skills: 'List of skill names to enable for this session. Example: ["discovery-classify", "publish-and-pr"]',
     config: 'Additional configuration as a JSON object.',
     custom_metadata: 'User-defined metadata as a JSON object. Useful for tracking tickets, projects, etc.',
 };
@@ -38,6 +39,7 @@ export const StartSessionSchema = z.object({
         .optional()
         .describe(PARAM_DESCRIPTIONS.execution_provider),
     mcp_servers: z.array(z.string()).optional().describe(PARAM_DESCRIPTIONS.mcp_servers),
+    skills: z.array(z.string()).optional().describe(PARAM_DESCRIPTIONS.skills),
     config: z.record(z.unknown()).optional().describe(PARAM_DESCRIPTIONS.config),
     custom_metadata: z.record(z.unknown()).optional().describe(PARAM_DESCRIPTIONS.custom_metadata),
 });
@@ -51,10 +53,12 @@ const TOOL_DESCRIPTION = `Start a new agent session in the Agent Orchestrator.
 - If a prompt is provided, the agent job is automatically queued to start
 - If no prompt is provided, creates a clone-only session that can be started later with action_session
 
+**Defaults from Agent Roots:** When starting a session that matches a preconfigured agent root (from \`get_configs\`), the agent root defines \`default_mcp_servers\` and \`default_skills\`. In most cases, you should pass these defaults through — omitting them means the session won't have MCP servers or skills configured. You can reduce the set (e.g., drop an MCP server you don't need), but when \`ALLOWED_AGENT_ROOTS\` is active, you cannot add servers beyond the defaults. Skills are not constrained — you can augment with additional skills as needed. As a rule of thumb: always pass the defaults unless you have a specific reason to remove something.
+
 **Use cases:**
 - Start a new agent task on a repository
 - Create a session to work on a specific branch
-- Set up an agent with specific MCP servers enabled
+- Set up an agent with specific MCP servers and skills enabled
 - Create a session with custom metadata for tracking`;
 export function startSessionTool(_server, clientFactory) {
     return {
@@ -105,6 +109,11 @@ export function startSessionTool(_server, clientFactory) {
                     items: { type: 'string' },
                     description: PARAM_DESCRIPTIONS.mcp_servers,
                 },
+                skills: {
+                    type: 'array',
+                    items: { type: 'string' },
+                    description: PARAM_DESCRIPTIONS.skills,
+                },
                 config: {
                     type: 'object',
                     description: PARAM_DESCRIPTIONS.config,
@@ -137,7 +146,21 @@ export function startSessionTool(_server, clientFactory) {
                         };
                     }
                 }
-                const session = await client.createSession(validatedArgs);
+                // Resolve stop_condition ID to its description so the agent receives
+                // meaningful context about when to stop, not just an opaque identifier.
+                let createArgs = validatedArgs;
+                if (validatedArgs.stop_condition) {
+                    let configs = getConfigsCache();
+                    if (!configs) {
+                        configs = await client.getConfigs();
+                        setConfigsCache(configs);
+                    }
+                    const match = configs.stop_conditions.find((sc) => sc.id === validatedArgs.stop_condition);
+                    if (match) {
+                        createArgs = { ...validatedArgs, stop_condition: match.description };
+                    }
+                }
+                const session = await client.createSession(createArgs);
                 const lines = [
                     `## Session Started Successfully`,
                     '',

package/shared/types.d.ts

@@ -142,6 +142,7 @@ export interface CreateSessionRequest {
     stop_condition?: string;
     execution_provider?: string;
     mcp_servers?: string[];
+    skills?: string[];
     config?: Record<string, unknown>;
     custom_metadata?: Record<string, unknown>;
 }