@mariodefe/sap-datasphere-mcp 1.0.9

package/CHANGELOG_v1.0.9.md

@@ -0,0 +1,196 @@
+# Changelog v1.0.9 - Enhanced Aggregation & Improved Logging
+
+**Release Date:** December 15, 2025
+**Type:** Enhancement Release
+**Based on:** Kiro's v1.0.8 testing feedback
+
+---
+
+## 🎉 v1.0.8 Success Confirmed
+
+Kiro's testing confirmed v1.0.8 fixed the critical aggregation fallback bug:
+- ✅ GROUP BY queries with client-side aggregation working correctly
+- ✅ LIMIT optimization working perfectly
+- ✅ Enhanced error messages with actionable steps
+- ✅ Basic queries functioning flawlessly
+
+---
+
+## ✨ Enhancements in v1.0.9
+
+### 1. Simple Aggregation Support (WITHOUT GROUP BY)
+
+**Issue:** Queries like `SELECT COUNT(*) FROM table` failed because client-side aggregation required GROUP BY.
+
+**Fix:** Enhanced `perform_client_side_aggregation()` to support both modes:
+- **Case 1:** Simple aggregation (no GROUP BY) - Returns single row with aggregate over all data
+- **Case 2:** GROUP BY aggregation - Returns grouped results (existing behavior)
+
+**Examples Now Supported:**
+```sql
+-- Simple COUNT
+SELECT COUNT(*) FROM SAP_SC_FI_V_ProductsDim
+-- Returns: {"COUNT_ALL": 35}
+
+-- Simple aggregations
+SELECT COUNT(*), AVG(PRICE), MAX(PRICE) FROM SAP_SC_FI_V_ProductsDim
+-- Returns single row with all aggregates
+
+-- GROUP BY (already worked)
+SELECT CATEGORY, COUNT(*), AVG(PRICE) FROM Products GROUP BY CATEGORY
+-- Returns: Multiple rows, one per category
+```
+
+**Implementation:**
+- Checks for GROUP BY clause first
+- If no GROUP BY: aggregates over entire dataset, returns single row
+- If GROUP BY: groups data and aggregates per group (existing logic)
+- Improved regex to avoid ORDER BY false positives
+
+---
+
+### 2. Enhanced Asset Capability Detection
+
+**Issue:** False "Asset not found" warnings even when queries succeeded.
+
+**Fix:** Multi-strategy asset search:
+1. **Strategy 1:** Exact name match (`name eq 'table'`)
+2. **Strategy 2:** Contains match for views with schema prefixes
+3. **Fallback:** Assume asset exists (avoid false negatives)
+
+**Benefits:**
+- Fewer false warnings
+- Better case-insensitive matching
+- Graceful degradation when catalog search has limitations
+
+---
+
+### 3. Improved Logging & User Experience
+
+**Before:**
+```
+⚠️  Asset 'SAP_SC_FI_V_ProductsDim' not found - will attempt query anyway
+```
+
+**After:**
+```
+ℹ️  Asset not in catalog search - may still exist
+✓ Asset found: type=view, analytical=false
+```
+
+**Changes:**
+- Changed warning emoji (⚠️) to info emoji (ℹ️) for non-critical messages
+- More accurate descriptions ("not in catalog" vs "not found")
+- Only show suggestions when query likely to fail
+- Clearer asset capability reporting
+
+---
+
+## 📊 What's Fixed
+
+| Issue | v1.0.8 | v1.0.9 |
+|-------|--------|--------|
+| **Simple COUNT(*)** | Failed (no GROUP BY) | Works ✅ |
+| **Simple aggregations** | Failed (no GROUP BY) | Works ✅ |
+| **GROUP BY queries** | Works ✅ | Works ✅ |
+| **False "not found" warnings** | Frequent ⚠️ | Rare ℹ️ |
+| **Asset detection** | Exact match only | Multi-strategy ✅ |
+| **ORDER BY in GROUP BY** | Parse issues | Improved regex ✅ |
+
+---
+
+## 🔧 Technical Details
+
+### Enhanced Aggregation Function
+
+**Location:** [sap_datasphere_mcp_server.py:2326-2455](sap_datasphere_mcp_server.py#L2326-L2455)
+
+**Key Changes:**
+```python
+# New: Check for GROUP BY with improved regex
+group_by_match = re.search(
+    r'GROUP\s+BY\s+([\w,\s]+?)(?:\s+ORDER\s+BY|\s+HAVING|\s+LIMIT|$)',
+    query_str,
+    re.IGNORECASE
+)
+
+# Case 1: Simple aggregation (NEW in v1.0.9)
+if not group_by_match:
+    result = {}
+    # Aggregate over all data
+    for func, column, alias in agg_functions:
+        # Calculate aggregate...
+    return [result]  # Single row
+
+# Case 2: GROUP BY aggregation (existing)
+# ... group and aggregate logic ...
+```
+
+### Enhanced Asset Detection
+
+**Location:** [sap_datasphere_mcp_server.py:2285-2336](sap_datasphere_mcp_server.py#L2285-L2336)
+
+**Multi-Strategy Search:**
+1. Exact match
+2. Contains match with case-insensitive filter
+3. Graceful fallback
+
+---
+
+## 📈 Statistics
+
+**v1.0.9 by the numbers:**
+- **2 major enhancements** (simple aggregation + asset detection)
+- **1 UX improvement** (better logging)
+- **~130 lines** of code modified
+- **100% backward compatible**
+- **45 tools** total (no new tools)
+
+---
+
+## 🚀 Upgrade Instructions
+
+```bash
+pip install --upgrade sap-datasphere-mcp
+```
+
+**Verification:**
+```bash
+python -c "import sap_datasphere_mcp; print(sap_datasphere_mcp.__version__)"
+# Should output: 1.0.9
+```
+
+---
+
+## 🙏 Acknowledgments
+
+- **Kiro (Testing Agent):** Comprehensive v1.0.8 testing & detailed feedback
+- Identified simple aggregation edge cases
+- Reported false "not found" warnings
+- Suggested ORDER BY parsing improvements
+
+---
+
+## 📚 Related Documentation
+
+- [CHANGELOG_v1.0.8.md](CHANGELOG_v1.0.8.md) - Previous hotfix
+- [CHANGELOG_v1.0.7.md](CHANGELOG_v1.0.7.md) - Original aggregation implementation
+- [V1.0.7_IMPLEMENTATION_PLAN.md](V1.0.7_IMPLEMENTATION_PLAN.md) - Implementation plan
+- [README.md](README.md) - Main documentation
+
+---
+
+## Summary
+
+v1.0.9 completes the aggregation feature by adding support for simple aggregations without GROUP BY, improves asset detection to reduce false warnings, and enhances logging for better user experience.
+
+**All Query Types Now Supported:**
+1. ✅ Simple queries: `SELECT * FROM table`
+2. ✅ Simple aggregations: `SELECT COUNT(*) FROM table`
+3. ✅ GROUP BY aggregations: `SELECT category, COUNT(*) FROM table GROUP BY category`
+4. ✅ Complex GROUP BY with ORDER BY: `SELECT category, COUNT(*) FROM table GROUP BY category ORDER BY COUNT(*) DESC`
+
+---
+
+**Full Changelog:** https://github.com/MarioDeFelipe/sap-datasphere-mcp/compare/v1.0.8...v1.0.9
+**PyPI Release:** https://pypi.org/project/sap-datasphere-mcp/1.0.9/

package/LICENSE

@@ -0,0 +1,21 @@
+MIT License
+
+Copyright (c) 2025 SAP Datasphere AWS Glue Metadata Sync Platform
+
+Permission is hereby granted, free of charge, to any person obtaining a copy
+of this software and associated documentation files (the "Software"), to deal
+in the Software without restriction, including without limitation the rights
+to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+copies of the Software, and to permit persons to whom the Software is
+furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in all
+copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+SOFTWARE.