Skip to content

Fix: 이벤트·활동 시각을 넣은 순간(clock_timestamp)으로 (019) - #36

Merged
oyeonghun merged 1 commit into
devfrom
fix/event-clock-timestamp
Oct 10, 2026
Merged

oyeonghun merged 1 commit into
devfrom
fix/event-clock-timestamp

Conversation

@oyeonghun

Copy link
Copy Markdown
Collaborator

요약

  • events.ts·agent_activity.ts 기본값 now() → clock_timestamp() (마이그레이션 019, 기본값만 변경·기존 행 유지).
  • 원인: now()는 BEGIN 시각이라, 먼저 BEGIN한 트랜잭션이 READ COMMITTED로 뒤에 커밋된 변화를 읽고 남긴 이벤트가 원인보다 과거 시각을 받았다. 룸 피드에서 "실행해 주세요"가 "프로젝트를 시작합니다"보다 앞에 정렬되어 CI(cli-v0.1.6 배포)에서 가끔 실패했다.
  • 같은 모양을 고정하는 테스트 추가 — now()면 실패, clock_timestamp()면 통과를 확인.
  • 룸 테스트 준비 단계는 태스크 생성 신호를 기다린 뒤 시작한다.

테스트

  • 전체 vitest, typecheck 통과.

🤖 Generated with Claude Code

…작보다 앞에 정렬되던 것 (019)

- now()는 BEGIN 시각이라 READ COMMITTED에서 뒤에 커밋된 변화를 읽고 남긴 이벤트가 원인보다 과거 시각을 받았다.
  룸 피드 테스트가 CI에서 가끔 '실행해 주세요'를 '프로젝트를 시작합니다'보다 앞에 냈다.
- 같은 모양(먼저 연 트랜잭션 A, 그 사이 커밋된 B)을 고정하는 테스트 — now()면 실패, clock_timestamp()면 통과를 확인.
- 룸 테스트 준비 단계는 태스크 생성 신호를 기다린 뒤 시작한다.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@oyeonghun
oyeonghun merged commit dcefb0a into dev Oct 10, 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