Skip to content

fix(cloud-task): bundle a local skill tagged on the first message#3056

Merged
tatoalo merged 1 commit into
mainfrom
cloud-skills-first-message-bundle
Jul 1, 2026
Merged

fix(cloud-task): bundle a local skill tagged on the first message#3056
tatoalo merged 1 commit into
mainfrom
cloud-skills-first-message-bundle

Conversation

@tatoalo

@tatoalo tatoalo commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Problem

starting a cloud task whose first message is a local-skill slash command failed, the skill's bundle was never uploaded, so the sandbox had no such skill.
The initial task-creation path did not resolved a typed local-skill command into the skill tag

Changes

resolve a leading local-skill slash command into a <skill …/> tag in the initial task-creation path, mirroring what the follow-up path already does

tatoalo commented Jul 1, 2026

Copy link
Copy Markdown
Contributor Author

@github-actions

github-actions Bot commented Jul 1, 2026

Copy link
Copy Markdown

React Doctor found no issues in the changed files. 🎉

Reviewed by React Doctor for commit aad38ac.

@greptile-apps

greptile-apps Bot commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Reviews (1): Last reviewed commit: "fix(cloud-task): bundle a local skill ta..." | Re-trigger Greptile

Comment thread packages/core/src/task-detail/taskCreationSaga.test.ts
@tatoalo tatoalo self-assigned this Jul 1, 2026
@tatoalo tatoalo added the Stamphog This will request an autostamp by stamphog on small changes label Jul 1, 2026
@tatoalo tatoalo force-pushed the cloud-skills-first-message-bundle branch from 6cf04be to aad38ac Compare July 1, 2026 13:35
@tatoalo tatoalo marked this pull request as ready for review July 1, 2026 13:40

@github-actions github-actions Bot 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.

Focused bug fix that adds skill-bundle resolution on first cloud message, mirroring the identical pattern already used in the session service host. Single implementation of the interface, tests cover the happy path and pass-through cases, and the one substantive bot review comment was addressed.

@greptile-apps

greptile-apps Bot commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Reviews (2): Last reviewed commit: "fix(cloud-task): bundle a local skill ta..." | Re-trigger Greptile

@tatoalo tatoalo merged commit 9dbf01e into main Jul 1, 2026
28 checks passed
@tatoalo tatoalo deleted the cloud-skills-first-message-bundle branch July 1, 2026 13:57
@richardsolomou

Copy link
Copy Markdown
Member

@tatoalo this is still happening for me on latest version

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Stamphog This will request an autostamp by stamphog on small changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants