docs(guardrails): document BYOG admin configuration [AL-513] - #2550
docs(guardrails): document BYOG admin configuration [AL-513]#2550apetraru-uipath wants to merge 2 commits into
Conversation
…yo-configurations) Split out of the AL-513 agent-authoring PR so this platform-owned reference doesn't need extra CODEOWNERS approval on that PR. New reference: skills/uipath-platform/references/guardrails/byo-configurations.md, covering the full lifecycle (list/create/update/delete) of bring-your-own guardrail (BYOG) configurations — the tenant-admin side of registering an external guardrail provider against an Integration Service connection. Mirrors the existing BYO LLM Gateway doc's structure and conventions. Key semantics documented: ValidatorName is unique per tenant and is the only value agents reference (no connection_id passed by the agent — the platform resolves the connection server-side); update has merge semantics with ValidatorName/ValidatorType locked at creation; delete requires --force; the 404 split between ByoGuardrailsUnavailable (feature flag off) and ByoGuardrailConfigurationNotFound (bad id); the endpoint requires an org-admin user token and rejects application tokens. Wired into skills/uipath-platform/SKILL.md (frontmatter triggers, command table, references list). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
Claude finished @apetraru-uipath's task in 4m 12s —— View job PR Review: docs(guardrails): document BYOG admin configuration
SummaryAdds Change-by-Change Review1.
|
| Area | Status | Notes |
|---|---|---|
| Frontmatter | Issue | description is 1038 chars, 14 over the 1024-char limit |
| E2E Tests | OK | Docs-only change; agent behavior tested via #2516 |
| Skill Body | OK | Bullet, table rows, and reference link are correct bridge content |
| References & Assets | OK | New reference is thorough, well-structured, and correctly linked |
| Repo Hygiene | OK | No secrets, no cross-skill file references, changes scoped to uipath-platform |
Issues for Manual Review
- CLI command accuracy: Verify that
uip guardrails byo-configurationssubcommands (list/create/update/delete) and their flags match the actual CLI. The--forcerequirement ondelete,--disabled/--enabledmutual exclusivity, and the merge semantics ofupdateshould be spot-checked against the implementation. - Error code names: Confirm
ByoGuardrailsUnavailable,ByoGuardrailConfigurationNotFound,ByoGuardrailConfigurationCreated,ByoGuardrailConfigurationUpdated,ByoGuardrailConfigurationDeleted, andByoGuardrailConfigurationsListare the actual response codes. - CODEOWNERS ownership: Decide whether
/skills/uipath-platform/references/guardrails/needs a dedicated entry for a guardrails-specific team, or whether the platform-team parent entry is intentionally correct.
Conclusion
One High issue: the frontmatter description exceeds the 1024-char repo cap by 14 characters — this needs trimming before merge to pass the validation hook. The reference document itself is clean and well-structured. Recommend fixing the description length and optionally adding a CODEOWNERS entry for the guardrails subdirectory.
…before-save gap Review (valentinabojan): byoConfigurationId is not a real field an agent uses — ByoValidatorName is what both low-code and coded agents reference a BYOG configuration by. The doc's own top-line summary already said "agents reference by ValidatorName alone"; the Id-field row and the low-code bullet contradicted it. Fixed both, and the Id field's row now says plainly that agent authors don't reference it directly. Also added a "Known gap" section per review: unlike BYO LLM connections, byo-configurations create/update run no server-side validation of the connection or provider before saving — confirmed against the CLI source (no validate/probe call in guardrails-tool, unlike llmgw-tool's mandatory POST .../validate). A CLI-side validate verb is a real feature gap, not something a docs PR can add; `is connections ping` is noted as the closest partial check available today (confirms the connection is alive, not that it implements the guardrail contract). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
This file/doc moved here from #2516 (agent-authoring PR) so it doesn't need
|
…ationId Same review correction (valentinabojan, on the admin-doc PR #2550) applies here too, and this is the more important place to fix it — this is what actually teaches an agent how to write agent.json. byoConfigurationId was never a real field in the guardrail schema; ByoValidatorName is what both low-code and coded agents reference a BYOG configuration by, and it's unique per tenant so no id lookup is needed. Fixed across: lowcode guardrails.md (mapping table, authoring guidance, anti-pattern rule), lowcode + review disambiguation notes, and the byog_pinning test (task description + check script's actual match key, re-verified against positive/negative stubs — including a stub asserting the old byoConfigurationId-only shape now correctly fails). Left untouched: mentions of ByoConfigurationId as a real discovery-output field (`uip agent guardrails list --byo`, `uip guardrails byo-configurations list`) — that field genuinely exists for admin-side identification, it's just not what the guardrail JSON authors. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
What changed?
Split out of #2516 (agent-authoring side of bring-your-own guardrails) so this platform-owned reference doesn't need to ride on that PR's approvals.
skills/uipath-platform/references/guardrails/byo-configurations.md— the tenant-admin side of BYOG: the fulluip guardrails byo-configurationslifecycle (list/create/update/delete), mirroring the existing BYO LLM Gateway doc's structure.skills/uipath-platform/SKILL.md: frontmatter triggers, command table, references list wired to the new doc.Key semantics documented:
ValidatorNameis unique per tenant and is the only value agents reference — noconnection_idpassed by the agent; the platform resolves the connection server-side.updatehas merge semantics;ValidatorName/ValidatorTypeare locked at creation (delete + re-create to change them).deleterequires--force.ByoGuardrailsUnavailable(feature flag off) vs.ByoGuardrailConfigurationNotFound(bad id).#2516 already references this content only via the skill-level
/uipath:uipath-platformlink (never a literal file path), so it isn't blocked on this PR merging first — no cross-PR ordering dependency.How has this been tested?
scripts/check-skill-verbs.py: no hard-blocking (matched_prefix: null) findings against the current catalog (uip guardrails byo-configurations *resolves).scripts/check-task-driver.py: clean.byog_pinningin docs(guardrails): document bring-your-own (BYOG) guardrail configurations [AL-513] #2516.Are there any breaking changes?