[BUG] canonicalize Path Fallback May Return Relative Path

[xyz080125]

Project

vgrep

Description

In search.rs line 43, std::fs::canonicalize(path).unwrap_or_else(|_| path.to_path_buf()) falls back to the original path if canonicalization fails. If the original path is relative, this could cause issues with database queries that expect absolute paths.

Error Message

Debug Logs

System Information

OS: Any
Rust Version: 1.75+

Screenshots

No response

Steps to Reproduce

1. Search with a relative path that doesn't exist yet (e.g., a new directory)
2. Observe query behavior

Expected Behavior

Either fail clearly if path doesn't exist, or convert to absolute path using current_dir

Actual Behavior

May use relative path for database query, causing incorrect path matching

Additional Context

No response

[echobt]

Marking as invalid: A duplicate PR already exists in the target repository addressing this issue: CortexLM/vgrep#69. The existing PR modifies the exact same line in 'src/core/search.rs' to handle canonicalization errors correctly.

[echobt]

Thank you for your report. After careful review, we are unable to validate this issue for the following reason: OUT OF SCOPE - Issue is about vgrep project, not Cortex CLI. This issue explicitly targets the vgrep project (CortexLM/vgrep), referencing search.rs canonicalize path handling from vgrep codebase. This is outside the scope of our validation for Cortex CLI, web-auth, and web-app components. Additionally, this issue is already CLOSED with an existing PR #69 addressing it. This issue does not meet our validation criteria for the Cortex CLI project. If you believe this is an error, please provide additional details or reproduction steps.