Skip to content

feat(vscode): Marketplace#341

Open
bernaferrari wants to merge 5 commits intoKilo-Org:devfrom
bernaferrari:marketplace
Open

feat(vscode): Marketplace#341
bernaferrari wants to merge 5 commits intoKilo-Org:devfrom
bernaferrari:marketplace

Conversation

@bernaferrari
Copy link
Contributor

@bernaferrari bernaferrari commented Feb 16, 2026

There are a few minor differences with Kilo current extension:

  • You can apply mcps to project or globally
  • You can specify the API on the marketplace screen

If you want full fidelity I can do that too. I think project/globally is probably a feature that opencode provides, so not sure we want or not to use it.

The only thing missing: I need to translate some strings into every other language. I just want to be sure you like the PR before I do that.

@github-actions
Copy link
Contributor

Hey! Your PR title feat(vscode) Marketplace doesn't follow conventional commit format.

Please update it to start with one of:

  • feat: or feat(scope): new feature
  • fix: or fix(scope): bug fix
  • docs: or docs(scope): documentation changes
  • chore: or chore(scope): maintenance tasks
  • refactor: or refactor(scope): code refactoring
  • test: or test(scope): adding or updating tests

Where scope is the package name (e.g., app, desktop, kilo).

See CONTRIBUTING.md for details.

@bernaferrari bernaferrari changed the title feat(vscode) Marketplace feat(vscode): Marketplace Feb 16, 2026
@github-actions
Copy link
Contributor

Thanks for your contribution!

This PR doesn't have a linked issue. All PRs must reference an existing issue.

Please:

  1. Open an issue describing the bug/feature (if one doesn't exist)
  2. Add Fixes #<number> or Closes #<number> to this PR description

See CONTRIBUTING.md for details.

* Fetch extension settings for the authenticated user/organization.
* These settings include org-managed Marketplace policy fields.
*/
export async function fetchExtensionSettings(
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm not sure where this came from, if it's in the plan or based no the current extension, but we dont actually have org settings in our current marketplace. Roo does, and the code is there in the kilocode extension, but that is actually dead. Can you remove all this organization checking logic?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, I'll get rid of that

Copy link
Contributor

@markijbema markijbema left a comment

Choose a reason for hiding this comment

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

The org stuff shouldnt be there, and those are also the changes that span extension and cli-backend. Can you please revert those? Then there is some small stuff, but I can do that postmerge as well if you want (like adding other translations, which isnt hard but just costs a lot of tokens)

@bernaferrari
Copy link
Contributor Author

Yes, I even did that in a separate commit, but without them the marketplace won't work, is that OK?

@markijbema
Copy link
Contributor

Why wouldnt it work, the marketplace is publicly available endpoints like http://app.kilo.ai/api/marketplace/mcps

There was functionality in the old extension to disable certain parts of the marketplace for orgs (which never did anything in kilo, but was from roo), but the marketplace should just work fine as is?

@bernaferrari
Copy link
Contributor Author

I'm still a noob on this. Got it working. Thanks for your patience. I also fixed translations. I made another PR (much simpler) improving the profile UI while trying to not mess with server.

@bernaferrari bernaferrari marked this pull request as ready for review February 17, 2026 19:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants