ace-support-items 0.15.15 → 0.15.17

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ee3fcee97215b4d7243d870366200e78369a85fc74fbe6c7ea8107efa23f8053
-  data.tar.gz: 8c9914130df26e74948b15d84b3311beb326855ff657ab6ce41d8b13bb039ccd
+  metadata.gz: 377e58a49a35c4a328970fd2f2e88dc410eb7df235a56c5e513c3444f097f4d2
+  data.tar.gz: 4551619979ce7de032986074aba3a3b2da7f0bddd1825636a146ccba44266b63
 SHA512:
-  metadata.gz: 90551754056d747f9f8a341f8f50601095b4c29e43725550652929b95479edac2e2b92e7e922c2ab552efb09decbf0340cd24765f255c47c1babb7ef0d4c7d9e
-  data.tar.gz: d315633aeb2cce808bdeb0a4eb1bf626a50079fba04d74f35849ca06dd649b639d65d71cc69930ab5611aaf7ee9b5248b0457319dcaa3dcfc1d173b1e6fc3368
+  metadata.gz: 00fee712d7aa01022e168a41d4b096fc8d74802d2bcef7f57e50fea5afd129a0cef59dc970fa616947d7d0db3429fe1e5934cf9b247d50cd96eb8200913cb476
+  data.tar.gz: 42ba9cf8bdc9c901b95afaf0ac994938b1c167c3349c27ba5f1b6e71db3cbc298c03b732fabdd855eeacfca40917185df9dc938375471cbba2bdc9414940d789

data/CHANGELOG.md

@@ -7,6 +7,19 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.15.17] - 2026-11-10
+
+### Fixed
+
+- Allow accessible-only discovery to skip unreadable descendants while preserving strict complete inventory and root checks.
+
+## [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/directory_scanner.rb

@@ -40,17 +40,17 @@ module Ace
 
           # Scan root directory recursively for items
           # @return [Array<ScanResult>] List of scan results, sorted by ID (chronological)
-          def scan
+          def scan(accessible_only: false)
             return [] unless Dir.exist?(@root_dir)
 
             results = []
-            scan_directory(@root_dir, results)
+            scan_directory(@root_dir, results, accessible_only: accessible_only)
             results.sort_by(&:id)
           end
 
           private
 
-          def scan_directory(dir, results)
+          def scan_directory(dir, results, accessible_only:)
             Dir.entries(dir).sort.each do |entry|
               next if entry.start_with?(".")
 
@@ -65,13 +65,17 @@ module Ace
                   results << result
                 else
                   # Folder has spec files but isn't an item (e.g., _maybe with orphan files) — recurse
-                  scan_directory(full_path, results)
+                  scan_directory(full_path, results, accessible_only: accessible_only)
                 end
               else
                 # Recurse into subdirectory (for special folders like _maybe/)
-                scan_directory(full_path, results)
+                scan_directory(full_path, results, accessible_only: accessible_only)
               end
             end
+          rescue Errno::EACCES, Errno::EPERM
+            # Only discovery may omit inaccessible descendants. A requested
+            # root and the default complete inventory always fail closed.
+            raise unless accessible_only && dir != @root_dir
           end
 
           def build_result(dir_path, file_path)

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::Exception => 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.17"
     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.17
 platform: ruby
 authors:
 - Michal Czyz
 bindir: bin
 cert_chain: []
-date: 2026-10-20 00:00:00.000000000 Z
+date: 2026-11-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ace-support-core