Skip to content

fix: 매칭 이력 PR 자동 머지 및 6/29 중복 기록 제거#74

Merged
DaleSeo merged 1 commit into
mainfrom
fix/match-history-staleness
Jun 29, 2026
Merged

fix: 매칭 이력 PR 자동 머지 및 6/29 중복 기록 제거#74
DaleSeo merged 1 commit into
mainfrom
fix/match-history-staleness

Conversation

@DaleSeo

@DaleSeo DaleSeo commented Jun 29, 2026

Copy link
Copy Markdown
Member

문제

6/29 매칭이 두 가지로 오작동했습니다 (디스코드 제보):

  1. 스케줄 무시 — coffee-ui는 월 1회(28일)인데 직전 매칭(6/22) 7일 만에 재실행
  2. 동일 매칭 — 직전 커피챗과 완전히 같은 조합으로 매칭

원인

매칭 결과 Discord 발표는 cron 실행 중 자동으로 되는 반면, 이력 저장은 수동 머지가 필요한 PR로 처리됩니다. 직전 이력 PR(#72, 6/22 생성)이 6/29 02:55까지 방치된 상태에서 6/29 02:35 cron이 실행되어, history.json이 과거 시점(coffee-ui=5/25, coffee=6/8)으로 보였습니다.

  • schedule.ts는 마지막 매칭일로 실행 여부를 판단 → stale 날짜 기준으로 "주기 도래"로 오판 → 재실행
  • matcher.ts의 중복 회피는 마지막 라운드 페어를 회피 → 6/22 페어가 없으니 그대로 재현 → 6/22와 세 조 모두 동일한 결과

변경

  • match.yml: PR 생성 직후 gh pr merge --squash --delete-branch로 즉시 머지. ruleset 승인 요구가 0개라 즉시 머지 가능하며, 다음 cron 전에 history가 main에 확정되어 stale 창이 사라집니다.
  • data/coffee/history.json, data/coffee-ui/history.json: 오작동으로 생성된 6/29 중복 기록 제거. 마지막 매칭이 6/22로 복원되어 다음 주기가 정상화됩니다 (coffee 7/6, coffee-ui 7/20).

참고

  • Discord에는 이미 6/29 조별 쓰레드가 생성되어 있어 별도 정리가 필요합니다 (이 PR 범위 밖).

매칭 결과 Discord 발표는 자동인 반면 이력 저장은 수동 머지 PR이라,
직전 PR(#72)이 방치된 상태에서 6/29 cron이 stale history로 실행되어
스케줄을 무시하고 직전과 동일하게 재매칭되는 문제가 있었다.

- match.yml: PR 생성 직후 gh pr merge로 즉시 머지하여 다음 cron 전에
  history가 main에 반영되도록 변경 (ruleset 승인 0개라 즉시 머지 가능)
- coffee/coffee-ui: 오작동으로 생성된 6/29 중복 기록 제거
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jun 29, 2026

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
coffee 81f1a16 Commit Preview URL

Branch Preview URL
Jun 29 2026, 03:10 AM

@DaleSeo DaleSeo requested a review from sounmind June 29, 2026 03:11
@DaleSeo DaleSeo merged commit 49390e2 into main Jun 29, 2026
2 checks passed
@DaleSeo DaleSeo deleted the fix/match-history-staleness branch June 29, 2026 03: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.

2 participants