~bigbes/sr-ht-dolt · parade

main · last commit 6 hours ago · ti41arbu

← Back to the parade

sr-ht-dolt-3s9 Adopt browse's NULL mask in the readers Past Stand

status: closed P3 task
bd reopen sr-ht-dolt-3s9
Created byEugene Blikh
Ownerbigbes@gmail.com
Created2026-08-13T07:06:41Z
Started2026-08-13T07:18:50Z
Updated2026-08-13T07:36:37Z
Closed2026-08-13T07:36:37Z
Description
Once browse can say which cells are actually NULL (sr-ht-dolt-0df), the readers should use it: mcpsrv read_rows should render a real NULL as JSON null instead of the string "NULL" and drop the warning from its tool description, and beads.cell() should keep treating a real NULL as empty while no longer flattening a stored "NULL" string to empty. The browse half is deliberately separate so the placeholder rendering the HTML pages depend on does not change under them.

Depends on

No outgoing dependencies.

Depended on by

Nothing depends on this issue.

No comments.

Close reason

Landed in 6307350 (read_rows cells are [][]*string — a real NULL is JSON null on the wire, asserted on the marshalled payload; the description's warning removed) and c82f106 (beads rowCells binds a row to its mask, so cell() reads the mask and ~100 call sites did not move). A hand-built page with no mask keeps the older reading, documented and pinned rather than left implicit.
  • Eugene Blikh created the issue · 2026-08-13T10:06:40Z
  • Eugene Blikh changed status to in_progress · 2026-08-13T10:18:49Z
  • Eugene Blikh closed the issue · 2026-08-13T10:36:37Z
    Landed in 6307350 (read_rows cells are [][]*string — a real NULL is JSON null on the wire, asserted on the marshalled payload; the description's warning removed) and c82f106 (beads rowCells binds a row to its mask, so cell() reads the mask and ~100 call sites did not move). A hand-built page with no mask keeps the older reading, documented and pinned rather than left implicit.
Stored rows — what this pane was built from, as read
issues 1 row
id sr-ht-dolt-3s9
content_hash dce98d223440919c25ba0e8e76711bd6f573a359b6c581ba922a0b73f56fd3fc
title Adopt browse's NULL mask in the readers
description Once browse can say which cells are actually NULL (sr-ht-dolt-0df), the readers should use it: mcpsrv read_rows should render a real NULL as JSON null instead of the string "NULL" and drop the warning from its tool description, and beads.cell() should keep treating a real NULL as empty while no longer flattening a stored "NULL" string to empty. The browse half is deliberately separate so the placeholder rendering the HTML pages depend on does not change under them.
design
acceptance_criteria
notes
status closed
priority 3
issue_type task
assignee NULL
estimated_minutes NULL
created_at 2026-08-13T07:06:41Z
created_by Eugene Blikh
owner bigbes@gmail.com
updated_at 2026-08-13T07:36:37Z
closed_at 2026-08-13T07:36:37Z
closed_by_session
external_ref NULL
spec_id
compaction_level 0
compacted_at NULL
compacted_at_commit NULL
original_size NULL
sender
ephemeral 0
wisp_type
pinned 0
is_template 0
mol_type
work_type
source_system
metadata �{}
source_repo
close_reason Landed in 6307350 (read_rows cells are [][]*string — a real NULL is JSON null on the wire, asserted on the marshalled payload; the description's warning removed) and c82f106 (beads rowCells binds a row to its mask, so cell() reads the mask and ~100 call sites did not move). A hand-built page with no mask keeps the older reading, documented and pinned rather than left implicit.
event_kind
actor
target
payload
await_type
await_id
timeout_ns 0
waiters
hook_bead
role_bead
agent_state
last_activity NULL
role_type
rig
due_at NULL
defer_until NULL
no_history 0
started_at 2026-08-13T07:18:50Z
is_blocked 0
events 3 rows
id 019ff9f1-ca7a-78a6-bbde-b02cee16b5c0
issue_id sr-ht-dolt-3s9
event_type created
actor Eugene Blikh
old_value
new_value
comment NULL
created_at 2026-08-13T10:06:40Z
id 019ff9fc-ea3b-73a9-87ca-87f1529b472e
issue_id sr-ht-dolt-3s9
event_type status_changed
actor Eugene Blikh
old_value {"id":"sr-ht-dolt-3s9","title":"Adopt browse's NULL mask in the readers","description":"Once browse can say which cells are actually NULL (sr-ht-dolt-0df), the readers should use it: mcpsrv read_rows should render a real NULL as JSON null instead of the string \"NULL\" and drop the warning from its tool description, and beads.cell() should keep treating a real NULL as empty while no longer flattening a stored \"NULL\" string to empty. The browse half is deliberately separate so the placeholder rendering the HTML pages depend on does not change under them.","status":"open","priority":3,"issue_type":"task","owner":"bigbes@gmail.com","created_at":"2026-08-13T07:06:41Z","created_by":"Eugene Blikh","updated_at":"2026-08-13T07:06:41Z"}
new_value {"status":"in_progress"}
comment NULL
created_at 2026-08-13T10:18:49Z
id 019ffa0d-3420-76f1-b48d-4b016f001b29
issue_id sr-ht-dolt-3s9
event_type closed
actor Eugene Blikh
old_value
new_value Landed in 6307350 (read_rows cells are [][]*string — a real NULL is JSON null on the wire, asserted on the marshalled payload; the description's warning removed) and c82f106 (beads rowCells binds a row to its mask, so cell() reads the mask and ~100 call sites did not move). A hand-built page with no mask keeps the older reading, documented and pinned rather than left implicit.
comment NULL
created_at 2026-08-13T10:36:37Z