Add PaperKit Copilot skills for routing and defaults#29
Add PaperKit Copilot skills for routing and defaults#29peternicholls wants to merge 3 commits intomasterfrom
Conversation
Summary of ChangesHello @peternicholls, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request enhances PaperKit's discoverability and standardization by introducing new Copilot Agent Skills. These skills enable external tools to effectively route user requests to the correct PaperKit agents and apply consistent workflow defaults, streamlining interactions and ensuring predictable behavior without altering any core runtime logic. Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
There was a problem hiding this comment.
Code Review
This pull request introduces Copilot Agent Skills for PaperKit, defining routing rules and default configurations in new markdown files. The changes are well-structured and effectively document the intended agent behaviors. My review focuses on enhancing the clarity and usability of these new skill and documentation files. I've suggested making the routing logic more deterministic to prevent ambiguity and recommended adding relative links in the documentation to improve developer experience. Overall, these are solid additions that make the system's logic more discoverable.
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
This PR adds Copilot Agent Skills for PaperKit to enable tool-based routing and workflow defaults discovery. The skills encode orchestrator tie-break rules for routing user requests to specific agents and document standard workflow configuration values.
Changes:
- Add routing skill descriptor with agent selection tie-break rules
- Add defaults skill descriptor with citation style, paths, and document class defaults
- Add skills documentation pointing to sources of truth in
.paperkit/core
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
.paperkit/docs/skills.md |
Documents the two new skills and references their sources of truth |
.github/skills/paperkit-routing/SKILL.md |
Implements routing tie-break rules matching orchestrator agent selection logic |
.github/skills/paperkit-defaults/SKILL.md |
Declares workflow defaults from core configuration for citation style, paths, and document settings |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| - "polish / rewrite / tighten" -> quality-refiner | ||
| - "synthesize / summarize / consolidate" -> research-consolidator | ||
| - "brainstorm / ideate / suggest" -> brainstorm | ||
| - otherwise: research-consolidator |
There was a problem hiding this comment.
The fallback rule differs from the source of truth in orchestrator.md. The orchestrator specifies "otherwise: research-consolidator (if synthesis) or brainstorm (if ideation)", but this skill unconditionally routes to research-consolidator. Consider either matching the orchestrator's nuanced fallback or documenting why the simplification was chosen.
| - otherwise: research-consolidator | |
| - otherwise: research-consolidator (if synthesis) or brainstorm (if ideation) |
|
@copilot open a new pull request to apply changes based on the comments in this thread |
|
@peternicholls I've opened a new pull request, #30, to work on those changes. Once the pull request is ready, I'll request review from you. |
Motivation
.paperkit/coreconfiguration and agent docs.Description
.github/skills/paperkit-routing/SKILL.mdimplementing routing tie-break rules that map request intents to agents likelibrarian,tutor,problem-solver, and others..github/skills/paperkit-defaults/SKILL.mddeclaring defaults such as citation styleharvard, document classarticle, and default paths like.paperand.paperkit/data..paperkit/docs/skills.mddocumenting the new skills and pointing to the sources of truth at.paperkit/core/agents/orchestrator.mdand.paperkit/core/config.yaml.Testing
Codex Task