feat(api-workflow): teach connector-event triggers - #3437
Draft
rares-baesu-uipath wants to merge 4 commits into
Draft
rares-baesu-uipath wants to merge 4 commits into
rares-baesu-uipath wants to merge 4 commits into
Conversation
Example connector payloads hardcoded a real personal email address as the recipient. Replace every occurrence with <RECIPIENT_EMAIL>. Per .claude/rules/content-quality.md, any example value carrying the shape of an identifier or personal data must be a placeholder: customer content- inspection gates match on shape alone and block the whole published package. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
An API workflow can start from a connector event (Slack button clicked, new Outlook calendar entry, new Salesforce record) instead of an HTTP call. The skill had no guidance for this and actively told agents triggers could not be authored at all. A trigger is an activity with call: "UiPath.IntSvcEvent" placed first in the root sequence, AND a matching "EventTrigger" entry in bindings_v2.json. That entry is what registers the Orchestrator event trigger on deploy: without it the workflow validates, packs, publishes and deploys cleanly and then never fires, with every gate reporting success. That silent failure is the mistake this guidance exists to prevent. Adds: - references/trigger-authoring-guide.md — discovery flow, the activity shape, the EventTrigger binding contract, JMESPath filter composition, and what polling vs webhooks means for a local run - Critical Rule 16a, a When-to-Use entry, and reference/template navigation - assets/templates/trigger-workflow-template.json (passes validate) - two coder-eval tasks: an offline edit task that grades the stale-binding trap, and an authed e2e that grades the full authoring path Corrects two now-stale claims that trigger types cannot be stubbed, and one retired CLI verb (solution resource refresh -> solution resources refresh, retired at uip 1.196.0). Studio Web derives the binding itself and cannot run the local CLI write commands, so the bindings and local-run passages are enclosed in flavor blocks with sparse studioweb overrides. The flavor contract test swept a fixed list of built references that excluded new files, so the new reference is added to that list; verified non-vacuous. Requires the matching uip CLI support (registry resolve --kind trigger, registry stub for trigger types, bindings sync EventTrigger generation). The e2e task cannot pass until that ships. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
…laims - halve the trigger reference and rule 16a; drop repeated facts - Studio Web debug falls back to debug polling for webhooks triggers - document path/query routing and the Property companion binding Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
…dio Web flavor Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Teaches the
uipath-api-workflowskill how to start a workflow from a connector event (a Slack button click, a new Outlook calendar entry, a new Salesforce record). The skill previously had no guidance for this and actively told agents that triggers could not be authored at all.Two commits — the first is an unrelated sanitization fix, kept separate for review.
The fact this guidance exists to convey
A trigger is an activity with
call: "UiPath.IntSvcEvent"placed first in the root sequence, and a matching"EventTrigger"entry inbindings_v2.json. That entry is what registers the Orchestrator event trigger on deploy.Omit it and the workflow validates, packs, publishes and deploys cleanly — and then never fires. Every gate reports success, so nothing else catches it. That silent failure is the most expensive mistake in this area and is what the new rule, the guide and the offline test all point at.
Added
references/trigger-authoring-guide.md— discovery flow, the activity shape, theEventTriggerbinding contract, JMESPath filter composition, and what polling vs webhooks means for a local runassets/templates/trigger-workflow-template.json— passesuip api-workflow validateCorrected
solution resource refresh→solution resources refresh(retired at uip 1.196.0), including one pre-existing occurrenceStudio Web flavor
Studio Web derives the binding itself and cannot run the local CLI write commands, so the bindings and local-run passages are enclosed in flavor blocks with sparse overrides.
Worth a look during review: the flavor contract test swept a fixed list of built references that excluded any new file, so the new reference initially leaked CLI-only guidance into the Studio Web build with nothing to catch it. The new reference is now on that list, and the guard was checked to be non-vacuous (planting a banned command in the canonical guide fails the test).
Verification
skills:validate,skills:check-links(6866 links),check-skill-status,check-skills-sh,check-cli-verbs— all OK"Status": "Valid"skills:testUpdate coded skill to use uipcli instead of rpa-tool #20 (pack builds complete…) fails here, but it fails identically on a clean tree — pre-existing, unrelatedDependency
Requires the matching
uipCLI support (registry resolve --kind trigger,registry stubfor trigger types,bindings syncEventTrigger generation) — UiPath/cli#4365. That should merge and ship first: the e2e task drives the real CLI and cannot pass against a published build without--kind. The offline task is unaffected.🤖 Generated with Claude Code