Refactor(kclvm-tools): move 'printer' from 'kclvm-tools' to 'kclvm-ast'.

kclvm/Cargo.toml

@@ -48,6 +48,7 @@ lto = true
 members = [
     "capi",
     "ast",
+    "ast_pretty",
     "compiler",
     "config",
     "error",

kclvm/ast_pretty/Cargo.toml

@@ -0,0 +1,14 @@
+[package]
+name = "kclvm-ast-pretty"
+version = "0.1.0"
+edition = "2021"
+
+# See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html
+
+[dependencies]
+kclvm-parser = {path = "../parser", version = "0.1.0"}
+kclvm-error = {path = "../error", version = "0.1.0"}
+kclvm-ast = {path = "../ast", version = "0.1.0"}
+indexmap = "1.0"
+fancy-regex = "0.7.1"
+pretty_assertions = "1.3.0"

kclvm/tools/src/printer/mod.rs → kclvm/ast_pretty/src/lib.rs

@@ -1,8 +1,11 @@
 use indexmap::IndexMap;
+use kclvm_ast::{
+    ast::{self, Module},
+    token::TokenKind,
+    walker::MutSelfTypedResultWalker,
+};
 use std::collections::VecDeque;
 
-use kclvm_ast::{ast, token::TokenKind, walker::MutSelfTypedResultWalker};
-
 mod node;
 
 #[cfg(test)]
@@ -244,7 +247,7 @@ impl<'p> Printer<'p> {
 }
 
 /// Print AST to string
-pub fn print_ast_module(module: &ast::Module) -> String {
+pub fn print_ast_module(module: &Module) -> String {
     let mut printer = Printer::default();
     printer.write_module(module);
     printer.out

kclvm/tools/src/printer/node.rs → kclvm/ast_pretty/src/node.rs

@@ -1,8 +1,11 @@
 use std::collections::HashSet;
 
-use kclvm_ast::ast::{self, CallExpr};
-use kclvm_ast::token::{DelimToken, TokenKind};
-use kclvm_ast::walker::MutSelfTypedResultWalker;
+use kclvm_ast::{
+    ast::{self, CallExpr},
+    token::{DelimToken, TokenKind},
+    walker::MutSelfTypedResultWalker,
+};
+use kclvm_error::bug;
 
 use super::{Indentation, Printer};
 

kclvm/tools/src/printer/tests.rs → kclvm/ast_pretty/src/tests.rs

@@ -1,4 +1,6 @@
-use super::*;
+use std::path::PathBuf;
+
+use super::print_ast_module;
 use kclvm_parser::parse_file;
 use pretty_assertions::assert_eq;
 
@@ -23,18 +25,19 @@ const TEST_CASES: &[&'static str; 15] = &[
 ];
 
 fn read_data(data_name: &str) -> (String, String) {
-    let module = parse_file(
-        &format!("./src/printer/test_data/{}{}", data_name, FILE_INPUT_SUFFIX),
-        None,
-    );
+    let mut filename = PathBuf::from(env!("CARGO_MANIFEST_DIR"));
+    filename.push(&format!("src/test_data/{}{}", data_name, FILE_INPUT_SUFFIX));
+
+    let module = parse_file(filename.to_str().unwrap(), None);
 
+    let mut filename_expect = PathBuf::from(env!("CARGO_MANIFEST_DIR"));
+    filename_expect.push(&format!(
+        "src/test_data/{}{}",
+        data_name, FILE_OUTPUT_SUFFIX
+    ));
     (
         print_ast_module(&module.unwrap()),
-        std::fs::read_to_string(&format!(
-            "./src/printer/test_data/{}{}",
-            data_name, FILE_OUTPUT_SUFFIX
-        ))
-        .unwrap(),
+        std::fs::read_to_string(filename_expect.to_str().unwrap()).unwrap(),
     )
 }
 

kclvm/tools/Cargo.toml

@@ -16,6 +16,8 @@ kclvm-error = {path = "../error", version = "0.1.0"}
 kclvm-parser = {path = "../parser", version = "0.1.0"}
 kclvm-sema = {path = "../sema", version = "0.1.0"}
 kclvm-config = {path = "../config", version = "0.1.0"}
+kclvm-ast-pretty = {path = "../ast_pretty", version = "0.1.0"}
+
 serde_json = "1.0.85"
 serde_yaml = "0.9.13"
 

kclvm/tools/src/format/mod.rs

@@ -6,9 +6,10 @@
 //! AST Module, and then use the AST printer [kclvm_tools::printer::print_ast_module]
 //! to print it as source code string.
 use anyhow::{anyhow, Result};
+use kclvm_ast_pretty::print_ast_module;
 use std::path::Path;
 
-use crate::{printer::print_ast_module, util::get_kcl_files};
+use crate::util::get_kcl_files;
 use kclvm_parser::parse_file;
 
 #[cfg(test)]

kclvm/tools/src/lib.rs

@@ -1,6 +1,5 @@
 pub mod format;
 pub mod lint;
-pub mod printer;
 pub mod query;
 mod util;
 pub mod vet;

kclvm/tools/src/query/mod.rs

@@ -11,12 +11,12 @@ mod util;
 
 use anyhow::{anyhow, Result};
 use kclvm_ast::ast;
+use kclvm_ast_pretty::print_ast_module;
 use kclvm_parser::parse_file;
 
 pub use r#override::{apply_override_on_module, apply_overrides};
 
 use self::r#override::parse_override_spec;
-use crate::printer::print_ast_module;
 
 /// Override and rewrite a file with override specifications. Please note that this is an external user API,
 /// and it can directly modify the KCL file in place.

kclvm/tools/src/query/override.rs

@@ -6,11 +6,10 @@ use kclvm_ast::config::try_get_config_expr_mut;
 use kclvm_ast::path::{get_attr_paths_from_config_expr, get_key_path};
 use kclvm_ast::walker::MutSelfMutWalker;
 use kclvm_ast::{ast, walk_if_mut};
+use kclvm_ast_pretty::print_ast_module;
 use kclvm_parser::parse_expr;
 use kclvm_sema::pre_process::{fix_config_expr_nest_attr, transform_multi_assign};
 
-use crate::printer::print_ast_module;
-
 use super::util::{invalid_spec_error, split_field_path};
 
 /// Import statement column offset always start with 1.

kclvm/tools/src/query/tests.rs

@@ -1,5 +1,4 @@
 use super::{r#override::apply_override_on_module, *};
-use crate::printer::print_ast_module;
 use kclvm_ast::ast;
 use kclvm_parser::parse_file;
 use pretty_assertions::assert_eq;