docs: update Claude docs from PR review analysis

[claude]

Summary

Weekly analysis of PR comments (2026-06-08 -> 2026-06-15).
Analyzed 19 PRs with resolved review threads. Found 6 actionable insights.

Changes

agent_docs/rules.md

• @link in @param is redundant — TypeDoc auto-links parameter types
  Source: PR feat(agents): add AgentMemory service (memory timeline, calls timeline, top spaces) #483, PR feat(agents): Agents service foundation + getAll (1/5) [PLT-103787] #492 — reviewer noted {@link TypeName} in @param lines adds noise; TypeDoc links parameter types automatically. Explicit {@link} is only needed in @returns.

• Sibling methods sharing a configuration pattern (e.g., excludeFromPrefix) each need independent tests for that config
  Source: PR feat(agents): getErrors — paginated agent errors (2/5) [PLT-103787] #503 — getIncidents was missing the OData-prefix exclusion test that getAll had. If excludeFromPrefix were dropped from one method, no unit test would catch it unless each method's suite independently verifies the behavior.

agent_docs/conventions.md

• Time-range parameters must be Date objects, not strings
  Source: PR feat(agents): Agents service foundation + getAll (1/5) [PLT-103787] #492, PR feat(agents): add AgentMemory service (memory timeline, calls timeline, top spaces) #483 — multiple reviewers (deepeshrai-tech, ninja-shreyash) flagged startTime/endTime as strings; the SDK should accept Date objects and convert to the API format internally, matching the Maestro getTopRunCount() precedent.

• Omit vs ? optional for fields with server-side defaults
  Source: PR feat(tasks): support assignment criteria for group task assignment #499 — reviewer (Sandeepan-Ghosh-0312) flagged Omit<..., 'assignmentCriteria'> when the field has a server default (SingleUser). Using Omit implies the field causes errors if included; ? optional is correct when the API simply defaults the value.

• Named wrapper types for pure array responses are unnecessary
  Source: PR feat(agents): add AgentMemory service (memory timeline, calls timeline, top spaces) #483, PR feat(governance): add Governance getPolicyTraces service #464 — reviewer (ninja-shreyash) asked to use T[] directly when the response is a plain collection with no pagination metadata. Creating a named type just to hold items: T[] adds indirection with no value.

• Header-only options must be destructured before calling PaginationHelpers.getAll()
  Source: PR feat(data-fabric): add folder support to entity schema and attachment ops #490 — folderKey leaked into GET query params and POST body because it wasn't destructured out of options before PaginationHelpers.getAll(). The pattern: const { folderKey, ...pagedOptions } = options ?? {}, then pass folderKey via createHeaders().

agent_docs/architecture.md

• Subpath exports are always flat — nested paths break core import resolution
  Source: PR feat(agents): add AgentMemory service (memory timeline, calls timeline, top spaces) #483 — discussion resolved that /agents/memory conflicts with the core import pattern; the correct approach is a flat dash-separated name like @uipath/uipath-typescript/agent-memory.

No changes

• CLAUDE.md — no relevant insights found
• Agents.md — no relevant insights found

PRs Analyzed

PR	Title	Comments
#520	test(action-center): assign tasks to a configured group in integration tests	2
#515	fix: traces docs	0
#507	docs: add SETUP.md for Agentic Inner Loop KPI	1
#509	feat(data-fabric): add includeAllFolders option to entities.getAll	0
#503	feat(agents): getErrors — paginated agent errors (2/5)	4
#499	feat(tasks): support assignment criteria for group task assignment	13
#472	feat(buckets): support folderKey/folderPath in file ops	8
#506	chore: bump version to 1.4.0	0
#483	feat(agents): add AgentMemory service	15
#493	doc: integration testing guide	4
#501	fix(data-fabric): make importRecordsById public	1
#497	feat(telemetry): report CloudUserId on telemetry events	3
#492	feat(agents): Agents service foundation + getAll (1/5)	10
#490	feat(data-fabric): add folder support to entity schema and attachment ops	12
#484	fix: fix conversational paginated response and delete output	8
#498	docs: update Claude docs from PR review analysis	0
#471	feat(governance): add Governance getOperationSummary service	7
#464	feat(governance): add Governance getPolicyTraces service	20
#494	feat(telemetry): add org/tenant id and CloudUserId attributes	4

🤖 Generated with Claude Code

[claude]

✅ No issues found. Checked for bugs and CLAUDE.md compliance.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud