feat: clarify project manager continuations and skills - #2859
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What changed
search_skillsretrieval, andmanage_skillsmutation routing from the currentdevarchitecture.Why
Agent-owned scheduler wakeups were stored in the same message direction as dashboard prompts, so the UI could make an automated continuation look like something the user sent. Persistent skill attachments were also discoverable through search but were not summarized as an authoritative inventory, which made broad skill-availability questions easy to answer incorrectly.
User and developer impact
Scheduled follow-ups are now visually auditable and clearly attributed to the Project Manager. Project managers receive linked skill names, descriptions, and versions in runtime context while retaining vector-backed retrieval for detailed guidance. The branch is synchronized with current
dev, including containerized Git storage and optimized memory retrieval.Validation
pnpm run lintpnpm exec vitest run tests/backend/services/skill-service.test.ts tests/backend/infrastructure/providers/cli/mockup-cli-provider.test.ts tests/dashboard/v2/chat-message-bubbles.test.tsx— 74 passedpnpm run test:backend— 5,077 passed, 7 skippedpnpm run test:dashboard— 1,991 passed, 2 skippedpnpm run sync:docs-webpnpm run check:docs-webpnpm run buildRisk and rollback
Risk is limited to chat presentation and additional descriptor text in skill-enabled agent prompts. Revert the three feature commits to restore prior rendering, prompt inventory behavior, and worker guidance independently.