fix: add default values for optional config fields #3

orion · 2026-02-26T23:50:15Z

orion commented

2026-02-26 23:50:15 +00:00

Add default values for enableDiscordControlTool, enableWhispergatePolicyTool, discordControlApiBaseUrl, enableDebugLogs, debugLogChannelIds to fix issue where whispergateway_tools tool was not exposed due to missing config fields in openclaw.json

orion added 28 commits 2026-02-26 23:50:16 +00:00

docs(test): add phase test report with pending e2e checklist c912ceed79

feat(installer): add openclaw config-set installer with automatic rollback 2f269c25b4

feat(installer): add --install/--uninstall with recorded full rollback 9d752ca090

fix(installer): atomic plugins write and valid json defaults for install flags 6ff9858b18

fix(installer): make uninstall atomic for plugins and pass rollback test c119697f7f

chore(installer): change default no-reply provider/model ids f5ec6f9b15

fix(plugin-path): align packaged dir with plugin id to remove mismatch warning 7f1d6bb3f7

feat(tool): register optional discord_control tool in whispergate plugin and align defaults 0f526346f4

feat(config): add hot-reload config + listMode (human-list/agent-list) 6d463a4572

feat(policy): add per-channel channelPolicies with hot-reload list mode/lists d6f908b813

feat(policy-file): move channel overrides to standalone channelPoliciesFile with hot reload 682d9a336e

feat(policy-runtime): in-memory policy state with whispergate_policy tool and atomic persist e5999743fe

fix(installer): pass CHANNEL_POLICIES_FILE env when resolving policy path 714168e4bf

refactor(installer): replace bash installer logic with node-only implementation 46e56c6760

refactor(tooling): merge discord_control and whispergate_policy into whispergateway_tools ca96779159

fix(rules): inject end-marker prompt for every non-no-reply discord turn 46ea43b3fe

feat(debug): add whispergate hook diagnostics with channel-scoped debug logs 51149dd6a0

fix(hooks): move first-pass decision to before_model_resolve and keep message_received for debug only 8eade446b3

fix(installer): uninstall now selects latest install record instead of latest pointer 1746fb33ad

feat(installer): restart gateway and validate custom no-reply model visibility after install 875cd66d34

fix(installer): write providers map by object key to avoid quoted provider id and restore providers atomically on uninstall 8d34bf257b

fix(installer): raise no-reply model contextWindow/maxTokens to satisfy OpenClaw minimums 15975e3970

fix: remove gateway restart from installer, let user restart manually fd6c4dd3a2

Root cause: installer called 'openclaw gateway restart' (async via systemd)
then immediately validated model visibility — race condition caused validation
to fail and rollback the correct config.

Fix: remove restart + validation from script entirely. Script only writes config.
User restarts gateway manually after install completes.

Also fix CONFIG.example.json: contextWindow 4096->200000, maxTokens 64->8192
(OpenClaw requires minimum 16000 contextWindow).

fix: extract senderId from event.prompt instead of ctx in hooks f33dc13af4

Root cause: PluginHookAgentContext in before_model_resolve only has
agentId, sessionKey, sessionId, workspaceDir, messageProvider.
senderId, channelId, input are NOT available in this hook phase.

The plugin was reading ctx.senderId (undefined) -> inHumanList=false
for ALL Discord sessions -> shouldUseNoReply=true -> all silenced.

Fix: use event.prompt which contains the full user message including
the 'Conversation info (untrusted metadata)' JSON block, and extract
sender_id from there. Same fix applied to before_prompt_build.

Merge pull request 'fix: wait for gateway ready before post-install model validation' (#2 ) from zhi/WhisperGate:feat/whispergate-mvp into feat/whispergate-mvp 6b3d89634a

Reviewed-on: orion/WhisperGate#2

chore(debug): log parsed sender/channel fields in hook decision recompute 211a94233f

fix: restore model after no-reply executes (needsRestore flag) 4622173787

fix: add default values for optional config fields a4836097e4

- Add default values for enableDiscordControlTool, enableWhispergatePolicyTool,
  discordControlApiBaseUrl, enableDebugLogs, debugLogChannelIds
- Merge defaults in both baseConfig and getLivePluginConfig
- Fixes issue where whispergateway_tools tool was not exposed due to missing
  config fields in openclaw.json

hzhang added 6 commits 2026-02-27 15:30:12 +00:00

fix: bypass DM sessions without metadata and make tool globally visible f23d9049a7

1. DM bypass: when neither senderId nor channelId can be extracted from
   the prompt (DM sessions lack untrusted conversation info), skip the
   no-reply gate and allow the message through with end-marker injection.

2. Tool visibility: change whispergateway_tools registration from
   optional=true to optional=false so all agents can see the tool
   without needing explicit tools.allow entries.

fix(installer): resolve plugin path relative to repo instead of hardcoded operator path f74b3978e7

PLUGIN_PATH defaulted to /root/.openclaw/workspace-operator/... regardless
of which workspace the installer was run from. Now resolves relative to
the script location (../dist/whispergate).

fix(rules): strip trailing metadata blocks before checking end symbol 75d659787c

getLastChar was checking the last character of the full event.prompt,
which includes Conversation/Sender metadata blocks appended by OpenClaw
after the actual message. This meant end symbols like 🔚 at the end of
the message body were invisible — the last char was always backtick or
whitespace from the metadata JSON block.

Fix: strip trailing '(untrusted metadata)' blocks before extracting
the last character. This only affects non-humanList senders (humanList
senders bypass end symbol check via human_list_sender reason).

fix: use configured endSymbols in injected prompt and exempt gateway keywords 3749de981f

- buildEndMarkerInstruction() replaces hardcoded END_MARKER_INSTRUCTION,
  dynamically using the resolved policy's endSymbols
- Instruction now explicitly exempts gateway keywords (NO_REPLY, HEARTBEAT_OK)
  from requiring end symbols
- Export resolvePolicy from rules.ts for reuse in before_prompt_build hook

fix(rules): handle multi-byte emoji in getLastChar via Array.from 75f358001b

getLastChar used t[t.length-1] which only gets the trailing surrogate
of emoji like 🔚 (U+1F51A, a surrogate pair in UTF-16). This meant
end symbol matching ALWAYS failed for emoji symbols, causing every
non-humanList message to hit rule_match_no_end_symbol -> no-reply.

Fix: use Array.from(t) to correctly split by Unicode code points.

Merge pull request 'fix: bypass DM sessions without metadata and make tool globally visible' (#4 ) from fix/dm-bypass-and-tool-visibility into feat/whispergate-mvp 9047d3ad8f

Reviewed-on: orion/WhisperGate#4

hzhang merged commit 52a613fdcc into main

2026-02-27 15:31:33 +00:00

hzhang referenced this issue from a commit

2026-02-27 15:31:34 +00:00

Merge pull request 'fix: add default values for optional config fields' (#3) from feat/whispergate-mvp into main

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: nav/Dirigent#3