ast-grep-cli 0.32.2__tar.gz → 0.32.3__tar.gz

{ast_grep_cli-0.32.2 → ast_grep_cli-0.32.3}/Cargo.lock

@@ -92,9 +92,9 @@ dependencies = [
 
 [[package]]
 name = "anyhow"
-version = "1.0.94"
+version = "1.0.95"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "c1fd03a028ef38ba2276dce7e33fcd6369c158a1bca17946c4b1b701891c1ff7"
+checksum = "34ac096ce696dc2fcabef30516bb13c0a68a11d30131d3df6f04711467681b04"
 
 [[package]]
 name = "assert_cmd"
@@ -114,7 +114,7 @@ dependencies = [
 
 [[package]]
 name = "ast-grep"
-version = "0.32.2"
+version = "0.32.3"
 dependencies = [
  "ansi_term",
  "anyhow",
@@ -144,7 +144,7 @@ dependencies = [
 
 [[package]]
 name = "ast-grep-config"
-version = "0.32.2"
+version = "0.32.3"
 dependencies = [
  "anyhow",
  "ast-grep-core",
@@ -160,7 +160,7 @@ dependencies = [
 
 [[package]]
 name = "ast-grep-core"
-version = "0.32.2"
+version = "0.32.3"
 dependencies = [
  "bit-set",
  "regex",
@@ -171,7 +171,7 @@ dependencies = [
 
 [[package]]
 name = "ast-grep-dynamic"
-version = "0.32.2"
+version = "0.32.3"
 dependencies = [
  "ast-grep-core",
  "ignore",
@@ -183,7 +183,7 @@ dependencies = [
 
 [[package]]
 name = "ast-grep-language"
-version = "0.32.2"
+version = "0.32.3"
 dependencies = [
  "ast-grep-core",
  "ignore",
@@ -215,7 +215,7 @@ dependencies = [
 
 [[package]]
 name = "ast-grep-lsp"
-version = "0.32.2"
+version = "0.32.3"
 dependencies = [
  "ast-grep-config",
  "ast-grep-core",
@@ -229,7 +229,7 @@ dependencies = [
 
 [[package]]
 name = "ast-grep-napi"
-version = "0.32.2"
+version = "0.32.3"
 dependencies = [
  "ast-grep-config",
  "ast-grep-core",
@@ -244,7 +244,7 @@ dependencies = [
 
 [[package]]
 name = "ast-grep-py"
-version = "0.32.2"
+version = "0.32.3"
 dependencies = [
  "anyhow",
  "ast-grep-config",
@@ -312,7 +312,7 @@ dependencies = [
 
 [[package]]
 name = "benches"
-version = "0.32.2"
+version = "0.32.3"
 dependencies = [
  "ast-grep-config",
  "ast-grep-core",
@@ -448,9 +448,9 @@ dependencies = [
 
 [[package]]
 name = "clap_complete"
-version = "4.5.39"
+version = "4.5.40"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "fd4db298d517d5fa00b2b84bbe044efd3fde43874a41db0d46f91994646a2da4"
+checksum = "ac2e663e3e3bed2d32d065a8404024dad306e699a04263ec59919529f803aee9"
 dependencies = [
  "clap",
 ]
@@ -691,9 +691,9 @@ checksum = "e8c02a5121d4ea3eb16a80748c74f5549a5665e4c21333c6098f283870fbdea6"
 
 [[package]]
 name = "float-cmp"
-version = "0.9.0"
+version = "0.10.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "98de4bbd547a563b716d8dfa9aad1cb19bfab00f4fa09a6a4ed21dbcf44ce9c4"
+checksum = "b09cf3155332e944990140d967ff5eceb70df778b34f77d8075db46e4704e6d8"
 dependencies = [
  "num-traits",
 ]
@@ -1266,9 +1266,9 @@ checksum = "7170ef9988bc169ba16dd36a7fa041e5c4cbeb6a35b76d4c03daded371eae7c0"
 
 [[package]]
 name = "predicates"
-version = "3.1.2"
+version = "3.1.3"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "7e9086cc7640c29a356d1a29fd134380bee9d8f79a17410aa76e7ad295f42c97"
+checksum = "a5d19ee57562043d37e82899fade9a22ebab7be9cef5026b07fda9cdd4293573"
 dependencies = [
  "anstyle",
  "difflib",
@@ -1547,9 +1547,9 @@ dependencies = [
 
 [[package]]
 name = "serde_json"
-version = "1.0.133"
+version = "1.0.134"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "c7fceb2473b9166b2294ef05efcb65a3db80803f0b03ef86a5fc88a2b85ee377"
+checksum = "d00f4175c42ee48b15416f6193a959ba3a0d67fc699a0db9ad12df9f83991c7d"
 dependencies = [
  "indexmap",
  "itoa",
@@ -1699,18 +1699,18 @@ checksum = "3369f5ac52d5eb6ab48c6b4ffdc8efbcad6b89c765749064ba298f2c68a16a76"
 
 [[package]]
 name = "thiserror"
-version = "2.0.6"
+version = "2.0.9"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "8fec2a1820ebd077e2b90c4df007bebf344cd394098a13c563957d0afc83ea47"
+checksum = "f072643fd0190df67a8bab670c20ef5d8737177d6ac6b2e9a236cb096206b2cc"
 dependencies = [
  "thiserror-impl",
 ]
 
 [[package]]
 name = "thiserror-impl"
-version = "2.0.6"
+version = "2.0.9"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "d65750cab40f4ff1929fb1ba509e9914eb756131cef4210da8d5d700d26f6312"
+checksum = "7b50fa271071aae2e6ee85f842e2e28ba8cd2c5fb67f11fcb1fd70b276f9e7d4"
 dependencies = [
  "proc-macro2",
  "quote",

{ast_grep_cli-0.32.2 → ast_grep_cli-0.32.3}/Cargo.toml

@@ -7,7 +7,7 @@ resolver = "2"
 lto = true
 
 [workspace.package]
-version = "0.32.2"
+version = "0.32.3"
 authors = ["Herrington Darkholme <2883231+HerringtonDarkholme@users.noreply.github.com>"]
 edition = "2021"
 license = "MIT"
@@ -18,11 +18,11 @@ rust-version = "1.67"
 readme = "README.md"
 
 [workspace.dependencies]
-ast-grep-core = { path = "crates/core", version = "0.32.2" }
-ast-grep-config = { path = "crates/config", version = "0.32.2" }
-ast-grep-dynamic = { path = "crates/dynamic", version = "0.32.2" }
-ast-grep-language = { path = "crates/language", version = "0.32.2" }
-ast-grep-lsp = { path = "crates/lsp", version = "0.32.2" }
+ast-grep-core = { path = "crates/core", version = "0.32.3" }
+ast-grep-config = { path = "crates/config", version = "0.32.3" }
+ast-grep-dynamic = { path = "crates/dynamic", version = "0.32.3" }
+ast-grep-language = { path = "crates/language", version = "0.32.3" }
+ast-grep-lsp = { path = "crates/lsp", version = "0.32.3" }
 
 bit-set = { version = "0.8.0" }
 ignore = { version = "0.4.22" }

{ast_grep_cli-0.32.2 → ast_grep_cli-0.32.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: ast-grep-cli
-Version: 0.32.2
+Version: 0.32.3
 Classifier: Development Status :: 3 - Alpha
 Classifier: Environment :: Console
 Classifier: Intended Audience :: Developers

{ast_grep_cli-0.32.2 → ast_grep_cli-0.32.3}/crates/cli/src/print/cloud_print.rs

@@ -7,7 +7,6 @@ use anyhow::Result;
 use ast_grep_core::{NodeMatch as SgNodeMatch, StrDoc};
 use codespan_reporting::files::SimpleFile;
 use std::io::{Stdout, Write};
-use std::sync::Mutex;
 
 type NodeMatch<'a, L> = SgNodeMatch<'a, StrDoc<L>>;
 
@@ -28,15 +27,13 @@ pub enum Platform {
   GitHub,
 }
 
-pub struct CloudPrinter<W: Write + Send + Sync> {
-  writer: Mutex<W>,
+pub struct CloudPrinter<W: Write> {
+  writer: W,
 }
 
-impl<W: Write + Send + Sync> CloudPrinter<W> {
-  pub fn new(w: W) -> Self {
-    Self {
-      writer: Mutex::new(w),
-    }
+impl<W: Write> CloudPrinter<W> {
+  pub fn new(writer: W) -> Self {
+    Self { writer }
   }
 }
 
@@ -46,9 +43,9 @@ impl CloudPrinter<Stdout> {
   }
 }
 
-impl<W: Write + Send + Sync> Printer for CloudPrinter<W> {
+impl<W: Write> Printer for CloudPrinter<W> {
   fn print_rule<'a>(
-    &self,
+    &mut self,
     matches: Matches!('a),
     file: SimpleFile<Cow<str>, &String>,
     rule: &RuleConfig<SgLang>,
@@ -57,16 +54,16 @@ impl<W: Write + Send + Sync> Printer for CloudPrinter<W> {
     print_rule(self, matches, &path, rule)
   }
 
-  fn print_matches<'a>(&self, _m: Matches!('a), _p: &Path) -> Result<()> {
+  fn print_matches<'a>(&mut self, _m: Matches!('a), _p: &Path) -> Result<()> {
     unreachable!()
   }
 
-  fn print_diffs<'a>(&self, _d: Diffs!('a), _p: &Path) -> Result<()> {
+  fn print_diffs<'a>(&mut self, _d: Diffs!('a), _p: &Path) -> Result<()> {
     unreachable!()
   }
 
   fn print_rule_diffs(
-    &self,
+    &mut self,
     diffs: Vec<(Diff<'_>, &RuleConfig<SgLang>)>,
     path: &Path,
   ) -> Result<()> {
@@ -77,13 +74,13 @@ impl<W: Write + Send + Sync> Printer for CloudPrinter<W> {
   }
 }
 
-fn print_rule<'a, W: Write + Send + Sync>(
-  p: &CloudPrinter<W>,
+fn print_rule<'a, W: Write>(
+  p: &mut CloudPrinter<W>,
   matches: Matches!('a),
   path: &Path,
   rule: &RuleConfig<SgLang>,
 ) -> Result<()> {
-  let mut writer = p.writer.lock().expect("should work");
+  let writer = &mut p.writer;
   let level = match rule.severity {
     Severity::Error => "error",
     Severity::Warning => "warning",
@@ -98,7 +95,7 @@ fn print_rule<'a, W: Write + Send + Sync>(
     let end_line = m.end_pos().line() + 1;
     let message = rule.get_message(&m);
     writeln!(
-      &mut writer,
+      writer,
       "::{level} file={name},line={line},endLine={end_line},title={title}::{message}"
     )?;
   }
@@ -115,8 +112,8 @@ mod test {
   fn make_test_printer() -> CloudPrinter<Buffer> {
     CloudPrinter::new(Buffer::no_color())
   }
-  fn get_text(printer: &CloudPrinter<Buffer>) -> String {
-    let buffer = printer.writer.lock().expect("should work");
+  fn get_text(printer: &mut CloudPrinter<Buffer>) -> String {
+    let buffer = &mut printer.writer;
     let bytes = buffer.as_slice();
     std::str::from_utf8(bytes)
       .expect("buffer should be valid utf8")
@@ -142,13 +139,13 @@ language: TypeScript
 
   fn test_output(src: &str, rule_str: &str, expect: &str) {
     let src = src.to_owned();
-    let printer = make_test_printer();
+    let mut printer = make_test_printer();
     let grep = SgLang::from(SupportLang::Tsx).ast_grep(&src);
     let rule = make_rule(rule_str);
     let matches = grep.root().find_all(&rule.matcher);
     let file = SimpleFile::new(Cow::Borrowed("test.tsx"), &src);
     printer.print_rule(matches, file, &rule).unwrap();
-    let actual = get_text(&printer);
+    let actual = get_text(&mut printer);
     assert_eq!(actual, expect);
   }
 

{ast_grep_cli-0.32.2 → ast_grep_cli-0.32.3}/crates/cli/src/print/colored_print/test.rs

@@ -11,7 +11,7 @@ fn make_test_printer() -> ColoredPrinter<Buffer> {
   ColoredPrinter::new(Buffer::no_color()).color(ColorChoice::Never)
 }
 fn get_text(printer: &ColoredPrinter<Buffer>) -> String {
-  let buffer = printer.writer.lock().expect("should work");
+  let buffer = &printer.writer;
   let bytes = buffer.as_slice();
   std::str::from_utf8(bytes)
     .expect("buffer should be valid utf8")
@@ -46,7 +46,7 @@ const MATCHES_CASES: &[Case] = &[
 fn test_print_matches() {
   for &(source, pattern, note) in MATCHES_CASES {
     // heading is required for CI
-    let printer = make_test_printer().heading(Heading::Always);
+    let mut printer = make_test_printer().heading(Heading::Always);
     let grep = SgLang::from(SupportLang::Tsx).ast_grep(source);
     let matches = grep.root().find_all(pattern);
     printer.print_matches(matches, "test.tsx".as_ref()).unwrap();
@@ -66,7 +66,7 @@ fn test_print_matches() {
 #[test]
 fn test_print_matches_without_heading() {
   for &(source, pattern, note) in MATCHES_CASES {
-    let printer = make_test_printer().heading(Heading::Never);
+    let mut printer = make_test_printer().heading(Heading::Never);
     let grep = SgLang::from(SupportLang::Tsx).ast_grep(source);
     let matches = grep.root().find_all(pattern);
     printer.print_matches(matches, "test.tsx".as_ref()).unwrap();
@@ -86,7 +86,7 @@ fn test_print_matches_without_heading() {
 fn test_print_rules() {
   let globals = GlobalRules::default();
   for &(source, pattern, note) in MATCHES_CASES {
-    let printer = make_test_printer()
+    let mut printer = make_test_printer()
       .heading(Heading::Never)
       .style(ReportStyle::Short);
     let grep = SgLang::from(SupportLang::TypeScript).ast_grep(source);
@@ -153,7 +153,7 @@ const DIFF_CASES: &[DiffCase] = &[
 fn test_print_diffs() {
   for &(source, pattern, rewrite, note) in DIFF_CASES {
     // heading is required for CI
-    let printer = make_test_printer().heading(Heading::Always);
+    let mut printer = make_test_printer().heading(Heading::Always);
     let lang = SgLang::from(SupportLang::Tsx);
     let fixer = Fixer::from_str(rewrite, &lang).expect("should work");
     let grep = lang.ast_grep(source);
@@ -170,7 +170,7 @@ fn test_overlap_print_impl(heading: Heading) {
     // empty
     Some(2)
   ";
-  let printer = make_test_printer().heading(heading).context((1, 1));
+  let mut printer = make_test_printer().heading(heading).context((1, 1));
   let lang = SgLang::from(SupportLang::Tsx);
   let grep = lang.ast_grep(src);
   let matches = grep.root().find_all("Some($A)");
@@ -195,7 +195,7 @@ fn test_non_overlap_print_impl(heading: Heading) {
     // empty
     Some(2)
   ";
-  let printer = make_test_printer().heading(heading);
+  let mut printer = make_test_printer().heading(heading);
   let lang = SgLang::from(SupportLang::Tsx);
   let grep = lang.ast_grep(src);
   let matches = grep.root().find_all("Some($A)");
@@ -217,7 +217,7 @@ fn test_non_overlap_print() {
 fn test_print_rule_diffs() {
   let globals = GlobalRules::default();
   for &(source, pattern, rewrite, note) in DIFF_CASES {
-    let printer = make_test_printer()
+    let mut printer = make_test_printer()
       .heading(Heading::Never)
       .style(ReportStyle::Short);
     let grep = SgLang::from(SupportLang::TypeScript).ast_grep(source);
@@ -264,7 +264,7 @@ fn test_before_after() {
   ";
   for b in 0..3 {
     for a in 0..3 {
-      let printer = make_test_printer().context((b, a));
+      let mut printer = make_test_printer().context((b, a));
       let lang = SgLang::from(SupportLang::Tsx);
       let grep = lang.ast_grep(src);
       let matches = grep.root().find_all("Some($A)");

{ast_grep_cli-0.32.2 → ast_grep_cli-0.32.3}/crates/cli/src/print/colored_print.rs

@@ -16,7 +16,6 @@ use std::borrow::Cow;
 use std::fmt::Display;
 use std::io::Write;
 use std::path::Path;
-use std::sync::Mutex;
 
 mod test;
 
@@ -62,8 +61,8 @@ impl Heading {
   }
 }
 
-pub struct ColoredPrinter<W: WriteColor + Send + Sync> {
-  writer: Mutex<W>,
+pub struct ColoredPrinter<W: WriteColor> {
+  writer: W,
   config: term::Config,
   styles: PrintStyles,
   heading: Heading,
@@ -76,10 +75,10 @@ impl ColoredPrinter<StandardStream> {
   }
 }
 
-impl<W: WriteColor + Send + Sync> ColoredPrinter<W> {
+impl<W: WriteColor> ColoredPrinter<W> {
   pub fn new(writer: W) -> Self {
     Self {
-      writer: Mutex::new(writer),
+      writer,
       styles: PrintStyles::from(ColorChoice::Auto),
       config: term::Config::default(),
       heading: Heading::Auto,
@@ -128,15 +127,15 @@ impl<W: WriteColor + Send + Sync> ColoredPrinter<W> {
   }
 }
 
-impl<W: WriteColor + Send + Sync> Printer for ColoredPrinter<W> {
+impl<W: WriteColor> Printer for ColoredPrinter<W> {
   fn print_rule<'a>(
-    &self,
+    &mut self,
     matches: Matches!('a),
     file: SimpleFile<Cow<str>, &String>,
     rule: &RuleConfig<SgLang>,
   ) -> Result<()> {
     let config = &self.config;
-    let mut writer = self.writer.lock().expect("should not fail");
+    let writer = &mut self.writer;
     let severity = match rule.severity {
       Severity::Error => diagnostic::Severity::Error,
       Severity::Warning => diagnostic::Severity::Warning,
@@ -163,7 +162,7 @@ impl<W: WriteColor + Send + Sync> Printer for ColoredPrinter<W> {
     Ok(())
   }
 
-  fn print_matches<'a>(&self, matches: Matches!('a), path: &Path) -> Result<()> {
+  fn print_matches<'a>(&mut self, matches: Matches!('a), path: &Path) -> Result<()> {
     if self.heading.should_print() {
       print_matches_with_heading(matches, path, self)
     } else {
@@ -171,18 +170,18 @@ impl<W: WriteColor + Send + Sync> Printer for ColoredPrinter<W> {
     }
   }
 
-  fn print_diffs<'a>(&self, diffs: Diffs!('a), path: &Path) -> Result<()> {
-    let writer = &mut *self.writer.lock().expect("should success");
+  fn print_diffs<'a>(&mut self, diffs: Diffs!('a), path: &Path) -> Result<()> {
     let context = self.diff_context();
+    let writer = &mut self.writer;
     print_diffs(diffs, path, &self.styles, writer, context)
   }
   fn print_rule_diffs(
-    &self,
+    &mut self,
     diffs: Vec<(Diff<'_>, &RuleConfig<SgLang>)>,
     path: &Path,
   ) -> Result<()> {
-    let writer = &mut *self.writer.lock().expect("should success");
     let context = self.diff_context();
+    let writer = &mut self.writer;
     let mut start = 0;
     print_prelude(path, &self.styles, writer)?;
     for (diff, rule) in diffs {
@@ -322,13 +321,14 @@ impl<'a> MatchMerger<'a> {
   }
 }
 
-fn print_matches_with_heading<'a, W: WriteColor + Send + Sync>(
+fn print_matches_with_heading<'a, W: WriteColor>(
   mut matches: Matches!('a),
   path: &Path,
-  printer: &ColoredPrinter<W>,
+  printer: &mut ColoredPrinter<W>,
 ) -> Result<()> {
   let styles = &printer.styles;
-  let writer = &mut *printer.writer.lock().expect("cannot get printer lock");
+  let context_span = printer.context_span();
+  let writer = &mut printer.writer;
   print_prelude(path, styles, writer)?;
   let Some(first_match) = matches.next() else {
     return Ok(());
@@ -360,7 +360,7 @@ fn print_matches_with_heading<'a, W: WriteColor + Send + Sync>(
     write!(writer, "{line_num:>width$}│")?; // initial line num
     print_highlight(ret.lines(), width, &mut num, writer, styles)?;
     writeln!(writer)?; // end match new line
-    if printer.context_span() > 0 {
+    if context_span > 0 {
       writeln!(writer, "{:╴>width$}┤", "")?; // make separation
     }
     merger.conclude_match(&nm);
@@ -379,13 +379,14 @@ fn print_matches_with_heading<'a, W: WriteColor + Send + Sync>(
   Ok(())
 }
 
-fn print_matches_with_prefix<'a, W: WriteColor + Send + Sync>(
+fn print_matches_with_prefix<'a, W: WriteColor>(
   mut matches: Matches!('a),
   path: &Path,
-  printer: &ColoredPrinter<W>,
+  printer: &mut ColoredPrinter<W>,
 ) -> Result<()> {
   let styles = &printer.styles;
-  let writer = &mut *printer.writer.lock().expect("cannot get printer lock");
+  let context_span = printer.context_span();
+  let writer = &mut printer.writer;
   let path = path.display();
   let Some(first_match) = matches.next() else {
     return Ok(());
@@ -412,7 +413,7 @@ fn print_matches_with_prefix<'a, W: WriteColor + Send + Sync>(
       let num = merger.last_start_line + n;
       writeln!(writer, "{path}:{num}:{line}")?;
     }
-    if printer.context_span() > 0 {
+    if context_span > 0 {
       writeln!(writer, "--")?; // make separation
     }
     merger.conclude_match(&nm);

{ast_grep_cli-0.32.2 → ast_grep_cli-0.32.3}/crates/cli/src/print/interactive_print.rs

@@ -8,8 +8,6 @@ use ast_grep_config::RuleConfig;
 use ast_grep_core::{NodeMatch as SgNodeMatch, StrDoc};
 use codespan_reporting::files::SimpleFile;
 
-use std::sync::atomic::{AtomicBool, Ordering};
-
 type NodeMatch<'a, L> = SgNodeMatch<'a, StrDoc<L>>;
 
 use std::borrow::Cow;
@@ -24,8 +22,9 @@ macro_rules! Diffs {
 }
 
 pub struct InteractivePrinter<P: Printer> {
-  accept_all: AtomicBool,
+  accept_all: bool,
   from_stdin: bool,
+  committed_cnt: usize,
   inner: P,
 }
 
@@ -35,15 +34,16 @@ impl<P: Printer> InteractivePrinter<P> {
       Err(anyhow::anyhow!(EC::StdInIsNotInteractive))
     } else {
       Ok(Self {
-        accept_all: AtomicBool::new(accept_all),
+        accept_all,
         from_stdin,
         inner,
+        committed_cnt: 0,
       })
     }
   }
 
   fn prompt_edit(&self) -> char {
-    if self.accept_all.load(Ordering::SeqCst) {
+    if self.accept_all {
       return 'a';
     }
     const EDIT_PROMPT: &str = "Accept change? (Yes[y], No[n], Accept All[a], Quit[q], Edit[e])";
@@ -51,7 +51,7 @@ impl<P: Printer> InteractivePrinter<P> {
   }
 
   fn prompt_view(&self) -> char {
-    if self.accept_all.load(Ordering::SeqCst) {
+    if self.accept_all {
       return '\n';
     }
     const VIEW_PROMPT: &str = "Next[enter], Quit[q], Edit[e]";
@@ -74,7 +74,7 @@ impl<P: Printer> InteractivePrinter<P> {
 
 impl<P: Printer> Printer for InteractivePrinter<P> {
   fn print_rule<'a>(
-    &self,
+    &mut self,
     matches: Matches!('a),
     file: SimpleFile<Cow<str>, &String>,
     rule: &RuleConfig<SgLang>,
@@ -99,22 +99,23 @@ impl<P: Printer> Printer for InteractivePrinter<P> {
     })
   }
 
-  fn print_matches<'a>(&self, matches: Matches!('a), path: &Path) -> Result<()> {
+  fn print_matches<'a>(&mut self, matches: Matches!('a), path: &Path) -> Result<()> {
     utils::run_in_alternate_screen(|| print_matches_and_confirm_next(self, matches, path))
   }
 
-  fn print_diffs<'a>(&self, diffs: Diffs!('a), path: &Path) -> Result<()> {
+  fn print_diffs<'a>(&mut self, diffs: Diffs!('a), path: &Path) -> Result<()> {
     let path = path.to_path_buf();
     let (confirmed, all) =
       print_diffs_interactive(self, &path, diffs.map(|d| (d, None)).collect())?;
     self.rewrite_action(confirmed, &path)?;
     if all {
-      self.accept_all.store(true, Ordering::SeqCst);
+      self.accept_all = true;
+      // self.accept_all.store(true, Ordering::SeqCst);
     }
     Ok(())
   }
   fn print_rule_diffs(
-    &self,
+    &mut self,
     diffs: Vec<(Diff<'_>, &RuleConfig<SgLang>)>,
     path: &Path,
   ) -> Result<()> {
@@ -126,19 +127,27 @@ impl<P: Printer> Printer for InteractivePrinter<P> {
     )?;
     self.rewrite_action(confirmed, &path)?;
     if all {
-      self.accept_all.store(true, Ordering::SeqCst);
+      self.accept_all = true;
+      // self.accept_all.store(true, Ordering::SeqCst);
     }
     Ok(())
   }
+
+  fn after_print(&mut self) -> Result<()> {
+    if self.committed_cnt > 0 {
+      println!("Applied {} changes", self.committed_cnt);
+    }
+    self.inner.after_print()
+  }
 }
 
 fn print_diffs_interactive<'a>(
-  interactive: &InteractivePrinter<impl Printer>,
+  interactive: &mut InteractivePrinter<impl Printer>,
   path: &Path,
   diffs: Vec<(Diff<'a>, Option<&RuleConfig<SgLang>>)>,
 ) -> Result<(Vec<Diff<'a>>, bool)> {
   let mut confirmed = vec![];
-  let mut all = interactive.accept_all.load(Ordering::SeqCst);
+  let mut all = interactive.accept_all;
   let mut end = 0;
   for (diff, rule) in diffs {
     if diff.range.start < end {
@@ -153,18 +162,19 @@ fn print_diffs_interactive<'a>(
     if confirm {
       end = diff.range.end;
       confirmed.push(diff);
+      interactive.committed_cnt = interactive.committed_cnt.saturating_add(1);
     }
   }
   Ok((confirmed, all))
 }
 /// returns if accept_current and accept_all
 fn print_diff_and_prompt_action(
-  interactive: &InteractivePrinter<impl Printer>,
+  interactive: &mut InteractivePrinter<impl Printer>,
   path: &Path,
   (diff, rule): (Diff, Option<&RuleConfig<SgLang>>),
 ) -> Result<(bool, bool)> {
-  let printer = &interactive.inner;
   utils::run_in_alternate_screen(|| {
+    let printer = &mut interactive.inner;
     if let Some(rule) = rule {
       printer.print_rule_diffs(vec![(diff.clone(), rule)], path)?;
     } else {
@@ -186,11 +196,11 @@ fn print_diff_and_prompt_action(
 }
 
 fn print_matches_and_confirm_next<'a>(
-  interactive: &InteractivePrinter<impl Printer>,
+  interactive: &mut InteractivePrinter<impl Printer>,
   matches: Matches!('a),
   path: &Path,
 ) -> Result<()> {
-  let printer = &interactive.inner;
+  let printer = &mut interactive.inner;
   let matches: Vec<_> = matches.collect();
   let first_match = match matches.first() {
     Some(n) => n.start_pos().line(),