fix(importers[tmuxinator]): Map pre to on_project_start instead of before_script

before_script calls run_before_script() which uses shlex.split +
Popen(shell=False) — it expects a file path. tmuxinator pre is a raw
shell command (e.g., pre: "mysql.server start"). Imported configs with
raw commands crashed with BeforeLoadScriptNotExists.

on_project_start uses run_hook_commands(shell=True) which handles raw
shell commands correctly, matching tmuxinator's template.erb behavior
where pre is emitted as a raw shell line.

Author: tony

docs/cli/import.md

@@ -84,7 +84,7 @@ $ tmuxp import tmuxinator /path/to/file.json
 
 The tmuxinator importer now supports:
 
-- **Hook mapping** — `pre` maps to `before_script`, `pre_window` maps to `shell_command_before`
+- **Hook mapping** — `pre` maps to `on_project_start`, `pre_window` maps to `shell_command_before`
 - **CLI args** — `cli_args` values (`-f`, `-S`, `-L`) are parsed into tmuxp config equivalents
 - **Synchronize** — `synchronize` window key is converted
 - **Startup focus** — `startup_window` / `startup_pane` convert to `focus: true`

src/tmuxp/workspace/importers.py

@@ -153,21 +153,16 @@ def import_tmuxinator(workspace_dict: dict[str, t.Any]) -> dict[str, t.Any]:
     if "tabs" in workspace_dict:
         workspace_dict["windows"] = workspace_dict.pop("tabs")
 
-    # Handle pre → before_script (independent of pre_window chain)
+    # Handle pre → on_project_start (independent of pre_window chain)
+    # tmuxinator's pre is a raw shell command emitted as a line in a bash script.
+    # on_project_start uses run_hook_commands(shell=True) which handles raw commands.
+    # before_script requires a file path and would crash on raw commands.
     if "pre" in workspace_dict:
         pre_val = workspace_dict["pre"]
         if isinstance(pre_val, list):
-            if (
-                workspace_dict.get("pre_window") is None
-                and workspace_dict.get("pre_tab") is None
-            ):
-                logger.info(
-                    "multi-command pre list mapped to before_script; "
-                    "consider splitting into before_script and shell_command_before",
-                )
-            tmuxp_workspace["before_script"] = "; ".join(pre_val)
+            tmuxp_workspace["on_project_start"] = "; ".join(pre_val)
         else:
-            tmuxp_workspace["before_script"] = pre_val
+            tmuxp_workspace["on_project_start"] = pre_val
 
     # Resolve shell_command_before using tmuxinator's exclusive precedence:
     # rbenv > rvm > pre_tab > pre_window (only ONE is selected)

tests/fixtures/import_tmuxinator/test2.py

@@ -49,7 +49,7 @@
     "socket_name": "foo",
     "config": "~/.tmux.mac.conf",
     "start_directory": "~/test",
-    "before_script": "sudo /etc/rc.d/mysqld start",
+    "on_project_start": "sudo /etc/rc.d/mysqld start",
     "shell_command_before": ["rbenv shell 2.0.0-p247"],
     "windows": [
         {

tests/fixtures/import_tmuxinator/test3.py

@@ -50,7 +50,7 @@
     "socket_name": "foo",
     "start_directory": "~/test",
     "config": "~/.tmux.mac.conf",
-    "before_script": "sudo /etc/rc.d/mysqld start",
+    "on_project_start": "sudo /etc/rc.d/mysqld start",
     "shell_command_before": ["rbenv shell 2.0.0-p247"],
     "windows": [
         {

tests/fixtures/import_tmuxinator/test5.py

@@ -23,7 +23,7 @@
 expected = {
     "session_name": "ruby-app",
     "start_directory": "~/projects/ruby-app",
-    "before_script": "./scripts/bootstrap.sh",
+    "on_project_start": "./scripts/bootstrap.sh",
     "shell_command_before": ["rvm use 2.1.1"],
     "windows": [
         {"window_name": "editor", "panes": ["vim"]},

tests/workspace/test_import_tmuxinator.py

@@ -96,23 +96,6 @@ def test_import_tmuxinator_logs_debug(
     assert getattr(records[0], "tmux_session", None) == "test"
 
 
-def test_logs_info_on_multi_command_pre_list(
-    caplog: pytest.LogCaptureFixture,
-) -> None:
-    """Test that multi-command pre list logs info about before_script mapping."""
-    workspace = {
-        "name": "multi-pre",
-        "root": "~/test",
-        "pre": ["cmd1", "cmd2"],
-        "windows": [{"editor": "vim"}],
-    }
-    with caplog.at_level(logging.INFO, logger="tmuxp.workspace.importers"):
-        importers.import_tmuxinator(workspace)
-
-    pre_records = [r for r in caplog.records if "multi-command pre list" in r.message]
-    assert len(pre_records) == 1
-
-
 def test_startup_window_sets_focus_by_name() -> None:
     """Startup_window sets focus on the matching window by name."""
     workspace = {
@@ -473,15 +456,15 @@ def test_import_tmuxinator_socket_name_same_no_warning(
     assert len(warning_records) == 0
 
 
-def test_import_tmuxinator_pre_list_joined_for_before_script() -> None:
-    """List pre values are joined with '; ' so expand() doesn't crash."""
+def test_import_tmuxinator_pre_list_joined_for_on_project_start() -> None:
+    """List pre values are joined with '; ' for on_project_start."""
     workspace = {
         "name": "pre-list",
         "windows": [{"editor": "vim"}],
         "pre": ["echo one", "echo two"],
     }
     result = importers.import_tmuxinator(workspace)
-    assert result["before_script"] == "echo one; echo two"
+    assert result["on_project_start"] == "echo one; echo two"
 
     # Verify it survives expand() without TypeError
     from tmuxp.workspace import loader
@@ -587,45 +570,45 @@ class PreWindowStandaloneFixture(t.NamedTuple):
     test_id: str
     config_extra: dict[str, t.Any]
     expect_shell_command_before: list[str] | None
-    expect_before_script: str | None
+    expect_on_project_start: str | None
 
 
 PRE_WINDOW_STANDALONE_FIXTURES: list[PreWindowStandaloneFixture] = [
     PreWindowStandaloneFixture(
         test_id="pre_window-only",
         config_extra={"pre_window": "echo PRE"},
         expect_shell_command_before=["echo PRE"],
-        expect_before_script=None,
+        expect_on_project_start=None,
     ),
     PreWindowStandaloneFixture(
         test_id="pre_tab-only",
         config_extra={"pre_tab": "rbenv shell 3.0"},
         expect_shell_command_before=["rbenv shell 3.0"],
-        expect_before_script=None,
+        expect_on_project_start=None,
     ),
     PreWindowStandaloneFixture(
         test_id="pre_window-list",
         config_extra={"pre_window": ["echo a", "echo b"]},
         expect_shell_command_before=["echo a; echo b"],
-        expect_before_script=None,
+        expect_on_project_start=None,
     ),
     PreWindowStandaloneFixture(
         test_id="pre-and-pre_window",
         config_extra={"pre": "sudo start", "pre_window": "echo PRE"},
         expect_shell_command_before=["echo PRE"],
-        expect_before_script="sudo start",
+        expect_on_project_start="sudo start",
     ),
     PreWindowStandaloneFixture(
         test_id="pre-and-pre_window-list",
         config_extra={"pre": "sudo start", "pre_window": ["cd /app", "nvm use 18"]},
         expect_shell_command_before=["cd /app; nvm use 18"],
-        expect_before_script="sudo start",
+        expect_on_project_start="sudo start",
     ),
     PreWindowStandaloneFixture(
         test_id="pre-only",
         config_extra={"pre": "sudo start"},
         expect_shell_command_before=None,
-        expect_before_script="sudo start",
+        expect_on_project_start="sudo start",
     ),
 ]
 
@@ -639,7 +622,7 @@ def test_import_tmuxinator_pre_window_standalone(
     test_id: str,
     config_extra: dict[str, t.Any],
     expect_shell_command_before: list[str] | None,
-    expect_before_script: str | None,
+    expect_on_project_start: str | None,
 ) -> None:
     """pre_window/pre_tab map to shell_command_before independently of pre."""
     workspace: dict[str, t.Any] = {
@@ -654,10 +637,10 @@ def test_import_tmuxinator_pre_window_standalone(
     else:
         assert "shell_command_before" not in result
 
-    if expect_before_script is not None:
-        assert result.get("before_script") == expect_before_script
+    if expect_on_project_start is not None:
+        assert result.get("on_project_start") == expect_on_project_start
     else:
-        assert "before_script" not in result
+        assert "on_project_start" not in result
 
 
 class PreWindowPrecedenceFixture(t.NamedTuple):