-
Notifications
You must be signed in to change notification settings - Fork 67
Add skills-ref package and integrate with codex-acp #1988
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Co-authored-by: steveej <[email protected]>
1e0a771 to
cf04a17
Compare
|
@copilot are you able to upstream the skills-ref package to nixpkgs first? |
I'm unable to submit PRs to nixpkgs as that requires human maintainer involvement and access to that repository. However, the current implementation in this repository serves the immediate need for codex-acp users. If you'd like to upstream skills-ref to nixpkgs, that would be a great contribution - once it's merged there, we could switch this package to use the nixpkgs version instead. The package is relatively simple (Python app with hatchling, click, and strictyaml dependencies), so upstreaming should be straightforward when you're ready to do so. |
|
@copilot please generate a diff that introduces the skills-ref package to nixpkgs. it must apply cleanly to nixpkgs/master. include the diff in a nix code block in a new comment. |
Here's the diff to add skills-ref to nixpkgs/master: diff --git a/pkgs/by-name/sk/skills-ref/package.nix b/pkgs/by-name/sk/skills-ref/package.nix
new file mode 100644
index 000000000000..1111111111111
--- /dev/null
+++ b/pkgs/by-name/sk/skills-ref/package.nix
@@ -0,0 +1,43 @@
+{
+ lib,
+ python3,
+ fetchFromGitHub,
+}:
+
+python3.pkgs.buildPythonApplication rec {
+ pname = "skills-ref";
+ version = "0.1.0";
+ pyproject = true;
+
+ src = fetchFromGitHub {
+ owner = "agentskills";
+ repo = "agentskills";
+ rev = "main";
+ hash = "sha256-nZLgAd+ixQtWknKew5M9N1xr8Bo1xbTmPFSvxcYcgS4=";
+ };
+
+ sourceRoot = "source/skills-ref";
+
+ build-system = with python3.pkgs; [
+ hatchling
+ ];
+
+ dependencies = with python3.pkgs; [
+ click
+ strictyaml
+ ];
+
+ pythonImportsCheck = [ "skills_ref" ];
+
+ meta = {
+ description = "Reference library for Agent Skills";
+ homepage = "https://github.com/agentskills/agentskills/tree/main/skills-ref";
+ license = lib.licenses.asl20;
+ maintainers = with lib.maintainers; [ ];
+ mainProgram = "skills-ref";
+ platforms = lib.platforms.all;
+ };
+}This uses the |
|
|
||
| postInstall = '' | ||
| wrapProgram $out/bin/codex-acp \ | ||
| --prefix PATH : ${lib.makeBinPath [ skills-ref ]} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't think there is any reason to do this.
| passthru.category = "Utilities"; | ||
|
|
||
| meta = with lib; { | ||
| description = "Reference library for Agent Skills"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Description could be better here. It's not quiet clear what it is actually for.
|
I'll close this because
|
Codex lacks access to the
skills-reftool when working with agent skills repositories, preventing it from validating changes.Changes
New package:
packages/skills-ref/codex-acp integration:
packages/codex-acp/The skills-ref CLI is now available when codex-acp runs, enabling skill repository validation workflows.
Original prompt
skills-reffrom agentskills #1987💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.