~bigbes/sr-ht-spec · parade

main · last commit 18 hours ago · 8tmadfpi

← Back to the parade

spec-t2y Remove or implement the stale render-cache claim in specs' doc comments Lined Up

status: open P3 task
bd update spec-t2y --claim bd close spec-t2y
Created byEugene Blikh
Ownerbigbes@gmail.com
Created2026-08-17T20:40:15Z
Updated2026-08-17T20:40:15Z
Description
What is wrong: specs documents a render cache that does not exist. `service/service.go::Config.Cache` is described (in its own doc comment and referenced from `gitx/read.go::Blob`, `service/read.go::Document.Blob`, `db/store.go` and `core/names.go`) as "the bleve index and the blob-sha-keyed render cache" — but `Service.CacheDir()` has zero non-test callers, and the only actual consumer of `cfg.Cache` is `filepath.Join(cfg.Cache, "index")` for the bleve search index. Four files agree with each other about something that isn't there.

Why it matters: a reader (or a future donor-copy) trusting these comments will believe there is a working blob-sha-keyed render cache and reason about performance or invalidation behaviour that does not exist — the comments are self-reinforcing precisely because four separate files repeat the same claim.

Fix: either implement the described blob-sha-keyed render cache under `Config.Cache`, or (more likely, given `CacheDir()` has no real callers) strip the "render cache" claim from the doc comments in `service/service.go`, `gitx/read.go`, `service/read.go`, `db/store.go` and `core/names.go`, leaving only the accurate description ("the bleve index directory").

Note: confidence reported — this specific item (unlike two other items in the same audit bundle) was not independently re-verified in this pass; confirm `CacheDir()` truly has zero non-test callers before treating the comments as stale rather than the code as unfinished.

Audit: D48 (cross-repo audit, 2026-08-17)

Depends on

No outgoing dependencies.

Depended on by

Nothing depends on this issue.

No comments.

  • Eugene Blikh created the issue · 2026-08-17T23:40:14Z
Stored rows — what this pane was built from, as read
issues 1 row
id spec-t2y
content_hash 625debda0cb70222426b62dcf1097be54a5c4df2fe9164735757b573e6d703d8
title Remove or implement the stale render-cache claim in specs' doc comments
description What is wrong: specs documents a render cache that does not exist. `service/service.go::Config.Cache` is described (in its own doc comment and referenced from `gitx/read.go::Blob`, `service/read.go::Document.Blob`, `db/store.go` and `core/names.go`) as "the bleve index and the blob-sha-keyed render cache" — but `Service.CacheDir()` has zero non-test callers, and the only actual consumer of `cfg.Cache` is `filepath.Join(cfg.Cache, "index")` for the bleve search index. Four files agree with each other about something that isn't there. Why it matters: a reader (or a future donor-copy) trusting these comments will believe there is a working blob-sha-keyed render cache and reason about performance or invalidation behaviour that does not exist — the comments are self-reinforcing precisely because four separate files repeat the same claim. Fix: either implement the described blob-sha-keyed render cache under `Config.Cache`, or (more likely, given `CacheDir()` has no real callers) strip the "render cache" claim from the doc comments in `service/service.go`, `gitx/read.go`, `service/read.go`, `db/store.go` and `core/names.go`, leaving only the accurate description ("the bleve index directory"). Note: confidence reported — this specific item (unlike two other items in the same audit bundle) was not independently re-verified in this pass; confirm `CacheDir()` truly has zero non-test callers before treating the comments as stale rather than the code as unfinished. Audit: D48 (cross-repo audit, 2026-08-17)
design
acceptance_criteria
notes
status open
priority 3
issue_type task
assignee NULL
estimated_minutes NULL
created_at 2026-08-17T20:40:15Z
created_by Eugene Blikh
owner bigbes@gmail.com
updated_at 2026-08-17T20:40:15Z
closed_at NULL
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
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 NULL
is_blocked 0
events 1 row
id 01a01174-11bb-7497-88d8-dca6cbc8fd2b
issue_id spec-t2y
event_type created
actor Eugene Blikh
old_value
new_value
comment NULL
created_at 2026-08-17T23:40:14Z