From 6133721af775ae9c845b95d1363bc96dda505afa Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Mon, 8 Jun 2026 13:35:32 +0000
Subject: [PATCH] =?UTF-8?q?tools:=20remove=20readOnly=20from=20fbq-add-sou?=
 =?UTF-8?q?rce.sh=20=E2=80=94=20not=20a=20valid=20Quantum=20config=20field?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Quantum's YAML parser rejects 'readOnly' under source config with
'unknown field'. Removed the prompt, the written field, and the
display column entirely.

https://claude.ai/code/session_014CCYqVwW6d6f5dw1qRokYt
---
 tools/fbq-add-source.sh | 12 +++---------
 1 file changed, 3 insertions(+), 9 deletions(-)

diff --git a/tools/fbq-add-source.sh b/tools/fbq-add-source.sh
index ce044bb..af49815 100644
--- a/tools/fbq-add-source.sh
+++ b/tools/fbq-add-source.sh
@@ -177,10 +177,6 @@ cmd_add() {
         read -r -p "  Enable for all users by default? [y/N]: " yn
         [[ "${yn,,}" == "y" ]] && default_enabled_bool="true"
 
-        local read_only_bool="false"
-        read -r -p "  Read-only? [y/N]: " read_only_bool_raw
-        [[ "${read_only_bool_raw,,}" == "y" ]] && read_only_bool="true"
-
         backup "$config"
         # Strip inline comments first so yq edits cleanly
         local tmp; tmp=$(mktemp)
@@ -189,8 +185,7 @@ cmd_add() {
             \"path\": \"${container_path}\",
             \"name\": \"${source_name}\",
             \"config\": {
-                \"defaultEnabled\": ${default_enabled_bool},
-                \"readOnly\": ${read_only_bool}
+                \"defaultEnabled\": ${default_enabled_bool}
             }
         }]" "$config"
 
@@ -265,9 +260,8 @@ cmd_show() {
         local name enabled ro
         name=$(yq e ".server.sources[] | select(.path == \"$p\") | .name // \"(unnamed)\"" "$config" 2>/dev/null || echo "?")
         enabled=$(yq e ".server.sources[] | select(.path == \"$p\") | .config.defaultEnabled // false" "$config" 2>/dev/null || echo "false")
-        ro=$(yq e ".server.sources[] | select(.path == \"$p\") | .config.readOnly // false" "$config" 2>/dev/null || echo "false")
-        printf "  %-20s  %-35s  defaultEnabled=%-5s  readOnly=%s\n" \
-            "${name}" "${p}" "${enabled}" "${ro}"
+        printf "  %-20s  %-35s  defaultEnabled=%s\n" \
+            "${name}" "${p}" "${enabled}"
         count=$(( count + 1 ))
     done < <(list_sources "$config")