Fix skipped HDA missing hid in _create_skipped_output

Use history.add_dataset() to properly assign hid via _next_hid()
instead of passing history= to constructor which skips hid assignment.
Fixes __strict_check_before_flush__ validation in CI.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
John Chilton
2026-03-22 19:15:04 -04:00
co-authored by Claude Opus 4.6
parent c810dcc37d
commit bce5b360c8
+1 -1
View File
@@ -2127,10 +2127,10 @@ class PickValueModule(WorkflowModule):
history = invocation.history
hda = model.HistoryDatasetAssociation(
name="Pick Value - skipped",
history=history,
create_dataset=True,
flush=False,
)
history.add_dataset(hda)
object_store_populator = ObjectStorePopulator(trans.app, trans.user)
hda.set_skipped(object_store_populator, replace_dataset=False)
trans.sa_session.add(hda)