Commit 95512d0
Add listLibrariesMissingMetadata task/goal for Gradle and Maven (#877)
* Introduce listMissingMetadataLibs gradle task and maven goal
* Never cache gradle listMissingMetadataLibs task
* Attempt to fetch gh authentication when it's present
* Remove brittle, hard-coded tests
* Move from testing fork ticket opening, to actual upstream oracle repo
* Fix checkstyle in listMissingMetadataLibs task
* Rename listMissingMetadataLibs to listLibrariesMissingMetadata
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* Add unit tests for direct-runtime dependency selection
- ListLibrariesMissingMetadataTest covers the Gradle path: only direct
external module dependencies survive, project components and
unresolved entries are filtered, transitive children are not walked.
- ListLibrariesMissingMetadataMojoTest covers the Maven path:
isDirectRuntimeDependency accepts compile/runtime/null scope at
trail length 2, rejects test/provided, transitive trails, missing
trail, and project-only trails.
Both tests run as plain Spock unit tests and require no GitHub access,
so they cover the dependency-selection contract without the brittle
live-metadata-repo dependency that the earlier functional tests had.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* Group listLibrariesMissingMetadata console output and surface createIssues=true
The old output interleaved each missing library's coordinates with a
long, heavily URL-encoded prefilled issue link, then trailed a dense
seven-counter summary line. The call-to-action for filing issues
automatically (createIssues=true) was only reachable by reading the
docs.
The new rendering:
- Groups results by status: "Already requested" (existing open
issues), "Requested support for N libraries" (when createIssues
succeeded), and a manual "Or request support manually" section
for the remaining ones. Errors get their own block at the end.
- Places a prominent call-to-action above the manual list that
shows the exact re-run command tailored to the build tool
(`-PcreateIssues=true` / `-DcreateIssues=true`) and the token
resolution order on the next line.
- Collapses prefilled issue URLs into numbered footnotes ([1],
[E1], [C1]) so library names line up vertically instead of
drowning in 200-character URLs.
- Trims `&maven_coordinates=...` from the prefilled URL (the title
already carries the GAV) and keeps colons readable in the title
query value.
- Appends `Full report: <path>` as a one-line footer when the
plugins pass in the report-file path via the new
renderConsoleOutput(reportFilePath) overload.
- For all-supported runs, collapses to a single sentence.
Tests in MissingMetadataCommandSupportTest are updated to assert on
the new grouped output (headline, CTA, token sources, footnote
indices) and on the trimmed prefilled URL shape.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* Fix missing metadata forced version and GitHub host resolution
* Fix ASCII-only checkstyle violation in metadata test
* Restore prefilled coordinates in metadata issue links
* Add schema for missing metadata reports
* Clarify missing metadata tip on docs landing page
* Fail listLibrariesMissingMetadata when issue creation fails
---------
Co-authored-by: Vojin Jovanovic <vojin.jovanovic@oracle.com>
Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>1 parent 0dcda78 commit 95512d0
14 files changed
Lines changed: 2406 additions & 4 deletions
File tree
- common/graalvm-reachability-metadata/src
- main/java/org/graalvm/reachability
- test/java/org/graalvm/reachability
- docs/src/docs/asciidoc
- native-gradle-plugin/src
- main/java/org/graalvm/buildtools/gradle
- tasks
- test/groovy/org/graalvm/buildtools/gradle/tasks
- native-maven-plugin/src
- main/java/org/graalvm/buildtools/maven
- test/groovy/org/graalvm/buildtools/maven
- schemas
Lines changed: 945 additions & 0 deletions
Large diffs are not rendered by default.
Lines changed: 536 additions & 0 deletions
Large diffs are not rendered by default.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
235 | 235 | | |
236 | 236 | | |
237 | 237 | | |
238 | | - | |
| 238 | + | |
| 239 | + | |
| 240 | + | |
| 241 | + | |
| 242 | + | |
| 243 | + | |
| 244 | + | |
| 245 | + | |
| 246 | + | |
| 247 | + | |
| 248 | + | |
239 | 249 | | |
240 | 250 | | |
241 | 251 | | |
| |||
443 | 453 | | |
444 | 454 | | |
445 | 455 | | |
446 | | - | |
| 456 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
333 | 333 | | |
334 | 334 | | |
335 | 335 | | |
336 | | - | |
| 336 | + | |
| 337 | + | |
| 338 | + | |
| 339 | + | |
| 340 | + | |
| 341 | + | |
| 342 | + | |
| 343 | + | |
| 344 | + | |
| 345 | + | |
| 346 | + | |
337 | 347 | | |
338 | 348 | | |
339 | 349 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
825 | 825 | | |
826 | 826 | | |
827 | 827 | | |
| 828 | + | |
| 829 | + | |
| 830 | + | |
| 831 | + | |
| 832 | + | |
| 833 | + | |
| 834 | + | |
| 835 | + | |
| 836 | + | |
| 837 | + | |
| 838 | + | |
| 839 | + | |
| 840 | + | |
| 841 | + | |
| 842 | + | |
| 843 | + | |
| 844 | + | |
| 845 | + | |
| 846 | + | |
| 847 | + | |
| 848 | + | |
| 849 | + | |
| 850 | + | |
| 851 | + | |
| 852 | + | |
| 853 | + | |
| 854 | + | |
| 855 | + | |
| 856 | + | |
| 857 | + | |
| 858 | + | |
| 859 | + | |
| 860 | + | |
| 861 | + | |
| 862 | + | |
| 863 | + | |
| 864 | + | |
| 865 | + | |
828 | 866 | | |
829 | 867 | | |
830 | 868 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
10 | 18 | | |
11 | 19 | | |
12 | 20 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
842 | 842 | | |
843 | 843 | | |
844 | 844 | | |
| 845 | + | |
| 846 | + | |
| 847 | + | |
| 848 | + | |
| 849 | + | |
| 850 | + | |
| 851 | + | |
| 852 | + | |
| 853 | + | |
| 854 | + | |
| 855 | + | |
| 856 | + | |
| 857 | + | |
| 858 | + | |
| 859 | + | |
| 860 | + | |
| 861 | + | |
| 862 | + | |
| 863 | + | |
| 864 | + | |
| 865 | + | |
| 866 | + | |
| 867 | + | |
| 868 | + | |
| 869 | + | |
| 870 | + | |
| 871 | + | |
| 872 | + | |
| 873 | + | |
| 874 | + | |
| 875 | + | |
| 876 | + | |
| 877 | + | |
| 878 | + | |
| 879 | + | |
| 880 | + | |
| 881 | + | |
| 882 | + | |
845 | 883 | | |
846 | 884 | | |
847 | 885 | | |
| |||
Lines changed: 31 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
61 | 61 | | |
62 | 62 | | |
63 | 63 | | |
| 64 | + | |
64 | 65 | | |
65 | 66 | | |
66 | 67 | | |
| |||
71 | 72 | | |
72 | 73 | | |
73 | 74 | | |
| 75 | + | |
74 | 76 | | |
75 | 77 | | |
76 | 78 | | |
| |||
326 | 328 | | |
327 | 329 | | |
328 | 330 | | |
| 331 | + | |
| 332 | + | |
| 333 | + | |
| 334 | + | |
| 335 | + | |
| 336 | + | |
| 337 | + | |
| 338 | + | |
| 339 | + | |
| 340 | + | |
| 341 | + | |
| 342 | + | |
| 343 | + | |
| 344 | + | |
| 345 | + | |
| 346 | + | |
| 347 | + | |
| 348 | + | |
| 349 | + | |
| 350 | + | |
| 351 | + | |
| 352 | + | |
| 353 | + | |
| 354 | + | |
| 355 | + | |
| 356 | + | |
| 357 | + | |
| 358 | + | |
| 359 | + | |
329 | 360 | | |
330 | 361 | | |
331 | 362 | | |
| |||
Lines changed: 174 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
| 41 | + | |
| 42 | + | |
| 43 | + | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
| 47 | + | |
| 48 | + | |
| 49 | + | |
| 50 | + | |
| 51 | + | |
| 52 | + | |
| 53 | + | |
| 54 | + | |
| 55 | + | |
| 56 | + | |
| 57 | + | |
| 58 | + | |
| 59 | + | |
| 60 | + | |
| 61 | + | |
| 62 | + | |
| 63 | + | |
| 64 | + | |
| 65 | + | |
| 66 | + | |
| 67 | + | |
| 68 | + | |
| 69 | + | |
| 70 | + | |
| 71 | + | |
| 72 | + | |
| 73 | + | |
| 74 | + | |
| 75 | + | |
| 76 | + | |
| 77 | + | |
| 78 | + | |
| 79 | + | |
| 80 | + | |
| 81 | + | |
| 82 | + | |
| 83 | + | |
| 84 | + | |
| 85 | + | |
| 86 | + | |
| 87 | + | |
| 88 | + | |
| 89 | + | |
| 90 | + | |
| 91 | + | |
| 92 | + | |
| 93 | + | |
| 94 | + | |
| 95 | + | |
| 96 | + | |
| 97 | + | |
| 98 | + | |
| 99 | + | |
| 100 | + | |
| 101 | + | |
| 102 | + | |
| 103 | + | |
| 104 | + | |
| 105 | + | |
| 106 | + | |
| 107 | + | |
| 108 | + | |
| 109 | + | |
| 110 | + | |
| 111 | + | |
| 112 | + | |
| 113 | + | |
| 114 | + | |
| 115 | + | |
| 116 | + | |
| 117 | + | |
| 118 | + | |
| 119 | + | |
| 120 | + | |
| 121 | + | |
| 122 | + | |
| 123 | + | |
| 124 | + | |
| 125 | + | |
| 126 | + | |
| 127 | + | |
| 128 | + | |
| 129 | + | |
| 130 | + | |
| 131 | + | |
| 132 | + | |
| 133 | + | |
| 134 | + | |
| 135 | + | |
| 136 | + | |
| 137 | + | |
| 138 | + | |
| 139 | + | |
| 140 | + | |
| 141 | + | |
| 142 | + | |
| 143 | + | |
| 144 | + | |
| 145 | + | |
| 146 | + | |
| 147 | + | |
| 148 | + | |
| 149 | + | |
| 150 | + | |
| 151 | + | |
| 152 | + | |
| 153 | + | |
| 154 | + | |
| 155 | + | |
| 156 | + | |
| 157 | + | |
| 158 | + | |
| 159 | + | |
| 160 | + | |
| 161 | + | |
| 162 | + | |
| 163 | + | |
| 164 | + | |
| 165 | + | |
| 166 | + | |
| 167 | + | |
| 168 | + | |
| 169 | + | |
| 170 | + | |
| 171 | + | |
| 172 | + | |
| 173 | + | |
| 174 | + | |
0 commit comments