Skip to content

POK-89: Deep-merge POST /api/config with validation#29

Merged
saheljalal merged 1 commit into
mainfrom
pok-89-api-config-deep-merge
Jul 3, 2026
Merged

POK-89: Deep-merge POST /api/config with validation#29
saheljalal merged 1 commit into
mainfrom
pok-89-api-config-deep-merge

Conversation

@saheljalal

Copy link
Copy Markdown
Collaborator

Summary

  • POST /api/config now deep-merges partial updates into the loaded config instead of replacing the file wholesale
  • Validates top-level keys (version, agents, settings) and rejects unknown keys or wrong types with HTTP 400
  • Documents merge semantics in docs/aikit.md

Test plan

  • pytest tests/test_aikit_api_config.py (5 tests)
  • Verified partial POST with empty agents object preserves existing agent entries

Closes POK-89

Prevent wholesale config replacement by merging partial updates into the
loaded config and rejecting unknown keys or invalid types.

Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: multica-agent <github@multica.ai>
@saheljalal saheljalal merged commit 556d88e into main Jul 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.

1 participant