feat: context provider api support for java #1489

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.

Sign up for GitHub

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

Jump to bottom

Merged

wenytang-ms merged 13 commits into main from wenyt/contextprovidersupport

Sep 27, 2025

Contributor

wenytang-ms commented Sep 18, 2025 •

edited

Loading

https://exp.microsoft.com/features/e770d1ba-bcd3-540b-9eec-fcd3e15a0117?workspaceId=dbd12384-5001-4e90-a863-526321eaf233

Testing against some popular java repo
jbang
Mixin
Velocity
java deps cost - execute java command time cost
pack cost - execute context provider to collect and return context whole time cost

file url	java-deps cost	pack cost
Configuration.java	9.82 ms	13 ms 7 items
PathTypeAdapter.java	4.36ms	15ms 0 items
VersionChecker.java	4.6ms	13ms 3 items
RuntimeDecompiler.java	20ms	25ms 4 items
ModLauncherClassTracker.java	7.74ms	19ms 3items
CookieReceiveEvent.java	4.5ms	8ms 6 items
ResultEvent.java	5.2 ms	12 ms 0 items

wenytang-ms added 4 commits

September 18, 2025 15:20


          feat: context provider api support for java

86c38e4


          feat: register registerCopilotContextProviders

0d78220


          feat: register registerCopilotContextProviders

1896f00


          feat: update

1d257e9

wenytang-ms marked this pull request as ready for review

September 19, 2025 00:57

wenytang-ms requested review from chagong, jdneo and testforstephen as code owners

September 19, 2025 00:57

chagong previously approved these changes

View reviewed changes

testforstephen reviewed

View reviewed changes

src/copilot/context/contextCache.ts Outdated Show resolved Hide resolved

testforstephen reviewed

View reviewed changes

src/copilot/context/copilotHelper.ts Show resolved Hide resolved

testforstephen reviewed

View reviewed changes

src/copilot/context/contextCache.ts Outdated Show resolved Hide resolved

testforstephen reviewed

View reviewed changes

src/copilot/contextProvider.ts Outdated Show resolved Hide resolved


          fix: update code according to the comments

42a47ce

wenytang-ms dismissed chagong’s stale review via

42a47ce

September 22, 2025 03:23

testforstephen reviewed

View reviewed changes

src/copilot/context/contextCache.ts Outdated Show resolved Hide resolved


          fix: update the docu hash logic

d982414

testforstephen reviewed

View reviewed changes

src/copilot/context/contextCache.ts Outdated Show resolved Hide resolved

testforstephen reviewed

View reviewed changes

src/copilot/context/copilotHelper.ts Show resolved Hide resolved

testforstephen reviewed

View reviewed changes

src/copilot/contextProvider.ts Outdated Show resolved Hide resolved

testforstephen reviewed

View reviewed changes

src/copilot/context/contextCache.ts Outdated Show resolved Hide resolved

testforstephen reviewed

View reviewed changes

src/copilot/context/contextCache.ts Outdated Show resolved Hide resolved

wenytang-ms added 3 commits

September 25, 2025 09:37


          fix: update code

c88b1fb


          fix: update the code logic according to comments

465260a


          fix: update code according to comments

363c754

testforstephen reviewed

View reviewed changes

src/recommendation/index.ts Outdated Show resolved Hide resolved

wenytang-ms added 4 commits

September 25, 2025 14:04


          feat: remove cache design

d730437


          feat: add telemetry data report

1ba9788


          feat: remove get java version

ada5f17


          fix: update

718abf5

Contributor

chagong commented Sep 26, 2025

I don't see anywhere you check exp feature flag. is it something all handled by exp platform?

Contributor Author

wenytang-ms commented Sep 26, 2025

@chagong Copilot-client will fetch the feature from exp, and turn on the context provider apis, then java-pack will turn on this process.
the Exp vars is here.
https://github.com/microsoft/copilot-client/blob/7afc1d2731be3258a746182ce13a646d39c19f0a/lib/src/experiments/expConfig.ts#L88

chagong approved these changes

View reviewed changes

testforstephen approved these changes

View reviewed changes

wenytang-ms merged commit bfcbddd into main

wenytang-ms deleted the wenyt/contextprovidersupport branch

September 27, 2025 13:49

wenytang-ms added a commit that referenced this pull request


          Revert "feat: context provider api support for java (#1489)"

bb1b868

This reverts commit bfcbddd.

wenytang-ms added a commit that referenced this pull request


          revert: revert all the context provider implement in java pack (#1533)

b9901a7

* Revert "fix: update the project uri (#1520)"

This reverts commit 2712e70.

* Revert "feat: Enhance the context returned by the context provider (#1518)"

This reverts commit 2e90376.

* Revert "feat: context provider api support for java (#1489)"

This reverts commit bfcbddd.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet