You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@superset.apache.org by "I3eka (via GitHub)" <gi...@apache.org> on 2026/10/26 03:45:36 UTC

[PR] fix(ai): expose chart type schema discovery to authoring profiles [superset]

I3eka opened a new pull request, #45725:
URL: https://github.com/apache/superset/pull/45725

### Summary

**ON HOLD — pending SIP-224** (#42806) and the parent Native AI proposal #42805. Draft follow-up to the opt-in authoring work in #43133.

The chart generation contract tells the agent to call `get_chart_type_schema`, but Native AI does not register that tool. The agent therefore cannot inspect the supported configuration before creating a chart.

Expose the existing registered MCP discovery tool in the opt-in authoring bundle and configuration example. The adapter uses its flat argument contract and the same authenticated MCP execution path as the authoring tools. Schemas and examples remain owned by the existing MCP implementation; no chart-type list is duplicated.

### Validation

- 154 authoring, policy, and tool tests pass, including registry availability and the exact MCP argument shape.
- Pre-commit passes for all branch-changed files.
- This branch includes the current master compatibility merges and prerequisite authoring work; no deployment was performed.

--
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(ai): expose chart type schema discovery to authoring profiles [superset]

Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.

github-actions[bot] commented on PR #45725:
URL: https://github.com/apache/superset/pull/45725#issuecomment-6461098472

### ⚠️ Possible DB migration conflict

@I3eka the base branch `master` picked up a new database migration under `superset/migrations/` after this PR was opened, and this PR adds or modifies migration files too. Alembic migrations form a single linear chain, so if both branch off the same parent revision, merging this PR as-is could leave the migration history with two heads instead of one, which makes `superset db upgrade` fail with `Multiple head revisions are present`.

To check and, if needed, fix this:

1. Rebase this branch onto the latest `master`.
2. Run `superset db heads`. If it lists more than one revision, you have a conflict.
3. In your new migration's script, update `down_revision` (and the `Revises` comment) to point at the other head instead, so the chain has a single head again.

See [Merging DB Migrations](https://superset.apache.org/docs/contributing/development#merging-db-migrations) in the contributing guide for a worked example.

--
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

Re: [PR] fix(ai): expose chart type schema discovery to authoring profiles [superset]

Posted by "I3eka (via GitHub)" <gi...@apache.org>.

I3eka commented on PR #45725:
URL: https://github.com/apache/superset/pull/45725#issuecomment-6435327668

Reviewed the current discussion and the focused implementation, and merged Apache master `037abb7d0b433c1274ab55f95e5613310f132462` without rewriting the branch. Published head: `1efc821f7e6c24fb9278d3691867a88e8e74500d`. There are no inline review threads or outstanding substantive reviewer findings on this PR in this audit.

- 716 tests passed: the full AI unit suite and the single-migration-head check.
- 105 AI frontend tests passed on the identical full frontend Git tree `91c05bee9972b32d81248e1a3575e8f4e31edd75` in #44497; reused that result. Frontend type/lint hooks were rerun on this branch.
- All applicable pre-commit hooks passed for the 122 files in this PR delta.

Includes the prerequisite opt-in authoring work in #43133. Tests check registry inclusion, policy configuration and exact MCP argument shape; no live authoring deployment tested.

Updated the description to match this head and its validation. This stays **Draft / ON HOLD** behind #42805 and SIP #42806. No deployment, live database migration or external provider call was performed; GitHub CI is separate from these local checks.

--
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

Re: [PR] fix(ai): expose chart type schema discovery to authoring profiles [superset]

Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.

github-actions[bot] commented on PR #45725:
URL: https://github.com/apache/superset/pull/45725#issuecomment-6455912608

### ⚠️ Possible DB migration conflict

To check and, if needed, fix this:

See [Merging DB Migrations](https://superset.apache.org/docs/contributing/development#merging-db-migrations) in the contributing guide for a worked example.

--
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

Re: [PR] fix(ai): expose chart type schema discovery to authoring profiles [superset]

Posted by "codecov[bot] (via GitHub)" <gi...@apache.org>.

codecov[bot] commented on PR #45725:
URL: https://github.com/apache/superset/pull/45725#issuecomment-6407851769

   ## [Codecov](https://app.codecov.io/gh/apache/superset/pull/45725?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) Report
   :white_check_mark: All modified and coverable lines are covered by tests.
   :white_check_mark: Project coverage is 55.76%. Comparing base ([`fc01645`](https://app.codecov.io/gh/apache/superset/commit/fc016454fb0874b3a242253d9f6d9c04355c7a43?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)) to head ([`98d72af`](https://app.codecov.io/gh/apache/superset/commit/98d72af64f5cd285f05f8bde4dcf60220c3beb72?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)).
   :warning: Report is 931 commits behind head on master.
   
   <details><summary>Additional details and impacted files</summary>
   
   
   
   ```diff
   @@             Coverage Diff             @@
   ##           master   #45725       +/-   ##
   ===========================================
   - Coverage   80.65%   55.76%   -24.89%     
   ===========================================
     Files        2942     3107      +165     
     Lines      175691   206872    +31181     
     Branches    40788    47811     +7023     
   ===========================================
   - Hits       141704   115365    -26339     
   - Misses      31325    90420    +59095     
   + Partials     2662     1087     -1575     
   ```
   
   | [Flag](https://app.codecov.io/gh/apache/superset/pull/45725/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | |
   |---|---|---|
   | [mysql](https://app.codecov.io/gh/apache/superset/pull/45725/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `?` | |
   | [postgres](https://app.codecov.io/gh/apache/superset/pull/45725/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `?` | |
   | [sqlite](https://app.codecov.io/gh/apache/superset/pull/45725/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `?` | |
   | [superset-extensions-cli](https://app.codecov.io/gh/apache/superset/pull/45725/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `91.24% <ø> (?)` | |
   | [unit](https://app.codecov.io/gh/apache/superset/pull/45725/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `?` | |
   
   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 Harness](https://app.codecov.io/gh/apache/superset/pull/45725?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(ai): expose chart type schema discovery to authoring profiles [superset]

Posted by "netlify[bot] (via GitHub)" <gi...@apache.org>.

netlify[bot] commented on PR #45725:
URL: https://github.com/apache/superset/pull/45725#issuecomment-6407775281

   ### <span aria-hidden="true">✅</span> Deploy Preview for *superset-docs-preview* ready!
   
   
   |  Name | Link |
   |:-:|------------------------|
   |<span aria-hidden="true">🔨</span> Latest commit | 98d72af64f5cd285f05f8bde4dcf60220c3beb72 |
   |<span aria-hidden="true">🔍</span> Latest deploy log | https://app.netlify.com/projects/superset-docs-preview/deploys/6adeccde9f077a0f83b2eebf |
   |<span aria-hidden="true">😎</span> Deploy Preview | [https://deploy-preview-45725--superset-docs-preview.netlify.app](https://deploy-preview-45725--superset-docs-preview.netlify.app) |
   |<span aria-hidden="true">📱</span> Preview on mobile | <details><summary> Toggle QR Code... </summary><br /><br />![QR Code](https://app.netlify.com/qr-code/eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJ1cmwiOiJodHRwczovL2RlcGxveS1wcmV2aWV3LTQ1NzI1LS1zdXBlcnNldC1kb2NzLXByZXZpZXcubmV0bGlmeS5hcHAifQ.h2ANgzjhMaFxOMQzTo3QqN0OGgWa0MJHs-Pzf0joSnE)<br /><br />_Use your smartphone camera to open QR code link._</details> |
   |<span aria-hidden="true">🤖</span> Make changes | [Run an agent on this branch](https://app.netlify.com/projects/superset-docs-preview/agent-runs#DcpBCoAgEADAv-w5SQqy-kHPWNddFDLFPCTR3_M2h3khlxRzhR1gAFvwIt8t4RkxKPJYqqots7rJc8R-HOcztcP1taBjIsebaGNQyzrbidkKfD8) |
   ---
   <!-- [superset-docs-preview Preview](https://deploy-preview-45725--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