From 4e692d0e9a377c7755d34af9ca22c6c60403a215 Mon Sep 17 00:00:00 2001 From: Andrew Yim Date: Mon, 27 Apr 2026 11:17:11 +0900 Subject: [PATCH] fix(notion-writer): add "object": "block" to create_callout_block Code review caught that all 9 other create_*_block factories include "object": "block" as the first key, but the new callout factory was missing it. The Notion API accepts both forms, but the inconsistency would compound; align with the existing convention. 22/22 tests still pass. Co-Authored-By: Claude Opus 4.7 (1M context) --- custom-skills/32-notion-writer/code/scripts/notion_writer.py | 1 + 1 file changed, 1 insertion(+) diff --git a/custom-skills/32-notion-writer/code/scripts/notion_writer.py b/custom-skills/32-notion-writer/code/scripts/notion_writer.py index 8bbb20f..3c91093 100644 --- a/custom-skills/32-notion-writer/code/scripts/notion_writer.py +++ b/custom-skills/32-notion-writer/code/scripts/notion_writer.py @@ -318,6 +318,7 @@ def create_callout_block(text: str, alert_type: str) -> Dict[str, Any]: """Create a callout block with icon + color from a GitHub alert type.""" icon, color = ALERT_TYPES[alert_type] return { + "object": "block", "type": "callout", "callout": { "rich_text": parse_rich_text(text),