Skip to content

Add PipelineDehydrator(extract_subgraphs=False) seam (v0.1.30) - #75

Merged
Volv-G merged 1 commit into
masterfrom
piforge/tangle-pipeline-crud/dehydrator-extract-subgraphs-opt-b7f418e
Oct 3, 2026
Merged

Volv-G merged 1 commit into
masterfrom
piforge/tangle-pipeline-crud/dehydrator-extract-subgraphs-opt-b7f418e

Conversation

@Volv-G

@Volv-G Volv-G commented Oct 3, 2026 •

Copy link
Copy Markdown
Collaborator

(AI-assisted)

Part 1 of 2 in a stack. Part 2: #76 (YAML→Python decompiler, v0.1.31), based on this branch.

What

A keyword-only PipelineDehydrator(extract_subgraphs: bool = True) option, plus the v0.1.30 version bump.

False keeps every nested-graph boundary inline and dehydrates only the leaf components inside it, in place. It is for callers that represent nested graphs themselves; the decompiler in part 2 turns each one into a Python @pipeline.

Evidence

  • The default is True, so ordinary dehydration and pipelines dehydrate are byte-for-byte unchanged. No existing test line is changed or removed.
  • One new parametrized test covers DIGEST, FILE and AUTO: the boundary stays inline, the leaf is dehydrated, and no subgraph file is written.

How

The new branch sits in dehydrate() before the existing AUTO/FILE/URL and DIGEST/NAME subgraph-extraction branches, which are otherwise unchanged.

Failure modes

Only callers that pass extract_subgraphs=False are affected, and they get a deliberately not-fully-dehydrated document.

Testing

  • 3.10 and 3.12 full non-packaging suites: 2090 passed each.
  • Packaging: 10/10. Locally this needed the uv index config bypassed; that is a local issue, not a repo one.
  • ruff: clean on the touched files. The repo-wide count is 36, the same as master.
  • git diff --check: clean.
  • Independent local review found no blocking findings (option-ignored mutant killed).

Review focus

The option's semantics and its placement in dehydrate().

Keyword-only, default True: ordinary dehydration is unchanged. False keeps every
nested-graph boundary inline and dehydrates only the leaf components inside it,
in place, for callers that represent nested graphs themselves (the YAML->Python
decompiler turns each into a Python @pipeline).
@Volv-G
Volv-G requested a review from Ark-kun as a code owner October 3, 2026 01:56
@Volv-G
Volv-G merged commit 1faa8a4 into master Oct 3, 2026
6 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