Skip to content

Fix CMS clear write validation#7466

Merged
ar2rsawseen merged 10 commits intonewarchitecturefrom
codex/upload-validation-guards
Apr 14, 2026
Merged

Fix CMS clear write validation#7466
ar2rsawseen merged 10 commits intonewarchitecturefrom
codex/upload-validation-guards

Conversation

@ar2rsawseen
Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings April 10, 2026 17:49
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Fixes CMS /i/cms/clear write validation wiring in the API request router and adds an API test to cover saving, reading, and clearing CMS cache entries.

Changes:

  • Corrects the argument order for validateUserForWrite when handling /i/cms/clear.
  • Adds a new API test covering /i/cms/save_entries, /o/cms/entries, and /i/cms/clear.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
api/utils/requestProcessor.js Fixes validateUserForWrite invocation so /i/cms/clear is validated and executed correctly.
test/2.api/08b.cms.js Adds an integration test for CMS save/read/clear flows (currently has response-shape assertion issues).

Comment thread test/2.api/08b.cms.js Outdated
Comment thread test/2.api/08b.cms.js Outdated
@ar2rsawseen ar2rsawseen force-pushed the codex/upload-validation-guards branch from 40f5f23 to 7bca5af Compare April 14, 2026 13:34
@ar2rsawseen ar2rsawseen merged commit 73a97a9 into newarchitecture Apr 14, 2026
15 of 19 checks 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