You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@superset.apache.org by "michael-s-molina (via GitHub)" <gi...@apache.org> on 2026/01/19 14:14:18 UTC
[PR] fix(mcp): prevent MCP service from corrupting shared appbuilder singleton [superset]
michael-s-molina opened a new pull request, #37252:
URL: https://github.com/apache/superset/pull/37252
### SUMMARY
Fixes a bug where the MCP service would corrupt Flask-AppBuilder's shared `appbuilder` singleton, causing `BuildError: Could not build url for endpoint 'None.login'` errors when accessing Superset.
**Root Cause:**
PR #37190 (`fix(mcp): push Flask app context in mcp_auth_hook for tool execution`) added `from superset.mcp_service.flask_singleton import get_flask_app` to `auth.py`. During main app initialization, when MCP tools are decorated with `@tool` (which uses `mcp_auth_hook`), this import triggers `flask_singleton.py` to be loaded. The `flask_singleton.py` module then calls `create_app()` creating a second Flask app that re-initializes the shared `appbuilder` singleton.
When `appbuilder.init_app()` is called a second time:
1. Flask-AppBuilder's `add_view_no_menu()` detects the view class is already registered
2. It returns a **new instance** without calling `create_blueprint()` (which sets the `endpoint` attribute)
3. The view's `endpoint` remains `None`
4. When the main app tries to build URLs (e.g., `get_url_for_login`), it fails with `BuildError`
**The Fix:**
Modified `superset/mcp_service/flask_singleton.py` to check if `appbuilder.app` is already initialized. If so, it reuses the existing Flask app instead of creating a new one via `create_app()`. This prevents corruption of the shared appbuilder singleton.
```python
# Before: Always created a new app
_temp_app = create_app()
# After: Reuse existing app if appbuilder is already initialized
if appbuilder.app is not None:
app = appbuilder.app
else:
_temp_app = create_app()
# ... initialization code
```
### TESTING INSTRUCTIONS
1. Enable MCP service in your Superset configuration
2. Start Superset in debug mode: `superset run -p 8088 --reload --debugger`
3. Navigate to `http://localhost:8088/superset/welcome/`
4. Verify the login page loads without errors
5. Verify you can log in successfully
### ADDITIONAL INFORMATION
<!--- Check any relevant boxes with "x" -->
<!--- HINT: Include "Fixes #nnn" if you are fixing an existing issue -->
- [ ] Has associated issue:
- [ ] Required feature flags:
- [ ] Changes UI
- [ ] Includes DB Migration (follow approval process in [SIP-59](https://github.com/apache/superset/issues/13351))
- [ ] Migration is atomic, supports rollback & is backwards-compatible
- [ ] Confirm DB migration upgrade and downgrade tested
- [ ] Runtime estimates and downtime expectations provided
- [ ] Introduces new feature or API
- [ ] Removes existing feature or API
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
Re: [PR] fix(mcp): prevent MCP service from corrupting shared appbuilder singleton [superset]
Posted by "bito-code-review[bot] (via GitHub)" <gi...@apache.org>.
bito-code-review[bot] commented on PR #37252:
URL: https://github.com/apache/superset/pull/37252#issuecomment-3768904059
AI Code Review is in progress (usually takes 3 to 15 minutes unless it's a very large PR).
<details>
<summary>
Bito Usage Guide
</summary>
**Commands**
Type the following command in the pull request comment and save the comment.
- `/review` - Manually triggers a full AI review.
- `/pause` - Pauses automatic reviews on this pull request.
- `/resume` - Resumes automatic reviews.
- `/resolve` - Marks all Bito-posted review comments as resolved.
- `/abort` - Cancels all in-progress reviews.
Refer to the <a href="https://docs.bito.ai/bito-dev-agents/ai-code-review-agent/available-commands" target="_blank" rel="noopener noreferrer">documentation</a> for additional commands.
**Configuration**
This repository uses `Superset` You can customize the agent settings <a href="https://alpha.bito.ai/home/ai-agents/code-review-agent" target="_blank" rel="noopener noreferrer">here</a> or contact your Bito workspace admin at evan@preset.io.
**Documentation & Help**
- <a href="https://docs.bito.ai/bito-dev-agents/ai-code-review-agent/getting-started/install-run-using-bito-cloud/create-or-customize-an-agent-instance" target="_blank" rel="noopener noreferrer">Customize agent settings</a>
- <a href="https://docs.bito.ai/bito-dev-agents/ai-code-review-agent/implementing-custom-code-review-rules" target="_blank" rel="noopener noreferrer">Review rules</a>
- <a href="https://docs.bito.ai/bito-dev-agents/ai-code-review-agent" target="_blank" rel="noopener noreferrer">General documentation</a>
- <a href="https://docs.bito.ai/bito-dev-agents/ai-code-review-agent/faqs" target="_blank" rel="noopener noreferrer">FAQ</a>
</details>
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
Re: [PR] fix(mcp): prevent MCP service from corrupting shared appbuilder singleton [superset]
Posted by "michael-s-molina (via GitHub)" <gi...@apache.org>.
michael-s-molina commented on code in PR #37252:
URL: https://github.com/apache/superset/pull/37252#discussion_r2704995795
##########
superset/mcp_service/flask_singleton.py:
##########
@@ -33,34 +33,47 @@
logger.info("Creating Flask app instance for MCP service")
try:
- from superset.app import create_app
- from superset.mcp_service.mcp_config import get_mcp_config
-
- # Create a temporary context to avoid
- # "Working outside of application context" errors.
- _temp_app = create_app()
-
- # Push an application context and initialize core dependencies and extensions
- with _temp_app.app_context():
- # Apply MCP configuration - reads from app.config first, falls back to defaults
- mcp_config = get_mcp_config(_temp_app.config)
- _temp_app.config.update(mcp_config)
- try:
- from superset.initialization import SupersetAppInitializer
-
- # Create initializer and run only dependency injection
- # NOT the full init_app_in_ctx which includes web views
- initializer = SupersetAppInitializer(_temp_app)
- initializer.init_all_dependencies_and_extensions()
-
- logger.info("Core dependencies and extensions initialized for MCP service")
- except Exception as e:
- logger.warning("Failed to initialize dependencies for MCP service: %s", e)
-
- # Store the app instance for later use
- app = _temp_app
-
- logger.info("Minimal Flask app instance created successfully for MCP service")
+ from superset.extensions import appbuilder
+
+ # Check if appbuilder is already initialized (main Superset app is running).
+ # If so, reuse that app to avoid corrupting the shared appbuilder singleton.
+ # Calling create_app() again would re-initialize appbuilder and break view endpoints.
+ if appbuilder.app is not None:
+ logger.info("Reusing existing Flask app from appbuilder for MCP service")
+ app = appbuilder.app
+ else:
+ from superset.app import create_app
+ from superset.mcp_service.mcp_config import get_mcp_config
+
+ # Create a temporary context to avoid
+ # "Working outside of application context" errors.
+ _temp_app = create_app()
Review Comment:
Resolved!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
Re: [PR] fix(mcp): prevent MCP service from corrupting shared appbuilder singleton [superset]
Posted by "codecov[bot] (via GitHub)" <gi...@apache.org>.
codecov[bot] commented on PR #37252:
URL: https://github.com/apache/superset/pull/37252#issuecomment-3768590468
## [Codecov](https://app.codecov.io/gh/apache/superset/pull/37252?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) Report
:x: Patch coverage is `0%` with `19 lines` in your changes missing coverage. Please review.
:white_check_mark: Project coverage is 48.12%. Comparing base ([`2900258`](https://app.codecov.io/gh/apache/superset/commit/2900258e0515bb9f73bbc2c6c0c2d0e336d075d1?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)) to head ([`b699020`](https://app.codecov.io/gh/apache/superset/commit/b699020828b8e46af7e69a0aeb995f11df3b90f2?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)).
| [Files with missing lines](https://app.codecov.io/gh/apache/superset/pull/37252?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Patch % | Lines |
|---|---|---|
| [superset/mcp\_service/flask\_singleton.py](https://app.codecov.io/gh/apache/superset/pull/37252?src=pr&el=tree&filepath=superset%2Fmcp_service%2Fflask_singleton.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-c3VwZXJzZXQvbWNwX3NlcnZpY2UvZmxhc2tfc2luZ2xldG9uLnB5) | 0.00% | [19 Missing :warning: ](https://app.codecov.io/gh/apache/superset/pull/37252?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
<details><summary>Additional details and impacted files</summary>
```diff
@@ Coverage Diff @@
## master #37252 +/- ##
===========================================
+ Coverage 0 48.12% +48.12%
===========================================
Files 0 644 +644
Lines 0 48229 +48229
Branches 0 5286 +5286
===========================================
+ Hits 0 23210 +23210
- Misses 0 24356 +24356
- Partials 0 663 +663
```
| [Flag](https://app.codecov.io/gh/apache/superset/pull/37252/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Ξ | |
|---|---|---|
| [hive](https://app.codecov.io/gh/apache/superset/pull/37252/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `42.75% <0.00%> (?)` | |
| [presto](https://app.codecov.io/gh/apache/superset/pull/37252/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `46.34% <0.00%> (?)` | |
| [python](https://app.codecov.io/gh/apache/superset/pull/37252/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `48.12% <0.00%> (?)` | |
| [unit](https://app.codecov.io/gh/apache/superset/pull/37252/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `100.00% <ΓΈ> (?)` | |
Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#carryforward-flags-in-the-pull-request-comment) to find out more.
</details>
[:umbrella: View full report in Codecov by Sentry](https://app.codecov.io/gh/apache/superset/pull/37252?dropdown=coverage&src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache).
:loudspeaker: Have feedback on the report? [Share it here](https://about.codecov.io/codecov-pr-comment-feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache).
<details><summary> :rocket: New features to boost your workflow: </summary>
- :snowflake: [Test Analytics](https://docs.codecov.com/docs/test-analytics): Detect flaky tests, report on failures, and find test suite problems.
- :package: [JS Bundle Analysis](https://docs.codecov.com/docs/javascript-bundle-analysis): Save yourself from yourself by tracking and limiting bundle sizes in JS merges.
</details>
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
Re: [PR] fix(mcp): prevent MCP service from corrupting shared appbuilder singleton [superset]
Posted by "codeant-ai-for-open-source[bot] (via GitHub)" <gi...@apache.org>.
codeant-ai-for-open-source[bot] commented on PR #37252:
URL: https://github.com/apache/superset/pull/37252#issuecomment-3768559645
CodeAnt AI finished reviewing your PR.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
Re: [PR] fix(mcp): prevent MCP service from corrupting shared appbuilder singleton [superset]
Posted by "codeant-ai-for-open-source[bot] (via GitHub)" <gi...@apache.org>.
codeant-ai-for-open-source[bot] commented on code in PR #37252:
URL: https://github.com/apache/superset/pull/37252#discussion_r2704957755
##########
superset/mcp_service/flask_singleton.py:
##########
@@ -33,34 +33,47 @@
logger.info("Creating Flask app instance for MCP service")
try:
- from superset.app import create_app
- from superset.mcp_service.mcp_config import get_mcp_config
-
- # Create a temporary context to avoid
- # "Working outside of application context" errors.
- _temp_app = create_app()
-
- # Push an application context and initialize core dependencies and extensions
- with _temp_app.app_context():
- # Apply MCP configuration - reads from app.config first, falls back to defaults
- mcp_config = get_mcp_config(_temp_app.config)
- _temp_app.config.update(mcp_config)
- try:
- from superset.initialization import SupersetAppInitializer
-
- # Create initializer and run only dependency injection
- # NOT the full init_app_in_ctx which includes web views
- initializer = SupersetAppInitializer(_temp_app)
- initializer.init_all_dependencies_and_extensions()
-
- logger.info("Core dependencies and extensions initialized for MCP service")
- except Exception as e:
- logger.warning("Failed to initialize dependencies for MCP service: %s", e)
-
- # Store the app instance for later use
- app = _temp_app
-
- logger.info("Minimal Flask app instance created successfully for MCP service")
+ from superset.extensions import appbuilder
+
+ # Check if appbuilder is already initialized (main Superset app is running).
+ # If so, reuse that app to avoid corrupting the shared appbuilder singleton.
+ # Calling create_app() again would re-initialize appbuilder and break view endpoints.
+ if appbuilder.app is not None:
+ logger.info("Reusing existing Flask app from appbuilder for MCP service")
+ app = appbuilder.app
+ else:
+ from superset.app import create_app
+ from superset.mcp_service.mcp_config import get_mcp_config
+
+ # Create a temporary context to avoid
+ # "Working outside of application context" errors.
+ _temp_app = create_app()
Review Comment:
**Suggestion:** Logic error: calling `create_app()` here will run the full application bootstrap (including `app_initializer.init_app()`), which re-initializes Flask-AppBuilder and can recreate the shared `appbuilder` singleton β exactly what this PR was trying to avoid. Replace creation of a fully-initialized app with creating a minimal app instance (e.g., `SupersetApp(__name__)`) and explicitly disable debug to avoid side-effects like starting a file-watcher. [logic error]
<details>
<summary><b>Severity Level:</b> Critical π¨</summary>
```mdx
- β Login page URL building fails after double initialization.
- β View endpoints become None causing BuildError.
- β οΈ MCP tool import can corrupt shared appbuilder singleton.
- β οΈ _temp app runs full initializer during module import.
```
</details>
```suggestion
from superset.app import SupersetApp
from superset.mcp_service.mcp_config import get_mcp_config
# Create a minimal Superset app instance (don't run the full initializer).
# This avoids calling `app_initializer.init_app()` which would register views
# and re-initialize the shared appbuilder singleton.
_temp_app = SupersetApp(__name__)
# Ensure debug is disabled to avoid starting debug-only background threads.
_temp_app.debug = False
```
<details>
<summary><b>Steps of Reproduction β
</b></summary>
```mdx
1. Start Superset with MCP enabled (follow PR testing steps). The main process imports MCP
helpers during module import.
2. During import, `superset/mcp_service/flask_singleton.py` is executed (see file lines
where import occurs: line 36 imports `appbuilder`, lines 45-50 call `create_app()`).
3. `create_app()` in `superset/app.py` (shown in provided context starting at line 47)
executes `app_initializer.init_app()` as part of its normal flow (see
`app_initializer.init_app()` in `superset/app.py:~79`), registering views and extensions.
4. That registration can reinitialize Flask-AppBuilder, causing duplicate registration and
leaving view `.endpoint` attributes unset β reproducing the BuildError described in the PR
(login URL build failures observed when `create_app()` is invoked here).
Note: existing PR already guards re-initialization when `appbuilder.app` is present, but
when `appbuilder.app` is None this path runs the full app bootstrap (via `create_app()`),
which is the exact root cause this suggestion highlights.
```
</details>
<details>
<summary><b>Prompt for AI Agent π€ </b></summary>
```mdx
This is a comment left during a code review.
**Path:** superset/mcp_service/flask_singleton.py
**Line:** 45:50
**Comment:**
*Logic Error: Logic error: calling `create_app()` here will run the full application bootstrap (including `app_initializer.init_app()`), which re-initializes Flask-AppBuilder and can recreate the shared `appbuilder` singleton β exactly what this PR was trying to avoid. Replace creation of a fully-initialized app with creating a minimal app instance (e.g., `SupersetApp(__name__)`) and explicitly disable debug to avoid side-effects like starting a file-watcher.
Validate the correctness of the flagged issue. If correct, How can I resolve this? If you propose a fix, implement it and please make it concise.
```
</details>
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
Re: [PR] fix(mcp): prevent MCP service from corrupting shared appbuilder singleton [superset]
Posted by "codeant-ai-for-open-source[bot] (via GitHub)" <gi...@apache.org>.
codeant-ai-for-open-source[bot] commented on PR #37252:
URL: https://github.com/apache/superset/pull/37252#issuecomment-3768546531
CodeAnt AI is reviewing your PR.
---
### Thanks for using CodeAnt! π
We're free for open-source projects. if you're enjoying it, help us grow by sharing.
[Share on X](https://twitter.com/intent/tweet?text=Just%20tried%20%40CodeAntAI%20for%20automated%20code%20review%20and%20I%27m%20impressed%21%20Free%20for%20open%20source%20with%20a%20free%20trial%20for%20private%20repos.%20Worth%20checking%20out%3A&url=https%3A//codeant.ai) Β·
[Reddit](https://www.reddit.com/submit?title=Check%20out%20CodeAnt%20for%20automated%20code%20review&text=Just%20tried%20CodeAnt%20for%20automated%20code%20review%20and%20I%27m%20impressed%21%20Free%20for%20open%20source%20with%20a%20free%20trial%20for%20private%20repos.%20Worth%20checking%20out%3A%20https%3A//codeant.ai) Β·
[LinkedIn](https://www.linkedin.com/sharing/share-offsite/?url=https%3A%2F%2Fcodeant.ai&mini=true&title=Check%20out%20CodeAnt%20for%20automated%20code%20review&summary=Just%20tried%20CodeAnt%20for%20automated%20code%20review%20and%20I%27m%20impressed%21%20Free%20for%20open%20source%20with%20a%20free%20trial%20for%20private%20repos)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
Re: [PR] fix(mcp): prevent MCP service from corrupting shared appbuilder singleton [superset]
Posted by "netlify[bot] (via GitHub)" <gi...@apache.org>.
netlify[bot] commented on PR #37252:
URL: https://github.com/apache/superset/pull/37252#issuecomment-3768559492
### <span aria-hidden="true">β
</span> Deploy Preview for *superset-docs-preview* ready!
| Name | Link |
|:-:|------------------------|
|<span aria-hidden="true">π¨</span> Latest commit | b699020828b8e46af7e69a0aeb995f11df3b90f2 |
|<span aria-hidden="true">π</span> Latest deploy log | https://app.netlify.com/projects/superset-docs-preview/deploys/696e3c3ad181d9000852c1b4 |
|<span aria-hidden="true">π</span> Deploy Preview | [https://deploy-preview-37252--superset-docs-preview.netlify.app](https://deploy-preview-37252--superset-docs-preview.netlify.app) |
|<span aria-hidden="true">π±</span> Preview on mobile | <details><summary> Toggle QR Code... </summary><br /><br /><br /><br />_Use your smartphone camera to open QR code link._</details> |
---
<!-- [superset-docs-preview Preview](https://deploy-preview-37252--superset-docs-preview.netlify.app) -->
_To edit notification comments on pull requests, go to your [Netlify project configuration](https://app.netlify.com/projects/superset-docs-preview/configuration/notifications#deploy-notifications)._
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
Re: [PR] fix(mcp): prevent MCP service from corrupting shared appbuilder singleton [superset]
Posted by "bito-code-review[bot] (via GitHub)" <gi...@apache.org>.
bito-code-review[bot] commented on PR #37252:
URL: https://github.com/apache/superset/pull/37252#issuecomment-3768553470
AI Code Review is in progress (usually takes 3 to 15 minutes unless it's a very large PR).
<details>
<summary>
Bito Usage Guide
</summary>
**Commands**
Type the following command in the pull request comment and save the comment.
- `/review` - Manually triggers a full AI review.
- `/pause` - Pauses automatic reviews on this pull request.
- `/resume` - Resumes automatic reviews.
- `/resolve` - Marks all Bito-posted review comments as resolved.
- `/abort` - Cancels all in-progress reviews.
Refer to the <a href="https://docs.bito.ai/bito-dev-agents/ai-code-review-agent/available-commands" target="_blank" rel="noopener noreferrer">documentation</a> for additional commands.
**Configuration**
This repository uses `Superset` You can customize the agent settings <a href="https://alpha.bito.ai/home/ai-agents/code-review-agent" target="_blank" rel="noopener noreferrer">here</a> or contact your Bito workspace admin at evan@preset.io.
**Documentation & Help**
- <a href="https://docs.bito.ai/bito-dev-agents/ai-code-review-agent/getting-started/install-run-using-bito-cloud/create-or-customize-an-agent-instance" target="_blank" rel="noopener noreferrer">Customize agent settings</a>
- <a href="https://docs.bito.ai/bito-dev-agents/ai-code-review-agent/implementing-custom-code-review-rules" target="_blank" rel="noopener noreferrer">Review rules</a>
- <a href="https://docs.bito.ai/bito-dev-agents/ai-code-review-agent" target="_blank" rel="noopener noreferrer">General documentation</a>
- <a href="https://docs.bito.ai/bito-dev-agents/ai-code-review-agent/faqs" target="_blank" rel="noopener noreferrer">FAQ</a>
</details>
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
Re: [PR] fix(mcp): prevent MCP service from corrupting shared appbuilder singleton [superset]
Posted by "michael-s-molina (via GitHub)" <gi...@apache.org>.
michael-s-molina merged PR #37252:
URL: https://github.com/apache/superset/pull/37252
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
Re: [PR] fix(mcp): prevent MCP service from corrupting shared appbuilder singleton [superset]
Posted by "michael-s-molina (via GitHub)" <gi...@apache.org>.
michael-s-molina commented on PR #37252:
URL: https://github.com/apache/superset/pull/37252#issuecomment-3768548680
cc @aminghadersohi
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org