Skip to content

fix: CI failures + Two-Way Sync + API fixes (pre-1.0)#17

Merged
ozymandiashh merged 1 commit intomainfrom
fix/pre-1.0-final
Mar 22, 2026
Merged

fix: CI failures + Two-Way Sync + API fixes (pre-1.0)#17
ozymandiashh merged 1 commit intomainfrom
fix/pre-1.0-final

Conversation

@ozymandiashh
Copy link
Copy Markdown
Owner

Summary

Final fixes before 1.0 release:

  • CI: Fix 3 test failures on Linux/Windows CI (battery tests need _has_battery = True to bypass Darwin guard; taskkill test needs timeout=10 kwarg)
  • UX: "Two-Way Mirror" reverted to "Two-Way Sync" in wizard.html (only one-way Sync is renamed to Mirror)
  • V901: API exclude parameter now accepts both excludes (list) and exclude (string), with string-to-list splitting
  • V906: Cloud root wrapper infers source_type when not provided by API callers (paths without : are treated as local)

Test plan

  • python3 -m pytest -v all pass (including battery + taskkill tests)
  • CI green on Ubuntu, Windows, macOS
  • Mode selector shows: Copy, Mirror, Two-Way Sync
  • API transfer with excludes actually excludes files
  • API transfer to cloud root creates wrapper folder

- Fix battery tests to work with InternalBattery guard (F501)
- Fix taskkill test to work with timeout parameter (T502)
- Revert "Two-Way Mirror" back to "Two-Way Sync" (keep Mirror for one-way)
- Fix exclude parameter handling in API start endpoint (V901)
- Fix cloud root wrapper to apply via API path (V906)

Co-Authored-By: Claude <noreply@anthropic.com>
@ozymandiashh ozymandiashh merged commit b22ddff into main Mar 22, 2026
15 checks passed
@ozymandiashh ozymandiashh deleted the fix/pre-1.0-final branch March 22, 2026 13:12
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