Skip to content

Add PaperKit Copilot skills for routing and defaults#29

Open
peternicholls wants to merge 3 commits intomasterfrom
codex/plan-improvements-for-/paperkit/-with-skills
Open

Add PaperKit Copilot skills for routing and defaults#29
peternicholls wants to merge 3 commits intomasterfrom
codex/plan-improvements-for-/paperkit/-with-skills

Conversation

@peternicholls
Copy link
Owner

Motivation

  • Provide discoverable Copilot Agent Skills for PaperKit so tools can route user requests and apply workflow defaults.
  • Encode the orchestrator tie-break rules for selecting a single agent without performing the task in the router.
  • Record standard workflow defaults such as citation style, document class, and project paths for consistent behavior.
  • Surface the skills alongside their sources of truth to enable tooling to reference .paperkit/core configuration and agent docs.

Description

  • Add .github/skills/paperkit-routing/SKILL.md implementing routing tie-break rules that map request intents to agents like librarian, tutor, problem-solver, and others.
  • Add .github/skills/paperkit-defaults/SKILL.md declaring defaults such as citation style harvard, document class article, and default paths like .paper and .paperkit/data.
  • Add .paperkit/docs/skills.md documenting the new skills and pointing to the sources of truth at .paperkit/core/agents/orchestrator.md and .paperkit/core/config.yaml.
  • Changes are documentation/skill descriptor files only and do not modify runtime code.

Testing

  • No automated tests or linters were run because the change is documentation-only and therefore outside the scope of code tests.
  • No test failures occurred since no tests were executed.

Codex Task

Copilot AI review requested due to automatic review settings January 10, 2026 21:34
@gemini-code-assist
Copy link

Summary of Changes

Hello @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

  • New Copilot Agent Skills: Introduces two new Copilot Agent Skills for PaperKit: paperkit-routing and paperkit-defaults.
  • Routing Skill: The paperkit-routing skill defines orchestrator tie-break rules for routing user requests to specific PaperKit agents like librarian, tutor, and problem-solver.
  • Defaults Skill: The paperkit-defaults skill establishes standard workflow defaults, including citation style (harvard), document class (article), and default project paths (.paper, .paperkit/data).
  • Documentation: New documentation at .paperkit/docs/skills.md details these skills and points to their sources of truth in .paperkit/core/agents/orchestrator.md and .paperkit/core/config.yaml.
  • Documentation-Only Changes: All changes in this pull request are documentation-only, meaning no runtime code was modified and no automated tests were executed.
Using Gemini Code Assist

The 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 /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature Command Description
Code Review /gemini review Performs a code review for the current pull request in its current state.
Pull Request Summary /gemini summary Provides a summary of the current pull request in its current state.
Comment @gemini-code-assist Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help /gemini help Displays a list of available commands.

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 .gemini/ folder in the base of the repository. Detailed instructions can be found here.

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

  1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution.

Copy link

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

peternicholls and others added 2 commits January 10, 2026 21:36
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>
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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
Copy link

Copilot AI Jan 10, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Suggested change
- otherwise: research-consolidator
- otherwise: research-consolidator (if synthesis) or brainstorm (if ideation)

Copilot uses AI. Check for mistakes.
@peternicholls
Copy link
Owner Author

@copilot open a new pull request to apply changes based on the comments in this thread

Copy link
Contributor

Copilot AI commented Jan 10, 2026

@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.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants