fix: unify agentType value domain to 'claude-code' | 'codex'
The streamed-output-evaluator used 'claude' while the rest of the codebase used 'claude-code'. Align the evaluator type and all call sites (message-agent-executor, task-scheduler) to use the canonical 'claude-code' | 'codex' domain consistently. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
@@ -40,7 +40,7 @@ function freshState(): StreamedOutputState {
|
||||
}
|
||||
|
||||
const claudeOpts: EvaluateStreamedOutputOptions = {
|
||||
agentType: 'claude',
|
||||
agentType: 'claude-code',
|
||||
provider: 'claude',
|
||||
};
|
||||
|
||||
@@ -91,7 +91,7 @@ describe('evaluateStreamedOutput', () => {
|
||||
const result = evaluateStreamedOutput(
|
||||
successOutput('Hello'),
|
||||
freshState(),
|
||||
{ agentType: 'claude', provider: 'fallback' },
|
||||
{ agentType: 'claude-code', provider: 'fallback' },
|
||||
);
|
||||
expect(result.shouldForwardOutput).toBe(true);
|
||||
expect(result.state.sawOutput).toBe(true);
|
||||
@@ -130,7 +130,7 @@ describe('evaluateStreamedOutput', () => {
|
||||
const result = evaluateStreamedOutput(
|
||||
successOutput('banner text'),
|
||||
freshState(),
|
||||
{ agentType: 'claude', provider: 'fallback' },
|
||||
{ agentType: 'claude-code', provider: 'fallback' },
|
||||
);
|
||||
// Non-primary: skip banner check, forward normally
|
||||
expect(result.shouldForwardOutput).toBe(true);
|
||||
@@ -244,7 +244,7 @@ describe('evaluateStreamedOutput', () => {
|
||||
|
||||
it('does not track for non-primary Claude', () => {
|
||||
const result = evaluateStreamedOutput(successOutput(null), freshState(), {
|
||||
agentType: 'claude',
|
||||
agentType: 'claude-code',
|
||||
provider: 'fallback',
|
||||
trackSuccessNullResult: true,
|
||||
});
|
||||
|
||||
Reference in New Issue
Block a user