Skip to content

fix: add databasePath alias to plugin schema#241

Merged
jalehman merged 1 commit into
Martian-Engineering:mainfrom
mhprol:fix/config-schema-missing-properties
Apr 3, 2026
Merged

fix: add databasePath alias to plugin schema#241
jalehman merged 1 commit into
Martian-Engineering:mainfrom
mhprol:fix/config-schema-missing-properties

Conversation

@mhprol

@mhprol mhprol commented Apr 2, 2026

Copy link
Copy Markdown
Contributor

What

This PR adds databasePath to openclaw.plugin.json so the manifest schema accepts the runtime-supported alias for dbPath.

Why

Lossless-claw's runtime config loader accepts both dbPath and databasePath, but OpenClaw validates plugin config against openclaw.plugin.json with additionalProperties: false. Without this alias in the schema, a valid runtime config key can still be rejected before startup.

Changes

  • Add databasePath to plugin manifest schema

Testing

  • Parsed openclaw.plugin.json after the schema update
  • Expected outcome: manifest remains valid JSON and OpenClaw accepts databasePath as a config alias

@ifsheldon

Copy link
Copy Markdown

how about newSessionRetainDepth? I added this according to README but I got

Config invalid
File: ~/.openclaw/openclaw.json
Problem:
  - plugins.entries.lossless-claw.config: invalid config: must NOT have additional properties

@jalehman

jalehman commented Apr 2, 2026

Copy link
Copy Markdown
Contributor

how about newSessionRetainDepth? I added this according to README but I got

Config invalid
File: ~/.openclaw/openclaw.json
Problem:
  - plugins.entries.lossless-claw.config: invalid config: must NOT have additional properties

This hasn't been released yet — you'd need to run off of main for this one.

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 jalehman force-pushed the fix/config-schema-missing-properties branch from 50b2e8c to 69cc85b Compare April 3, 2026 19:24
@jalehman

jalehman commented Apr 3, 2026

Copy link
Copy Markdown
Contributor

I missed your earlier PR here, and that was my mistake.

I had already merged overlapping schema-sync work in #252 before I realized you had a similar fix open. Rather than closing this out as redundant, I rebased your branch onto current main and trimmed it down to the one remaining schema/runtime mismatch that was still unique from your branch: accepting databasePath as the manifest alias for dbPath.

I wanted to keep that piece on your PR so you still get contribution credit for it. Thanks for spotting the schema gap.

@jalehman jalehman changed the title fix: add missing config properties to openclaw.plugin.json schema fix: add databasePath alias to plugin schema Apr 3, 2026
@jalehman jalehman merged commit f62ea97 into Martian-Engineering:main Apr 3, 2026
1 check passed
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.

3 participants