DefStream 0.1.10__tar.gz → 0.1.12__tar.gz

{defstream-0.1.10 → defstream-0.1.12}/DefStream.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: DefStream
-Version: 0.1.10
+Version: 0.1.12
 Summary: Synchronous message processing with pluggable broker adapters
 Home-page: https://github.com/yong5eon/DefStream
 Author: Heo Yongseon
@@ -571,6 +571,8 @@ class Add(
 - `format`의 기본값은 `application/json`입니다. 문서의 미디어 타입만 바꾸며 브로커 코덱은 별도 설정입니다.
 - Run의 `name`은 AsyncAPI Operation 키와 Metrics 작업 키에 함께 사용합니다.
   생략하면 `모듈명.__qualname__`을 사용하며, Run의 `description`은 AsyncAPI Operation에 표시됩니다.
+  명시적 이름은 `[A-Za-z0-9_.-]+` 형식이며 공백·한국어는 허용하지 않습니다.
+  대소문자와 입력값을 그대로 보존합니다.
 - 실행 순서는 초기화 훅 → 요청 검증 → `run()` → 응답 검증 → 완료 훅 → 응답 발행·ACK입니다.
 
 검증 예외는 작업 실패로 처리되어 오류 훅과 `on_error` 정책을 따릅니다.

{defstream-0.1.10 → defstream-0.1.12}/DefStream.egg-info/SOURCES.txt

@@ -21,6 +21,7 @@ issues/feature/8/ci.refs.json
 issues/fix/10/IMPLEMENTS.md
 issues/fix/10/README.md
 issues/fix/10/REQUIREMENTS.md
+issues/fix/13/README.md
 issues/fix/3/IMPLEMENTS.md
 issues/fix/3/README.md
 issues/fix/3/REQUIREMENTS.md

{defstream-0.1.10 → defstream-0.1.12}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: DefStream
-Version: 0.1.10
+Version: 0.1.12
 Summary: Synchronous message processing with pluggable broker adapters
 Home-page: https://github.com/yong5eon/DefStream
 Author: Heo Yongseon
@@ -571,6 +571,8 @@ class Add(
 - `format`의 기본값은 `application/json`입니다. 문서의 미디어 타입만 바꾸며 브로커 코덱은 별도 설정입니다.
 - Run의 `name`은 AsyncAPI Operation 키와 Metrics 작업 키에 함께 사용합니다.
   생략하면 `모듈명.__qualname__`을 사용하며, Run의 `description`은 AsyncAPI Operation에 표시됩니다.
+  명시적 이름은 `[A-Za-z0-9_.-]+` 형식이며 공백·한국어는 허용하지 않습니다.
+  대소문자와 입력값을 그대로 보존합니다.
 - 실행 순서는 초기화 훅 → 요청 검증 → `run()` → 응답 검증 → 완료 훅 → 응답 발행·ACK입니다.
 
 검증 예외는 작업 실패로 처리되어 오류 훅과 `on_error` 정책을 따릅니다.

{defstream-0.1.10 → defstream-0.1.12}/README.en.md

@@ -507,6 +507,8 @@ class Add(
 - `format` defaults to `application/json`. It changes documented media types, not broker codecs.
 - Run's `name` is used as both the AsyncAPI Operation key and the Metrics task key.
   When omitted, `module.__qualname__` is used; Run's `description` appears on the AsyncAPI Operation.
+  Explicit names must match `[A-Za-z0-9_.-]+`; whitespace and Korean characters are not allowed.
+  Names are case-sensitive and preserved exactly as provided.
 - Execution order: setup hooks → request validation → `run()` → response validation → completion hooks → reply/ACK.
 
 Validation exceptions are task failures and follow error hooks and the `on_error`

{defstream-0.1.10 → defstream-0.1.12}/README.md

@@ -538,6 +538,8 @@ class Add(
 - `format`의 기본값은 `application/json`입니다. 문서의 미디어 타입만 바꾸며 브로커 코덱은 별도 설정입니다.
 - Run의 `name`은 AsyncAPI Operation 키와 Metrics 작업 키에 함께 사용합니다.
   생략하면 `모듈명.__qualname__`을 사용하며, Run의 `description`은 AsyncAPI Operation에 표시됩니다.
+  명시적 이름은 `[A-Za-z0-9_.-]+` 형식이며 공백·한국어는 허용하지 않습니다.
+  대소문자와 입력값을 그대로 보존합니다.
 - 실행 순서는 초기화 훅 → 요청 검증 → `run()` → 응답 검증 → 완료 훅 → 응답 발행·ACK입니다.
 
 검증 예외는 작업 실패로 처리되어 오류 훅과 `on_error` 정책을 따릅니다.

{defstream-0.1.10 → defstream-0.1.12}/issues/fix/10/IMPLEMENTS.md

@@ -13,8 +13,8 @@
 
 - [Run](../../../src/Worker/Run.py) 클래스 정의에서 `name`과 `description`
   키워드를 받는다. `RunMeta`가 이름을 내부 키워드 `run_name`으로 중계하고,
-  `__init_subclass__`가 이름과 설명을 처리한다. 이름은 Python 식별자이며
-  예약어가 아닌지 검사하고, 설명은 문자열 또는 `None`으로 검증한다.
+  `__init_subclass__`가 이름과 설명을 처리한다. 명시적 이름은
+  `[A-Za-z0-9_.-]+` 전체 일치로 검사하고, 설명은 문자열 또는 `None`으로 검증한다.
 - 클래스 생성 시 이름이 생략되면 해당 클래스의 `__module__`과 `__qualname__`으로
   기본 이름을 만든다. 하위 Run에는 상위 Run의 이름과 설명을 상속하지 않는다.
 - [Worker](../../../src/Worker/Worker.py)는 Run 등록 시 동일한 이름이 이미
@@ -133,3 +133,18 @@ Python 3.9.13과 3.10.11에서 클래스 정의의 `name=` 키워드가
 Python 3.9의 첫 전체 실행에서는 무관한 Console HTTP 테스트에 Windows
 `ConnectionAbortedError`가 발생했으나, 해당 테스트의 단독 실행과 전체 재실행은
 통과했다. 변경한 Run 소스의 Pylance 진단에도 오류가 없다.
+
+## 7. 명시적 이름의 허용 문자 변경
+
+Python 식별자 제한을 제거하고 영문 대소문자·숫자·`_`·`.`·`-`를 하나 이상
+허용하도록 변경했다. 위치 제한은 없으며 Python 예약어도 허용한다.
+빈 문자열, 공백·개행, 한국어 등 다른 문자는 거부한다.
+대소문자 변환이나 공백 제거 없이 지정한 이름을 그대로 사용한다.
+미지정 이름의 `모듈명.__qualname__` 생성 규칙과 중복 검사는 유지한다.
+
+허용·거부 문자 조합과 입력값 보존, AsyncAPI Operation 키 및 Metrics
+스냅샷·Prometheus 레이블의 동일 이름 사용을 회귀 테스트로 검증한다.
+
+- Python 3.12.10: 전체 201개 실행, 193개 통과, 선택적 테스트 8개 건너뜀.
+- Python 3.9.13·3.10.11: 각 관련 테스트 43개 실행, 40개 통과, 3개 건너뜀.
+- 변경한 Run 소스의 Pylance 진단 및 변경 파일 공백 검사 통과.

{defstream-0.1.10 → defstream-0.1.12}/issues/fix/10/README.md

@@ -28,8 +28,10 @@ class Add(
 ```
 
 Run 등록 후 AsyncAPI Operation 키와 Metrics 작업 식별자는 `add`입니다.
-이름은 Python 식별자여야 하며, Worker에 같은 이름을 가진 Run을 함께 등록할 수
-없습니다.
+명시적 이름은 영문 대소문자·숫자·`_`·`.`·`-`로만 구성된 한 글자 이상의 문자열입니다
+(`Orders.create-v2_Job` 등). 위치 제한은 없고 Python 예약어도 허용합니다.
+공백·한국어·그 외 문자는 허용하지 않으며, 입력을 변환하지 않고 대소문자를
+구분합니다. Worker에 같은 이름을 가진 Run을 함께 등록할 수 없습니다.
 
 ## 기존 코드에 미치는 영향
 

{defstream-0.1.10 → defstream-0.1.12}/issues/fix/10/REQUIREMENTS.md

@@ -46,7 +46,7 @@ Run을 정의할 때 이름과 설명을 선택적으로 지정할 수 있도록
 |---|---|
 | SPECNAME-01 | Run 정의 시 선택적 `name`을 지정할 수 있어야 한다. 지정하지 않은 경우의 기본값은 미지정 상태여야 한다. |
 | SPECNAME-02 | Run 정의 시 선택적 `description`을 지정할 수 있어야 한다. 지정하지 않은 경우 설명은 없어야 한다. |
-| SPECNAME-03 | `name`은 Python 문법상 유효한 식별자여야 하며 예약어는 허용하지 않는다. 이름을 자동 정규화하거나 변경하지 않는다. 잘못된 타입은 `TypeError`, 유효하지 않은 식별자나 예약어는 `ValueError`로 거부한다. |
+| SPECNAME-03 | 명시적 `name`은 `[A-Za-z0-9_.-]+` 전체 일치 문자열이어야 한다. 영문 대소문자·숫자·`_`·`.`·`-`를 위치 제한 없이 허용하고 Python 예약어도 허용한다. 빈 문자열, 공백, 한국어 및 그 외 문자는 `ValueError`, 문자열과 `None` 이외의 타입은 `TypeError`로 거부한다. 입력을 정규화하지 않고 대소문자를 구분하여 보존한다. |
 | SPECNAME-04 | `description`은 문자열 또는 `None`이어야 한다. 다른 타입은 `TypeError`로 명확히 거부한다. 빈 설명 문자열은 그대로 보존한다. |
 | SPECNAME-05 | 이름이 지정되지 않은 Run의 유효 이름은 `Run.__module__ + '.' + Run.__qualname__`이어야 한다. 중첩 클래스도 `__qualname__`을 포함해야 한다. |
 | SPECNAME-06 | AsyncAPI 문서의 `operations` 객체에서 각 Run의 키로 유효 Run 이름을 사용해야 한다. 이 요구사항은 Operation 객체의 별도 필드가 아니라 `operations` 맵의 키에 적용한다. |
@@ -79,7 +79,7 @@ Run을 정의할 때 이름과 설명을 선택적으로 지정할 수 있도록
 | WAT-01 | `Run` 정의에 `name='orders'`를 지정하면 AsyncAPI `operations`와 Metrics에서 `orders` 키로 식별된다. |
 | WAT-02 | 이름을 지정하지 않은 모듈 최상위 Run의 AsyncAPI 키와 Metrics 키가 모두 `모듈명.클래스명`이다. |
 | WAT-03 | 이름을 지정하지 않은 중첩 Run의 기본 이름에 모듈명과 중첩 경로를 포함한 `__qualname__`이 반영된다. |
-| WAT-04 | `name`이 Python 문법상 식별자가 아니거나 예약어이거나 문자열이 아니면 각각 `ValueError` 또는 `TypeError`가 발생한다. |
+| WAT-04 | `Orders.create-v2_Job`, 숫자만 있는 이름, Python 예약어 및 구분자로 시작·끝나는 이름을 그대로 허용한다. 빈 문자열, 한국어, 공백·개행 및 허용 범위 밖 문자는 `ValueError`, 문자열과 `None` 이외의 타입은 `TypeError`로 거부한다. |
 | WAT-05 | `description`이 문자열이면 AsyncAPI Operation 설명에 반영되고, `None`이면 Operation에 설명 필드가 없다. |
 | WAT-06 | `description`에 문자열 또는 `None` 이외의 값을 전달하면 `TypeError`가 발생한다. |
 | WAT-07 | 두 Run에 같은 사용자 지정 이름을 지정하면 Run 등록 시 충돌을 감지하고, 문서 생성에서도 충돌을 검증한다. 기존 항목을 덮어쓰거나 통계를 합치지 않는다. |

defstream-0.1.12/issues/fix/13/README.md

@@ -0,0 +1,31 @@
+# Worker 시작 배너와 샘플 로깅 순서
+
+## 변경 요약
+
+Worker 시작 배너를 로깅 설정 여부와 관계없이 먼저 확인할 수 있도록 Worker
+샘플의 로깅 설정 시점을 `WorkerSetup` 훅으로 옮겼습니다. 시작 배너는 터미널에서
+청록·파랑 계열로 강조하며, ASCII 아트의 모양과 문구는 유지합니다.
+
+## 변경 내용
+
+- [Worker 샘플](../../../sample/Worker/Sample.py)은 `configure_logging()`을
+  실행 진입점에서 호출하는 대신 `SetupLogger` setup 훅에서 수행합니다.
+  Worker 생성·작업 등록 시점의 로그는 아직 설정되지 않은 핸들러에 의해 출력되지
+  않으며, 시작 배너가 샘플의 첫 화면 출력이 됩니다.
+- [Startup](../../../src/Worker/Startup.py)은 stdout이 터미널이고 `NO_COLOR`
+  환경 변수가 설정되지 않은 경우에만 로고에 색을 적용합니다. `Starting Worker`
+  제목도 굵은 청록색으로 표시합니다. 파일·파이프 등으로 출력을 리다이렉트하거나
+  `NO_COLOR`를 설정하면 ANSI 색상 코드를 출력하지 않습니다.
+- `Worker.run()`의 로그와 배너 호출 순서는 변경하지 않았습니다. 따라서
+  로깅 핸들러가 미리 설정된 애플리케이션에서는 `Starting Worker` 로그가 배너보다
+  먼저 기록될 수 있습니다. 이 변경은 샘플의 로깅 설정 시점을 조정해 샘플 출력에서
+  배너를 먼저 보이게 합니다.
+
+## 호환성 및 검증 범위
+
+시작 배너의 ASCII 문자와 구성 출력은 유지됩니다. 색상은 터미널 조건에서만
+추가되며 `NO_COLOR`를 통해 끌 수 있습니다.
+
+로고 색상 적용 후 ANSI 코드를 제거한 결과가 원본과 동일한지, 수정한 Python 소스의
+문법이 유효한지 확인했습니다. Worker 시작 테스트는 환경에 `pika`가 없어 실행하지
+못했으며, 샘플의 로깅 순서 변경은 별도 실행으로 검증하지 않았습니다.

{defstream-0.1.10 → defstream-0.1.12}/sample/Worker/RabbitMQ/Client.py

@@ -3,7 +3,7 @@ from random import randint
 from Tasks import (
 	Add,
 	Divide,
-	LoggingSetup,
+	TaskLoggingSetup,
 	logger,
 )
 
@@ -16,7 +16,7 @@ from DefStream.Worker import request
 
 
 if __name__ == '__main__':
-	LoggingSetup()()
+	TaskLoggingSetup()()
 	broker = Connection(Configuration(
 		host='127.0.0.1',
 		port=5672,

{defstream-0.1.10 → defstream-0.1.12}/sample/Worker/RabbitMQ/Tasks.py

@@ -28,9 +28,11 @@ from DefStream.Validator.Patterns.Object import (
 	IsRequiredIn,
 )
 from DefStream.Worker import (
+	WorkerSetup,
 	PoolSetup,
 	Run,
 	Specification,
+	Worker,
 )
 
 __all__ = (
@@ -42,7 +44,27 @@ __all__ = (
 logger = getLogger('Sample')
 
 
-class LoggingSetup(PoolSetup):
+class SetupLogger(WorkerSetup):
+	"""부모와 spawn 자식 프로세스의 샘플 로그 출력을 설정한다."""
+
+	def __call__(
+		self,
+		worker: Worker,
+	) -> None:
+		handler = StreamHandler()
+		handler.setFormatter(Formatter('%(asctime)s %(processName)s %(name)s %(levelname)s %(message)s'))
+		sources = (
+			Filter('Sample'),
+			Filter('DefStream'),
+		)
+		handler.addFilter(lambda record: any(source.filter(record) for source in sources))
+		root = getLogger()
+		root.setLevel(DEBUG)
+		root.addHandler(handler)
+		return
+
+
+class TaskLoggingSetup(PoolSetup):
 	"""부모와 spawn 자식 프로세스의 샘플 로그 출력을 설정한다."""
 
 	def __call__(self) -> None:

{defstream-0.1.10 → defstream-0.1.12}/sample/Worker/RabbitMQ/Worker.py

@@ -7,7 +7,8 @@ from signal import (
 from Tasks import (
 	Add,
 	Divide,
-	LoggingSetup,
+	TaskLoggingSetup,
+	SetupLogger,
 	logger,
 )
 
@@ -76,7 +77,6 @@ class Cleanup(WorkerCleanup):
 
 
 if __name__ == '__main__':
-	LoggingSetup()()
 	worker = Worker(
 		Connection(Configuration(
 			host='127.0.0.1',
@@ -86,9 +86,12 @@ if __name__ == '__main__':
 		)),
 		ProcessPool(
 			10,
-			setups=LoggingSetup(),
+			setups=TaskLoggingSetup(),
+		),
+		setups=(
+			SetupLogger(),
+			Prepare(),
 		),
-		setups=Prepare(),
 		cleanups=Cleanup(),
 		inspector=Inspector(
 			host='127.0.0.1',

{defstream-0.1.10 → defstream-0.1.12}/sample/Worker/Redis/Sample.py

@@ -39,19 +39,6 @@ from DefStream.Worker import (
 logger = getLogger('Sample')
 
 
-def configure_logging() -> None:
-	handler = StreamHandler()
-	handler.setFormatter(Formatter('%(asctime)s %(name)s %(levelname)s %(message)s'))
-	sources = (
-		Filter('Sample'),
-		Filter('DefStream'),
-	)
-	handler.addFilter(lambda record: any(source.filter(record) for source in sources))
-	root = getLogger()
-	root.setLevel(DEBUG)
-	root.addHandler(handler)
-	return
-
 
 class Add(
 	Run,
@@ -96,6 +83,24 @@ class Divide(
 		return result
 
 
+class SetupLogger(WorkerSetup):
+	def __call__(
+		self,
+		worker: Worker,
+	) -> None:
+		handler = StreamHandler()
+		handler.setFormatter(Formatter('%(asctime)s %(name)s %(levelname)s %(message)s'))
+		sources = (
+			Filter('Sample'),
+			Filter('DefStream'),
+		)
+		handler.addFilter(lambda record: any(source.filter(record) for source in sources))
+		root = getLogger()
+		root.setLevel(DEBUG)
+		root.addHandler(handler)
+		return
+
+
 class Prepare(WorkerSetup):
 	def __call__(
 		self,
@@ -139,11 +144,13 @@ class Cleanup(WorkerCleanup):
 
 
 def main() -> None:
-	configure_logging()
 	worker = Worker(
 		Connection(Configuration()),
 		ThreadPool(10),
-		setups=Prepare(),
+		setups=(
+			SetupLogger(),
+			Prepare(),
+		),
 		cleanups=Cleanup(),
 		inspector=Inspector(
 			host='127.0.0.1',

{defstream-0.1.10 → defstream-0.1.12}/sample/Worker/Sample.py

@@ -35,19 +35,6 @@ from DefStream.Worker import (
 logger = getLogger('Sample')
 
 
-def configure_logging() -> None:
-	handler = StreamHandler()
-	handler.setFormatter(Formatter('%(asctime)s %(name)s %(levelname)s %(message)s'))
-	sources = (
-		Filter('Sample'),
-		Filter('DefStream'),
-	)
-	handler.addFilter(lambda record: any(source.filter(record) for source in sources))
-	root = getLogger()
-	root.setLevel(DEBUG)
-	root.addHandler(handler)
-	return
-
 
 class Queue(Sendable):
 	def __init__(self):
@@ -219,6 +206,24 @@ class Add(
 		)
 
 
+class SetupLogger(WorkerSetup):
+	def __call__(
+		self,
+		worker: Worker,
+	) -> None:
+		handler = StreamHandler()
+		handler.setFormatter(Formatter('%(asctime)s %(name)s %(levelname)s %(message)s'))
+		sources = (
+			Filter('Sample'),
+			Filter('DefStream'),
+		)
+		handler.addFilter(lambda record: any(source.filter(record) for source in sources))
+		root = getLogger()
+		root.setLevel(DEBUG)
+		root.addHandler(handler)
+		return
+
+
 class Prepare(WorkerSetup):
 	def __call__(
 		self,
@@ -252,12 +257,16 @@ class Cleanup(WorkerCleanup):
 
 
 if __name__ == '__main__':
-	configure_logging()
 	worker = Worker(
 		Broker(),
 		ThreadPool(1),
-		setups=Prepare(),
-		cleanups=Cleanup(),
+		setups=(
+			SetupLogger(),
+			Prepare(),
+		),
+		cleanups=(
+			Cleanup(),
+		),
 	)
 	worker.add(Add)
 	worker.run()

{defstream-0.1.10 → defstream-0.1.12}/src/Worker/Run.py

@@ -4,8 +4,8 @@ from abc import (
 	abstractmethod,
 )
 from collections.abc import Mapping
-from keyword import iskeyword
 from logging import getLogger
+from re import fullmatch
 from time import monotonic
 from typing import (
 	Any,
@@ -138,8 +138,11 @@ class Run(ABC, metaclass=RunMeta):
 			str,
 		):
 			raise TypeError('Run name must be a string or None')
-		if run_name is not None and (not run_name.isidentifier() or iskeyword(run_name)):
-			raise ValueError('Run name must be a valid Python identifier')
+		if run_name is not None and fullmatch(
+			r'[A-Za-z0-9_.-]+',
+			run_name,
+		) is None:
+			raise ValueError('Run name must contain only ASCII letters, digits, underscores, dots or hyphens')
 		if description is not None and not isinstance(
 			description,
 			str,

{defstream-0.1.10 → defstream-0.1.12}/src/Worker/Startup.py

@@ -1,5 +1,6 @@
 from collections.abc import Mapping
-from os import getpid
+from os import environ, getpid
+import sys
 from typing import (
 	Any,
 	TYPE_CHECKING,
@@ -19,9 +20,42 @@ LOGO = r"""
 ███  ███ ▄█▀█▄ ▀██▀  ▀████▄  ▀██▀▀ ████▄ ▄█▀█▄  ▀▀█▄ ███▄███▄ 
 ███  ███ ██▄█▀  ██     ▀████  ██   ██ ▀▀ ██▄█▀ ▄█▀██ ██ ██ ██ 
 ██████▀  ▀█▄▄▄  ██  ███████▀  ██   ██    ▀█▄▄▄ ▀█▄██ ██ ██ ██ 
+
+Just def. Let workers do the work. You don't need to await this.
 """
 
 
+def supports_color() -> bool:
+	return (
+		getattr(sys.stdout, 'isatty', lambda: False)()
+		and not environ.get('NO_COLOR')
+	)
+
+
+def colorize_logo(logo: str) -> str:
+	colors = (
+		'\033[38;5;39m',
+		'\033[38;5;45m',
+		'\033[38;5;51m',
+		'\033[38;5;45m',
+		'\033[38;5;39m',
+	)
+	reset = '\033[0m'
+	colorized_lines = []
+	art_line = 0
+	for line in logo.splitlines(keepends=True):
+		if line.strip() and art_line < len(colors):
+			newline = '\n' if line.endswith('\n') else ''
+			colorized_lines.append(
+				f'{colors[art_line]}{line[:-1] if newline else line}'
+				f'{reset}{newline}',
+			)
+			art_line += 1
+		else:
+			colorized_lines.append(line)
+	return ''.join(colorized_lines)
+
+
 def subscription_value(
 	parameters: Any,
 	name: str,
@@ -90,10 +124,15 @@ def startup(worker: 'Worker') -> None:
 			str,
 		) and vhost:
 			broker_name += '/' + vhost.lstrip('/')
+	logo = LOGO.rstrip()
+	starting_worker = 'Starting Worker'
+	if supports_color():
+		logo = colorize_logo(logo)
+		starting_worker = f'\033[1;38;5;51m{starting_worker}\033[0m'
 	lines = [
-		LOGO.rstrip(),
+		logo,
 		'',
-		'Starting Worker',
+		starting_worker,
 		f'  PID: {getpid()}',
 		f'  Broker: {broker_name}',
 	]

{defstream-0.1.10 → defstream-0.1.12}/test/test_inspector.py

@@ -498,7 +498,7 @@ class InspectorTests(TestCase):
 		class NamedAdd(
 			Add,
 			parameters={'queue': 'named-add'},
-			name='named_add',
+			name='Orders.create-v2_Job',
 		):
 			pass
 
@@ -514,7 +514,21 @@ class InspectorTests(TestCase):
 		self.addCleanup(worker.pool.stop)
 		self.assertEqual(
 			set(metrics.tasks),
-			{'named_add'},
+			{'Orders.create-v2_Job'},
+		)
+		started = metrics.begin(NamedAdd.name)
+		metrics.end(
+			NamedAdd.name,
+			started,
+			True,
+		)
+		self.assertEqual(
+			metrics.snapshot(inspector)['tasks'][NamedAdd.name]['success'],
+			1,
+		)
+		self.assertIn(
+			'defstream_tasks_total{task="Orders.create-v2_Job",outcome="success"} 1',
+			metrics.render(inspector),
 		)
 
 	def test_metrics_reject_duplicate_run_names_without_overwriting(self):

{defstream-0.1.10 → defstream-0.1.12}/test/test_logging.py

@@ -274,7 +274,7 @@ class LoggingTests(TestCase):
 		try:
 			spec.loader.exec_module(sample)
 			self.assertEqual(root.handlers, handlers)
-			sample.configure_logging()
+			sample.SetupLogger()(Mock())
 			handler = root.handlers[-1]
 			for name, expected in (
 				(

{defstream-0.1.10 → defstream-0.1.12}/test/test_specifications.py

@@ -426,7 +426,7 @@ class DocumentTests(TestCase):
 		class Described(
 			Add,
 			parameters={'queue': 'described'},
-			name='described',
+			name='Orders.create-v2_Job',
 			description='Add request description',
 		):
 			"""Not the operation description."""
@@ -444,7 +444,7 @@ class DocumentTests(TestCase):
 		inspector.worker.add(Empty)
 		operations = Specifications('/spec').document(inspector)['operations']
 		self.assertEqual(
-			operations['described']['description'],
+			operations['Orders.create-v2_Job']['description'],
 			'Add request description',
 		)
 		self.assertEqual(
@@ -469,12 +469,42 @@ class DocumentTests(TestCase):
 			Local.__module__ + '.' + Local.__qualname__,
 		)
 
-	def test_run_name_must_be_a_python_identifier(self):
+	def test_run_names_preserve_allowed_characters(self):
+		for name in (
+			'Orders.create-v2_Job',
+			'123',
+			'class',
+			'.-_',
+			'-Job.',
+		):
+			with self.subTest(name=name):
+				class Named(
+					Run,
+					name=name,
+				):
+					def run(self):
+						pass
+
+				self.assertEqual(
+					Named.name,
+					name,
+				)
+
+	def test_run_names_reject_empty_strings_and_unsupported_characters(self):
 		for invalid in (
 			'',
-			'not-valid',
 			'has space',
-			'class',
+			' leading',
+			'trailing ',
+			'job\n',
+			'job\t',
+			'job/name',
+			'job:name',
+			'job"name',
+			'job\\name',
+			'\uC791\uC5C5',
+			'caf\u00e9',
+			'\uff11',
 		):
 			with self.subTest(name=invalid), self.assertRaises(ValueError):
 				class Invalid(