Skip to content

feat(tasks): bundle local skills for cloud runs#2908

Closed
tatoalo wants to merge 3 commits into
mainfrom
feat/cloud-agent/local-skill-to-sandbox
Closed

feat(tasks): bundle local skills for cloud runs#2908
tatoalo wants to merge 3 commits into
mainfrom
feat/cloud-agent/local-skill-to-sandbox

Conversation

@tatoalo

@tatoalo tatoalo commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Problem

for cloud tasks, local skill invocations only sent the slash command text. User-local and repo-local skills were available in the local editor but not in the cloud sandbox

Changes

  • let's bundle uploadable local skills into zip artifacts with SHA-256 metadata and send them through the existing cloud artifact upload flow.

  • install skill_bundle artifacts in the agent server before prompt delivery, inject the bundled skill instructions for the invoked turn, and suppress the raw slash command from the model-facing prompt so local skills behave like actual skills rather than unsupported commands

  • refresh slash command metadata when a follow-up command is submitted, so newly created local skills can be selected and invoked without starting a brand-new task.

closes #2260

@tatoalo tatoalo self-assigned this Jun 24, 2026
@github-actions

github-actions Bot commented Jun 24, 2026

Copy link
Copy Markdown

React Doctor found no issues in the changed files. 🎉

Reviewed by React Doctor for commit 9a1a7aa.

@greptile-apps

greptile-apps Bot commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Comments Outside Diff (1)

  1. packages/core/src/sessions/sessionEvents.ts, line 17-42 (link)

    P2 decodeXmlAttr is a local reimplementation of unescapeXmlAttr already exported from @posthog/shared. Three other files that parse skill tags in this PR (content.ts, cloudPrompt.ts, cloud-prompt.ts) all import unescapeXmlAttr from @posthog/shared. This local copy is a direct violation of OnceAndOnlyOnce and drifts silently if the shared version is ever updated.

    Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!

Reviews (1): Last reviewed commit: "fix(tasks): update queued cloud prompt t..." | Re-trigger Greptile

Comment thread packages/core/src/sessions/cloudPrompt.ts Outdated
@tatoalo tatoalo force-pushed the feat/cloud-agent/local-skill-to-sandbox branch from ba09cda to 9a1a7aa Compare June 25, 2026 09:07
@tatoalo

tatoalo commented Jun 25, 2026

Copy link
Copy Markdown
Contributor Author

Superseded by a stacked split for easier review (same change, rebased onto current main):

Closing in favour of the stack.

@tatoalo tatoalo closed this Jun 25, 2026
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.

Cloud tasks should support local skill injection

1 participant