~bigbes/sr-ht-dolt · parade

main · last commit 6 hours ago · ti41arbu

← Back to the parade

sr-ht-dolt-0qf.4 mcpsrv: generic tools over the browse seam Past Stand

status: closed P2 task
bd reopen sr-ht-dolt-0qf.4
Created byEugene Blikh
Ownerbigbes@gmail.com
Created2026-08-12T20:11:45Z
Started2026-08-13T05:27:34Z
Updated2026-08-13T05:46:43Z
Closed2026-08-13T05:46:43Z
Description
list_branches, list_tables, read_rows, get_commit_log, get_commit_diff — the browse surface as tools, addressing a database as {owner, name} and defaulting ref to the repository default branch. Caps: read_rows limit<=500 (default 100), get_commit_log limit<=100 (default 25); a truncated result says so and carries the true total.
Design
docs/DESIGN.mcp.md §9.1, §9.3. Visibility is core.Allowed/OpBrowse with NotFoundForPrivate masking — never a distinguishable forbidden.
Acceptance criteria
Visibility matrix per tool over PUBLIC/UNLISTED/PRIVATE x anonymous/stranger/grantee/owner; cap tests.

Depends on

  • sr-ht-dolt-0qf — MCP surface: a read-only agent door to hosted databases parent-child closed
  • sr-ht-dolt-0qf.3 — mcpsrv: skeleton, Host allowlist, grant gate, list_databases blocks closed

Depended on by

Prerequisite chain — everything this waits on, transitively

  • sr-ht-dolt-0qf — MCP surface: a read-only agent door to hosted databases parent-child closed
  • sr-ht-dolt-0qf.3 — mcpsrv: skeleton, Host allowlist, grant gate, list_databases blocks closed
  • sr-ht-dolt-0qf.2 — authn: the tokens.sr.ht bearer plane and core.GrantRead blocks closed

No comments.

Close reason

Landed in 7a77409: list_branches, list_tables, read_rows, get_commit_log, get_commit_diff in mcpsrv/browse.go. Caps clamp and say so (limit echoes what was applied); a masked database and a missing one share a byte-identical sentence; a bad ref/table is a plain answer, not the mask. 176 tests, cap test mutation-checked.
  • Eugene Blikh added under epic sr-ht-dolt-0qf · 2026-08-12T23:11:44Z
  • Eugene Blikh created the issue · 2026-08-12T23:11:44Z
  • Eugene Blikh added dependency on sr-ht-dolt-0qf.3 · 2026-08-12T23:12:55Z
  • Eugene Blikh sr-ht-dolt-0qf.6 now depends on this · 2026-08-12T23:12:56Z
  • Eugene Blikh changed status to in_progress · 2026-08-13T08:27:34Z
  • Eugene Blikh closed the issue · 2026-08-13T08:46:43Z
    Landed in 7a77409: list_branches, list_tables, read_rows, get_commit_log, get_commit_diff in mcpsrv/browse.go. Caps clamp and say so (limit echoes what was applied); a masked database and a missing one share a byte-identical sentence; a bad ref/table is a plain answer, not the mask. 176 tests, cap test mutation-checked.
Stored rows — what this pane was built from, as read
issues 1 row
id sr-ht-dolt-0qf.4
content_hash bd8dd472bd71f282d5eaea3072a8e8cd448fb2c7d330444408baed331e9121f6
title mcpsrv: generic tools over the browse seam
description list_branches, list_tables, read_rows, get_commit_log, get_commit_diff — the browse surface as tools, addressing a database as {owner, name} and defaulting ref to the repository default branch. Caps: read_rows limit<=500 (default 100), get_commit_log limit<=100 (default 25); a truncated result says so and carries the true total.
design docs/DESIGN.mcp.md §9.1, §9.3. Visibility is core.Allowed/OpBrowse with NotFoundForPrivate masking — never a distinguishable forbidden.
acceptance_criteria Visibility matrix per tool over PUBLIC/UNLISTED/PRIVATE x anonymous/stranger/grantee/owner; cap tests.
notes
status closed
priority 2
issue_type task
assignee NULL
estimated_minutes NULL
created_at 2026-08-12T20:11:45Z
created_by Eugene Blikh
owner bigbes@gmail.com
updated_at 2026-08-13T05:46:43Z
closed_at 2026-08-13T05:46:43Z
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 7a77409: list_branches, list_tables, read_rows, get_commit_log, get_commit_diff in mcpsrv/browse.go. Caps clamp and say so (limit echoes what was applied); a masked database and a missing one share a byte-identical sentence; a bad ref/table is a plain answer, not the mask. 176 tests, cap test mutation-checked.
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-13T05:27:34Z
is_blocked 0
dependencies 3 rows
id 03f3e80e-f338-5ac8-9e05-18bcd396d161
issue_id sr-ht-dolt-0qf.4
type parent-child
created_at 2026-08-12T23:11:44Z
created_by Eugene Blikh
metadata �{}
thread_id
depends_on_issue_id sr-ht-dolt-0qf
depends_on_wisp_id NULL
depends_on_external NULL
id 2f47f333-ffc0-5034-8f29-e2d752909357
issue_id sr-ht-dolt-0qf.6
type blocks
created_at 2026-08-12T23:12:56Z
created_by Eugene Blikh
metadata �{}
thread_id
depends_on_issue_id sr-ht-dolt-0qf.4
depends_on_wisp_id NULL
depends_on_external NULL
id 4f91155c-b2d3-56ea-881f-6ecdb0c45b70
issue_id sr-ht-dolt-0qf.4
type blocks
created_at 2026-08-12T23:12:55Z
created_by Eugene Blikh
metadata �{}
thread_id
depends_on_issue_id sr-ht-dolt-0qf.3
depends_on_wisp_id NULL
depends_on_external NULL
events 3 rows
id 019ff79a-2fae-7124-8f23-45dfbded2c88
issue_id sr-ht-dolt-0qf.4
event_type created
actor Eugene Blikh
old_value
new_value
comment NULL
created_at 2026-08-12T23:11:44Z
id 019ff997-0f0a-7357-afb2-47557a45d8ae
issue_id sr-ht-dolt-0qf.4
event_type status_changed
actor Eugene Blikh
old_value {"id":"sr-ht-dolt-0qf.4","title":"mcpsrv: generic tools over the browse seam","description":"list_branches, list_tables, read_rows, get_commit_log, get_commit_diff — the browse surface as tools, addressing a database as {owner, name} and defaulting ref to the repository default branch. Caps: read_rows limit\u003c=500 (default 100), get_commit_log limit\u003c=100 (default 25); a truncated result says so and carries the true total.","design":"docs/DESIGN.mcp.md §9.1, §9.3. Visibility is core.Allowed/OpBrowse with NotFoundForPrivate masking — never a distinguishable forbidden.","acceptance_criteria":"Visibility matrix per tool over PUBLIC/UNLISTED/PRIVATE x anonymous/stranger/grantee/owner; cap tests.","status":"open","priority":2,"issue_type":"task","owner":"bigbes@gmail.com","created_at":"2026-08-12T20:11:45Z","created_by":"Eugene Blikh","updated_at":"2026-08-12T20:11:45Z"}
new_value {"status":"in_progress"}
comment NULL
created_at 2026-08-13T08:27:34Z
id 019ff9a8-970a-7101-bbfa-63ea6d3a7c0a
issue_id sr-ht-dolt-0qf.4
event_type closed
actor Eugene Blikh
old_value
new_value Landed in 7a77409: list_branches, list_tables, read_rows, get_commit_log, get_commit_diff in mcpsrv/browse.go. Caps clamp and say so (limit echoes what was applied); a masked database and a missing one share a byte-identical sentence; a bad ref/table is a plain answer, not the mask. 176 tests, cap test mutation-checked.
comment NULL
created_at 2026-08-13T08:46:43Z