Skip to content

Recognize zeebe:adHoc#outputCollection#69

Open
nikku wants to merge 6 commits intomainfrom
ad-hoc-output-mapping
Open

Recognize zeebe:adHoc#outputCollection#69
nikku wants to merge 6 commits intomainfrom
ad-hoc-output-mapping

Conversation

@nikku
Copy link
Member

@nikku nikku commented Feb 13, 2026

Proposed Changes

Support <zeebe:adHoc outputCollection="..." /> definition, to make variables visible in agentic use-cases:

image

Defines the following variables (assuming toolCallResult and toolCallResults are AI Agent local variables:

[
  { name: 'agent', origin: [ 'AI_Agent' ], scope: 'ai-agent-chat-with-tools' },
  { name: 'toolCallResult', origin: [ 'AI_Agent', 'GetDateAndTime', 'SuperfluxProduct' ], scope: 'AI_Agent' },
  { name: 'toolCallResults', origin: [ 'AI_Agent' ], scope: 'AI_Agent' },
  { name: 'data.response.includeAgentContext' },
  { name: 'data.response.includeAssistantMessage' },
  { name: 'data.response.format.parseJson' },
  { name: 'data.response.format.type' },
  { name: 'data.events.behavior' },
  { name: 'data.limits.maxModelCalls' },
  { name: 'data.memory.contextWindowSize' },
  { name: 'data.memory.storage.type' },
  { name: 'agentContext', origin: [ 'AI_Agent' ], scope: 'AI_Agent' },
  { name: 'data.userPrompt.documents' },
  { name: 'data.userPrompt.prompt' },
  { name: 'data.systemPrompt.prompt' },
  { name: 'provider.openai.model.model' },
  { name: 'provider.openai.authentication.apiKey' },
  { name: 'provider.type' }
]

Depends on bpmn-io/extract-process-variables#34

Related to camunda/camunda-modeler#5606

Checklist

Ensure you provide everything we need to review your contribution:

  • Contribution meets our definition of done
  • Pull request establishes context
    • Link to related issue(s), i.e. Closes {LINK_TO_ISSUE} or Related to {LINK_TO_ISSUE}
    • Brief textual description of the changes
    • Screenshots or short videos showing UI/UX changes
    • Steps to try out, i.e. using the @bpmn-io/sr tool

@bpmn-io-tasks bpmn-io-tasks bot added the in progress Currently worked on label Feb 13, 2026
@nikku nikku force-pushed the ad-hoc-output-mapping branch from 9e23ca9 to 7402bb0 Compare February 13, 2026 21:48
@nikku nikku force-pushed the ad-hoc-output-mapping branch from 7402bb0 to 9845abc Compare February 13, 2026 22:00
@nikku nikku marked this pull request as ready for review February 13, 2026 22:01
Copilot AI review requested due to automatic review settings February 13, 2026 22:01
@bpmn-io-tasks bpmn-io-tasks bot added needs review Review pending and removed in progress Currently worked on labels Feb 13, 2026
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR extends the Zeebe variable extraction test suite to cover additional scoping/mapping scenarios, including agentic ad-hoc sub-processes using zeebe:adHoc#outputCollection, and updates a few related dependencies.

Changes:

  • Add new Zeebe BPMN fixtures + spec coverage for sub-process scoping edge cases, connector output mapping, connector sub-process scoping, and agentic ad-hoc sub-process variables.
  • Improve variableEqual assertion diagnostics in tests.
  • Bump @bpmn-io/lezer-feel and @bpmn-io/properties-panel (plus related lockfile updates).

Reviewed changes

Copilot reviewed 9 out of 10 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
test/spec/zeebe/ZeebeVariableResolver.spec.js Adds/adjusts tests for new scoping + agentic ad-hoc coverage; switches some calls to pass diagram elements.
test/globals.js Updates the custom variableEqual chai assertion to provide more descriptive failure messages.
test/fixtures/zeebe/sub-process.no-output-mapping.bpmn New fixture for sub-process scoping without output mapping.
test/fixtures/zeebe/connectors.sub-process.bpmn New fixture for connectors inside a sub-process.
test/fixtures/zeebe/connectors.output-mapping.bpmn New fixture for connector output mappings.
test/fixtures/zeebe/connectors.bpmn Extends existing connector fixture with an additional participant/sub-process scenario.
test/fixtures/zeebe/complex.sub-process-mapping-conflict.bpmn New fixture for conflicting mapping behavior.
test/fixtures/zeebe/ad-hoc-sub-process.agentic.bpmn New agentic ad-hoc sub-process fixture using zeebe:adHoc outputCollection.
package.json Dependency bumps for FEEL parser + properties panel.
package-lock.json Lockfile updates reflecting dependency bumps.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

* outputCollection defines variable
* validate agentic use-case
@nikku nikku force-pushed the ad-hoc-output-mapping branch from 9845abc to f04ca51 Compare February 13, 2026 22:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs review Review pending

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant