DefStream 0.1.21__tar.gz → 0.1.22__tar.gz

{defstream-0.1.21 → defstream-0.1.22}/DefStream.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: DefStream
-Version: 0.1.21
+Version: 0.1.22
 Summary: Synchronous message processing with pluggable broker adapters
 Home-page: https://github.com/yong5eon/DefStream
 Author: Heo Yongseon

{defstream-0.1.21 → defstream-0.1.22}/DefStream.egg-info/SOURCES.txt

@@ -141,6 +141,14 @@ issues/fix/43/implements/1793261447018.md
 issues/fix/43/tasks/1793260818661.md
 issues/fix/43/tasks/1793261447019.md
 issues/fix/43/tasks/1793262314955.md
+issues/fix/50/CHANGELOG.md
+issues/fix/50/IMPLEMENTS.md
+issues/fix/50/PLAN.md
+issues/fix/50/README.md
+issues/fix/50/REQUIREMENTS.md
+issues/fix/50/TASKS.md
+issues/fix/50/implements/1793282630353.md
+issues/fix/50/tasks/1793282630354.md
 issues/fix/6/IMPLEMENTS.md
 issues/fix/6/README.md
 issues/fix/6/REQUIREMENTS.md

{defstream-0.1.21 → defstream-0.1.22}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: DefStream
-Version: 0.1.21
+Version: 0.1.22
 Summary: Synchronous message processing with pluggable broker adapters
 Home-page: https://github.com/yong5eon/DefStream
 Author: Heo Yongseon

{defstream-0.1.21 → defstream-0.1.22}/docs/RabbitMQ.en.md

@@ -325,7 +325,9 @@ print(result)
 
 - `timeout` is in **seconds**. Missing replies raise `TimeoutError`.
   This does not cancel delivered work or strictly bound total connection and cleanup time.
-- Each call owns a dedicated connection and temporary reply queue, cleaned up on exit.
+- Queue-based RPC uses a dedicated connection and creates a temporary reply queue named
+  `{request queue}.{uuid4().hex}`, which is cleaned up on exit. Exchange RPC reply queue
+  names are not affected by this rule.
 - Server rejection raises `DefStream.Broker.RemoteError`.
   Requeue can retry without sending a final error response.
 - Reply publication and request ACK are not atomic. Handle duplicate execution with RPC too.

{defstream-0.1.21 → defstream-0.1.22}/docs/RabbitMQ.md

@@ -315,7 +315,8 @@ print(result)
 
 - `timeout`은 **초** 단위이며 응답이 없으면 `TimeoutError`가 발생합니다.
   이미 전달된 서버 작업을 취소하거나 전체 연결·정리 시간을 엄격히 제한하는 값은 아닙니다.
-- 각 호출은 전용 연결과 임시 응답 큐를 사용하고 종료 시 정리합니다.
+- 요청 큐 RPC는 전용 연결을 사용하고 `{요청 큐}.{uuid4().hex}` 형식의 임시 응답 큐를 생성합니다.
+  큐는 종료 시 정리됩니다. Exchange RPC의 응답 큐 이름은 이 규칙의 영향을 받지 않습니다.
 - 서버가 요청을 거부하면 `DefStream.Broker.RemoteError`가 발생합니다.
   재큐잉은 최종 오류 응답 없이 재시도할 수 있습니다.
 - 응답 발행과 요청 ACK는 원자적이지 않습니다. RPC에서도 중복 실행에 대비하세요.

defstream-0.1.22/issues/fix/50/CHANGELOG.md

@@ -0,0 +1,5 @@
+# 변경 로그
+
+## 수정
+
+- RabbitMQ 큐 기반 RPC의 임시 응답 큐 이름에 요청 큐와 UUID를 포함해, 브로커에서 응답 큐가 어떤 요청 큐에 대응하는지 식별할 수 있습니다. Exchange RPC의 응답 큐 이름은 변경되지 않습니다.

defstream-0.1.22/issues/fix/50/IMPLEMENTS.md

@@ -0,0 +1,8 @@
+# 구현 결과
+
+요청 큐 기반 RPC의 임시 응답 큐를 `{요청 큐}.{uuid4().hex}` 형식으로 선언한다. 전용 응답 큐의 exclusive·auto-delete 수명, 요청 `reply_to`, `correlation_id` 응답 매칭과 Exchange RPC 동작은 유지한다.
+
+- 구현 및 회귀 테스트: [요청 큐 기반 RPC 응답 큐 이름 적용](./implements/1793282630353.md)
+- 검증 결과: [이슈 분석 및 RPC 회귀 검증](./tasks/1793282630354.md)
+
+`tests/test_rabbitmq_rpc.py` 16개와 `tests/test_rabbitmq_broker_methods.py` 1개가 통과했다. RabbitMQ 서버가 필요한 통합 테스트는 실행하지 않았다.

defstream-0.1.22/issues/fix/50/PLAN.md

@@ -0,0 +1,15 @@
+# 작업 계획
+
+## 대상과 순서
+
+1. 완료: `src/Broker/Implements/RabbitMQ/Queue.py`에서 요청 큐 이름 기반 응답 큐를 선언했다.
+2. 완료: `tests/test_rabbitmq_rpc.py`에 응답 큐 형식, 기존 선언 옵션과 `reply_to` 연결을 검증하는 회귀 테스트를 추가했다.
+3. 완료: `docs/RabbitMQ.md`와 `docs/RabbitMQ.en.md`의 동기 RPC 설명을 함께 갱신했다.
+4. 완료: RPC 테스트를 실행하고 결과를 기록했다.
+5. 완료: 종합 문서, 단계별 기록과 `CHANGELOG.md`를 실제 결과에 맞춰 정리했다.
+
+## 검증
+
+- `tests/test_rabbitmq_rpc.py`의 Queue 및 Exchange RPC 테스트로 요청 큐 경로와 Exchange 제외 범위를 확인한다.
+- RPC 테스트에서 응답 큐 이름 형식, exclusive·auto-delete 옵션, `reply_to` 연결 및 기존 성공·실패 처리를 확인한다.
+- 문서 링크와 한국어·영어 동기화 여부를 확인한다.

defstream-0.1.22/issues/fix/50/README.md

@@ -0,0 +1,36 @@
+# RPC 임시 응답 큐 이름을 요청 큐 기반으로 생성
+
+## 이슈와 작업 정보
+
+- 원본 이슈: [#50](https://github.com/yong5eon/DefStream/issues/50)
+- 상태: Open
+- 유형: `fix` (원본 레이블에 적용됨)
+- 추가 레이블: `rabbitmq`
+- 저장소: `yong5eon/DefStream`
+- 작업 브랜치: `fix/50`
+- 분기 기준: `master` (`origin/master`와 동일)
+- 요구사항: [REQUIREMENTS.md](./REQUIREMENTS.md)
+- 계획: [PLAN.md](./PLAN.md)
+- 구현 종합: [IMPLEMENTS.md](./IMPLEMENTS.md)
+- 작업·검증 종합: [TASKS.md](./TASKS.md)
+- 변경 로그: [CHANGELOG.md](./CHANGELOG.md)
+- 최신 구현 기록: [1793282630353.md](./implements/1793282630353.md)
+- 최신 작업 기록: [1793282630354.md](./tasks/1793282630354.md)
+
+## 이슈 원문
+
+### 해결하려는 문제
+
+RabbitMQ RPC가 요청마다 RabbitMQ 자동 생성 이름(`amq.gen-*`)의 임시 응답 큐를 사용해 큐 이름만으로는 어떤 요청 큐의 응답인지 알아보기 어렵습니다.
+
+### 제안하는 해결 방법
+
+`Queue.call()`을 포함해 요청 큐를 대상으로 하는 RPC에서 임시 응답 큐 이름을 `{요청큐}.{uuid4().hex}` 형식으로 생성합니다. 예를 들어 요청 큐가 `orders`이면 응답 큐 이름은 `orders.0123456789abcdef...` 형태입니다. 응답 큐의 exclusive·auto-delete 동작과 요청의 `reply_to`/`correlation_id` 처리 방식은 유지합니다.
+
+### 고려한 대안
+
+RabbitMQ가 생성하는 기본 임시 큐 이름(`amq.gen-*`)을 그대로 사용합니다.
+
+### 추가 정보
+
+Exchange RPC는 특정 요청 큐 이름을 대상으로 하지 않으므로 이번 요청 큐 기반 이름 규칙의 범위에 포함하지 않습니다.

defstream-0.1.22/issues/fix/50/REQUIREMENTS.md

@@ -0,0 +1,23 @@
+# 요구사항
+
+## 분석
+
+- 현재 `Queue.call()`은 RabbitMQ에 빈 큐 이름을 전달해 `amq.gen-*` 임시 응답 큐를 생성한다. 큐 이름만으로는 요청 큐를 식별하기 어렵다.
+- `RabbitMQ.Connection.request()`는 `Queue.call()`에 위임하므로 해당 경로도 같은 응답 큐 이름 규칙을 사용해야 한다.
+- `Exchange.call()`은 특정 요청 큐를 대상으로 하지 않으며 이 변경 범위에서 제외한다.
+- 기존 응답 큐 수명과 RPC 상관관계 계약은 변경하지 않는다.
+
+## 확정 요구사항
+
+1. 요청 큐 기반 RPC의 임시 응답 큐 이름을 `{요청 큐 이름}.{uuid4().hex}` 형식으로 생성한다.
+2. 응답 큐를 exclusive 및 auto-delete로 선언하는 기존 동작을 유지한다.
+3. 요청의 `reply_to`는 생성한 응답 큐를 가리키고, `correlation_id` 기반 응답 일치 처리는 유지한다.
+4. Exchange RPC의 임시 응답 큐 이름과 동작은 변경하지 않는다.
+5. RabbitMQ 상세 매뉴얼의 한국어·영어 설명을 일치시켜 갱신한다.
+
+## 인수 기준
+
+- `Queue.call()`이 요청 큐 이름 뒤에 32자리 소문자 16진수 UUID를 붙여 응답 큐를 선언한다.
+- 선언된 큐가 `exclusive=True`, `auto_delete=True`이고 요청의 `reply_to`에 사용된다.
+- 기존 RPC 성공·오류·타임아웃 및 상관관계 테스트가 유지된다.
+- Exchange RPC 동작은 변경되지 않는다.

defstream-0.1.22/issues/fix/50/TASKS.md

@@ -0,0 +1,8 @@
+# 작업 및 검증 결과
+
+이슈 원문과 기존 구현을 대조해 요청 큐 기반 RPC만 변경하고 Exchange RPC는 제외했다. `fix` 레이블이 이미 적용되어 있었고, 요청한 `fix/50` 브랜치를 `master`에서 생성했다.
+
+- [이슈 분석 및 RPC 회귀 검증](./tasks/1793282630354.md)
+- [구현 결과](./IMPLEMENTS.md)
+
+워크스페이스 Python 환경에서 RPC 테스트 16개와 RabbitMQ 브로커 메서드 테스트 1개가 통과했다. RabbitMQ 서버 기반 통합 테스트는 실행하지 않았다.

defstream-0.1.22/issues/fix/50/implements/1793282630353.md

@@ -0,0 +1,7 @@
+# 요청 큐 기반 RPC 응답 큐 이름 적용
+
+`src/Broker/Implements/RabbitMQ/Queue.py`의 `Queue.call()`이 RabbitMQ에 빈 큐 이름을 전달하던 부분을 `{요청 큐}.{uuid4().hex}` 형식으로 변경했다. 응답 큐는 계속 `exclusive=True`, `auto_delete=True`로 선언되며, 생성된 큐 이름을 사용한 소비와 요청 `reply_to`, 기존 `correlation_id` 응답 일치 처리는 유지된다.
+
+`Exchange.call()`은 별도 구현이므로 변경하지 않았다. 회귀 테스트에서 큐 RPC의 이름 형식과 옵션, `reply_to` 연결을 확인하고 Exchange RPC는 빈 이름을 계속 요청하는지 확인한다. RabbitMQ 상세 매뉴얼도 한국어·영어판을 함께 갱신했다.
+
+검증 결과는 [작업·검증 기록](../tasks/1793282630354.md)에 정리했다.

defstream-0.1.22/issues/fix/50/tasks/1793282630354.md

@@ -0,0 +1,12 @@
+# 이슈 분석 및 RPC 회귀 검증
+
+- GitHub 이슈 #50의 원문과 레이블을 확인했다. `fix` 유형 레이블이 이미 적용되어 있어 변경하지 않았다.
+- `fix/50`을 `master` (`origin/master`와 동일)에서 생성했다. 요청 큐 RPC는 `Queue.call()`을 거치고 Exchange RPC는 별도 구현임을 확인했다.
+- 이슈 README, 요구사항, 계획 및 구현·작업 종합 기록을 작성했다.
+- 기본 시스템 Python에는 `pika`가 없어 첫 테스트 수집이 실패했다. 별도 설치 없이 워크스페이스 Python 환경에서 다음 테스트를 재실행했다.
+  - 저장소 루트에서 `python -m unittest discover -s tests -p test_rabbitmq_rpc.py`: 16개 통과
+  - 저장소 루트에서 `python -m unittest discover -s tests -p test_rabbitmq_broker_methods.py`: 1개 통과
+- 테스트는 큐 응답 이름 형식, `exclusive`·`auto_delete`, 요청 `reply_to` 연결과 Exchange RPC의 기존 이름 요청 동작을 확인한다.
+- 고객에게 보이는 응답 큐 식별 동작을 변경 로그에 기록하고 RabbitMQ 한국어·영어 상세 매뉴얼을 맞춰 갱신했다.
+
+RabbitMQ 서버를 사용하는 통합 테스트는 실행하지 않았다.

{defstream-0.1.21 → defstream-0.1.22}/src/Broker/Implements/RabbitMQ/Queue.py

@@ -175,7 +175,7 @@ class Queue(
 		try:
 			channel = connection.channel()
 			reply_queue = channel.queue_declare(
-				queue='',
+				queue=f'{self.queue}.{uuid4().hex}',
 				exclusive=True,
 				auto_delete=True,
 			).method.queue

{defstream-0.1.21 → defstream-0.1.22}/tests/test_rabbitmq_rpc.py

@@ -3,6 +3,7 @@ from json import (
 	dumps,
 	loads,
 )
+from typing import List
 from unittest import (
 	TestCase,
 	main,
@@ -310,6 +311,20 @@ class QueueClientTests(TestCase):
 		connect = self.connect
 		connection = connect.return_value
 		channel = connection.channel.return_value
+		reply_queues: List[str] = []
+
+		def declare_reply_queue(
+			*,
+			queue: str,
+			exclusive: bool,
+			auto_delete: bool,
+		) -> Mock:
+			declaration: Mock = Mock()
+			declaration.method.queue = queue if queue else 'amq.gen-test'
+			reply_queues.append(declaration.method.queue)
+			return declaration
+
+		channel.queue_declare.side_effect = declare_reply_queue
 
 		def respond(**kwargs):
 			callback = channel.basic_consume.call_args.kwargs['on_message_callback']
@@ -338,7 +353,28 @@ class QueueClientTests(TestCase):
 			7,
 		)
 		connection.close.assert_called_once_with()
-		self.assertTrue(channel.queue_declare.call_args.kwargs['exclusive'])
+		queue_declaration = channel.queue_declare.call_args.kwargs
+		if self.client_class is Queue:
+			self.assertRegex(
+				queue_declaration['queue'],
+				r'^requests\.[0-9a-f]{32}$',
+			)
+		else:
+			self.assertEqual(
+				queue_declaration['queue'],
+				'',
+			)
+		self.assertTrue(queue_declaration['exclusive'])
+		self.assertTrue(queue_declaration['auto_delete'])
+		properties = channel.basic_publish.call_args.kwargs['properties']
+		self.assertEqual(
+			channel.basic_consume.call_args.args[0],
+			reply_queues[0],
+		)
+		self.assertEqual(
+			properties.reply_to,
+			reply_queues[0],
+		)
 		self.assertTrue(channel.basic_publish.call_args.kwargs['mandatory'])
 
 	def test_timeout_closes_connection(self):