PR #59336: fix: Config Raw mode permanently disabled due to round-trip check regression

• Repository: openclaw/openclaw
• Author: rayseling
• State: open | merged: False
• Link: https://github.com/openclaw/openclaw/pull/59336

Description (problem / solution / changelog)

Summary

Fixes #59330 — Control UI Config editor permanently shows "Raw mode disabled (snapshot cannot safely round-trip raw text)" since 2026.3.31.

Root Cause

Two issues cause shouldFallbackToStructuredRawRedaction() to always return true:

1. normalizeExecSafeBinProfilesInConfig assigns void 0 instead of deleting keys

When safeBinProfiles/safeBinTrustedDirs are empty, the normalize function assigns undefined:

typedExec.safeBinProfiles = ... ? normalizedProfiles : undefined;

This creates keys with undefined values. JSON-parsed objects can never have undefined-valued keys, so isDeepStrictEqual() always fails:

isDeepStrictEqual({ safeBinProfiles: undefined }, {}) // → false

2. shouldFallbackToStructuredRawRedaction compares against materialized config

The round-trip check compares the restored (raw-parsed) config against snapshot.config (the materialized runtime config), which contains transformations not present in the raw source:

• ~/ paths expanded to absolute paths
• Model api field inherited from provider level
• Auto-generated alias fields on model entries
• Default empty config: {} added to plugin entries

The comparison should use snapshot.parsed (the source config before materialization).

Changes

1. src/config/normalize-exec-safe-bin.ts: Use delete instead of assigning undefined when profiles/dirs are empty.

2. src/config/redact-snapshot.raw.ts:

   • Accept optional sourceConfig parameter for comparison target
   • Normalize both sides with JSON.parse(JSON.stringify()) to strip any remaining undefined-valued keys

3. src/config/redact-snapshot.ts: Pass snapshot.parsed as sourceConfig to the round-trip check.

Testing

• Verified locally: redactConfigSnapshot() now returns raw as a string (not null)
• Raw mode is re-enabled in Control UI after these changes
• Confirmed the fix handles all materialization diffs (path expansion, field inheritance, alias generation, default objects)

🤖 AI-assisted (Claude via OpenClaw). Fully tested locally against real config. I understand what the code does.

Changed files

• src/config/normalize-exec-safe-bin.ts (modified, +10/-4)
• src/config/redact-snapshot.raw.ts (modified, +26/-1)
• src/config/redact-snapshot.ts (modified, +1/-0)