fix(skills): split SQL example out of go-fenced block in NULL-safe scans guidance

63f745b879731b7e3225b00c020f4d586ca06021 · 2026-05-16 16:22:48 -0700 · Trevin Chow

Per Greptile review on PR #1530: the COALESCE example was appended
inside the same ```go fenced block as the Go snippets, so syntax
highlighters, AI tools that read SKILL.md, and lint passes that
validate fenced blocks would all interpret the bare SELECT as Go.
Move the SQL into its own ```sql fenced block with a short
introductory sentence, leaving the Go snippets self-contained.

Files touched

M skills/printing-press/SKILL.md

Diff

commit 63f745b879731b7e3225b00c020f4d586ca06021
Author: Trevin Chow <trevin@trevinchow.com>
Date:   Sat May 16 16:22:48 2026 -0700

    fix(skills): split SQL example out of go-fenced block in NULL-safe scans guidance
    
    Per Greptile review on PR #1530: the COALESCE example was appended
    inside the same ```go fenced block as the Go snippets, so syntax
    highlighters, AI tools that read SKILL.md, and lint passes that
    validate fenced blocks would all interpret the bare SELECT as Go.
    Move the SQL into its own ```sql fenced block with a short
    introductory sentence, leaving the Go snippets self-contained.
---
 skills/printing-press/SKILL.md | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/skills/printing-press/SKILL.md b/skills/printing-press/SKILL.md
index c4198fbf..d45318a7 100644
--- a/skills/printing-press/SKILL.md
+++ b/skills/printing-press/SKILL.md
@@ -2591,8 +2591,11 @@ if err := rows.Scan(&id, &name); err != nil { continue }
 var name sql.NullString
 if err := rows.Scan(&id, &name); err != nil { continue }
 result.Name = name.String
+```
+
+Also right: push the default into the query so the scan target stays bare.
 
-// Also right — push the default into the query so the scan target stays bare.
+```sql
 SELECT id, COALESCE(json_extract(data, '$.name'), '') FROM resources WHERE ...
 ```

← f7365c26 fix(ci): close jq if-then-else with end so >10-thread path d · back to Cli Printing Press · fix(cli): drive browser-sniff http_transport from HAR HTTP-v 833213ff →