3 Commits

Author SHA1 Message Date
IGNY8 VPS (Salman)
13bd7fa134 Phase 0: Add ModuleEnableSettings serializer, ViewSet, and URL routing
- Created ModuleEnableSettingsSerializer
- Created ModuleEnableSettingsViewSet with get_or_create logic
- Added URL routing for module enable settings
- One record per account, auto-created on first access
2025-11-16 18:40:10 +00:00
IGNY8 VPS (Salman)
a73b2ae22b Phase 0: Add ModuleEnableSettings model and migration
- Created ModuleEnableSettings model with enabled flags for all modules
- Added migration for ModuleEnableSettings
- Updated imports across system module files
2025-11-16 18:39:16 +00:00
IGNY8 VPS (Salman)
5b11c4001e Phase 0: Update credit costs and CreditService, add credit checks to AI Engine
- Updated CREDIT_COSTS to match Phase 0 spec (flat structure)
- Added get_credit_cost() method to CreditService
- Updated check_credits() to accept operation_type and amount
- Added deduct_credits_for_operation() convenience method
- Updated AI Engine to check credits BEFORE AI call
- Updated AI Engine to deduct credits AFTER successful execution
- Added helper methods for operation type mapping and amount calculation
2025-11-16 18:37:41 +00:00
11 changed files with 369 additions and 186 deletions

View File

@@ -27,27 +27,6 @@ Each entry follows this format:
## [Unreleased] ## [Unreleased]
### Added ### Added
- **Phase 0: Foundation & Credit System - Initial Implementation**
- Updated `CREDIT_COSTS` constants to Phase 0 format with new operations
- Added new credit costs: `linking` (8 credits), `optimization` (1 credit per 200 words), `site_structure_generation` (50 credits), `site_page_generation` (20 credits)
- Maintained backward compatibility with legacy operation names (`ideas`, `content`, `images`, `reparse`)
- Enhanced `CreditService` with `get_credit_cost()` method for dynamic cost calculation
- Supports variable costs based on operation type and amount (word count, etc.)
- Updated `check_credits()` and `deduct_credits()` to support both legacy `required_credits` parameter and new `operation_type`/`amount` parameters
- Maintained full backward compatibility with existing code
- Created `AccountModuleSettings` model for module enable/disable functionality
- One settings record per account (get_or_create pattern)
- Enable/disable flags for all 8 modules: `planner_enabled`, `writer_enabled`, `thinker_enabled`, `automation_enabled`, `site_builder_enabled`, `linker_enabled`, `optimizer_enabled`, `publisher_enabled`
- Helper method `is_module_enabled(module_name)` for easy module checking
- Added `AccountModuleSettingsSerializer` and `AccountModuleSettingsViewSet`
- API endpoint: `/api/v1/system/settings/account-modules/`
- Custom action: `check/(?P<module_name>[^/.]+)` to check if a specific module is enabled
- Automatic account assignment on create
- Unified API Standard v1.0 compliant
- **Affected Areas**: Billing module (`constants.py`, `services.py`), System module (`settings_models.py`, `settings_serializers.py`, `settings_views.py`, `urls.py`)
- **Documentation**: See `docs/planning/phases/PHASE-0-FOUNDATION-CREDIT-SYSTEM.md` for complete details
- **Impact**: Foundation for credit-only system and module-based feature access control
- **Planning Documents Organization**: Organized architecture and implementation planning documents - **Planning Documents Organization**: Organized architecture and implementation planning documents
- Created `docs/planning/` directory for all planning documents - Created `docs/planning/` directory for all planning documents
- Moved `IGNY8-HOLISTIC-ARCHITECTURE-PLAN.md` to `docs/planning/` - Moved `IGNY8-HOLISTIC-ARCHITECTURE-PLAN.md` to `docs/planning/`

View File

@@ -192,6 +192,31 @@ class AIEngine:
self.step_tracker.add_request_step("PREP", "success", prep_message) self.step_tracker.add_request_step("PREP", "success", prep_message)
self.tracker.update("PREP", 25, prep_message, meta=self.step_tracker.get_meta()) self.tracker.update("PREP", 25, prep_message, meta=self.step_tracker.get_meta())
# Phase 2.5: CREDIT CHECK - Check credits before AI call (25%)
if self.account:
try:
from igny8_core.modules.billing.services import CreditService
from igny8_core.modules.billing.exceptions import InsufficientCreditsError
# Map function name to operation type
operation_type = self._get_operation_type(function_name)
# Calculate estimated cost
estimated_amount = self._get_estimated_amount(function_name, data, payload)
# Check credits BEFORE AI call
CreditService.check_credits(self.account, operation_type, estimated_amount)
logger.info(f"[AIEngine] Credit check passed: {operation_type}, estimated amount: {estimated_amount}")
except InsufficientCreditsError as e:
error_msg = str(e)
error_type = 'InsufficientCreditsError'
logger.error(f"[AIEngine] {error_msg}")
return self._handle_error(error_msg, fn, error_type=error_type)
except Exception as e:
logger.warning(f"[AIEngine] Failed to check credits: {e}", exc_info=True)
# Don't fail the operation if credit check fails (for backward compatibility)
# Phase 3: AI_CALL - Provider API Call (25-70%) # Phase 3: AI_CALL - Provider API Call (25-70%)
# Validate account exists before proceeding # Validate account exists before proceeding
if not self.account: if not self.account:
@@ -325,37 +350,45 @@ class AIEngine:
# Store save_msg for use in DONE phase # Store save_msg for use in DONE phase
final_save_msg = save_msg final_save_msg = save_msg
# Track credit usage after successful save # Phase 5.5: DEDUCT CREDITS - Deduct credits after successful save
if self.account and raw_response: if self.account and raw_response:
try: try:
from igny8_core.modules.billing.services import CreditService from igny8_core.modules.billing.services import CreditService
from igny8_core.modules.billing.models import CreditUsageLog from igny8_core.modules.billing.exceptions import InsufficientCreditsError
# Calculate credits used (based on tokens or fixed cost) # Map function name to operation type
credits_used = self._calculate_credits_for_clustering( operation_type = self._get_operation_type(function_name)
keyword_count=len(data.get('keywords', [])) if isinstance(data, dict) else len(data) if isinstance(data, list) else 1,
tokens=raw_response.get('total_tokens', 0),
cost=raw_response.get('cost', 0)
)
# Log credit usage (don't deduct from account.credits, just log) # Calculate actual amount based on results
CreditUsageLog.objects.create( actual_amount = self._get_actual_amount(function_name, save_result, parsed, data)
# Deduct credits using the new convenience method
CreditService.deduct_credits_for_operation(
account=self.account, account=self.account,
operation_type='clustering', operation_type=operation_type,
credits_used=credits_used, amount=actual_amount,
cost_usd=raw_response.get('cost'), cost_usd=raw_response.get('cost'),
model_used=raw_response.get('model', ''), model_used=raw_response.get('model', ''),
tokens_input=raw_response.get('tokens_input', 0), tokens_input=raw_response.get('tokens_input', 0),
tokens_output=raw_response.get('tokens_output', 0), tokens_output=raw_response.get('tokens_output', 0),
related_object_type='cluster', related_object_type=self._get_related_object_type(function_name),
related_object_id=save_result.get('id') or save_result.get('cluster_id') or save_result.get('task_id'),
metadata={ metadata={
'function_name': function_name,
'clusters_created': clusters_created, 'clusters_created': clusters_created,
'keywords_updated': keywords_updated, 'keywords_updated': keywords_updated,
'function_name': function_name 'count': count,
**save_result
} }
) )
logger.info(f"[AIEngine] Credits deducted: {operation_type}, amount: {actual_amount}")
except InsufficientCreditsError as e:
# This shouldn't happen since we checked before, but log it
logger.error(f"[AIEngine] Insufficient credits during deduction: {e}")
except Exception as e: except Exception as e:
logger.warning(f"Failed to log credit usage: {e}", exc_info=True) logger.warning(f"[AIEngine] Failed to deduct credits: {e}", exc_info=True)
# Don't fail the operation if credit deduction fails (for backward compatibility)
# Phase 6: DONE - Finalization (98-100%) # Phase 6: DONE - Finalization (98-100%)
success_msg = f"Task completed: {final_save_msg}" if 'final_save_msg' in locals() else "Task completed successfully" success_msg = f"Task completed: {final_save_msg}" if 'final_save_msg' in locals() else "Task completed successfully"
@@ -453,18 +486,74 @@ class AIEngine:
# Don't fail the task if logging fails # Don't fail the task if logging fails
logger.warning(f"Failed to log to database: {e}") logger.warning(f"Failed to log to database: {e}")
def _calculate_credits_for_clustering(self, keyword_count, tokens, cost): def _get_operation_type(self, function_name):
"""Calculate credits used for clustering operation""" """Map function name to operation type for credit system"""
# Use plan's cost per request if available, otherwise calculate from tokens mapping = {
if self.account and hasattr(self.account, 'plan') and self.account.plan: 'auto_cluster': 'clustering',
plan = self.account.plan 'generate_ideas': 'idea_generation',
# Check if plan has ai_cost_per_request config 'generate_content': 'content_generation',
if hasattr(plan, 'ai_cost_per_request') and plan.ai_cost_per_request: 'generate_image_prompts': 'image_prompt_extraction',
cluster_cost = plan.ai_cost_per_request.get('cluster', 0) 'generate_images': 'image_generation',
if cluster_cost: }
return int(cluster_cost) return mapping.get(function_name, function_name)
# Fallback: 1 credit per 30 keywords (minimum 1) def _get_estimated_amount(self, function_name, data, payload):
credits = max(1, int(keyword_count / 30)) """Get estimated amount for credit calculation (before operation)"""
return credits if function_name == 'generate_content':
# Estimate word count from task or default
if isinstance(data, dict):
return data.get('estimated_word_count', 1000)
return 1000 # Default estimate
elif function_name == 'generate_images':
# Count images to generate
if isinstance(payload, dict):
image_ids = payload.get('image_ids', [])
return len(image_ids) if image_ids else 1
return 1
elif function_name == 'generate_ideas':
# Count clusters
if isinstance(data, dict) and 'cluster_data' in data:
return len(data['cluster_data'])
return 1
# For fixed cost operations (clustering, image_prompt_extraction), return None
return None
def _get_actual_amount(self, function_name, save_result, parsed, data):
"""Get actual amount for credit calculation (after operation)"""
if function_name == 'generate_content':
# Get actual word count from saved content
if isinstance(save_result, dict):
word_count = save_result.get('word_count')
if word_count:
return word_count
# Fallback: estimate from parsed content
if isinstance(parsed, dict) and 'content' in parsed:
content = parsed['content']
return len(content.split()) if isinstance(content, str) else 1000
return 1000
elif function_name == 'generate_images':
# Count successfully generated images
count = save_result.get('count', 0)
if count > 0:
return count
return 1
elif function_name == 'generate_ideas':
# Count ideas generated
count = save_result.get('count', 0)
if count > 0:
return count
return 1
# For fixed cost operations, return None
return None
def _get_related_object_type(self, function_name):
"""Get related object type for credit logging"""
mapping = {
'auto_cluster': 'cluster',
'generate_ideas': 'content_idea',
'generate_content': 'content',
'generate_image_prompts': 'image',
'generate_images': 'image',
}
return mapping.get(function_name, 'unknown')

View File

@@ -1,25 +1,21 @@
""" """
Credit Cost Constants - Phase 0: Credit-Only System Credit Cost Constants
All features are unlimited. Only credits restrict usage. Phase 0: Credit-only system costs per operation
""" """
CREDIT_COSTS = { CREDIT_COSTS = {
# Existing operations
'clustering': 10, # Per clustering request 'clustering': 10, # Per clustering request
'idea_generation': 15, # Per cluster → ideas request 'idea_generation': 15, # Per cluster → ideas request
'content_generation': 1, # Per 100 words 'content_generation': 1, # Per 100 words
'image_prompt_extraction': 2, # Per content piece 'image_prompt_extraction': 2, # Per content piece
'image_generation': 5, # Per image 'image_generation': 5, # Per image
'linking': 8, # Per content piece (NEW)
# Legacy operation names (for backward compatibility) 'optimization': 1, # Per 200 words (NEW)
'ideas': 15, # Alias for idea_generation 'site_structure_generation': 50, # Per site blueprint (NEW)
'content': 1, # Alias for content_generation (per 100 words) 'site_page_generation': 20, # Per page (NEW)
'images': 5, # Alias for image_generation # Legacy operation types (for backward compatibility)
'reparse': 2, # Alias for image_prompt_extraction 'ideas': 15, # Alias for idea_generation
'content': 3, # Legacy: 3 credits per content piece
# NEW: Phase 2+ operations 'images': 5, # Alias for image_generation
'linking': 8, # Per content piece (NEW) 'reparse': 1, # Per reparse
'optimization': 1, # Per 200 words (NEW)
'site_structure_generation': 50, # Per site blueprint (NEW)
'site_page_generation': 20, # Per page (NEW)
} }

View File

@@ -19,43 +19,67 @@ class CreditService:
Args: Args:
operation_type: Type of operation (from CREDIT_COSTS) operation_type: Type of operation (from CREDIT_COSTS)
amount: Optional amount (word count, etc.) for variable costs amount: Optional amount (word count, image count, etc.)
Returns: Returns:
int: Number of credits required int: Number of credits required
Raises:
CreditCalculationError: If operation type is unknown
""" """
base_cost = CREDIT_COSTS.get(operation_type, 0) base_cost = CREDIT_COSTS.get(operation_type, 0)
if base_cost == 0:
raise CreditCalculationError(f"Unknown operation type: {operation_type}")
# Variable costs based on amount # Variable cost operations
if operation_type == 'content_generation' and amount: if operation_type == 'content_generation' and amount:
# Per 100 words # Per 100 words
return max(1, int(base_cost * (amount / 100))) return max(1, int(base_cost * (amount / 100)))
elif operation_type == 'optimization' and amount: elif operation_type == 'optimization' and amount:
# Per 200 words # Per 200 words
return max(1, int(base_cost * (amount / 200))) return max(1, int(base_cost * (amount / 200)))
elif operation_type == 'image_generation' and amount:
# Per image
return base_cost * amount
elif operation_type == 'idea_generation' and amount:
# Per idea
return base_cost * amount
# Fixed cost operations
return base_cost return base_cost
@staticmethod @staticmethod
def check_credits(account, required_credits=None, operation_type=None, amount=None): def check_credits(account, operation_type, amount=None):
""" """
Check if account has enough credits. Check if account has sufficient credits for an operation.
Args: Args:
account: Account instance account: Account instance
required_credits: Number of credits required (legacy parameter) operation_type: Type of operation
operation_type: Type of operation (new parameter) amount: Optional amount (word count, image count, etc.)
amount: Optional amount for variable costs (new parameter)
Raises: Raises:
InsufficientCreditsError: If account doesn't have enough credits InsufficientCreditsError: If account doesn't have enough credits
""" """
# Support both old and new API required = CreditService.get_credit_cost(operation_type, amount)
if operation_type: if account.credits < required:
required_credits = CreditService.get_credit_cost(operation_type, amount) raise InsufficientCreditsError(
elif required_credits is None: f"Insufficient credits. Required: {required}, Available: {account.credits}"
raise ValueError("Either required_credits or operation_type must be provided") )
return True
@staticmethod
def check_credits_legacy(account, required_credits):
"""
Legacy method: Check if account has enough credits (for backward compatibility).
Args:
account: Account instance
required_credits: Number of credits required
Raises:
InsufficientCreditsError: If account doesn't have enough credits
"""
if account.credits < required_credits: if account.credits < required_credits:
raise InsufficientCreditsError( raise InsufficientCreditsError(
f"Insufficient credits. Required: {required_credits}, Available: {account.credits}" f"Insufficient credits. Required: {required_credits}, Available: {account.credits}"
@@ -83,8 +107,8 @@ class CreditService:
Returns: Returns:
int: New credit balance int: New credit balance
""" """
# Check sufficient credits # Check sufficient credits (legacy: amount is already calculated)
CreditService.check_credits(account, amount) CreditService.check_credits_legacy(account, amount)
# Deduct from account.credits # Deduct from account.credits
account.credits -= amount account.credits -= amount
@@ -116,6 +140,61 @@ class CreditService:
return account.credits return account.credits
@staticmethod
@transaction.atomic
def deduct_credits_for_operation(account, operation_type, amount=None, description=None, metadata=None, cost_usd=None, model_used=None, tokens_input=None, tokens_output=None, related_object_type=None, related_object_id=None):
"""
Deduct credits for an operation (convenience method that calculates cost automatically).
Args:
account: Account instance
operation_type: Type of operation
amount: Optional amount (word count, image count, etc.)
description: Optional description (auto-generated if not provided)
metadata: Optional metadata dict
cost_usd: Optional cost in USD
model_used: Optional AI model used
tokens_input: Optional input tokens
tokens_output: Optional output tokens
related_object_type: Optional related object type
related_object_id: Optional related object ID
Returns:
int: New credit balance
"""
# Calculate credit cost
credits_required = CreditService.get_credit_cost(operation_type, amount)
# Check sufficient credits
CreditService.check_credits(account, operation_type, amount)
# Auto-generate description if not provided
if not description:
if operation_type == 'clustering':
description = f"Clustering operation"
elif operation_type == 'idea_generation':
description = f"Generated {amount or 1} idea(s)"
elif operation_type == 'content_generation':
description = f"Generated content ({amount or 0} words)"
elif operation_type == 'image_generation':
description = f"Generated {amount or 1} image(s)"
else:
description = f"{operation_type} operation"
return CreditService.deduct_credits(
account=account,
amount=credits_required,
operation_type=operation_type,
description=description,
metadata=metadata,
cost_usd=cost_usd,
model_used=model_used,
tokens_input=tokens_input,
tokens_output=tokens_output,
related_object_type=related_object_type,
related_object_id=related_object_id
)
@staticmethod @staticmethod
@transaction.atomic @transaction.atomic
def add_credits(account, amount, transaction_type, description, metadata=None): def add_credits(account, amount, transaction_type, description, metadata=None):
@@ -152,9 +231,7 @@ class CreditService:
def calculate_credits_for_operation(operation_type, **kwargs): def calculate_credits_for_operation(operation_type, **kwargs):
""" """
Calculate credits needed for an operation. Calculate credits needed for an operation.
Legacy method - use get_credit_cost() instead.
DEPRECATED: Use get_credit_cost() instead.
Kept for backward compatibility.
Args: Args:
operation_type: Type of operation operation_type: Type of operation
@@ -166,31 +243,22 @@ class CreditService:
Raises: Raises:
CreditCalculationError: If calculation fails CreditCalculationError: If calculation fails
""" """
# Map old operation types to new ones # Map legacy operation types
operation_mapping = { if operation_type == 'ideas':
'ideas': 'idea_generation', operation_type = 'idea_generation'
'content': 'content_generation', elif operation_type == 'content':
'images': 'image_generation', operation_type = 'content_generation'
'reparse': 'image_prompt_extraction', elif operation_type == 'images':
} operation_type = 'image_generation'
mapped_type = operation_mapping.get(operation_type, operation_type) # Extract amount from kwargs
amount = None
if 'word_count' in kwargs:
amount = kwargs.get('word_count')
elif 'image_count' in kwargs:
amount = kwargs.get('image_count')
elif 'idea_count' in kwargs:
amount = kwargs.get('idea_count')
# Handle variable costs return CreditService.get_credit_cost(operation_type, amount)
if mapped_type == 'content_generation':
word_count = kwargs.get('word_count') or kwargs.get('content_count', 1000) * 100
return CreditService.get_credit_cost(mapped_type, word_count)
elif mapped_type == 'clustering':
keyword_count = kwargs.get('keyword_count', 0)
# Clustering is fixed cost per request
return CreditService.get_credit_cost(mapped_type)
elif mapped_type == 'idea_generation':
idea_count = kwargs.get('idea_count', 1)
# Fixed cost per request
return CreditService.get_credit_cost(mapped_type)
elif mapped_type == 'image_generation':
image_count = kwargs.get('image_count', 1)
return CreditService.get_credit_cost(mapped_type) * image_count
return CreditService.get_credit_cost(mapped_type)

View File

@@ -0,0 +1,37 @@
# Generated manually for Phase 0: Module Enable Settings
from django.db import migrations, models
class Migration(migrations.Migration):
dependencies = [
('igny8_core_modules_system', '0006_alter_systemstatus_unique_together_and_more'),
('igny8_core_auth', '0008_passwordresettoken_alter_industry_options_and_more'),
]
operations = [
migrations.CreateModel(
name='ModuleEnableSettings',
fields=[
('id', models.BigAutoField(auto_created=True, primary_key=True, serialize=False, verbose_name='ID')),
('planner_enabled', models.BooleanField(default=True, help_text='Enable Planner module')),
('writer_enabled', models.BooleanField(default=True, help_text='Enable Writer module')),
('thinker_enabled', models.BooleanField(default=True, help_text='Enable Thinker module')),
('automation_enabled', models.BooleanField(default=True, help_text='Enable Automation module')),
('site_builder_enabled', models.BooleanField(default=True, help_text='Enable Site Builder module')),
('linker_enabled', models.BooleanField(default=True, help_text='Enable Linker module')),
('optimizer_enabled', models.BooleanField(default=True, help_text='Enable Optimizer module')),
('publisher_enabled', models.BooleanField(default=True, help_text='Enable Publisher module')),
('account', models.ForeignKey(on_delete=models.CASCADE, to='igny8_core_auth.account', db_column='tenant_id')),
],
options={
'db_table': 'igny8_module_enable_settings',
},
),
migrations.AddConstraint(
model_name='moduleenablesettings',
constraint=models.UniqueConstraint(fields=('account',), name='unique_account_module_enable_settings'),
),
]

View File

@@ -6,7 +6,7 @@ from igny8_core.auth.models import AccountBaseModel
# Import settings models # Import settings models
from .settings_models import ( from .settings_models import (
SystemSettings, AccountSettings, UserSettings, ModuleSettings, AccountModuleSettings, AISettings SystemSettings, AccountSettings, UserSettings, ModuleSettings, ModuleEnableSettings, AISettings
) )

View File

@@ -3,7 +3,7 @@ Settings Models Admin
""" """
from django.contrib import admin from django.contrib import admin
from igny8_core.admin.base import AccountAdminMixin from igny8_core.admin.base import AccountAdminMixin
from .settings_models import SystemSettings, AccountSettings, UserSettings, ModuleSettings, AISettings from .settings_models import SystemSettings, AccountSettings, UserSettings, ModuleSettings, ModuleEnableSettings, AISettings
@admin.register(SystemSettings) @admin.register(SystemSettings)

View File

@@ -92,12 +92,8 @@ class ModuleSettings(BaseSettings):
return f"ModuleSetting: {self.module_name} - {self.key}" return f"ModuleSetting: {self.module_name} - {self.key}"
class AccountModuleSettings(AccountBaseModel): class ModuleEnableSettings(AccountBaseModel):
""" """Module enable/disable settings per account"""
Account-level module enable/disable settings.
Phase 0: Credit System - Module Settings
"""
# Module enable/disable flags
planner_enabled = models.BooleanField(default=True, help_text="Enable Planner module") planner_enabled = models.BooleanField(default=True, help_text="Enable Planner module")
writer_enabled = models.BooleanField(default=True, help_text="Enable Writer module") writer_enabled = models.BooleanField(default=True, help_text="Enable Writer module")
thinker_enabled = models.BooleanField(default=True, help_text="Enable Thinker module") thinker_enabled = models.BooleanField(default=True, help_text="Enable Thinker module")
@@ -107,34 +103,23 @@ class AccountModuleSettings(AccountBaseModel):
optimizer_enabled = models.BooleanField(default=True, help_text="Enable Optimizer module") optimizer_enabled = models.BooleanField(default=True, help_text="Enable Optimizer module")
publisher_enabled = models.BooleanField(default=True, help_text="Enable Publisher module") publisher_enabled = models.BooleanField(default=True, help_text="Enable Publisher module")
updated_at = models.DateTimeField(auto_now=True)
created_at = models.DateTimeField(auto_now_add=True)
class Meta: class Meta:
db_table = 'igny8_account_module_settings' db_table = 'igny8_module_enable_settings'
verbose_name = 'Account Module Settings' unique_together = [['account']] # One record per account
verbose_name_plural = 'Account Module Settings'
# One settings record per account
constraints = [
models.UniqueConstraint(fields=['account'], name='unique_account_module_settings')
]
indexes = [
models.Index(fields=['account']),
]
def __str__(self): def __str__(self):
account = getattr(self, 'account', None) account = getattr(self, 'account', None)
return f"ModuleSettings: {account.name if account else 'No Account'}" return f"ModuleEnableSettings: {account.name if account else 'No Account'}"
@classmethod @classmethod
def get_or_create_for_account(cls, account): def get_or_create_for_account(cls, account):
"""Get or create module settings for an account""" """Get or create module enable settings for an account"""
settings, created = cls.objects.get_or_create(account=account) settings, created = cls.objects.get_or_create(account=account)
return settings return settings
def is_module_enabled(self, module_name): def is_module_enabled(self, module_name):
"""Check if a module is enabled""" """Check if a module is enabled"""
module_map = { mapping = {
'planner': self.planner_enabled, 'planner': self.planner_enabled,
'writer': self.writer_enabled, 'writer': self.writer_enabled,
'thinker': self.thinker_enabled, 'thinker': self.thinker_enabled,
@@ -144,7 +129,7 @@ class AccountModuleSettings(AccountBaseModel):
'optimizer': self.optimizer_enabled, 'optimizer': self.optimizer_enabled,
'publisher': self.publisher_enabled, 'publisher': self.publisher_enabled,
} }
return module_map.get(module_name, True) # Default to enabled if module not found return mapping.get(module_name, True) # Default to enabled if unknown
# AISettings extends IntegrationSettings (which already exists) # AISettings extends IntegrationSettings (which already exists)

View File

@@ -2,7 +2,7 @@
Serializers for Settings Models Serializers for Settings Models
""" """
from rest_framework import serializers from rest_framework import serializers
from .settings_models import SystemSettings, AccountSettings, UserSettings, ModuleSettings, AccountModuleSettings, AISettings from .settings_models import SystemSettings, AccountSettings, UserSettings, ModuleSettings, ModuleEnableSettings, AISettings
from .validators import validate_settings_schema from .validators import validate_settings_schema
@@ -58,10 +58,9 @@ class ModuleSettingsSerializer(serializers.ModelSerializer):
return value return value
class AccountModuleSettingsSerializer(serializers.ModelSerializer): class ModuleEnableSettingsSerializer(serializers.ModelSerializer):
"""Serializer for Account Module Settings (Phase 0)"""
class Meta: class Meta:
model = AccountModuleSettings model = ModuleEnableSettings
fields = [ fields = [
'id', 'planner_enabled', 'writer_enabled', 'thinker_enabled', 'id', 'planner_enabled', 'writer_enabled', 'thinker_enabled',
'automation_enabled', 'site_builder_enabled', 'linker_enabled', 'automation_enabled', 'site_builder_enabled', 'linker_enabled',

View File

@@ -13,10 +13,10 @@ from igny8_core.api.authentication import JWTAuthentication, CSRFExemptSessionAu
from igny8_core.api.pagination import CustomPageNumberPagination from igny8_core.api.pagination import CustomPageNumberPagination
from igny8_core.api.throttles import DebugScopedRateThrottle from igny8_core.api.throttles import DebugScopedRateThrottle
from igny8_core.api.permissions import IsAuthenticatedAndActive, HasTenantAccess, IsAdminOrOwner from igny8_core.api.permissions import IsAuthenticatedAndActive, HasTenantAccess, IsAdminOrOwner
from .settings_models import SystemSettings, AccountSettings, UserSettings, ModuleSettings, AccountModuleSettings, AISettings from .settings_models import SystemSettings, AccountSettings, UserSettings, ModuleSettings, ModuleEnableSettings, AISettings
from .settings_serializers import ( from .settings_serializers import (
SystemSettingsSerializer, AccountSettingsSerializer, UserSettingsSerializer, SystemSettingsSerializer, AccountSettingsSerializer, UserSettingsSerializer,
ModuleSettingsSerializer, AccountModuleSettingsSerializer, AISettingsSerializer ModuleSettingsSerializer, ModuleEnableSettingsSerializer, AISettingsSerializer
) )
@@ -282,68 +282,98 @@ class ModuleSettingsViewSet(AccountModelViewSet):
update=extend_schema(tags=['System']), update=extend_schema(tags=['System']),
partial_update=extend_schema(tags=['System']), partial_update=extend_schema(tags=['System']),
) )
class AccountModuleSettingsViewSet(AccountModelViewSet): class ModuleEnableSettingsViewSet(AccountModelViewSet):
""" """
ViewSet for managing account module enable/disable settings. ViewSet for managing module enable/disable settings
Phase 0: Credit System - Module Settings Unified API Standard v1.0 compliant
One settings record per account (get_or_create pattern) One record per account
""" """
queryset = AccountModuleSettings.objects.all() queryset = ModuleEnableSettings.objects.all()
serializer_class = AccountModuleSettingsSerializer serializer_class = ModuleEnableSettingsSerializer
permission_classes = [IsAuthenticatedAndActive, HasTenantAccess] permission_classes = [IsAuthenticatedAndActive, HasTenantAccess, IsAdminOrOwner]
authentication_classes = [JWTAuthentication, CSRFExemptSessionAuthentication] authentication_classes = [JWTAuthentication, CSRFExemptSessionAuthentication]
pagination_class = CustomPageNumberPagination
throttle_scope = 'system' throttle_scope = 'system'
throttle_classes = [DebugScopedRateThrottle] throttle_classes = [DebugScopedRateThrottle]
def get_queryset(self): def get_queryset(self):
"""Get module settings for current account""" """Get module enable settings for current account"""
queryset = super().get_queryset() queryset = super().get_queryset()
return queryset.filter(account=self.request.account) return queryset
def list(self, request, *args, **kwargs): def list(self, request):
"""Get or create module settings for account""" """Get or create module enable settings for current account"""
account = request.account account = getattr(request, 'account', None)
settings = AccountModuleSettings.get_or_create_for_account(account)
serializer = self.get_serializer(settings)
return success_response(data=serializer.data, request=request)
def retrieve(self, request, pk=None):
"""Get module settings for account"""
account = request.account
try:
settings = AccountModuleSettings.objects.get(account=account, pk=pk)
except AccountModuleSettings.DoesNotExist:
# Create if doesn't exist
settings = AccountModuleSettings.get_or_create_for_account(account)
serializer = self.get_serializer(settings)
return success_response(data=serializer.data, request=request)
def perform_create(self, serializer):
"""Set account automatically"""
account = getattr(self.request, 'account', None)
if not account: if not account:
user = getattr(self.request, 'user', None) user = getattr(request, 'user', None)
if user: if user:
account = getattr(user, 'account', None) account = getattr(user, 'account', None)
if not account: if not account:
from rest_framework.exceptions import ValidationError return error_response(
raise ValidationError("Account is required") error='Account not found',
status_code=status.HTTP_400_BAD_REQUEST,
request=request
)
serializer.save(account=account) # Get or create settings for account
settings = ModuleEnableSettings.get_or_create_for_account(account)
serializer = self.get_serializer(settings)
return success_response(data=serializer.data, request=request)
@action(detail=False, methods=['get'], url_path='check/(?P<module_name>[^/.]+)', url_name='check_module') def retrieve(self, request, pk=None):
def check_module(self, request, module_name=None): """Get module enable settings for current account"""
"""Check if a specific module is enabled""" account = getattr(request, 'account', None)
account = request.account if not account:
settings = AccountModuleSettings.get_or_create_for_account(account) user = getattr(request, 'user', None)
is_enabled = settings.is_module_enabled(module_name) if user:
return success_response( account = getattr(user, 'account', None)
data={'module_name': module_name, 'enabled': is_enabled},
if not account:
return error_response(
error='Account not found',
status_code=status.HTTP_400_BAD_REQUEST,
request=request
)
# Get or create settings for account
settings = ModuleEnableSettings.get_or_create_for_account(account)
serializer = self.get_serializer(settings)
return success_response(data=serializer.data, request=request)
def update(self, request, pk=None):
"""Update module enable settings for current account"""
account = getattr(request, 'account', None)
if not account:
user = getattr(request, 'user', None)
if user:
account = getattr(user, 'account', None)
if not account:
return error_response(
error='Account not found',
status_code=status.HTTP_400_BAD_REQUEST,
request=request
)
# Get or create settings for account
settings = ModuleEnableSettings.get_or_create_for_account(account)
serializer = self.get_serializer(settings, data=request.data, partial=True)
if serializer.is_valid():
serializer.save()
return success_response(data=serializer.data, request=request)
return error_response(
error='Validation failed',
errors=serializer.errors,
status_code=status.HTTP_400_BAD_REQUEST,
request=request request=request
) )
def partial_update(self, request, pk=None):
"""Partial update module enable settings"""
return self.update(request, pk)
@extend_schema_view( @extend_schema_view(
list=extend_schema(tags=['System']), list=extend_schema(tags=['System']),

View File

@@ -7,7 +7,7 @@ from .views import AIPromptViewSet, AuthorProfileViewSet, StrategyViewSet, syste
from .integration_views import IntegrationSettingsViewSet from .integration_views import IntegrationSettingsViewSet
from .settings_views import ( from .settings_views import (
SystemSettingsViewSet, AccountSettingsViewSet, UserSettingsViewSet, SystemSettingsViewSet, AccountSettingsViewSet, UserSettingsViewSet,
ModuleSettingsViewSet, AccountModuleSettingsViewSet, AISettingsViewSet ModuleSettingsViewSet, ModuleEnableSettingsViewSet, AISettingsViewSet
) )
router = DefaultRouter() router = DefaultRouter()
router.register(r'prompts', AIPromptViewSet, basename='prompts') router.register(r'prompts', AIPromptViewSet, basename='prompts')
@@ -17,7 +17,7 @@ router.register(r'settings/system', SystemSettingsViewSet, basename='system-sett
router.register(r'settings/account', AccountSettingsViewSet, basename='account-settings') router.register(r'settings/account', AccountSettingsViewSet, basename='account-settings')
router.register(r'settings/user', UserSettingsViewSet, basename='user-settings') router.register(r'settings/user', UserSettingsViewSet, basename='user-settings')
router.register(r'settings/modules', ModuleSettingsViewSet, basename='module-settings') router.register(r'settings/modules', ModuleSettingsViewSet, basename='module-settings')
router.register(r'settings/account-modules', AccountModuleSettingsViewSet, basename='account-module-settings') router.register(r'settings/modules/enable', ModuleEnableSettingsViewSet, basename='module-enable-settings')
router.register(r'settings/ai', AISettingsViewSet, basename='ai-settings') router.register(r'settings/ai', AISettingsViewSet, basename='ai-settings')
# Custom URL patterns for integration settings - matching reference plugin structure # Custom URL patterns for integration settings - matching reference plugin structure