Merge pull request #20 from yysd5/fix/command-execution-delay

Fix/command execution delay

Author: yysd5

Cargo.lock

@@ -1,12 +1,12 @@
 [package]
 name = "tsps"
-version = "0.5.0"
+version = "0.6.0"
 edition = "2024"
 description = "Quickly set up tmux workspaces by splitting windows into multiple panes"
 license = "MIT"
-repository = "https://github.com/yyossy5/tsps"
+repository = "https://github.com/yysd5/tsps"
 authors = ["yyossy5 <hm.hr.yossy@gmail.com>"]
-homepage = "https://github.com/yyossy5/tsps"
+homepage = "https://github.com/yysd5/tsps"
 readme = "README.md"
 keywords = ["tmux", "cli", "terminal", "workspace"]
 categories = ["command-line-utilities"]

Cargo.toml

@@ -8,7 +8,7 @@
 ## Building from source
 
 ```bash
-git clone https://github.com/yyossy5/tsps.git
+git clone https://github.com/yysd5/tsps.git
 cd tsps
 cargo build --release
 ```

DEVELOPMENT.md

@@ -2,7 +2,7 @@
 
 [![Crates.io](https://img.shields.io/crates/v/tsps)](https://crates.io/crates/tsps)
 [![Downloads](https://img.shields.io/crates/d/tsps)](https://crates.io/crates/tsps)
-[![License](https://img.shields.io/crates/l/tsps)](https://github.com/yyossy5/tsps/blob/main/LICENSE)
+[![License](https://img.shields.io/crates/l/tsps)](https://github.com/yysd5/tsps/blob/main/LICENSE)
 
 Quickly set up your tmux workspace by splitting a window into multiple panes at a specified directory with a single command.
 
@@ -34,7 +34,7 @@ cargo install tsps
 
 ```bash
 # Download the latest release
-curl -L https://github.com/yyossy5/tsps/releases/latest/download/tsps-aarch64-apple-darwin -o tsps
+curl -L https://github.com/yysd5/tsps/releases/latest/download/tsps-aarch64-apple-darwin -o tsps
 
 # Make it executable
 chmod +x tsps
@@ -74,6 +74,9 @@ tsps --layout examples/dev.yaml --directory /path/to/different/project
 
 # Short form with directory override
 tsps -l examples/dev.yaml -d /path/to/project
+
+# Adjust the delay before executing commands (in milliseconds, default: 2000)
+tsps -l examples/dev.yaml --delay 3000
 ```
 
 #### Example Layout File
@@ -138,6 +141,18 @@ tsps -l ~/layouts/dev.yaml -d ~/projects/mobile
 
 This is particularly useful when you have standardized development workflows but work on multiple projects.
 
+#### Command Execution Delay
+
+If your environment causes commands to fail or display incorrectly, you can increase the delay before commands are executed:
+
+```bash
+# Wait for 3 seconds instead of the default 2 seconds
+tsps -l ~/layouts/dev.yaml --delay 3000
+```
+
+The default delay is 2000ms (2 seconds), which should be sufficient for most environments.
+Increase this value if you find that commands are being sent before the shell is ready.
+
 #### Size Specification Examples
 
 ```yaml
@@ -176,7 +191,7 @@ cargo install-update tsps
 
 ```bash
 # Download the latest release
-curl -L https://github.com/yyossy5/tsps/releases/latest/download/tsps-aarch64-apple-darwin -o tsps
+curl -L https://github.com/yysd5/tsps/releases/latest/download/tsps-aarch64-apple-darwin -o tsps
 
 # Make it executable
 chmod +x tsps

README.md

@@ -11,12 +11,12 @@ panes:
       - "nvim ."
     focus: true
 
-  # Claude Code (top-right, smaller)
-  - id: "claude"
+  # Gemini CLI (top-right, smaller)
+  - id: "gemini"
     split: "vertical"
     size: "38%"
     commands:
-      - "claude"
+      - "gemini"
 
   # General terminal (bottom-left)
   - id: "terminal"

examples/dev.yaml

@@ -43,7 +43,7 @@ impl LayoutConfig {
     }
 
     /// Apply layout to tmux
-    pub fn apply_to_tmux(&self) -> Result<(), Box<dyn std::error::Error>> {
+    pub fn apply_to_tmux(&self, delay: u64) -> Result<(), Box<dyn std::error::Error>> {
         // Check if directory exists
         let target_dir = Path::new(&self.workspace.directory);
         if !target_dir.exists() {
@@ -69,7 +69,7 @@ impl LayoutConfig {
         self.adjust_pane_sizes()?;
 
         // Execute commands in each pane
-        self.execute_commands()?;
+        self.execute_commands(delay)?;
 
         // Set focus
         self.set_focus()?;
@@ -118,8 +118,8 @@ impl LayoutConfig {
 
     /// Adjust pane sizes after layout is applied
     fn adjust_pane_sizes(&self) -> Result<(), Box<dyn std::error::Error>> {
-        // Wait a moment for layout to settle
-        std::thread::sleep(std::time::Duration::from_millis(100));
+        // Wait for layout to settle
+        std::thread::sleep(std::time::Duration::from_millis(500));
 
         // First pass: adjust individual pane sizes
         for (index, pane) in self.panes.iter().enumerate() {
@@ -208,13 +208,23 @@ impl LayoutConfig {
     }
 
     /// Execute commands in each pane
-    fn execute_commands(&self) -> Result<(), Box<dyn std::error::Error>> {
+    fn execute_commands(&self, delay: u64) -> Result<(), Box<dyn std::error::Error>> {
+        // Wait for all panes to settle after creation and resizing
+        std::thread::sleep(std::time::Duration::from_millis(delay));
+
         for (index, pane) in self.panes.iter().enumerate() {
+            if pane.commands.is_empty() {
+                continue;
+            }
+
             // Select pane
             Command::new("tmux")
                 .args(["select-pane", "-t", &index.to_string()])
                 .status()?;
 
+            // Delay between pane selection and command execution
+            std::thread::sleep(std::time::Duration::from_millis(300));
+
             // Execute commands
             for command in &pane.commands {
                 Command::new("tmux")

src/layout.rs

@@ -19,6 +19,10 @@ struct Cli {
     #[arg(short, long, value_name = "DIRECTORY")]
     directory: Option<PathBuf>,
 
+    /// Delay before executing commands in milliseconds
+    #[arg(short = 'D', long, value_name = "MS", default_value = "2000")]
+    delay: u64,
+
     /// Remaining arguments for traditional usage (pane_count and directory)
     #[arg(trailing_var_arg = true, allow_hyphen_values = true)]
     args: Vec<String>,
@@ -35,7 +39,7 @@ fn main() {
 
     // If layout file is specified
     if let Some(layout_file) = cli.layout {
-        match apply_layout_file(&layout_file, cli.directory.as_ref()) {
+        match apply_layout_file(&layout_file, cli.directory.as_ref(), cli.delay) {
             Ok(()) => return,
             Err(e) => {
                 eprintln!("Error: Failed to apply layout: {}", e);
@@ -125,6 +129,7 @@ fn main() {
 fn apply_layout_file(
     layout_file: &PathBuf,
     override_directory: Option<&PathBuf>,
+    delay: u64,
 ) -> Result<(), Box<dyn std::error::Error>> {
     let mut layout = LayoutConfig::from_file(layout_file)?;
 
@@ -133,6 +138,6 @@ fn apply_layout_file(
         layout.workspace.directory = dir.to_string_lossy().to_string();
     }
 
-    layout.apply_to_tmux()?;
+    layout.apply_to_tmux(delay)?;
     Ok(())
 }