Fix core dump parsing transaction entry with only comment (#26)

Author: mondeja

CHANGELOG.md

@@ -1,5 +1,11 @@
 # CHANGELOG
 
+## 2025-05-20 - [0.2.8]
+
+### Bug fixes
+
+- Fix core dump parsing transaction entry with only comment.
+
 ## 2025-05-19 - [0.2.7]
 
 ### Bug fixes
@@ -102,6 +108,7 @@
 
 First beta release
 
+[0.2.8]: https://github.com/mondeja/hledger-fmt/compare/v0.2.7...v0.2.8
 [0.2.7]: https://github.com/mondeja/hledger-fmt/compare/v0.2.6...v0.2.7
 [0.2.6]: https://github.com/mondeja/hledger-fmt/compare/v0.2.5...v0.2.6
 [0.2.5]: https://github.com/mondeja/hledger-fmt/compare/v0.2.4...v0.2.5

Cargo.lock

@@ -1,6 +1,6 @@
 [package]
 name = "hledger-fmt"
-version = "0.2.7"
+version = "0.2.8"
 rust-version = "1.74.1"
 edition = "2021"
 description = "An opinionated hledger's journal files formatter."

Cargo.toml

@@ -490,3 +490,15 @@ fn lots() {
 "#,
     );
 }
+
+#[test]
+fn issue25() {
+    assert_format(
+        r#"1/1/1 * transaction
+	; vacation  $2350 hawaii flight
+"#,
+        r#"1/1/1 * transaction
+; vacation  $2350 hawaii flight
+"#,
+    );
+}

src/formatter/tests.rs

@@ -367,7 +367,7 @@ pub fn parse_content(content: &str) -> Result<JournalFile, errors::SyntaxError>
                                 }
                             }
                         } else {
-                            // inside transaction entry
+                            // maybe inside transaction entry
                             let mut at_indent = c != '\t';
                             let mut indent = if at_indent { 1 } else { 4 };
                             let mut entry_name = String::with_capacity(64);
@@ -381,7 +381,7 @@ pub fn parse_content(content: &str) -> Result<JournalFile, errors::SyntaxError>
                                         indent += 1;
                                     } else if c == ';' || c == '#' {
                                         // transaction entry with empty value
-                                        let comment = parse_inline_comment(
+                                        let maybe_comment = parse_inline_comment(
                                             &mut chars_iter,
                                             lineno,
                                             coln + 1,
@@ -391,7 +391,7 @@ pub fn parse_content(content: &str) -> Result<JournalFile, errors::SyntaxError>
                                                 CommentPrefix::Semicolon
                                             }),
                                         );
-                                        if comment.is_some() {
+                                        if let Some(comment) = maybe_comment {
                                             is_comment_only = true;
                                             // if the first comment is indented with >=2 and first entry indent
                                             // is not setted, set it
@@ -400,11 +400,8 @@ pub fn parse_content(content: &str) -> Result<JournalFile, errors::SyntaxError>
                                             if indent >= 2 && data.first_entry_indent == 0 {
                                                 data.first_entry_indent = indent;
                                             }
-                                            data.transaction_entries.push(
-                                                TransactionNode::SingleLineComment(
-                                                    comment.unwrap(),
-                                                ),
-                                            );
+                                            data.transaction_entries
+                                                .push(TransactionNode::SingleLineComment(comment));
                                         }
                                         break;
                                     } else {
@@ -422,6 +419,23 @@ pub fn parse_content(content: &str) -> Result<JournalFile, errors::SyntaxError>
                                         prev_was_whitespace = true;
                                     } else {
                                         prev_was_whitespace = false;
+
+                                        if c == ';' && entry_name.is_empty() {
+                                            // inside comment
+                                            let maybe_comment = parse_inline_comment(
+                                                &mut chars_iter,
+                                                lineno,
+                                                coln + 1,
+                                                Some(CommentPrefix::Semicolon),
+                                            );
+                                            if let Some(comment) = maybe_comment {
+                                                is_comment_only = true;
+                                                data.transaction_entries.push(
+                                                    TransactionNode::SingleLineComment(comment),
+                                                );
+                                            }
+                                            break;
+                                        }
                                     }
                                     entry_name.push(c);
                                 }