Merge pull request 'fix: rewrite flexbox tasks to describe outcomes instead of answers' (#5) from 003-flexbox-task-wording into main

lessons/flexbox.json

@@ -9,7 +9,7 @@
 			"id": "flexbox-1",
 			"title": "Container",
 			"description": "Before flexbox, creating even simple layouts required floats, positioning hacks, or table-based layouts. Flexbox (Flexible Box Layout) revolutionized CSS by providing a one-dimensional layout system designed specifically for distributing space and aligning content.<br><br><strong>How it works:</strong> When you set <kbd>display: flex</kbd> on an element, it becomes a <em>flex container</em>. Its direct children automatically become <em>flex items</em> that flow along a main axis (horizontal by default). This single property transforms stacked block elements into a horizontal row.<br><br><strong>The two axes:</strong><br>• <em>Main axis</em> – The primary direction items flow (row = left→right)<br>• <em>Cross axis</em> – Perpendicular to main (row = top→bottom)<br><br><pre>.nav {\n  display: flex;\n}</pre>",
-			"task": "This navigation menu stacks vertically. Add <kbd>display: flex</kbd> to <kbd>.nav</kbd> to arrange the links horizontally.",
+			"task": "The navigation links are stacking vertically. Make them display side by side in a horizontal row.",
 			"previewHTML": "<nav class=\"nav\"><a href=\"#\">Home</a><a href=\"#\">Products</a><a href=\"#\">About</a><a href=\"#\">Contact</a></nav>",
 			"previewBaseCSS": "body { font-family: system-ui, sans-serif; margin: 0; } .nav { background: #1a1a2e; padding: 1rem; } .nav a { color: white; text-decoration: none; padding: 8px 1rem; border-radius: 4px; } .nav a:hover { background: rgba(255,255,255,0.1); }",
 			"sandboxCSS": "",
@@ -22,7 +22,7 @@
 				{
 					"type": "property_value",
 					"value": { "property": "display", "expected": "flex" },
-					"message": "Set <kbd>display: flex</kbd>"
+					"message": "Try changing the display mode to create a flex container"
 				}
 			]
 		},
@@ -30,7 +30,7 @@
 			"id": "flexbox-2",
 			"title": "Gap",
 			"description": "The <kbd>gap</kbd> property adds consistent spacing between flex items without needing margins. It only creates space between items, not around the edges.",
-			"task": "Add <kbd>gap: 1rem</kbd> to space out the navigation links evenly.",
+			"task": "The navigation links are crammed together with no breathing room. Add 1rem of spacing between them.",
 			"previewHTML": "<nav class=\"nav\"><a href=\"#\">Home</a><a href=\"#\">Products</a><a href=\"#\">About</a><a href=\"#\">Contact</a></nav>",
 			"previewBaseCSS": "body { font-family: system-ui, sans-serif; margin: 0; } .nav { background: #1a1a2e; padding: 1rem; display: flex; } .nav a { color: white; text-decoration: none; padding: 8px 1rem; border-radius: 4px; background: rgba(255,255,255,0.1); }",
 			"sandboxCSS": "",
@@ -43,7 +43,7 @@
 				{
 					"type": "property_value",
 					"value": { "property": "gap", "expected": "1rem" },
-					"message": "Set <kbd>gap: 1rem</kbd>"
+					"message": "Use the property that adds spacing between flex items"
 				}
 			]
 		},
@@ -51,7 +51,7 @@
 			"id": "flexbox-3",
 			"title": "Justify Content",
 			"description": "<kbd>justify-content</kbd> distributes items along the main axis. Common values:<br>• <kbd>flex-start</kbd> – pack items at the start<br>• <kbd>flex-end</kbd> – pack at the end<br>• <kbd>center</kbd> – center items<br>• <kbd>space-between</kbd> – equal space between items<br>• <kbd>space-around</kbd> – equal space around items",
-			"task": "Push the \"Login\" button to the right by setting <kbd>justify-content: space-between</kbd> on the nav.",
+			"task": "The Login button should sit on the far right, with the other links staying on the left. Distribute the space between them.",
 			"previewHTML": "<nav class=\"nav\"><div class=\"links\"><a href=\"#\">Home</a><a href=\"#\">Products</a><a href=\"#\">About</a></div><a href=\"#\" class=\"login\">Login</a></nav>",
 			"previewBaseCSS": "body { font-family: system-ui, sans-serif; margin: 0; } .nav { background: #1a1a2e; padding: 1rem; display: flex; } .links { display: flex; gap: 8px; } .nav a { color: white; text-decoration: none; padding: 8px 1rem; border-radius: 4px; } .nav a:hover { background: rgba(255,255,255,0.1); } .login { background: steelblue; }",
 			"sandboxCSS": "",
@@ -64,7 +64,7 @@
 				{
 					"type": "property_value",
 					"value": { "property": "justify-content", "expected": "space-between" },
-					"message": "Set <kbd>justify-content: space-between</kbd>"
+					"message": "Use the property that distributes items along the main axis"
 				}
 			]
 		},
@@ -72,7 +72,7 @@
 			"id": "flexbox-4",
 			"title": "Align Items",
 			"description": "<kbd>align-items</kbd> controls alignment on the cross axis (vertical when flex-direction is row). Values include:<br>• <kbd>stretch</kbd> – stretch to fill (default)<br>• <kbd>flex-start</kbd> – align to top<br>• <kbd>flex-end</kbd> – align to bottom<br>• <kbd>center</kbd> – center vertically",
-			"task": "The logo and nav links have different heights. Center them vertically with <kbd>align-items: center</kbd>.",
+			"task": "The logo and nav links sit at different heights. Center them vertically so they line up.",
 			"previewHTML": "<header class=\"header\"><div class=\"logo\">ACME</div><nav><a href=\"#\">Products</a><a href=\"#\">Pricing</a><a href=\"#\">Docs</a></nav></header>",
 			"previewBaseCSS": "body { font-family: system-ui, sans-serif; margin: 0; } .header { background: white; padding: 1rem 2rem; display: flex; justify-content: space-between; border-bottom: 1px solid #eee; } .logo { font-size: 1.5rem; font-weight: bold; color: steelblue; } nav { display: flex; gap: 1rem; } nav a { color: #333; text-decoration: none; font-size: 0.9rem; }",
 			"sandboxCSS": "",
@@ -85,7 +85,7 @@
 				{
 					"type": "property_value",
 					"value": { "property": "align-items", "expected": "center" },
-					"message": "Set <kbd>align-items: center</kbd>"
+					"message": "Use the property that controls cross-axis alignment"
 				}
 			]
 		},
@@ -93,7 +93,7 @@
 			"id": "flexbox-5",
 			"title": "Flex Wrap",
 			"description": "By default, flex items squeeze onto one line. <kbd>flex-wrap: wrap</kbd> allows items to flow onto multiple lines when they run out of space.",
-			"task": "These cards overflow the container. Add <kbd>flex-wrap: wrap</kbd> to allow them to wrap to new rows.",
+			"task": "The cards overflow the container instead of fitting within it. Allow the items to flow onto new rows when they run out of space.",
 			"previewHTML": "<div class=\"cards\"><article class=\"card\">Card 1</article><article class=\"card\">Card 2</article><article class=\"card\">Card 3</article><article class=\"card\">Card 4</article><article class=\"card\">Card 5</article><article class=\"card\">Card 6</article></div>",
 			"previewBaseCSS": "body { font-family: system-ui, sans-serif; padding: 1rem; background: #f5f5f5; } .cards { display: flex; gap: 1rem; } .card { background: white; padding: 2rem; border-radius: 8px; box-shadow: 0 2px 4px rgba(0,0,0,0.1); min-width: 120px; text-align: center; }",
 			"sandboxCSS": "",
@@ -106,7 +106,7 @@
 				{
 					"type": "property_value",
 					"value": { "property": "flex-wrap", "expected": "wrap" },
-					"message": "Set <kbd>flex-wrap: wrap</kbd>"
+					"message": "Use the property that allows flex items to wrap onto new lines"
 				}
 			]
 		},
@@ -114,7 +114,7 @@
 			"id": "flexbox-6",
 			"title": "Flex Grow",
 			"description": "The <kbd>flex</kbd> property on items controls how they grow and shrink. <kbd>flex: 1</kbd> makes an item grow to fill available space. Multiple items with <kbd>flex: 1</kbd> share space equally.",
-			"task": "Make the search input expand to fill available space by setting <kbd>flex: 1</kbd> on <kbd>.search</kbd>.",
+			"task": "The search input is too narrow. Make it stretch to fill all the remaining space in the toolbar.",
 			"previewHTML": "<div class=\"toolbar\"><input class=\"search\" type=\"text\" placeholder=\"Search...\"><button class=\"btn\">Search</button><button class=\"btn\">Filters</button></div>",
 			"previewBaseCSS": "body { font-family: system-ui, sans-serif; padding: 1rem; } .toolbar { display: flex; gap: 8px; padding: 1rem; background: #f5f5f5; border-radius: 8px; } .search { padding: 8px 1rem; border: 1px solid #ddd; border-radius: 4px; font-size: 1rem; } .btn { padding: 8px 1rem; background: steelblue; color: white; border: none; border-radius: 4px; cursor: pointer; }",
 			"sandboxCSS": "",
@@ -125,9 +125,9 @@
 			"previewContainer": "preview-area",
 			"validations": [
 				{
-					"type": "property_value",
-					"value": { "property": "flex", "expected": "1" },
-					"message": "Set <kbd>flex: 1</kbd>"
+					"type": "regex",
+					"value": "(flex\\s*:\\s*1|flex-grow\\s*:\\s*1)",
+					"message": "Use the property that makes a flex item grow to fill available space"
 				}
 			]
 		}

specs/003-flexbox-task-wording/plan.md

@@ -0,0 +1,76 @@
+# Implementation Plan
+
+## Objective
+
+Rewrite all 6 flexbox lesson task descriptions to describe the desired visual outcome instead of giving the exact CSS declaration. Update validation messages to hint without revealing answers, and accept alternative valid solutions where applicable.
+
+## Approach
+
+This is a content-only change to a single JSON file (`lessons/flexbox.json`). Each lesson needs three edits:
+
+1. **Task text**: Replace copy-pasteable CSS declarations with outcome-oriented descriptions
+2. **Validation messages**: Replace answer-revealing messages with pedagogical hints
+3. **Validations array**: Add alternative accepted solutions where multiple CSS approaches achieve the same visual result
+
+The lesson `description` fields (which teach concepts with code examples) remain unchanged — they are the learning material, not the exercise prompt.
+
+## File Mapping
+
+| File | Action | Description |
+|------|--------|-------------|
+| `lessons/flexbox.json` | modify | Rewrite `task` and validation `message` fields for all 6 lessons; add alternative validations for flexbox-6 |
+
+No new files need to be created. No validator code changes needed — the existing `property_value` and `regex` validation types already support everything required.
+
+## Detailed Changes Per Lesson
+
+### flexbox-1 (Container)
+- **Task**: Describe that nav links stack vertically and should display side by side
+- **Validation msg**: Hint at display property for flex layout
+- **Alt solutions**: None — `display: flex` is the only correct answer (inline-flex changes block behavior)
+
+### flexbox-2 (Gap)
+- **Task**: Describe that links are crammed together and need 1rem of spacing between them
+- **Validation msg**: Hint at the gap property
+- **Alt solutions**: None — `gap: 1rem` is the specific expected value
+
+### flexbox-3 (Justify Content)
+- **Task**: Describe that Login button should be pushed to the far right, with nav links on the left
+- **Validation msg**: Hint at main-axis distribution property
+- **Alt solutions**: None — `justify-content: space-between` is the only property that works when targeting `.nav`
+
+### flexbox-4 (Align Items)
+- **Task**: Describe the visual misalignment and ask for vertical centering
+- **Validation msg**: Hint at cross-axis alignment property
+- **Alt solutions**: None — `align-items: center` is the correct answer
+
+### flexbox-5 (Flex Wrap)
+- **Task**: Describe cards overflowing and needing to flow onto new rows
+- **Validation msg**: Hint at wrapping property
+- **Alt solutions**: None — `flex-wrap: wrap` is the only answer
+
+### flexbox-6 (Flex Grow)
+- **Task**: Describe that the search input should stretch to fill remaining space
+- **Validation msg**: Hint at flex growth property
+- **Alt solutions**: Accept both `flex: 1` and `flex-grow: 1` via regex validation
+
+## Architecture Decisions
+
+1. **No validator code changes**: The existing `regex` validation type can handle alternative solutions for flexbox-6. No need to add a new validation type.
+2. **Keep values in tasks where needed**: Some tasks mention target values like "1rem" since the validator checks exact values and students need to know the amount. The key change is removing the *property name* from the task.
+3. **Solution field unchanged**: The `solution` field is used for the "show solution" feature and should remain as the canonical answer.
+4. **codePrefix unchanged**: The existing codePrefix already shows the selector context (e.g., `.nav {`), which is enough guidance for students.
+
+## Risks
+
+| Risk | Likelihood | Mitigation |
+|------|-----------|------------|
+| Tasks become too vague for beginners | Low | Descriptions still teach the property; tasks describe specific visual outcomes |
+| Alternative regex validation too permissive | Low | Regex will be specific to `flex:\s*1` and `flex-grow:\s*1` patterns |
+| Validation messages too cryptic | Low | Messages will hint at the property category without giving the exact declaration |
+
+## Testing Strategy
+
+1. **Run existing test suite**: `npm run test` — all tests should pass since no code or module structure changes
+2. **Manual verification**: Validate that each rewritten task accurately describes the visual outcome shown in the preview
+3. **JSON schema validation**: Ensure `lessons/flexbox.json` still conforms to the module schema

specs/003-flexbox-task-wording/spec.md

@@ -0,0 +1,35 @@
+# fix: remove answers from flexbox task descriptions (copy-paste score 95%)
+
+**Issue**: [libretech/code-crispies#3](https://git.librete.ch/libretech/code-crispies/issues/3)
+**State**: open
+**Author**: libretech
+**Labels**: none
+**Complexity**: simple
+
+## Issue Body
+
+Pedagogy audit: All 6 flexbox exercises give the exact CSS declaration in the task text. Students type without understanding. Rewrite tasks to describe the DESIRED OUTCOME instead of the exact code. Example: 'Add display: flex' → 'The navigation links stack vertically. Make them display side by side.' Accept multiple valid solutions in validations.
+
+## Current State
+
+All 6 lessons in `lessons/flexbox.json` have task descriptions that include the exact CSS declaration students need to type:
+
+| Lesson | Current Task (gives away answer) |
+|--------|----------------------------------|
+| flexbox-1 | "Add `display: flex` to `.nav`" |
+| flexbox-2 | "Add `gap: 1rem` to space out..." |
+| flexbox-3 | "setting `justify-content: space-between` on the nav" |
+| flexbox-4 | "Center them vertically with `align-items: center`" |
+| flexbox-5 | "Add `flex-wrap: wrap` to allow them to wrap" |
+| flexbox-6 | "setting `flex: 1` on `.search`" |
+
+Validation error messages also give away answers (e.g., "Set `display: flex`").
+
+## Acceptance Criteria
+
+1. All 6 flexbox task descriptions rewritten to describe the desired visual outcome, not the exact CSS code
+2. Students cannot copy-paste from the task into the editor to pass
+3. Validation error messages updated to provide hints without revealing the exact declaration
+4. Where applicable, validations accept multiple valid CSS solutions (e.g., `flex: 1` and `flex-grow: 1`)
+5. Existing tests continue to pass
+6. Lesson descriptions (which teach the concepts) remain unchanged

specs/003-flexbox-task-wording/tasks.md

@@ -0,0 +1,13 @@
+# Tasks
+
+## Phase 1: Core Content Changes
+- [X] Task 1.1: Rewrite task text for all 6 flexbox lessons to describe visual outcomes [P]
+- [X] Task 1.2: Rewrite validation error messages to hint without revealing answers [P]
+
+## Phase 2: Alternative Validations
+- [X] Task 2.1: Add regex validation for flexbox-6 to accept both `flex: 1` and `flex-grow: 1`
+
+## Phase 3: Validation
+- [X] Task 3.1: Run existing test suite to confirm no regressions
+- [X] Task 3.2: Verify flexbox.json still conforms to module schema
+- [X] Task 3.3: Run lesson format check (`npm run format.lessons`)