Skip to content

CMS hardening + refactor foundation#2

Draft
Ultraivanov wants to merge 5 commits intomainfrom
codex/cms-hardening-refactor
Draft

CMS hardening + refactor foundation#2
Ultraivanov wants to merge 5 commits intomainfrom
codex/cms-hardening-refactor

Conversation

@Ultraivanov
Copy link
Copy Markdown
Owner

Why: CMS is currently the highest risk area for content update speed and stability.

This PR:

  • Adds execution plan and test matrix in docs/cms-hardening-refactor-plan.md
  • Starts isolated implementation on a dedicated branch

Next commits planned in this PR:

  1. Server-side validation + path allowlist for CMS APIs
  2. GitHub client hardening (timeouts/retries/error normalization)
  3. Admin decomposition and regression tests
  4. Rendering correctness + content loading refactor

Safety:

  • Draft PR until P0 hardening + tests are complete.

@vercel
Copy link
Copy Markdown

vercel bot commented Apr 13, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
portfolio Ready Ready Preview, Comment Apr 13, 2026 4:38pm

addBlock used .push() and removeBlock assigned directly to the original
section's blocks array instead of creating new objects. This caused React
to miss re-renders — newly added blocks would not appear in the UI.

Now both functions create immutable copies, consistent with updateBlockValue
and moveBlock. Added an immutability regression test.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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