fix(notion-search): correct --filter example syntax in docs

Final review caught that both --filter example locations used simplified
JSON ({"Status": "Done"}) that Notion's data_sources.query API rejects
with a 400. The script passes --filter verbatim, so users copy-pasting
the example would hit a confusing error.

Replace with Notion's actual filter shape:
  {"property": "Status", "status": {"equals": "Done"}}

Also added a compound (and/or) example in CLAUDE.md so users have a
reference for combining filters.

30/30 tests still pass.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This commit is contained in:
2026-04-28 13:57:29 +09:00
parent c1061dcc71
commit 69526d345a
2 changed files with 7 additions and 3 deletions

View File

@@ -28,9 +28,13 @@ python scripts/notion_search.py "AI agents" --json | jq '.[].id'
# Constrain to specific databases # Constrain to specific databases
python scripts/notion_search.py "MCP" --databases f8f19ede-32bd-43ac-9f60-0651f6f40afe python scripts/notion_search.py "MCP" --databases f8f19ede-32bd-43ac-9f60-0651f6f40afe
# Property filter # Property filter (Notion's filter shape — passed verbatim to data_sources.query)
python scripts/notion_search.py "MCP" --databases ID \ python scripts/notion_search.py "MCP" --databases ID \
--filter '{"Status": "Done", "Topic": "AI"}' --filter '{"property": "Status", "status": {"equals": "Done"}}'
# Compound filter (and/or)
python scripts/notion_search.py "MCP" --databases ID \
--filter '{"and": [{"property": "Status", "status": {"equals": "Done"}}, {"property": "Topic", "multi_select": {"contains": "AI"}}]}'
# Fast mode (skip LLM stages) # Fast mode (skip LLM stages)
python scripts/notion_search.py "exact term" --no-rerank --no-expand python scripts/notion_search.py "exact term" --no-rerank --no-expand

View File

@@ -19,7 +19,7 @@ python3 notion_search.py "$ARGUMENTS"
- **Default browse mode** (terminal table): `notion-search "AI agents in 2026"` - **Default browse mode** (terminal table): `notion-search "AI agents in 2026"`
- **JSON for piping** (e.g. into the future notion-to-notebooklm push skill): `notion-search "AI agents" --json | jq '.[].id'` - **JSON for piping** (e.g. into the future notion-to-notebooklm push skill): `notion-search "AI agents" --json | jq '.[].id'`
- **Constrain to specific databases**: `notion-search "MCP" --databases f8f19ede-32bd-43ac-9f60-0651f6f40afe` - **Constrain to specific databases**: `notion-search "MCP" --databases f8f19ede-32bd-43ac-9f60-0651f6f40afe`
- **Property filter** (per-database mode): `notion-search "MCP" --databases ID --filter '{"Status": "Done"}'` - **Property filter** (per-database mode, Notion's native filter shape): `notion-search "MCP" --databases ID --filter '{"property": "Status", "status": {"equals": "Done"}}'`
- **Fast mode (no LLM)**: `notion-search "exact phrase" --no-rerank --no-expand` - **Fast mode (no LLM)**: `notion-search "exact phrase" --no-rerank --no-expand`
## Requirements ## Requirements