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

   ## Status
   
   **Draft / ON HOLD pending SIP-224:** parent proposal #42805 and discussion 
#42806. This Native AI follow-up is not ready to merge before the parent 
proposal is accepted. It builds on the authoring adapter in #45726; focused 
commits are `02c67ff70b` and `0873d7ee1f`.
   
   ## Problem and change
   
   An unsaved chart preview in chat depended on a temporary Explore form-data 
key. After the cache entry expired, reopening the retained conversation could 
no longer render the chart or open it in Explore.
   
   Keep the bounded form-data snapshot in the successful tool display and 
replay it through the existing chart renderer and Explore helper. Match 
snapshots to the exact tool display key; reject incomplete/truncated snapshots. 
Chart data is requested with current permissions and filters. This retains 
configuration, not historical result rows. Legacy messages without a complete 
snapshot continue using their temporary key and cannot recover a configuration 
that has already expired.
   
   ## Validation
   
   - 19 frontend tests, including retained-history replay after a simulated 
expired key, preserved filters, Explore action, legacy resolution and 
truncated-snapshot rejection.
   - 14 backend authoring tests.
   - Branch pre-commit and corrected fixture checks pass, including frontend 
types.
   - Full panel verification is deferred to integration with the existing 
empty-chat fix #44498. No deployment or live browser/database validation.
   


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to