feat: allow optional seperate review model

Author: jahilldev

api/src/agent/agent.model.ts

@@ -19,6 +19,10 @@ export interface AgentRunOptions {
   skills?: string[];
   /** Override the default per-invocation timeout. */
   timeoutMs?: number;
+  /** Override the model for this specific invocation (e.g. a review-only model). */
+  model?: string;
+  /** Override the provider for this specific invocation. */
+  provider?: string;
 }
 
 export interface AgentRunResult {

api/src/agent/agent.service.ts

@@ -21,7 +21,8 @@ export class HermesAgentService {
   ) {}
 
   async run(opts: AgentRunOptions): Promise<AgentRunResult> {
-    const model = this.config.get('HERMES_MODEL') || undefined;
+    const model = opts.model ?? this.config.get('HERMES_MODEL') ?? undefined;
+    const provider = opts.provider ?? this.config.get('HERMES_PROVIDER') ?? undefined;
     const sandboxMode = this.config.get('SANDBOX_MODE');
     const hermesHome = this.config.get('HERMES_HOME') || undefined;
 
@@ -37,7 +38,7 @@ export class HermesAgentService {
       cwd: opts.cwd,
       prompt: opts.prompt,
       model,
-      provider: this.config.get('HERMES_PROVIDER') || undefined,
+      provider,
       toolsets: opts.toolsets,
       skills: opts.skills,
     });

api/src/config/config.model.ts

@@ -27,6 +27,8 @@ export const envSchema = z.object({
   HERMES_HOME: z.string().optional(),
   HERMES_MODEL: z.string().optional(),
   HERMES_PROVIDER: z.string().optional(),
+  HERMES_REVIEW_MODEL: z.string().optional(),
+  HERMES_REVIEW_PROVIDER: z.string().optional(),
   HERMES_TIMEOUT_MS: intFromString(7_200_000),
 
   // orchestration policy

api/src/orchestrator/orchestrator.service.ts

@@ -578,6 +578,8 @@ export class OrchestratorService {
         phase: 'REVIEW',
         cwd: ws.dir,
         prompt: reviewPrompt,
+        model: this.config.get('HERMES_REVIEW_MODEL') || undefined,
+        provider: this.config.get('HERMES_REVIEW_PROVIDER') || undefined,
       });
       if (res.status !== 'SUCCEEDED') {
         throw new Error(`review agent ${res.status}; ${res.stderr.slice(0, 300)}`);
@@ -613,6 +615,8 @@ export class OrchestratorService {
           phase: 'REVISE',
           cwd: ws.dir,
           prompt: revisePrompt,
+          model: this.config.get('HERMES_REVIEW_MODEL') || undefined,
+          provider: this.config.get('HERMES_REVIEW_PROVIDER') || undefined,
         });
         if (rev.status !== 'SUCCEEDED') {
           // Don't let a failed/timed-out revise kill the whole task — the workspace is