Skip to content

feat: push notes to remote GitHub repo on save#530

Merged
kwannoel merged 6 commits intomainfrom
session-4-9c3a34
Mar 20, 2026
Merged

feat: push notes to remote GitHub repo on save#530
kwannoel merged 6 commits intomainfrom
session-4-9c3a34

Conversation

@kwannoel
Copy link
Copy Markdown
Owner

Summary

  • Add push_to_remote() to notes module — checks for origin remote via git2, shells out to git push origin HEAD if present
  • Integrate into command layer so every note commit triggers a background push
  • Show error toast on sync failure via notes-sync-error Tauri event

Setup

cd ~/.the-controller/notes
git remote add origin git@github.com:user/my-notes.git

Test plan

  • push_to_remote returns Ok when no remote configured
  • push_to_remote pushes to bare repo remote in test
  • All 7 note mutation commands trigger background push
  • commit_notes triggers push when content changes
  • Frontend shows toast on sync error
  • All Rust tests pass (399+)
  • All frontend tests pass (291)

🤖 Generated with Claude Code

kwannoel and others added 6 commits March 19, 2026 23:01
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@kwannoel kwannoel merged commit 6bf5bd7 into main Mar 20, 2026
1 of 3 checks passed
@kwannoel kwannoel deleted the session-4-9c3a34 branch March 20, 2026 06:04
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