From 4c338535750e88172c5c14a4f5667e253a6e84d7 Mon Sep 17 00:00:00 2001
From: Patrick Roy <roypat@amazon.co.uk>
Date: Wed, 27 Nov 2024 10:24:35 +0000
Subject: [PATCH] fix: ensure ci artifacts are present when using devtool
 sandbox

Without ensure_ci_artifacts called inside cmd_sandbox, you will see an
index out of bounds errors during set up of the argument parser, since
build/img will be an empty directory (e.g. there are no kernels to
select, and the -1 index will be out of bounds).

Reported-by: Jack Thomson <jackabt@amazon.co.uk>
Signed-off-by: Patrick Roy <roypat@amazon.co.uk>
---
 tools/devtool | 1 +
 1 file changed, 1 insertion(+)

diff --git a/tools/devtool b/tools/devtool
index c3b7b27ed98..3e0bc857ed5 100755
--- a/tools/devtool
+++ b/tools/devtool
@@ -890,6 +890,7 @@ cmd_sh() {
 
 cmd_sandbox() {
     cmd_build --release
+    ensure_ci_artifacts
     cmd_sh "tmux new env PYTEST_ADDOPTS=--pdbcls=IPython.terminal.debugger:TerminalPdb PYTHONPATH=tests IPYTHONDIR=\$PWD/.ipython ipython -i ./tools/sandbox.py $@"
 }