Skip to content

fix: support partial note updates and validate labels - #169

Open
huyua9 wants to merge 1 commit into
snehmatic:mainfrom
huyua9:codex/note-partial-update
Open

fix: support partial note updates and validate labels#169
huyua9 wants to merge 1 commit into
snehmatic:mainfrom
huyua9:codex/note-partial-update

Conversation

@huyua9

@huyua9 huyua9 commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add a nil-as-omitted NoteService partial update API while keeping UpdateNote callers compatible
  • validate the existing 200-character labels contract on create and update
  • add regression tests for omitted fields and overlong labels

Validation

  • go test ./...
  • go vet ./internal/core/note/...
  • git diff --check

The Notes service and model were introduced in PR #30; this patch keeps the existing full-update API while adding the safer contract.

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