refactor(ssm): split update_document and list_documents in documents.rs

[vieiralucas]

Summary

• `update_document`: extract `validate_update_document_target_version`, `validate_update_document_unique_content` (the per-version SHA-256 dedupe check), and `validate_update_document_unique_version_name`. The outer function now reads as parse → 3 named validators → mutate → render instead of inlining the validation logic between the lookup and the version push.
• `list_documents`: extract `document_matches_list_filters` (Owner / TargetType / Name filter dispatch — was a 30-line nested for-loop+match inside a closure) and `document_identifier_json` (the `DocumentIdentifier` JSON shape). The outer function is now a 12-line filter+paginate flow.

Test plan

• `cargo fmt`
• `cargo clippy -p fakecloud-ssm --all-targets -- -D warnings`
• `cargo test -p fakecloud-ssm --lib` — 60 passed
• `cargo test --workspace --exclude fakecloud-e2e --exclude fakecloud-conformance`

---

Summary by cubic

Refactored fakecloud-ssm documents service by splitting update_document and list_documents into small helpers for validation, filtering, and JSON shaping. Flows are now parse → validate → mutate → render and filter → paginate, with no behavior changes.

• Refactors
  • update_document: extracted validate_update_document_target_version, validate_update_document_unique_content (SHA-256 dedupe), and validate_update_document_unique_version_name.
  • list_documents: extracted document_matches_list_filters (Owner/TargetType/Name) and document_identifier_json.

^{Written for commit cd6026d. Summary will update on new commits.}

[cubic-dev-ai]

No issues found across 1 file