Skip to content

fix: harden sync status progress contracts#80

Merged
eunaverse merged 1 commit into
mainfrom
feature/fix-dotenv-loading
Jun 19, 2026
Merged

fix: harden sync status progress contracts#80
eunaverse merged 1 commit into
mainfrom
feature/fix-dotenv-loading

Conversation

@eunaverse

Copy link
Copy Markdown
Owner

요약

  • get_sync_status()에 장시간 Notion upstream fetch 진행 힌트를 안정적으로 노출하도록 sync/status 흐름을 정리했습니다.
  • sync_all()의 public contract를 고정해서 error shape, all-blocked 상태, hidden-only vs true-empty failure 케이스를 일관되게 처리하도록 보강했습니다.
  • local uv/.env/GitHub/Obsidian 설정과 startup vs refresh failure 설명이 현재 동작과 맞도록 README와 architecture/plan 문서를 업데이트했습니다.

리뷰 포인트

  • observer stop와 inactive-job stop를 분리했고 observer-cancelled background failure만 replay-once 하도록 제한했습니다.
  • Notion request polling에서 stop 요청이 completed request error보다 우선되도록 race를 정리했습니다.
  • 최종 fresh 5-reviewer subagent pass에서 actionable finding 없이 종료했습니다.
  • 관련 이슈 없음

검증

  • uv run pytest tests/api/test_tools_contract.py tests/fetching/test_connectors.py tests/fetching/test_notion.py tests/indexing/test_ingestion_service.py tests/storage/test_metadata_store.py -q (237 passed)
  • python -m compileall api core environments fetching indexing search storage main.py
  • ./scripts/verify_functional_e2e.sh (25 passed)

@eunaverse eunaverse merged commit 9495ada into main Jun 19, 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