Repository navigation
fix: link the discover-candidates skills into .claude/skills - #82
Merged
Merged
Conversation
.claude/skills/ symlinked eight of the ten public skills into skills/, omitting discover-company-candidates and discover-university-candidates. Both are part of the public skill set and both are delegation targets: find-university-chairs and find-company-thesis-options instruct the agent to invoke them for live candidate discovery, so that step dead-ended in any session using the project skill directory. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_015mBAtaUv5Br2nML9wYLpej
ValentinJSchmidt
requested review from
dxmme and
mxs01
and
a lite review from Copilot
September 7, 2026 05:46
Contributor
There was a problem hiding this comment.
🟢 Approval recommended
The change is limited to adding the two missing skill symlinks and aligns with existing delegation references and the expected skill set.
Pull request overview
This PR fixes the .claude/skills/ skill registry by adding the two missing discovery-skill symlinks so delegation from find-university-chairs / find-company-thesis-options can actually resolve and run in agent environments that rely on .claude/skills/.
Changes:
- Add
.claude/skills/discover-company-candidates -> ../../skills/discover-company-candidates(symlink) - Add
.claude/skills/discover-university-candidates -> ../../skills/discover-university-candidates(symlink)
File summaries
| File | Description |
|---|---|
.claude/skills/discover-company-candidates |
Adds missing symlink so agents can invoke discover-company-candidates via .claude/skills/. |
.claude/skills/discover-university-candidates |
Adds missing symlink so agents can invoke discover-university-candidates via .claude/skills/. |
Review details
- Files reviewed: 2/2 changed files
- Comments generated: 0
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Collaborator
Author
|
This is not relevant for the submission |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
.claude/skills/symlinked eight of the ten public skills intoskills/, but neverdiscover-company-candidatesordiscover-university-candidates. This adds the twomissing symlinks so the project agent can actually invoke them.
Motivation
Both are delegation targets, not optional helpers:
skills/find-university-chairs/SKILL.md:36— "Invokediscover-university-candidates"skills/find-company-thesis-options/SKILL.md:35— "Invokediscover-company-candidates"Both are also in
EXPECTED_SKILLSinskills/tests/test_skill_package.py. Without thesymlinks, an agent running either discovery track dead-ends at the candidate step.
Git history shows they were never linked — an oversight in 1d43de7, not a deliberate
exclusion.
What Changed
.claude/skills/discover-company-candidates -> ../../skills/discover-company-candidates.claude/skills/discover-university-candidates -> ../../skills/discover-university-candidatesStored as symlinks (mode
120000), matching the existing eight, so they stay in syncwith
skills/rather than drifting as copies.How to Run / Test
Verified end-to-end in a session: both skills registered and were used to produce a
full university + company option map.
🤖 Generated with Claude Code
https://claude.ai/code/session_015mBAtaUv5Br2nML9wYLpej