fix(uipath-ixp): hard DO-NOT-TRIGGER for flow work in frontmatter description#2101
Conversation
…cription The body-level "When NOT to Use This Skill" deferral (RE-12440) is only read after the skill activates; skill selection happens on the frontmatter description, where the deferral was one soft trailing sentence that lost to the strong positive triggers during flow discovery. Replace it with an explicit DO NOT TRIGGER clause (same idiom as uipath-maestro-flow). ACTV-89470 Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_0164ykfgunBCrvhrUMHbaGEQ
|
Claude finished @andrei-uipath's task in 2m 19s —— View job PR Review: fix(uipath-ixp): hard DO-NOT-TRIGGER for flow work in frontmatter description
SummarySingle-line frontmatter change in Change-by-Change Review1.
|
| Area | Status | Notes |
|---|---|---|
| Frontmatter | OK | Valid YAML, 751 chars, identity front-loaded, hard negative trigger matches repo idiom |
| E2E Tests | OK | No new capabilities added — existing tests unaffected |
| Skill Body | OK | Unchanged, body-level deferral already in place |
| References & Assets | OK | No reference changes needed |
| Repo Hygiene | OK | Scoped change, no secrets, no cross-skill deps |
Issues for Manual Review
- Activation-gate eval: The hard
DO NOT TRIGGERis the right fix at the description level, but the actual misfire rate (ACTV-89470) should be validated with an activation eval to confirm the new wording reliably prevents the false trigger. Theactivation-gate.ymlCI workflow should cover this if configured for this skill.
Conclusion
Clean, well-scoped fix. The change correctly escalates a soft trailing redirect to a hard negative trigger in the frontmatter description — the only place where skill selection happens. The idiom is consistent with how uipath-maestro-flow already handles the symmetric exclusion. No issues found. Approve.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_0164ykfgunBCrvhrUMHbaGEQ
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_0164ykfgunBCrvhrUMHbaGEQ
richsilv
left a comment
There was a problem hiding this comment.
I think @rockymadden will want to see this given that it's a change to front-matter.
Address review feedback (#2101): "standalone" is vague and adds no precision — the positive scope is already carried by the capability list, and the exclusion is stated precisely by the DO NOT TRIGGER clause. Remove the qualifier and rely on the negative trigger. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
richsilv
left a comment
There was a problem hiding this comment.
Looks good to me. Not sure if you need a coding agents team reviewer to change the front matter.
Skill selection happens on the frontmatter description alone — the body-level "When NOT to Use This Skill" deferral shipped in #1348 (RE-12440) is only read after uipath-ixp activates, so it never prevents the misfire ACTV-89470 describes: agents picking uipath-ixp during flow discovery. In the description, the deferral was one soft trailing sentence ("For IxP nodes or model listings inside a .flow / Maestro flow→uipath-maestro-flow") that loses to the strong positive triggers ("pull metrics and model versions, publish/tag/roll back model versions").
This replaces it with a hard negative trigger in the same idiom uipath-maestro-flow already uses: "DO NOT TRIGGER during .flow / Maestro Flow work — discovering or listing IxP / document-extraction models, extractors, or nodes available to Maestro Flow, and adding or wiring an IxP node, belong to uipath-maestro-flow even when they sound like IXP model management." (Per review, the earlier "Standalone IXP projects ONLY" qualifier was dropped as ambiguous — the capability list already carries the positive scope and the negative trigger states the exclusion precisely.) Description is 727 chars (cap 1024); body text unchanged.
🤖 Generated with Claude Code
https://claude.ai/code/session_0164ykfgunBCrvhrUMHbaGEQ