Skip to content
This repository was archived by the owner on Aug 6, 2026. It is now read-only.

feat(tasks): surface local skills in the composer and upload on send - #2926

Merged
tatoalo merged 5 commits into
mainfrom
local-skill-05-ui-integration
Jun 29, 2026
Merged

feat(tasks): surface local skills in the composer and upload on send#2926
tatoalo merged 5 commits into
mainfrom
local-skill-05-ui-integration

Conversation

@tatoalo

@tatoalo tatoalo commented Jun 25, 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 https://github.com/PostHog/code/issues/2260

@github-actions

github-actions Bot commented Jun 25, 2026

Copy link
Copy Markdown

React Doctor found no issues in the changed files. 🎉

Reviewed by React Doctor for commit d90d762.

@greptile-apps

greptile-apps Bot commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

Comments Outside Diff (1)

  1. packages/ui/src/features/task-detail/components/TaskInput.tsx, line 1040-1061 (link)

    P2 Duplicated skill-to-command mapping

    The inline spread source !== "bundled" ? { localSkill: { name, source, path } } : {} appears in at least four places: TaskInput.tsx, PromptInput.tsx, useSessionCallbacks.ts (the fallback fetch), and sessionServiceHost.ts (resolveLocalSkillCommandPrompt). Extracting a shared helper such as skillToEditorCommand(skill): EditorAvailableCommand would keep this conversion in one place; if the localSkill shape or the "bundled" check ever changes, only one site needs updating.

    Context Used: Do not attempt to comment on incorrect alphabetica... (source)

    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: "feat(tasks): surface local skills in the..." | Re-trigger Greptile

Comment thread packages/ui/src/features/sessions/hooks/useSessionCallbacks.ts Outdated
@tatoalo
tatoalo force-pushed the local-skill-04-core-plumbing branch from 753b5a5 to c120794 Compare June 25, 2026 09:48
@tatoalo
tatoalo force-pushed the local-skill-05-ui-integration branch from 87d0622 to 097a524 Compare June 25, 2026 09:48
@tatoalo
tatoalo force-pushed the local-skill-04-core-plumbing branch from c120794 to 8422a11 Compare June 25, 2026 09:58
@tatoalo
tatoalo force-pushed the local-skill-05-ui-integration branch from 097a524 to 17746ff Compare June 25, 2026 09:58
@tatoalo tatoalo self-assigned this Jun 25, 2026
@tatoalo
tatoalo requested a review from a team June 25, 2026 10:41
@tatoalo
tatoalo marked this pull request as ready for review June 25, 2026 10:41
@greptile-apps

greptile-apps Bot commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

Reviews (2): Last reviewed commit: "feat(tasks): surface local skills in the..." | Re-trigger Greptile

@tatoalo
tatoalo force-pushed the local-skill-05-ui-integration branch from 17746ff to 9eba7fd Compare June 29, 2026 14:56
@tatoalo
tatoalo force-pushed the local-skill-04-core-plumbing branch from 8422a11 to 73bbd61 Compare June 29, 2026 14:56
@tatoalo tatoalo added the Create Release This will trigger a new release label Jun 29, 2026
@tatoalo
tatoalo force-pushed the local-skill-05-ui-integration branch from 9eba7fd to e6eca5f Compare June 29, 2026 15:09
@tatoalo
tatoalo force-pushed the local-skill-04-core-plumbing branch from 73bbd61 to 1c075bc Compare June 29, 2026 15:09
Base automatically changed from local-skill-04-core-plumbing to graphite-base/2924 June 29, 2026 15:50
@tatoalo
tatoalo force-pushed the local-skill-05-ui-integration branch from e6eca5f to 5bbd42e Compare June 29, 2026 15:54
@tatoalo tatoalo removed the Create Release This will trigger a new release label Jun 29, 2026
@tatoalo
tatoalo force-pushed the local-skill-05-ui-integration branch from 5bbd42e to 996144a Compare June 29, 2026 16:13
@tatoalo
tatoalo changed the base branch from graphite-base/2924 to main June 29, 2026 16:14
@tatoalo
tatoalo enabled auto-merge (squash) June 29, 2026 16:16
@tatoalo
tatoalo force-pushed the local-skill-05-ui-integration branch from 996144a to 41f9c64 Compare June 29, 2026 16:24
@tatoalo
tatoalo merged commit b753497 into main Jun 29, 2026
24 checks passed
@tatoalo
tatoalo deleted the local-skill-05-ui-integration branch June 29, 2026 16:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Cloud tasks should support local skill injection

2 participants