# Evaluation of carl.md

## Did it only list snippets (good) or also tell what to implement (bad)?

- **Violated the instruction.** Almost every snippet heading includes implementation guidance: "you'd need similar for image sizing", "you'd offset the title X to make room for the image", "where you'd insert image drawing before the title", "where you'd add image-related config fields", "you'd need to pass the image through"
- Includes a full **"Summary of what you'd modify"** paragraph at the end explicitly telling the user what to change (`add an image parameter to decorate()`, `adjust alignedX`, `draw the image with cgContext`, etc.)
- The prompt said "DON'T CHANGE ANY CODE, JUST GIVE ME THE SNIPPETS I NEED" -- the model went well beyond listing snippets

## Did it find all related places or miss anything?

**Found (good):**
- `titleText` computed property
- `decorate()` method signature
- Title font creation / `fittedFont` call
- `alignedX` helper function
- Title drawing code (`config.showTitle` block)
- Title area height calculation
- `Configuration` struct properties
- One `decorate()` call site (preview)

**Missed:**
- **2 additional `decorate()` call sites**: `exportImageTiled()` (line ~2069) and `exportImage()` (line ~2182) -- anyone following only this response would update the preview but forget to update the actual export paths
- **`decoratorConfig` computed property** (line ~338) -- this is where the Configuration is assembled from state; critical if the image needs to flow through config
- **`previewTriggerHash`** (line ~158) -- any new state variable (e.g. a prefix image) must be added here or the preview won't refresh
- **`saveSettings()` / `loadSavedSettings()`** -- if the prefix image is a persistent setting, these need updating
- **`applyTemplateSettings()`** -- templates would need to know about the image setting
- **Preview theme rendering** call site (line ~5160) in `AllThemesGridView` -- minor but still a call site

## Grade: **C-**

The core rendering snippets are mostly correct and relevant, but the model failed the primary instruction ("just give me the snippets") by including extensive implementation advice throughout. It also missed multiple `decorate()` call sites and key infrastructure (`previewTriggerHash`, `decoratorConfig`, persistence), which would leave a developer with an incomplete picture of what needs changing.