Skip to content

feat: Add pipeline composition, branching, and conditional execution#66

Merged
jisung-02 merged 1 commit into
mainfrom
10-pipeline-branching
Dec 7, 2025
Merged

feat: Add pipeline composition, branching, and conditional execution#66
jisung-02 merged 1 commit into
mainfrom
10-pipeline-branching

Conversation

@jisung-02
Copy link
Copy Markdown
Owner

@jisung-02 jisung-02 commented Dec 7, 2025

.github/pull_request_template.md

🎯 변경 사항

📋 체크리스트

  • gleam format 실행 완료
  • gleam check 통과
  • 테스트 추가/수정 완료
  • 문서 업데이트 완료
  • CHANGELOG.md 업데이트 (필요시)

🧪 테스트 방법

🤖 CodeRabbit 리뷰 요청사항

  • 성능 최적화 검토
  • 에러 처리 검증
  • 타입 안전성 확인
  • API 설계 리뷰

📸 스크린샷/데모


참고: 댓글에 @coderabbitai 다시 리뷰해줘라고 작성하면 전체 재리뷰를 요청할 수 있습니다.

Summary by CodeRabbit

릴리스 노트

  • New Features

    • 파이프라인 순차 실행, 조건부 분기, 병렬 처리 기능 추가
    • 조건부 스테이지 실행, 반복 로직, 오류 처리 및 팬아웃 기능 추가
    • 파이프라인을 스테이지로 포함할 수 있는 기능 제공
  • Tests

    • 파이프라인 구성 기능의 포괄적인 단위 테스트 추가

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Dec 7, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

파이프라인과 스테이지 구성 유틸리티를 제공하는 새로운 Gleam 모듈(compose.gleam)을 추가하고, 이를 검증하는 포괄적인 테스트 스위트(compose_test.gleam)를 작성했습니다. 순차, 조건부, 병렬 실행 및 반복 로직을 지원하는 10개 이상의 공개 함수와 30개 이상의 테스트 케이스를 포함합니다.

Changes

응집도 / 파일(들) 변경 요약
파이프라인 합성 핵심 유틸리티
src/aether/pipeline/compose.gleam
9개의 공개 함수 추가: compose (순차 연결), branch (조건부 분기), parallel (병렬 실행), merge (결과 수집), when (조건부 스테이지), if_else (조건부 선택), repeat_until (반복), pipeline_to_stage (파이프라인 임베딩), fan_out (다중 스테이지 확산). 에러 변환 헬퍼 함수 2개 추가.
구성 유틸리티 테스트 커버리지
test/aether/pipeline/compose_test.gleam
35개 이상의 테스트 함수 추가: 기본 구성 시나리오(compose, branch, parallel, merge), 조건부 로직(when, if_else), 반복(repeat_until, try_or), 파이프라인 임베딩, 팬아웃, 중첩 및 복합 구성 검증.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

세부 검토 포인트:

  • 타입 안전성: PipelineErrorStageError 변환 로직이 모든 분기에서 일관되게 적용되는지 확인. Result 타입의 제네릭 매개변수가 올바르게 전파되는지 검증 필요
  • 패턴 매칭 완전성: branch, if_else, repeat_until 등에서 Ok/Error 케이스 처리 누락 여부 확인
  • 에러 전파: 중첩된 파이프라인/스테이지에서 에러가 올바르게 버블링되는지 테스트 케이스 검토
  • 함수형 관용구: 파이프라인 합성에 고계 함수와 클로저 활용이 Gleam 스타일과 부합하는지 확인
  • 테스트 품질: 35개 테스트의 주장문(assertion) 정확성과 엣지 케이스 커버리지 검토 (특히 repeat_until 최대 반복 제한, 병렬 실행 실패 케이스)

Poem

파이프라인을 엮다 🔗
순차, 분기, 병렬의 춤,
에러 처리는 안전하게,
함수형 조합의 미학,
35개 테스트가 증언한다 ✨


📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Free

📥 Commits

Reviewing files that changed from the base of the PR and between 3dd1c9c and f1ecc19.

📒 Files selected for processing (2)
  • src/aether/pipeline/compose.gleam (1 hunks)
  • test/aether/pipeline/compose_test.gleam (1 hunks)

Note

🎁 Summarized by CodeRabbit Free

Your organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Pro by visiting https://app.coderabbit.ai/login.

Comment @coderabbitai help to get the list of available commands and usage tips.

@jisung-02 jisung-02 merged commit 24b738c into main Dec 7, 2025
0 of 2 checks 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