backend: complete dev-notes CRUD and health proof slice - #75
Merged
Merged
Conversation
juscghwe
marked this pull request as ready for review
August 1, 2026 08:13
juscghwe
changed the title
## Purpose Refactor the existing dev-notes implementation into a contract-driven module with clear HTTP, validation, service, repository, SQLite and health boundaries. --- ## Changes Added: * Resource contract and operation policies. * Generic command and field validation. * Repository port and shared SQLite repository. * Contract-derived schema, SQL, mapping and health helpers. * Module-local router, service and architecture documentation. Changed: * Replaced the existing dev-notes implementation with the rebuilt module. * Unified temporary and persistent storage behind the same repository contract. * Mounted the router exported by the dev-notes module. * Updated backend health integration and schema smoke coverage. * Updated the module README and dependency documentation. Removed: * Legacy facade, validation and duplicated SQLite adapters. * Separate hard-coded schemas and seed helpers. * Standalone dev-notes route wrapper. * Temporary `dev-notes-new` path. --- ## Validation * [x] Backend health smoke passed. * [x] Dev-notes health smoke passed. * [x] Schema smoke passed for `temp` and `persistent`. * [x] CRUD smoke passed for `temp` and `persistent`. --- ## Future Work [open] * Add targeted validation and service-level tests. * Continue frontend dev-notes integration in the parent branch.
juscghwe
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Purpose
Complete the M1 dev-notes backend proof slice with configurable storage, CRUD routes, schema validation, health reporting and integration smoke coverage.
Changes
Added:
Changed:
Removed:
Validation
tempandpersistent.tempandpersistent.Future Work [open]