byterover-cli 1.0.4 → 1.1.0

package/dist/tui/contexts/onboarding-context.d.ts

@@ -22,6 +22,8 @@ export interface OnboardingContextValue {
     hasCurated: boolean;
     /** Whether query has been completed at least once */
     hasQueried: boolean;
+    /** Whether user has acknowledged init completion */
+    initAcknowledged: boolean;
     /** Whether the project is initialized (brvConfig exists) */
     isInitialized: boolean;
     /** Whether we're still loading the dismissed state */
@@ -30,6 +32,8 @@ export interface OnboardingContextValue {
     queryAcknowledged: boolean;
     /** Set curate acknowledged state */
     setCurateAcknowledged: (value: boolean) => void;
+    /** Set init acknowledged state */
+    setInitAcknowledged: (value: boolean) => void;
     /** Set query acknowledged state */
     setQueryAcknowledged: (value: boolean) => void;
     /** Whether onboarding should be shown */

package/dist/tui/contexts/onboarding-context.js

@@ -65,6 +65,7 @@ export function OnboardingProvider({ children }) {
         }
     }, [isInitialConfigLoaded, isInitialized]);
     // Track acknowledgment for completed steps (user pressed Enter after seeing output)
+    const [initAcknowledged, setInitAcknowledgedState] = useState(false);
     const [curateAcknowledged, setCurateAcknowledgedState] = useState(false);
     const [queryAcknowledged, setQueryAcknowledgedState] = useState(false);
     // Track if init was completed during this onboarding session (to avoid duplicate tracking)
@@ -76,6 +77,10 @@ export function OnboardingProvider({ children }) {
             trackingService.track('onboarding:init_completed');
         }
     }, [isInitialized, trackingService]);
+    // Wrapper for setInitAcknowledged that also tracks
+    const setInitAcknowledged = useCallback((value) => {
+        setInitAcknowledgedState(value);
+    }, []);
     // Wrapper for setCurateAcknowledged that also tracks
     const setCurateAcknowledged = useCallback((value) => {
         setCurateAcknowledgedState(value);
@@ -110,10 +115,13 @@ export function OnboardingProvider({ children }) {
         return { hasCurated: curateCompleted, hasQueried: queryCompleted };
     }, [tasks]);
     // Derive current step (considering acknowledgment)
-    // Stay on curate/query step until user acknowledges the completion
+    // Stay on each step until user acknowledges the completion
     const currentStep = useMemo(() => {
         if (!isInitialized)
             return 'init';
+        // isInitialized is true but not yet acknowledged -> stay on init
+        if (!initAcknowledged)
+            return 'init';
         if (!hasCurated)
             return 'curate';
         // hasCurated is true but not yet acknowledged -> stay on curate
@@ -125,7 +133,7 @@ export function OnboardingProvider({ children }) {
         if (!queryAcknowledged)
             return 'query';
         return 'complete';
-    }, [isInitialized, hasCurated, hasQueried, curateAcknowledged, queryAcknowledged]);
+    }, [isInitialized, initAcknowledged, hasCurated, hasQueried, curateAcknowledged, queryAcknowledged]);
     // Show onboarding if:
     // 1. Project was not initialized after initial config check, AND
     // 2. User has never dismissed onboarding before (persisted)
@@ -146,10 +154,12 @@ export function OnboardingProvider({ children }) {
         currentStep,
         hasCurated,
         hasQueried,
+        initAcknowledged,
         isInitialized,
         isLoadingDismissed,
         queryAcknowledged,
         setCurateAcknowledged,
+        setInitAcknowledged,
         setQueryAcknowledged,
         shouldShowOnboarding,
         totalSteps: 3, // init, curate, query (complete is not counted)
@@ -160,9 +170,11 @@ export function OnboardingProvider({ children }) {
         hasCurated,
         hasQueried,
         hasDismissed,
+        initAcknowledged,
         isInitialized,
         isLoadingDismissed,
         queryAcknowledged,
+        setInitAcknowledged,
         shouldShowOnboarding,
     ]);
     return _jsx(OnboardingContext.Provider, { value: contextValue, children: children });

package/dist/tui/views/command-view.js

@@ -231,6 +231,7 @@ export const CommandView = ({ availableHeight }) => {
             };
         }
     }, [activePrompt?.type, appendShortcuts, removeShortcuts]);
+    /* eslint-disable complexity -- Command execution requires handling multiple command types and states */
     const executeCommand = useCallback(async (value) => {
         const trimmed = value.trim();
         if (!trimmed)
@@ -301,6 +302,23 @@ export const CommandView = ({ availableHeight }) => {
                 setActivePrompt(null);
                 const needReloadAuth = trimmed.startsWith('/login') || trimmed.startsWith('/logout');
                 const needReloadBrvConfig = trimmed.startsWith('/space switch') || trimmed.startsWith('/init');
+                const needNewSession = trimmed.startsWith('/new');
+                // Handle /new command - create new session and clear messages
+                if (needNewSession && client) {
+                    try {
+                        const response = await client.request('agent:newSession', { reason: 'User requested new session' });
+                        /* eslint-disable max-depth -- UI state handling requires this nesting level */
+                        if (response.success) {
+                            // Clear the messages to start fresh
+                            setMessages([]);
+                            clearTasks();
+                        }
+                    }
+                    catch {
+                        // Error handling - the command already showed feedback
+                    }
+                    /* eslint-enable max-depth */
+                }
                 // Refresh state after commands that change auth or project state
                 if (needReloadAuth || needReloadBrvConfig) {
                     clearTasks();
@@ -323,6 +341,7 @@ export const CommandView = ({ availableHeight }) => {
             }
         }
     }, [clearTasks, client, exit, handleSlashCommand, reloadAuth, reloadBrvConfig]);
+    /* eslint-enable complexity */
     const handleSubmit = useCallback(async (value) => {
         if (mode === 'console' && !isStreaming) {
             await executeCommand(value);

package/dist/utils/file-validator.d.ts

@@ -3,7 +3,7 @@
  * Checks:
  * 1. File exists
  * 2. File is within project directory (where .brv exists)
- * 3. File is text/code file (not binary)
+ * 3. File is supported by read_file tool (text, images, or PDFs)
  *
  * @param filePath - The file path to validate (can be relative, absolute, or tilde)
  * @param projectRoot - The project root directory (current working directory)

package/dist/utils/file-validator.js

@@ -1,17 +1,18 @@
 import fs from 'node:fs';
 import os from 'node:os';
 import path from 'node:path';
+import { isBinaryFile, isMediaFile } from '../infra/cipher/file-system/binary-utils.js';
 /**
  * Normalize file path - handles relative, absolute, tilde, symlinks
  * Returns absolute canonical path
  * @param filePath - The file path to normalize
  * @returns Normalized absolute path
  */
-function normalizeFilePath(filePath) {
+function normalizeFilePath(filePath, baseDir) {
     // Expand tilde to home directory
     const expanded = filePath.startsWith('~') ? filePath.replace(/^~/, os.homedir()) : filePath;
-    // Resolve to absolute path
-    const absolute = path.resolve(expanded);
+    // Resolve to absolute path using baseDir for relative paths
+    const absolute = path.isAbsolute(expanded) ? expanded : path.resolve(baseDir ?? process.cwd(), expanded);
     // Resolve symlinks (only if file exists)
     try {
         return fs.realpathSync(absolute);
@@ -21,44 +22,23 @@ function normalizeFilePath(filePath) {
         return absolute;
     }
 }
-/**
- * Check if file is text (no null bytes in first 8KB)
- * Returns false for binary files (images, PDFs, compiled binaries, etc.)
- * @param filePath - The file path to check
- * @returns true if file is text, false if binary
- */
-function isTextFile(filePath) {
-    try {
-        const buffer = Buffer.alloc(8192);
-        const fd = fs.openSync(filePath, 'r');
-        const bytesRead = fs.readSync(fd, buffer, 0, 8192, 0);
-        fs.closeSync(fd);
-        // Check for null bytes (indicates binary file)
-        for (let i = 0; i < bytesRead; i++) {
-            if (buffer[i] === 0)
-                return false;
-        }
-        return true;
-    }
-    catch {
-        // If we can't read the file, consider it non-text
-        return false;
-    }
-}
 /**
  * Validate file for --files flag in brv curate command
  * Checks:
  * 1. File exists
  * 2. File is within project directory (where .brv exists)
- * 3. File is text/code file (not binary)
+ * 3. File is supported by read_file tool (text, images, or PDFs)
  *
  * @param filePath - The file path to validate (can be relative, absolute, or tilde)
  * @param projectRoot - The project root directory (current working directory)
  * @returns Validation result with normalized path or error message
  */
 export function validateFileForCurate(filePath, projectRoot) {
-    // Normalize path
-    const normalized = normalizeFilePath(filePath);
+    // Normalize projectRoot first to ensure consistent behavior
+    // This handles cases where projectRoot might be relative
+    const normalizedProjectRoot = normalizeFilePath(projectRoot);
+    // Normalize file path using normalizedProjectRoot as base for relative paths
+    const normalized = normalizeFilePath(filePath, normalizedProjectRoot);
     // Check existence
     if (!fs.existsSync(normalized)) {
         return { error: `File does not exist: ${filePath}`, valid: false };
@@ -68,14 +48,33 @@ export function validateFileForCurate(filePath, projectRoot) {
     if (!stats.isFile()) {
         return { error: `Path is not a file: ${filePath}`, valid: false };
     }
-    // Check within project (normalized paths for reliable comparison)
-    const normalizedProjectRoot = normalizeFilePath(projectRoot);
+    // Check within project (both paths are already normalized)
     if (!normalized.startsWith(normalizedProjectRoot + path.sep) && normalized !== normalizedProjectRoot) {
         return { error: `File is outside project directory: ${filePath}`, valid: false };
     }
-    // Check is text file
-    if (!isTextFile(normalized)) {
-        return { error: `File is not a text/code file (binary detected): ${filePath}`, valid: false };
+    // Read sample buffer for file type detection
+    let buffer;
+    let bytesRead;
+    try {
+        buffer = Buffer.alloc(4096);
+        const fd = fs.openSync(normalized, 'r');
+        bytesRead = fs.readSync(fd, buffer, 0, 4096, 0);
+        fs.closeSync(fd);
+    }
+    catch {
+        return { error: `Cannot read file: ${filePath}`, valid: false };
+    }
+    const sampleBuffer = buffer.subarray(0, bytesRead);
+    // Check file type using binary-utils (same logic as read_file tool)
+    // Allow media files (images/PDFs) - read_file can handle these
+    // For PDFs, also validate magic bytes to reject fake PDFs (e.g., binary.pdf)
+    if (isMediaFile(normalized, sampleBuffer)) {
+        return { normalizedPath: normalized, valid: true };
+    }
+    // Check if it's a binary file (using same logic as read_file tool)
+    if (isBinaryFile(normalized, sampleBuffer)) {
+        return { error: `File type not supported: ${filePath}`, valid: false };
     }
+    // It's a text file - supported
     return { normalizedPath: normalized, valid: true };
 }

package/dist/utils/type-guards.d.ts

@@ -0,0 +1,5 @@
+/**
+ * Type guard to check if value is a non-null object (Record).
+ * Useful for safely narrowing unknown values before accessing properties.
+ */
+export declare function isRecord(value: unknown): value is Record<string, unknown>;

package/dist/utils/type-guards.js

@@ -0,0 +1,7 @@
+/**
+ * Type guard to check if value is a non-null object (Record).
+ * Useful for safely narrowing unknown values before accessing properties.
+ */
+export function isRecord(value) {
+    return typeof value === 'object' && value !== null && !Array.isArray(value);
+}

package/oclif.manifest.json

@@ -9,7 +9,7 @@
           "required": false
         }
       },
-      "description": "Curate context to the context tree (connects to running brv instance)\n\nRequires a running brv instance. Start one with: brv start\n\nGood examples:\n- \"Auth uses JWT with 24h expiry. Tokens stored in httpOnly cookies via authMiddleware.ts\"\n- \"API rate limit is 100 req/min per user. Implemented using Redis with sliding window in rateLimiter.ts\"\nBad examples:\n- \"Authentication\" or \"JWT tokens\" (too vague, lacks context)\n- \"Rate limiting\" (no implementation details or file references)",
+      "description": "Curate context to the context tree (connects to running brv instance)\n\nRequires a running brv instance. Start one with: brv\n\nGood examples:\n- \"Auth uses JWT with 24h expiry. Tokens stored in httpOnly cookies via authMiddleware.ts\"\n- \"API rate limit is 100 req/min per user. Implemented using Redis with sliding window in rateLimiter.ts\"\nBad examples:\n- \"Authentication\" or \"JWT tokens\" (too vague, lacks context)\n- \"Rate limiting\" (no implementation details or file references)",
       "examples": [
         "# Curate context - queues task for background processing",
         "<%= config.bin %> <%= command.id %> \"Auth uses JWT with 24h expiry. Tokens stored in httpOnly cookies via authMiddleware.ts\"",
@@ -45,6 +45,27 @@
         "curate.js"
       ]
     },
+    "hook-prompt-submit": {
+      "aliases": [],
+      "args": {},
+      "description": "Internal: Pre-prompt hook for coding agents",
+      "flags": {},
+      "hasDynamicHelp": false,
+      "hidden": true,
+      "hiddenAliases": [],
+      "id": "hook-prompt-submit",
+      "pluginAlias": "byterover-cli",
+      "pluginName": "byterover-cli",
+      "pluginType": "core",
+      "strict": true,
+      "enableJsonFlag": false,
+      "isESM": true,
+      "relativePath": [
+        "dist",
+        "commands",
+        "hook-prompt-submit.js"
+      ]
+    },
     "main": {
       "aliases": [],
       "args": {},
@@ -75,7 +96,7 @@
           "required": true
         }
       },
-      "description": "Query and retrieve information from the context tree (connects to running brv instance)\n\nRequires a running brv instance. Start one with: brv start\n\nGood:\n- \"How is user authentication implemented?\"\n- \"What are the API rate limits and where are they enforced?\"\nBad:\n- \"auth\" or \"authentication\" (too vague, not a question)\n- \"show me code\" (not specific about what information is needed)",
+      "description": "Query and retrieve information from the context tree (connects to running brv instance)\n\nRequires a running brv instance. Start one with: brv\n\nGood:\n- \"How is user authentication implemented?\"\n- \"What are the API rate limits and where are they enforced?\"\nBad:\n- \"auth\" or \"authentication\" (too vague, not a question)\n- \"show me code\" (not specific about what information is needed)",
       "examples": [
         "# Ask questions about patterns, decisions, or implementation details",
         "<%= config.bin %> <%= command.id %> What are the coding standards?",
@@ -109,9 +130,14 @@
       "description": "Show CLI status and project information. Display local context tree managed by ByteRover CLI",
       "examples": [
         "<%= config.bin %> <%= command.id %>",
-        "# Check status after login:\n<%= config.bin %> login\n<%= config.bin %> <%= command.id %>",
-        "# Verify project initialization:\n<%= config.bin %> init\n<%= config.bin %> <%= command.id %>",
-        "<%= config.bin %> <%= command.id %>",
+        "# Check status after login (in REPL):",
+        "/login",
+        "/status",
+        "",
+        "# Verify project initialization (in REPL):",
+        "/init",
+        "/status",
+        "",
         "<%= config.bin %> <%= command.id %> /path/to/project",
         "<%= config.bin %> <%= command.id %> --format json"
       ],
@@ -198,5 +224,5 @@
       ]
     }
   },
-  "version": "1.0.4"
+  "version": "1.1.0"
 }

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "byterover-cli",
   "description": "ByteRover's CLI",
-  "version": "1.0.4",
+  "version": "1.1.0",
   "author": "ByteRover",
   "bin": {
     "brv": "./bin/run.js"
@@ -32,6 +32,7 @@
     "inquirer-file-selector": "^1.0.1",
     "js-yaml": "^4.1.1",
     "keytar": "^7.9.0",
+    "minisearch": "^7.2.0",
     "mixpanel": "^0.18.1",
     "nanoid": "^5.1.6",
     "open": "^10.2.0",
@@ -39,6 +40,7 @@
     "react": "^19.2.1",
     "socket.io": "^4.8.1",
     "socket.io-client": "^4.8.1",
+    "stopword": "^3.1.5",
     "update-notifier": "^7.3.1",
     "zod": "^3.25.76",
     "zod-to-json-schema": "^3.24.6"
@@ -55,6 +57,7 @@
     "@types/node": "^20",
     "@types/react": "^19.2.7",
     "@types/sinon": "^17.0.4",
+    "@types/stopword": "^2.0.3",
     "chai": "^4",
     "eslint": "^9",
     "eslint-config-oclif": "^6",

package/dist/config/context-tree-domains.d.ts

@@ -1,29 +0,0 @@
-/**
- * Domain configurations for the context tree structure.
- * Each domain represents a specific area of knowledge in the project.
- *
- * @deprecated Domains are now created dynamically based on content semantics.
- * This file is kept for backward compatibility only.
- */
-export interface DomainConfig {
-    description: string;
-    name: string;
-}
-/**
- * Example domain names for reference only.
- * Domains are now created dynamically by the agent based on content.
- *
- * @deprecated Domains are created dynamically. These are kept as examples only.
- */
-export declare const EXAMPLE_DOMAIN_NAMES: readonly ["authentication", "api_design", "data_models", "error_handling", "ui_components", "testing_patterns", "configuration", "logging", "security", "performance"];
-/**
- * @deprecated Domains are now created dynamically based on content semantics.
- * The agent will create domain names that are semantically meaningful for the curated content.
- * This constant is kept for backward compatibility but is no longer used.
- */
-export declare const DEFAULT_CONTEXT_TREE_DOMAINS: DomainConfig[];
-/**
- * Alias for backward compatibility.
- * @deprecated Domains are created dynamically. This constant is no longer used.
- */
-export declare const CONTEXT_TREE_DOMAINS: DomainConfig[];

package/dist/config/context-tree-domains.js

@@ -1,29 +0,0 @@
-/**
- * Example domain names for reference only.
- * Domains are now created dynamically by the agent based on content.
- *
- * @deprecated Domains are created dynamically. These are kept as examples only.
- */
-export const EXAMPLE_DOMAIN_NAMES = [
-    'authentication',
-    'api_design',
-    'data_models',
-    'error_handling',
-    'ui_components',
-    'testing_patterns',
-    'configuration',
-    'logging',
-    'security',
-    'performance',
-];
-/**
- * @deprecated Domains are now created dynamically based on content semantics.
- * The agent will create domain names that are semantically meaningful for the curated content.
- * This constant is kept for backward compatibility but is no longer used.
- */
-export const DEFAULT_CONTEXT_TREE_DOMAINS = [];
-/**
- * Alias for backward compatibility.
- * @deprecated Domains are created dynamically. This constant is no longer used.
- */
-export const CONTEXT_TREE_DOMAINS = DEFAULT_CONTEXT_TREE_DOMAINS;

package/dist/core/interfaces/usecase/i-generate-rules-use-case.d.ts

@@ -1,3 +0,0 @@
-export interface IGenerateRulesUseCase {
-    run(): Promise<void>;
-}

package/dist/infra/repl/commands/gen-rules-command.d.ts

@@ -1,7 +0,0 @@
-import { SlashCommand } from '../../../tui/types.js';
-/**
- * Generate rules command
- *
- * Creates and runs GenerateRulesUseCase with ReplTerminal for TUI integration.
- */
-export declare const genRulesCommand: SlashCommand;

package/dist/infra/rule/agent-rule-config.d.ts

@@ -1,19 +0,0 @@
-import { type Agent } from '../../core/domain/entities/agent.js';
-import { type WriteMode } from '../../core/interfaces/i-file-service.js';
-/**
- * Configuration for agent-specific rule files.
- */
-export type AgentRuleConfig = {
-    /**
-     * The file path where the agent's rules should be written.
-     */
-    filePath: string;
-    /**
-     * The write mode to use when writing the rule file.
-     */
-    writeMode: WriteMode;
-};
-/**
- * Mapping of agents to their rule file configurations.
- */
-export declare const AGENT_RULE_CONFIGS: Record<Agent, AgentRuleConfig>;

package/dist/infra/usecase/generate-rules-use-case.d.ts

@@ -1,61 +0,0 @@
-import type { Agent } from '../../core/domain/entities/agent.js';
-import type { IFileService } from '../../core/interfaces/i-file-service.js';
-import type { IRuleTemplateService } from '../../core/interfaces/i-rule-template-service.js';
-import type { ITerminal } from '../../core/interfaces/i-terminal.js';
-import type { ITrackingService } from '../../core/interfaces/i-tracking-service.js';
-import type { IGenerateRulesUseCase } from '../../core/interfaces/usecase/i-generate-rules-use-case.js';
-import { LegacyRuleDetector } from '../rule/legacy-rule-detector.js';
-type CleanupStrategy = 'automatic' | 'manual';
-export declare class GenerateRulesUseCase implements IGenerateRulesUseCase {
-    private readonly fileService;
-    private readonly legacyRuleDetector;
-    private readonly templateService;
-    private readonly terminal;
-    private readonly trackingService;
-    constructor(fileService: IFileService, legacyRuleDetector: LegacyRuleDetector, templateService: IRuleTemplateService, terminal: ITerminal, trackingService: ITrackingService);
-    /**
-     * Prompts the user to select an agent.
-     * @returns The selected agent
-     */
-    protected promptForAgentSelection(): Promise<Agent>;
-    /**
-     * Prompts the user to choose cleanup strategy for legacy rules.
-     * @returns The chosen cleanup strategy
-     */
-    protected promptForCleanupStrategy(): Promise<CleanupStrategy>;
-    /**
-     * Prompts the user to create a new rule file.
-     * This method is protected to allow test overrides.
-     * @param agent The agent for which the rule file doesn't exist
-     * @param filePath The path where the file would be created
-     * @returns True if the user wants to create the file, false otherwise
-     */
-    protected promptForFileCreation(agent: Agent, filePath: string): Promise<boolean>;
-    /**
-     * Prompts the user to confirm overwriting an existing rule file.
-     * This method is protected to allow test overrides.
-     * @param agent The agent for which the rule file exists
-     * @returns True if the user confirms overwrite, false otherwise
-     */
-    protected promptForOverwriteConfirmation(agent: Agent): Promise<boolean>;
-    run(): Promise<void>;
-    /**
-     * Appends ByteRover rules to a file that has no ByteRover content.
-     */
-    private appendRulesToFile;
-    /**
-     * Creates a new rule file with ByteRover rules.
-     */
-    private createNewRuleFile;
-    /**
-     * Handles legacy rules cleanup with user choice of automatic or manual.
-     */
-    private handleLegacyRulesCleanup;
-    private performAutomaticCleanup;
-    private performManualCleanup;
-    /**
-     * Replaces existing ByteRover rules (with boundary markers) with new rules.
-     */
-    private replaceExistingRules;
-}
-export {};