Skip to content

Conversation

@nangua
Copy link

@nangua nangua commented Mar 11, 2025

Description

Logs output:

Unknown message type: subprocess

Checklist

  • The relevant docs, if any, have been updated or created
  • The relevant tests, if any, have been updated or created

Screenshots

Not involved.

Testing instructions

Not involved.

@netlify
Copy link

netlify bot commented Mar 11, 2025

Deploy Preview for continuedev canceled.

Name Link
🔨 Latest commit c2a21a5
🔍 Latest deploy log https://app.netlify.com/sites/continuedev/deploys/67cfbbeb8a294300092417ef

@sestinj
Copy link
Contributor

sestinj commented Mar 16, 2025

@nangua I think we are going to probably deprecate this message soon along with config.ts. I assume you are using config.ts, but could you share what your use case is for "subprocess"?

@sestinj sestinj self-requested a review March 16, 2025 19:39
@sestinj
Copy link
Contributor

sestinj commented Apr 9, 2025

@nangua I've not heard back so I'm going to close this PR. If you want to continue the conversation, please feel free to open an issue!

@sestinj sestinj closed this Apr 9, 2025
@nangua
Copy link
Author

nangua commented Apr 23, 2025

@sestinj Yes, I am indeed using config. ts,I read the official doc Custom Context Providers with Submenu or Query, which claims that ide. subprocess can be used in the Custom Context Provider. However, I happened to need to use some lightweight MCP like logic in the provider that also requires interaction with the operating system through ide. subprocess, so I used it.

For example, I need to analyze the risk of dependency components in Maven projects. In order to reduce tokens, I hope to first generate a list of dependency components for the entire project through mvn dependency:list in the Custom Context Provider, and then parse it into a compact string in getContextitems(). Although there may be other more elegant solutions, I believe the current approach is the most convenient.

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.

3 participants