Skip to content

#1722: implement copilot url updater#1805

Open
MarvMa wants to merge 7 commits intodevonfw:mainfrom
MarvMa:feature/#1722-copilot-url-updater
Open

#1722: implement copilot url updater#1805
MarvMa wants to merge 7 commits intodevonfw:mainfrom
MarvMa:feature/#1722-copilot-url-updater

Conversation

@MarvMa
Copy link
Copy Markdown
Contributor

@MarvMa MarvMa commented Apr 9, 2026

This PR fixes #1722

Implemented changes:

  • Added CopilotUrlUpdater using GithubUrlReleaseUpdater

Checklist for this PR

Make sure everything is checked before merging this PR. For further info please also see
our DoD.

  • When running mvn clean test locally all tests pass and build is successful
  • PR title is of the form #«issue-id»: «brief summary» (e.g. #921: fixed setup.bat). If no issue ID exists, title only.
  • PR top-level comment summarizes what has been done and contains link to addressed issue(s)
  • PR and issue(s) have suitable labels
  • Issue is set to In Progress and assigned to you or there is no issue (might happen for very small PRs)
  • You followed all coding conventions
  • You have added the issue implemented by your PR in CHANGELOG.adoc unless issue is labeled
    with internal

@github-project-automation github-project-automation bot moved this to 🆕 New in IDEasy board Apr 9, 2026
@MarvMa MarvMa marked this pull request as draft April 9, 2026 07:49
@coveralls
Copy link
Copy Markdown
Collaborator

coveralls commented Apr 9, 2026

Coverage Report for CI Build 24190181285

Coverage increased (+0.03%) to 70.5%

Details

  • Coverage increased (+0.03%) from the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • 17 coverage regressions across 1 file.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

17 previously-covered lines in 1 file lost coverage.

File Lines Losing Coverage Coverage
com/devonfw/tools/ide/url/updater/UpdateManager.java 17 0.0%

Coverage Stats

Coverage Status
Relevant Lines: 15083
Covered Lines: 11085
Line Coverage: 73.49%
Relevant Branches: 6690
Covered Branches: 4265
Branch Coverage: 63.75%
Branches in Coverage %: Yes
Coverage Strength: 3.1 hits per line

💛 - Coveralls

@MarvMa
Copy link
Copy Markdown
Contributor Author

MarvMa commented Apr 9, 2026

According to the NVD Search there is no CPE for copilot-cli. But theres a CVE https://nvd.nist.gov/vuln/detail/CVE-2026-29783 that describes a security vulnerability for this tool. As long as we have the current implementation of CVE checks I would prefer to leave the cpe vendor and product empty, to prevent confusion.

@MarvMa MarvMa changed the title Feature/#1722 copilot url updater #1722: implement copilot url updater Apr 9, 2026
@MarvMa MarvMa added enhancement New feature or request AI artificial intelligence, GenAI, etc. copilot labels Apr 9, 2026
@MarvMa MarvMa moved this from 🆕 New to 👀 In review in IDEasy board Apr 9, 2026
@MarvMa MarvMa marked this pull request as ready for review April 9, 2026 08:14
@MarvMa MarvMa self-assigned this Apr 9, 2026
@MarvMa MarvMa marked this pull request as draft April 9, 2026 08:22
@MarvMa MarvMa marked this pull request as ready for review April 9, 2026 12:31
@MarvMa MarvMa moved this from 👀 In review to Team Review in IDEasy board Apr 9, 2026
@jakozian jakozian requested review from jakozian and removed request for jakozian April 9, 2026 12:41
@ducminh02 ducminh02 self-requested a review April 10, 2026 06:37
@ducminh02 ducminh02 self-assigned this Apr 10, 2026
@MarvMa MarvMa added the internal Nothing to be added to CHANGELOG, only internal story label Apr 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AI artificial intelligence, GenAI, etc. copilot enhancement New feature or request internal Nothing to be added to CHANGELOG, only internal story

Projects

Status: Team Review

Development

Successfully merging this pull request may close these issues.

Create CopilotUrlUpdater

3 participants