[nishantmonu51]

• moveItemAcrossContainers spliced the raw YAML item into the destination without reading the if of the source tab group, tab or row, so an item dragged into another tab, out of a tab group or onto the top-level rows became visible to every viewer. In embeds and public URLs restricted to the canvas, its metrics view became queryable too.
• Fold the source container's conditions that the destination doesn't already apply into the item's own if, as the same-container move and deleteTab already do.
• Covers tab to tab in one group, tab to top-level rows, top-level rows to tab, and tab group to tab group, including the "new row" branch.

Steps to reproduce:

1. Create a canvas with a tab group. On tab "Finance", set if: '{{ has "finance" .user.groups }}' and add a KPI.
2. In the editor, drag the KPI onto tab "Overview" or out of the tab group.
3. Before this fix the YAML shows the KPI with no if, and "View as" a user outside finance shows it.

Introduced by #10006. Not present in 0.90.

Checklist:

• Covered by tests
• Ran it and it works as intended
• Reviewed the diff before requesting a review
• Checked for unhandled edge cases
• Linked the issues it closes
• Checked if the docs need to be updated. If so, create a separate Linear DOCS issue
• Intend to cherry-pick into the release branch
• I'm proud of this work!