calm-artist-7206
03/31/2026, 9:42 PMSkill tool for, I think, loading an AgentSkill.
I… am not sure why it would need that?salmon-lock-64866
03/31/2026, 11:38 PMcalm-artist-7206
03/31/2026, 11:39 PMcalm-artist-7206
03/31/2026, 11:40 PMsalmon-lock-64866
03/31/2026, 11:44 PMcalm-artist-7206
03/31/2026, 11:51 PMlimited-student-10747
03/31/2026, 11:52 PMcalm-artist-7206
04/01/2026, 12:13 AMsalmon-lock-64866
04/01/2026, 12:56 AMcalm-artist-7206
04/01/2026, 1:01 AMcalm-artist-7206
04/01/2026, 1:02 AMsalmon-lock-64866
04/01/2026, 1:08 AMcalm-artist-7206
04/01/2026, 1:09 AMnutritious-balloon-65098
04/01/2026, 1:09 AMcalm-artist-7206
04/01/2026, 1:10 AMcalm-artist-7206
04/01/2026, 1:10 AMnutritious-balloon-65098
04/01/2026, 1:19 AMnutritious-balloon-65098
04/01/2026, 1:24 AMopenhands-workspace-2Uhx7DjJeGPO2Hx2uTqFs0
- PR opened: #2635
- PR URL: https://github.com/OpenHands/software-agent-sdk/pull/2635
The PR title is:
- chore: capture skill prompt truncation investigation
The committed work is on:
- d94962c9calm-artist-7206
04/01/2026, 1:26 AM<description>Babysit a GitHub pull request by continuously polling CI checks/workflow runs, new review comments, and mergeability state until the PR is ready to merge (or merged/closed). Diagnose failures, retry l... [352 characters truncated. View /workspace/project/extensions/skills/babysit-pr/SKILL.md for complete information]</description>limited-student-10747
04/01/2026, 1:27 AMlimited-student-10747
04/01/2026, 1:27 AMcalm-artist-7206
04/01/2026, 1:27 AMcalm-artist-7206
04/01/2026, 1:35 AMextensions/ repo, and disallow descriptions larger than 200 characters
• send an LLM call to a small LLM where possible, to summarize them? but it risks losing data just the same, and it introduces randomness in behaviorlimited-student-10747
04/01/2026, 1:40 AMnutritious-balloon-65098
04/01/2026, 1:40 AM.pr/ by force-pushing the PR branch with lease
- Branch: openhands-workspace-2Uhx7DjJeGPO2Hx2uTqFs0
- New branch tip: e99049a6
- PR remains: https://github.com/OpenHands/software-agent-sdk/pull/2635
The remote automated commit only deleted .pr/, so I replaced it with the branch state that includes:
- the 1000-character truncation change
- updated tests
- regenerated .pr/skill_prompt_probe/ artifacts showing all three largest descriptions are now sent in fullnutritious-balloon-65098
04/01/2026, 1:45 AM23827870512 (PR Review by OpenHands) is now completed with conclusion cancelled
- The other related run 23827873769 is completed with conclusion skipped
Direct links:
- Cancelled run: https://github.com/OpenHands/software-agent-sdk/actions/runs/23827870512
- Skipped run: https://github.com/OpenHands/software-agent-sdk/actions/runs/23827873769
If you want, I can also check whether that workflow is likely to retrigger again on this PR and tell you how to avoid it.nutritious-balloon-65098
04/01/2026, 1:48 AMto_prompt()
- the three largest tested skills are now documented as being sent in full
- the focused test file is called out
- AGENTS.md is mentioned as updated
I also added a dedicated Reviewer artifacts section so reviewers know where to look:
- .pr/skill_prompt_probe.py
- .pr/skill_prompt_probe/report.md
- .pr/skill_prompt_probe/description_comparison.json
- .pr/skill_prompt_probe/available_skills_prompt.xml
- .pr/skill_prompt_probe/system_prompt.txt
- .pr/skill_prompt_probe/request_payload.json
- .pr/skill_prompt_probe/completions/*.json
- .pr/skill_prompt_probe/top_skills.json
I preserved the existing agent-server image block in the PR body too.
PR:
- https://github.com/OpenHands/software-agent-sdk/pull/2635
One note: I left the “Is the github CI passing?” checkbox unchecked, since I didn’t want to claim full green status while workflow state is still settling / one review workflow was intentionally cancelled.calm-artist-7206
04/01/2026, 1:53 AMnutritious-balloon-65098
04/01/2026, 1:53 AMnutritious-balloon-65098
04/01/2026, 1:56 AMcalm-artist-7206
04/01/2026, 1:58 AMnutritious-balloon-65098
04/01/2026, 3:08 AMgh, found that thread, verified the spec, and updated the PR.
What I verified
- <https://agentskills.io/specification> says the AgentSkills description field is:
- “Max 1024 characters”
- “Must be 1-1024 characters”
What I changed
- Added the rationale in code:
- openhands-sdk/openhands/sdk/context/skills/skill.py
- Added the rationale in repo memory:
- AGENTS.md
- Updated the PR description to say 1024 matches the AgentSkills spec
Commit pushed
- 429077be — docs(sdk): note AgentSkills 1024-char standard
Review thread
- I replied on the inline thread explaining the fix and the spec verification
- I resolved that thread
I only addressed the “why 1024” thread you pointed at. If you want, I can also go through the remaining unresolved review thread(s) and handle them too.salmon-lock-64866
04/01/2026, 3:29 AMlimited-student-10747
04/01/2026, 3:35 AMcalm-artist-7206
04/01/2026, 4:22 AMfile read tool is restricted to the workspace, then, well, skills can be global, in Path.home() somewhere 🤔, or in caches
Though, a concept of allowed directories would be enough...