test: D1-10 cover TransformPipelineContinuation with first-transform failure

SPECGAPS.md

@@ -124,7 +124,7 @@ All reconciled to follow code. Specs must be self-consistent and match code.
 | D1-7 | ~~LiquidOperatorSubset~~ | template.allium:210 | **Resolved:** `LiquidParser.validate/1` now walks the parsed AST for `{:op, _}` nodes and rejects any comparison operator outside the allowed `==`/`>` subset (`!=`, `<`, `>=`, `<=`, `contains`), sharing the publish gate and MCP `validate_template` surface with the tag/filter checks; spec `LiquidOperatorSubset` annotated with enforcement note; 10 tests added (5 unsupported operators rejected at publish, 5 supported `==`/`>`/`and`/`or`/bare-truthy expressions accepted). |
 | D1-8 | ~~MacroTimeout guarantee~~ | script.allium:94-95 | **Resolved:** added test in `api_test.exs` — an infinite-loop `render()` macro run with `max_reductions: :none` (forces the luerl sandbox onto its wall-clock path) and a 150ms `timeout` returns `{:error, :timeout}` and terminates within budget (<2s), proving the macro is killed near its budget rather than the default multi-minute script timeout |
 | D1-9 | ~~ExecuteTransform rule (pipeline, ordering, toast budget)~~ | script.allium:229-263 | **Resolved:** the `ExecuteTransform` rule had no engine — added `BDS.Scripts.Transforms.run/3` (+ `Scripts.list_transform_scripts/1` ordered by updated_at→slug→id and `Scripts.resolved_content/1`). The pipeline runs enabled project transforms sequentially on the blogmark candidate with a `{source="blogmark", url}` context, captures per-script errors without rolling back the last valid candidate (TransformPipelineContinuation), and enforces the toast budget (`transform_max_toasts_per_script`/`transform_max_toasts_total`/`transform_max_toast_length`, new config keys). 6 tests added (ordering, project/disabled scoping, continuation, context, per-script + total toast caps with truncation). Deep-link OS routing into this engine remains future work. |
-| D1-10 | TransformPipelineContinuation | script.allium:247-249 | Write test: error in transform doesn't halt pipeline |
+| D1-10 | ~~TransformPipelineContinuation~~ | script.allium:247-249 | **Resolved:** added focused test in `transforms_test.exs` — a failing *first* transform (no prior valid state) does not halt the pipeline: the original input survives, a later enabled transform still runs against it, and every failure is captured per-script in pipeline order tagged with its slug |
 | D1-11 | ChatContextTruncation invariant | ai.allium:375-379 | Write test: long chat history trimmed to context window |
 | D1-12 | BoundedToolLoop enforcement | ai.allium:381-385 | Write test: tool rounds bounded by chat_max_tool_rounds |
 | D1-13 | DiscardPostChangesSideEffects | engine_side_effects.allium:99-104 | Write test: FTS updated after discard |

test/bds/scripts/transforms_test.exs

@@ -167,6 +167,54 @@ defmodule BDS.Scripts.TransformsTest do
     assert [%{reason: _}] = result.errors
   end
 
+  test "a failing first transform does not halt the pipeline (TransformPipelineContinuation)", %{
+    project: project
+  } do
+    # First transform fails before producing any valid state; later transforms
+    # must still run against the original input, and each failure is captured
+    # with its slug without halting the pipeline.
+    boom =
+      transform(project.id, "Boom", """
+      function main(_data, _ctx)
+        error("boom")
+      end
+      """)
+
+    Process.sleep(5)
+
+    transform(project.id, "Append", """
+    function main(data, _ctx)
+      data.content = data.content .. "A"
+      return data
+    end
+    """)
+
+    Process.sleep(5)
+
+    kaboom =
+      transform(project.id, "Kaboom", """
+      function main(_data, _ctx)
+        error("kaboom")
+      end
+      """)
+
+    data = %{
+      "title" => "t",
+      "content" => "seed",
+      "tags" => [],
+      "categories" => [],
+      "url" => "http://x"
+    }
+
+    assert {:ok, result} = Transforms.run(project.id, data)
+    # Original input survives the first failure; the surviving transform still runs.
+    assert result.data["content"] == "seedA"
+    # Both failures are captured, in pipeline order, each tagged with its slug.
+    assert [%{slug: first_slug}, %{slug: second_slug}] = result.errors
+    assert first_slug == boom.slug
+    assert second_slug == kaboom.slug
+  end
+
   test "receives blogmark context with source and originating url", %{project: project} do
     transform(project.id, "Ctx", """
     function main(data, ctx)