start-command 0.20.3 → 0.20.4

package/CHANGELOG.md

@@ -1,5 +1,35 @@
 # start-command
 
+## 0.20.4
+
+### Patch Changes
+
+- 688f1f5: fix: Add empty line before result marker for visual continuity
+  - Added empty line before result marker (✓/✗) after command output
+  - Ensures consistent visual formatting: command → empty line → output → empty line → result marker
+  - Applied to both docker pull virtual commands and user commands
+  - Tests document the expected visual format
+
+  Expected format:
+
+  ```
+  │
+  $ docker pull alpine:latest
+
+  latest: Pulling from library/alpine
+  ...
+
+  ✓
+  │
+  $ echo hi
+
+  hi
+
+  ✓
+  ```
+
+  Fixes #73
+
 ## 0.20.3
 
 ### Patch Changes

package/package.json

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

package/src/lib/docker-utils.js

@@ -142,7 +142,9 @@ function dockerPullImage(image) {
     success = false;
   }
 
-  // Print result marker and separator (no empty line needed - already printed after command)
+  // Print empty line before result marker for visual separation (issue #73)
+  // This ensures output is visually separated from the result marker
+  console.log();
   console.log(createVirtualCommandResult(success));
   console.log(createTimelineSeparator());
 

package/test/output-blocks.test.js

@@ -533,13 +533,41 @@ describe('output-blocks module', () => {
       // $ docker pull alpine:latest
       // <empty line>
       // <docker output>
+      // <empty line>
       // ✓
       // │
       // $ echo hi
       // <empty line>
       // hi
+      // <empty line>
       // ✓
     });
+
+    it('output formatting follows visual continuity pattern', () => {
+      // Issue #73: All commands should have consistent formatting:
+      // 1. Command line ($ ...)
+      // 2. Empty line (visual separation)
+      // 3. Command output
+      // 4. Empty line (visual separation)
+      // 5. Result marker (✓ or ✗)
+      //
+      // This test documents the expected output structure that
+      // dockerPullImage and runInDocker should produce.
+
+      // Verify createCommandLine produces the correct format
+      const commandLine = createCommandLine('docker pull alpine:latest');
+      expect(commandLine).toBe('$ docker pull alpine:latest');
+
+      // Verify createVirtualCommandBlock matches
+      const virtualCommandLine = createVirtualCommandBlock(
+        'docker pull alpine:latest'
+      );
+      expect(virtualCommandLine).toBe(commandLine);
+
+      // Verify result markers
+      expect(createVirtualCommandResult(true)).toBe('✓');
+      expect(createVirtualCommandResult(false)).toBe('✗');
+    });
   });
 });