Skip to content

Add note editing command - #172

Open
Gidr0Gen wants to merge 7 commits into
mate-academy:mainfrom
Gidr0Gen:review-me
Open

Add note editing command#172
Gidr0Gen wants to merge 7 commits into
mate-academy:mainfrom
Gidr0Gen:review-me

Conversation

@Gidr0Gen

@Gidr0Gen Gidr0Gen commented Sep 2, 2026

Copy link
Copy Markdown

Adds an edit <id> <text> command to update an existing note. This PR is intentionally opened before conflict resolution so the change can be reviewed against the current main branch.

@Gidr0Gen

Gidr0Gen commented Sep 2, 2026

Copy link
Copy Markdown
Author

Claude caught the planted bug: edit() dereferences note.text without checking whether the requested note ID exists, so editing a missing note crashes with a TypeError.

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.

2 participants