Skip to content

Conversation

@Scra3
Copy link
Member

@Scra3 Scra3 commented Dec 22, 2025

Summary

  • Add new delete MCP tool to delete one or more records from a collection
  • Accepts collectionName and recordIds array
  • Activity logging with 'delete' action
  • Returns success message with count of deleted records

Dependencies

This PR is based on #1386 (feat/list-related) and should be merged after it.

Test plan

  • Unit tests for delete tool (14 tests)
  • All mcp-server tests pass (335 tests)

🤖 Generated with Claude Code

@qltysh
Copy link

qltysh bot commented Dec 22, 2025

1 new issue

Tool Category Rule Count
qlty Structure Function with many parameters (count = 4): declareDeleteTool 1

@Scra3 Scra3 mentioned this pull request Dec 22, 2025
2 tasks
@qltysh
Copy link

qltysh bot commented Dec 22, 2025

Qlty

Coverage Impact

⬇️ Merging this pull request will decrease total coverage on main by 0.41%.

Modified Files with Diff Coverage (3)

RatingFile% DiffUncovered Line #s
New file Coverage rating: A
packages/mcp-server/src/tools/update.ts100.0%
New file Coverage rating: A
packages/mcp-server/src/tools/delete.ts100.0%
New file Coverage rating: B
packages/mcp-server/src/server.ts100.0%
Total100.0%
🚦 See full report on Qlty Cloud »

🛟 Help
  • Diff Coverage: Coverage for added or modified lines of code (excludes deleted files). Learn more.

  • Total Coverage: Coverage for the whole repository, calculated as the sum of all File Coverage. Learn more.

  • File Coverage: Covered Lines divided by Covered Lines plus Missed Lines. (Excludes non-executable lines including blank lines and comments.)

    • Indirect Changes: Changes to File Coverage for files that were not modified in this PR. Learn more.

@Scra3 Scra3 force-pushed the feat/list-related branch 2 times, most recently from 4b19b0c to 11e8f63 Compare December 23, 2025 10:50
Base automatically changed from feat/list-related to main December 23, 2025 16:02
@Scra3 Scra3 force-pushed the feat/delete branch 6 times, most recently from 86803ce to cac3897 Compare December 24, 2025 14:33
@Scra3 Scra3 force-pushed the feat/delete branch 3 times, most recently from 85da574 to 9e994ee Compare December 24, 2025 14:53
Comment on lines 1 to 5
module.exports = {
rules: {
'import/extensions': 'off',
},
};
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To delete

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Eslint does not support .js extension. How do you want to fix this ?

alban bertolini and others added 2 commits December 24, 2025 16:18
Add a new MCP tool to delete one or more records from a collection:
- Accepts collectionName and recordIds array
- Activity logging with 'delete' action
- Returns success message with count of deleted records

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <[email protected]>
@VincentMolinie VincentMolinie merged commit d633d5c into main Dec 24, 2025
24 checks passed
@VincentMolinie VincentMolinie deleted the feat/delete branch December 24, 2025 15:41
forest-bot added a commit that referenced this pull request Dec 24, 2025
# @forestadmin/mcp-server [1.5.0](https://github.com/ForestAdmin/agent-nodejs/compare/@forestadmin/[email protected]...@forestadmin/[email protected]) (2025-12-24)

### Features

* **mcp-server:** add delete tool ([#1387](#1387)) ([d633d5c](d633d5c))
* **mcp-server:** add update tool ([#1389](#1389)) ([93f352d](93f352d))
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.

3 participants