Skip to content

fix(external-sources): clarify MCP and agent reviews - #1735

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

fix(external-sources): clarify MCP and agent reviews#1735
limityan merged 1 commit into
GCWing:mainfrom
limityan:yanzhn/external-sources-review-details

Conversation

@limityan

@limityan limityan commented Jul 24, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • keep pending MCP reviews compact while leaving the command or remote URL and a concise execution/data risk warning visible before approval
  • keep working directory, environment, header, and full configuration details behind an explicit disclosure, and associate the risk note with the enable action for assistive technology
  • report exact unavailable tool names and setting-specific OpenCode agent diagnostics while preserving older host and snapshot compatibility

Verification

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

A broad workspace/Core compile was intentionally not run locally because this is a focused change; CI remains the compilation gate.

@limityan
limityan marked this pull request as ready for review July 24, 2026 07:21
@limityan
limityan force-pushed the yanzhn/external-sources-review-details branch from 5f69785 to 1e89edd Compare July 24, 2026 07:46
@limityan
limityan merged commit f7289ce 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