Skip to content

chore(void): remove dead linkSkillsForAgents - #241

Merged
Brooooooklyn merged 1 commit into
mainfrom
07-02-chore_void_remove_dead_linkskillsforagents
Jul 2, 2026
Merged

Brooooooklyn merged 1 commit into
mainfrom
07-02-chore_void_remove_dead_linkskillsforagents

Conversation

@shulaoda

@shulaoda shulaoda commented Jul 1, 2026

Copy link
Copy Markdown
Member

What

This removes the exported linkSkillsForAgents function from packages/void/src/cli/skills.ts and drops the detectAgents import that only that function used.

Why

The function is dead code. linkSkillsForAgents is never called anywhere in the monorepo. It was superseded by linkSkillsForSpecificAgents, which is what void init actually calls, and its own doc comment falsely claimed it was used by void init. With the function gone, the detectAgents import became unused, so it is removed as well.

Safety

There is no behavior change. The function had no callers. The shared helpers parseSkills and linkSkills, the sibling linkSkillsForSpecificAgents, and the getPackageDir, AgentConfig, and log imports are all untouched because they are still used.

Verification

There are zero references to linkSkillsForAgents across packages including tests. The void package typechecks cleanly with tsgo and passes oxlint. The change is one insertion and 26 deletions.

@Brooooooklyn
Brooooooklyn merged commit 83307db into main Jul 2, 2026
6 checks passed
@Brooooooklyn
Brooooooklyn deleted the 07-02-chore_void_remove_dead_linkskillsforagents branch July 2, 2026 02:13
cpojer pushed a commit that referenced this pull request Sep 11, 2026
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
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