Skip to content

feat(uipath-platform): document tenant licensing CLI commands [PLT-100796]#605

Open
biftime wants to merge 1 commit intomainfrom
feature/licensing-skills
Open

feat(uipath-platform): document tenant licensing CLI commands [PLT-100796]#605
biftime wants to merge 1 commit intomainfrom
feature/licensing-skills

Conversation

@biftime
Copy link
Copy Markdown

@biftime biftime commented May 6, 2026

Add reference for uip platform tenants show-licenses and allocate-licenses, covering overlay semantics, product code translation, and a conversational workflow for handling friendly-name intent. Wire the new reference into SKILL.md task navigation and the uip command index.

Add reference for `uip platform tenants show-licenses` and `allocate-licenses`,
covering overlay semantics, product code translation, and a conversational
workflow for handling friendly-name intent. Wire the new reference into
SKILL.md task navigation and the uip command index.

[PLT-100796]

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 6, 2026

Claude finished @biftime's task in 1m 55s —— View job


Summary

Adds a new reference doc (references/platform/licensing.md) documenting uip platform tenants show-licenses and allocate-licenses commands, wires it into SKILL.md's Task Navigation table and CLI Overview, and adds a matching section to the uip-commands.md index.

Change-by-Change Review

1. skills/uipath-platform/references/platform/licensing.md (new file, +187 lines)

Severity: Low

Well-structured reference. Concepts section clearly explains overlay semantics, service license auto-routing, and the current-interval filter — all non-obvious behaviors an agent needs to get right. The product code translation table and conversational workflow are solid LLM-usability patterns.

Issues found:

  • Missing trailing newline (line 187): File ends without a final newline. Most editors and git diff will flag this with \ No newline at end of file.
    Fix this →

  • Code block comments could be stripped (lines 79, 82, 86, 137): Per token-optimization.md, comments like # Full summary, # Just one product, # Just the codes the tenant has, and # Verify restate surrounding prose. The section headings and workflow step 8 ("Verify") already convey intent. Consider removing these per repo token-optimization rules, though this is minor since they do aid scanability within the code blocks.

2. skills/uipath-platform/SKILL.md (modified, +3 lines)

Severity: Low

Three clean additions:

  • Line 19: "When to Use" bullet for licensing — appropriate scope, not overlapping other bullets.
  • Line 93: Task Navigation row linking to the new reference — correct relative path, resolves to an existing file.
  • Line 193: CLI Overview table row for platform command group — consistent with existing table format.

One gap:

  • References section not updated (lines 265–273): The bottom ## References section lists top-level reference categories (Orchestrator, Resources, Solutions, etc.) but does not include a Platform entry pointing to references/platform/licensing.md. While the Task Navigation table covers discoverability, the References section is the canonical index — adding a line like - **[Platform](references/platform/licensing.md)** — Tenant licensing (view, allocate) would keep it consistent.
    Fix this →

3. skills/uipath-platform/references/uip-commands.md (modified, +10 lines)

Severity: OK

New ## Platform section (lines 87–94) follows the exact pattern of sibling sections (Solution, Traces). Relative link platform/licensing.md resolves correctly from the references/ directory. Table format is consistent with existing command groups. Placement between Solution and Traces is logical.

What's Missing

  • No test task for the new licensing capability. tests/tasks/uipath-platform/ only has integration-service/ tasks. There's no smoke or e2e task covering licensing commands. Flagging as Medium — consistent with repo-wide gap (most existing capabilities also lack test tasks).
  • References section entry in SKILL.md as noted above (Low).
  • Trailing newline in licensing.md (Low).

Area Ratings

Area Status Notes
Frontmatter OK name matches folder, description is 244 chars, well within 1024 limit
E2E Tests Issue (Medium) No test tasks for licensing commands — consistent with existing gaps
Skill Body OK Task Navigation, CLI Overview, and When to Use updated correctly
References & Assets OK New reference is well-structured, all links resolve, no cross-skill refs
Repo Hygiene OK CODEOWNERS covers /skills/uipath-platform/, no secrets, no binary files, changes scoped to skill

Issues for Manual Review

  • CLI command accuracy: Verify that uip platform tenants show-licenses and uip platform tenants allocate-licenses are the actual command names, flags (--input, --organization), and output field names (availableForAllocation, allocatedAcrossOtherTenants, etc.) match the real CLI behavior.
  • Product code table completeness: The 17-entry product code → friendly name mapping table may be incomplete or may drift as new products are added. Confirm this is the current set.
  • Error messages: The 6 error messages in the Error Modes table should be validated against actual CLI error output.

Conclusion

Clean, well-structured addition. The licensing reference follows repo conventions, is prescriptive and LLM-friendly, and correctly wires into the skill's navigation. Two minor fixes recommended (trailing newline, References section entry). No blocking issues — approve with minor suggestions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants