geek-cafe-saas-sdk 0.7.2__tar.gz → 0.7.3__tar.gz

{geek_cafe_saas_sdk-0.7.2 → geek_cafe_saas_sdk-0.7.3}/.gitignore

@@ -148,4 +148,5 @@ activate.sh
 .pypirc
 .prompts
 .DS_Store
-.local
+.local
+.windsurf_historical

{geek_cafe_saas_sdk-0.7.2 → geek_cafe_saas_sdk-0.7.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: geek_cafe_saas_sdk
-Version: 0.7.2
+Version: 0.7.3
 Summary: Base Reusable Services for SaaS
 Project-URL: Homepage, https://github.com/geekcafe/geek-cafe-services
 Project-URL: Documentation, https://github.com/geekcafe/geek-cafe-services/blob/main/README.md

{geek_cafe_saas_sdk-0.7.2 → geek_cafe_saas_sdk-0.7.3}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "geek_cafe_saas_sdk"
-version = "0.7.2"
+version = "0.7.3"
 description = "Base Reusable Services for SaaS"
 authors = [{ name = "Eric Wilson", email = "eric.wilson@geekcafe.com" }]
 requires-python = ">=3.8"

{geek_cafe_saas_sdk-0.7.2 → geek_cafe_saas_sdk-0.7.3}/requirements.dev.txt

@@ -13,4 +13,5 @@ mypy_boto3_dynamodb
 pytest-cov
 pytest-html
 pyyaml
-joserfc
+joserfc
+python-dotenv  # For remote access tests

{geek_cafe_saas_sdk-0.7.2 → geek_cafe_saas_sdk-0.7.3}/src/geek_cafe_saas_sdk/__init__.py

@@ -3,7 +3,7 @@ Geek Cafe Services - Base Reusable Services for SaaS
 
 Version 0.6.0 adds File System Service
 """
-__version__ = "0.7.2"
+__version__ = "0.7.3"
 
 # Import main modules for easier access
 from . import services

{geek_cafe_saas_sdk-0.7.2 → geek_cafe_saas_sdk-0.7.3}/src/geek_cafe_saas_sdk/domains/files/services/file_version_service.py

@@ -515,21 +515,12 @@ class FileVersionService(DatabaseService[FileVersion]):
     def _get_file(self, tenant_id: str, file_id: str, user_id: str) -> ServiceResult[File]:
         """Get file with access control."""
         try:
-            file = File()
-            file.id = file_id
-            file.tenant_id = tenant_id
+            # Use DatabaseService helper to get file
+            file = self._get_model_by_id_with_tenant_check(file_id, File, tenant_id)
             
-            result = self.dynamodb.get(
-                table_name=self.table_name,
-                model=file
-            )
-            
-            if not result or 'Item' not in result:
+            if not file:
                 raise NotFoundError(f"File not found: {file_id}")
             
-            file = File()
-            file.map(result['Item'])
-            
             if file.owner_id != user_id:
                 raise AccessDeniedError("You do not have access to this file")
             
@@ -544,24 +535,14 @@ class FileVersionService(DatabaseService[FileVersion]):
     def _get_latest_version_number(self, tenant_id: str, file_id: str) -> int:
         """Get the latest version number for a file."""
         try:
-            file = File()
-            file.id = file_id
-            file.tenant_id = tenant_id
+            # Query versions for this file using DatabaseService helper
+            temp_version = FileVersion()
+            temp_version.file_id = file_id
             
-            key = file.get_key("gsi1")
-            results = self.dynamodb.query(
-                key=key,
-                table_name=self.table_name,
-                index_name="gsi1",
-                limit=1,
-                ascending=False  # Get highest version number
-            )
+            result = self._query_by_index(temp_version, "gsi1", limit=1, ascending=False)
             
-            items = results.get('Items', [])
-            if items:
-                version = FileVersion()
-                version.map(items[0])
-                return version.version_number
+            if result.success and result.data:
+                return result.data[0].version_number
             
             return 0  # No versions yet
             
@@ -576,19 +557,10 @@ class FileVersionService(DatabaseService[FileVersion]):
     ) -> None:
         """Mark a version as not current."""
         try:
-            version = FileVersion()
-            version.id = version_id
-            version.tenant_id = tenant_id
-            version.file_id = file_id
-            
-            result = self.dynamodb.get(
-                table_name=self.table_name,
-                model=version
-            )
+            # Use DatabaseService helper to get version
+            version = self._get_model_by_id_with_tenant_check(version_id, FileVersion, tenant_id)
             
-            if result and 'Item' in result:
-                version = FileVersion()
-                version.map(result['Item'])
+            if version:
                 version.is_current = False
                 
                 version.prep_for_save()
@@ -605,12 +577,15 @@ class FileVersionService(DatabaseService[FileVersion]):
     ) -> None:
         """Update file record with current version info."""
         try:
-            pk = f"FILE#{tenant_id}#{file_id}"
-            sk = "metadata"
+            file = File()
+            file.id = file_id
+            file.tenant_id = tenant_id
+            
+            key = file.get_key("gsi1")
             
             result = self.dynamodb.get(
                 table_name=self.table_name,
-                key={"pk": pk, "sk": sk}
+                key=key
             )
             
             if result and 'Item' in result:

{geek_cafe_saas_sdk-0.7.2 → geek_cafe_saas_sdk-0.7.3}/src/geek_cafe_saas_sdk/domains/messaging/models/contact_thread.py

@@ -33,13 +33,13 @@ class ContactThread(BaseModel):
         super().__init__()
         self._subject: str | None = None
         self._status: str = "open"  # open, in_progress, resolved, closed
-        self._priority: str = "medium"  # low, medium, high, urgent
+        self._priority: Optional[str] = "medium"  # low, medium, high, urgent
         
         # Sender information (guest or authenticated user)
         self._sender: Dict[str, Any] = {}  # {id, name, email, session_id}
         
         # Assignment and routing
-        self._assigned_to: str | None = None  # Staff user ID
+        self._assigned_to: Optional[str] = None  # Staff user ID
         self._inbox_id: str = "support"  # support, sales, billing, etc.
         
         # Messages embedded in thread (suitable for low volume)
@@ -178,7 +178,7 @@ class ContactThread(BaseModel):
 
     @priority.setter
     def priority(self, value: str | None):
-        valid_priorities = ["low", "medium", "high", "urgent"]
+        valid_priorities = ["low", "medium", "high", "urgent", None]
         if value in valid_priorities:
             self._priority = value
         else:

{geek_cafe_saas_sdk-0.7.2 → geek_cafe_saas_sdk-0.7.3}/src/geek_cafe_saas_sdk/domains/messaging/services/contact_thread_service.py

@@ -245,7 +245,7 @@ class ContactThreadService(DatabaseService[ContactThread]):
                                                 tenant_id=tenant_id, status=status)
 
     def list_by_assigned_user(self, assigned_to: str, tenant_id: str, 
-                              status: str = None, limit: int = 50) -> ServiceResult[List[ContactThread]]:
+                              *, status: str = None, priority: str = None, limit: int = 50) -> ServiceResult[List[ContactThread]]:
         """
         List contact threads assigned to a specific user using GSI3.
         
@@ -263,6 +263,13 @@ class ContactThreadService(DatabaseService[ContactThread]):
             temp_thread.assigned_to = assigned_to
             if status:
                 temp_thread.status = status
+            else:
+                temp_thread.status = None
+            
+            if priority:
+                temp_thread.priority = priority
+            else:
+                temp_thread.priority = None
 
             result = self._query_by_index(
                 temp_thread,

{geek_cafe_saas_sdk-0.7.2 → geek_cafe_saas_sdk-0.7.3}/src/geek_cafe_saas_sdk/services/database_service.py

@@ -2,14 +2,17 @@
 
 from abc import ABC, abstractmethod
 from typing import Generic, TypeVar, Dict, Any, List, Optional
-from boto3_assist.dynamodb.dynamodb import DynamoDB
+from boto3_assist.dynamodb.dynamodb import DynamoDB, DynamoDBIndex
 from ..core.service_result import ServiceResult
 from ..core.service_errors import ValidationError, AccessDeniedError, NotFoundError
 from ..core.error_codes import ErrorCode
 import os
+from aws_lambda_powertools import Logger
 
 T = TypeVar("T")
 
+logger = Logger()
+
 
 class DatabaseService(ABC, Generic[T]):
     """Base service class for database operations."""
@@ -23,6 +26,8 @@ class DatabaseService(ABC, Generic[T]):
         if not self.table_name:
             raise ValueError("Table name is required")
 
+        self.LOG_DYNAMO_DB_QUERY = os.getenv("LOG_DYNAMO_DB_QUERY", False)
+
     @abstractmethod
     def create(self, tenant_id: str, user_id: str, **kwargs) -> ServiceResult[T]:
         """Create a new resource."""
@@ -226,6 +231,26 @@ class DatabaseService(ABC, Generic[T]):
             # Get the key for the specified index from the provided model
             key = model.get_key(index_name).key()
 
+            # key.to_string()
+            # extract_key_values(key_expression) is coming soon from boto3_assist
+            if self.LOG_DYNAMO_DB_QUERY:
+                if hasattr(DynamoDBIndex, 'extract_key_values'):
+                    values = DynamoDBIndex.extract_key_values(key_expression)
+                    logger.info(f"Querying index {index_name} with key {values}")
+                else:
+                    try:
+                        value = {
+                            'pk': key._values[0]._values[1],
+                            'sk': key._values[1]._values[1],
+                            'operator': key._values[1].expression_operator,
+                            'format': key._values[1].expression_format,
+                            'index_name': index_name
+                        }
+                        logger.info(f"Querying index {index_name} with key {value}")
+                    except Exception as e:
+                        logger.error(f"Failed to extract key values: {e}")
+                
+
             # Execute the query
             response = self.dynamodb.query(
                 table_name=self.table_name,

geek_cafe_saas_sdk-0.7.2/CODING_STANDARDS_IMPLEMENTATION.md

@@ -1,252 +0,0 @@
-# Coding Standards Implementation - Complete ✅
-
-**Date:** October 17, 2025  
-**Status:** COMPLETE
-
----
-
-## 🎯 What Was Implemented
-
-1. **Global Rules** - Created 3 persistent memory rules that apply to all future work
-2. **Documentation** - Created comprehensive `CODING_STANDARDS.md` shareable across projects
-3. **Code Fixes** - Fixed 2 instances of deprecated `.load_from_dictionary()` pattern
-
----
-
-## 📝 Global Rules Created (Memory System)
-
-These rules are now **permanently stored** in my memory and will be **automatically retrieved** for all future work:
-
-### Rule 1: Model Index Pattern
-- **Title:** "Model Index Pattern - _setup_indexes() Required"
-- **Enforces:** All models must use `_setup_indexes()` method
-- **Tags:** coding_standards, models, dynamodb, indexes
-
-### Rule 2: Service Helper Pattern  
-- **Title:** "Service Pattern - Use DatabaseService Helper Methods"
-- **Enforces:** Services use `_save_model()`, `_get_model_by_id()`, `_query_by_index()`
-- **Tags:** coding_standards, services, dynamodb, helpers
-
-### Rule 3: Model Mapping Pattern
-- **Title:** "Model Mapping Pattern - Use .map() Method"
-- **Enforces:** Always use `.map()` instead of key iteration or `load_from_dictionary()`
-- **Tags:** coding_standards, models, mapping, data_loading
-
----
-
-## 📚 Documentation Created
-
-### File: `docs/guidelines/CODING_STANDARDS.md`
-
-Comprehensive coding standards document with:
-
-1. **Model Patterns**
-   - ✅ Index definition with `_setup_indexes()`
-   - ✅ Complete example with primary + GSI indexes
-   - ✅ Why these patterns matter
-
-2. **Service Patterns**
-   - ✅ Using `_save_model()` for saves
-   - ✅ Using `_get_model_by_id()` for retrieval
-   - ✅ Using `_query_by_index()` for queries
-   - ✅ Complete before/after examples
-
-3. **Data Mapping Patterns**
-   - ✅ Using `.map()` method
-   - ✅ Avoiding manual iteration
-   - ✅ List comprehension examples
-
-4. **Lambda Handler Patterns**
-   - ✅ Using `create_handler()` framework
-   - ✅ Complete handler example
-
-5. **Error Handling**
-   - ✅ ServiceResult pattern
-   - ✅ Exception wrapping
-
-6. **Testing Standards**
-   - ✅ pytest with moto
-   - ✅ Fixture patterns
-
-**Summary Checklist** included at end of document.
-
----
-
-## 🔧 Code Fixes Applied
-
-### Fixed: NotificationService
-
-**File:** `src/geek_cafe_saas_sdk/domains/notifications/services/notification_service.py`
-
-#### Fix 1: get_user_preferences() - Line 270
-
-**Before:**
-```python
-prefs = NotificationPreference()
-prefs.load_from_dictionary(result["Item"])
-```
-
-**After:**
-```python
-# Use .map() instead of load_from_dictionary
-prefs = NotificationPreference().map(result["Item"])
-```
-
-#### Fix 2: list_webhook_subscriptions() - Line 471
-
-**Before:**
-```python
-subscription = WebhookSubscription()
-subscription.load_from_dictionary(item)
-```
-
-**After:**
-```python
-# Use .map() instead of load_from_dictionary
-subscription = WebhookSubscription().map(item)
-```
-
----
-
-## 📊 Audit Results
-
-### Pattern Usage Analysis
-
-**Searched for:** `load_from_dictionary`
-- ✅ **Fixed:** 2 instances in NotificationService
-- ⚠️ **Remaining:** 1 instance in BaseModel (method definition - OK)
-
-**Searched for:** Manual key iteration (`for key, value in dict.items()`)
-- ✅ **Reviewed:** All instances are for **kwargs** or **updates** dictionaries
-- ✅ **Appropriate:** These are NOT database items, pattern is correct for optional field setting
-
-### Key Insight
-
-The iteration pattern `for key, value in kwargs.items()` is **appropriate** when:
-- ✅ Setting optional fields from `**kwargs` in constructors
-- ✅ Applying partial updates from `updates` dictionary
-- ✅ Not loading full database items
-
-The `.map()` pattern is **required** when:
-- ✅ Loading items from DynamoDB
-- ✅ Populating model from complete dictionary representation
-
----
-
-## ✅ Compliance Status
-
-### Models
-- ✅ All 7 recent models have `_setup_indexes()`
-- ✅ All models inherit from `BaseModel`
-- ✅ All models import `DynamoDBIndex`, `DynamoDBKey`
-
-### Services
-- ✅ SubscriptionManagerService uses helper methods
-- ✅ NotificationService uses helper methods
-- ✅ All services use `.map()` for database items
-
-### Lambda Handlers
-- ✅ All handlers use `create_handler()` framework
-- ✅ No manual request parsing
-
-### Error Handling
-- ✅ All service methods return `ServiceResult[T]`
-- ✅ Exceptions wrapped in `ServiceResult.exception_result()`
-
----
-
-## 🎓 How This Works Going Forward
-
-### For AI Assistant (Cascade)
-
-1. **Before any code changes**, I automatically retrieve these global rules from memory
-2. **During implementation**, I follow the patterns defined in the rules
-3. **If uncertain**, I reference `CODING_STANDARDS.md` for examples
-
-### For Developers
-
-1. **New to project?** Read `docs/guidelines/CODING_STANDARDS.md`
-2. **Writing models?** Add `_setup_indexes()` method
-3. **Writing services?** Use `_save_model()`, `_get_model_by_id()`, `_query_by_index()`
-4. **Loading database items?** Use `.map()` not iteration
-5. **Code review?** Check against the summary checklist
-
-### For Code Reviews
-
-The `CODING_STANDARDS.md` document provides:
-- ✅ Clear examples of correct vs incorrect patterns
-- ✅ Rationale for each pattern
-- ✅ Summary checklist for reviewers
-
----
-
-## 📤 Sharing Across Projects
-
-The `docs/guidelines/CODING_STANDARDS.md` file can be:
-- **Copied** to other projects as-is
-- **Adapted** for project-specific patterns
-- **Referenced** in onboarding documentation
-- **Linked** in PR templates
-
-**To use in another project:**
-```bash
-# Copy to new project
-cp docs/guidelines/CODING_STANDARDS.md \
-   /path/to/other-project/docs/guidelines/
-
-# Reference in PR template
-echo "- [ ] Code follows patterns in [CODING_STANDARDS.md](docs/guidelines/CODING_STANDARDS.md)" \
-   >> .github/pull_request_template.md
-```
-
----
-
-## 🔮 Future Enhancements
-
-Potential additions to standards:
-- Security patterns (input validation, sanitization)
-- Performance patterns (caching, batching)
-- Observability patterns (logging, metrics)
-- API design patterns (REST conventions)
-
----
-
-## 📈 Impact
-
-### Immediate Benefits
-- ✅ Consistent codebase patterns
-- ✅ Easier onboarding for new developers
-- ✅ Clearer code reviews
-- ✅ Reduced bugs from pattern violations
-
-### Long-term Benefits
-- ✅ Maintainable codebase
-- ✅ Easier refactoring
-- ✅ Knowledge transfer across team
-- ✅ Pattern reuse across projects
-
----
-
-## 🎉 Summary
-
-**Implementation Complete:**
-- ✅ 3 global rules created in memory system
-- ✅ Comprehensive CODING_STANDARDS.md document created
-- ✅ 2 code violations fixed
-- ✅ All recent code audited for compliance
-- ✅ Shareable documentation for other projects
-
-**The codebase now has:**
-- Consistent model index patterns
-- Consistent service query patterns  
-- Consistent data mapping patterns
-- Clear documentation for all developers
-
-**Future work will automatically follow these patterns!** 🚀
-
----
-
-**Status:** COMPLETE ✅  
-**Documentation:** docs/guidelines/CODING_STANDARDS.md  
-**Global Rules:** Stored in memory system  
-**Code Fixes:** 2/2 applied