Replace log-update terminal UI with Ink components#149
Open
ddwang wants to merge 9 commits intomagnitudedev:mainfrom
Open
Replace log-update terminal UI with Ink components#149ddwang wants to merge 9 commits intomagnitudedev:mainfrom
ddwang wants to merge 9 commits intomagnitudedev:mainfrom
Conversation
Allows pausing the agent's action loop at clean boundaries (between actions and between batches) for human-in-the-loop review or conditional execution, then resuming without losing state. stop() while paused unblocks the loop to prevent deadlocks.
Migrate the test runner rendering layer from manual ANSI string building with log-update to Ink components. State bridge pattern handles the async mount race condition. No changes to TestRenderer interface or TestStateTracker. Closes magnitudedev#94
Allow the LLM planner to hover over elements to reveal dropdown menus, tooltips, and other hover-triggered content. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Idempotent static method that closes all active browsers, clears the activeBrowsers record, and removes the singleton from globalThis. Safe to call when no browsers exist or when browsers are already closed. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Use Promise.allSettled for concurrent browser closing instead of sequential awaits. Replace per-key deletes with single object reset. Remove unused context variables in tests. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Lets consumers write `import { z } from 'magnitude-core'` instead of reaching into node_modules, avoiding version mismatch and breakage on internal restructuring. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> Summary only read modelUsage[0], missing tokens from additional models. Also memoize groupRegisteredTestsForDisplay and testContextMap since their inputs are stable across re-renders. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Rename package to @ddwang/magnitude-core with prerelease version 0.3.1-ddwang.1 to avoid colliding with upstream. Add scripts/publish.sh for manual npm publishing with dry-run confirmation. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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.
Summary
log-update+ manual ANSI string rendering layer inpackages/magnitude-test/src/term-app/with Ink (React for the terminal)ink,ink-spinner,reactdependencies; removeslog-updateand@types/terminal-kitTestRendererinterface,TestStateTracker, orrunner/state.tsTest plan
bun src/cli.ts examples/tests/example.mag.ts— visual comparison with previous output--plain/--debugmode still uses DebugRenderer (not Ink)showActions/showThoughtsconfig respectedworkerCount > 1— no rendering glitches from concurrent updatesCloses #94