ourdigital/our-claude-skills
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(notion-writer): add "object": "block" to create_callout_block

Browse Source 
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) <noreply@anthropic.com>
This commit is contained in:
Andrew Yim
2026-04-27 11:17:11 +09:00
parent e0f93e6add
commit 4e692d0e9a
1 changed files with 1 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
custom-skills/32-notion-writer/code/scripts/notion_writer.py
View File

@@ -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.""" """Create a callout block with icon + color from a GitHub alert type."""
icon, color = ALERT_TYPES[alert_type] icon, color = ALERT_TYPES[alert_type]
return { return {
"object": "block",
"type": "callout", "type": "callout",
"callout": { "callout": {
"rich_text": parse_rich_text(text), "rich_text": parse_rich_text(text),