simulate(tui): in-TUI scenario save (no overwrite) + copy-to-clipboard; drop export-scenarios subcommand#867
Open
theomonnom wants to merge 19 commits into
Open
simulate(tui): in-TUI scenario save (no overwrite) + copy-to-clipboard; drop export-scenarios subcommand#867theomonnom wants to merge 19 commits into
theomonnom wants to merge 19 commits into
Conversation
…erwrite), copy scenario to clipboard, print generated scenarios path on exit
…nto scrollActive (shared by keys + wheel)
…n/jobIconStylePtr/jobIcon) into one jobStatusIcon
- Pre-flight: reuse agentfs.CheckSDKVersion (pinned to 1.6.0, the thin-CLI baseline) in startAgent so start/dev/console/simulate fail fast with a clear 'too old, upgrade to ...' instead of a cryptic subprocess error. - On early exit / failed register/connect, surface the agent's OWN output (the real error) plus the full log path - no string-match guessing of the cause. Wired into simulate (CI + TUI) and console. Verified against staging with a broken .venv (shows the ModuleNotFoundError) and an old declared SDK (shows the upgrade message).
…has no agent description (scenario-file runs)
…kers don't survive and hold the port
…gisters but never joins a room
…ingle no-join timeout (which can be cloud connection pacing)
…ough end of log) so tracebacks aren't truncated
…file and surfaced errors stay plain text
The agent logs in colored format for the TUI; the temp log file kept the raw escape sequences.
The header listed every scenario label (bare bullets when labels are empty) and the job list rendered them all again. Replace it with one line: '<check> Loaded N scenarios from <file> <group name>' — the group name only when set.
'Loaded N scenarios from <file> — <name>' renders as the first (already done) step in file mode, alongside Starting agent / Creating simulation, instead of a separate header line.
Log lines were stripped of ANSI codes at scanner ingest, so the TUI rendered everything colorless. Keep raw lines in memory (bubbletea/ lipgloss render ANSI fine) and strip only where plain text is needed: the log file, fatal-marker matching, surfaced error excerpts, and CI stdout.
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.
No description provided.