chore: remove 3 dead tool-name converters + fix unused-arg lint warnings in runtime-artifact-conversion.cts

[trek-e]

Pre-submission checklist

• This does not change user-facing behavior (commands, output, file formats, config)
• I have searched existing issues — this has not already been filed

What is the maintenance task?

Clear 4 of the 12 @typescript-eslint/no-unused-vars warnings in src/runtime-artifact-conversion.cts that are pure lint hygiene: delete 3 genuinely-dead functions and rename 1 signature-placeholder argument.

Type of maintenance

Refactoring / code quality

Current state

npm run lint emits 12 no-unused-vars warnings in src/runtime-artifact-conversion.cts. Investigation split them into three buckets; this issue covers the 4 trivially-correct ones (the other 8 are an unwired-migration gap tracked in #1173):

• 3 genuinely dead — convertCursorToolName, convertWindsurfToolName, convertAugmentToolName. No call site in src/runtime-artifact-conversion.cts or bin/install.js; the corresponding agent converters perform tool-name mapping inline (regex) and never call these. Orphaned in both source and the generated installer.
• 1 signature placeholder — the runtime arg of convertClaudeCommandToClineSkill (~line 1508). All skill converters share the 5-arg signature (content, skillName, runtime, cmdNames, isGlobal); Cline needs no runtime branching so the arg is unused. Lines ~326–327 already use the _-prefix convention for exactly this case.

Warnings are deferred errors; suppressing without understanding is not acceptable — these 4 are honestly resolvable (delete what's dead, follow the existing _-prefix convention for a real placeholder).

Proposed work

• Delete convertCursorToolName, convertWindsurfToolName, convertAugmentToolName from src/runtime-artifact-conversion.cts (and from bin/install.js if it is hand-maintained rather than generated from the .cts).
• Rename the unused runtime arg of convertClaudeCommandToClineSkill to _runtime, matching the established convention.
• Rebuild (npm run build) and confirm the lint warning count drops by 4 (12 → 8, the remaining 8 owned by enhancement: wire the 8 extracted agent converters into the descriptor-driven install path (ADR-457 follow-up to #1099) #1173).
• Confirm no behavior change (runtime conversion tests stay green; gsd-test docker green).

Done when

• The 3 dead convert*ToolName functions are removed from source (and generated installer)
• convertClaudeCommandToClineSkill's placeholder arg is _runtime
• npm run lint shows 8 warnings in that file (down from 12), 0 errors
• gsd-test docker green; no user-facing change

Area affected

Core library code

Additional context

Surfaced during the #1170 lint review. Companion to #1173 (which wires the remaining 8 agent converters). Splitting keeps one-concern-per-pr: this is mechanical hygiene; #1173 is a behavioral migration.

[github-actions]

Branch chore/1175-chore-remove-3-dead-tool-name-converters created.

git fetch origin && git checkout chore/1175-chore-remove-3-dead-tool-name-converters

[trek-e]

This was generated by AI during triage.

Agent Brief

Category: enhancement (chore)
Summary: Remove three dead tool-name converters (convertCursorToolName, convertWindsurfToolName, convertAugmentToolName) and clear the associated unused-arg lint warnings.

⚠️ Scope correction from triage (investigate before starting): the three dead functions appear to live only in the generated bin/install.js, not in src/runtime-artifact-conversion.cts. The _runtime underscore-rename the issue describes already appears to be applied in the .cts source. So the real work is generator-side removal + a re-audited lint count — confirm the actual current state first.

Current behavior:
Three convert*ToolName functions are exported but have zero call sites. The issue also reports an unused-argument lint warning on a Cline command converter, but that signature already uses the _runtime underscore convention in the source module.

Desired behavior:
The three unreferenced converters are removed from wherever they are authoritatively defined (the generator source, since bin/install.js is generated), the build regenerates cleanly, and the unused-arg lint warning count drops accordingly.

Key interfaces:

• The runtime-artifact conversion surface — drop the three dead convert*ToolName exports and definitions at their authoritative (non-generated) source.

Acceptance criteria:

• The three functions have confirmed zero call sites and are removed; npm run build regenerates bin/install.js without them.
• The lint warning count drops; the remaining count is re-audited against current source (the issue's "8 remaining" target may be stale).
• gsd-test (docker) is green.

Out of scope:

• The 8 convertClaudeAgentTo* agent converters (those are bug: extracted convertClaudeAgentToCopilotAgent references undefined claudeToCopilotTools (blocks #1173 wiring) #1182/enhancement: wire the 8 extracted agent converters into the descriptor-driven install path (ADR-457 follow-up to #1099) #1173's wiring set — do not remove).
• convertClaudeCommandToClineSkill, which is live and heavily referenced — do not delete.

Sequencing: touches the same generated conversion surface as #1182/#1173 — land after or coordinate with that pair to avoid conflicts.

[github-actions]

Branch feat/1175-chore-remove-3-dead-tool-name-converters created.

git fetch origin && git checkout feat/1175-chore-remove-3-dead-tool-name-converters