draft PR for marketplace cleanup #3787
Closed
Conversation
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
- Removed src/services/package-manager/YamlParser.ts - Verified no test files or imports existed - All tests passing (1263 pass, 0 fail, 23 pending) Task ID: 4d3f7a2e-8c1b-4f9d-b5e2-9c1d8f3a6b4d
- Renamed prepare_logs to iterations - Updated task manager to use new terminology - Simplified CLI interface - Added better TypeScript types - Improved error handling Task ID: 4d3f7a2e-8c1b-4f9d-b5e2-9c1d8f3a6b4d
…tests - Removed e2e/src/suite/package-manager.test.ts - Reverted src/__mocks__/vscode.js to simpler version - Added enhanced unit tests with better coverage: * Cache directory handling * Localization testing * External items validation * Security checks * Error handling Task ID: e2e_analysis_20250412
…t-validation workflow
- Stop scanning at package boundaries - Remove items array handling (only for external refs) - Add comprehensive test coverage
- Add recursive scanning for nested components - Fix path handling for nested directories - Improve test coverage for package subcomponents - Fix timestamp handling in git-based dates
…search - Add test cases for case-insensitive matching - Add test cases for partial string matching - Add test cases using real data from package-manager-template - Verify search behavior with actual component data
…rving subcomponent handling - Replace complex word boundary/regex matching with simple string.includes() - Maintain all subcomponents in search results - Preserve proper matchInfo for packages and subcomponents
* chore: reword all `PackageManager` => `Marketplace` * chore: document refresh, `mcp-server` => `mcp` * chore: correct word * chore: remove unnecessary condition * chore: remove unnecessary bits * chore: add note for failed test
This is a breaking change that reword a lot of thing, revise the config props, and adds some more features.
* feat(wip): installation UI * chore: rebase, refactor and fixes --------- Co-authored-by: elianiva <[email protected]>
* feat: better UI for marketplace item list * feat: better source config UI * refactor: change how we fetch items * fix: update state more optimistically * fix: incorrect tags filter * fix: better tags filtering * feat: more consistent UI * feat: marketplace animation * fix: remove cache, it's fast enough * refactor: make the UI more consistent across themes * feat: integrate install metadata to UI * feat: add translation files * test: add marketplace UI
SmartManoj
pushed a commit
to SmartManoj/Raa-Code
that referenced
this pull request
Jun 13, 2025
* Stop tracking auto-generated files, respect .gitignore * migrate showAccountViewClicked --------- Co-authored-by: Elephant Lumps <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
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.
I'm just creating this PR to check the diffs and cleaning up unnecessary/unintended changes, correcting bad changes, refer to #2772 for the main marketplace PR.