Skip to content

docs: add Javadoc to all service classes and package-info.java files#31

Merged
tkahng merged 2 commits into
mainfrom
feat/documentation
May 27, 2026
Merged

docs: add Javadoc to all service classes and package-info.java files#31
tkahng merged 2 commits into
mainfrom
feat/documentation

Conversation

@tkahng

@tkahng tkahng commented May 27, 2026

Copy link
Copy Markdown
Owner

Summary

  • Added class-level and public method-level Javadoc to all 57 service classes across 30 domain modules
  • Created ~80 package-info.java files to give module-level orientation for new developers

What's covered

Domain group Services documented
Auth / Account / User AuthService, JwtService, TokenService, EmailService, ImpersonationService, AccountService
Product / Collection / Inventory / Blob ProductService, VariantService, OptionService, CollectionService, InventoryService, BlobService, + more
Cart / Order / Payment / Fulfillment CartService, OrderService, PaymentService, FulfillmentService, ReturnRequestService, + more
Discount / GiftCard / Quote / Shipping DiscountService, GiftCardService, QuoteService, ShippingService, + more
B2B / IAM / Admin CompanyService, PriceListService, CreditAccountService, IamService, AdminUserService, + more
Search / Stats / Webhook / Newsletter / Content / Wishlist / Review / Audit / + more All remaining services

Javadoc approach

  • Class-level: domain responsibility, key collaborators, important invariants
  • Method-level: business purpose and side effects — not a literal re-read of the code
  • Package-level: package-info.java in every package so new developers get oriented before reading individual classes

tkahng added 2 commits May 26, 2026 19:04
Covers all 57 service classes across 30 domain modules. Each service
gets class-level doc (domain responsibility, key collaborators) and
public method-level doc (business purpose, invariants, side effects).

Adds ~80 package-info.java files so new developers get module-level
orientation before diving into individual classes.
@tkahng tkahng enabled auto-merge (squash) May 27, 2026 02:05
@tkahng tkahng merged commit 58b49e6 into main May 27, 2026
3 checks passed
@tkahng tkahng deleted the feat/documentation branch May 29, 2026 20:46
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