[bug] New indent is added to ignored code block on each run (of formatting)

[Andrew15-5]

Before formatting

#let template(doc) = {
  //typstfmt::off
  show figure.where(kind: table): it => align(center)[
    #it.body
  ]
  //typstfmt::on

  // END ======================================================================
  doc
}

#let template(doc) = [
  //typstfmt::off
  #show figure.where(kind: table): it => align(center)[
    #it.body
  ]
  //typstfmt::on

  // END ======================================================================
  #doc
]

#let template(doc) = [
//typstfmt::off
#show figure.where(kind: table): it => align(center)[
  #it.body
]
//typstfmt::on

// END ======================================================================
#doc
]

After 1st run

#let template(doc) = {
  //typstfmt::off
  show figure.where(kind: table): it => align(center)[
      #it.body
    ]
    //typstfmt::on

  // END ======================================================================
  doc
}

#let template(doc) = [
//typstfmt::off
  #show figure.where(kind: table): it => align(center)[
    #it.body
  ]
  //typstfmt::on

// END ======================================================================
#doc
]

#let template(doc) = [
//typstfmt::off
#show figure.where(kind: table): it => align(center)[
  #it.body
]
//typstfmt::on

// END ======================================================================
#doc
]

After 10th run

#let template(doc) = {
  //typstfmt::off
  show figure.where(kind: table): it => align(center)[
                        #it.body
                      ]
                      //typstfmt::on

  // END ======================================================================
  doc
}

#let template(doc) = [
//typstfmt::off
  #show figure.where(kind: table): it => align(center)[
    #it.body
  ]
  //typstfmt::on

// END ======================================================================
#doc
]

#let template(doc) = [
//typstfmt::off
#show figure.where(kind: table): it => align(center)[
  #it.body
]
//typstfmt::on

// END ======================================================================
#doc
]

Smaller examples:

#let template(doc) = {
  //typstfmt::off
  let a = ""
  //typstfmt::on
  // END ======================================================================
  doc
}

#let template(doc) = {
  let a = ""
  //typstfmt::off
  //typstfmt::on
  // END ======================================================================
  doc
}

Config:

ident_space = 2
max_line_length = 80
experimental_args_breaking_consecutive = false

Version: https://github.com/astrale-sharp/typstfmt/commit/578d39fb304020d0c26118e4eeab272868c9d525

[Andrew15-5]

Here, block isn't ignored at all:

#{
  let a = ""

  // typstfmt::off
  ```anylang
  anything
  ```
  // typstfmt::on
}

[astrale-sharp]

wow this skipped my attention, I'll have to take a look, thanks!

[astrale-sharp]

I have an idea of what's going on but it's going to be a pain to fix I suspect

[astrale-sharp]

This is fixed in code blocks but testing would need to be done for content blocks, args (breaking and tight) etc

[jeffa5]

Related case: https://github.com/astrale-sharp/typstfmt/pull/101

[LDemetrios]

Happens not only in ignored blocks.

/* multiline
    comments */

also do float each typstfmt call, and so do raw blocks

[astrale-sharp]

Happens not only in ignored blocks.

/* multiline
    comments */

also do float each typstfmt call, and so do raw blocks

That's supposed to be fixed, are you using the last version? (not on vscode yet probably)

[miliog]

Any updates on this? This is a major reason why I can't use typstfmt yet.

[Andrew15-5]

I think ignore code block still doesn't work as intended and to fix this and many other things, a big refactoring needs to be done to make code more maintainable. At least this information was accurate about a month ago.

This is indeed a major feature without which the formatting becomes much less flexible. If I want to put 1 row per line in a table, this feature would make it possible and can increase readability of tables. Right now, formatter puts 1 cell per line. BTW, this could be a new feature where typstfmt would read the number of columns and try to put x amount of args per line (opt-in or opt-out option).

But there are many other cases where ignoring only part of a file would be beneficial.

[astrale-sharp]

Yes I would love to see dev in this area as well, It's quite difficult for me to handle it right now