byterover-cli 1.2.0 → 1.3.0

package/dist/infra/cipher/agent/agent-schemas.d.ts

@@ -11,18 +11,18 @@ export declare const LLMConfigSchema: z.ZodObject<{
     verbose: z.ZodDefault<z.ZodBoolean>;
 }, "strict", z.ZodTypeAny, {
     maxIterations: number;
-    verbose: boolean;
-    maxTokens: number;
     temperature: number;
+    maxTokens: number;
+    verbose: boolean;
     topK?: number | undefined;
     topP?: number | undefined;
 }, {
     maxIterations?: number | undefined;
-    verbose?: boolean | undefined;
-    maxTokens?: number | undefined;
     temperature?: number | undefined;
+    maxTokens?: number | undefined;
     topK?: number | undefined;
     topP?: number | undefined;
+    verbose?: boolean | undefined;
 }>;
 export type LLMConfig = z.input<typeof LLMConfigSchema>;
 export type ValidatedLLMConfig = z.output<typeof LLMConfigSchema>;
@@ -49,12 +49,12 @@ export declare const FileSystemConfigSchema: z.ZodObject<{
     maxFileSize: z.ZodOptional<z.ZodNumber>;
     workingDirectory: z.ZodOptional<z.ZodString>;
 }, "strict", z.ZodTypeAny, {
-    workingDirectory?: string | undefined;
     allowedExtensions?: string[] | undefined;
+    workingDirectory?: string | undefined;
     maxFileSize?: number | undefined;
 }, {
-    workingDirectory?: string | undefined;
     allowedExtensions?: string[] | undefined;
+    workingDirectory?: string | undefined;
     maxFileSize?: number | undefined;
 }>;
 export type FileSystemConfig = z.input<typeof FileSystemConfigSchema>;
@@ -67,13 +67,13 @@ export declare const BlobStorageConfigSchema: z.ZodObject<{
     maxTotalSize: z.ZodDefault<z.ZodNumber>;
     storageDir: z.ZodString;
 }, "strict", z.ZodTypeAny, {
-    maxBlobSize: number;
     maxTotalSize: number;
+    maxBlobSize: number;
     storageDir: string;
 }, {
     storageDir: string;
-    maxBlobSize?: number | undefined;
     maxTotalSize?: number | undefined;
+    maxBlobSize?: number | undefined;
 }>;
 export type BlobStorageConfig = z.input<typeof BlobStorageConfigSchema>;
 export type ValidatedBlobStorageConfig = z.output<typeof BlobStorageConfigSchema>;
@@ -92,25 +92,25 @@ export declare const AgentConfigSchema: z.ZodObject<{
         maxTotalSize: z.ZodDefault<z.ZodNumber>;
         storageDir: z.ZodString;
     }, "strict", z.ZodTypeAny, {
-        maxBlobSize: number;
         maxTotalSize: number;
+        maxBlobSize: number;
         storageDir: string;
     }, {
         storageDir: string;
-        maxBlobSize?: number | undefined;
         maxTotalSize?: number | undefined;
+        maxBlobSize?: number | undefined;
     }>>;
     fileSystem: z.ZodOptional<z.ZodObject<{
         allowedExtensions: z.ZodOptional<z.ZodArray<z.ZodString, "many">>;
         maxFileSize: z.ZodOptional<z.ZodNumber>;
         workingDirectory: z.ZodOptional<z.ZodString>;
     }, "strict", z.ZodTypeAny, {
-        workingDirectory?: string | undefined;
         allowedExtensions?: string[] | undefined;
+        workingDirectory?: string | undefined;
         maxFileSize?: number | undefined;
     }, {
-        workingDirectory?: string | undefined;
         allowedExtensions?: string[] | undefined;
+        workingDirectory?: string | undefined;
         maxFileSize?: number | undefined;
     }>>;
     httpReferer: z.ZodOptional<z.ZodString>;
@@ -123,18 +123,18 @@ export declare const AgentConfigSchema: z.ZodObject<{
         verbose: z.ZodDefault<z.ZodBoolean>;
     }, "strict", z.ZodTypeAny, {
         maxIterations: number;
-        verbose: boolean;
-        maxTokens: number;
         temperature: number;
+        maxTokens: number;
+        verbose: boolean;
         topK?: number | undefined;
         topP?: number | undefined;
     }, {
         maxIterations?: number | undefined;
-        verbose?: boolean | undefined;
-        maxTokens?: number | undefined;
         temperature?: number | undefined;
+        maxTokens?: number | undefined;
         topK?: number | undefined;
         topP?: number | undefined;
+        verbose?: boolean | undefined;
     }>>;
     model: z.ZodString;
     openRouterApiKey: z.ZodOptional<z.ZodString>;
@@ -156,34 +156,34 @@ export declare const AgentConfigSchema: z.ZodObject<{
     teamId: z.ZodOptional<z.ZodString>;
     useGranularStorage: z.ZodDefault<z.ZodBoolean>;
 }, "strict", z.ZodTypeAny, {
-    accessToken: string;
-    sessionKey: string;
     model: string;
-    sessions: {
-        maxSessions: number;
-        sessionTTL: number;
-    };
-    apiBaseUrl: string;
     llm: {
         maxIterations: number;
-        verbose: boolean;
-        maxTokens: number;
         temperature: number;
+        maxTokens: number;
+        verbose: boolean;
         topK?: number | undefined;
         topP?: number | undefined;
     };
+    sessions: {
+        maxSessions: number;
+        sessionTTL: number;
+    };
+    accessToken: string;
+    sessionKey: string;
+    apiBaseUrl: string;
     projectId: string;
     useGranularStorage: boolean;
     teamId?: string | undefined;
     spaceId?: string | undefined;
     blobStorage?: {
-        maxBlobSize: number;
         maxTotalSize: number;
+        maxBlobSize: number;
         storageDir: string;
     } | undefined;
     fileSystem?: {
-        workingDirectory?: string | undefined;
         allowedExtensions?: string[] | undefined;
+        workingDirectory?: string | undefined;
         maxFileSize?: number | undefined;
     } | undefined;
     httpReferer?: string | undefined;
@@ -191,11 +191,19 @@ export declare const AgentConfigSchema: z.ZodObject<{
     region?: string | undefined;
     siteName?: string | undefined;
 }, {
+    model: string;
     accessToken: string;
     sessionKey: string;
-    model: string;
     apiBaseUrl: string;
     projectId: string;
+    llm?: {
+        maxIterations?: number | undefined;
+        temperature?: number | undefined;
+        maxTokens?: number | undefined;
+        topK?: number | undefined;
+        topP?: number | undefined;
+        verbose?: boolean | undefined;
+    } | undefined;
     sessions?: {
         maxSessions?: number | undefined;
         sessionTTL?: number | undefined;
@@ -204,23 +212,15 @@ export declare const AgentConfigSchema: z.ZodObject<{
     spaceId?: string | undefined;
     blobStorage?: {
         storageDir: string;
-        maxBlobSize?: number | undefined;
         maxTotalSize?: number | undefined;
+        maxBlobSize?: number | undefined;
     } | undefined;
     fileSystem?: {
-        workingDirectory?: string | undefined;
         allowedExtensions?: string[] | undefined;
+        workingDirectory?: string | undefined;
         maxFileSize?: number | undefined;
     } | undefined;
     httpReferer?: string | undefined;
-    llm?: {
-        maxIterations?: number | undefined;
-        verbose?: boolean | undefined;
-        maxTokens?: number | undefined;
-        temperature?: number | undefined;
-        topK?: number | undefined;
-        topP?: number | undefined;
-    } | undefined;
     openRouterApiKey?: string | undefined;
     region?: string | undefined;
     siteName?: string | undefined;
@@ -241,15 +241,15 @@ export declare const LLMUpdatesSchema: z.ZodObject<{
 }, "strict", z.ZodTypeAny, {
     maxIterations?: number | undefined;
     model?: string | undefined;
-    verbose?: boolean | undefined;
-    maxTokens?: number | undefined;
     temperature?: number | undefined;
+    maxTokens?: number | undefined;
+    verbose?: boolean | undefined;
 }, {
     maxIterations?: number | undefined;
     model?: string | undefined;
-    verbose?: boolean | undefined;
-    maxTokens?: number | undefined;
     temperature?: number | undefined;
+    maxTokens?: number | undefined;
+    verbose?: boolean | undefined;
 }>;
 export type LLMUpdates = z.input<typeof LLMUpdatesSchema>;
 /**

package/dist/infra/cipher/llm/context/context-manager.js

@@ -200,15 +200,13 @@ export class ContextManager {
             content,
             role: 'user',
         };
-        await this.mutex.withLock(async () => {
-            this.messages.push(message);
-            try {
-                await this.persistHistory();
-            }
-            catch (error) {
-                this.logger.error('Failed to persist history after user message', { error, sessionId: this.sessionId });
-            }
-        });
+        this.messages.push(message);
+        try {
+            await this.persistHistory();
+        }
+        catch (error) {
+            this.logger.error('Failed to persist history after user message', { error, sessionId: this.sessionId });
+        }
     }
     /**
      * Clear all messages from the conversation history.

package/dist/infra/cipher/llm/internal-llm-service.d.ts

@@ -72,6 +72,7 @@ export declare class ByteRoverLLMService implements ILLMService {
     private readonly loopDetector;
     private readonly memoryManager?;
     private readonly metadataHandler;
+    private readonly mutex;
     private readonly outputProcessor;
     private readonly providerType;
     private readonly sessionEventBus;
@@ -257,10 +258,13 @@ export declare class ByteRoverLLMService implements ILLMService {
      * Execute a single iteration of the agentic loop.
      *
      * @param options - Iteration options
+     * @param options.executionContext - Optional execution context
+     * @param options.fileData - Optional file data (only used on first iteration)
+     * @param options.imageData - Optional image data (only used on first iteration)
      * @param options.iterationCount - Current iteration number
      * @param options.taskId - Task ID from usecase for billing tracking
+     * @param options.textInput - User input text (only used on first iteration)
      * @param options.tools - Available tools for this iteration
-     * @param options.executionContext - Optional execution context
      * @returns Final response string if complete, null if more iterations needed
      */
     private executeAgenticIteration;

package/dist/infra/cipher/llm/internal-llm-service.js

@@ -6,6 +6,7 @@ import { NoOpLogger } from '../../../core/interfaces/cipher/i-logger.js';
 import { getErrorMessage } from '../../../utils/error-helpers.js';
 import { EnvironmentContextBuilder } from '../system-prompt/environment-context-builder.js';
 import { ToolMetadataHandler } from '../tools/streaming/metadata-handler.js';
+import { AsyncMutex } from './context/async-mutex.js';
 import { ContextManager } from './context/context-manager.js';
 import { LoopDetector } from './context/loop-detector.js';
 import { ClaudeMessageFormatter } from './formatters/claude-formatter.js';
@@ -44,6 +45,7 @@ export class ByteRoverLLMService {
     loopDetector;
     memoryManager;
     metadataHandler;
+    mutex = new AsyncMutex();
     outputProcessor;
     providerType;
     sessionEventBus;
@@ -147,8 +149,6 @@ export class ByteRoverLLMService {
     async completeTask(textInput, options) {
         // Extract options with defaults
         const { executionContext, fileData, imageData, signal, taskId } = options ?? {};
-        // Add user message to context
-        await this.contextManager.addUserMessage(textInput, imageData, fileData);
         // Get filtered tools based on command type (e.g., only read-only tools for 'query')
         const toolSet = this.toolManager.getToolsForCommand(options?.executionContext?.commandType);
         // Create state machine with configured limits
@@ -171,8 +171,11 @@ export class ByteRoverLLMService {
                 // eslint-disable-next-line no-await-in-loop -- Sequential iterations required for agentic loop
                 const result = await this.executeAgenticIteration({
                     executionContext,
+                    fileData,
+                    imageData,
                     iterationCount: stateMachine.getContext().turnCount,
                     taskId,
+                    textInput,
                     tools: toolSet,
                 });
                 if (result !== null) {
@@ -477,14 +480,17 @@ export class ByteRoverLLMService {
      * Execute a single iteration of the agentic loop.
      *
      * @param options - Iteration options
+     * @param options.executionContext - Optional execution context
+     * @param options.fileData - Optional file data (only used on first iteration)
+     * @param options.imageData - Optional image data (only used on first iteration)
      * @param options.iterationCount - Current iteration number
      * @param options.taskId - Task ID from usecase for billing tracking
+     * @param options.textInput - User input text (only used on first iteration)
      * @param options.tools - Available tools for this iteration
-     * @param options.executionContext - Optional execution context
      * @returns Final response string if complete, null if more iterations needed
      */
     async executeAgenticIteration(options) {
-        const { executionContext, iterationCount, taskId, tools } = options;
+        const { executionContext, fileData, imageData, iterationCount, taskId, textInput, tools } = options;
         // Build system prompt using SystemPromptManager (before compression for correct token accounting)
         // Use filtered tool names based on command type (e.g., only read-only tools for 'query')
         const availableTools = this.toolManager.getToolNamesForCommand(executionContext?.commandType);
@@ -533,54 +539,59 @@ export class ByteRoverLLMService {
                 reflectionType,
             });
         }
-        // Get token count for logging (using system prompt for token accounting)
-        const systemPromptTokens = this.generator.estimateTokensSync(systemPrompt);
-        const messages = this.contextManager.getMessages();
-        const messageTokenCounts = messages.map((msg) => this.generator.estimateTokensSync(typeof msg.content === 'string' ? msg.content : JSON.stringify(msg.content)));
-        const maxMessageTokens = this.config.maxInputTokens - systemPromptTokens;
-        // Target utilization to leave headroom for response
-        const targetMessageTokens = Math.floor(maxMessageTokens * TARGET_MESSAGE_TOKEN_UTILIZATION);
-        this.contextManager.compressMessage(targetMessageTokens, messageTokenCounts);
-        // Calculate tokens after compression
-        const compressedMessagesTokens = this.contextManager
-            .getMessages()
-            .reduce((total, msg) => total +
-            this.generator.estimateTokensSync(typeof msg.content === 'string' ? msg.content : JSON.stringify(msg.content)), 0);
-        const tokensUsed = systemPromptTokens + compressedMessagesTokens;
-        // Verbose: Log messages that will be sent to LLM
-        if (this.config.verbose) {
-            console.log('\n========== MESSAGES (Sent to LLM) ==========');
-            console.log(JSON.stringify(this.contextManager.getMessages(), null, 2));
-            console.log('========== END MESSAGES ==========\n');
-            // Log token usage for monitoring compression behavior
-            console.log(`[ByteRoverLLMService] [Iter ${iterationCount + 1}/${this.config.maxIterations}] Sending to LLM: ${tokensUsed} tokens (max: ${this.config.maxInputTokens})`);
-        }
         // Final iteration optimization for query: strip tools (reflection already added above)
         let toolsForThisIteration = tools;
         if (executionContext?.commandType === 'query' && iterationCount === this.config.maxIterations - 1) {
             toolsForThisIteration = {}; // Empty toolset forces text response
         }
-        // Build generation request
-        const request = this.buildGenerateContentRequest({
-            executionContext,
-            systemPrompt,
-            taskId,
-            tools: toolsForThisIteration,
-        });
-        // Call LLM via generator (retry + logging handled by decorators)
-        const lastMessage = await this.callLLMAndParseResponse(request);
-        // Check if there are tool calls
-        if (!lastMessage.toolCalls || lastMessage.toolCalls.length === 0) {
-            const response = await this.handleFinalResponse(lastMessage, taskId);
-            // Auto-compaction check after assistant response
+        // Get token count for logging (using system prompt for token accounting)
+        const systemPromptTokens = this.generator.estimateTokensSync(systemPrompt);
+        // Add user message and compress context within mutex lock
+        return this.mutex.withLock(async () => {
+            // Add user message to context only on the first iteration
+            await this.contextManager.addUserMessage(textInput, imageData, fileData);
+            const messages = this.contextManager.getMessages();
+            const messageTokenCounts = messages.map((msg) => this.generator.estimateTokensSync(typeof msg.content === 'string' ? msg.content : JSON.stringify(msg.content)));
+            const maxMessageTokens = this.config.maxInputTokens - systemPromptTokens;
+            // Target utilization to leave headroom for response
+            const targetMessageTokens = Math.floor(maxMessageTokens * TARGET_MESSAGE_TOKEN_UTILIZATION);
+            this.contextManager.compressMessage(targetMessageTokens, messageTokenCounts);
+            // Calculate tokens after compression
+            const compressedMessagesTokens = this.contextManager
+                .getMessages()
+                .reduce((total, msg) => total +
+                this.generator.estimateTokensSync(typeof msg.content === 'string' ? msg.content : JSON.stringify(msg.content)), 0);
+            const tokensUsed = systemPromptTokens + compressedMessagesTokens;
+            // Verbose: Log messages that will be sent to LLM
+            if (this.config.verbose) {
+                console.log('\n========== MESSAGES (Sent to LLM) ==========');
+                console.log(JSON.stringify(this.contextManager.getMessages(), null, 2));
+                console.log('========== END MESSAGES ==========\n');
+                // Log token usage for monitoring compression behavior
+                console.log(`[ByteRoverLLMService] [Iter ${iterationCount + 1}/${this.config.maxIterations}] Sending to LLM: ${tokensUsed} tokens (max: ${this.config.maxInputTokens})`);
+            }
+            // Build generation request
+            const request = this.buildGenerateContentRequest({
+                executionContext,
+                systemPrompt,
+                taskId,
+                tools: toolsForThisIteration,
+            });
+            // Call LLM via generator (retry + logging handled by decorators)
+            const lastMessage = await this.callLLMAndParseResponse(request);
+            // Check if there are tool calls
+            if (!lastMessage.toolCalls || lastMessage.toolCalls.length === 0) {
+                const response = await this.handleFinalResponse(lastMessage, taskId);
+                // Auto-compaction check after assistant response
+                await this.checkAndTriggerCompaction(taskId ?? '');
+                return response;
+            }
+            // Has tool calls - handle them (pass taskId for subagent billing)
+            await this.handleToolCalls(lastMessage, taskId);
+            // Auto-compaction check after tool execution batch
             await this.checkAndTriggerCompaction(taskId ?? '');
-            return response;
-        }
-        // Has tool calls - handle them (pass taskId for subagent billing)
-        await this.handleToolCalls(lastMessage, taskId);
-        // Auto-compaction check after tool execution batch
-        await this.checkAndTriggerCompaction(taskId ?? '');
-        return null;
+            return null;
+        });
     }
     /**
      * Execute a single tool call in parallel (without adding to context).

package/dist/infra/cipher/session/chat-session.d.ts

@@ -84,9 +84,11 @@ export declare class ChatSession implements IChatSession {
      * @param input - User message
      * @param options - Execution options
      * @param options.executionContext - Optional execution context
-     * @param options.taskId - Optional task ID for concurrent task isolation
+     * @param options.taskId - Optional task ID for billing tracking
+     * @param options.emitTaskId - Whether to include taskId in emitted events (default: true)
      */
     run(input: string, options?: {
+        emitTaskId?: boolean;
         executionContext?: ExecutionContext;
         taskId?: string;
     }): Promise<string>;

package/dist/infra/cipher/session/chat-session.js

@@ -168,10 +168,12 @@ export class ChatSession {
      * @param input - User message
      * @param options - Execution options
      * @param options.executionContext - Optional execution context
-     * @param options.taskId - Optional task ID for concurrent task isolation
+     * @param options.taskId - Optional task ID for billing tracking
+     * @param options.emitTaskId - Whether to include taskId in emitted events (default: true)
      */
     async run(input, options) {
         const taskId = options?.taskId;
+        const emitTaskId = options?.emitTaskId !== false;
         const controller = new AbortController();
         // Track controller per-task for concurrent execution support
         if (taskId) {
@@ -180,8 +182,8 @@ export class ChatSession {
         else {
             this.currentController = controller;
         }
-        // Store taskId for event forwarding (last-write-wins for concurrent tasks)
-        this.currentTaskId = taskId;
+        // Store taskId for event forwarding only if emitTaskId is true
+        this.currentTaskId = emitTaskId ? taskId : undefined;
         this.isExecuting = true;
         sessionStatusManager.setBusy(this.id, this.eventBus);
         try {

package/dist/infra/cipher/system-prompt/contributor-schemas.d.ts

@@ -12,14 +12,14 @@ export declare const StaticContributorConfigSchema: z.ZodObject<{
     type: z.ZodLiteral<"static">;
 }, "strict", z.ZodTypeAny, {
     type: "static";
-    content: string;
     id: string;
+    content: string;
     enabled: boolean;
     priority: number;
 }, {
     type: "static";
-    content: string;
     id: string;
+    content: string;
     priority: number;
     enabled?: boolean | undefined;
 }>;
@@ -179,14 +179,14 @@ export declare const ContributorConfigSchema: z.ZodDiscriminatedUnion<"type", [z
     type: z.ZodLiteral<"static">;
 }, "strict", z.ZodTypeAny, {
     type: "static";
-    content: string;
     id: string;
+    content: string;
     enabled: boolean;
     priority: number;
 }, {
     type: "static";
-    content: string;
     id: string;
+    content: string;
     priority: number;
     enabled?: boolean | undefined;
 }>, z.ZodObject<{
@@ -311,14 +311,14 @@ export declare const SystemPromptManagerConfigSchema: z.ZodObject<{
         type: z.ZodLiteral<"static">;
     }, "strict", z.ZodTypeAny, {
         type: "static";
-        content: string;
         id: string;
+        content: string;
         enabled: boolean;
         priority: number;
     }, {
         type: "static";
-        content: string;
         id: string;
+        content: string;
         priority: number;
         enabled?: boolean | undefined;
     }>, z.ZodObject<{
@@ -433,8 +433,8 @@ export declare const SystemPromptManagerConfigSchema: z.ZodObject<{
 }, "strict", z.ZodTypeAny, {
     contributors: ({
         type: "static";
-        content: string;
         id: string;
+        content: string;
         enabled: boolean;
         priority: number;
     } | {
@@ -471,8 +471,8 @@ export declare const SystemPromptManagerConfigSchema: z.ZodObject<{
 }, {
     contributors: ({
         type: "static";
-        content: string;
         id: string;
+        content: string;
         priority: number;
         enabled?: boolean | undefined;
     } | {

package/dist/infra/cipher/system-prompt/schemas.d.ts

@@ -12,28 +12,28 @@ export declare const PromptConfigSchema: z.ZodEffects<z.ZodEffects<z.ZodObject<{
     prompt: z.ZodOptional<z.ZodString>;
     prompts: z.ZodOptional<z.ZodRecord<z.ZodString, z.ZodString>>;
 }, "strict", z.ZodTypeAny, {
-    prompt?: string | undefined;
     description?: string | undefined;
+    prompt?: string | undefined;
     excludedTools?: string[] | undefined;
     prompts?: Record<string, string> | undefined;
 }, {
-    prompt?: string | undefined;
     description?: string | undefined;
+    prompt?: string | undefined;
     excludedTools?: string[] | undefined;
     prompts?: Record<string, string> | undefined;
 }>, {
-    prompt?: string | undefined;
     description?: string | undefined;
+    prompt?: string | undefined;
     excludedTools?: string[] | undefined;
     prompts?: Record<string, string> | undefined;
 }, {
-    prompt?: string | undefined;
     description?: string | undefined;
+    prompt?: string | undefined;
     excludedTools?: string[] | undefined;
     prompts?: Record<string, string> | undefined;
 }>, {
-    prompt?: string | undefined;
     description?: string | undefined;
+    prompt?: string | undefined;
     excludedTools?: string[] | undefined;
     prompts?: Record<string, string> | undefined;
 }, unknown>;

package/dist/infra/cipher/tools/implementations/task-tool.js

@@ -95,7 +95,7 @@ export function createTaskTool(dependencies) {
     const registry = getAgentRegistry();
     return {
         description: buildTaskToolDescription(registry),
-        // eslint-disable-next-line complexity -- Inherent complexity: validates agent, manages sessions, handles errors
+        // eslint-disable-next-line complexity
         async execute(input, context) {
             const params = input;
             const { contextTreeOnly, description, prompt, sessionId, subagentType } = params;
@@ -181,8 +181,8 @@ export function createTaskTool(dependencies) {
                         progress: 10,
                     });
                 }
-                // Execute the subagent session with parent's taskId for billing tracking
-                const response = await session.run(fullPrompt, { taskId: context?.taskId });
+                // Pass taskId for billing but don't emit it in events (ENG-917)
+                const response = await session.run(fullPrompt, { emitTaskId: false, taskId: context?.taskId });
                 // Stream completion update
                 if (context?.metadata) {
                     context.metadata({

package/dist/infra/connectors/connector-manager.js

@@ -3,6 +3,7 @@ import { CONNECTOR_TYPES } from '../../core/domain/entities/connector-type.js';
 import { HookConnector } from './hook/hook-connector.js';
 import { McpConnector } from './mcp/mcp-connector.js';
 import { RulesConnector } from './rules/rules-connector.js';
+import { SkillConnector } from './skill/skill-connector.js';
 /**
  * Factory and orchestration layer for connectors.
  * Creates connector instances and manages connector operations.
@@ -16,6 +17,7 @@ export class ConnectorManager {
             ['hook', new HookConnector({ fileService, projectRoot })],
             ['mcp', new McpConnector({ fileService, projectRoot, templateService })],
             ['rules', new RulesConnector({ fileService, projectRoot, templateService })],
+            ['skill', new SkillConnector({ fileService, projectRoot })],
         ]);
     }
     async getAllInstalledConnectors() {

package/dist/infra/connectors/hook/hook-connector.d.ts

@@ -21,7 +21,7 @@ type HookConnectorOptions = {
  * - Safe uninstall: Only removes ByteRover hooks by command match
  */
 export declare class HookConnector implements IConnector {
-    readonly type: ConnectorType;
+    readonly connectorType: ConnectorType;
     private readonly fileService;
     private readonly projectRoot;
     private readonly supportedAgents;

package/dist/infra/connectors/hook/hook-connector.js

@@ -23,7 +23,7 @@ function parseJsonAsRecord(content) {
  * - Safe uninstall: Only removes ByteRover hooks by command match
  */
 export class HookConnector {
-    type = 'hook';
+    connectorType = 'hook';
     fileService;
     projectRoot;
     supportedAgents;
@@ -31,7 +31,7 @@ export class HookConnector {
         this.fileService = options.fileService;
         this.projectRoot = options.projectRoot;
         this.supportedAgents = Object.entries(AGENT_CONNECTOR_CONFIG)
-            .filter(([_, config]) => config.supported.includes(this.type))
+            .filter(([_, config]) => config.supported.includes(this.connectorType))
             .map(([agent]) => agent);
     }
     getConfigPath(agent) {
@@ -104,7 +104,7 @@ export class HookConnector {
         }
     }
     isSupported(agent) {
-        return AGENT_CONNECTOR_CONFIG[agent].supported.includes(this.type);
+        return AGENT_CONNECTOR_CONFIG[agent].supported.includes(this.connectorType);
     }
     async status(agent) {
         if (!this.isSupported(agent)) {

package/dist/infra/connectors/mcp/mcp-connector.d.ts

@@ -24,7 +24,7 @@ type McpConnectorOptions = {
  * - Safe uninstall: Only removes ByteRover's MCP server entry and rule content
  */
 export declare class McpConnector implements IConnector {
-    readonly type: ConnectorType;
+    readonly connectorType: ConnectorType;
     private readonly fileService;
     private readonly projectRoot;
     private readonly ruleFileManager;