Auto-spawn control plane and notify user when new zellij session is created #219
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
Change zellij.EnsureSession() to return (bool, error) where the boolean indicates whether a new session was created (true) or an existing session was reused (false). This enables callers to take additional actions when a session is first created, such as spawning the control plane. Updated all call sites to handle the new return value. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Create a higher-level InitializeSession function that: 1. Ensures a zellij session exists using EnsureSession 2. Automatically spawns the control plane when a new session is created 3. Returns SessionInitResult with session creation info for display This consolidates the session + control plane initialization logic. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
When a new zellij session is created (via InitializeSession), display a notification to help users attach to the session: CLI (co work create): - Shows a prominent box with the session name and attach command - Works for both regular and --auto workflows TUI: - Extended planWorkCreatedMsg with sessionCreated and sessionName fields - Status bar shows attach command when new session is created Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
…pawn-a-new-zellij-session
newhook
commented
Jan 27, 2026
newhook
commented
Jan 27, 2026
Moved the InitializeSession call before the if flagAutoRun block, eliminating the duplicate code that existed both inside and outside the conditional branch. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
The err variable is already in scope from the earlier CollectIssueIDsForAutomatedWorkflow call and gets reassigned later by CreateWorkAsync, so using a separate sessionErr variable was unnecessary. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
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.
Summary
When the orchestrator spawns a new zellij session, this change automatically:
Previously, control plane spawning and user notification were inconsistent when a new zellij session was created.
Changes
internal/zellij/zellij.go: ModifiedEnsureSessionto return a boolean indicating whether a new session was createdinternal/control/spawn.go: AddedInitializeSessionhelper function that consolidates session + control plane initializationcmd/work.go:printSessionCreatedNotificationto display attach instructionsInitializeSessioninrunWorkCreatefor both--autoand regular pathsinternal/tui/tui_plan.go: ExtendedplanWorkCreatedMsgto carry session creation statusinternal/tui/tui_plan_work.go: IntegratedInitializeSessionand updated status message to show attach command when new session createdinternal/claude/runner.go: Updated allEnsureSessioncall sites to use new signature (ignoring return value where appropriate)Issues Resolved
ac-yssc: zellij: when we spawn a new zellij sessionac-yssc.1: Modify EnsureSession to return created flagac-yssc.2: Create session initialization helper with control plane spawnac-yssc.3: Display user notification when new zellij session is spawnedac-yssc.4: Update call sites to use new session initializationTesting
go buildcompletes successfullygo test ./...passes🤖 Generated with Claude Code