fix: TUI tar single-file upload, [d] keybind labels, benchmark metadata nav

Author: jason-rl

src/components/DevboxCreatePage.tsx

@@ -3127,7 +3127,7 @@ export const DevboxCreatePage = ({
                       paddingX={1}
                     >
                       <Text color={colors.textDim} dimColor>
-                        {`${figures.arrowUp}${figures.arrowDown} Navigate • [Enter] ${selectedGatewayIndex === 0 ? "Attach" : selectedGatewayIndex === maxGatewayIndex ? "Done" : "Select"} • [d] Remove • [esc] Back`}
+                        {`${figures.arrowUp}${figures.arrowDown} Navigate • [Enter] ${selectedGatewayIndex === 0 ? "Attach" : selectedGatewayIndex === maxGatewayIndex ? "Done" : "Select"} • [d] Delete • [esc] Back`}
                       </Text>
                     </Box>
                   </>
@@ -3408,7 +3408,7 @@ export const DevboxCreatePage = ({
                       paddingX={1}
                     >
                       <Text color={colors.textDim} dimColor>
-                        {`${figures.arrowUp}${figures.arrowDown} Navigate • [Enter] ${selectedMcpIndex === 0 ? "Attach" : selectedMcpIndex === maxMcpIndex ? "Done" : "Select"} • [d] Remove • [esc] Back`}
+                        {`${figures.arrowUp}${figures.arrowDown} Navigate • [Enter] ${selectedMcpIndex === 0 ? "Attach" : selectedMcpIndex === maxMcpIndex ? "Done" : "Select"} • [d] Delete • [esc] Back`}
                       </Text>
                     </Box>
                   </>
@@ -3564,7 +3564,7 @@ export const DevboxCreatePage = ({
                       <Text color={colors.textDim} dimColor>
                         {editingAgentMountPath
                           ? "Type to edit path • [Enter/esc] Done"
-                          : `${figures.arrowUp}${figures.arrowDown} Navigate • [Enter] Select • [e] Edit path • [d] Remove • [esc] Back`}
+                          : `${figures.arrowUp}${figures.arrowDown} Navigate • [Enter] Select • [e] Edit path • [d] Delete • [esc] Back`}
                       </Text>
                     </Box>
                   </Box>
@@ -3693,7 +3693,7 @@ export const DevboxCreatePage = ({
                       <Text color={colors.textDim} dimColor>
                         {editingObjectMountPath
                           ? "Type to edit path • [Enter/esc] Done"
-                          : `${figures.arrowUp}${figures.arrowDown} Navigate • [Enter] Select • [e] Edit path • [d] Remove • [esc] Back`}
+                          : `${figures.arrowUp}${figures.arrowDown} Navigate • [Enter] Select • [e] Edit path • [d] Delete • [esc] Back`}
                       </Text>
                     </Box>
                   </Box>

src/components/ObjectCreatePage.tsx

@@ -394,7 +394,10 @@ export const ObjectCreatePage = ({
 
         let buffer: Buffer;
 
-        if (paths.length > 1 || isTar) {
+        const singleIsDir =
+          paths.length === 1 && (await lstat(resolvedPaths[0])).isDirectory();
+
+        if (paths.length > 1 || (isTar && singleIsDir)) {
           const isGzip = formData.content_type === "tgz";
           buffer = await createTarBuffer(resolvedPaths, isGzip);
         } else {

src/screens/BenchmarkJobCreateScreen.tsx

@@ -911,68 +911,75 @@ export function BenchmarkJobCreateScreen({
   }, [formData, isFormValid, cloneAgentConfigs, cloneOrchestratorConfig]);
 
   // Handle input
-  useInput((input, key) => {
-    if (screenState !== "form") return;
+  useInput(
+    (input, key) => {
+      if (screenState !== "form") return;
 
-    // Handle source type toggle with left/right arrows
-    if (currentField === "source_type" && (key.leftArrow || key.rightArrow)) {
-      setFormData((prev) => ({
-        ...prev,
-        sourceType: prev.sourceType === "benchmark" ? "scenarios" : "benchmark",
-        // Clear the other source when switching
-        benchmarkId: prev.sourceType === "scenarios" ? "" : prev.benchmarkId,
-        benchmarkName:
-          prev.sourceType === "scenarios" ? "" : prev.benchmarkName,
-        scenarioIds: prev.sourceType === "benchmark" ? [] : prev.scenarioIds,
-        scenarioNames:
-          prev.sourceType === "benchmark" ? [] : prev.scenarioNames,
-      }));
-      return;
-    }
+      // Handle source type toggle with left/right arrows
+      if (currentField === "source_type" && (key.leftArrow || key.rightArrow)) {
+        setFormData((prev) => ({
+          ...prev,
+          sourceType:
+            prev.sourceType === "benchmark" ? "scenarios" : "benchmark",
+          // Clear the other source when switching
+          benchmarkId: prev.sourceType === "scenarios" ? "" : prev.benchmarkId,
+          benchmarkName:
+            prev.sourceType === "scenarios" ? "" : prev.benchmarkName,
+          scenarioIds: prev.sourceType === "benchmark" ? [] : prev.scenarioIds,
+          scenarioNames:
+            prev.sourceType === "benchmark" ? [] : prev.scenarioNames,
+        }));
+        return;
+      }
 
-    // Navigate between fields
-    if (key.upArrow && currentFieldIndex > 0) {
-      setCurrentField(fieldKeys[currentFieldIndex - 1]);
-    } else if (key.downArrow && currentFieldIndex < fieldKeys.length - 1) {
-      setCurrentField(fieldKeys[currentFieldIndex + 1]);
-    } else if (key.escape) {
-      goBack();
-    } else if (key.return) {
-      if (currentFieldDef?.type === "picker" && currentField === "benchmark") {
-        setScreenState("picking_benchmark");
-      } else if (
-        currentFieldDef?.type === "picker" &&
-        currentField === "scenarios"
-      ) {
-        setScreenState("picking_scenarios");
-      } else if (
-        currentFieldDef?.type === "picker" &&
-        currentField === "agents"
-      ) {
-        setScreenState("picking_agents");
-      } else if (
-        currentFieldDef?.type === "picker" &&
-        currentField === "secrets"
-      ) {
-        setScreenState("secrets_config");
-        setSecretsConfigSelectedIndex(0);
-      } else if (
-        currentFieldDef?.type === "metadata" &&
-        currentField === "metadata"
-      ) {
-        setInMetadataSection(true);
-        setSelectedMetadataIndex(0);
-      } else if (
-        currentFieldDef?.type === "action" &&
-        currentField === "create"
-      ) {
-        handleCreate();
-      } else if (currentFieldIndex < fieldKeys.length - 1) {
-        // Move to next field on Enter for text inputs
+      // Navigate between fields
+      if (key.upArrow && currentFieldIndex > 0) {
+        setCurrentField(fieldKeys[currentFieldIndex - 1]);
+      } else if (key.downArrow && currentFieldIndex < fieldKeys.length - 1) {
         setCurrentField(fieldKeys[currentFieldIndex + 1]);
+      } else if (key.escape) {
+        goBack();
+      } else if (key.return) {
+        if (
+          currentFieldDef?.type === "picker" &&
+          currentField === "benchmark"
+        ) {
+          setScreenState("picking_benchmark");
+        } else if (
+          currentFieldDef?.type === "picker" &&
+          currentField === "scenarios"
+        ) {
+          setScreenState("picking_scenarios");
+        } else if (
+          currentFieldDef?.type === "picker" &&
+          currentField === "agents"
+        ) {
+          setScreenState("picking_agents");
+        } else if (
+          currentFieldDef?.type === "picker" &&
+          currentField === "secrets"
+        ) {
+          setScreenState("secrets_config");
+          setSecretsConfigSelectedIndex(0);
+        } else if (
+          currentFieldDef?.type === "metadata" &&
+          currentField === "metadata"
+        ) {
+          setInMetadataSection(true);
+          setSelectedMetadataIndex(0);
+        } else if (
+          currentFieldDef?.type === "action" &&
+          currentField === "create"
+        ) {
+          handleCreate();
+        } else if (currentFieldIndex < fieldKeys.length - 1) {
+          // Move to next field on Enter for text inputs
+          setCurrentField(fieldKeys[currentFieldIndex + 1]);
+        }
       }
-    }
-  });
+    },
+    { isActive: !inMetadataSection },
+  );
 
   useInput(
     (input, key) => {