fix(executor): fix dependency resolution, allow blocks with multiple inputs to execute

[waleedlatif1]

Description

• the dependency checker was incorrectly waiting for ALL incoming connections to be satisfied (including the non-selected API path), so we fixed it by checking if a source block is intentionally excluded from execution first, which automatically satisfies that dependency.

Type of change

• Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Ran the workflow pictured below and it worked, despite not having a value for one of the referenced blocks later in the execution.

Checklist:

• My code follows the style guidelines of this project
• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have added tests that prove my fix is effective or that my feature works
• All tests pass locally and in CI (bun run test)
• My changes generate no new warnings
• Any dependent changes have been merged and published in downstream modules
• I have updated version numbers as needed (if needed)
• I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

Security Considerations:

• My changes do not introduce any new security vulnerabilities
• I have considered the security implications of my changes

Additional Information:

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
sim	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jul 1, 2025 4:27pm

1 Skipped Deployment

Name	Status	Preview	Comments	Updated (UTC)
docs	⬜️ Skipped (Inspect)			Jul 1, 2025 4:27pm

[greptile-apps]

PR Summary

Fixed a critical bug in the workflow execution engine where blocks with multiple inputs were incorrectly blocked from executing when some inputs came from inactive paths.

• Modified PathTracker in apps/sim/executor/path.ts to add recursive activateDownstreamPaths for proper path activation
• Updated dependency resolution in apps/sim/executor/index.ts to check for intentionally inactive paths before dependency validation
• Added comprehensive test coverage in apps/sim/executor/path.test.ts for router downstream path activation scenarios
• Fixed potential cyclic workflow issues by implementing loop prevention in path activation logic

_{4 files reviewed, 2 comments
Edit PR Review Bot Settings | Greptile}

[delve-auditor]

✅ No security or compliance issues detected. Reviewed everything up to c04e047.

Security Overview

• 🔎 Scanned files: 4 changed file(s)

Detected Code Changes

Change Type	Relevant files
Bug Fix	► executor/index.ts     Fix dependency resolution logic for blocks with multiple inputs ► executor/index.test.ts     Add tests for dependency checking logic ► executor/path.ts     Improve router downstream path activation ► executor/path.test.ts     Add tests for router downstream path activation

Reply to this PR with @delve-auditor followed by a description of what change you want and we'll auto-submit a change to this PR to implement it.