This feature request affects the functionality of golang/tools/cmd/goimports.

What version of Go are you using (go version)?

go version go1.7.3

What operating system and processor architecture are you using (go env)?

linux/amd64

Background

1. This is not a bug report, but a feature request.
2. I would like to contribute and implement this feature.
3. The purpose of filing this issue is to indicate that I would like to work on this and to agree on the implementation.

The feature

Google released Bazel. Bazel's supports for Go is via the Skylark go_rules.

Bazel organizes all source code under a WORKSPACE directory, say /some/path/to/workspace further referred to as the bazel_workspace. The go_rules require the user to set a WORPSKACE-wide variable go_prefix which is the global prefix used to fully qualify all Go import targets. This may be yourcompany further referred to as the bazel_go_prefix.

With this setup a package named foo located in file /some/path/to/workspace/subdir/foo/foo.go should be imported as:

<bazel_go_prefix>/subdir/foo

With the above mentioned bazel_go_prefix becomes:

"yourcompany/subdir/foo"

The desired feature is to enable goimports to find packages under a Bazel WORKSPACE and to add correct import paths for these packages.

Proposed implementation

I would add two new flags into the goimports main:

1. -bazel_workspace for setting the absolute path to the Bazel's WORKSPACE .
2. -bazel_go_prefix for setting the go prefix.

The code in golang/tools/imports/fix.go would be modified as follows:

1. Two new global vars will be added to hold the values of the new flags.
2. A new function scanBazelWorkspace() will be added, that when called would scan the Bazel WORKSPACE similarly to how the Go directories are scanned today, but with Bazel specific path manipulation.
3. The scanBazelWorkspace() function would be called from findImportGoPath() if both of the new flags were set.
4. The scanBazelWorkspace() function would be called concurrently with scanGoRoot() and scanGoPath().
5. Sorting of the discovered candidates would be modified, if the two new flags were specified, the import paths within the Bazel WORKSPACE would be preferred, the rest of the candidates would be sorted as today - based on the length, preferring the shortest.

I have a prototype that I tested this locally. Please let me know if you agree with the proposal above, in which case I will start the code review process.

Alternatively feedback or suggestions are welcome.