Adjust variable editor row layout

This commit is contained in:
2025-10-03 15:59:38 -06:00
parent b3c68932b0
commit aacc7ccec2
4 changed files with 172 additions and 101 deletions

View File

@@ -5,14 +5,6 @@
"category": "script", "category": "script",
"type": "powershell", "type": "powershell",
"script": "# Define the file path\n$filePath = \"C:\\Canary.txt\"\n\n# Write some content into the file\n\"SYSTEM Canary is alive.\"ddss | Out-File -FilePath $filePath -Encoding UTF8\n", "script": "# Define the file path\n$filePath = \"C:\\Canary.txt\"\n\n# Write some content into the file\n\"SYSTEM Canary is alive.\"ddss | Out-File -FilePath $filePath -Encoding UTF8\n",
"script_lines": [
"# Define the file path",
"$filePath = \"C:\\Canary.txt\"",
"",
"# Write some content into the file",
"\"SYSTEM Canary is alive.\"ddss | Out-File -FilePath $filePath -Encoding UTF8",
""
],
"timeout_seconds": 3600, "timeout_seconds": 3600,
"sites": { "sites": {
"mode": "specific", "mode": "specific",

View File

@@ -212,13 +212,11 @@ function fromServerDocument(doc = {}, defaultType = "powershell") {
assembly.description = doc.description || ""; assembly.description = doc.description || "";
assembly.category = doc.category || assembly.category; assembly.category = doc.category || assembly.category;
assembly.type = doc.type || assembly.type; assembly.type = doc.type || assembly.type;
if (Array.isArray(doc.script_lines)) { const legacyScript = Array.isArray(doc.script_lines)
assembly.script = doc.script_lines ? doc.script_lines.map((line) => (line == null ? "" : String(line))).join("\n")
.map((line) => (line == null ? "" : String(line))) : "";
.join("\n"); const script = doc.script ?? doc.content ?? legacyScript;
} else { assembly.script = typeof script === "string" ? script : legacyScript;
assembly.script = doc.script ?? doc.content ?? "";
}
const timeout = doc.timeout_seconds ?? doc.timeout ?? assembly.timeoutSeconds; const timeout = doc.timeout_seconds ?? doc.timeout ?? assembly.timeoutSeconds;
assembly.timeoutSeconds = Number.isFinite(Number(timeout)) assembly.timeoutSeconds = Number.isFinite(Number(timeout))
? Number(timeout) ? Number(timeout)
@@ -238,7 +236,6 @@ function toServerDocument(assembly) {
const normalizedScript = typeof assembly.script === "string" const normalizedScript = typeof assembly.script === "string"
? assembly.script.replace(/\r\n/g, "\n") ? assembly.script.replace(/\r\n/g, "\n")
: ""; : "";
const scriptLines = normalizedScript ? normalizedScript.split("\n") : [];
const timeoutNumeric = Number(assembly.timeoutSeconds); const timeoutNumeric = Number(assembly.timeoutSeconds);
const timeoutSeconds = Number.isFinite(timeoutNumeric) ? Math.max(0, Math.round(timeoutNumeric)) : 3600; const timeoutSeconds = Number.isFinite(timeoutNumeric) ? Math.max(0, Math.round(timeoutNumeric)) : 3600;
return { return {
@@ -248,7 +245,6 @@ function toServerDocument(assembly) {
category: assembly.category || "script", category: assembly.category || "script",
type: assembly.type || "powershell", type: assembly.type || "powershell",
script: normalizedScript, script: normalizedScript,
script_lines: scriptLines,
timeout_seconds: timeoutSeconds, timeout_seconds: timeoutSeconds,
sites: { sites: {
mode: assembly.sites?.mode === "specific" ? "specific" : "all", mode: assembly.sites?.mode === "specific" ? "specific" : "all",
@@ -728,12 +724,16 @@ export default function AssemblyEditor({
value={assembly.description} value={assembly.description}
onChange={(e) => updateAssembly({ description: e.target.value })} onChange={(e) => updateAssembly({ description: e.target.value })}
multiline multiline
minRows={3} minRows={2}
maxRows={8}
fullWidth fullWidth
variant="outlined" variant="outlined"
sx={{ sx={{
...INPUT_BASE_SX, ...INPUT_BASE_SX,
"& .MuiOutlinedInput-inputMultiline": { padding: "4px 8px" } "& .MuiOutlinedInput-inputMultiline": {
padding: "6px 12px",
lineHeight: 1.4
}
}} }}
/> />
</Grid> </Grid>
@@ -902,92 +902,177 @@ export default function AssemblyEditor({
key={variable.id} key={variable.id}
sx={{ p: 2, bgcolor: BACKGROUND_COLORS.field, border: "1px solid #2b3544", borderRadius: 1 }} sx={{ p: 2, bgcolor: BACKGROUND_COLORS.field, border: "1px solid #2b3544", borderRadius: 1 }}
> >
<Grid container spacing={2} alignItems="center"> <Box sx={{ display: "flex", flexDirection: "column", gap: { xs: 2, lg: 1.5 } }}>
<Grid item xs={12} md={3}> <Box
<TextField sx={{
label="Variable Name" display: "flex",
value={variable.name} flexWrap: "wrap",
onChange={(e) => updateVariable(variable.id, { name: e.target.value })} alignItems: "center",
fullWidth gap: { xs: 2, lg: 1.5 },
variant="outlined" pt: 0.5
sx={INPUT_BASE_SX} }}
/> >
</Grid> <Box sx={{ flex: { xs: "1 1 100%", lg: "0 1 28%" }, minWidth: { lg: 220 } }}>
<Grid item xs={12} md={3}> <Tooltip
<TextField title="This is the name of the variable you will be referencing inside of the script. Within the script you will reference this variable with a prefixed \"$env:<variable>\". For example, a variable named \"message\" would be written in the script as \"$env:message\"."
label="Display Label" arrow
value={variable.label} placement="top-start"
onChange={(e) => updateVariable(variable.id, { label: e.target.value })} >
fullWidth
variant="outlined"
sx={INPUT_BASE_SX}
/>
</Grid>
<Grid item xs={12} md={2}>
<TextField
select
fullWidth
label="Type"
value={variable.type}
onChange={(e) => updateVariable(variable.id, { type: e.target.value })}
sx={SELECT_BASE_SX}
SelectProps={{ MenuProps: MENU_PROPS }}
>
{VARIABLE_TYPE_OPTIONS.map((opt) => (
<MenuItem key={opt.key} value={opt.key}>{opt.label}</MenuItem>
))}
</TextField>
</Grid>
<Grid item xs={12} md={3}>
{variable.type === "boolean" ? (
<FormControlLabel
control={
<Checkbox
checked={Boolean(variable.defaultValue)}
onChange={(e) => updateVariable(variable.id, { defaultValue: e.target.checked })}
sx={{ color: "#58a6ff" }}
/>
}
label="Default Value"
/>
) : (
<TextField <TextField
label="Default Value" label="Variable"
value={variable.defaultValue ?? ""} value={variable.name}
onChange={(e) => updateVariable(variable.id, { defaultValue: e.target.value })} onChange={(e) => updateVariable(variable.id, { name: e.target.value })}
fullWidth fullWidth
variant="outlined" variant="outlined"
sx={INPUT_BASE_SX} sx={INPUT_BASE_SX}
/> />
</Tooltip>
</Box>
<Box sx={{ flex: { xs: "1 1 100%", lg: "0 1 22%" }, minWidth: { lg: 180 } }}>
<Tooltip
title="This is the name that will be given to the variable and seen by Borealis server operators."
arrow
placement="top-start"
>
<TextField
label="Display Label"
value={variable.label}
onChange={(e) => updateVariable(variable.id, { label: e.target.value })}
fullWidth
variant="outlined"
sx={INPUT_BASE_SX}
/>
</Tooltip>
</Box>
<Box sx={{ flex: { xs: "1 1 100%", lg: "0 1 18%" }, minWidth: { lg: 160 } }}>
<Tooltip
title="This defines the type of variable data the script should expect."
arrow
placement="top-start"
>
<TextField
select
fullWidth
label="Type"
value={variable.type}
onChange={(e) => updateVariable(variable.id, { type: e.target.value })}
sx={SELECT_BASE_SX}
SelectProps={{ MenuProps: MENU_PROPS }}
>
{VARIABLE_TYPE_OPTIONS.map((opt) => (
<MenuItem key={opt.key} value={opt.key}>{opt.label}</MenuItem>
))}
</TextField>
</Tooltip>
</Box>
<Box
sx={{
flex: { xs: "1 1 100%", lg: "0 1 24%" },
minWidth: { lg: 220 },
display: "flex",
alignItems: "center"
}}
>
{variable.type === "boolean" ? (
<Tooltip
title="This is the value that will be pre-populated in the assembly when ran. Use a sensible default value."
arrow
placement="top-start"
>
<FormControlLabel
control={
<Checkbox
checked={Boolean(variable.defaultValue)}
onChange={(e) => updateVariable(variable.id, { defaultValue: e.target.checked })}
sx={{ color: "#58a6ff" }}
/>
}
label="Default Value"
sx={{
color: "#9ba3b4",
m: 0,
"& .MuiFormControlLabel-label": { fontSize: "0.95rem" }
}}
/>
</Tooltip>
) : (
<Tooltip
title="This is the value that will be pre-populated in the assembly when ran. Use a sensible default value."
arrow
placement="top-start"
>
<TextField
label="Default Value"
value={variable.defaultValue ?? ""}
onChange={(e) => updateVariable(variable.id, { defaultValue: e.target.value })}
fullWidth
variant="outlined"
sx={INPUT_BASE_SX}
/>
</Tooltip>
)} )}
</Grid> </Box>
<Grid item xs={12} md={1} sx={{ display: "flex", justifyContent: "center" }}> <Box
<Tooltip title="Required"> sx={{
flex: { xs: "1 1 100%", lg: "0 0 110px" },
minWidth: { lg: 110 },
display: "flex",
alignItems: "center",
justifyContent: "center",
gap: 1
}}
>
<Typography variant="caption" sx={{ color: "#9ba3b4", fontSize: "0.75rem" }}>
Required
</Typography>
<Checkbox <Checkbox
checked={Boolean(variable.required)} checked={Boolean(variable.required)}
onChange={(e) => updateVariable(variable.id, { required: e.target.checked })} onChange={(e) => updateVariable(variable.id, { required: e.target.checked })}
sx={{ color: "#58a6ff" }} sx={{ color: "#58a6ff", p: 0.5 }}
inputProps={{ "aria-label": "Required" }}
/> />
</Tooltip> </Box>
</Grid> <Box
<Grid item xs={12}> sx={{
<TextField flex: { xs: "1 1 100%", lg: "0 0 auto" },
label="Description" display: "flex",
value={variable.description} justifyContent: "flex-end",
onChange={(e) => updateVariable(variable.id, { description: e.target.value })} alignItems: "center",
fullWidth ml: { lg: "auto" }
multiline }}
minRows={2} >
variant="outlined" <Tooltip title="Remove this Variable." arrow>
sx={INPUT_BASE_SX} <IconButton onClick={() => removeVariable(variable.id)} sx={{ color: "#ff6b6b" }}>
/> <DeleteIcon />
</Grid> </IconButton>
<Grid item xs={12} sx={{ display: "flex", justifyContent: "flex-end" }}> </Tooltip>
<IconButton onClick={() => removeVariable(variable.id)} sx={{ color: "#ff6b6b" }}> </Box>
<DeleteIcon /> </Box>
</IconButton> <Box
</Grid> sx={{
</Grid> display: "flex",
flexWrap: "wrap",
gap: { xs: 2, lg: 1.5 }
}}
>
<Box sx={{ flex: { xs: "1 1 100%", lg: "0 1 50%" }, minWidth: { lg: 260 } }}>
<Tooltip
title="Instruct the operator in why this variable exists and how to set it appropriately."
arrow
placement="top-start"
>
<TextField
label="Description"
value={variable.description}
onChange={(e) => updateVariable(variable.id, { description: e.target.value })}
fullWidth
variant="outlined"
sx={INPUT_BASE_SX}
/>
</Tooltip>
</Box>
</Box>
</Box>
</Paper> </Paper>
))} ))}
</Box> </Box>

View File

@@ -182,7 +182,6 @@ class JobScheduler:
"category": "application" if default_type == "ansible" else "script", "category": "application" if default_type == "ansible" else "script",
"type": default_type, "type": default_type,
"script": "", "script": "",
"script_lines": [],
"variables": [], "variables": [],
"files": [], "files": [],
"timeout_seconds": 3600, "timeout_seconds": 3600,
@@ -217,7 +216,6 @@ class JobScheduler:
doc["script"] = content_val doc["script"] = content_val
normalized_script = (doc["script"] or "").replace("\r\n", "\n") normalized_script = (doc["script"] or "").replace("\r\n", "\n")
doc["script"] = normalized_script doc["script"] = normalized_script
doc["script_lines"] = normalized_script.split("\n") if normalized_script else []
try: try:
timeout_raw = data.get("timeout_seconds", data.get("timeout")) timeout_raw = data.get("timeout_seconds", data.get("timeout"))
if timeout_raw is None: if timeout_raw is None:
@@ -271,7 +269,6 @@ class JobScheduler:
content = "" content = ""
normalized_script = (content or "").replace("\r\n", "\n") normalized_script = (content or "").replace("\r\n", "\n")
doc["script"] = normalized_script doc["script"] = normalized_script
doc["script_lines"] = normalized_script.split("\n") if normalized_script else []
return doc return doc
def _ansible_root(self) -> str: def _ansible_root(self) -> str:

View File

@@ -682,7 +682,6 @@ def _empty_assembly_document(default_type: str = "powershell") -> Dict[str, Any]
"category": "application" if (default_type or "").lower() == "ansible" else "script", "category": "application" if (default_type or "").lower() == "ansible" else "script",
"type": default_type or "powershell", "type": default_type or "powershell",
"script": "", "script": "",
"script_lines": [],
"timeout_seconds": 3600, "timeout_seconds": 3600,
"sites": {"mode": "all", "values": []}, "sites": {"mode": "all", "values": []},
"variables": [], "variables": [],
@@ -718,7 +717,6 @@ def _normalize_assembly_document(obj: Any, default_type: str, base_name: str) ->
doc["script"] = content_val doc["script"] = content_val
normalized_script = (doc["script"] or "").replace("\r\n", "\n") normalized_script = (doc["script"] or "").replace("\r\n", "\n")
doc["script"] = normalized_script doc["script"] = normalized_script
doc["script_lines"] = normalized_script.split("\n") if normalized_script else []
timeout_val = obj.get("timeout_seconds", obj.get("timeout")) timeout_val = obj.get("timeout_seconds", obj.get("timeout"))
if timeout_val is not None: if timeout_val is not None:
try: try:
@@ -798,7 +796,6 @@ def _load_assembly_document(abs_path: str, island: str, type_hint: str = "") ->
doc["name"] = base_name doc["name"] = base_name
normalized_script = (content or "").replace("\r\n", "\n") normalized_script = (content or "").replace("\r\n", "\n")
doc["script"] = normalized_script doc["script"] = normalized_script
doc["script_lines"] = normalized_script.split("\n") if normalized_script else []
if default_type == "ansible": if default_type == "ansible":
doc["category"] = "application" doc["category"] = "application"
return doc return doc