fix(perc-system): cms.objectstore serialVersionUID inventory + test companion (#2403)

[natechadwick-intsof]

Summary

#2403 (parent #2022 / grandparent #2200, slice 3e after #2366): clear -Xlint:serial missing-serialVersionUID for remaining cms.objectstore types.

Inventory result (this PR)

Reflection + -Xlint:serial compile of com.percussion.cms.objectstore shows 0 missing serialVersionUID diagnostics. All 20 true Serializable types already declare UIDs from the #2297 foundation batch:

Source of Serializable	Types (examples)
Explicit / PSComponent / PSKey	PSKey, PSSimpleKey, PSItemChildLocator, PSFolder, PSComponentSummary, PSSite, cloning request/options, PSAaRelationship
IPSCatalogSummary → IPSCatalogIdentifier → Serializable	PSAction, PSSearch, PSDisplayFormat, PSItemDefSummary, PSItemDefinition
Exceptions	PSInvalid*, PSRemoteException, PSCorruptDatabaseException
PSObjectPermissions	PSFolderPermissions

Non-Serializable PSDbComponent collection subclasses correctly have no UID (same policy as design.objectstore dead-UID cleanup #994baa0).

Companion test

New PSCmsObjectStoreSerialVersionUidTest:

• Asserts every inventory type is Serializable and declares a non-zero static final serialVersionUID
• Java serialization round-trips for keys, folder permissions, cloning options/request, and exceptions

Out of scope (already filed or residual)

• this-escape → issue 2022 slice 3f: design.objectstore this-escape ctor/fromXml after #2366 #2404
• serial-field on hottest editors / PSItemDefinition fields → issue 2022 slice 3g: serial-field hottest design.objectstore editors after #2366 #2405
• com.percussion.cms.PSCmsException missing UID (parent package, not objectstore) → residual below
• rawtypes → issue 2022 slice 2e: residual cms.objectstore rawtypes (after #2376) #2401

Test plan

• cd system && ../mvnw.cmd clean install — BUILD SUCCESS
• Tests run: 1278, Failures: 0, Errors: 0 (Skipped: 240 baseline)
• Focused: PSCmsObjectStoreSerialVersionUidTest — 4 tests, 0 failures
• No new production warnings introduced by this change (test-only)

Gates evidence

• Erlang-style self-review: inventory-only production delta (none); test covers true Serializable set; portable paths N/A
• Per-module standalone clean install: pass (cd system)
• No monorepo-wide reformat

Residuals (filed with this PR)

• PSCmsException missing serialVersionUID under com.percussion.cms (adjacent package)
• Spring circular dependency [8.2] p1: Spring circular dependency folderHelper→recycle→assetDao→contentItemDao blocks Jetty Rhythmyx startup (QA blocked) #2423 implementation slices (p1 QA blocker residuals)

Operator

Operator: Grok: night-issue-prs (model grok-4.5)

Fixes #2403
Refs #2022 #2200 #2297 #2366

Co-Authored by Grok Build using grok-4.5 with agent main.

[natechadwick-intsof]

Residuals filed with this PR:

Issue	Parent	Priority	Notes
#2434	#2022	p8	PSCmsException missing serialVersionUID (adjacent package)
#2435	#2423	p1	Break folderHelper Spring cycle
#2436	#2423	p1	Spring startup regression test
#2437	#2423	p1	Docker qa-up health/login smoke

Sibling objectstore work remains open: #2404 this-escape, #2405 serial-field.

Co-Authored by Grok Build using grok-4.5 with agent main.

[kilo-code-bot]

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (1 file)

• system/src/test/java/com/percussion/cms/objectstore/PSCmsObjectStoreSerialVersionUidTest.java

---

_{Reviewed by step-3.7-flash · Input: 46.8K · Output: 3.9K · Cached: 188.5K}

_{Review guidance: REVIEW.md from base branch main}