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

Add autoInclude param for mentions #199

Merged
merged 1 commit into from
Sep 2, 2024
Merged

Conversation

thenamankumar
Copy link
Member

  • Removed unused mentionSelectors params.
  • Added meta.mentions.autoInclude to auto-include mentions as initial context.
  • Added uri? and codebase? params for mentions.

@thenamankumar thenamankumar merged commit 50490a4 into main Sep 2, 2024
12 checks passed
@thenamankumar thenamankumar deleted the naman/auto-include branch September 2, 2024 07:24
thenamankumar added a commit to sourcegraph/cody that referenced this pull request Sep 3, 2024
context:
https://linear.app/sourcegraph/issue/PROD-264/one-pager-for-how-to-get-this-set-up-using-openctx

We have [added](sourcegraph/openctx#199)
`meta.mentions.autoInclude` property for OpenCtx providers. The
providers which set that to `true` will be used to include the mentions
they return as initial context items. We are also passing the active
file uri and codebase names to the `mentions` function.


## Test plan
Set the following as the vs code setting and see the hello world mention
item chip appearing as the default context item.
```json
"openctx.providers": {
        "https://openctx.org/npm/@openctx/provider-hello-world": true
}
```

## Changelog

Include initial context from OpenCtx providers.
bevzzz pushed a commit to bevzzz/openctx that referenced this pull request Dec 12, 2024
- Removed unused mentionSelectors params.
- Added `meta.mentions.autoInclude` to auto-include mentions as initial
context.
- Added `uri?` and `codebase?` params for `mentions`.
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