Skip to content

feat: context provider api support for java - #1489

Merged
wenyt (wenytang-ms) merged 13 commits into
mainfrom
wenyt/contextprovidersupport
Sep 27, 2025
Merged

feat: context provider api support for java#1489
wenyt (wenytang-ms) merged 13 commits into
mainfrom
wenyt/contextprovidersupport

Conversation

@wenytang-ms

@wenytang-ms wenyt (wenytang-ms) commented Sep 18, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor

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
wenyt (wenytang-ms) marked this pull request as ready for review September 19, 2025 00:57
testforstephen
Jinbo Wang (testforstephen) reviewed Sep 19, 2025
View reviewed changes
Comment thread src/copilot/context/contextCache.ts Outdated
testforstephen
Jinbo Wang (testforstephen) reviewed Sep 19, 2025
View reviewed changes
Comment thread src/copilot/context/copilotHelper.ts
testforstephen
Jinbo Wang (testforstephen) reviewed Sep 19, 2025
View reviewed changes
Comment thread src/copilot/context/contextCache.ts Outdated
testforstephen
Jinbo Wang (testforstephen) reviewed Sep 19, 2025
View reviewed changes
Comment thread src/copilot/contextProvider.ts Outdated
testforstephen
Jinbo Wang (testforstephen) reviewed Sep 23, 2025
View reviewed changes
Comment thread src/copilot/context/contextCache.ts Outdated
testforstephen
Jinbo Wang (testforstephen) reviewed Sep 24, 2025
View reviewed changes
Comment thread src/copilot/context/contextCache.ts Outdated
testforstephen
Jinbo Wang (testforstephen) reviewed Sep 24, 2025
View reviewed changes
Comment thread src/copilot/context/copilotHelper.ts
testforstephen
Jinbo Wang (testforstephen) reviewed Sep 24, 2025
View reviewed changes
Comment thread src/copilot/contextProvider.ts Outdated
testforstephen
Jinbo Wang (testforstephen) reviewed Sep 24, 2025
View reviewed changes
Comment thread src/copilot/context/contextCache.ts Outdated
testforstephen
Jinbo Wang (testforstephen) reviewed Sep 24, 2025
View reviewed changes
Comment thread src/copilot/context/contextCache.ts Outdated
testforstephen
Jinbo Wang (testforstephen) reviewed Sep 25, 2025
View reviewed changes
Comment thread src/recommendation/index.ts Outdated
@chagong

Changyong Gong (chagong) commented Sep 26, 2025

Copy link
Copy Markdown
Contributor

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

@wenytang-ms

wenyt (wenytang-ms) commented Sep 26, 2025

Copy link
Copy Markdown
Contributor Author

Changyong Gong (@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

@wenytang-ms
wenyt (wenytang-ms) merged commit bfcbddd into main Sep 27, 2025
@wenytang-ms
wenyt (wenytang-ms) deleted the wenyt/contextprovidersupport branch September 27, 2025 13:49
wenyt (wenytang-ms) added a commit that referenced this pull request Nov 10, 2025
@wenytang-ms
Revert "feat: context provider api support for java (#1489)"
bb1b868 
This reverts commit bfcbddd.
wenyt (wenytang-ms) added a commit that referenced this pull request Nov 10, 2025
@wenytang-ms
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

Reviewers

@chagong Changyong Gong (chagong) chagong approved these changes
@testforstephen Jinbo Wang (testforstephen) testforstephen approved these changes
@jdneo Sheng Chen (jdneo) Awaiting requested review from jdneo jdneo is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@wenytang-ms @chagong @testforstephen