Skip to content

fix(external-sources): default external integrations to disabled - #1734

Merged
limityan merged 1 commit into
GCWing:mainfrom
limityan:yanzhn/external-sources-default-disabled
Jul 24, 2026
Merged

fix(external-sources): default external integrations to disabled#1734
limityan merged 1 commit into
GCWing:mainfrom
limityan:yanzhn/external-sources-default-disabled

Conversation

@limityan

Copy link
Copy Markdown
Collaborator

Summary

  • default external AI integrations to disabled for new or missing policy state
  • keep explicitly persisted user choices unchanged
  • fail closed in the Web UI when an older host omits the enabled flag
  • preserve the existing recommended access profile after a user explicitly enables integrations

Why

The shared policy default and the Web UI fallback both previously treated external integrations as enabled. That could make external AI application content available before a user knowingly opted in.

User impact

New or unconfigured users now see external AI application access turned off and can enable it from settings. Existing explicit enabled or disabled values continue to be respected; this change does not migrate or overwrite saved choices.

Validation

  • cargo test -p bitfun-product-domains --features external-sources --test external_source_contracts (37 passed)
  • pnpm --dir src/web-ui run test:run src/infrastructure/config/components/ExternalSourcesConfig.test.tsx (46 passed)
  • pnpm run type-check:web
  • node scripts/check-core-boundaries.mjs
  • pnpm run check:repo-hygiene
  • git diff --check

@limityan
limityan marked this pull request as ready for review July 24, 2026 07:10
@limityan
limityan force-pushed the yanzhn/external-sources-default-disabled branch from 449fa0d to ecea0cc Compare July 24, 2026 07:29
@limityan
limityan force-pushed the yanzhn/external-sources-default-disabled branch from ecea0cc to c34fa25 Compare July 24, 2026 07:30
@limityan
limityan merged commit 76addac into GCWing:main Jul 24, 2026
5 checks passed
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