CI: cat gatekeeper time fix (self-check)#3
Open
lemotw wants to merge 2 commits into
Open
Conversation
Adds a "Compact context" button to the context-usage popover plus a composer-scoped Cmd/Ctrl+Shift+K shortcut. Both call a new POST /api/compact endpoint that runs pi's dedicated `compact` rpc command (session.compact()). Compaction is NOT a chat prompt: pi's rpc prompt path only expands extension/skill/template commands, so sending "/compact" as a message would reach the model as literal text and never compact. The backend adds BuildCompactCommand, piRPCWorker.Compact, Manager.Compact and handleCompact (async; broadcasts a reload once compaction completes). UX: - "Compacting…" banner above the composer + a status label pinned to "compacting" while it runs (guarded against firing while the worker is busy). - updateContextUsage is now compaction-aware: it mirrors pi's estimateContextTokens so the context ring reflects the reduced size after a compaction entry (summary + kept entries, chars/4) instead of stale pre-compaction usage, switching to the measured value on the next turn. Tests: vitest (compact endpoint path, banner show/hide, context estimate vs measured) and Playwright (real compaction marker, banner, no prompt echo) with a stub pi that implements the compact command. Closes ygncode#41
The cat gatekeeper's bedtime "sleep" overlay (default 23:00-07:00) is driven by the live browser Date.now(), so skip-to-break showed the sleepy cat instead of the break whenever CI ran during that window. Use page.clock.setFixedTime() to pin a non-bedtime hour (setFixedTime, not install, so app timers keep running).
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Self-check PR to run the fork's CI on
fix/cat-e2e-time(branched off feat/compact-button).Adds
page.clock.setFixedTime()to make the cat gatekeeper e2e time-independent (no longer fails during the 23:00–07:00 bedtime window). Includes the /compact feature from feat/compact-button.