Repository navigation
[P2] Add Notion as a catalog platform (via localthought/overlays) #72
Description
Activity
Verified already resolved on
mainbefore this issue was filed — Notion is registered in the application's actual default catalog pin (config::DEFAULT_CATALOG_PATH, bumped in commit 8f22a85, two days before this issue) with a working auth overlay (notionOAuthsecurity scheme) and existing fixture tests (tests/fixtures/notion-composed.yaml,composed_notion_preserves_base_path_and_resolves_version_header,composed_notion_fixture_builds_json_token_request). #73 adds a further regression test (default_catalog_pin_selects_google_offline_and_spotify_pkce_profiles) that loads the live default pin over the network and confirmsoauth_provider("notion")resolves successfully alongside the other platforms.This issue's own hand-off framing correctly identified that step 3 (bumping
CATALOG_PATH) belongs inintegration-proxy— that bump had already happened. No furtherintegration-proxycode change is needed. Closing with no additional code change.
Generated by Claude Code
Summary
The Notion connector in atomic-server's Integrations UI fails to connect. Root cause:
notionis not a registered platform in the LocalThought catalog — there is no OpenAPI source or overlay set for it inlocalthought/overlays'catalog.json, andintegration-proxytherefore has nothing to compose or serve at/catalog/notion.yaml.Why this isn't an
integration-proxycode changePer AGENTS.md, platform-specific data (OpenAPI documents, CRUD/collection structure,
crudResources, pagination, auth, throttling) must not be hand-authored in this repo. It belongs inlocalthought/overlays, referenced by pinned, immutable URLs from that repo'scatalog.json. This repo only ever gets a one-line bump of the pinnedCATALOG_PATHdefault in src/config.rs once the new revision exists.Hand-off plan
localthought/overlays: author Notion's platform data, following the pattern used forgithub-issues/google-calendar/moneybird/todoist/spotify/discord:localthought/openapi-directoryif a suitable OAD already exists there, otherwise author one).auth-overlay.yaml— OAuth2 security scheme + flow.pagination-overlay.yaml— Notion's cursor-based pagination scheme.crud-causality-overlay.yaml—components.crudResourcesfor the resources the plugin needs (e.g. pages, databases), matching the shape validated bysrc/catalog.rs's pinned-fixture tests for other platforms.notionentry tolocalthought/overlays' rootcatalog.jsonreferencing the pinned commit URLs from step 1, and publish/commit it.integration-proxy: bump the defaultCATALOG_PATHinsrc/config.rsto the newcatalog.jsoncommit, restart the service, and optionally add a pinned-fixture test (in the style of the existing#[ignore]d tests insrc/catalog.rs) asserting the composed Notion document looks right.References
browser/data-browser/src/chunks/PluginRuns/ConnectNotion.tsxnotionentry): https://raw.githubusercontent.com/localthought/overlays/d83c3ce0afd9f8ca0e4c42e142fa89d5fa9d8f70/catalog.jsonNo implementation changes are proposed here; this issue is a hand-off record only.