fix: add bindings comment regression test (#23458) (thanks @echoVic)

This commit is contained in:
Peter Steinberger
2026-02-22 11:46:34 +01:00
parent 56f01bc493
commit 812bf7c8e1
2 changed files with 11 additions and 0 deletions

View File

@@ -94,6 +94,7 @@ Docs: https://docs.openclaw.ai
- Cron: persist `delivered` state in cron job records so delivery failures remain visible in status and logs. (#19174) Thanks @simonemacario.
- Config/Doctor: only repair the OAuth credentials directory when affected channels are configured, avoiding fresh-install noise.
- Config/Channels: whitelist `channels.modelByChannel` in config validation and exclude it from plugin auto-enable channel detection so model overrides no longer trigger `unknown channel id` validation errors or bogus `modelByChannel` plugin enables. (#23412) Thanks @ProspectOre.
- Config/Bindings: allow optional `bindings[].comment` in strict config validation so annotated binding entries no longer fail load. (#23458) Thanks @echoVic.
- Usage/Pricing: correct MiniMax M2.5 pricing defaults to fix inflated cost reporting. (#22755) Thanks @miloudbelarebia.
- Gateway/Daemon: verify gateway health after daemon restart.
- Agents/UI text: stop rewriting normal assistant billing/payment language outside explicit error contexts. (#17834) Thanks @niceysam.

View File

@@ -363,6 +363,16 @@ describe("legacy config detection", () => {
expectedValue: "work",
});
});
it("accepts bindings[].comment on load", () => {
expectValidConfigValue({
config: {
bindings: [{ agentId: "main", comment: "primary route", match: { channel: "telegram" } }],
},
readValue: (config) =>
(config as { bindings?: Array<{ comment?: string }> }).bindings?.[0]?.comment,
expectedValue: "primary route",
});
});
it("rejects session.sendPolicy.rules[].match.provider on load", async () => {
await withTempHome(async (home) => {
const configPath = path.join(home, ".openclaw", "openclaw.json");