[matej21]

Fixes #136

Problem

DataGridHasOneColumn wraps every cell in the include/exclude filter tooltip when the related handle has an id. A disconnected has-one still returns a placeholder id (__placeholder_…), so an empty cell got the tooltip too. „Filter" or „Exclude" there stored the placeholder id in the filter artifact, and the server rejected the resulting { id: { eq: "__placeholder_…" } } condition as an invalid id.

Change

• has-one-column.tsx: wrap the cell only when the related id is a persisted id (isPersistedId). An empty relation, and a temporary id, render the plain cell content.
• An empty value stays filterable through the column's null („N/A") filter.
• has-many-column.tsx is not changed: a has-many handle has no id, so its wrapper already returns the plain content.

Tests

• New tests/react/dataview/hasOneColumnEmptyCellFilter.test.tsx: a grid with one connected and one empty author. The connected cell has the tooltip trigger; the empty cell has none. It fails on main (commit c741fad) and passes with the fix.
• bun run typecheck and bun run test (2199 pass) pass locally. The browser suite was not run locally.

🤖 Generated with Claude Code

https://claude.ai/code/session_013L98ArhvRJ1p7riCvTRXBf

[matej21]

Review round 1

• Reviewer: review agent (claude-opus-5-5)
• Commit reviewed: 05fcae8 (base 7ae3d15)
• Verdict: approve

The guard in renderCellWrapper now skips the include/exclude tooltip unless the related id is a persisted id. That covers both the disconnected placeholder id and a temporary id from an unsaved entity. isPersistedId is exported from @contember/bindx. The empty value stays filterable through the column's null filter. The regression test renders a connected and an empty cell and checks the tooltip trigger on each, so it covers the changed behaviour.

Findings

• nit: ?? null was dropped, so id is now string | undefined. No behaviour change, because !id handles both.

No correctness, security or test-coverage issues found.