Implement actionable mecatui exit handoff - #2047
Merged
Merged
Conversation
Co-authored-by: Mecatl <noreply@mecatl.dev>
Co-authored-by: Mecatl <noreply@mecatl.dev>
Co-authored-by: Mecatl <noreply@mecatl.dev>
Replace duplicate title sanitization with the shared terminal-safe helper and repair the connected-resume guide code span. Co-authored-by: Mecatl <noreply@mecatl.dev>
Co-authored-by: Mecatl <noreply@mecatl.dev>
Co-authored-by: Mecatl <noreply@mecatl.dev>
This branch was successfully deployed
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
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.
Implementation
Closes #1992. Implements the embedded-session exit handoff approved in Plan / Interface PR #2039, approved squash-merge baseline
dc28cce781edd95969770b2b6cd22998b0cb0aef.After a clean embedded exit,
mecatuiretains the byte-exact JSON final-session-ID record and prints a safe, copyable exact resume command with a qualified--resume-latestalternative. When available, the final session snapshot supplies the display title, model-call count, and lifetime main input/output/cache counts. Lookup happens before client close with a one-second bound; output follows terminal teardown and cleanup. Connected-mode human guidance is optional under the plan and is not included; the existing connected ID record remains unchanged. An operator-requested follow-up adds a blank line before the exit handoff and removes the redundant startup notice containing the embedded server's private socket address.Acceptance and verification
runWithOptions/client/server composition, final-vs-startup ID, and malicious-ID/title cases.user-docs/mecatui/sessions.mdguide and built the site.task docs,task test,task lint,task test:race,task site:build,task ac-trace-strict,task api:check, and offlinego run ./cmd/mecademopassed. For commit231278731, focused exit/diagnostics tests,task docs,task site:build, andtask buildpassed. The full test and race suites were not rerun, as requested for fast iteration.PANEL: ship_blockers=0 important=0 advisory=0 reviewer_failures=0.Interfaces match approved contract: Yes for the required handoff. No amendment was needed for that implementation. The blank line and removal of the embedded socket startup notice are additional routine presentation changes requested after the plan merged; the optional connected human command remains deferred.