Skip to content

Chore: Polish Renderer Comments#168

Merged
jlukic merged 1 commit intomainfrom
chore/polish-renderer-comments
Apr 30, 2026
Merged

Chore: Polish Renderer Comments#168
jlukic merged 1 commit intomainfrom
chore/polish-renderer-comments

Conversation

@jlukic
Copy link
Copy Markdown
Member

@jlukic jlukic commented Apr 30, 2026

Comment cleanup pass on packages/renderer/. Strip internal plan references and one-off profile numbers, drop a temporary cache-hit diagnostic, restore intent/insight notes at function sites that need them.

Changes

  • Remove __cbhCache global + hit/miss counters from the native renderer
  • Strip Plan-XX references and ai/plans / ai/workspace paths
  • Drop file-line breadcrumbs and ad-hoc profile millisecond figures
  • Restore load-bearing invariant notes (server/client getItemID sync, item-data proxy fallthrough, Phase 3 notify() rationale) at the sites that need them

A follow-up will add a hydration bench fixture so the cache header can cite a verifiable workload number.

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 30, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
semantic-next Ready Ready Preview, Comment Apr 30, 2026 7:08pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
mcp Ignored Ignored Apr 30, 2026 7:08pm

Request Review

@jlukic jlukic merged commit 9d35359 into main Apr 30, 2026
12 checks passed
@jlukic jlukic deleted the chore/polish-renderer-comments branch April 30, 2026 19:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant