Skip to content

docs(sdk): document launch-time skill overlays - #770

Open
kyletser wants to merge 1 commit into
OpenHands:mainfrom
kyletser:fix/acp-profile-launch-skills-16905
Open

docs(sdk): document launch-time skill overlays#770
kyletser wants to merge 1 commit into
OpenHands:mainfrom
kyletser:fix/acp-profile-launch-skills-16905

Conversation

@kyletser

@kyletser kyletser commented Aug 30, 2026

Copy link
Copy Markdown
  • I have read and reviewed the documentation changes to the best of my ability.
  • If the change is significant, I have run the documentation site locally and confirmed it renders as expected.

Summary of changes

Document agent_launch_additions.skills_append in the Agent Server architecture guide.

  • Provide a request example that validates against the current SDK model.
  • Explain same-name precedence, disabled_skills, persistence across server restarts, and disable_model_invocation for trigger-only skills.
  • Clarify that launch additions change the materialized conversation agent without modifying the stored Agent Profile.

Validation:

  • The JSON example passed StartConversationRequest.model_validate against the matching SDK branch.
  • git diff --check passed.
  • mint broken-links parsed the updated page. The repository-wide check still reports 928 existing broken links; this section adds no links.

Implementation: OpenHands/software-agent-sdk#4717

Related to OpenHands/OpenHands#16905.

@kyletser
kyletser marked this pull request as ready for review August 30, 2026 17:48
@kyletser
kyletser requested a review from enyst as a code owner August 30, 2026 17:48
Copilot AI lite review requested due to automatic review settings August 30, 2026 17:48

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

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