Skip to content

Remove deprecated Connector._robot_session property#71

Merged
b-Tomas merged 1 commit into
nextfrom
breaking/remove-deprecated-robot-session
May 20, 2026
Merged

Remove deprecated Connector._robot_session property#71
b-Tomas merged 1 commit into
nextfrom
breaking/remove-deprecated-robot-session

Conversation

@b-Tomas
Copy link
Copy Markdown
Member

@b-Tomas b-Tomas commented May 19, 2026

Summary

  • Remove the deprecated Connector._robot_session property and its @deprecated decorator from connector.py
  • Remove the deprecated decorator import (Python 3.13+ compat block) — no longer used anywhere
  • Remove the "Deprecated: _robot_session" section from the connector specification docs

Breaking changes

Connector._robot_session property has been removed. It was deprecated in favor of _get_session().

Migration

# Before (v2.x)
session = self._robot_session

# After (v3.0)
session = self._get_session()

_get_session() is unchanged and remains the canonical way to get the edge-sdk robot session for the current robot.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR removes the deprecated Connector._robot_session property (and its now-unused deprecated decorator import shim) and updates the connector specification docs accordingly, making _get_session() the sole supported way to access the single-robot Edge SDK session.

Changes:

  • Removed Connector._robot_session property (previously deprecated in favor of _get_session()).
  • Removed the Python 3.13+ compatibility import block for the deprecated decorator (no longer used).
  • Removed the deprecated _robot_session section from the connector specification documentation.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
inorbit_connector/connector.py Removes the deprecated _robot_session property and the unused deprecated import compatibility block.
docs/contents/specification/connector.md Deletes the spec section documenting the deprecated _robot_session property to align docs with the API removal.

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

@b-Tomas b-Tomas self-assigned this May 19, 2026
@b-Tomas b-Tomas marked this pull request as ready for review May 19, 2026 22:20
@b-Tomas b-Tomas requested a review from miguelgarcia May 19, 2026 23:13
BREAKING CHANGE: `Connector._robot_session` property has been removed.
Use `Connector._get_session()` instead.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@b-Tomas b-Tomas force-pushed the breaking/remove-deprecated-robot-session branch from e91b87b to eee9d38 Compare May 19, 2026 23:21
@b-Tomas b-Tomas merged commit 2fa7e58 into next May 20, 2026
13 checks passed
@b-Tomas b-Tomas deleted the breaking/remove-deprecated-robot-session branch May 20, 2026 12:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants