fix(ci): resolve YAML parsing error in amber-auto-review workflow (#419)

## Summary

Fixes the CI failure affecting PR #417 and all future pull requests.

## Problem

The `amber-auto-review.yml` workflow was failing with:
```
SyntaxError: Unexpected end of input
```

**Root Cause**: The YAML parser was confused by the `---` markdown
separator on line 168, which appeared as a standalone line inside a
JavaScript template literal. YAML interprets `---` as a document
separator, causing the workflow file structure to become invalid.

## Solution

Converted the JavaScript template literal to string concatenation to
avoid having `---` appear as a standalone line in the YAML structure.
This prevents the YAML parser from misinterpreting markdown formatting
as YAML syntax.

### Changes
- Replaced multiline template literal (lines 166-189) with string
concatenation
- No functional changes - the generated markdown is identical
- Resolved yamllint syntax errors

## Testing

- ✅ Local YAML validation passes (yamllint)
- ✅ File structure validated
- ✅ Will be tested on this PR's CI run

## Impact

- Fixes PR #417 CI failure
- Prevents future failures on all PRs with Amber code review enabled

---

Closes #417 (CI failure)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude <noreply@anthropic.com>

Author: jeremyeder

.github/workflows/amber-auto-review.yml

@@ -163,30 +163,20 @@ jobs:
               return;
             }
 
-            const transparencySection = `
-
----
-🔍 [View AI decision process](${serverUrl}/${repository}/actions/runs/${runId}) (logs available for 90 days)
-
-<details>
-<summary>📋 View memory system files loaded (click to expand)</summary>
-
-### What Amber Loaded for Code Review
-
-Amber automatically loaded these repository standards from the memory system:
-
-1. **CLAUDE.md** - Master project instructions, development standards
-2. **backend-development.md** - Go backend, K8s integration patterns
-3. **frontend-development.md** - NextJS, Shadcn UI, React Query patterns
-4. **security-standards.md** - Auth, RBAC, token handling
-5. **k8s-client-usage.md** - User token vs service account patterns
-6. **error-handling.md** - Consistent error patterns
-7. **react-query-usage.md** - Data fetching patterns
-
-**Impact**: This review used your repository's specific code quality standards, security patterns, and best practices from the memory system (PRs #359, #360) - not just generic code review guidelines.
-
-</details>
-`;
+            const transparencySection = '\n\n---\n🔍 [View AI decision process](' + serverUrl + '/' + repository + '/actions/runs/' + runId + ') (logs available for 90 days)\n\n' +
+              '<details>\n' +
+              '<summary>📋 View memory system files loaded (click to expand)</summary>\n\n' +
+              '### What Amber Loaded for Code Review\n\n' +
+              'Amber automatically loaded these repository standards from the memory system:\n\n' +
+              '1. **CLAUDE.md** - Master project instructions, development standards\n' +
+              '2. **backend-development.md** - Go backend, K8s integration patterns\n' +
+              '3. **frontend-development.md** - NextJS, Shadcn UI, React Query patterns\n' +
+              '4. **security-standards.md** - Auth, RBAC, token handling\n' +
+              '5. **k8s-client-usage.md** - User token vs service account patterns\n' +
+              '6. **error-handling.md** - Consistent error patterns\n' +
+              '7. **react-query-usage.md** - Data fetching patterns\n\n' +
+              '**Impact**: This review used your repository\'s specific code quality standards, security patterns, and best practices from the memory system (PRs #359, #360) - not just generic code review guidelines.\n\n' +
+              '</details>';
 
             const updatedBody = reviewComment.body + transparencySection;