refactor(governance): deprecate is_governance_enabled, drop CLI caller

[viswa-uipath]

Summary

The CLI run / debug governance bootstrap no longer gates on the EnablePythonGovernanceChecker feature flag. resolve_governance always fetches policy and lets the backend-supplied enforcement mode decide whether governance runs.

is_governance_enabled() is a released public symbol (uipath-core 0.5.18, tag v2.11.0) and external agents may still gate their own evaluator wiring on the flag, so it is kept for backward compatibility rather than removed:

• uipath-core: is_governance_enabled() now emits a DeprecationWarning but otherwise reads the flag exactly as before (same resolution order, same off-by-default behavior). Export and signature are unchanged. It will be removed in a future major release.
• uipath: removed the is_governance_enabled import and the if not is_governance_enabled(): return None gate at the top of resolve_governance in _cli/_governance_bootstrap.py — the only in-SDK caller.

Why deprecate instead of hardcoding return False

Hardcoding return False would silently break external callers that gate on the flag — they'd skip governance even when the operator set UIPATH_FEATURE_EnablePythonGovernanceChecker=true. A silent behavior change is worse than a loud one. Keeping the original flag-reading behavior preserves correct semantics; the warning signals the migration path.

Testing

• packages/uipath-core: pytest tests/governance/test_config.py — 6 passed (existing behavior tests now assert the DeprecationWarning; added a dedicated deprecation test)
• packages/uipath: pytest tests/cli/test_governance_bootstrap.py — 16 passed
• Ruff lint + format clean on all changed files

🤖 Generated with Claude Code

[Copilot]

Pull request overview

This PR removes the is_governance_enabled() feature-flag gate from uipath-core and stops the CLI governance bootstrap (resolve_governance) from short-circuiting based on that flag, relying instead on backend policy responses and existing error-handling to keep governance optional.

Changes:

• Deleted is_governance_enabled() from uipath-core governance config and removed it from package exports/tests.
• Removed the is_governance_enabled import and early-return gate from uipath CLI governance bootstrap.
• Updated CLI tests to no longer monkeypatch the feature-flag gate.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
packages/uipath/tests/cli/test_governance_bootstrap.py	Removes tests/stubs that depended on the feature-flag gate in CLI bootstrap.
packages/uipath/src/uipath/_cli/_governance_bootstrap.py	Removes the feature-flag short-circuit so governance resolution proceeds to backend policy fetch.
packages/uipath-core/tests/governance/test_config.py	Removes tests for is_governance_enabled() and keeps only the constant stability check.
packages/uipath-core/src/uipath/core/governance/config.py	Removes the gate function and leaves only GOVERNANCE_FEATURE_FLAG.
packages/uipath-core/src/uipath/core/governance/init.py	Stops exporting is_governance_enabled from the public governance surface.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[github-actions]

🚨 Heads up: uipath-integrations cross-tests are FAILING 🚨

Your changes may break one or more integrations in uipath-integrations-python:

• uipath-openai-agents
• uipath-google-adk
• uipath-agent-framework
• uipath-llamaindex
• uipath-pydantic-ai

⚠️ These checks are NOT enforced by branch protection rules. Please review the failures before merging.

🔍 Inspect the failed run →

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

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

See analysis details on SonarQube Cloud

[Copilot]

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (1)

packages/uipath/src/uipath/_cli/_governance_bootstrap.py:103

• The try/except wraps UiPath() construction and provider setup as well as the actual get_policy_async call, so exceptions from SDK/provider initialization will be reported as a “policy fetch failed”. That message is misleading now that this code path always runs; consider making the message cover bootstrap/setup failures (or narrowing the try to only the fetch).

    try:
        sdk = UiPath()
        provider = UiPathPlatformGovernanceProvider(service=sdk.governance)
        response = await provider.get_policy_async(context)
    except Exception as exc:
        console.warning(
            f"Governance policy fetch failed - continuing without governance: {exc}"
        )
        return None