sourceroot not used by cli tool

[stuhood]

The CLI tool does not use the sourceroot value that is passed in, and the result is that the files that are operated on are always the ones relative to CWD.

The Input values that come from the scalameta database will be Input.LabeledString (afaict), with relative paths names as the labels. scalafix will need to convert these to absolute paths using the supplied sourceroot.

[olafurpg]

Good catch! In sbt, the working directory is the same as CWD so this didn't get caugt in tests.

The absolute path is inferred from the working directory here:

scalafix/scalafix-cli/src/main/scala/scalafix/cli/CliRunner.scala

Line 82 in 13e9681

def path = input.path(common.workingPath)

The CliRunner should instead use a sourceroot field passed as a constructor parameter here

scalafix/scalafix-cli/src/main/scala/scalafix/cli/CliRunner.scala

Line 40 in 13e9681

inputs: Seq[Input],

I will try to take a look at this at the spree today.

PS. This reminds me scalafix should have a RemoveUnusedDefns rewrite to remove unused definitions like resolvedSourceroot.

[stuhood]

Thanks for #183!