Feat: CLI optimization (#389)

* feat: remove python cli and disable python -m openviking

* feat: ls, tree, find, search, grep all use --node-limit as the result limiting arg

* feat: add ls -n

* feat: update agfs to support grep -n

* feat: update agfs to support grep -n

* docs: cancel modify

---------

Co-authored-by: openviking <openviking@example.com>

Author: MaojiaSheng

crates/ov_cli/src/client.rs

@@ -369,13 +369,13 @@ impl HttpClient {
         &self,
         query: String,
         uri: String,
-        limit: i32,
+        node_limit: i32,
         threshold: Option<f64>,
     ) -> Result<serde_json::Value> {
         let body = serde_json::json!({
             "query": query,
             "target_uri": uri,
-            "limit": limit,
+            "limit": node_limit,
             "score_threshold": threshold,
         });
         self.post("/api/v1/search/find", &body).await
@@ -386,28 +386,30 @@ impl HttpClient {
         query: String,
         uri: String,
         session_id: Option<String>,
-        limit: i32,
+        node_limit: i32,
         threshold: Option<f64>,
     ) -> Result<serde_json::Value> {
         let body = serde_json::json!({
             "query": query,
             "target_uri": uri,
             "session_id": session_id,
-            "limit": limit,
+            "limit": node_limit,
             "score_threshold": threshold,
         });
         self.post("/api/v1/search/search", &body).await
     }
 
-    pub async fn grep(&self, uri: &str, pattern: &str, ignore_case: bool) -> Result<serde_json::Value> {
+    pub async fn grep(&self, uri: &str, pattern: &str, ignore_case: bool, node_limit: i32) -> Result<serde_json::Value> {
         let body = serde_json::json!({
             "uri": uri,
             "pattern": pattern,
             "case_insensitive": ignore_case,
+            "node_limit": node_limit,
         });
         self.post("/api/v1/search/grep", &body).await
     }
 
+
     pub async fn glob(&self, pattern: &str, uri: &str) -> Result<serde_json::Value> {
         let body = serde_json::json!({
             "pattern": pattern,

crates/ov_cli/src/commands/search.rs

@@ -6,12 +6,12 @@ pub async fn find(
     client: &HttpClient,
     query: &str,
     uri: &str,
-    limit: i32,
+    node_limit: i32,
     threshold: Option<f64>,
     output_format: OutputFormat,
     compact: bool,
 ) -> Result<()> {
-    let result = client.find(query.to_string(), uri.to_string(), limit, threshold).await?;
+    let result = client.find(query.to_string(), uri.to_string(), node_limit, threshold).await?;
     output_success(&result, output_format, compact);
     Ok(())
 }
@@ -21,12 +21,12 @@ pub async fn search(
     query: &str,
     uri: &str,
     session_id: Option<String>,
-    limit: i32,
+    node_limit: i32,
     threshold: Option<f64>,
     output_format: OutputFormat,
     compact: bool,
 ) -> Result<()> {
-    let result = client.search(query.to_string(), uri.to_string(), session_id, limit, threshold).await?;
+    let result = client.search(query.to_string(), uri.to_string(), session_id, node_limit, threshold).await?;
     output_success(&result, output_format, compact);
     Ok(())
 }
@@ -36,14 +36,16 @@ pub async fn grep(
     uri: &str,
     pattern: &str,
     ignore_case: bool,
+    node_limit: i32,
     output_format: OutputFormat,
     compact: bool,
 ) -> Result<()> {
-    let result = client.grep(uri, pattern, ignore_case).await?;
+    let result = client.grep(uri, pattern, ignore_case, node_limit).await?;
     output_success(&result, output_format, compact);
     Ok(())
 }
 
+
 pub async fn glob(
     client: &HttpClient,
     pattern: &str,

crates/ov_cli/src/main.rs

@@ -195,7 +195,7 @@ enum Commands {
         #[arg(short, long)]
         all: bool,
         /// Maximum number of nodes to list
-        #[arg(long = "node-limit", short = 'n', default_value = "256")]
+        #[arg(long = "node-limit", short = 'n', alias = "limit", default_value = "256")]
         node_limit: i32,
     },
     /// Get directory tree
@@ -209,7 +209,7 @@ enum Commands {
         #[arg(short, long)]
         all: bool,
         /// Maximum number of nodes to list
-        #[arg(long = "node-limit", short = 'n', default_value = "256")]
+        #[arg(long = "node-limit", short = 'n', alias = "limit", default_value = "256")]
         node_limit: i32,
         /// Maximum depth level to traverse (default: 3)
         #[arg(short = 'L', long = "level-limit", default_value = "3")]
@@ -265,8 +265,8 @@ enum Commands {
         #[arg(short, long, default_value = "")]
         uri: String,
         /// Maximum number of results
-        #[arg(short = 'n', long, default_value = "10")]
-        limit: i32,
+        #[arg(short = 'n', long = "node-limit", alias = "limit", default_value = "10")]
+        node_limit: i32,
         /// Score threshold
         #[arg(short, long)]
         threshold: Option<f64>,
@@ -282,8 +282,8 @@ enum Commands {
         #[arg(long)]
         session_id: Option<String>,
         /// Maximum number of results
-        #[arg(short = 'n', long, default_value = "10")]
-        limit: i32,
+        #[arg(short = 'n', long = "node-limit", alias = "limit", default_value = "10")]
+        node_limit: i32,
         /// Score threshold
         #[arg(short, long)]
         threshold: Option<f64>,
@@ -298,6 +298,9 @@ enum Commands {
         /// Case insensitive
         #[arg(short, long)]
         ignore_case: bool,
+        /// Maximum number of results
+        #[arg(short = 'n', long = "node-limit", alias = "limit", default_value = "256")]
+        node_limit: i32,
     },
     /// Run file glob pattern search
     Glob {
@@ -562,15 +565,16 @@ async fn main() {
         Commands::Read { uri } => handle_read(uri, ctx).await,
         Commands::Abstract { uri } => handle_abstract(uri, ctx).await,
         Commands::Overview { uri } => handle_overview(uri, ctx).await,
-        Commands::Find { query, uri, limit, threshold } => {
-            handle_find(query, uri, limit, threshold, ctx).await
+        Commands::Find { query, uri, node_limit, threshold } => {
+            handle_find(query, uri, node_limit, threshold, ctx).await
         }
-        Commands::Search { query, uri, session_id, limit, threshold } => {
-            handle_search(query, uri, session_id, limit, threshold, ctx).await
+        Commands::Search { query, uri, session_id, node_limit, threshold } => {
+            handle_search(query, uri, session_id, node_limit, threshold, ctx).await
         }
-        Commands::Grep { uri, pattern, ignore_case } => {
-            handle_grep(uri, pattern, ignore_case, ctx).await
+        Commands::Grep { uri, pattern, ignore_case, node_limit } => {
+            handle_grep(uri, pattern, ignore_case, node_limit, ctx).await
         }
+
         Commands::Glob { pattern, uri } => {
             handle_glob(pattern, uri, ctx).await
         }
@@ -860,24 +864,24 @@ async fn handle_overview(uri: String, ctx: CliContext) -> Result<()> {
 async fn handle_find(
     query: String,
     uri: String,
-    limit: i32,
+    node_limit: i32,
     threshold: Option<f64>,
     ctx: CliContext,
 ) -> Result<()> {
     let client = ctx.get_client();
-    commands::search::find(&client, &query, &uri, limit, threshold, ctx.output_format, ctx.compact).await
+    commands::search::find(&client, &query, &uri, node_limit, threshold, ctx.output_format, ctx.compact).await
 }
 
 async fn handle_search(
     query: String,
     uri: String,
     session_id: Option<String>,
-    limit: i32,
+    node_limit: i32,
     threshold: Option<f64>,
     ctx: CliContext,
 ) -> Result<()> {
     let client = ctx.get_client();
-    commands::search::search(&client, &query, &uri, session_id, limit, threshold, ctx.output_format, ctx.compact).await
+    commands::search::search(&client, &query, &uri, session_id, node_limit, threshold, ctx.output_format, ctx.compact).await
 }
 
 /// Print command with specified parameters for debugging
@@ -938,11 +942,12 @@ async fn handle_stat(uri: String, ctx: CliContext) -> Result<()> {
     commands::filesystem::stat(&client, &uri, ctx.output_format, ctx.compact).await
 }
 
-async fn handle_grep(uri: String, pattern: String, ignore_case: bool, ctx: CliContext) -> Result<()> {
+async fn handle_grep(uri: String, pattern: String, ignore_case: bool, node_limit: i32, ctx: CliContext) -> Result<()> {
     let client = ctx.get_client();
-    commands::search::grep(&client, &uri, &pattern, ignore_case, ctx.output_format, ctx.compact).await
+    commands::search::grep(&client, &uri, &pattern, ignore_case, node_limit, ctx.output_format, ctx.compact).await
 }
 
+
 async fn handle_glob(pattern: String, uri: String, ctx: CliContext) -> Result<()> {
     let client = ctx.get_client();
     commands::search::glob(&client, &pattern, &uri, ctx.output_format, ctx.compact).await