docs: update Claude docs from PR review analysis

[claude]

Summary

Weekly analysis of PR comments (2026-08-31 -> 2026-09-07).
Analyzed 12 PRs with ~100 review threads. Found 5 actionable insights.

Changes

agent_docs/rules.md

• NEVER include RBAC permissions in docs/oauth-scopes.md — OAuth scopes only
  Source: PR feat(maestro): add Business Apps CRUD service #671 — reviewer commented on docs/oauth-scopes.md:

  "These are RBAC permissions which arent documented in this page. Lets remove"

• When adding a resource type to ResourceRegistry, wire cleanupAllTestResources() too
  Source: PR feat(maestro): add Business Apps CRUD service #671 — reviewer noted on tests/integration/utils/cleanup.ts:

  "`businessApps` registry slot wired but `cleanupAllTestResources()` was not updated — `registerResource('businessApps', ...)` calls are inert and orphaned apps won't be deleted"

• Sample app GIFs must be HD; config template named uipath.json.example; no hardcoded env values; no internal errors in UI toasts
  Source: PR feat(samples): add queues sample app [PLT-104203] #676 — reviewer comments:

  "quality of this gif is poor, can you have HD gifs?" / "please dont hardcode folder id etc." / "dont surface internals in end user toast messages" / "call it uipath.json.example"

agent_docs/conventions.md

• Shared regex patterns across services should be extracted to src/utils/
  Source: PR feat(assets): add ref-based updateValue; deprecate updateValueById #684 — reviewer commented:

  "same regex is used in queues and a few other places. consider extracting and reusing this constant."

• Ref resolution must carry resolved folder context through to mutations (effectiveFolder left side of ??)
  Source: PR feat(assets): add ref-based updateValue; deprecate updateValueById #684 — reviewer commented:

  "resolveRefToId returns `{ id, effectiveFolder }` so name/key lookups can carry the resolved folder up to the follow-up mutation. Here, effectiveFolder is dropped on both branches."

No changes

• CLAUDE.md — no relevant insights found
• Agents.md — no relevant insights found
• agent_docs/architecture.md — no relevant insights found

PRs Analyzed

PR	Title	Threads
#671	feat(maestro): add Business Apps CRUD service	12
#684	feat(assets): add ref-based updateValue; deprecate updateValueById	31
#682	feat(data-fabric): by-name addressing for entity ops	29
#707	test(data-fabric): split entities integration tests	4
#708	fix(integration-service): simplify user facing exports	1
#706	docs(coded-apps): document redirectUri override	3
#695	feat: add httpRequest and wait runtime helpers	16
#676	feat(samples): add queues sample app	10
#700	feat(auth): add enforceSso option	5
#698	refactor(integration-service): remove reauthenticate API	0
#635	feat(data-fabric): mark schema methods @experimental	15
#690	chore: bump version to 1.7.0	0

[claude]

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

[claude]

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

[claude]

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