fix: Revert enum type annotations to correct approach

Reverted keyof typeof changes back to direct enum types. The original
issue was resolved by earlier fixes to core package enum exports.
Direct enum type annotations (AgentProvider, AgentRole) are the
correct approach for enum values in function parameters.

Build now passes successfully.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: alpsla

packages/agents/src/multi-agent/factory.ts

@@ -156,13 +156,13 @@ export class MultiAgentFactory {
     name: string,
     strategy: AnalysisStrategy,
     primaryAgentSelection: {
-      provider: keyof typeof AgentProvider,
-      role: keyof typeof AgentRole,
+      provider: AgentProvider,
+      role: AgentRole,
       position?: AgentPosition
     },
     secondaryAgentSelections: Array<{
-      provider: keyof typeof AgentProvider,
-      role: keyof typeof AgentRole,
+      provider: AgentProvider,
+      role: AgentRole,
       position?: AgentPosition
     }> = [],
     options: {
@@ -234,18 +234,18 @@ export class MultiAgentFactory {
     name: string,
     strategy: AnalysisStrategy,
     primaryConfig: {
-      provider: keyof typeof AgentProvider,
-      role: keyof typeof AgentRole,
+      provider: AgentProvider,
+      role: AgentRole,
       position: AgentPosition
     },
     secondaryConfigs: Array<{
-      provider: keyof typeof AgentProvider,
-      role: keyof typeof AgentRole,
+      provider: AgentProvider,
+      role: AgentRole,
       position: AgentPosition
     }>,
     fallbackConfigs?: Array<{
-      provider: keyof typeof AgentProvider,
-      role: keyof typeof AgentRole,
+      provider: AgentProvider,
+      role: AgentRole,
       position: AgentPosition,
       priority?: number
     }>,