ace-support-items 0.15.15 → 0.15.16

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ee3fcee97215b4d7243d870366200e78369a85fc74fbe6c7ea8107efa23f8053
-  data.tar.gz: 8c9914130df26e74948b15d84b3311beb326855ff657ab6ce41d8b13bb039ccd
+  metadata.gz: a38caecfe7baec161ef92edffb6b0682c8347a2aa5595bf16659806af50556c0
+  data.tar.gz: f1a1c662c2cf9f570e7cded9997632ec0462953acb5aa772a6fe2c951d0ed394
 SHA512:
-  metadata.gz: 90551754056d747f9f8a341f8f50601095b4c29e43725550652929b95479edac2e2b92e7e922c2ab552efb09decbf0340cd24765f255c47c1babb7ef0d4c7d9e
-  data.tar.gz: d315633aeb2cce808bdeb0a4eb1bf626a50079fba04d74f35849ca06dd649b639d65d71cc69930ab5611aaf7ee9b5248b0457319dcaa3dcfc1d173b1e6fc3368
+  metadata.gz: 784f3f6d39072de1b71fdab2fa6a22bc5c8ac06289cb642fb724bac3c6b062260b1f53a27e882259ed8d248cb34805d89bb2479c220cacfbe098811cb27b27cc
+  data.tar.gz: f8b3d574bfc6c5033bc9138f813010ecf48b46d7dd979f27a9d45a908a35a59788b3b6c4b2b4374ab5dd7790034bbc9cb8185c9478a5eef4c23cf94685c3d2bb

data/CHANGELOG.md

@@ -7,6 +7,13 @@ 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

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/version.rb

@@ -3,7 +3,7 @@
 module Ace
   module Support
     module Items
-      VERSION = "0.15.15"
+      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.15
+  version: 0.15.16
 platform: ruby
 authors:
 - Michal Czyz
 bindir: bin
 cert_chain: []
-date: 2026-10-20 00:00:00.000000000 Z
+date: 2026-10-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ace-support-core