Skip to content

Conversation

@plusplusoneplusplus
Copy link
Owner

Summary

This PR removes unused functionality from the Wu Wei agent panel system. The codebase has evolved to exclusively use which includes prompt integration and TSX-based composition.

Changes Made

  • AgentPanelProvider: Removed unused case from message handler
  • AgentPanelProvider: Removed method that was no longer called
  • Extension: Removed VS Code command registration
  • Package.json: Removed command definition from VS Code extension manifest

Technical Details

The system architecture has moved to a prompt-centric approach where:

  • All agent requests must include a prompt template
  • The UI enforces prompt selection before sending requests
  • Enhanced TSX-based prompt composition is used
  • The handler provides sophisticated prompt enhancement logic

Testing

  • Verified no references to removed functionality remain
  • Confirmed UI still enforces prompt selection
  • Validated that path works correctly
  • No breaking changes to existing functionality

Code Quality

  • Removes 82 lines of unused code
  • Simplifies message handling logic
  • Maintains backward compatibility for existing prompt-based workflows
  • Follows conventional commit format

This refactoring improves code maintainability by removing dead code paths while preserving all active functionality.

- Remove unused sendAgentRequest case from message handler
- Remove handleAgentRequest method that's no longer called
- Remove wu-wei.sendAgentRequest VS Code command registration
- Remove command definition from package.json

The system now exclusively uses sendAgentRequestWithPrompt which
includes prompt integration and TSX-based composition. (Cursor)
@plusplusoneplusplus plusplusoneplusplus enabled auto-merge (squash) June 28, 2025 23:09
@plusplusoneplusplus plusplusoneplusplus merged commit bb5cda1 into main Jun 28, 2025
14 checks passed
@plusplusoneplusplus plusplusoneplusplus deleted the refactor/remove-unused-sendAgentRequest branch June 28, 2025 23:14
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