feat(pipeline-run): fall back to file stem for a nameless submit spec

[Silin144]

What

Relaxes OSS submit-time validation so a pipeline spec with no declared name falls back to the pipeline file stem instead of hard-failing with name must be a non-empty string. A declared, non-empty name always wins; only the genuinely-nameless case is filled.

This restores the long-standing tangle-deploy behaviour (submitting my_pipeline.yaml with no name produces a run named my_pipeline) after submit-time validation was consolidated upstream (#28/#30).

Approach — resolve-then-validate

Rather than loosening the schema (which would let genuinely-invalid specs through), the fallback name is resolved into the spec first, and then the fully-resolved spec is validated. Everything else stays strict.

A single shared helper does the resolution:

@staticmethod
def _resolve_fallback_pipeline_name(pipeline_spec, fallback_name):
    # A declared non-empty name always wins.
    # No-op when the spec is not a mapping or no fallback is available.

It is wired into both submit code paths so OSS CLI and downstream (tangle-deploy) submits behave identically:

1. PipelineRunner.prepare_pipeline_for_run (pipeline_runner.py) — validates first in the run lifecycle; passes the hook-resolved pipeline_name (which already honours any initial_pipeline_name override) so it can't reject a nameless-but-otherwise-valid spec before path 2.
2. PipelineRunManager.prepare_submit_payload_from_spec (pipeline_run_manager.py) — the shared submit chokepoint; resolves after apply_run_name_template, before validation.

The run name flows from the spec's name (root_task.componentRef.spec.name), so injecting name into the spec is the correct mechanism — the resulting run is named after the stem.

Worked example

my_pipeline.yaml:

implementation:
  graph:
    tasks: {}

tangle pipeline-runs submit my_pipeline.yaml --no-hydrate → submitted spec name = "my_pipeline" (file stem). If the file instead declares name: Demo Pipeline, the run is named Demo Pipeline (declared name wins over the config stem).

Changes

• pipeline_run_manager.py — shared _resolve_fallback_pipeline_name helper + resolve step in the submit chokepoint.
• pipeline_runner.py — resolve step before first-path validation.
• tests/test_pipeline_runs_cli.py — two tests: stem fallback when nameless, and declared-name-wins-over-stem.

Testing

• Full OSS suite green (774 standalone); ruff clean.

🤖 Generated with Claude Code

[Volv-G]

(AI-assisted) [nit] A whitespace-only declared name skips the fallback. isinstance(name, str) and name treats " " as a valid declared name, so a blank-ish name is kept and the file-stem fallback isn't applied. Matches the literal "non-empty" contract, but a whitespace-only name arguably should also fall back to the stem (name.strip()). Trivial.

[Silin144]

Fixed in 45353fc — switched to name.strip() so a whitespace-only name is treated as nameless and takes the stem fallback, plus a regression test covering the " " case.

[Volv-G]

(AI-assisted) Correct resolve-then-validate approach — injects the file-stem fallback into the spec and keeps validation strict rather than loosening the schema. Verified the shared _resolve_fallback_pipeline_name on PipelineRunManager is reached from both submit paths (PipelineRunner inherits it), and declared-name-wins vs stem-fallback are both tested end-to-end. One trivial nit inline (whitespace-only name). LGTM.