Class: Ace::Review::Molecules::FeedbackFileReader

Inherits:
Object
  • Object
show all
Defined in:
lib/ace/review/molecules/feedback_file_reader.rb

Overview

Reads feedback files from disk and parses them into FeedbackItem instances.

Handles YAML frontmatter parsing and markdown section extraction. Returns error hashes for malformed files rather than raising exceptions.

Examples:

Read a single file

reader = FeedbackFileReader.new
result = reader.read("/path/to/8o7abc-missing-error.s.md")
result[:success]       #=> true
result[:feedback_item] #=> FeedbackItem instance

Read all files in a directory

reader = FeedbackFileReader.new
results = reader.read_all("/path/to/feedback")
results #=> [FeedbackItem, FeedbackItem, ...]

Constant Summary collapse

FRONTMATTER_PATTERN =

YAML frontmatter pattern: content between — markers at start of file

/\A---\n(.*?)\n---\n/m
SECTION_PATTERN =

Section header pattern: ## Section Name

/^## (\w+)\n/

Instance Method Summary collapse

Instance Method Details

#read(file_path) ⇒ Hash

Read a single feedback file

Parameters:

  • file_path (String) —

    Path to the .s.md file

Returns:

  • (Hash) —

    Result with :success and :feedback_item or :error



35
36
37
38
39
40
41
42
43
44
45
46
47
48
 
# File 'lib/ace/review/molecules/feedback_file_reader.rb', line 35

def read(file_path)
  validate_file_path(file_path)

  content = File.read(file_path)
  parse_content(content, file_path)
rescue Errno::ENOENT
  {success: false, error: "File not found: #{file_path}"}
rescue Errno::EACCES
  {success: false, error: "Permission denied: #{file_path}"}
rescue ArgumentError => e
  {success: false, error: e.message}
rescue SystemCallError, IOError => e
  {success: false, error: "Failed to read file: #{e.message}"}
end

#read_all(directory) ⇒ Array<Models::FeedbackItem>

Read all .s.md files in a directory

Parameters:

  • directory (String) —

    Path to the feedback directory

Returns:



54
55
56
57
58
59
60
61
62
63
64
65
66
 
# File 'lib/ace/review/molecules/feedback_file_reader.rb', line 54

def read_all(directory)
  return [] unless Dir.exist?(directory)

  files = Dir.glob(File.join(directory, "*.s.md"))
  items = []

  files.each do |file_path|
    result = read(file_path)
    items << result[:feedback_item] if result[:success]
  end

  items
end

#read_by_status(directory, status) ⇒ Array<Models::FeedbackItem>

Read feedback files filtered by status

Parameters:

  • directory (String) —

    Path to the feedback directory

  • status (String) —

    Status to filter by (draft, pending, invalid, skip, done)

Returns:



73
74
75
 
# File 'lib/ace/review/molecules/feedback_file_reader.rb', line 73

def read_by_status(directory, status)
  read_all(directory).select { |item| item.status == status }
end