Skip to content

Link .agents skills into .claude/skills - #302

Merged
bholmesdev merged 1 commit into
mainfrom
claude/link-agent-skills
Oct 18, 2026
Merged

bholmesdev merged 1 commit into
mainfrom
claude/link-agent-skills

Conversation

@bholmesdev

Copy link
Copy Markdown
Owner

Description

  • Symlinks code-comments, implementation, review-cycle, taste-review, test-desktop-app, write-product-spec, write-tech-spec from .agents/skills into .claude/skills. Unlinked, Claude Code's Skill tool can't see them.
  • AGENTS.md: note that skills are written in .agents/skills and symlinked into .claude/skills.

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Documentation update

Testing

  • Existing tests pass
  • Added new tests for changes
  • Tested manually (describe below)

Manual Testing Details:
Linked skills appear in Claude Code's Skill tool list.

Checklist

  • I discussed this change in a GitHub issue before submitting this PR
  • I have run the linter, formatter, and tests to ensure my code is ready for review

🤖 Generated with Claude Code

Unlinked skills are invisible to Claude Code's Skill tool.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@vercel

vercel Bot commented Sep 28, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
hubble-md-web Ready Ready Preview Sep 28, 2026 1:18pm UTC

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Overview

Links seven existing .agents/skills (code-comments, implementation, review-cycle, taste-review, test-desktop-app, write-product-spec, write-tech-spec) into .claude/skills with the same relative symlink pattern already used there, and documents that convention in AGENTS.md.

Concerns

None. Targets exist with SKILL.md; paths match working links (../../.agents/skills/<name>); no code/test changes.

Verdict

Found: 0 critical, 0 important, 0 suggestions

Approve


Reviewed by a Warp Factory agent.

@bholmesdev

bholmesdev commented Oct 18, 2026 •

Copy link
Copy Markdown
Owner Author

Merge risk: low

Impact: small ✅

  • Contributor and agent tooling only; no app code.
  • Symlinks become plain-text files on Windows checkouts without core.symlinks. That's already true of the existing .claude/skills links.

Rollback cost: low ✅

Error detection: N/A

Verification: good ✅

  • Checked manually in Claude Code. All 18 .agents/skills entries now have a matching link.

Alignment: good ✅

@bholmesdev
bholmesdev merged commit 5fb285a into main Oct 18, 2026
7 checks passed

This branch was successfully deployed

1 active deployment
Preview — dc30e1b4 Deployed Sep 28, 2026 by vercel[bot]
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.

1 participant