ace-support-items 0.15.14 → 0.15.16

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 36c95e46f3570063b5651e2b3d91bf46edafd37278c5c3250d1a70c73071545a
-  data.tar.gz: b1b2894f143a00d647c6d984ef2af96c419bbb1be25f82f1ce85cb40a7c14048
+  metadata.gz: a38caecfe7baec161ef92edffb6b0682c8347a2aa5595bf16659806af50556c0
+  data.tar.gz: f1a1c662c2cf9f570e7cded9997632ec0462953acb5aa772a6fe2c951d0ed394
 SHA512:
-  metadata.gz: 975ade85ed637013761e3a953dc0f6eb9c9c3a82932b70e269ab17b7eead7cd52f4426e6b0c68a901697ebc3acbfdb3bd5c229f84de89f1d31e933c5b5788a62
-  data.tar.gz: e51e3e3eff0172dad565df58d7df98e8f663ccd926ad47c13d1a9df904d1819bab0dc1489359713607fcdc22ac025302859f0635d021f06cb09c9f202b21bff8
+  metadata.gz: 784f3f6d39072de1b71fdab2fa6a22bc5c8ac06289cb642fb724bac3c6b062260b1f53a27e882259ed8d248cb34805d89bb2479c220cacfbe098811cb27b27cc
+  data.tar.gz: f8b3d574bfc6c5033bc9138f813010ecf48b46d7dd979f27a9d45a908a35a59788b3b6c4b2b4374ab5dd7790034bbc9cb8185c9478a5eef4c23cf94685c3d2bb

data/CHANGELOG.md

@@ -7,6 +7,21 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.15.16] - 2026-10-29
+
+### Fixed
+
+- Frontmatter flow sequences serialize every string element with the same quoting rules as flat values, so a question or tag containing `: ` parses back as the same string instead of a single-pair hash; compound elements raise `ArgumentError` instead of emitting Ruby rocket syntax that is not YAML (the 8xsdfo ask-flow corruption: recipient/reply_to frontmatter was silently dropped, so native sync refused the captain's reply and the answer wake never fired).
+- `FrontmatterParser.parse!` gives mutating paths a strict parse, and `FieldUpdater` refuses to rewrite a file whose frontmatter YAML is invalid — an unparsable block now aborts the update loudly instead of rebuilding the file from an empty frontmatter.
+
+## [0.15.15] - 2026-10-18
+
+### Added
+
+- `FolderMover#destination_path` computes the destination a move would create
+  without touching the filesystem, so callers can run pre-flight checks (for
+  example a Git-ignored destination guard).
+
 ## [0.15.14] - 2026-10-06
 
 ### Fixed

data/lib/ace/support/items/atoms/frontmatter_parser.rb

@@ -13,6 +13,21 @@ module Ace
           # @param content [String] The markdown content with optional frontmatter
           # @return [Hash] Parsed frontmatter data (empty hash if no frontmatter)
           def self.parse_frontmatter(content)
+            parse_frontmatter!(content)
+          rescue Psych::SyntaxError
+            # Read paths tolerate arbitrary user files: an unparsable block
+            # yields an empty hash instead of crashing scans.
+            {}
+          end
+
+          # Strict variant for mutating paths (FieldUpdater and friends): a
+          # syntax error must stop the change instead of silently rebuilding
+          # the file from an empty frontmatter (the 8xsdfo ask-flow corruption
+          # amplified one bad line into total frontmatter loss).
+          # @param content [String] The markdown content with optional frontmatter
+          # @return [Hash] Parsed frontmatter data (empty hash if no frontmatter)
+          # @raise [Psych::SyntaxError] when a frontmatter block is unparsable
+          def self.parse_frontmatter!(content)
             return {} if content.nil? || content.empty?
             return {} unless content.start_with?("---\n")
 
@@ -21,11 +36,7 @@ module Ace
 
             yaml_content = content[4...end_index]
 
-            begin
-              YAML.safe_load(yaml_content, permitted_classes: [Date, Time, Symbol]) || {}
-            rescue Psych::SyntaxError
-              {}
-            end
+            YAML.safe_load(yaml_content, permitted_classes: [Date, Time, Symbol]) || {}
           end
 
           # Extract content after frontmatter
@@ -50,6 +61,14 @@ module Ace
           def self.parse(content)
             [parse_frontmatter(content), extract_body(content)]
           end
+
+          # Strict parse for mutating paths: raises on an unparsable block.
+          # @param content [String] The markdown content
+          # @return [Array(Hash, String)] Tuple of [frontmatter, body]
+          # @raise [Psych::SyntaxError] when a frontmatter block is unparsable
+          def self.parse!(content)
+            [parse_frontmatter!(content), extract_body(content)]
+          end
         end
       end
     end

data/lib/ace/support/items/atoms/frontmatter_serializer.rb

@@ -31,7 +31,7 @@ module Ace
           end
 
           # Serialize a single key-value entry with indentation support.
-          def self.serialize_entry(lines, key, value, indent:)
+          private_class_method def self.serialize_entry(lines, key, value, indent:)
             prefix = "  " * indent
             case value
             when Hash
@@ -43,7 +43,7 @@ module Ace
               lines << if value.empty?
                 "#{prefix}#{key}: []"
               else
-                "#{prefix}#{key}: [#{value.join(", ")}]"
+                "#{prefix}#{key}: [#{value.map { |element| flow_element(element, key) }.join(", ")}]"
               end
             when String
               lines << if needs_quoting?(value)
@@ -56,7 +56,27 @@ module Ace
             end
           end
 
-          private_class_method :serialize_entry
+          # One element inside a flow sequence: strings follow the same
+          # quoting rules as flat values (a plain scalar containing `: ` would
+          # parse back as a hash, silently corrupting the value across
+          # read-modify-write cycles). Numbers, booleans and nil serialize to
+          # their YAML literals; compound values have no safe flow form and
+          # fail loudly instead of emitting YAML that parses back as a
+          # different structure (the 8xsdfo ask-flow corruption).
+          def self.flow_element(element, key)
+            case element
+            when String
+              needs_quoting?(element) ? "\"#{escape_yaml_string(element)}\"" : element
+            when Integer, Float, true, false
+              element.to_s
+            when nil
+              "null"
+            else
+              raise ArgumentError,
+                "frontmatter array '#{key}' cannot safely carry a #{element.class} element; use scalar strings"
+            end
+          end
+          private_class_method :flow_element
 
           # Check if a string value needs YAML quoting
           # @param value [String] Value to check

data/lib/ace/support/items/molecules/field_updater.rb

@@ -25,7 +25,7 @@ module Ace
 
               file.rewind
               content = file.read
-              frontmatter, body = Atoms::FrontmatterParser.parse(content)
+              frontmatter, body = parse_strict(file_path, content)
               # Strip leading newline from body so rebuild doesn't double-space
               body = body.sub(/\A\n/, "")
 
@@ -118,6 +118,16 @@ module Ace
             file.fsync
           end
 
+          # Strict parse with an actionable message: an unparsable block must
+          # abort the update (the rewrite would silently drop the whole
+          # frontmatter), never proceed with an empty hash.
+          def self.parse_strict(file_path, content)
+            Atoms::FrontmatterParser.parse!(content)
+          rescue Psych::SyntaxError => e
+            raise ArgumentError,
+              "#{file_path}: frontmatter YAML is invalid (#{e.message}); refusing to rewrite the file"
+          end
+
           private_class_method :apply_nested_set, :rewrite_locked_file
         end
       end

data/lib/ace/support/items/molecules/folder_mover.rb

@@ -17,35 +17,38 @@ module Ace
             @root_dir = root_dir
           end
 
-          # Move an item folder to a target location.
+          # Compute the destination path for an item without moving it.
+          # Used by pre-flight checks (e.g. Git-ignored task guards).
           #
           # @param item [#path] Item with a path attribute (directory to move)
           # @param to [String] Target folder name (short or full, e.g., "maybe", "_archive")
           # @param date [Time, nil] Date used to compute archive partition (default: Time.now)
-          # @return [String] New path of the item directory
-          def move(item, to:, date: nil)
+          # @return [String] Destination directory path
+          def destination_path(item, to:, date: nil)
             if Atoms::SpecialFolderDetector.virtual_filter?(to)
               raise ArgumentError, "Cannot move to virtual filter '#{to}' — it is not a physical folder"
             end
 
             normalized = Atoms::SpecialFolderDetector.normalize(to)
-
-            target_parent = if normalized == "_archive"
-              partition = Atoms::DatePartitionPath.compute(date || Time.now)
-              File.expand_path(File.join(@root_dir, normalized, partition))
-            else
-              File.expand_path(File.join(@root_dir, normalized))
-            end
-
+            target_parent = destination_parent(normalized, date)
             validate_path_traversal!(target_parent)
-            FileUtils.mkdir_p(target_parent)
 
-            folder_name = File.basename(item.path)
-            new_path = File.join(target_parent, folder_name)
+            File.join(target_parent, File.basename(item.path))
+          end
+
+          # Move an item folder to a target location.
+          #
+          # @param item [#path] Item with a path attribute (directory to move)
+          # @param to [String] Target folder name (short or full, e.g., "maybe", "_archive")
+          # @param date [Time, nil] Date used to compute archive partition (default: Time.now)
+          # @return [String] New path of the item directory
+          def move(item, to:, date: nil)
+            new_path = destination_path(item, to: to, date: date)
 
             # Same-location no-op check
             return item.path if File.expand_path(item.path) == File.expand_path(new_path)
 
+            FileUtils.mkdir_p(File.dirname(new_path))
             atomic_move(item.path, new_path)
           end
 
@@ -65,6 +68,15 @@ module Ace
 
           private
 
+          def destination_parent(normalized, date)
+            if normalized == "_archive"
+              partition = Atoms::DatePartitionPath.compute(date || Time.now)
+              File.expand_path(File.join(@root_dir, normalized, partition))
+            else
+              File.expand_path(File.join(@root_dir, normalized))
+            end
+          end
+
           def validate_path_traversal!(target_parent)
             root_real = File.expand_path(@root_dir)
             unless target_parent.start_with?(root_real + File::SEPARATOR)

data/lib/ace/support/items/version.rb

@@ -3,7 +3,7 @@
 module Ace
   module Support
     module Items
-      VERSION = "0.15.14"
+      VERSION = "0.15.16"
     end
   end
 end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: ace-support-items
 version: !ruby/object:Gem::Version
-  version: 0.15.14
+  version: 0.15.16
 platform: ruby
 authors:
 - Michal Czyz
 bindir: bin
 cert_chain: []
-date: 2026-10-08 00:00:00.000000000 Z
+date: 2026-10-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ace-support-core