Fix Obsidian sync by running sync-setup before sync #32

Open
ryan wants to merge 1 commits from fix/obsidian-sync-setup into main
Owner

Summary

  • Run ob sync-setup with env vars before attempting ob sync --continuous
  • Skip sync gracefully if setup fails
  • Fixes "No sync configuration found" error on container startup

Test plan

  • Set OBSIDIAN_CONTINUOUS_SYNC=true with valid vault credentials and verify sync starts
  • Set OBSIDIAN_CONTINUOUS_SYNC=true with invalid/missing credentials and verify graceful skip
  • Set OBSIDIAN_CONTINUOUS_SYNC=false and verify no sync-related output
## Summary - Run `ob sync-setup` with env vars before attempting `ob sync --continuous` - Skip sync gracefully if setup fails - Fixes "No sync configuration found" error on container startup ## Test plan - [ ] Set `OBSIDIAN_CONTINUOUS_SYNC=true` with valid vault credentials and verify sync starts - [ ] Set `OBSIDIAN_CONTINUOUS_SYNC=true` with invalid/missing credentials and verify graceful skip - [ ] Set `OBSIDIAN_CONTINUOUS_SYNC=false` and verify no sync-related output
ryan added 1 commit 2026-04-24 09:04:39 -04:00
The container was failing with "No sync configuration found" because
ob sync was called without first running ob sync-setup. Now startup.sh
configures sync using environment variables before starting continuous
sync, and gracefully skips sync if setup fails.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Checking for merge conflicts…
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin fix/obsidian-sync-setup:fix/obsidian-sync-setup
git checkout fix/obsidian-sync-setup
Sign in to join this conversation.
No Reviewers
No Label
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: ryan/simbarag#32