improvement(provenance): inherit copied-path reasons once per copy

waleedlatif1 · waleedlatif1 · commit 335f2c10f64d · 2026-08-09T20:14:45.000-07:00
Semantics are unchanged — reasons are still inherited only when at least one
incomplete path was actually copied — but the source set is walked once rather
than once per copied path.
diff --git a/apps/sim/executor/utils/resolved-secret-trace-registry.ts b/apps/sim/executor/utils/resolved-secret-trace-registry.ts
@@ -1818,11 +1818,13 @@ export class ResolvedSecretTraceRegistry {
     target: ResolvedSecretTraceRegistry,
     roots?: readonly ResolvedSecretInputPath[]
   ): void {
+    let copied = false
     for (const [key, path] of this.incompleteInputPaths) {
       if (roots && !roots.some((root) => inputPathsOverlap(path, root))) continue
       target.incompleteInputPaths.set(key, [...path])
-      target.inheritIncompletenessReasonsFrom(this)
+      copied = true
     }
+    if (copied) target.inheritIncompletenessReasonsFrom(this)
   }
 
   private addActiveEntry(entry: ActiveSecretEntry, options: { propagated?: boolean } = {}): void {

Original file line number	Diff line number	Diff line change
`@@ -1818,11 +1818,13 @@ export class ResolvedSecretTraceRegistry {`
`1818`	`1818`	`target: ResolvedSecretTraceRegistry,`
`1819`	`1819`	`roots?: readonly ResolvedSecretInputPath[]`
`1820`	`1820`	`): void {`
	`1821`	`+ let copied = false`
`1821`	`1822`	`for (const [key, path] of this.incompleteInputPaths) {`
`1822`	`1823`	`if (roots && !roots.some((root) => inputPathsOverlap(path, root))) continue`
`1823`	`1824`	`target.incompleteInputPaths.set(key, [...path])`
`1824`		`- target.inheritIncompletenessReasonsFrom(this)`
	`1825`	`+ copied = true`
`1825`	`1826`	`}`
	`1827`	`+ if (copied) target.inheritIncompletenessReasonsFrom(this)`
`1826`	`1828`	`}`
`1827`	`1829`
`1828`	`1830`	`private addActiveEntry(entry: ActiveSecretEntry, options: { propagated?: boolean } = {}): void {`