working settings page logic for all cfg items

libs/comfyui.py

@@ -32,12 +32,9 @@ def get_available_models() -> list:
     response = requests.get(url)
     if response.status_code == 200:
         data = response.json()
-        return (
+        general = data.get("CheckpointLoaderSimple", {}).get("input", {}).get("required", {}).get("ckpt_name", [])[0]
-            data.get("CheckpointLoaderSimple", {})
+        flux = data.get("UnetLoaderGGUF", {}).get("input", {}).get("required", {}).get("unet_name", [])[0]
-            .get("input", {})
+        return general + flux
-            .get("required", {})
-            .get("ckpt_name", [])[0]
-        )
     else:
         print(f"Failed to fetch models: {response.status_code}")
         return []

libs/generic.py

@@ -102,12 +102,15 @@ def get_current_version():
 def load_models_from_config():
     flux_models = load_config()["comfyui:flux"]["models"].split(",")
     sdxl_models = load_config()["comfyui"]["models"].split(",")
-    all_models = flux_models + sdxl_models
+    sorted_flux_models = sorted(flux_models, key=str.lower)
-    return all_models
+    sorted_sdxl_models = sorted(sdxl_models, key=str.lower)
+    return sorted_sdxl_models, sorted_flux_models
+
 
 def load_topics_from_config():
-    topics = load_config()["comfyui"]["topics"].split(", ")
+    topics = load_config()["comfyui"]["topics"].split(",")
-    return topics
+    sorted_topics = sorted(topics, key=str.lower)
+    return sorted_topics
 
 user_config = load_config()
 output_folder = user_config["comfyui"]["output_dir"]

routes/create_routes.py

@@ -1,6 +1,6 @@
 from flask import Blueprint, request, render_template, redirect, url_for, session
 import threading
-from libs.comfyui import create_image, select_model
+from libs.comfyui import create_image, select_model, get_available_models
 from libs.ollama import create_prompt_on_openwebui
 from libs.generic import load_models_from_config, load_topics_from_config
 import os
@@ -21,7 +21,7 @@ def create():
         threading.Thread(target=lambda: create_image(prompt, model)).start()
         return redirect(url_for("create_routes.image_queued", prompt=prompt, model=model.split(".")[0]))
 
-    return render_template("create_image.html", models=load_models_from_config(), topics=load_topics_from_config())
+    return render_template("create_image.html", models=load_models_from_config()[0]+load_models_from_config()[1], topics=load_topics_from_config())
 
 @bp.route("/image_queued")
 def image_queued():
@@ -33,7 +33,7 @@ def image_queued():
 def create_image_page():
     if user_config["frame"]["create_requires_auth"] == "True" and not session.get("authenticated"):
         return redirect(url_for("auth_routes.login", next=request.path))
-    return render_template("create_image.html", models=load_models_from_config(), topics=load_topics_from_config())
+    return render_template("create_image.html", models=load_models_from_config()[0]+load_models_from_config()[1], topics=load_topics_from_config())
 
 
 def init_app(config):

routes/settings_routes.py

@@ -1,40 +1,94 @@
 from flask import Blueprint, render_template, request, redirect, url_for, session
 import configparser
-from libs.generic import load_topics_from_config
+from libs.generic import load_topics_from_config, load_models_from_config
 
 bp = Blueprint('settings_route', __name__)
 CONFIG_PATH = "./user_config.cfg"
 
-def save_items(items):
-    config = configparser.ConfigParser()
-    config.read(CONFIG_PATH)
-
-    # Make sure the section exists
-    if not config.has_section('comfyui'):
-        config.add_section('comfyui')
-
-    # Save updated list to the 'topics' key
-    config.set('comfyui', 'topics', ', '.join(items))
-
-    with open(CONFIG_PATH, 'w') as configfile:
-        config.write(configfile)
-
 @bp.route('/settings', methods=['GET', 'POST'])
 def config_editor():
     if not session.get("authenticated"):
         return redirect(url_for("auth_routes.login", next=request.path))
-    items = load_topics_from_config()  # should return list[str]
+
+    config = configparser.ConfigParser()
+    config.read(CONFIG_PATH)
+
+    # Load from config directly — no helper functions needed anymore
+    topics = config.get('comfyui', 'topics', fallback='').split(',')
+    general_models = config.get('comfyui', 'models', fallback='').split(',')
+    flux_models = config.get('comfyui:flux', 'models', fallback='').split(',')
+
+    topics = [t.strip() for t in topics if t.strip()]
+    general_models = [m.strip() for m in general_models if m.strip()]
+    flux_models = [m.strip() for m in flux_models if m.strip()]
 
     if request.method == 'POST':
         if 'new_topic' in request.form:
-            new_item = request.form.get('new_topic', '').strip()
+            new_topic = request.form.get('new_topic', '').strip()
-            if new_item and new_item not in items:
+            if new_topic and new_topic not in topics:
-                items.append(new_item)
+                topics.append(new_topic)
-        elif 'delete_topic' in request.form:
+
-            to_delete = request.form.getlist('delete_topic')
+        if 'delete_topic' in request.form:
-            items = [item for item in items if item not in to_delete]
+            to_delete = request.form.getlist('delete_topic')
+            topics = [topic for topic in topics if topic not in to_delete]
+
+        if 'new_model' in request.form:
+            new_model = request.form.get('new_model', '').strip()
+            if new_model:
+                if 'flux' in new_model and new_model not in flux_models:
+                    flux_models.append(new_model)
+                elif 'flux' not in new_model and new_model not in general_models:
+                    general_models.append(new_model)
+
+        if 'delete_model' in request.form:
+            to_delete = request.form.getlist('delete_model')
+            general_models = [m for m in general_models if m not in to_delete]
+            flux_models = [m for m in flux_models if m not in to_delete]
+
+        # Save models/topics into the shared config object
+        if not config.has_section('comfyui'):
+            config.add_section('comfyui')
+        if not config.has_section('comfyui:flux'):
+            config.add_section('comfyui:flux')
+
+        config.set('comfyui', 'models', ','.join(general_models))
+        config.set('comfyui:flux', 'models', ','.join(flux_models))
+        config.set('comfyui', 'topics', ','.join(topics))
+
+        # Handle dynamic CFG field updates (excluding DEFAULT and protected keys)
+        for section in config.sections():
+            for key in config[section]:
+                if key == 'models' and section in ('comfyui', 'comfyui:flux'):
+                    continue
+                if key == 'topics' and section == 'comfyui':
+                    continue
+                form_key = f"{section}:{key}"
+                if form_key in request.form:
+                    config[section][key] = request.form[form_key]
+
+        # Save everything at once
+        with open(CONFIG_PATH, 'w') as configfile:
+            config.write(configfile)
 
-        save_items(items)
         return redirect(url_for('settings_route.config_editor'))
 
-    return render_template('settings.html', topics=items)
+    # Prepare filtered config for display
+    filtered_config = {}
+    for section in config.sections():
+        items = {
+            k: v for k, v in config[section].items()
+            if not (
+                (k == 'models' and section in ('comfyui', 'comfyui:flux')) or
+                (k == 'topics' and section == 'comfyui')
+            )
+        }
+        if items:  # only include non-empty sections
+            filtered_config[section] = items
+
+    return render_template(
+        'settings.html',
+        topics=topics,
+        models=general_models + flux_models,
+        config_sections=filtered_config.keys(),
+        config_values=filtered_config
+    )

templates/index.html

@@ -92,10 +92,20 @@
             font-size: 12px;
             font-family: monospace;
             user-select: none;
-            pointer-events: none;
             opacity: 0.6;
         }
 
+        .version a {
+            color: inherit;
+            text-decoration: none;
+            cursor: pointer;
+        }
+
+        .version a:hover {
+            text-decoration: underline;
+        }
+
+
         @media (max-width: 768px) {
             .image-container {
                 max-width: 100vw;
@@ -148,7 +158,9 @@
     {% endif %}
 
     <!-- Version number at bottom right -->
-    <div class="version">v{{ version }}</div>
+    <div class="version">
+        <a href="{{ url_for('settings_route.config_editor') }}">v{{ version }}</a>
+    </div>
 </body>
 
 </html>

templates/settings.html

@@ -147,6 +147,31 @@
       <button name="new_model" type="submit">Add Model</button>
     </form>
   </div>
+  <div class="box" style="width: 100%;">
+    <h2>Config Values</h2>
+    <form method="post" style="display:flex; flex-wrap:wrap; justify-content:space-between;">
+      {% for section in config_sections %}
+      <div class="box">
+        <h2>[{{ section }}]</h2>
+        {% for key, value in config_values[section].items() %}
+        <label>{{ key }}</label>
+        {% if value.lower() in ['true', 'false'] %}
+        <select name="{{ section }}:{{ key }}">
+          <option value="True" {% if value.lower()=='true' %}selected{% endif %}>True</option>
+          <option value="False" {% if value.lower()=='false' %}selected{% endif %}>False</option>
+        </select>
+        {% else %}
+        <input type="text" name="{{ section }}:{{ key }}" value="{{ value }}">
+        {% endif %}
+
+        {% endfor %}
+      </div>
+      {% endfor %}
+      <div class="box" style="width: 100%;">
+        <button type="submit">Save Config</button>
+      </div>
+    </form>
+  </div>
 
   <div class="back-button-wrapper">
     <a href="/" class="button-link">Back to Home</a>