Skip to content

feat: add is_rooted_to_debug_job property to ConfigurationManager#1464

Merged
smflorentino merged 9 commits intomainfrom
fix/maestro-debug-licensing
Mar 19, 2026
Merged

feat: add is_rooted_to_debug_job property to ConfigurationManager#1464
smflorentino merged 9 commits intomainfrom
fix/maestro-debug-licensing

Conversation

@smflorentino
Copy link
Collaborator

@smflorentino smflorentino commented Mar 19, 2026

Summary

  • Adds is_rooted_to_debug_job property to ConfigurationManager that reads isDebug from uipath.json's runtime.internalArguments
  • Orchestrator sets this flag for jobs whose root parent is a debug session (e.g. Maestro solution debug)
  • Follows the same pattern as licensing_context (public property wrapping _read_internal_argument)

Test plan

  • Verify property returns True when isDebug: true is in uipath.json internalArguments
  • Verify property returns False when key is absent or false

🤖 Generated with Claude Code

Exposes isDebug from Orchestrator's InternalArguments as a public
property. Used by uipath-agents to override the licensing header
for Maestro solution debug agent jobs.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@github-actions github-actions bot added the test:uipath-langchain Triggers tests in the uipath-langchain-python repository label Mar 19, 2026
smflorentino and others added 3 commits March 18, 2026 22:09
Version bump for is_rooted_to_debug_job property.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Required for is_rooted_to_debug_job property.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@smflorentino smflorentino marked this pull request as ready for review March 19, 2026 05:13
smflorentino and others added 3 commits March 18, 2026 22:16
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Detail lives in _get_agenthub_config in uipath-agents-python.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Moves the debug licensing override logic to uipath-platform so all
SDKs (langchain, openai-agents, pydantic-ai, etc.) can use it.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The override logic belongs in uipath-agents-python where the header
values originate. uipath-platform only exposes is_rooted_to_debug_job.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@property
def is_rooted_to_debug_job(self) -> bool:
"""Whether this job, which may be a deployed process, is rooted to a debug session (e.g. Maestro solution debug)."""
return self._read_internal_argument("isDebug") is True
Copy link
Collaborator

Choose a reason for hiding this comment

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

Seemed strange to check the is True for a boolean value so I took a deeper look and the problem stands with the underlying methods.

Might be worth updating in a future PR:

  1. _read_internal_argument to not type hint the return to str | None as the value here it's actually a bool
  2. def _internal_arguments(self) -> dict[str, str] | None: to be def _internal_arguments(self) -> dict[str, Any] | None:
    as the underlying method called is json.load which effectively returns Any
    def _read_internal_argument(self, key: str) -> str | None:
        internal_args = self._internal_arguments
        return internal_args.get(key) if internal_args else None

Copy link
Collaborator

Choose a reason for hiding this comment

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

raised an issue here to not forget about it #1465

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

makes sense

@UiPath UiPath deleted a comment from cotovanu-cristian Mar 19, 2026
@smflorentino smflorentino merged commit 9848d16 into main Mar 19, 2026
117 checks passed
@smflorentino smflorentino deleted the fix/maestro-debug-licensing branch March 19, 2026 06:51
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants