feat: support document and collection summary

[iziang]

No description provided.

[iziang]

bugbot run

[cursor]

Bug: Callback Parameters Mismatch

The on_summary_failed callback is called with incorrect parameters. It expects (summary_id, error_message, target_version) but receives (collection_id, str(e)). This incorrectly uses collection_id instead of summary_id and omits target_version, preventing the failure callback from working properly.

config/celery_tasks.py#L806-L810

ApeRAG/config/celery_tasks.py

Lines 806 to 810 in 28f7178

	# Mark as failed using callback if we've exhausted retries
	if self.request.retries >= self.max_retries:
	from aperag.service.collection_summary_service import collection_summary_callbacks
	collection_summary_callbacks.on_summary_failed(collection_id, str(e))

Fix in Cursor • Fix in Web

---

Bug: Mismatched Collection Models in Summary Generation

The generate_collection_summary_view endpoint passes a view_models.Collection object (returned by collection_service.get_collection) to collection_summary_service.trigger_collection_summary_generation. This service expects a db_models.Collection database model, leading to runtime errors. Specifically, the config attribute's type differs (CollectionConfig in the view model vs. Text in the database model), causing issues when parseCollectionConfig() is called.

aperag/views/main.py#L112-L118

ApeRAG/aperag/views/main.py

Lines 112 to 118 in 28f7178

	# Check if collection exists
	collection = await collection_service.get_collection(str(user.id), collection_id)
	if not collection:
	raise HTTPException(status_code=404, detail="Collection not found")
	# Trigger async summary generation
	task_triggered = await collection_summary_service.trigger_collection_summary_generation(collection)

Fix in Cursor • Fix in Web

---

Bug: Null Check Error in Configuration Parsing

Missing null check: The parseCollectionConfig() function might return None, but the code immediately attempts to access config.enable_summary without verifying if config is None. This can lead to an AttributeError at runtime.

aperag/tasks/document.py#L128-L131

ApeRAG/aperag/tasks/document.py

Lines 128 to 131 in 28f7178

	# Check if summary is enabled in collection config
	config = parseCollectionConfig(collection.config)
	if not config.enable_summary:

aperag/tasks/document.py#L291-L294

ApeRAG/aperag/tasks/document.py

Lines 291 to 294 in 28f7178

	from aperag.schema.utils import parseCollectionConfig
	# Check if summary is enabled in collection config
	config = parseCollectionConfig(collection.config)

Fix in Cursor • Fix in Web

---

Was this report helpful? Give feedback by reacting with 👍 or 👎