fix: can miss sharepoint name on partial falure

[Dakuan]

Description

Fixes SharePoint knowledge sources sometimes persisting without their site name or URL when the connect-time site listing omits the selected site. The selected site metadata is now submitted with the connection request and used as a fallback preventing the loading placeholder or raw SharePoint site ID from becoming the permanent display name.

---

Summary by cubic

Prevents SharePoint knowledge sources from saving without a site name or URL by sending the selected site’s metadata with the connection request and using it as a fallback when the site listing is missing.

• Bug Fixes

  • Builder now submits a site object (id, name, webUrl) instead of siteId.
  • Server accepts site, prefers current listing data, and falls back to submitted metadata to avoid placeholder or raw ID names.
  • Validation and types updated; adds tests for fallback behavior and empty site id rejection.

• Migration

  • Update clients to send site: { id: string, name?: string, webUrl?: string } instead of siteId.

^{Written for commit 34a71a6. Summary will update on new commits.}

[cubic-dev-ai]

No issues found across 5 files

Confidence score: 5/5

• Automated review surfaced no issues in the provided summaries.
• No files require special attention.

_{Re-trigger cubic}

[deanhannigan]

LGTM! Nice one! 🚀