From 47118dcbf23b1674afbfe8b0167f38f6ca7c0ea5 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Carlos=20Mestre=20Cebri=C3=A1n?= Date: Wed, 15 Jul 2026 07:52:07 +0200 Subject: [PATCH] docs(readme): make install loops failure-isolated per iteration (#157) A failed iteration in the Quick start install loops (e.g. bun missing from PATH) skipped the cd back to the repo root, so every remaining tool's cd failed in cascade and the shell ended up stranded inside .agents/skills//cli with nothing else installed. Run each bash iteration in a subshell and use Push-Location/Pop-Location in PowerShell so a failure stays contained to its own tool and the loop always returns to the repo root. Claude-Session: https://claude.ai/code/session_015EQ2xeixvVdnvbihce3aSt Co-authored-by: Claude Fable 5 --- README.md | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/README.md b/README.md index 1411d79..594520d 100644 --- a/README.md +++ b/README.md @@ -83,9 +83,9 @@ PowerShell: ```powershell $tools = @("jobbank-search", "jobdanmark-search", "jobindex-search", "jobnet-search", "linkedin-search", "freehire-search") foreach ($tool in $tools) { - Set-Location ".agents/skills/$tool/cli" + Push-Location ".agents/skills/$tool/cli" bun install - Set-Location "..\..\..\.." + Pop-Location } ``` @@ -93,7 +93,7 @@ Bash / zsh / Git Bash: ```bash for tool in jobbank-search jobdanmark-search jobindex-search jobnet-search linkedin-search freehire-search; do - cd .agents/skills/$tool/cli && bun install && cd ../../../.. + (cd .agents/skills/$tool/cli && bun install) done ```