Skip to content

Clarify cache-score read-premium amortization#201

Merged
ictechgy merged 1 commit into
mainfrom
ultragoal/token-savings-batch1-final-cache-fix
Jun 14, 2026
Merged

Clarify cache-score read-premium amortization#201
ictechgy merged 1 commit into
mainfrom
ultragoal/token-savings-batch1-final-cache-fix

Conversation

@ictechgy

Copy link
Copy Markdown
Owner

Summary\n- avoid reporting monotonic break-even reuse counts when cache-read multipliers are above uncached cost\n- add max_profitable_reuses for bounded positive read-premium cases\n- add regression coverage for read-premium positive and negative cases\n\n## Validation\n- python3 scripts/sync_plugin_copies.py --check\n- python3 -m py_compile context-guard-kit/cache_score.py plugins/context-guard/bin/context-guard-cache-score tests/test_context_guard_kit.py\n- PYTHONDONTWRITEBYTECODE=1 python3 -m unittest tests.test_context_guard_kit.ClaudeTokenKitTests -k cache_score\n- PYTHONDONTWRITEBYTECODE=1 python3 scripts/prepublish_check.py --skip-tests\n- python3 scripts/release_smoke.py --timeout 20\n- PYTHONDONTWRITEBYTECODE=1 python3 scripts/prepublish_check.py (691 tests)\n- git diff --check\n\nFollow-up to PR #200 final code-review finding.

@ictechgy ictechgy force-pushed the ultragoal/token-savings-batch1-final-cache-fix branch 4 times, most recently from 04d7797 to 752ef42 Compare June 14, 2026 14:51
@ictechgy ictechgy force-pushed the ultragoal/token-savings-batch1-final-cache-fix branch from 752ef42 to 1a24c85 Compare June 14, 2026 15:13
@ictechgy

Copy link
Copy Markdown
Owner Author

Final ultragoal follow-up evidence for PR #201 (read-premium cache-score amortization)

Reason for follow-up:

Local validation:

  • python3 scripts/sync_plugin_copies.py --check
  • python3 -m py_compile context-guard-kit/cache_score.py plugins/context-guard/bin/context-guard-cache-score tests/test_context_guard_kit.py
  • PYTHONDONTWRITEBYTECODE=1 python3 -m unittest tests.test_context_guard_kit.ClaudeTokenKitTests -k cache_score ✅ (3 tests)
  • PYTHONDONTWRITEBYTECODE=1 python3 scripts/prepublish_check.py --skip-tests
  • python3 scripts/release_smoke.py --timeout 20
  • PYTHONDONTWRITEBYTECODE=1 python3 scripts/prepublish_check.py ✅ (691 tests)
  • git diff --check

Review gates:

  • Code review r6: no actionable correctness regressions (.omx/artifacts/token-savings-batch1-pr201-code-review-r6.md) ✅
  • Architect review r2: architecturalStatus=CLEAR, blockerCount=0 (.omx/artifacts/token-savings-batch1-pr201-architect-review-r2.md) ✅

CI:

Release boundary:

  • No package release/version mutation in this PR; v0.4.10 remains unchanged. Future source release must be 0.4.11+.

@ictechgy ictechgy merged commit b893828 into main Jun 14, 2026
3 checks passed
@ictechgy ictechgy deleted the ultragoal/token-savings-batch1-final-cache-fix branch June 14, 2026 15:20
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