Skip to content

fix(search): expose provider configuration updates in Sources - #7704

Merged
icecrasher321 merged 2 commits into
stagingfrom
codex/search-provider-config-refresh
Sep 10, 2026
Merged

fix(search): expose provider configuration updates in Sources#7704
icecrasher321 merged 2 commits into
stagingfrom
codex/search-provider-config-refresh

Conversation

@icecrasher321

Copy link
Copy Markdown
Collaborator

Summary

  • Expose the existing provider configuration update action in organization Sources settings so Search admins can recover after an OAuth App configuration changes.
  • Preserve provider identities and Slack configuration through shared option mapping, and correct the recovery instructions in Search and self-hosting docs.

Type of Change

  • Bug fix

Testing

34 focused UI and credential-group tests passed. App type check, lint, all 46 repository audits, block registry check, and docs manifest check passed.

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

@vercel

vercel Bot commented Sep 10, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
docs Ready Ready Preview Sep 10, 2026 12:58am UTC

Request Review

@greptile-apps

greptile-apps Bot commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

The PR appears safe to merge; no actionable regression or outstanding previous finding remains.

Summary

  • Reuses a shared option mapper so provider identities and Slack-specific configuration survive policy refreshes.
  • Makes organization-specific configuration errors direct administrators to the new Sources action.
  • Adds focused tests for successful, failed, unavailable, and pending update states.
  • The previously reported missing TSDoc was added, and that thread is resolved.

Diagram

%%{init: {'theme': 'neutral'}}%%
flowchart LR
  A[Organization admin opens Sources] --> B[Load organization account group]
  B --> C[Update configurations]
  C --> D[Map existing provider options]
  D --> E[Authorized organization-account update]
  E --> F[Refresh managed OAuth policies]
  F --> G[Invalidate affected account and Search queries]
Loading

Reviews (2) · Last reviewed commit: "chore(search): document provider refresh..."

Comment thread apps/sim/lib/credential-groups/organization-account-options.ts
@icecrasher321
icecrasher321 force-pushed the codex/search-provider-config-refresh branch from 1ffc5d2 to 39438f8 Compare September 10, 2026 00:56
@icecrasher321

Copy link
Copy Markdown
Collaborator Author

@greptile

@icecrasher321

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@icecrasher321 I have started the AI code review. It will take a few minutes to complete.

@icecrasher321
icecrasher321 merged commit 57c1c7c into staging Sep 10, 2026
33 checks passed
@waleedlatif1
waleedlatif1 deleted the codex/search-provider-config-refresh branch September 10, 2026 01:36
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