Skip to content

feat(ai): invoke a project skill with / in every chat surface - #9932

Merged
nishantmonu51 merged 1 commit into
mainfrom
nishant/skills-all-chat-surfaces
Sep 25, 2026
Merged

nishantmonu51 merged 1 commit into
mainfrom
nishant/skills-all-chat-surfaces

Conversation

@nishantmonu51

Copy link
Copy Markdown
Collaborator
  • Typing / (or clicking the / button) opens the skill picker in every AI chat surface, not just the project chat: explore and canvas dashboard chats, the embed chat, and the developer chat.
  • The picker lists the project's skills for the agent that answers the chat (analyst or developer), derived from the ChatConfig agent; the per-surface skills flag is gone.
  • The developer agent now pre-invokes load_skill for skills referenced in the prompt, like the analyst. Since the developer's model only sees the current turn's tool calls, the "already loaded" check is scoped to the current invocation, so a skill referenced again in a later turn is reloaded.
  • Refactors the analyst's loop into a shared loadReferencedSkills helper and adds developer-agent tests.

Follows up on #9892.

Checklist:

  • Covered by tests
  • Ran it and it works as intended
  • Reviewed the diff before requesting a review
  • Checked for unhandled edge cases
  • Linked the issues it closes
  • Checked if the docs need to be updated. If so, create a separate Linear DOCS issue
  • Intend to cherry-pick into the release branch
  • I'm proud of this work!

The `/` skill picker was only enabled for the project chat. It now
opens in every chat: the picker lists the skills for the agent that
answers the chat (analyst for project, embed, explore and canvas chats;
developer for the developer chat), and the developer agent pre-invokes
load_skill for the skills referenced in the prompt like the analyst
already did. Since the developer's model only sees the current turn's
tool calls, a referenced skill is loaded again on each turn.
@nishantmonu51
nishantmonu51 merged commit cff9085 into main Sep 25, 2026
21 of 23 checks passed
@nishantmonu51
nishantmonu51 deleted the nishant/skills-all-chat-surfaces branch September 25, 2026 05:45
nishantmonu51 added a commit that referenced this pull request Sep 25, 2026
The `/` skill picker was only enabled for the project chat. It now
opens in every chat: the picker lists the skills for the agent that
answers the chat (analyst for project, embed, explore and canvas chats;
developer for the developer chat), and the developer agent pre-invokes
load_skill for the skills referenced in the prompt like the analyst
already did. Since the developer's model only sees the current turn's
tool calls, a referenced skill is loaded again on each turn.
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