Improve error message for non-existent database

[lucas-koontz]

No description provided.

[github-actions]

Coverage Report

File	Stmts	Miss	Cover	Missing
mindsdb_sdk
agents.py	179	47	74%	29, 101, 104, 107, 110, 118, 126, 146, 167, 178, 181, 185, 187, 189, 191, 193, 195, 197, 261, 274, 285, 296, 299–303, 318, 327–331, 375, 383–384, 442, 450–451, 455, 486–488, 490–491, 518–519, 521
databases.py	58	7	88%	85–86, 149, 177, 188–194
handlers.py	37	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	211	19	91%	109, 140–141, 222, 231, 233, 303, 339, 349, 373, 398, 492, 500, 519, 535, 543, 568, 572, 585
projects.py	61	1	98%	158
query.py	13	1	92%	14
server.py	31	2	94%	86–87
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	244	56	77%	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	1589	297	81%

Tests	Skipped	Failures	Errors	Time
26	0 💤	0 ❌	0 🔥	6.090s ⏱️

[entelligence-ai-pr-reviews]

Duplicate Code: ⚠️ Duplicate Code Detected (Similarity: 92%)

This function Databases.get duplicates existing code.

📍 Original Location:

mindsdb_sdk/projects.py:152-161

Function: Projects.get

💡 Recommendation:
Extract this pattern into a template method in the CollectionBase parent class:

class CollectionBase:
    def _get_with_validation(self, name: str, list_method, constructor, resource_type: str):
        """Template method for validated get operations"""
        items = list_method()
        if name not in items:
            raise AttributeError(f"{resource_type} '{name}' doesn't exist")
        return constructor(name, items) if callable(constructor) else items[name]

Then both Databases.get() and Projects.get() can call this template method, reducing duplication and ensuring consistent error messages across all collection managers.

Consider importing and reusing the existing function instead of duplicating the logic.

[MinuraPunchihewa]

@lucas-koontz If you want to release this as a patch, you will need to update the version as well.