Skip to content

fix: sync plugin config schema#252

Merged
jalehman merged 1 commit into
mainfrom
codex/plugin-schema-sync
Apr 3, 2026
Merged

fix: sync plugin config schema#252
jalehman merged 1 commit into
mainfrom
codex/plugin-schema-sync

Conversation

@jalehman

@jalehman jalehman commented Apr 3, 2026

Copy link
Copy Markdown
Contributor

What

This PR syncs openclaw.plugin.json with the runtime-supported plugin config surface so valid lossless-claw settings are accepted by OpenClaw instead of failing schema validation. It also removes the dead autocompactDisabled option from the advertised config surface and adds release metadata plus regression coverage to keep the manifest from drifting again.

Why

Issue #247 reported that the published docs and runtime supported config keys that the published plugin manifest rejected as additional properties, which could prevent the gateway from starting. The manifest is the schema OpenClaw enforces, so it needs to stay aligned with the actual config loader and documented settings.

Changes

  • Add missing manifest schema entries
  • Add missing manifest UI hints
  • Remove dead autocompactDisabled option
  • Add config-schema regression coverage
  • Add maintainer changeset for release
  • Add repo guidance for schema sync
File Change
openclaw.plugin.json Added missing schema and uiHints entries for supported config keys
src/db/config.ts Removed dead autocompactDisabled parsing from runtime config
README.md Removed unsupported LCM_AUTOCOMPACT_DISABLED documentation
test/config.test.ts Added manifest schema coverage and removed dead option expectations
test/engine.test.ts Removed dead config fixture field
test/expansion.test.ts Removed dead config fixture field
test/lcm-expand-query-tool.test.ts Removed dead config fixture field
test/lcm-expand-tool.test.ts Removed dead config fixture field
test/lcm-tools.test.ts Removed dead config fixture field
test/session-operation-queues.test.ts Removed dead config fixture field
test/summarize.test.ts Removed dead config fixture field
AGENTS.md Added rule to keep manifest and runtime config in sync
.changeset/plugin-config-schema-sync.md Added patch release note for the next publish

Testing

  • npx vitest run test/config.test.ts test/engine.test.ts test/expansion.test.ts test/lcm-expand-query-tool.test.ts test/lcm-expand-tool.test.ts test/lcm-tools.test.ts test/session-operation-queues.test.ts test/summarize.test.ts
  • Expected outcome: all listed Vitest suites pass

Fixes #247

Update the published plugin manifest so runtime-supported config keys are accepted by OpenClaw instead of being rejected as additional properties. Add a patch changeset for the next release, remove the dead autocompactDisabled config option from the advertised surface, and add regression coverage plus repo instructions to keep the manifest in sync with runtime config changes.

Regeneration-Prompt: |
  Prepare the lossless-claw branch for the next release after users reported that documented plugin config keys were rejected by the published openclaw.plugin.json schema. Keep the fix additive and release-ready: sync the manifest uiHints and configSchema with the actual runtime-supported plugin config surface, remove any config option that is parsed but not wired to behavior instead of advertising it, add regression tests so future config drift is caught, add a patch changeset because this affects user-visible package behavior, and update the repo instructions so future config changes also update the manifest.
@jalehman jalehman force-pushed the codex/plugin-schema-sync branch from 63b2b8a to f3d9606 Compare April 3, 2026 17:52
@jalehman jalehman merged commit e843638 into main Apr 3, 2026
2 checks passed
@jalehman jalehman deleted the codex/plugin-schema-sync branch April 3, 2026 17:53
@github-actions github-actions Bot mentioned this pull request Apr 3, 2026
jalehman pushed a commit to mhprol/lossless-claw that referenced this pull request Apr 3, 2026
Keep the plugin manifest schema aligned with the runtime config aliases by accepting `databasePath` alongside `dbPath`. This narrows the original schema-sync contribution down to the one remaining alias that was not already merged through PR Martian-Engineering#252.
jalehman pushed a commit that referenced this pull request Apr 3, 2026
Keep the plugin manifest schema aligned with the runtime config aliases by accepting `databasePath` alongside `dbPath`. This narrows the original schema-sync contribution down to the one remaining alias that was not already merged through PR #252.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

v0.5.3: documented/runtime config keys rejected by plugin manifest schema

1 participant