Added the Update Operation for Databases

[MinuraPunchihewa]

This PR adds the update operation (con.databases.update()) to the SDK.

[github-actions]

Coverage Report

File	Stmts	Miss	Cover	Missing
mindsdb_sdk
agents.py	217	67	69%	30, 106, 109, 112, 115, 123, 131, 151, 172, 183, 186, 190, 192, 194, 196, 198, 200, 202, 204, 206, 213, 277, 290, 301, 312, 315–319, 334, 343–347, 391, 399–400, 458, 466–467, 471, 501–509, 517–519, 521–522, 548–561, 573–574, 576
databases.py	57	7	88%	85–86, 148, 176, 187–193
handlers.py	39	1	97%	77
jobs.py	97	7	93%	40, 52, 80, 84, 146–149
knowledge_bases.py	133	18	86%	70–73, 132–136, 160, 189, 196, 200–202, 206, 228–232, 242
ml_engines.py	42	3	93%	94, 126, 128
models.py	210	19	91%	109, 140–141, 222, 231, 233, 303, 339, 348, 372, 397, 491, 499, 518, 534, 542, 567, 571, 584
projects.py	63	1	98%	160
query.py	13	1	92%	14
server.py	33	2	94%	88–89
skills.py	53	9	83%	43, 45, 49, 58–60, 76–80, 122
tables.py	130	15	88%	140–142, 145, 165, 192, 203–204, 209, 224, 227, 321, 342–347, 356, 376
tree.py	32	16	50%	18–19, 24–28, 41–57
views.py	37	2	95%	105, 138
mindsdb_sdk/connectors
rest_api.py	271	56	79%	19–29, 35–36, 51, 55, 58–59, 79–81, 87–89, 108, 111, 118–121, 144, 158–166, 187–188, 223–226, 240–241, 295–300, 304–316
mindsdb_sdk/utils
agents.py	50	4	92%	72, 79–81
mind.py	47	47	0%	1–128
openai.py	85	30	65%	37–40, 83–85, 107, 148–158, 215–216, 234–240, 258–276
table_schema.py	21	21	0%	1–54
TOTAL	1711	326	81%

Tests	Skipped	Failures	Errors	Time
31	0 💤	0 ❌	0 🔥	6.042s ⏱️

[entelligence-ai-pr-reviews]

🔒 Entelligence AI Vulnerability Scanner

✅ No security vulnerabilities found!

Your code passed our comprehensive security analysis.

📊 Files Analyzed: 2 files

---

[entelligence-ai-pr-reviews]

Review Summary

🏷️ Draft Comments (2)

Skipped posting 2 draft comments that were valid but scored below your review threshold (>=13/15). Feel free to update them here.

mindsdb_sdk/databases.py (2)

178-193: update method in Databases does not check if the database exists before attempting to update, leading to a misleading success if the database does not exist.

📊 Impact Scores:

• Production Impact: 4/5
• Fix Specificity: 5/5
• Urgency Impact: 3/5
• Total Score: 12/15

🤖 AI Agent Prompt (Copy & Paste Ready):

In mindsdb_sdk/databases.py, lines 178-193, the `update` method does not check if the database exists before attempting to update it. This can result in a misleading success if the database does not exist. Please add a check at the start of the method to verify the database exists (using self._list_databases()), and raise an AttributeError if it does not, matching the behavior of the `get` method.

---

116-126: _list_databases fetches all databases and iterates through all rows on every call, causing O(n) overhead for each get, list, or update operation; this is inefficient for large numbers of databases.

📊 Impact Scores:

• Production Impact: 2/5
• Fix Specificity: 4/5
• Urgency Impact: 2/5
• Total Score: 8/15

🤖 AI Agent Prompt (Copy & Paste Ready):

Optimize the `_list_databases` method in `mindsdb_sdk/databases.py` (lines 116-126). Currently, it fetches and iterates through all databases on every call, causing O(n) overhead for each `get`, `list`, or `update` operation. Add a simple in-memory cache (e.g., `self._cached_databases`) to avoid repeated full scans, and invalidate the cache in `create`, `drop`, and `update` methods after changes.

---