Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Context Providers Audit #3630

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Context Providers Audit #3630

wants to merge 5 commits into from

Conversation

RomneyDa
Copy link
Collaborator

@RomneyDa RomneyDa commented Jan 7, 2025

Systematic audit of context providers to make sure types are prim and proper, all are functional with normal usage in both VS code and jetbrains

Changes

  • Docs: added missing docs for problems, prompt files (not really missing but added a linking section), and greptile. Rearranged docs a bit to put more-used/basic up top
  • Added various missing config schema enum and param type values
  • Added several more representative icons for the MentionList dropdown
  • Fix some small bugs

Findings/Things to fix in future PRs

  • @OPEN Files context provider appears broken: "Cannot read properties of undefined (reading 'content')"
  • @clipboard context provider doesn't yet support Jetbrains IDE copying and in VS code there seems to be a minor LRU cache bug when things are copied externally
  • @debugger returning nothing when there are variables in the debugger - VS Code
  • @google is questionably useful: it returns top results but only links and titles, no content. Intentional?
  • @Terminal throws error when terminal is empty: "Cannot read properties of undefined (reading 'content')"
  • @github Issue returns both PRs and Issues. Intentional?
  • @docs: several pre-indexed docs are broken - seems that they are not in S3
  • @codebase, @folder, and @docs retrieval quality issues are apparent
  • @discord schema isn't right, channels param isn't actually used
  • Highlights, Outline, and Continue Proxy context providers - these are the outliers that have some code but don't seem to be in use. Let's fix or remove
  • Most context providers still don't have tests

More complex context providers to test thoroughly in near future:

database, discord, gitlab-mr, greptile, http, jira, mcp, postgres
All on Jetbrains

Copy link

netlify bot commented Jan 7, 2025

Deploy Preview for continuedev ready!

Name Link
🔨 Latest commit bedcfee
🔍 Latest deploy log https://app.netlify.com/sites/continuedev/deploys/677f09a30e265b0008a882d5
😎 Deploy Preview https://deploy-preview-3630--continuedev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@RomneyDa RomneyDa marked this pull request as ready for review January 8, 2025 14:57
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.

1 participant