Library: a shelf holds fifty objects

[manager]

The per-shelf cap goes from 30 to 50, all types counted together. CMS side is keepsimple-cms-new #410. The full-shelf error is matched on its shape rather than its number, so the two sides cannot disagree mid-rollout.

🤖 Generated with Claude Code

[github-actions]

Claude finished @manager's task in 1m 5s —— View job

---

Review: Library shelf cap 30 → 50

• Read repo instructions (CLAUDE.md / AGENTS.md)
• Gather PR diff and changed files
• Review correctness & React best practices
• Review TypeScript quality
• Review project conventions
• Review styling / design tokens
• Post inline comments for issues found (none needed)
• Post final summary

Verdict: LGTM. This is a clean, minimal, low-risk change — the diff is only a constant bump and its accompanying comments.

What I checked:

• src/constants/library/common.ts:84 — MAX_OBJECTS_PER_SHELF bumped 30 → 50. Grepped the repo for any other hardcoded 30/shelf-cap references (source, tests under tests/) and found none — Shelf.tsx is the only consumer, via the constant.
• src/lib/library/shelfFull.ts — the regex /shelf cannot have more than \d+ objects/i already used \d+ before this PR, so no code change was needed there, only the comment. This confirms the PR body's claim: the full-shelf error is matched on message shape, not the number, so frontend/CMS can't disagree mid-rollout regardless of deploy order.
• AddObjectModal.tsx / ObjectOverviewModal.tsx / Shelf.tsx — only comment text updated to reflect the new cap; the underlying logic (isShelfFullError, atObjectLimit) is unchanged and still correct.
• No App Router/Tailwind/state-library/global-CSS violations, no accessibility or security surface touched by this diff.

Gap: I wasn't able to run yarn tsc/yarn lint in this sandbox (Bash command approval was denied for those invocations), but given the change is a numeric constant plus comment-only edits with no new syntax, that risk is minimal.

One dependency to flag per the PR body: this frontend bump should land together with (or after) keepsimple-cms-new #410 per the release process in scripts/release/README.md — the regex tolerance means order doesn't cause user-facing breakage, but confirm CMS side is merged before calling this release-ready.

[manager]

Folded into #215 (commit 01afd6a) so one pull request carries everything the next staging build ships. Closing this one.