Kotlin script files (.kts)

[russellw]

ktfmt looks good so far! Only deficiency I have found is that it doesn't work on script (.kts) files. Any chance of adding this?

[cgrushko]

Looks like ktfmt is parsing the file as if it were a .kt file, expecting a top-level declaration.

$ cat > ~/temp/foo.kts
println("Called with args:")
args.forEach {
    println("- $it")
}

$ java -jar ktfmt-0.13-SNAPSHOT-jar-with-dependencies.jar ~/temp/foo.kts
~/temp/foo.kts:1:1: error: Expecting a top level declaration

Something inside Parser.kt causes kotlin-compiler to expect a .kt file instead of .kts.
KotlinFileType.INSTANCE looks suspicious, but there's no kts equivalent.

[cgrushko]

Had to rollback the changes that added support for scripts, so reopening.

[jdemeulenaere]

Are there any news regarding support for Kotlin Script files? It would be very useful to us as we are currently using ktlint but we would like to migrate to ktfmt for both normal and script Kotlin files.

[cgrushko]

@jdemeulenaere glad you find ktfmt useful :)
Script support was added in c2cf421, but unfortunately had to be rolled back because it hurt preserving some newlines, as well as newlines between package and the rest of the file when there are no imports. We'll get to it soon-ish, I hope.

[cgrushko]

There's now preliminary support in the main branch. It'll do undesirable things like surrounding every statement in newlines, but at least it ktfmt no longer chokes on kts files.

[JavierSegoviaCordoba]

Is there a snapshot for it?

[cgrushko]

I uploaded a snapshot to

<dependency>
  <groupId>com.facebook</groupId>
  <artifactId>ktfmt</artifactId>
  <version>0.20-SNAPSHOT</version>
</dependency>

see https://stackoverflow.com/questions/7715321/how-to-download-snapshot-version-from-maven-snapshot-repository for hwo to add the OSSRH snapshot repo to your Maven repo list.

[MathieuLorber]

Hi, everything seems to be ok now, with just a small problem : formatting scripts starting by a hashbang will fail
#!/usr/bin/env kscript
Is there any chance to support it ?

[ZacSweers]

@cgrushko would you be open to a PR for the hashbang support via just peeking the first line and skipping it if it is a hashbang?

[cgrushko]

@ZacSweers was just implemented in d71487b :) next weekly release should pick it up.