Skip to content

Conversation

@chr1syy
Copy link

@chr1syy chr1syy commented Feb 1, 2026

Windows Command Line Length Fix

Resolved 'Die Befehlszeile ist zu lang' (command line too long) error on Windows by:

  • Modified inline document generation to use sendPromptViaStdin on Windows
    • Passes prompt via stdin instead of as command line argument
    • Bypasses Windows cmd.exe ~8KB command line length limit
    • Matches approach already used for SSH remote execution
  • Added --input-format stream-json when using stdin with stream-json compatible agents
  • Added logging for prompt length and stdin usage for debugging

OpenCode Agent Support

Extended inline wizard to support OpenCode agent alongside Claude Code and Codex:

  • Added 'opencode' to supported wizard agents list in useInlineWizard hook
  • OpenCode batch mode args handling already present in buildArgsForAgent functions
  • Added SSH-aware availability checking in both conversation and document generation phases

SSH Remote Configuration

Improved SSH remote session handling:

  • Added sessionSshRemoteConfig to DocumentGenerationConfig interface
  • Added sendPromptViaStdin and sendPromptViaStdinRaw to renderer ProcessConfig
  • Pass sessionSshRemoteConfig through entire wizard lifecycle (conversation → document generation)
  • Skip local agent availability checks when executing on remote hosts
  • Use agent type as command fallback for remote-only agents
  • Added logging to distinguish remote execution from local

Remote Agent Availability Fix

Fixed critical bug preventing remote SSH agents from being used:

  • Split agent availability checks to allow null agents for remote sessions
  • For remote sessions: skip both null-check AND availability-check
  • For local sessions: enforce both checks as before
  • Allows remote-only agents (like SSH-configured OpenCode) to work for both conversation and document generation

Test Updates

  • Fixed useInlineWizard.test.ts by adding window.maestro.agents.get mock
  • Mock returns agent info needed for availability checks
  • Added comprehensive test coverage for remote agent scenarios

## Windows Command Line Length Fix

Resolved 'Die Befehlszeile ist zu lang' (command line too long) error on Windows by:
- Modified inline document generation to use sendPromptViaStdin on Windows
  - Passes prompt via stdin instead of as command line argument
  - Bypasses Windows cmd.exe ~8KB command line length limit
  - Matches approach already used for SSH remote execution
- Added --input-format stream-json when using stdin with stream-json compatible agents
- Added logging for prompt length and stdin usage for debugging

## OpenCode Agent Support

Extended inline wizard to support OpenCode agent alongside Claude Code and Codex:
- Added 'opencode' to supported wizard agents list in useInlineWizard hook
- OpenCode batch mode args handling already present in buildArgsForAgent functions
- Added SSH-aware availability checking in both conversation and document generation phases

## SSH Remote Configuration

Improved SSH remote session handling:
- Added sessionSshRemoteConfig to DocumentGenerationConfig interface
- Added sendPromptViaStdin and sendPromptViaStdinRaw to renderer ProcessConfig
- Pass sessionSshRemoteConfig through entire wizard lifecycle (conversation → document generation)
- Skip local agent availability checks when executing on remote hosts
- Use agent type as command fallback for remote-only agents
- Added logging to distinguish remote execution from local

## Remote Agent Availability Fix

Fixed critical bug preventing remote SSH agents from being used:
- Split agent availability checks to allow null agents for remote sessions
- For remote sessions: skip both null-check AND availability-check
- For local sessions: enforce both checks as before
- Allows remote-only agents (like SSH-configured OpenCode) to work for both conversation and document generation

## Test Updates

- Fixed useInlineWizard.test.ts by adding window.maestro.agents.get mock
- Mock returns agent info needed for availability checks
- Added comprehensive test coverage for remote agent scenarios
@pedramamini pedramamini merged commit a164f16 into pedramamini:0.15.0-rc Feb 1, 2026
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants