feat: register #bedrockContext LM tool for Copilot Chat project context#541
Conversation
Agent-Logs-Url: https://github.com/Blockception/minecraft-bedrock-language-server/sessions/7ef8dcb3-c496-4273-a567-dff0c4f2bfc8 Co-authored-by: DaanV2 <2393905+DaanV2@users.noreply.github.com>
- Add RequestTypes.WorkspaceContext LSP request + WorkspaceContextSummary / WorkspacePackSummary types to ide/shared/src/constants.ts - Extend WorkspaceProjectDataCollections with packs arrays and add getWorkspaceContextSummary + handler in service.ts - Register blockception.minecraft.bedrockContext LM tool in language-model-tools.ts - Contribute new tool in ide/vscode/package.json with canBeReferencedInPrompt: true - Add 5 new tests for getWorkspaceContextSummary in service.test.ts Agent-Logs-Url: https://github.com/Blockception/minecraft-bedrock-language-server/sessions/7ef8dcb3-c496-4273-a567-dff0c4f2bfc8 Co-authored-by: DaanV2 <2393905+DaanV2@users.noreply.github.com>
|
@copilot there are pipelines errors during building |
…collection types Add CollectionKeys<T> helper type that filters to only keys whose values extend CollectionWithIds, preventing the packs: ReadonlyArray<PackEntry> field from being included in the key union. This fixes the 5 TypeScript errors the CI Windows build reported. Agent-Logs-Url: https://github.com/Blockception/minecraft-bedrock-language-server/sessions/33615915-486f-4c8f-81ce-03eae0f47ded Co-authored-by: DaanV2 <2393905+DaanV2@users.noreply.github.com>
Fixed in 7497cd9. The CI failure was 5 TypeScript Added a |
|
@copilot Following errors on windows pipeline: |
The dataset service test file on main imports WorkspaceProjectDataCollections from the language-model-tools service; adding packs as a required field there caused TS2741 errors in this test file too. Add packs: [] to both behaviorPacks and resourcePacks mock objects. Agent-Logs-Url: https://github.com/Blockception/minecraft-bedrock-language-server/sessions/c47c26ec-15a6-4a5d-95f1-39776d351bd3 Co-authored-by: DaanV2 <2393905+DaanV2@users.noreply.github.com>
... Fixed in 1a43926. The |
|
@copilot resolve the merge conflicts in this pull request |
…aset/service.test.ts Co-authored-by: DaanV2 <2393905+DaanV2@users.noreply.github.com>
Done in 0de3da1. There was one add/add conflict in |
|
@copilot now there are test issues |
…ntext activation event Agent-Logs-Url: https://github.com/Blockception/minecraft-bedrock-language-server/sessions/54b18e4a-ca58-4293-b796-d6f834761b4e Co-authored-by: DaanV2 <2393905+DaanV2@users.noreply.github.com>
Fixed in 6478518. The |
RequestTypes.WorkspaceContextandWorkspaceContextSummary/WorkspacePackSummarytypes toide/shared/src/constants.tsWorkspaceProjectDataCollectionswithpacksarrays and addgetWorkspaceContextSummary+ handler inide/base/server/src/lsp/language-model-tools/service.tsblockception.minecraft.bedrockContextLM tool inide/base/client/src/commands/language-model-tools.tside/vscode/package.jsonwithcanBeReferencedInPrompt: trueservice.test.ts(5 new tests, all passing)CollectionKeys<T>helper typepacks: []to mock objects indataset/service.test.tsonLanguageModelTool:blockception.minecraft.bedrockContexttoactivationEventsinide/vscode/package.json(required bylanguage-model-tools-manifest.test.ts)