start-command 0.20.1 → 0.20.3

package/CHANGELOG.md

@@ -1,5 +1,44 @@
 # start-command
 
+## 0.20.3
+
+### Patch Changes
+
+- 3598f38: Fix visual continuity in docker isolation mode (#73)
+
+  The empty line is now properly placed after the command line (e.g., `$ docker pull alpine:latest`)
+  instead of before it, maintaining consistent visual structure in the timeline output.
+
+  Before:
+
+  ```
+  │
+
+  $ docker pull alpine:latest
+  latest: Pulling from library/alpine
+  ```
+
+  After:
+
+  ```
+  │
+  $ docker pull alpine:latest
+
+  latest: Pulling from library/alpine
+  ```
+
+## 0.20.2
+
+### Patch Changes
+
+- 4749051: fix: Remove empty line after virtual command to maintain visual continuity
+  - Fixed visual continuity break in docker pull output
+  - Removed empty line between timeline marker (`│`) and virtual command line (`$ docker pull`)
+  - Output now flows continuously from timeline marker to command to output
+  - Applies to all virtual command blocks (currently docker pull operations)
+
+  Fixes #73
+
 ## 0.20.1
 
 ### Patch Changes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "start-command",
-  "version": "0.20.1",
+  "version": "0.20.3",
   "description": "Gamification of coding, execute any command with ability to auto-report issues on GitHub",
   "main": "src/bin/cli.js",
   "exports": {

package/src/bin/cli.js

@@ -523,7 +523,10 @@ async function runWithIsolation(
       deferCommand,
     })
   );
-  console.log('');
+  // Only print empty line when not deferring command (docker isolation handles its own spacing)
+  if (!deferCommand) {
+    console.log('');
+  }
 
   // Save initial execution record and set global reference for signal cleanup
   if (executionRecord && store) {

package/src/lib/docker-utils.js

@@ -115,7 +115,7 @@ function dockerPullImage(image) {
     createTimelineSeparator,
   } = require('./output-blocks');
 
-  // Print the virtual command line
+  // Print the virtual command line followed by empty line for visual separation
   console.log(createVirtualCommandBlock(`docker pull ${image}`));
   console.log();
 
@@ -142,8 +142,7 @@ function dockerPullImage(image) {
     success = false;
   }
 
-  // Print result marker and separator
-  console.log();
+  // Print result marker and separator (no empty line needed - already printed after command)
   console.log(createVirtualCommandResult(success));
   console.log(createTimelineSeparator());
 

package/test/output-blocks.test.js

@@ -458,6 +458,88 @@ describe('output-blocks module', () => {
       });
       expect(block).not.toContain('$ echo hello');
     });
+
+    it('should end with empty timeline line when deferCommand is true (issue #73)', () => {
+      // Issue #73: Visual continuity - when deferCommand is true,
+      // the start block should end with │ (empty timeline line)
+      // and no trailing empty line should be added by the CLI
+      const block = createStartBlock({
+        sessionId: 'test-uuid',
+        timestamp: '2025-01-01 00:00:00',
+        command: 'echo hello',
+        extraLines: [
+          '[Isolation] Environment: docker, Mode: attached',
+          '[Isolation] Image: alpine:latest',
+          '[Isolation] Session: docker-container-123',
+        ],
+        deferCommand: true,
+      });
+      const lines = block.split('\n');
+      // Last line should be empty timeline line (│)
+      expect(lines[lines.length - 1]).toBe('│');
+      // Should not contain the command
+      expect(block).not.toContain('$ echo hello');
+    });
+  });
+
+  describe('Visual Continuity (issue #73)', () => {
+    it('virtual command block should be followed by empty line for visual separation', () => {
+      // Issue #73: The empty line should come AFTER the command
+      // ($ docker pull ...), not BEFORE it
+      // This test verifies the expected format: command line followed by
+      // empty line before output
+      const virtualCommand = createVirtualCommandBlock(
+        'docker pull alpine:latest'
+      );
+      expect(virtualCommand).toBe('$ docker pull alpine:latest');
+      // The empty line is added by the caller (dockerPullImage)
+      // after the command and before running docker pull
+    });
+
+    it('timeline separator should be empty timeline line for visual structure', () => {
+      // After virtual command output, we print result marker and separator
+      const separator = createTimelineSeparator();
+      expect(separator).toBe('│');
+    });
+
+    it('start block with docker isolation should end cleanly for virtual commands', () => {
+      // When using docker isolation with defer_command, the block ends with │
+      // The virtual command (docker pull) starts immediately after
+      const block = createStartBlock({
+        sessionId: 'uuid-abc',
+        timestamp: '2026-01-08 12:00:00',
+        command: 'echo hi',
+        extraLines: [
+          '[Isolation] Environment: docker, Mode: attached',
+          '[Isolation] Image: alpine:latest',
+          '[Isolation] Session: docker-1234',
+        ],
+        deferCommand: true,
+      });
+
+      // Expected structure:
+      // │ session   uuid-abc
+      // │ start     2026-01-08 12:00:00
+      // │
+      // │ isolation docker
+      // │ mode      attached
+      // │ image     alpine:latest
+      // │ container docker-1234
+      // │   <-- ends here with empty timeline line
+      const lines = block.split('\n');
+      expect(lines[lines.length - 1]).toBe('│');
+
+      // Next should be virtual command (added separately by docker-utils):
+      // $ docker pull alpine:latest
+      // <empty line>
+      // <docker output>
+      // ✓
+      // │
+      // $ echo hi
+      // <empty line>
+      // hi
+      // ✓
+    });
   });
 });