fix(isc): codemod note escaping + comment stripping + array field parsing

Codemod fixes:
- Strip YAML inline comments from note text
- Unescape YAML escapes before re-escaping for ISC
- Prevents double-escaping of quotes in notes

DocumentBuilder fix:
- parse_array_field splits field_block content into array items
- URL and other array fields properly parsed from brace blocks

Result: 259/289 deep equivalent (up from 247)

Author: ronaldtse

lib/interscript/isc/codemod.rb

@@ -417,7 +417,14 @@ def emit_heredoc_note(indent)
       def emit_note_with_continuation(note_indent)
         @out << "\n#{note_indent}note \""
         text = @scanner.scan(/[^\n]+/).to_s
-        @out << text.gsub('"', '\\"')
+        # Strip YAML inline comments: "text # comment" → "text"
+        text = text.sub(/\s+#.*$/, "")
+        # Strip outer quotes if the YAML list item was quoted: - "text"
+        text = text[1..-2] if text.start_with?('"') && text.end_with?('"')
+        text = text[1..-2] if text.start_with?("'") && text.end_with?("'")
+        # Unescape YAML escape sequences, then re-escape for ISC
+        text = text.gsub('\\"', '"').gsub("\\\\", "\\")
+        @out << text.gsub('\\', '\\\\\\\\').gsub('"', '\\"')
         # Consume continuation lines: any subsequent line indented deeper
         # than the `- ` marker is part of the same note. Blank lines between
         # continuations are preserved as \n.

lib/interscript/isc/document_builder.rb

@@ -77,6 +77,16 @@ def unescape_braces(text)
         text.gsub(/\\([{}\\])/, '\1')
       end
 
+      def parse_array_field(val)
+        return [] if val.nil? || val.to_s.strip.empty?
+        # If the value contains newlines or `- ` markers, split into items
+        items = val.to_s.split(/\n+/)
+                       .map { |l| l.strip.sub(/\A-\s*/, "") }
+                       .reject(&:empty?)
+        return [val.to_s.strip] if items.empty?
+        items
+      end
+
       def normalize_heredoc(text)
         lines = text.lines.map(&:chomp)
         content_lines = lines.reject { |l| l.strip.empty? }
@@ -155,7 +165,7 @@ def extract_metadata(arr)
             end
             # DSL stores these as Arrays — match that convention.
             if ARRAY_METADATA_FIELDS.include?(name)
-              h[name] = val.to_s.empty? ? [] : [val]
+              h[name] = parse_array_field(val)
             else
               h[name] = val
             end