Skip to content

feat(sync): bidirectional + active hours (complete M7) - #37

Merged
ArianAr merged 1 commit into
mainfrom
feat/m7-complete
Jul 16, 2026
Merged

feat(sync): bidirectional + active hours (complete M7)#37
ArianAr merged 1 commit into
mainfrom
feat/m7-complete

Conversation

@ArianAr

@ArianAr ArianAr commented Jul 16, 2026

Copy link
Copy Markdown
Owner

Summary

Complete remaining M7 checklist items and mark M6/M7 done on the roadmap.

Closes #36

  • Rule bidirectional: forward sync then reverse (reverse never deletes / no extra_targets)
  • Rule active_hours_utc (HH:MM-HH:MM, multi + overnight): Start refuses; scheduler skips
  • Dashboard fields + dry-run reverse matrix
  • docs/ops.md + ROADMAP M6/M7 Done (optional OpenTelemetry deferred post-1.0)

Test

  • go test ./...
  • go vet ./...
  • Frontend build
  • Binary smoke: /healthz, /readyz, /api/version, /metrics, rule validation

Complete remaining M7 items for v1:
- bidirectional: forward then reverse (no delete/fan-out on reverse)
- active_hours_utc maintenance windows for start + cron skip
- UI, ops docs, unit tests; mark M6/M7 done (OTel deferred)

Closes #36
@ArianAr

ArianAr commented Jul 16, 2026

Copy link
Copy Markdown
Owner Author

Review: reverse safety looks correct (no delete/fan-out); active hours exclusive end is intentional. Approve when CI green.

@ArianAr
ArianAr enabled auto-merge (squash) July 16, 2026 19:15
@ArianAr
ArianAr merged commit 66babd7 into main Jul 16, 2026
8 of 9 checks 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.

feat: bidirectional sync and active-hours windows (M7 complete)

1 participant