Skip to content

feat: add DELETE /collection/:id/resources endpoint#835

Open
Husainbw786 wants to merge 1 commit intotestingfrom
feat/delete-collection-resources
Open

feat: add DELETE /collection/:id/resources endpoint#835
Husainbw786 wants to merge 1 commit intotestingfrom
feat/delete-collection-resources

Conversation

@Husainbw786
Copy link
Copy Markdown
Collaborator

What changed

Added a new DELETE endpoint for deleting resources from a collection by owner ID.

New endpoint: DELETE /collection/:id/resources?ownerId=<ownerId>

Changes made:

  1. Added deleteResourcesByCollectionAndOwner controller function in rag.controller.js
  2. Added deleteResourcesByCollectionQuerySchema validation schema in rag.validation.js
  3. Added the DELETE route in rag.routes.js

Why

To support deleting resources from a collection based on the collection ID and owner ID. This follows the existing pattern used for other RAG endpoints.

Notes

  • Uses the existing Hippocampus API endpoint: DELETE /collection/:collectionId/resources?ownerId=<ownerId>
  • Protected with middleware and checkAgentAccessMiddleware consistent with other resource management endpoints
  • ownerId is required as a query parameter

@windsurf-bot
Copy link
Copy Markdown
Contributor

windsurf-bot bot commented Mar 20, 2026

PR review rate limit exceeded

@Husainbw786 Husainbw786 force-pushed the feat/delete-collection-resources branch from ea9d13f to 1330e97 Compare March 20, 2026 13:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants