# Evaluation: gpt_20b response

## Did it only list snippets, or also tell what to implement?

- Mostly snippets-only, but the final paragraph crosses the line: *"Adding a prefix image would involve inserting drawing code before or alongside the `title.draw(at:)` call in the `decorate` method."*
- This is explicit implementation guidance, which the prompt asked to avoid ("DON'T CHANGE ANY CODE, JUST GIVE ME THE SNIPPETS I NEED")
- Minor violation but still a violation

## Did it find all related places?

**Included (good):**
- `titleText` computed property
- `subtitleContent` computed property
- `MapExportDecorator.decorate()` method (drawing section)
- `generatePreview()` as a call site
- `exportImage()` as a call site

**Missing:**
- `MapExportDecorator.Configuration` struct: needed to understand what configuration options exist and where to add a prefix image setting
- `decoratorConfig` computed property in ExportSheet: this is where configuration is assembled and passed to the decorator; any new prefix image setting would flow through here
- `exportImageTiled()` function: a third call site of `decorate()` that was completely missed (used for large/print exports)
- The `decorate()` method is heavily truncated with `// …` placeholders, hiding the critical layout/sizing code: `titleAreaHeight` calculation, `canvasHeight` calculation, and the `titleAreaY`/`mapFrameY` positioning logic. These are essential for understanding how to fit a prefix image into the layout
- ExportSheet state variables and `previewTriggerHash`: shows the pattern for how new visual settings are wired up (state var + hash entry + save/load)

## Summary

The response identifies the core rendering pipeline (title generation -> decoration -> preview/export) but misses important structural pieces. The heavily truncated `decorate()` method hides the layout math that would be the most critical code to understand for inserting a prefix image. Missing a full call site (`exportImageTiled`) and the configuration/wiring layer (`Configuration` struct, `decoratorConfig`, state management) means someone following this response would discover significant gaps when actually implementing the feature.

## Grade: C+

Finds the right general area but truncates the most important method too aggressively and misses the configuration layer and a call site. The unsolicited implementation advice is a minor negative.