fix: remove READ_ONLY_MODE from system prompt to prevent stale mid-session metadata

Read-only mode is enforced via CLI args (--read-only, YOLO flag filtering)
which update correctly per-message. The system prompt value was frozen at
session start, causing agents to refuse writes after toggling read-only off.

Author: pedramamini

src/prompts/maestro-system-prompt.md

@@ -55,7 +55,6 @@ When you need detailed guidance on a Maestro feature, fetch the relevant documen
 - **Git Branch:** {{GIT_BRANCH}}
 - **Session ID:** {{AGENT_SESSION_ID}}
 - **History File:** {{AGENT_HISTORY_PATH}}
-- **Read-Only Mode:** {{READ_ONLY_MODE}}
 
 ## Task Recall
 
@@ -212,17 +211,6 @@ If a user requests an operation that would write outside your assigned directory
 1. Change to the appropriate session/agent for that directory
 2. Explicitly confirm they want to override this safety measure
 
-### Read-Only / Plan Mode Behavior
-
-**Your current read-only mode status: {{READ_ONLY_MODE}}**
-
-When operating in read-only or plan mode (`{{READ_ONLY_MODE}}` = true), you MUST provide both:
-
-1. Any artifacts you create (documents, plans, specifications)
-2. A clear, detailed summary of your plan in your response to the user
-
-Do not assume the user will read generated files. Always explain your analysis, reasoning, and proposed approach directly in your response.
-
 **Asking questions:** When you need input from the user before proceeding, place ALL questions in a clearly labeled section at the **end** of your response using this exact format:
 
 ---

src/renderer/hooks/agent/useMergeTransferHandlers.ts

@@ -501,7 +501,6 @@ You are taking over this conversation. Based on the context above, provide a bri
 							groupId: targetSession.groupId,
 							activeTabId: newTabId,
 							conductorProfile,
-							readOnlyMode: false,
 						});
 					}
 

src/renderer/hooks/input/useInputProcessing.ts

@@ -1030,7 +1030,6 @@ export function useInputProcessing(deps: UseInputProcessingDeps): UseInputProces
 								activeTabId: freshSession.activeTabId,
 								historyFilePath,
 								conductorProfile,
-								readOnlyMode: isReadOnly,
 							});
 						}
 

src/renderer/stores/agentStore.ts

@@ -321,7 +321,6 @@ export const useAgentStore = create<AgentStore>()((set, get) => ({
 						groupId: session.groupId,
 						activeTabId: targetTab.id,
 						conductorProfile: deps.conductorProfile,
-						readOnlyMode: isReadOnly,
 					});
 				}
 
@@ -417,7 +416,6 @@ export const useAgentStore = create<AgentStore>()((set, get) => ({
 							groupId: session.groupId,
 							activeTabId: targetTab.id,
 							conductorProfile: deps.conductorProfile,
-							readOnlyMode: isReadOnly,
 						});
 					}
 

src/shared/templateVariables.ts

@@ -53,7 +53,6 @@ import { buildSessionDeepLink, buildGroupDeepLink } from './deep-link-urls';
  *
  * Maestro Variables:
  *   {{MAESTRO_CLI_PATH}}  - Platform-appropriate path to maestro-cli
- *   {{READ_ONLY_MODE}}    - "true" if agent is in read-only/plan mode, "false" otherwise
  *
  * Cue Variables (Cue automation only):
  *   {{CUE_EVENT_TYPE}}      - Cue event type (app.startup, time.heartbeat, time.scheduled, file.changed, agent.completed, github.*, task.pending)
@@ -161,8 +160,6 @@ export interface TemplateContext {
 	historyFilePath?: string;
 	// Conductor profile (user's About Me from settings)
 	conductorProfile?: string;
-	// Read-only / plan mode state
-	readOnlyMode?: boolean;
 	// Cue event context (for Cue automation prompts)
 	cue?: {
 		eventType?: string;
@@ -303,7 +300,6 @@ export const TEMPLATE_VARIABLES = [
 	},
 	{ variable: '{{MONTH}}', description: 'Month (01-12)' },
 	{ variable: '{{MAESTRO_CLI_PATH}}', description: 'Path to maestro-cli' },
-	{ variable: '{{READ_ONLY_MODE}}', description: 'Read-only/plan mode (true/false)' },
 	{ variable: '{{TAB_DEEP_LINK}}', description: 'Deep link to agent + active tab (maestro://)' },
 	{ variable: '{{TIME}}', description: 'Time (HH:MM:SS)' },
 	{ variable: '{{TIMESTAMP}}', description: 'Unix timestamp (ms)' },
@@ -334,7 +330,6 @@ export function substituteTemplateVariables(template: string, context: TemplateC
 		documentPath,
 		historyFilePath,
 		conductorProfile,
-		readOnlyMode,
 	} = context;
 	const now = new Date();
 
@@ -403,7 +398,6 @@ export function substituteTemplateVariables(template: string, context: TemplateC
 
 		// Maestro variables
 		MAESTRO_CLI_PATH: getMaestroCLIPath(),
-		READ_ONLY_MODE: String(readOnlyMode ?? false),
 
 		// Cue variables
 		CUE_EVENT_TYPE: context.cue?.eventType || '',