Do not require access to dependencies that aren't being documented

[shs96c]

It's not always practical to add a dependency on the bzl files that come from third party dependencies. stardoc should allow documentation to be generated without introspecting deeply into dependencies.

As an example, I use MavenInfo from rules_jvm_external in my own rules for building java modules. In order to generate docs using stardoc, I must somehow also include source files from rules_jvm_external

[burkpojken]

I agree with that it would be very good to not have to depend on visibility of third party bzl files that you do not want to generated documentation for.
We see the same problem when are using skylib features in our bzl files for which we want to generate documentation. Example we have our custom config settings in config.bzl With content: load( "@bazel_skylib//rules:common_settings.bzl", "bool_flag", "string_flag", )

def our_rule( ...... ) We want to generate documentation for our_rule() Since the visibility of in the latest version of @bazel_skylib//rules:common_settings.bzl has been changed to exports_files( glob(["*.bzl"]), visibility = ["//:subpackages"], ) We cannot use stardoc anymore with the latest version of skylib. It does not feel right to request changes in skylib because of this.

[brandjon]

See #69, this should be addressed when we rewrite the doc extraction to grab real objects from the loading phase instead of mocking it in a separate process.

[brandjon]

(P4ing this one as opposed to marking it a dup, but the underlying rewrite is actually a P1 FR that we're aiming to work on this quarter and next.)

[guw]

I also started to investigate Stardoc for us and quickly ran into this.

Currently unable to build because rules_pkg also has their .bzl files defined with limited visibility.

[tmhdgsn]

has there been any progress on this at all? Currently unable to build because the rule to be documented uses java_binary/java_library from rules_java.

'documentation generation failed: File external/rules_java/java/defs.bzl imported '//java/private:native.bzl'

[tetromino]

I tried, but alas, it turned out to be too difficult given realistic time constraints. There are systems out there which rely on bazel query to produce the set of targets on which any given target - such as the .md file emitted by stardoc - depends. And bazel query can only query the target graph, not the loading graph; allowing it to query the loading graph without significantly degrading performance on large repos would be technically quite challenging (as in: quite probably several months of work). Which means we still have to express stardoc's deps in the target graph, not just in the loading graph. Which means we have to continue declaring in your BUILD file the deps of the .bzl file being documented.

I'm not closing this issue yet: although we must require access to dependencies in the Stardoc rule, it might be possible to have a separate command (which gets invoked from the terminal, not from a rule) for generating the doc. But that certainly will not happen before next year.

[tetromino]

has there been any progress on this at all? Currently unable to build because the rule to be documented uses java_binary/java_library from rules_java.

'documentation generation failed: File external/rules_java/java/defs.bzl imported '//java/private:native.bzl'

@tmhdgsn, please ensure that

• your stardoc target has in "@rules_java//java:rules" in its transitive deps; and
• you are using rules_java 5.3.2 or newer.

Does that fix the problem? If not, please add detailed repro steps and we will try to help debug.

[tmhdgsn]

has there been any progress on this at all? Currently unable to build because the rule to be documented uses java_binary/java_library from rules_java. 'documentation generation failed: File external/rules_java/java/defs.bzl imported '//java/private:native.bzl'

@tmhdgsn, please ensure that

• your stardoc target has in "@rules_java//java:rules" in its transitive deps; and
• you are using rules_java 5.3.2 or newer.

Does that fix the problem? If not, please add detailed repro steps and we will try to help debug.

somehow I had managed to miss that target "@rules_java//java:rules", thank you !! all good now.

[hauserx]

We also hit this problem as we include e.g. @bazel_tools//tools/cpp:toolchain_utils.bzl in some rules. For now we used --spawn_strategy=local as a workaround, but it stopped working between stardoc 0.5.3 and 0.5.4 (where stardoc stopped to use --dep_roots flag and started to use runfiles, see https://github.com/bazelbuild/stardoc/commit/ffcb4fb051bc5bb9b89cf47ef3eb301de9f951b7 and https://github.com/bazelbuild/bazel/commit/fde247978fe106dda36cfb92f028f3245c125886)

@tetromino you mentioned there can be some two staged approach possible, where we execute bazel first to gather all dependencies. How could it look like, how hard would it be to implement as a workaround?

Other possible solution would be to change starlark parser to allow undefined deps, possibly by traversing just starlark AST in stardoc, which could work in some cases and be available as a special mode of stardoc rule, but not sure about this solution for now.

[tetromino]

@hauserx - one approach would be for Bazel to gain a new non-build command or command-line option - perhaps a new option to bazel dump or some new flavor of bazel query - which will take a label of a .bzl file and dump out a proto that could be transformed into Markdown by Stardoc's renderer.

Open design questions:

• what this command or option should be called
• should it operate on a single .bzl file or multiple ones
• should it output in Stardoc proto format, or do we also want to retrieve some additional loading-phase data for use by other tools
• for rendering to markdown, do we recommend running the Stardoc renderer from the command line, or via the currently private stardoc_markdown_renderer rule (which would have to be cleaned up a bit and made public)?

[hauserx]

Wonder whether it would be possible for bazel to create automatic targets for .bzl files with dependencies created out of load statements. Then stardoc rules could use regular aspects and harvest all dependencies, and need for bzl_library would be replaced by such mechanism.

The .bzl files by their nature are special, so possibly special handling of those could be possible. Already loadfiles function mentions something like that, maybe it's not only stardoc that would benefit from such approach: https://bazel.build/query/language#loadfiles

For posterity, for srcs from @bazel_tools you can create bzl_libray in your own workspace as below. Using it I was able to fix our doc generation to work in newer versions of stardoc.

bzl_library(
    name = "bazel_tools_bzl",
    srcs = ["@bazel_tools//tools:bzl_srcs"],
)