Skip to content

chore: item update: detaching a parent requires parent_id="" which MCP clients strip; add a clear_parent action (mirror clear_start_date/clear_due_date) - #952

Merged
getappz merged 3 commits into
masterfrom
task/772-item-update-detaching-a-parent-requires
Oct 24, 2026
Merged

getappz merged 3 commits into
masterfrom
task/772-item-update-detaching-a-parent-requires

Conversation

@getappz

@getappz getappz commented Oct 22, 2026

Copy link
Copy Markdown
Owner

Adds a dedicated clear_parent action that detaches through the existing update path using only id, so MCP clients do not need to serialize an empty value. Keeps empty and whitespace-only parent updates compatible, documents the recommended action, and adds regression coverage for child detachment, root items, and whitespace updates.


Opened by codex on i-flare-c997d745ae66 (flared:c997d745ae66) for item #772 via agentflare.

@coderabbitai

coderabbitai Bot commented Oct 22, 2026 •

Copy link
Copy Markdown

Warning

Review limit reached

Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.
You're only billed for reviews past your plan's rate limits ($0.25/file).

Next included review available in 6 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used all 2 included reviews currently available. Your 54 included PR review attempts over the past 7 days set your current allowance at 2 reviews per hour.

Learn how review limits work.

Review configuration:

Run configuration
Commits

Reviewing files that changed from the base of the PR and between 87fd9d4 and 35ee695.

Files selected for processing: 5
  • src/mcp_server.rs
  • src/mcp_server/item.rs
  • src/mcp_server/tests/item_parent_tests.rs
  • src/mcp_server/tests/mod.rs
  • src/mcp_server/types.rs
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Comment @coderabbitai help to get the list of available commands.

@getappz
getappz marked this pull request as ready for review October 22, 2026 09:45
@getappz
getappz force-pushed the task/772-item-update-detaching-a-parent-requires branch from 9561718 to 2b1df59 Compare October 24, 2026 07:20
Agentflare-Branch: task/772-item-update-detaching-a-parent-requires
Agentflare-Item: 772
Agentflare-Branch: task/772-item-update-detaching-a-parent-requires
Agentflare-Item: 772
Agentflare-Agent: claude-code_2-1-310_harness
Agentflare-Branch: task/772-item-update-detaching-a-parent-requires
Agentflare-Item: 772
Agentflare-Session: 6278f161-4bdc-41bb-a62e-8f0e3e29b9ed
@getappz
getappz force-pushed the task/772-item-update-detaching-a-parent-requires branch from 2b1df59 to 35ee695 Compare October 24, 2026 07:38
@getappz
getappz merged commit f72c996 into master Oct 24, 2026
14 of 16 checks passed
@getappz
getappz deleted the task/772-item-update-detaching-a-parent-requires branch October 24, 2026 07:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant