From 2af8f080bd0e0fadbb2b3532c77a86ec6231bc87 Mon Sep 17 00:00:00 2001
From: malkuthro <166633396+malkuthro@users.noreply.github.com>
Date: Mon, 18 May 2026 13:03:16 +0200
Subject: [PATCH] Potential fix for code scanning alert no. 1:
 Prototype-polluting assignment

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
---
 web/sidebar/workflows_store.js | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/web/sidebar/workflows_store.js b/web/sidebar/workflows_store.js
index 472084d..3796860 100644
--- a/web/sidebar/workflows_store.js
+++ b/web/sidebar/workflows_store.js
@@ -37,6 +37,10 @@ import {
 
 let workflowsCache = { directories: {} };
 
+function isSafeObjectKey(name) {
+    return name !== "__proto__" && name !== "constructor" && name !== "prototype";
+}
+
 function notifyWorkflowsChanged() {
     window.dispatchEvent(new CustomEvent(WORKFLOWS_CHANGED_EVENT));
 }
@@ -609,6 +613,7 @@ export function addDirectory(parentPath, name) {
 export function renameDirectory(parentPath, oldName, newName) {
     newName = (newName || "").trim();
     if (!newName || newName === oldName) return false;
+    if (!isSafeObjectKey(oldName) || !isSafeObjectKey(newName)) return false;
     if (parentPath.length > 0 && newName.toLowerCase() === ARCHIVE_RESERVED_NAME) return false;
     const parent = dirOf(parentPath);
     if (!parent || !parent.directories || !parent.directories[oldName]) return false;
@@ -619,6 +624,7 @@ export function renameDirectory(parentPath, oldName, newName) {
 }
 
 export function deleteDirectory(parentPath, name) {
+    if (!isSafeObjectKey(name)) return false;
     const parent = dirOf(parentPath);
     if (!parent || !parent.directories || !parent.directories[name]) return false;
     delete parent.directories[name];
@@ -629,6 +635,7 @@ export function deleteDirectory(parentPath, name) {
 // Workflow operations (path-addressed)
 // =============================================================================
 export function saveWorkflowEntry(path, wfName, graphData, options = {}) {
+    if (!isSafeObjectKey(wfName)) return false;
     const dir = ensureDirectoryAtPath(path);
     if (!dir) return false;
     let archivedAs = null;
@@ -655,6 +662,7 @@ export function saveWorkflowEntry(path, wfName, graphData, options = {}) {
 }
 
 export function archiveWorkflow(path, wfName) {
+    if (!isSafeObjectKey(wfName)) return false;
     const dir = dirOf(path);
     if (!dir || !dir.workflows[wfName]) return false;
     dir.workflows[wfName].archived = true;
@@ -662,6 +670,7 @@ export function archiveWorkflow(path, wfName) {
 }
 
 export function unarchiveWorkflow(path, wfName) {
+    if (!isSafeObjectKey(wfName)) return false;
     const dir = dirOf(path);
     if (!dir || !dir.workflows[wfName]) return false;
     delete dir.workflows[wfName].archived;
@@ -671,6 +680,7 @@ export function unarchiveWorkflow(path, wfName) {
 export function renameWorkflow(path, oldWfName, newWfName) {
     newWfName = (newWfName || "").trim();
     if (!newWfName || newWfName === oldWfName) return false;
+    if (!isSafeObjectKey(oldWfName) || !isSafeObjectKey(newWfName)) return false;
     const dir = dirOf(path);
     if (!dir || !dir.workflows[oldWfName]) return false;
     if (dir.workflows[newWfName]) return false;