Skip to content

Restrict deployment Notion access to shared content - #1375

Merged
mrubens merged 3 commits into
developfrom
codex/notion-deployment-access-mode
Aug 15, 2026
Merged

Restrict deployment Notion access to shared content#1375
mrubens merged 3 commits into
developfrom
codex/notion-deployment-access-mode

Conversation

@mrubens

@mrubens mrubens commented Aug 15, 2026

Copy link
Copy Markdown
Contributor

Summary

  • replace the hosted Notion MCP connection with a deployment-managed Notion internal integration
  • keep the integration secret server-side and let Notion enforce access only to pages and data sources explicitly shared with the integration
  • retain fail-closed read-only access by default, with an admin-controlled read-write option and per-tool overrides
  • clear legacy broad OAuth credentials when an administrator configures the restricted integration
  • update Settings guidance, runtime setup, tests, docs, and release notes

Validation

  • pnpm lint
  • pnpm check-types
  • pnpm knip
  • pnpm --filter @roomote/docs check
  • focused types, API, SDK, web, worker, cloud-agent, and Slack tests
  • full repository suite: 1 unrelated worker harness test timed out once; its complete 46-test file passed immediately on rerun

@roomote-community

roomote-community Bot commented Aug 15, 2026

Copy link
Copy Markdown
Contributor

No new code issues found. See task

  • Existing OAuth Notion configurations cannot be migrated in place from Settings (apps/web/src/components/settings/Integrations.tsx:1832).

Reviewed 936b642

Comment thread apps/web/src/components/settings/Integrations.tsx Outdated
@mrubens
mrubens merged commit 5748cae into develop Aug 15, 2026
18 checks passed
@mrubens
mrubens deleted the codex/notion-deployment-access-mode branch August 15, 2026 18:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant