Skip to content

ci(evals): add fork-vs-handoff cache demo workflow (workflow_dispatch) - #5998

Open
Thushanth Bengre (thushanth-bengre-langchain) wants to merge 10 commits into
mainfrom
bengret/fork-cache-demo-workflow
Open

ci(evals): add fork-vs-handoff cache demo workflow (workflow_dispatch)#5998
Thushanth Bengre (thushanth-bengre-langchain) wants to merge 10 commits into
mainfrom
bengret/fork-cache-demo-workflow

Conversation

@thushanth-bengre-langchain

Adds a manual-only GitHub Actions workflow (fork_cache_demo.yml) that quantifies the prompt-cache benefit of forking the general-purpose subagent: it runs a real PR review through 5 delegated lenses (correctness, backcompat, tests, performance, api_design) once per branch leg. Fork vs. handoff is driven entirely by which branch's deepagents is checked out.
Metrics (cache-read tokens, token/tool-call cost, wall-clock, delegation-directive size) come from a local callback handler, not LangSmith; Provider API keys are gated by inputs.model's prefix, matching _harbor_run.yml's convention.

@github-actions github-actions Bot added github_actions PR touching `.github` internal User is a member of the `langchain-ai` GitHub organization size: L 500-999 LOC labels Aug 31, 2026

@open-swe open-swe Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Open SWE Review found 2 potential issues.

Open in WebView Open SWE trace

Comment thread .github/workflows/fork_cache_demo.yml Outdated
Comment thread .github/scripts/evals/fork_cache_demo.py Outdated

@corridor-security corridor-security Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The workflow directly interpolates workflow_dispatch inputs into bash run blocks in the demo job (lines 131, 165–170), enabling command injection by anyone who can trigger the workflow and pass the evals environment gate — with provider API keys in scope.

Comment thread .github/workflows/fork_cache_demo.yml
@thushanth-bengre-langchain Thushanth Bengre (thushanth-bengre-langchain) changed the title eval: add fork-vs-handoff cache demo workflow (workflow_dispatch) ci: add fork-vs-handoff cache demo workflow (workflow_dispatch) Aug 31, 2026
@github-actions github-actions Bot added the infra Repo meta changes label Aug 31, 2026
@thushanth-bengre-langchain Thushanth Bengre (thushanth-bengre-langchain) changed the title ci: add fork-vs-handoff cache demo workflow (workflow_dispatch) ci(evals): add fork-vs-handoff cache demo workflow (workflow_dispatch) Aug 31, 2026
@github-actions github-actions Bot added the evals Evaluation suite and Harbor integration label Aug 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

evals Evaluation suite and Harbor integration github_actions PR touching `.github` infra Repo meta changes internal User is a member of the `langchain-ai` GitHub organization size: L 500-999 LOC

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant