Address additional PR review comments.

- Add caching for LLM model options to avoid repeated API calls
  when switching between preference categories (fixes slow OpenAI
  model loading issue).
- Add icon to AI Reports panel tab (fa-file-alt).
- Clear options cache on successful refresh to ensure fresh data.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: dpage

web/pgadmin/llm/static/js/ai_tools.js

@@ -404,6 +404,7 @@ define([
       handler.docker.openTab({
         id: panelId,
         title: panelTitle,
+        icon: 'fa fa-file-alt',
         content: (
           <AIReport
             sid={sid}

web/pgadmin/preferences/static/js/components/PreferencesHelper.jsx

@@ -20,6 +20,15 @@ import CloseIcon from '@mui/icons-material/CloseRounded';
 import HTMLReactParser from 'html-react-parser/lib/index';
 import getApiInstance from '../../../../static/js/api_instance';
 
+// Cache for dynamically loaded options to avoid repeated API calls
+const optionsCache = {};
+
+// Clear cache for a specific endpoint (used when refresh button updates options)
+export function clearOptionsCache(endpoint) {
+  if (endpoint) {
+    delete optionsCache[endpoint];
+  }
+}
 
 export async function reloadPgAdmin() {
   const { name: browser } = getBrowser();
@@ -123,13 +132,19 @@ export function prepareSubnodeData(node, subNode, nodeData, preferencesStore) {
           const optionsEndpoint = element.controlProps.optionsUrl;
           const staticOptions = element.options || [];
           element.options = () => {
+            // Check cache first to avoid repeated API calls
+            if (optionsCache[optionsEndpoint]) {
+              return Promise.resolve([...optionsCache[optionsEndpoint], ...staticOptions]);
+            }
             return new Promise((resolve) => {
               const api = getApiInstance();
               const optionsUrl = url_for(optionsEndpoint);
               api.get(optionsUrl)
                 .then((res) => {
                   if (res.data?.data?.models) {
                     const dynamicOptions = res.data.data.models;
+                    // Cache the results
+                    optionsCache[optionsEndpoint] = dynamicOptions;
                     resolve([...dynamicOptions, ...staticOptions]);
                   } else {
                     resolve(staticOptions);
@@ -147,6 +162,10 @@ export function prepareSubnodeData(node, subNode, nodeData, preferencesStore) {
         const staticOptions = element.options || [];
         // Replace options with a function that fetches from the URL
         element.options = () => {
+          // Check cache first to avoid repeated API calls
+          if (optionsCache[optionsEndpoint]) {
+            return Promise.resolve([...optionsCache[optionsEndpoint], ...staticOptions]);
+          }
           return new Promise((resolve) => {
             const api = getApiInstance();
             // Use url_for to resolve the endpoint to a proper URL
@@ -156,6 +175,8 @@ export function prepareSubnodeData(node, subNode, nodeData, preferencesStore) {
                 if (res.data?.data?.models) {
                   // Dynamic models loaded successfully
                   const dynamicOptions = res.data.data.models;
+                  // Cache the results
+                  optionsCache[optionsEndpoint] = dynamicOptions;
                   resolve([...dynamicOptions, ...staticOptions]);
                 } else {
                   // No models in response, use static options

web/pgadmin/static/js/components/SelectRefresh.jsx

@@ -20,6 +20,7 @@ import url_for from 'sources/url_for';
 import gettext from 'sources/gettext';
 import { SchemaStateContext } from '../SchemaView/SchemaState';
 import { usePgAdmin } from '../PgAdminProvider';
+import { clearOptionsCache } from '../../../preferences/static/js/components/PreferencesHelper';
 
 const StyledBox = styled(Box)(() => ({
   display: 'flex',
@@ -69,6 +70,7 @@ export function SelectRefresh({ required, className, label, helpMessage, testcid
   const {
     getOptionsOnRefresh,
     optionsRefreshUrl,
+    optionsUrl,
     refreshDeps,
     ...selectControlProps
   } = controlProps;
@@ -101,6 +103,10 @@ export function SelectRefresh({ required, className, label, helpMessage, testcid
             pgAdmin.Browser.notifier.error(res.data.data.error);
           } else if (res.data?.data?.models) {
             const models = res.data.data.models;
+            // Clear the cache so next time preferences opens, it uses the refreshed data
+            if (optionsUrl) {
+              clearOptionsCache(optionsUrl);
+            }
             setOptionsState((prev) => ({ options: models, reloadBasis: prev.reloadBasis + 1 }));
           } else {
             // No models returned - clear the list
@@ -132,7 +138,7 @@ export function SelectRefresh({ required, className, label, helpMessage, testcid
           setIsRefreshing(false);
         });
     }
-  }, [optionsRefreshUrl, refreshDeps, schemaState, getOptionsOnRefresh, pgAdmin]);
+  }, [optionsRefreshUrl, optionsUrl, refreshDeps, schemaState, getOptionsOnRefresh, pgAdmin]);
 
   return (
     <FormInput required={required} label={label} className={className} helpMessage={helpMessage} testcid={testcid}>