Refactor TasksService to Business Intent Volume 07 - Advanced Application Architecture | Phase 24
Before A service method that mixes policy, Prisma query construction, mapping, and database error handling becomes difficult to scan.
After async getTaskForUser( taskId: string, currentUser: CurrentUser, ) { const task = await this.tasks.findScopedById( taskId, currentUser.id, )
if (!task) { throw new TaskNotFoundException(taskId) }
return task }
Responsibility split Controller HTTP method, params/body/query, status codes, decorators.
Service Business policy, workflow, invariants, authorization decisions.
Repository Persistence queries, scopes supplied by the service, mapping, database-facing mechanics.
PrismaService Generated Prisma Client lifecycle and connection/adapter concerns.
The service still owns meaning A repository may return null. The service decides whether that means Task-not-found, hidden cross-tenant resource, optional relationship, or some other domain outcome.
Refactor TasksService to Business Intent matters because it changes a real boundary in TaskFlow. The goal is to understand the inputs, owner, observable output, and failure modes before adding more abstraction.
Restate the goal of “Refactor TasksService to Business Intent” as one observable behavior. Write the success signal before touching code.
Trace the boundary map: request → validated command → invariant → state transition → persisted result. Mark the one boundary this lesson changes.
Inspect the current repository state and record the exact files, route, table, or runtime involved. Avoid creating an abstraction before you can name the existing owner.
Implement the smallest vertical slice. Add one coherent change, run the narrowest verification command, and read the output before continuing.
Connect the slice to TaskFlow’s phase artifact: One TaskFlow aggregate/use case refactored behind a purposeful repository interface with real database integration tests.. Verify behavior from the outside as a user or API client would.
Create one controlled failure that is safe to reproduce. Predict the symptom first, capture evidence, locate the earliest broken boundary, then fix only that cause.
Close the reference and rebuild or explain the core change from memory. Record what you would monitor or test before trusting the same change in production.
if (task.status === "DONE") {
throw new TaskAlreadyCompletedError(task.id);
}
task.status = "DONE";
task.completedAt = new Date();Do not copy this as a finished answer. Predict what each boundary does, type the smallest useful slice, run it, then compare the result with your prediction.
Return to the nearest observable boundary: input, network, process log, database, or rendered UI. Compare actual evidence with the predicted path.
Change one variable at a time and reproduce the original failure after the fix; otherwise you have correlation, not diagnosis.
Without copying the final source, create a second small scenario that applies “Refactor TasksService to Business Intent” to a different TaskFlow resource or input. Write the acceptance criteria first, implement it, and provide the evidence that each criterion is satisfied.
Unit-test service behavior with a fake only where it proves useful behavior.
Use the smallest change that demonstrates this lesson. Stop after it works and capture evidence.GenericRepository<T> mirroring ORM methods
Predict the symptom before causing the failure. Then locate the earliest boundary where reality diverges from expectation.Why does this responsibility live here, what input does it trust, what output proves success, and what would go wrong if another layer owned it?
If your explanation is vague, repeat the boundary trace before moving on.Close the reference. Reproduce the core idea from memory or explain the exact request/data flow aloud. Then check your diff or notes for accidental shortcuts that bypass validation, authorization, error handling, or verification earned in earlier phases.