Skip to content

feat: add prompt cache miss warnings#3671

Merged
rumpl merged 1 commit into
docker:mainfrom
rumpl:feat/cache-miss-warning
Jul 15, 2026
Merged

feat: add prompt cache miss warnings#3671
rumpl merged 1 commit into
docker:mainfrom
rumpl:feat/cache-miss-warning

Conversation

@rumpl

@rumpl rumpl commented Jul 15, 2026

Copy link
Copy Markdown
Member

Summary

  • add an opt-in warn_on_cache_miss user setting
  • expose the setting in the Notifications tab of /settings
  • emit warning notifications when model turns after the first session response report no cached input tokens
  • support both regular model calls and coding harness responses

Validation

  • task build
  • go test ./pkg/runtime ./pkg/tui/page/chat
  • go test ./pkg/tui/dialog ./pkg/tui/page/chat ./pkg/tui ./pkg/userconfig
  • task test ⚠️ unrelated existing failures in the scheduler docs schema snippet and live Moonshot provider test
  • task lint ⚠️ unrelated existing Lint/DeferMutexUnlock offense in pkg/tools/builtin/scheduler/scheduler.go

@rumpl
rumpl requested a review from a team as a code owner July 15, 2026 20:43
@rumpl
rumpl force-pushed the feat/cache-miss-warning branch from 356be28 to 94b62e9 Compare July 15, 2026 20:47
@rumpl
rumpl enabled auto-merge July 15, 2026 20:47
@rumpl
rumpl merged commit c932e5d into docker:main Jul 15, 2026
8 checks passed
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.

2 participants