Skip to content

feat(skill): add ROS agent integration and permission recovery - #307

Merged
guima-why merged 7 commits into
mainfrom
feature-skill2
Aug 25, 2026
Merged

feat(skill): add ROS agent integration and permission recovery#307
guima-why merged 7 commits into
mainfrom
feature-skill2

Conversation

@guima-why

Copy link
Copy Markdown
Collaborator

Summary

  • add the alicloud-ros-agent external Skill with StartChat code/CLI transports, endpoint/profile controls, SSE event handling, and local relay coverage
  • add durable permission waiting/checkpoint recovery across A2A, Web, pipeline, and local Skill integrations while preserving sub-pipeline timeout behavior
  • classify cloud tools by read/write intent so read-only operations can be approved automatically while resource mutations remain gated
  • extend pipeline handoff/event routing, session backup recovery, and focused offline/real E2E coverage

Validation

  • uv run ruff check scripts/a2a/e2e/run_recovery_scenarios.py tests/a2a_e2e/test_run_recovery_scenarios.py
  • uv run pytest -q tests/a2a_e2e/test_run_recovery_scenarios.py (64 passed)
  • real A2A regressions passed: scenario1, scenario1-performance-backup, fault-after-snapshot, and ask-waiting
  • all four ROS stacks created by those regressions were deleted and verified as DELETE_COMPLETE

Follow-up observation

The corrected selection-during-backup runner now proves that the selection request is dispatched inside the injected backup window. The current product routing classifies that concurrent selection as a supplement interrupt instead of candidate input, so that real-cloud scenario remains a focused follow-up rather than a passing validation.

Comment thread tests/skill_bridge/start_chat_relay.py Fixed
@guima-why
guima-why marked this pull request as ready for review August 25, 2026 14:23
@guima-why
guima-why merged commit 9849551 into main Aug 25, 2026
26 checks passed
@guima-why
guima-why deleted the feature-skill2 branch August 25, 2026 14:23
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.

2 participants