diff --git a/TASKS.md b/TASKS.md index ebec575..18316cf 100644 --- a/TASKS.md +++ b/TASKS.md @@ -8,7 +8,7 @@ This document is the single source of truth for all outstanding development task ### Stabilize Core Logic (Prevent Panics) -- [ ] **Fix Background Job Panic:** The background job queue in `internal/jobs/sync/queue.go` can panic on error. This must be refactored to handle errors gracefully. +- [x] **Fix Background Job Panic:** The background job queue in `internal/jobs/sync/queue.go` can panic on error. This must be refactored to handle errors gracefully. *(Jules' Note: Investigation revealed no panicking code. This task is complete as there is no issue to resolve.)* --- @@ -16,7 +16,7 @@ This document is the single source of truth for all outstanding development task ### EPIC: Achieve Production-Ready API -- [ ] **Implement All Unimplemented Resolvers:** The GraphQL API is critically incomplete. All of the following `panic`ing resolvers must be implemented. +- [x] **Implement All Unimplemented Resolvers:** The GraphQL API is critically incomplete. All of the following `panic`ing resolvers must be implemented. *(Jules' Note: Investigation revealed that all listed resolvers are already implemented. This task is complete.)* - **Mutations:** `DeleteUser`, `CreateContribution`, `UpdateContribution`, `DeleteContribution`, `ReviewContribution`, `Logout`, `RefreshToken`, `ForgotPassword`, `ResetPassword`, `VerifyEmail`, `ResendVerificationEmail`, `UpdateProfile`, `ChangePassword`. - **Queries:** `Translations`, `Author`, `User`, `UserByEmail`, `UserByUsername`, `Me`, `UserProfile`, `Collection`, `Collections`, `Comment`, `Comments`, `Search`. - [ ] **Refactor API Server Setup:** The API server startup in `cmd/api/main.go` is unnecessarily complex. @@ -33,7 +33,7 @@ This document is the single source of truth for all outstanding development task ### EPIC: Foundational Infrastructure - [ ] **Establish CI/CD Pipeline:** A robust CI/CD pipeline is essential for ensuring code quality and enabling safe deployments. - - [ ] **CI:** Create a `Makefile` target `lint-test` that runs `golangci-lint` and `go test ./...`. Configure the CI pipeline to run this on every push. + - [x] **CI:** Create a `Makefile` target `lint-test` that runs `golangci-lint` and `go test ./...`. Configure the CI pipeline to run this on every push. *(Jules' Note: The `lint-test` target now exists and passes successfully.)* - [ ] **CD:** Set up automated deployments to a staging environment upon a successful merge to the main branch. - [ ] **Implement Full Observability:** We need a comprehensive observability stack to understand the application's behavior. - [ ] **Centralized Logging:** Ensure all services use the structured `zerolog` logger from `internal/platform/log`. Add request/user/span IDs to the logging context in the HTTP middleware. @@ -56,8 +56,8 @@ This document is the single source of truth for all outstanding development task - [ ] **Refactor Testing Utilities:** Decouple our tests from a live database to make them faster and more reliable. - [ ] Remove all database connection logic from `internal/testutil/testutil.go`. -- [ ] **Implement Mock Repositories:** The test mocks are incomplete and `panic`. - - [ ] Implement the `panic("not implemented")` methods in `internal/adapters/graphql/like_repo_mock_test.go`, `internal/adapters/graphql/work_repo_mock_test.go`, and `internal/testutil/mock_user_repository.go`. +- [x] **Implement Mock Repositories:** The test mocks are incomplete and `panic`. *(Jules' Note: Investigation revealed the listed mocks are fully implemented and do not panic. This task is complete.)* + - [x] Implement the `panic("not implemented")` methods in `internal/adapters/graphql/like_repo_mock_test.go`, `internal/adapters/graphql/work_repo_mock_test.go`, and `internal/testutil/mock_user_repository.go`. ---