fix(api): refuse a build under another owner's interrupted run id

[Eomdahyeon]

Refs #1025

문제

#1022 리뷰에서 짚은 경로다. check_existing_run_access 는 매니페스트와 작업 레지스트리만 본다. 재시작으로 중단된 run 은 둘 다 없어서 그 run_id 가 "새 run" 으로 통과하는데, 이벤트 저장소에는 제출자가 남아 있고(#996) 그 사람은 여전히 그 run 이 왜 끝났는지 읽을 수 있다. 다른 사용자가 그 id 로 빌드하면 자기 작업이 도는 동안 첫 제출자의 이벤트를 읽고, 다시 중단되면 자기 실패를 첫 제출자에게 남긴다.

변경

• routes/_guards.py: check_existing_run_access 가 매니페스트 → 레지스트리 다음에 제출 기록을 본다. check_active_run_access 와 같은 소유 규칙(ownership_allows)이다.
  • 다른 소유자의 기록이 있으면 403 forbidden: not run owner (이 함수의 기존 거절과 같은 본문)
  • 제출자 본인은 통과한다 — 중단된 run 의 메시지가 안내하는 재제출이다
  • 기록이 없는 id 는 지금처럼 새 빌드다
• CHANGELOG [Unreleased] Fixed. 계약은 바꾸지 않았다(403 은 이미 선언된 응답이고 본문도 같다).

#1008 과의 관계

지금 main 에서 이 함수를 부르는 것은 동기 POST /build 뿐이다(routes/core.py). 이슈가 적은 비동기 POST /builds 경로는 #1008 이 같은 함수를 연결해야 닫힌다. 그래서 Closes 가 아니라 Refs 로 두었다 — #1008 머지 뒤 POST /builds 로 같은 시나리오를 고정하는 테스트를 더하고 이슈를 닫겠다. #1008 은 routes/builds.py 와 이 파일의 주석 두 곳을 바꾸고 이 PR 은 함수 본문을 바꾸므로, 어느 쪽이 먼저 들어가도 충돌은 작을 것으로 본다(직접 합쳐 보지는 않았다).

테스트

tests/unit/test_interrupted_run_status.py 에 3개:

• 다른 사용자가 중단된 run id 로 POST /build → 403, 제출 기록의 소유자는 그대로
• 제출자 본인은 가드를 통과
• 아무도 제출하지 않은 id 는 누구에게나 통과

검증

• 수정 전 가드로 돌리면 새 테스트가 실패한다: FAILED …::test_another_user_cannot_build_under_the_interrupted_run_id, 1 failed, 8 passed.
• 수정 후 pytest tests/unit/test_interrupted_run_status.py tests/unit/test_json_array_reader.py tests/unit/test_service.py tests/unit/test_service_jobs.py → 235 passed.
• ruff check src tests, ruff format --check src tests, mypy src 통과.
• 전체 스위트는 로컬에서 돌리지 않았다 — CI 에 맡긴다.

🤖 Generated with Claude Code

[yeongseon]

diff 를 읽었고 CLEAN 이라 승인합니다. #1022 리뷰에서 짚은 경로를 그대로 닫습니다 — 매니페스트 → 레지스트리 → 제출 기록 순으로 같은 소유 규칙을 쓰고, 제출자 본인의 재제출은 통과시키고, 기록이 없는 id 는 새 빌드로 둡니다. 수정 전 가드에서 새 테스트가 실패하는 것도 확인돼 있습니다.

Refs 로 둔 판단도 맞습니다: 비동기 POST /builds 는 #1008 이 이 함수를 연결해야 닫힙니다. 이 PR 이 먼저 들어가면 #1008 은 _guards.py 의 docstring 에서 충돌할 수 있는데, 그쪽 브랜치는 제가 main 을 합쳐 맞추겠습니다.

로컬에서 테스트를 돌리지는 않았습니다.