dagflows 0.2.0__tar.gz → 0.2.2__tar.gz

{dagflows-0.2.0 → dagflows-0.2.2}/.gitignore

@@ -27,3 +27,4 @@ internal_refs/
 
 # Scratch venv used by `make wheel`
 .wheelcheck/
+.venv-*/

{dagflows-0.2.0 → dagflows-0.2.2}/Makefile

@@ -31,7 +31,7 @@ readme:  ## just the README, about five seconds
 matrix:  ## the suite on every version requires-python claims
 	@for v in 3.11 3.12 3.13; do \
 		printf 'python %s: ' $$v; \
-		uv run --python $$v --group dev pytest -q | tail -1; \
+		UV_PROJECT_ENVIRONMENT=.venv-matrix-$$v uv run --python $$v --group dev pytest -q | tail -1; \
 	done
 
 build:  ## build the artifacts and check the metadata is publishable
@@ -57,4 +57,4 @@ wheel: build  ## prove the built wheel works, not just the working tree
 	@$(VENV)/Scripts/python -m dagflows --help > /dev/null && echo "cli ok"
 
 clean:  ## remove build and test leftovers
-	rm -rf dist $(VENV) .pytest_cache
+	rm -rf dist $(VENV) .venv-matrix-* .pytest_cache

{dagflows-0.2.0 → dagflows-0.2.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: dagflows
-Version: 0.2.0
+Version: 0.2.2
 Summary: SDK for authoring and running Dagflows workflows
 Project-URL: Homepage, https://github.com/dagflows/sdk-python
 Project-URL: Repository, https://github.com/dagflows/sdk-python

{dagflows-0.2.0 → dagflows-0.2.2}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "dagflows"
-version = "0.2.0"
+version = "0.2.2"
 description = "SDK for authoring and running Dagflows workflows"
 readme = "README.md"
 requires-python = ">=3.11"
@@ -42,6 +42,9 @@ packages = ["src/dagflows"]
 
 [tool.pytest.ini_options]
 testpaths = ["tests"]
+# A hang self-reports with every thread's stack instead of eating the CI
+# timeout in silence. Found one deadlock already.
+faulthandler_timeout = 300
 
 [tool.ruff]
 line-length = 100

{dagflows-0.2.0 → dagflows-0.2.2}/src/dagflows/__init__.py

@@ -37,6 +37,9 @@ _AUTHORING = {
 
 
 def __getattr__(name: str) -> Any:
+    if name == "__version__":
+        return _version()
+
     module = _AUTHORING.get(name)
     if module is None:
         raise AttributeError(f"module {__name__!r} has no attribute {name!r}")
@@ -44,6 +47,26 @@ def __getattr__(name: str) -> Any:
     return getattr(import_module(module, __name__), name)
 
 
+def _version() -> str:
+    """Return the installed package version from metadata."""
+    global _cached_version
+    if _cached_version:
+        return _cached_version
+
+    from importlib.metadata import PackageNotFoundError, version
+
+    try:
+        _cached_version = version("dagflows")
+    except PackageNotFoundError:
+        # An uninstalled source tree has no metadata to ask.
+        _cached_version = "0.0.0"
+
+    return _cached_version
+
+
+_cached_version = ""
+
+
 def __dir__() -> list[str]:
     return sorted(__all__)
 
@@ -68,5 +91,3 @@ __all__ = [
     "Workflow",
     "run",
 ]
-
-__version__ = "0.1.0"

{dagflows-0.2.0 → dagflows-0.2.2}/src/dagflows/_runtime/outstream.py

@@ -8,6 +8,7 @@ from types import TracebackType
 from typing import Any, Self
 
 from .content import ContentType
+from .envelope import DEFAULT_INLINE_MAX_BYTES
 from .multipart import Multipart
 from .result import RowSink, Upload, Written, buffer_cap
 
@@ -23,11 +24,14 @@ class OutputStream:
         memory_limit_mb: int = 0,
         multipart: Multipart | None = None,
     ) -> None:
+        # Fall back to default limit when unset (0) to avoid immediately uploading or rejecting rows.
+        limit = inline_max_bytes if inline_max_bytes > 0 else DEFAULT_INLINE_MAX_BYTES
+
         self.content_type = content_type
         self._sink = RowSink(
             content_type,
-            inline_max_bytes,
-            buffer_cap(inline_max_bytes, upload, memory_limit_mb),
+            limit,
+            buffer_cap(limit, upload, memory_limit_mb),
             upload,
             multipart,
         )

{dagflows-0.2.0 → dagflows-0.2.2}/tests/test_input.py

@@ -5,7 +5,7 @@ Tests for inline and reference input handling and streaming.
 import asyncio
 import json
 import threading
-from http.server import BaseHTTPRequestHandler, HTTPServer
+from http.server import BaseHTTPRequestHandler, ThreadingHTTPServer
 
 import pytest
 
@@ -21,7 +21,8 @@ BODIES = {
     "/rows.json": b'[{"id": 1, "name": "ana"}, {"id": 2, "name": "bo"}]',
     "/object.json": b'{"id": 1}',
     "/blob.bin": bytes(range(256)) * 4,
-    "/big.ndjson": b"".join(b'{"i": %d}\n' % i for i in range(20_000)),
+    # Large enough to exceed socket buffers so early-stop tests do not race against buffering.
+    "/big.ndjson": b'{"i": 0}\n' + (b'{"i": 1, "pad": "' + b"x" * 100 + b'"}\n') * 550_000,
 }
 
 served = {"gets": 0, "bytes": 0}
@@ -55,7 +56,9 @@ class Handler(BaseHTTPRequestHandler):
 
 @pytest.fixture(scope="module")
 def base():
-    httpd = HTTPServer(("127.0.0.1", 0), Handler)
+    # Use daemon threads so server shutdown does not deadlock on blocked writes.
+    httpd = ThreadingHTTPServer(("127.0.0.1", 0), Handler)
+    httpd.daemon_threads = True
     threading.Thread(target=httpd.serve_forever, daemon=True).start()
     yield f"http://127.0.0.1:{httpd.server_address[1]}"
     httpd.shutdown()
@@ -162,6 +165,7 @@ def test_iteration_is_re_entrant_rather_than_a_spent_generator(base):
 
 
 def test_stopping_early_does_not_pay_for_the_rest(base):
+    """Verify breaking out of iteration stops downloading remaining data."""
     handle = reference(base, "/big.ndjson", "application/x-ndjson")
 
     for row in handle:
@@ -268,8 +272,12 @@ def test_stopping_early_reads_at_most_one_batch_further(base, monkeypatch):
     monkeypatch.setattr(http, "stream", counting)
 
     async def first_only():
-        async for row in reference(base, "/big.ndjson", "application/x-ndjson"):
-            return row
+        # Close explicitly to ensure the connection is promptly terminated.
+        records = aiter(reference(base, "/big.ndjson", "application/x-ndjson"))
+        try:
+            return await anext(records)
+        finally:
+            await records.aclose()
 
     assert asyncio.run(first_only()) == {"i": 0}
     assert consumed < len(BODIES["/big.ndjson"]) / 4, (

{dagflows-0.2.0 → dagflows-0.2.2}/tests/test_outstream.py

@@ -85,6 +85,23 @@ def test_a_small_stream_stays_inline(no_network):
     assert "body" not in no_network
 
 
+def test_a_zero_inline_limit_from_the_platform_means_the_default(no_network):
+    """Verify zero inline_max_bytes falls back to the default inline limit."""
+    raw = {"node_key": "n", "inline_max_bytes": 0, "memory_limit_mb": 512}
+
+    offered = Ctx.from_raw({**raw, "output_upload_url": "https://s/put", "output_key": "k"})
+    with offered.output_stream() as out:
+        out.write({"id": 1})
+
+    assert out.ref().block["type"] == "INLINE"
+    assert "body" not in no_network, "a small output must not be uploaded"
+
+    with Ctx.from_raw(raw).output_stream() as out:
+        out.write({"id": 1})
+
+    assert out.ref().block["type"] == "INLINE"
+
+
 def test_writing_outside_the_block_is_refused():
     stream = ctx().output_stream()
 

dagflows-0.2.2/tests/test_package.py

@@ -0,0 +1,47 @@
+"""
+The package's claims about itself.
+"""
+
+from pathlib import Path
+
+import pytest
+
+import dagflows
+
+
+def test_version_comes_from_the_installed_metadata():
+    """Ensure __version__ matches the installed package metadata."""
+    from importlib.metadata import PackageNotFoundError, version
+
+    try:
+        expected = version("dagflows")
+    except PackageNotFoundError:
+        pytest.skip("running from an uninstalled source tree, no metadata to compare")
+
+    assert dagflows.__version__ == expected
+
+
+def test_version_agrees_with_pyproject():
+    """The end to end claim: what the code reports is what the build declares."""
+    from importlib.metadata import PackageNotFoundError, version
+
+    try:
+        version("dagflows")
+    except PackageNotFoundError:
+        pytest.skip("running from an uninstalled source tree")
+
+    pyproject = Path(__file__).resolve().parents[1] / "pyproject.toml"
+    declared = ""
+    for line in pyproject.read_text(encoding="utf-8").splitlines():
+        if line.startswith("version = "):
+            declared = line.split('"')[1]
+            break
+
+    assert declared, "pyproject.toml declares no version"
+    assert dagflows.__version__ == declared
+
+
+def test_an_unknown_attribute_still_raises():
+    """The __getattr__ gained a branch, the error for everything else stays."""
+    with pytest.raises(AttributeError, match="nonsense"):
+        dagflows.nonsense  # noqa: B018 - the attribute access is what raises

{dagflows-0.2.0 → dagflows-0.2.2}/uv.lock

@@ -1,5 +1,5 @@
 version = 1
-revision = 2
+revision = 3
 requires-python = ">=3.11"
 
 [[package]]
@@ -35,7 +35,7 @@ wheels = [
 
 [[package]]
 name = "dagflows"
-version = "0.2.0"
+version = "0.2.2"
 source = { editable = "." }
 
 [package.optional-dependencies]