Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@uipreliga
Copy link
Collaborator

Implement comprehensive SDK service standardization across 5 core Orchestrator services

(to fill in a feature gap required for more complete CLI)

AssetsService:
- Add list() iterator with OData support and auto-pagination
- Add exists() boolean check for asset existence
- Add get_value() convenience method
- Add delete() for asset removal
- Add async variants for all new methods
- Fix retrieve() to raise LookupError when not found

FolderService:
- Add list() iterator with OData filtering
- Add retrieve() with dual-mode lookup (display_name or key)
- Add retrieve_by_path() for path-based lookup
- Add exists() and exists_async() checks
- Add create() and delete() folder operations
- Full async/sync parity across all methods

JobsService:
- Add list() iterator for job enumeration
- Add retrieve() by job_key or job_id
- Add exists() check for job existence
- Add stop(), suspend(), resume() control operations
- Full async variants for all operations

ProcessesService:
- Add list() iterator for process/release enumeration
- Add retrieve() by key or process_id
- Add exists() check for process existence
- Add invoke() convenience wrapper
- Full async/sync parity

QueuesService (Two-Level Architecture):
Queue Definitions:
- Add list_definitions() iterator with OData support
- Add retrieve_definition() by name or key
- Add create_definition() for queue creation
- Add delete_definition() for cleanup
- Add exists_definition() check

Queue Items:
- Add create_item() for adding queue items
- Add list_items() iterator with status filtering
- Full async variants for all operations

@github-actions github-actions bot added test:uipath-langchain Triggers tests in the uipath-langchain-python repository test:uipath-llamaindex Triggers tests in the uipath-llamaindex-python repository labels Oct 31, 2025
@uipreliga uipreliga force-pushed the feat/orchestrator-services-enhancements branch from 160e2c5 to 1857837 Compare October 31, 2025 22:22
@uipreliga uipreliga force-pushed the feat/orchestrator-services-enhancements branch from 444654b to f10c43d Compare November 4, 2025 02:44
@uipreliga uipreliga force-pushed the feat/orchestrator-services-enhancements branch from f10c43d to 5da6b53 Compare November 5, 2025 02:19
Comment on lines +119 to +124
MAX_PAGES = 10
skip = 0
top = 50
pages_fetched = 0

while True:
while pages_fetched < MAX_PAGES:

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is the point of the autopaging up to 10?

scoped: Literal["org", "tenant"] = "tenant",
**kwargs: Any,
) -> Response:
import asyncio

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's already difficult to review a 7500 lines PR. Adding this kind of (technically valid) weirdness makes me trust the AI-generated code even less. Could you please do a pass on the code and break the PR in more manageable pieces?

  Implement comprehensive SDK service standardization across 5 core Orchestrator
  services

  AssetsService:
    - Add list() iterator with OData support and auto-pagination
    - Add exists() boolean check for asset existence
    - Add get_value() convenience method
    - Add delete() for asset removal
    - Add async variants for all new methods
    - Fix retrieve() to raise LookupError when not found

  FolderService:
    - Add list() iterator with OData filtering
    - Add retrieve() with dual-mode lookup (display_name or key)
    - Add retrieve_by_path() for path-based lookup
    - Add exists() and exists_async() checks
    - Add create() and delete() folder operations
    - Full async/sync parity across all methods

  JobsService:
    - Add list() iterator for job enumeration
    - Add retrieve() by job_key or job_id
    - Add exists() check for job existence
    - Add stop(), suspend(), resume() control operations
    - Full async variants for all operations

  ProcessesService:
    - Add list() iterator for process/release enumeration
    - Add retrieve() by key or process_id
    - Add exists() check for process existence
    - Add invoke() convenience wrapper
    - Full async/sync parity

  QueuesService (Two-Level Architecture):
    Queue Definitions:
    - Add list_definitions() iterator with OData support
    - Add retrieve_definition() by name or key
    - Add create_definition() for queue creation
    - Add delete_definition() for cleanup
    - Add exists_definition() check

    Queue Items:
    - Add create_item() for adding queue items
    - Add list_items() iterator with status filtering
    - Full async variants for all operations
@uipreliga uipreliga force-pushed the feat/orchestrator-services-enhancements branch from 5da6b53 to 4ef740d Compare November 10, 2025 22:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test:uipath-langchain Triggers tests in the uipath-langchain-python repository test:uipath-llamaindex Triggers tests in the uipath-llamaindex-python repository

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants