Skip to content

fix(codeql-resolver): add .md extension to skill paths#59

Merged
JacobPEvans merged 1 commit intomainfrom
fix/codeql-skill-paths
Feb 27, 2026
Merged

fix(codeql-resolver): add .md extension to skill paths#59
JacobPEvans merged 1 commit intomainfrom
fix/codeql-skill-paths

Conversation

@JacobPEvans
Copy link
Owner

@JacobPEvans JacobPEvans commented Feb 27, 2026

Summary

  • Add .md extension to skill paths in codeql-resolver/.claude-plugin/plugin.json
  • Paths ./skills/codeql-permission-classification and ./skills/github-workflow-security-patterns were missing the .md extension, causing path resolution errors when Claude Code loaded the plugin

Test plan

  • After merge, nix flake update jacobpevans-cc-plugins in nix-ai picks up the fix
  • Claude Code loads codeql-resolver without skill path errors

🤖 Generated with Claude Code


Important

Add missing .md extensions to skill paths in plugin.json to fix path resolution errors.

  • Fixes:
    • Add .md extension to skill paths in codeql-resolver/.claude-plugin/plugin.json.
    • Corrects paths ./skills/codeql-permission-classification and ./skills/github-workflow-security-patterns to ./skills/codeql-permission-classification.md and ./skills/github-workflow-security-patterns.md.
  • Testing:
    • Ensure nix flake update jacobpevans-cc-plugins in nix-ai picks up the fix.
    • Verify Claude Code loads codeql-resolver without skill path errors.

This description was created by Ellipsis for 327bfed. You can customize this summary. It will automatically update as commits are pushed.

Copilot AI review requested due to automatic review settings February 27, 2026 14:17
Copy link

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

Important

Looks good to me! 👍

Reviewed everything up to 327bfed in 7 seconds. Click for details.
  • Reviewed 14 lines of code in 1 files
  • Skipped 0 files when reviewing.
  • Skipped posting 0 draft comments. View those below.
  • Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.

Workflow ID: wflow_XcOmRzexwv2OV8ph

You can customize Ellipsis by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.

@gemini-code-assist
Copy link

Summary of Changes

Hello, 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 fixes a path resolution issue in the codeql-resolver plugin that prevented Claude Code from loading it correctly. The fix involves adding the missing .md extensions to the skill paths in the plugin.json file.

Highlights

  • Path Resolution Fix: This PR addresses an issue where Claude Code was unable to load the codeql-resolver plugin due to missing .md extensions in skill paths.
  • Plugin.json Update: The plugin.json file in the codeql-resolver/.claude-plugin/ directory was modified to include the missing .md extensions for the codeql-permission-classification and github-workflow-security-patterns skill paths.
  • Testing: The test plan includes updating the jacobpevans-cc-plugins flake in nix-ai and verifying that Claude Code loads the codeql-resolver plugin without errors.
Changelog
  • codeql-resolver/.claude-plugin/plugin.json
    • Added '.md' extensions to skill paths for 'codeql-permission-classification' and 'github-workflow-security-patterns'.
Activity
  • The PR author, JacobPEvans, has provided a clear summary of the issue and the proposed solution.
  • The test plan is well-defined and includes steps to verify the fix in both nix-ai and Claude Code.
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 correctly fixes a path resolution error by adding the .md extension to skill paths in codeql-resolver/.claude-plugin/plugin.json. However, this change introduces a schema validation issue, as the pattern for skills in schemas/plugin.schema.json does not currently allow for the .md extension. I've left a comment detailing the issue and recommending an update to the schema file to ensure consistency and validity.

Copy link

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 attempts to fix path resolution errors in the codeql-resolver plugin by adding .md extensions to skill paths in plugin.json. The actual skill files (codeql-permission-classification.md and github-workflow-security-patterns.md) exist as direct .md files, and Claude Code's runtime requires the .md extension in the plugin manifest to resolve them correctly. However, this change conflicts with the JSON schema validation used in CI.

Changes:

  • Add .md extensions to two skill paths in codeql-resolver plugin.json

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Skill paths in plugin.json must reference directories containing SKILL.md,
not flat .md files. The two skills were stored as flat .md files which Claude
Code cannot load via the directory-based skill loading mechanism.

- codeql-permission-classification.md -> codeql-permission-classification/SKILL.md
- github-workflow-security-patterns.md -> github-workflow-security-patterns/SKILL.md

plugin.json paths remain unchanged (no .md extension) - this is correct per schema.

(claude)
@JacobPEvans JacobPEvans force-pushed the fix/codeql-skill-paths branch from 327bfed to 853bba9 Compare February 27, 2026 16:38
@JacobPEvans JacobPEvans merged commit 81ab6af into main Feb 27, 2026
5 checks passed
@JacobPEvans JacobPEvans deleted the fix/codeql-skill-paths branch February 27, 2026 17:20
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