Skip to content

fix: Check the status project in the create_personal_task MCP tool - EXO-90711 - #659

Merged
boubaker merged 1 commit into
feature/maintenancefrom
fix/mcp-personal-task-status-check
Oct 5, 2026
Merged

boubaker merged 1 commit into
feature/maintenancefrom
fix/mcp-personal-task-status-check

Conversation

@boubaker

Copy link
Copy Markdown
Member

Fix: the create_personal_task MCP tool (TaskMcpTool#createPersonalTask) loads the posted status, refuses an unknown one (ObjectNotFoundException), and requires the user to be able to create tasks in its project (checkCanCreateInProject), as create_task_in_project does. The tool definition is unchanged.

Tests: TaskMcpToolTest pins the refusal, the unknown status and the allowed case; each guard, reverted, fails its test.

Knowledge: to follow — the eng-standards pitfall ledger PR that records this surface, opened after its dedup with the parallel backfill; its entry is re-stamped once this fix reaches develop.

This change is classified N1 (computed on 8df063b87) — its approver must be an Archi/Dev who knows it is N1, not an approval on AI review alone; author ≠ approver.

🤖 Generated with Claude Code

@boubaker boubaker changed the title fix: Check the status project in the create_personal_task MCP tool EXO-TBD fix: Check the status project in the create_personal_task MCP tool - EXO-90711 Sep 28, 2026
@boubaker
boubaker requested a review from ahamdi September 28, 2026 16:32
@boubaker
boubaker marked this pull request as ready for review September 28, 2026 16:32
ahamdi
ahamdi previously approved these changes Sep 29, 2026
@exo-swf
exo-swf dismissed ahamdi’s stale review September 30, 2026 23:18

The merge-base changed after approval.

@exo-swf
exo-swf force-pushed the feature/maintenance branch 2 times, most recently from 5f3a58d to 2a5a860 Compare October 1, 2026 23:18
…EXO-TBD

The tool loads the posted status, refuses an unknown one, and requires
the user to be able to create tasks in its project, as
create_task_in_project does.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@boubaker
boubaker force-pushed the fix/mcp-personal-task-status-check branch from 8df063b to ae657bb Compare October 5, 2026 13:10
@boubaker
boubaker requested a review from Jihed525 October 5, 2026 13:11
@boubaker

boubaker commented Oct 5, 2026

Copy link
Copy Markdown
Member Author

Rebased on top of feature/maintenance

@sonarqubecloud

sonarqubecloud Bot commented Oct 5, 2026

Copy link
Copy Markdown

@boubaker
boubaker merged commit e416f9f into feature/maintenance Oct 5, 2026
9 checks passed
@boubaker
boubaker deleted the fix/mcp-personal-task-status-check branch October 5, 2026 13:40
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