Expose list of all artifacts, including transitive

README.md

@@ -452,6 +452,9 @@ To use it, add the load statement to the top of your BUILD file:
 load("@rules_jvm_external//:defs.bzl", "artifact")
 ```
 
+Full `group:artifact:[packaging:[classifier:]]version` maven coordinates are also
+supported and translate to corresponding versionless target.
+
 Note that usage of this macro makes BUILD file refactoring with tools like
 `buildozer` more difficult, because the macro hides the actual target label at
 the syntax level.
@@ -855,6 +858,31 @@ maven_install(
 )
 ```
 
+It is also possible to change strict visibility value from default `//visibility:private`
+to a value specified by `strict_visibility_value` attribute.
+
+### Accessing transitive dependencies list
+
+It is possible to retrieve full list of dependencies in the dependency tree, including
+transitive, source, javadoc and other artifacts. `maven_artifacts` list contains full
+versioned maven coordinate strings of all dependencies.
+
+For example:
+```python
+load("@maven//:defs.bzl", "maven_artifacts")
+
+load("@rules_jvm_external//:defs.bzl", "artifact")
+load("@rules_jvm_external//:specs.bzl", "parse")
+
+all_jar_coordinates = [c for c in maven_artifacts if parse.parse_maven_coordinate(c).get("packaging", "jar") == "jar"]
+all_jar_targets = [artifact(c) for c in all_jar_coordinates]
+
+java_library(
+  name = "depends_on_everything",
+  runtime_deps = all_jar_targets,
+)
+```
+
 ### Fetch and resolve timeout
 
 The default timeout to fetch and resolve artifacts is 600 seconds.  If you need

coursier.bzl

@@ -471,11 +471,13 @@ def _pinned_coursier_fetch_impl(repository_ctx):
         "load(\"@bazel_tools//tools/build_defs/repo:utils.bzl\", \"maybe\")",
         "def pinned_maven_install():",
     ]
+    maven_artifacts = []
     netrc_entries = {}
 
     for artifact in dep_tree["dependencies"]:
         if artifact.get("url") != None:
             http_file_repository_name = escape(artifact["coord"])
+            maven_artifacts.extend([artifact["coord"]])
             http_files.extend([
                 "    http_file(",
                 "        name = \"%s\"," % http_file_repository_name,
@@ -499,6 +501,8 @@ def _pinned_coursier_fetch_impl(repository_ctx):
 
     http_files.extend(_get_jq_http_files())
 
+    http_files.extend(["maven_artifacts = [\n%s\n]" % (",\n".join(["    \"%s\"" % artifact for artifact in maven_artifacts]))])
+
     repository_ctx.file("defs.bzl", "\n".join(http_files), executable = False)
     repository_ctx.file(
         "netrc",

private/coursier_utilities.bzl

@@ -46,6 +46,10 @@ def strip_packaging_and_classifier(coord):
     return ":".join(coordinates)
 
 def strip_packaging_and_classifier_and_version(coord):
+    coordinates = coord.split(":")
+    # Support for simplified versionless groupId:artifactId coordinate format
+    if len(coordinates) == 2:
+        return ":".join(coordinates)
     return ":".join(strip_packaging_and_classifier(coord).split(":")[:-1])
 
 # TODO: Should these methods be combined with _parse_maven_coordinate_string in specs.

private/rules/artifact.bzl

@@ -1,19 +1,25 @@
 load("//:specs.bzl", "parse")
 load("//private:constants.bzl", "DEFAULT_REPOSITORY_NAME")
+load("//private:coursier_utilities.bzl", "strip_packaging_and_classifier_and_version")
 
 def artifact(a, repository_name = DEFAULT_REPOSITORY_NAME):
-    artifact_obj = _parse_artifact_str(a) if type(a) == "string" else a
-    return "@%s//:%s" % (repository_name, _escape(artifact_obj["group"] + ":" + artifact_obj["artifact"]))
+    artifact_str = _make_artifact_str(a) if type(a) != "string" else a
+    return "@%s//:%s" % (repository_name, _escape(strip_packaging_and_classifier_and_version(artifact_str)))
 
 def maven_artifact(a):
     return artifact(a, repository_name = DEFAULT_REPOSITORY_NAME)
 
 def _escape(string):
     return string.replace(".", "_").replace("-", "_").replace(":", "_")
 
-def _parse_artifact_str(artifact_str):
-    pieces = artifact_str.split(":")
-    if len(pieces) == 2:
-        return {"group": pieces[0], "artifact": pieces[1]}
-    else:
-        return parse.parse_maven_coordinate(artifact_str)
+# inverse of parse_maven_coordinate
+def _make_artifact_str(artifact_obj):
+    # produce either simplified g:a or standard g:a:[p:[c:]]v Maven coordinate string
+    coord = [artifact_obj["group"], artifact_obj["artifact"]]
+    if "version" in artifact_obj:
+        if "packaging" in artifact_obj:
+           coord.extend([artifact_obj["packaging"]])
+           if "classifier" in artifact_obj:
+             coord.extend([artifact_obj["classifier"]])
+        coord.extend([artifact_obj["version"]])
+    return ":".join(coord)

tests/unit/coursier_utilities_test.bzl

@@ -80,6 +80,12 @@ def _strip_packaging_and_classifier_and_version_test_impl(ctx):
         "groupId:artifactId",
         strip_packaging_and_classifier_and_version("groupId:artifactId:pom:sources:version"),
     )
+    # versionless coordinates aren't standard Maven coordinates but are useful for the artifact() macro
+    asserts.equals(
+        env,
+        "groupId:artifactId",
+        strip_packaging_and_classifier_and_version("groupId:artifactId"),
+    )
     return unittest.end(env)
 
 strip_packaging_and_classifier_and_version_test = unittest.make(_strip_packaging_and_classifier_and_version_test_impl)