main · last commit
18 hours ago ·
8tmadfpi
spec-t2y Remove or implement the stale render-cache claim in specs' doc comments
Lined Up
bd update spec-t2y --claim
bd close spec-t2y
| Created by | Eugene Blikh |
| Owner | bigbes@gmail.com |
| Created | 2026-08-17T20:40:15Z |
| Updated | 2026-08-17T20:40:15Z |
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)
No outgoing dependencies.
Nothing depends on this issue.
| 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 |
| 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 |
No comments.