Skip to content

IP-7080: [compose] Operator 체이닝 메서드명을 의도에 맞게 변경#542

Merged
novdov merged 1 commit intomainfrom
ip-7080
Mar 10, 2026
Merged

IP-7080: [compose] Operator 체이닝 메서드명을 의도에 맞게 변경#542
novdov merged 1 commit intomainfrom
ip-7080

Conversation

@novdov
Copy link
Copy Markdown
Contributor

@novdov novdov commented Mar 10, 2026

목적

Operator의 체이닝 메서드 then이 Promise/비동기의 시간적 순서를 내포하여 실제 의도(표현식의 구조적 합성)와 맞지 않는 문제를 해결

변경 사항

  • Operator.then 메서드명을 pipe로 변경하여 "이전 결과를 다음 연산자 입력으로 전달"하는 의도를 명확히 표현
  • 테스트 파일명 및 함수명을 pipe에 맞게 일치시킴

Summary by CodeRabbit

릴리스 노트

  • Breaking Changes

    • Operator 체이닝 메서드 이름 변경: .then()에서 .pipe()로 업데이트되었습니다.
  • 개선 사항

    • 표현식 평가 메커니즘이 개선되어 더욱 일관된 동작을 제공합니다.
  • 문서

    • 설계 결정사항에 관한 문서가 업데이트되었습니다.

@novdov novdov self-assigned this Mar 10, 2026
@linear
Copy link
Copy Markdown

linear bot commented Mar 10, 2026

IP-7080 [compose] Operator 체이닝 메서드명을 의도에 맞게 변경

배경

  • Operator의 중첩 선언을 flat하게 체이닝하기 위한 메서드 then이 존재
  • then은 Promise/비동기 체이닝의 시간적 순서를 내포하여, 표현식의 구조적 합성이라는 실제 의도와 맞지 않음

목표

  • 메서드명이 "이전 결과를 다음 연산자의 입력으로 전달한다"는 의도를 직관적으로 드러냄

@pozalabs-workspace pozalabs-workspace bot added the Improvement New feature or request label Mar 10, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 10, 2026

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Free

Run ID: 468837a1-3196-45e2-ac1c-bc88c94a89b9

📥 Commits

Reviewing files that changed from the base of the PR and between b447c39 and 011dd38.

📒 Files selected for processing (2)
  • compose/query/mongo/op/base.py
  • tests/query/unit/test_operator_pipe.py

Walkthrough

Operator 클래스의 메서드 thenpipe로 이름 변경했으며, 관련 테스트도 업데이트되었습니다. 메서드의 동작과 서명은 변하지 않으며, 오직 API 명명만 변경되었습니다.

Changes

Cohort / File(s) Summary
메서드 이름 변경
compose/query/mongo/op/base.py
Operator 클래스의 then() 메서드를 pipe()로 이름 변경. 함수 시그니처와 동작은 유지.
테스트 업데이트
tests/query/unit/test_operator_pipe.py
.then() 호출을 .pipe()로 변경하고, 테스트 함수명을 test_fmap에서 test_pipe로 변경.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 Hop, hop, then becomes pipe so fine,
Chaining operators in elegant design,
No logic shifts, just names that align,
Simple rename makes the API shine! 🌟


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.

@novdov novdov merged commit cefafd4 into main Mar 10, 2026
5 checks passed
@novdov novdov deleted the ip-7080 branch March 10, 2026 06:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Improvement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant