Skip to content

feat(library): rename a note's title and description - #42

Merged
Atreus-X merged 2 commits into
mainfrom
feat/notes-rename
Sep 30, 2026
Merged

Atreus-X merged 2 commits into
mainfrom
feat/notes-rename

Conversation

@Atreus-X

Copy link
Copy Markdown
Owner

Summary

  • Library notes can be renamed: a pencil next to the title renames the .md file (refuses a name that already exists or is invalid).
  • A description field above the editor rewrites the note's first non-empty line (keeping any # prefix), which is what the list shows under the name.
  • Host notes are named after their connection, so only their description is editable.
  • New LibraryNoteRenameCommand RPC; Go unit test covers rename, collision, header rewrite and the host-note restriction.

Remote hosts

Notes are local files in the Wave config dir, so this is independent of the connection; host notes are keyed by connection name and work the same for plain SSH and wsh hosts. No wsh needed.

Test plan

  • Rename a general note: list, header and editor follow the new name
  • Rename onto an existing name shows an error and changes nothing
  • Edit the description: the note's first line and the list subtitle update
  • Host note: no pencil, description editable

Not checked in the running app.

🤖 Generated with Claude Code

https://claude.ai/code/session_01W6QMXXvRReaKxWk1JL5Et6

Atreus-X and others added 2 commits September 29, 2026 14:34
General notes get a pencil next to the title (renames the .md file) and a
description field above the editor (rewrites the note's first line, which is
what the list shows). Host notes keep their connection-derived name but can
still have their description edited. New LibraryNoteRenameCommand RPC.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01W6QMXXvRReaKxWk1JL5Et6
…g or selecting another note

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01W6QMXXvRReaKxWk1JL5Et6
@Atreus-X Atreus-X mentioned this pull request Sep 30, 2026
@Atreus-X
Atreus-X merged commit e9e51f0 into main Sep 30, 2026
1 check passed
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