Create the_case_for_an_ai_header.mdx#287
Conversation
|
@claude Assess for typos and propose a commit to fix please |
|
Claude finished @dwjohnston's task in 50s —— View job Claude finished the typo reviewTodo list:
Typos fixed
Pushed as commit Note: I didn't touch the nested Markdown/TypeScript code-fence structure (lines 16–39) since that's a formatting choice rather than a typo, and it wasn't in scope for this request. |
Co-authored-by: David Johnston <dwjohnston@users.noreply.github.com>
|
|
||
|
|
||
| This is simple enough prompt that is hard to get wrong. | ||
| This is a simple enough prompt that it's hard to get wrong. |
There was a problem hiding this comment.
There was a problem hiding this comment.
Claude finished @dwjohnston's task in 1m 15s —— View job
Todo
- Read the trigger comment and current file state
- Update line 42 wording per feedback
- Commit and push
- Summarize
Response
Fair pushback — you're right. Looking at it again:
- Original: "This is simple enough prompt that is hard to get wrong." — missing the article "a" (a genuine typo).
- My earlier "fix": "This is a simple enough prompt that it's hard to get wrong." — the "it's" swap wasn't needed; it was an unnecessary rewrite, not a typo fix.
- Your version: "This is a simple enough prompt that is hard to get wrong." — grammatically correct as-is (the "that" clause modifies "prompt" directly), and just fixes the missing article.
I've reverted the wording to your version and pushed it as commit aadf79f directly to this PR's branch.
Co-authored-by: David Johnston <dwjohnston@users.noreply.github.com>
|
|
||
| When creating a new file add the following header to the top of it | ||
|
|
||
| \`\`\`typescript |
There was a problem hiding this comment.
@claude this isn't playing very nicely - how am I meant to put these triple back ticks into a code block?
There was a problem hiding this comment.
Claude finished @dwjohnston's task in 1m 2s —— View job
Todo
- Read
src/routes/posts/the_case_for_an_ai_header.mdxto see the nested code-fence issue - Figure out how to properly escape/nest triple backticks in MDX
- Implement the fix and push
- Summarize
The issue
The old markup tried to nest a code block by escaping the inner backticks with backslashes:
```md
...
\`\`\`typescript
...
\`\`\`
...
That doesn't work here — MDX (via `@mdx-js/mdx` v3 / remark) follows CommonMark, and **backslash escapes are not processed inside fenced code blocks**. So `\`\`\`` is rendered literally as `\`\`\`` rather than becoming a real triple-backtick fence.
### The fix
The correct CommonMark way to nest a code block inside another is to make the **outer** fence *longer* than the inner one. I changed the outer fence from three backticks to four, and left the inner `typescript` fence as plain triple backticks:
````md
````md
## Agent Instructions
...
```typescript
...
...
This way the parser only closes the outer block when it hits a matching (4-backtick) fence, and the inner triple-backtick fence renders as a normal nested code block.
Pushed as commit e9015d8 directly to this PR's branch (the-case-for-an-ai-header).
Co-authored-by: David Johnston <dwjohnston@users.noreply.github.com>
No description provided.