byterover-cli 2.6.0 → 3.0.1

package/dist/server/infra/transport/handlers/space-handler.d.ts

@@ -1,5 +1,6 @@
 import type { ITokenStore } from '../../../core/interfaces/auth/i-token-store.js';
 import type { IContextTreeMerger } from '../../../core/interfaces/context-tree/i-context-tree-merger.js';
+import type { IContextTreeService } from '../../../core/interfaces/context-tree/i-context-tree-service.js';
 import type { IContextTreeSnapshotService } from '../../../core/interfaces/context-tree/i-context-tree-snapshot-service.js';
 import type { IContextTreeWriterService } from '../../../core/interfaces/context-tree/i-context-tree-writer-service.js';
 import type { ICogitPullService } from '../../../core/interfaces/services/i-cogit-pull-service.js';
@@ -12,6 +13,7 @@ export interface SpaceHandlerDeps {
     broadcastToProject: ProjectBroadcaster;
     cogitPullService: ICogitPullService;
     contextTreeMerger: IContextTreeMerger;
+    contextTreeService: IContextTreeService;
     contextTreeSnapshotService: IContextTreeSnapshotService;
     contextTreeWriterService: IContextTreeWriterService;
     projectConfigStore: IProjectConfigStore;
@@ -29,6 +31,7 @@ export declare class SpaceHandler {
     private readonly broadcastToProject;
     private readonly cogitPullService;
     private readonly contextTreeMerger;
+    private readonly contextTreeService;
     private readonly contextTreeSnapshotService;
     private readonly contextTreeWriterService;
     private readonly projectConfigStore;

package/dist/server/infra/transport/handlers/space-handler.js

@@ -3,7 +3,7 @@ import { SpaceEvents, } from '../../../../shared/transport/events/space-events.j
 import { BRV_DIR, CONTEXT_TREE_CONFLICT_DIR, DEFAULT_BRANCH } from '../../../constants.js';
 import { LocalChangesExistError, NotAuthenticatedError, ProjectNotInitError, SpaceNotFoundError, } from '../../../core/domain/errors/task-error.js';
 import { syncConfigToXdg } from '../../../utils/config-xdg-sync.js';
-import { hasAnyChanges, resolveRequiredProjectPath, } from './handler-types.js';
+import { guardAgainstGitVc, hasAnyChanges, resolveRequiredProjectPath, } from './handler-types.js';
 /**
  * Handles space:* events.
  * Business logic for space listing and switching — no terminal/UI calls.
@@ -12,6 +12,7 @@ export class SpaceHandler {
     broadcastToProject;
     cogitPullService;
     contextTreeMerger;
+    contextTreeService;
     contextTreeSnapshotService;
     contextTreeWriterService;
     projectConfigStore;
@@ -24,6 +25,7 @@ export class SpaceHandler {
         this.broadcastToProject = deps.broadcastToProject;
         this.cogitPullService = deps.cogitPullService;
         this.contextTreeMerger = deps.contextTreeMerger;
+        this.contextTreeService = deps.contextTreeService;
         this.contextTreeSnapshotService = deps.contextTreeSnapshotService;
         this.contextTreeWriterService = deps.contextTreeWriterService;
         this.projectConfigStore = deps.projectConfigStore;
@@ -66,6 +68,7 @@ export class SpaceHandler {
     }
     async handleSwitch(data, clientId) {
         const projectPath = resolveRequiredProjectPath(this.resolveProjectPath, clientId);
+        await guardAgainstGitVc({ contextTreeService: this.contextTreeService, projectPath });
         const token = await this.tokenStore.load();
         if (!token || !token.isValid()) {
             throw new NotAuthenticatedError();

package/dist/server/infra/transport/handlers/status-handler.d.ts

@@ -1,12 +1,16 @@
 import type { ITokenStore } from '../../../core/interfaces/auth/i-token-store.js';
 import type { IContextTreeService } from '../../../core/interfaces/context-tree/i-context-tree-service.js';
 import type { IContextTreeSnapshotService } from '../../../core/interfaces/context-tree/i-context-tree-snapshot-service.js';
+import type { ICurateLogStore } from '../../../core/interfaces/storage/i-curate-log-store.js';
 import type { IProjectConfigStore } from '../../../core/interfaces/storage/i-project-config-store.js';
 import type { ITransportServer } from '../../../core/interfaces/transport/i-transport-server.js';
 import { type ProjectPathResolver } from './handler-types.js';
+/** Factory that creates a curate log store scoped to a project directory. */
+export type CurateLogStoreFactory = (projectPath: string) => ICurateLogStore;
 export interface StatusHandlerDeps {
     contextTreeService: IContextTreeService;
     contextTreeSnapshotService: IContextTreeSnapshotService;
+    curateLogStoreFactory: CurateLogStoreFactory;
     projectConfigStore: IProjectConfigStore;
     resolveProjectPath: ProjectPathResolver;
     tokenStore: ITokenStore;
@@ -19,6 +23,7 @@ export interface StatusHandlerDeps {
 export declare class StatusHandler {
     private readonly contextTreeService;
     private readonly contextTreeSnapshotService;
+    private readonly curateLogStoreFactory;
     private readonly projectConfigStore;
     private readonly resolveProjectPath;
     private readonly tokenStore;

package/dist/server/infra/transport/handlers/status-handler.js

@@ -9,6 +9,7 @@ import { resolveRequiredProjectPath } from './handler-types.js';
 export class StatusHandler {
     contextTreeService;
     contextTreeSnapshotService;
+    curateLogStoreFactory;
     projectConfigStore;
     resolveProjectPath;
     tokenStore;
@@ -16,6 +17,7 @@ export class StatusHandler {
     constructor(deps) {
         this.contextTreeService = deps.contextTreeService;
         this.contextTreeSnapshotService = deps.contextTreeSnapshotService;
+        this.curateLogStoreFactory = deps.curateLogStoreFactory;
         this.projectConfigStore = deps.projectConfigStore;
         this.resolveProjectPath = deps.resolveProjectPath;
         this.tokenStore = deps.tokenStore;
@@ -67,24 +69,30 @@ export class StatusHandler {
         try {
             const contextTreeExists = await this.contextTreeService.exists(projectPath);
             if (contextTreeExists) {
-                result.contextTreeDir = join(projectPath, BRV_DIR, CONTEXT_TREE_DIR);
-                result.contextTreeRelativeDir = join(BRV_DIR, CONTEXT_TREE_DIR);
-                const hasSnapshot = await this.contextTreeSnapshotService.hasSnapshot(projectPath);
-                if (!hasSnapshot) {
-                    await this.contextTreeSnapshotService.initEmptySnapshot(projectPath);
-                }
-                const changes = await this.contextTreeSnapshotService.getChanges(projectPath);
-                const hasChanges = changes.added.length > 0 || changes.modified.length > 0 || changes.deleted.length > 0;
-                if (hasChanges) {
-                    result.contextTreeStatus = 'has_changes';
-                    result.contextTreeChanges = {
-                        added: changes.added,
-                        deleted: changes.deleted,
-                        modified: changes.modified,
-                    };
+                const hasGitVc = await this.contextTreeService.hasGitRepo(projectPath);
+                if (hasGitVc) {
+                    result.contextTreeStatus = 'git_vc';
                 }
                 else {
-                    result.contextTreeStatus = 'no_changes';
+                    result.contextTreeDir = join(projectPath, BRV_DIR, CONTEXT_TREE_DIR);
+                    result.contextTreeRelativeDir = join(BRV_DIR, CONTEXT_TREE_DIR);
+                    const hasSnapshot = await this.contextTreeSnapshotService.hasSnapshot(projectPath);
+                    if (!hasSnapshot) {
+                        await this.contextTreeSnapshotService.initEmptySnapshot(projectPath);
+                    }
+                    const changes = await this.contextTreeSnapshotService.getChanges(projectPath);
+                    const hasChanges = changes.added.length > 0 || changes.modified.length > 0 || changes.deleted.length > 0;
+                    if (hasChanges) {
+                        result.contextTreeStatus = 'has_changes';
+                        result.contextTreeChanges = {
+                            added: changes.added,
+                            deleted: changes.deleted,
+                            modified: changes.modified,
+                        };
+                    }
+                    else {
+                        result.contextTreeStatus = 'no_changes';
+                    }
                 }
             }
             else {
@@ -94,6 +102,33 @@ export class StatusHandler {
         catch {
             result.contextTreeStatus = 'unknown';
         }
+        // Pending review count (best-effort)
+        try {
+            const store = this.curateLogStoreFactory(projectPath);
+            const entries = await store.list({ limit: 100, status: ['completed'] });
+            const pendingFiles = new Set();
+            const contextTreeRoot = join(projectPath, BRV_DIR, CONTEXT_TREE_DIR);
+            for (const entry of entries) {
+                for (const op of entry.operations) {
+                    if (op.reviewStatus === 'pending' && op.filePath) {
+                        const prefix = contextTreeRoot + '/';
+                        const relativePath = op.filePath.startsWith(prefix) ? op.filePath.slice(prefix.length) : op.filePath;
+                        pendingFiles.add(relativePath);
+                    }
+                }
+            }
+            if (pendingFiles.size > 0) {
+                result.pendingReviewCount = pendingFiles.size;
+                const port = this.transport.getPort();
+                if (port) {
+                    const encoded = Buffer.from(projectPath).toString('base64url');
+                    result.reviewUrl = `http://127.0.0.1:${port}/review?project=${encoded}`;
+                }
+            }
+        }
+        catch {
+            // Best-effort — if the log is unavailable, skip review info
+        }
         return result;
     }
 }

package/dist/server/infra/transport/handlers/vc-handler.d.ts

@@ -0,0 +1,100 @@
+import type { ITokenStore } from '../../../core/interfaces/auth/i-token-store.js';
+import type { IContextTreeService } from '../../../core/interfaces/context-tree/i-context-tree-service.js';
+import type { IGitService } from '../../../core/interfaces/services/i-git-service.js';
+import type { ISpaceService } from '../../../core/interfaces/services/i-space-service.js';
+import type { ITeamService } from '../../../core/interfaces/services/i-team-service.js';
+import type { IProjectConfigStore } from '../../../core/interfaces/storage/i-project-config-store.js';
+import type { ITransportServer } from '../../../core/interfaces/transport/i-transport-server.js';
+import type { IVcGitConfigStore } from '../../../core/interfaces/vc/i-vc-git-config-store.js';
+import { type ProjectBroadcaster, type ProjectPathResolver } from './handler-types.js';
+export interface IVcHandlerDeps {
+    broadcastToProject: ProjectBroadcaster;
+    contextTreeService: IContextTreeService;
+    gitRemoteBaseUrl: string;
+    gitService: IGitService;
+    projectConfigStore: IProjectConfigStore;
+    resolveProjectPath: ProjectPathResolver;
+    spaceService: ISpaceService;
+    teamService: ITeamService;
+    tokenStore: ITokenStore;
+    transport: ITransportServer;
+    vcGitConfigStore: IVcGitConfigStore;
+    webAppUrl: string;
+}
+/**
+ * Handles vc:* events (Version Control commands).
+ */
+export declare class VcHandler {
+    private readonly broadcastToProject;
+    private readonly contextTreeService;
+    private readonly gitRemoteBaseUrl;
+    private readonly gitService;
+    private readonly projectConfigStore;
+    private readonly resolveProjectPath;
+    private readonly spaceService;
+    private readonly teamService;
+    private readonly tokenStore;
+    private readonly transport;
+    private readonly vcGitConfigStore;
+    private readonly webAppUrl;
+    constructor(deps: IVcHandlerDeps);
+    setup(): void;
+    private buildAuthorHint;
+    private buildNoRemoteMessage;
+    /**
+     * Writes a .gitignore to the context-tree directory only if one does not already exist.
+     */
+    private ensureGitignore;
+    /**
+     * When force is NOT set, checks for uncommitted changes and throws
+     * VcError(UNCOMMITTED_CHANGES) if the working tree is dirty.
+     * When force IS set, skips the check entirely (changes will be discarded).
+     */
+    private guardUncommittedChanges;
+    private handleAdd;
+    private handleBranch;
+    private handleBranchCreate;
+    private handleBranchDelete;
+    private handleBranchList;
+    private handleBranchSetUpstream;
+    private handleCheckout;
+    private handleClone;
+    private handleCommit;
+    private handleConfig;
+    private handleFetch;
+    private handleInit;
+    private handleLog;
+    private handleMerge;
+    private handlePull;
+    private handlePush;
+    private handleRemote;
+    private handleReset;
+    private handleStatus;
+    /**
+     * Resolve clone request data into a clean cogit URL + team/space info.
+     * Accepts either a URL or explicit teamName/spaceName.
+     * Auth is handled by IsomorphicGitService via headers, not URL credentials.
+     */
+    private resolveCloneInput;
+    /**
+     * Resolve a remote URL to a clean cogit URL + team/space info.
+     * Expected format: {domain}/{teamName}/{spaceName}.git
+     * Resolves names to IDs via API; rejects unknown formats.
+     *
+     * Auth is handled by IsomorphicGitService via headers, not URL credentials.
+     */
+    private resolveFullCogitUrl;
+    private resolveLogResult;
+    /**
+     * Resolves pull target branch: explicit → tracking config → error.
+     * Mirrors native git: `git pull` without tracking config errors.
+     */
+    private resolvePullBranch;
+    private resolveTargetBranch;
+    /**
+     * Resolve team/space names to IDs via API, build clean cogit URL.
+     */
+    private resolveTeamSpaceNames;
+    private validateBranchName;
+    private validateRemoteUrlDomain;
+}