Skip to content

refactor: remove unused imports from platform handlers and tests#1

Merged
nesjett merged 1 commit into
mainfrom
refactor/format-check
Feb 5, 2026
Merged

refactor: remove unused imports from platform handlers and tests#1
nesjett merged 1 commit into
mainfrom
refactor/format-check

Conversation

@nesjett

@nesjett nesjett commented Feb 5, 2026

Copy link
Copy Markdown
Owner

Removes unused imports.

Copilot AI review requested due to automatic review settings February 5, 2026 20:09
@github-actions

github-actions Bot commented Feb 5, 2026

Copy link
Copy Markdown
Contributor

LCOV of commit 83d8e5e during CI #2

Summary coverage rate:
  lines......: 98.7% (1207 of 1223 lines)
  functions..: 100.0% (84 of 84 functions)
  branches...: 99.1% (232 of 234 branches)

Files changed coverage rate: n/a

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR removes unused imports from platform handlers and test files to improve code cleanliness and maintainability. The changes are purely cosmetic refactoring with no functional impact.

Changes:

  • Removed unused type imports from platform handlers (AgentConfig, Platform, Skill, Rule types)
  • Removed unused test helper imports (writeFixture, readFixture)
  • Removed unused utility imports (log function)

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.

Show a summary per file
File Description
tests/platforms/base_test.ts Removed unused AgentConfig and Platform type imports
tests/commands/list_test.ts Removed unused writeFixture test helper import
tests/commands/import_test.ts Removed unused readFixture test helper import
src/platforms/windsurf.ts Removed unused Skill type import
src/platforms/cursor.ts Removed unused Rule type import (kept Skill as it's used)
src/platforms/claude.ts Removed unused Skill type import
src/commands/list.ts Removed unused log utility import

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@nesjett nesjett merged commit 2b54fcf into main Feb 5, 2026
7 checks passed
@nesjett nesjett deleted the refactor/format-check branch February 5, 2026 20:12
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