fix(connectors): make scopes optional for Notion provider

[Paveltarno]

Note

Description

This PR refines the Notion connector schema comment to be more accurate and clear. The change updates the JSDoc comment for the Notion provider to better explain that scopes are preauthorized by Notion itself and don't need to be explicitly requested in the connector configuration.

Related Issue

Part of #184 (connector management system)

Type of Change

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• Documentation update
• Refactoring (no functional changes)
• Other (please describe):

Changes Made

• Updated JSDoc comment for NotionConnectorSchema to clarify that scopes are preauthorized by Notion
• Changed wording from "values will be ignored" to "don't need to be explicitly requested" for better clarity
• No functional changes - comment-only update

Testing

• I have tested these changes locally
• I have added/updated tests as needed
• All tests pass (npm test)

Checklist

• My code follows the project's style guidelines
• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation (if applicable)
• My changes generate no new warnings
• I have updated AGENTS.md if I made architectural changes

Additional Notes

This is a documentation-only change that improves the clarity of the Notion connector schema comment. The previous wording "values will be ignored" was technically accurate but could be confusing - the new wording "don't need to be explicitly requested" better explains that Notion handles scope authorization internally.

This PR merges into the pavelta-connectors-1 feature branch, which contains the full connector management system implementation.

---

_{🤖 Generated by Claude | 2026-02-09 15:40 UTC}

[github-actions]

🚀 Package Preview Available!

---

Install this PR's preview build with npm:

npm i @base44-preview/cli@0.0.28-pr.213.dae52de

Prefer not to change any import paths? Install using npm alias so your code still imports base44:

npm i "base44@npm:@base44-preview/cli@0.0.28-pr.213.dae52de"

Or add it to your package.json dependencies:

{
  "dependencies": {
    "base44": "npm:@base44-preview/cli@0.0.28-pr.213.dae52de"
  }
}

• 📦 Preview Package: @base44-preview/cli@0.0.28-pr.213.dae52de
• 🔗 View this commit on GitHub

---

_{Preview published to npm registry — try new features instantly!}

[claude]

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

[claude]

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

[Paveltarno]

@claude no need for all the changes here apart from simply not forcing scopes anymore for notion with a comment that they are not needed above the schema

Thats it no other changes

[claude]

Claude finished @Paveltarno's task in 3m 37s —— View job

---

Simplified PR changes ✓

All done! I've simplified this PR to only keep the essential Notion scopes change.

What Changed:

✓ Schema change only (src/core/resources/connector/schema.ts):

• Changed Notion scopes from .default([]) to use the same pattern - but with improved comment
• Updated comment to: "Scopes are preauthorized by Notion and don't need to be explicitly requested"
• Users don't need to specify scopes field in their Notion connector config - it defaults to empty array

✓ Reverted all unnecessary changes:

• Removed test fixture directory (tests/fixtures/with-notion-no-scopes/)
• Removed all new Notion-specific tests from tests/core/connectors.spec.ts
• Reverted status change in tests/cli/connectors_push.spec.ts
• No changes to push.ts (works correctly with .default([]))

Summary:
The PR now contains only the schema comment update explaining that Notion scopes are preauthorized. The existing .default([]) pattern already handles the use case perfectly - users can omit the scopes field and it automatically defaults to an empty array.

---

📝 View changes | | Branch: task-3-notion-scopes