Skip to content

[Domain] 연결선 위치 정합성 검증#191

Merged
learncold merged 2 commits into
mainfrom
codex/validate-connection-span-placement
May 7, 2026
Merged

[Domain] 연결선 위치 정합성 검증#191
learncold merged 2 commits into
mainfrom
codex/validate-connection-span-placement

Conversation

@muzygosu
Copy link
Copy Markdown
Collaborator

@muzygosu muzygosu commented May 5, 2026

Summary

  • [Domain] 연결선(Connection2D.centerSpan)이 fromZoneId/toZoneId로 참조하는 실제 구역 경계와 맞닿는지 검증을 추가한다.
  • [Domain] 연결선이 구역 경계에서 벗어나면 InvalidGeometry blocking issue를 생성한다.
  • [Application] InvalidGeometry를 live validation issue로 분류해, 사용자가 연결선을 정상 위치로 되돌리면 기존 blocking issue가 사라지도록 한다.
  • [Domain Test] Demo layout 테스트에 이동된 연결선이 blocking issue로 잡히는 검증을 추가한다.

Related Issue

Area

  • Engine
  • Domain
  • Application
  • Docs
  • Build
  • Analysis
  • Chore

Architecture Check

  • I kept the dependency direction application -> domain -> engine.
  • I did not add Qt UI code to src/domain.
  • I did not add domain or application dependencies to src/engine.
  • I used src/ as the include root.

Verification

  • cmake --preset windows-debug
  • cmake --preset windows-debug-no-app
  • cmake --build --preset build-debug --target safecrowd_tests
  • cmake --build --preset build-no-app-debug --target safecrowd_tests
  • ctest --preset test-debug
  • ctest --preset test-no-app-debug
  • Not run (reason below)

Risks / Follow-up

  • InvalidGeometry live validation refresh를 위해 application layer의 issue 갱신 목록도 함께 업데이트했다.
  • 출구 연결은 기존 import 모델을 유지하기 위해 출구 zone 전체 경계가 아니라 walkable zone 경계 접촉을 기준으로 검증한다.

@muzygosu muzygosu requested a review from learncold as a code owner May 5, 2026 14:09
@learncold learncold merged commit 12c3cbd into main May 7, 2026
2 checks passed
@learncold learncold deleted the codex/validate-connection-span-placement branch May 7, 2026 08:20
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.

Task-연결선 위치와 구역 연결 정합성 검증

2 participants