Commit Graph

9 Commits

Author SHA1 Message Date
2653496530 docs: update Phase 3 implementation plan to v1.3
Applied all fixes from critical review v3:
- Issue 3.1: Added input validation to GetVersionIds
- Issue 3.2: Removed redundant WriteLock in Rollback
- Issue 3.3: Added audit entry for prior versions in DeleteVersion
- Issue 3.4: Fixed Rollback return type (compilation fix)
- Issue 3.6: Added clarifying comment for cancellation behavior

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-12-23 03:06:17 +00:00
672f7aab9b docs: add Phase 1 implementation summary
🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-12-23 00:27:26 +00:00
586ae51ccb docs: add Phase 2 implementation plan and review documents
- Implementation plan with 10 tasks
- 4 critical review iterations
- Completion summary

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-12-23 00:26:05 +00:00
4bb1b24f92 docs: mark Phase 2 complete in design document
🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-12-23 00:12:08 +00:00
d78238b247 docs: mark Phase 1 complete in design document
Phase 1 (CRUD Service) successfully implemented:
- ContentServiceBase abstract class created
- IContentCrudService interface defined (21 methods)
- ContentCrudService implementation complete (~750 lines)
- ContentService facade updated to delegate (reduced from 3823 to 3497 lines)
- Benchmark regression enforcement with 20% threshold
- All 16 integration tests + 8 unit tests passing
- Git tag: phase-1-crud-extraction

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-12-21 03:51:01 +00:00
9962df50ee feat(core): register IContentCrudService in DI container
Adds IContentCrudService registration to UmbracoBuilder alongside
IContentService. Both services are now resolvable from DI.

Includes integration test verifying successful resolution.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-12-21 03:31:23 +00:00
a079c44afb chore: capture Phase 0 baseline benchmarks
Records baseline performance metrics for ContentService operations
before refactoring begins. This data will be used to detect
regressions during subsequent phases.

Captured 32 benchmarks covering core operations:
- CRUD operations (Save, Delete, Copy, Move)
- Publishing (Publish, Unpublish, PublishBranch)
- Queries (GetById, GetByIds, GetAncestors, etc.)
- Bulk operations (batch saves, recycle bin)
- Performance-critical paths (1000-item operations)

Note: 4 benchmarks failed due to test setup issues but still
captured timing data. 1 benchmark (Rollback_ToVersion) did not
produce data due to version creation failure.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-12-20 19:50:19 +00:00
bf054e9d62 docs: add performance benchmarks to ContentService refactor design
- Added revision 1.5 with 33 performance benchmarks for baseline comparison
- Benchmarks cover CRUD (7), Query (6), Publish (7), Move (8), Version (4) operations
- Added baseline comparison infrastructure with JSON output format
- Includes execution commands and sample comparison output

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-12-20 01:36:07 +00:00
f4a01ed50d docs: add ContentService refactoring design plan
Design document for refactoring ContentService (~3800 lines) into:
- 3 public service interfaces (CRUD, Publishing, Move)
- 4 internal helper classes (Versioning, Query, Permission, Blueprint)
- Thin facade for backward compatibility

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-12-19 22:54:18 +00:00