Skip to content

refactor(select)!: remove selection API in favor of resources#1340

Merged
deathbeam merged 1 commit intoCopilotC-Nvim:mainfrom
deathbeam:remove-selection
Aug 25, 2025
Merged

refactor(select)!: remove selection API in favor of resources#1340
deathbeam merged 1 commit intoCopilotC-Nvim:mainfrom
deathbeam:remove-selection

Conversation

@deathbeam
Copy link
Copy Markdown
Collaborator

@deathbeam deathbeam commented Aug 17, 2025

BEGIN_COMMIT_OVERRIDE
refactor(select)!: remove selection API in favor of resources

Replaces the selection API with a unified resources mechanism for sharing
context with the LLM. Updates config, mappings, and internal logic to use
resources instead of selection. Deprecates selection-related functions and
highlights. Improves flexibility and future extensibility for context
injection.
END_COMMIT_OVERRIDE

@deathbeam deathbeam added enhancement New feature or request refactor labels Aug 17, 2025
@deathbeam deathbeam mentioned this pull request Aug 17, 2025
7 tasks
@deathbeam
Copy link
Copy Markdown
Collaborator Author

Actually not sure if this is even good idea because selection is pretty useful for highlighting etc

@deathbeam deathbeam force-pushed the remove-selection branch 2 times, most recently from df30e9d to 97937c7 Compare August 18, 2025 21:17
@deathbeam deathbeam changed the title feat(functions)!: remove selection and change it to resource function refactor(selection): remove legacy selection API Aug 18, 2025
@deathbeam deathbeam marked this pull request as ready for review August 18, 2025 21:17
@deathbeam deathbeam marked this pull request as draft August 18, 2025 21:18
@deathbeam deathbeam force-pushed the remove-selection branch 2 times, most recently from 716a11a to b763588 Compare August 25, 2025 17:41
Replaces the selection API with a unified resources mechanism for sharing
context with the LLM. Updates config, mappings, and internal logic to use
resources instead of selection. Deprecates selection-related functions and
highlights. Improves flexibility and future extensibility for context
injection.
@deathbeam deathbeam changed the title refactor(selection): remove legacy selection API refactor(core): remove selection API in favor of resources Aug 25, 2025
@deathbeam deathbeam marked this pull request as ready for review August 25, 2025 17:43
@deathbeam deathbeam merged commit a2429ed into CopilotC-Nvim:main Aug 25, 2025
4 checks passed
@deathbeam deathbeam deleted the remove-selection branch August 25, 2025 17:45
@deathbeam deathbeam changed the title refactor(core): remove selection API in favor of resources refactor(select)!: remove selection API in favor of resources Aug 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request refactor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant