[pull] main from commontoolsinc:main #149
Merged
Conversation
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
The shell debugger was causing context window blowout by serializing entire cell values, function objects, and large arrays into the DOM. This was particularly problematic because: 1. Cell update events include full before/after values 2. Scheduler events include entire function definitions with metadata 3. Search filtering was JSON.stringifying every marker on every keystroke 4. Expanded event views had no size limits Changes: - Add safeJsonStringify() helper with configurable max length - Add truncateValue() helper that recursively limits: - Max object depth to 3 levels - Max array items to 5 (with "N more items" indicator) - Max string length to 200 chars - Max object keys to 20 (shows "[Object with N keys]" for larger) - Functions replaced with "[Function: name]" - Apply truncation to: - Event detail rendering (100 char limit) - Search filtering (5000 char limit) - Expanded JSON view (10000 char limit with depth truncation) The debugger now shows useful summaries without blowing out context. NOTE: To see full untruncated data for any event, use the "Copy" button which copies the complete JSON to clipboard. Paste into any text editor to inspect the full values. 🤖 Generated with [Claude Code](https://claude.ai/code) via [Happy](https://happy.engineering) Co-authored-by: Claude <noreply@anthropic.com> Co-authored-by: Happy <yesreply@happy.engineering>
…ler (#2225) chore: Sync a cell from CharmsController when creating a CharmController. Displays synced cells in CLI when inspecting
… failures (#2229) chore: Disable ct-list, ct-tags integration tests due to intermittent failures. Re-enable once solved, or reimagine as a lighterweight test to test the web components.
feat: Improve scrabble pattern UI layout and interactions - Center board, rack, header and instructions with consistent 540px max-width - Move message display into rack bottom bar with styled text - Move New Game button to header, style with green theme - Style Clear Board button with brown theme matching rack - Change default message to "Click New Game to begin" - Add dropZoneRect to ct-drop-zone events for position calculations 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-authored-by: Tony Espinoza <tonyespinoza@Tonys-MacBook-Pro.local> Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
3 participants
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.
See Commits and Changes for more details.
Created by
pull[bot] (v2.0.0-alpha.4)
Can you help keep this open source service alive? 💖 Please sponsor : )