fails on some valid kdl syntax

[Alexthemediocre]

See AvengeMedia/DankMaterialShell#2230. This library seems to expect a space where one is not required by the kdl v1 spec, such as in the following kdl:

recent-windows {
    binds {
        Alt+Tab         { next-window scope="output"; }
        Alt+Shift+Tab   { previous-window scope="output"; }
        Alt+grave       { next-window filter="app-id"; }
        Alt+Shift+grave { previous-window filter="app-id"; }
        Alt+Escape      { next-window scope="all"; }
        Alt+Shift+Escape{ previous-window scope="all"; }
    }
}

Short reproducible example

package main

import (
	"strings"
	"fmt"
	"github.com/sblinch/kdl-go"
)

func main() {
	data := `
		recent-windows {
			binds {
				Alt+Tab         { next-window scope="output"; }
				Alt+Shift+Tab   { previous-window scope="output"; }
				Alt+grave       { next-window filter="app-id"; }
				Alt+Shift+grave { previous-window filter="app-id"; }
				Alt+Escape      { next-window scope="all"; }
				Alt+Shift+Escape{ previous-window scope="all"; }
			}
		}
	`

	_, err := kdl.Parse(strings.NewReader(data))
	if err == nil {
		fmt.Println("success")
	} else {
		fmt.Println(err)
	}
}

Above example prints:

parse failed: unexpected BraceOpen in state stateNode at line 8, column 23:
...}       Alt+Shift+Escape{...
                           ^

This same kdl snippet is considered valid by niri (e.g. niri validate) using kdl-rs.

[wheelsbot7]

I'm having a similar issue, this time with environment variables that start with an underscore.

environment {
    _JAVA_AWT_WM_NONREPARENTING "1"
}

This is considered valid by Niri, but gets the following error from kdl-go

scan failed: unexpected character _ at line 3, column 7
     _JAVA_AWT_WM_NONREPARENTING  #"1"
     ^

Also, as as far as I can tell, Niri doesn't use kdl-rs, but kdl-rs does accept both kdl snippets as valid.

Short reproducible example

use kdl;
use miette;

fn main() -> miette::Result<()> {
  let doc_str = r#"
  	recent-windows {
          binds {
              Alt+Tab         { next-window scope="output"; }
              Alt+Shift+Tab   { previous-window scope="output"; }
              Alt+grave       { next-window filter="app-id"; }
              Alt+Shift+grave { previous-window filter="app-id"; }
              Alt+Escape      { next-window scope="all"; }
              Alt+Shift+Escape{ previous-window scope="all"; }
          }
      }
  "#;
  doc_str.parse::<kdl::KdlDocument>()?;
  Ok(())
}

use kdl;
use miette;

fn main() -> miette::Result<()> {
    let doc_str = r#"
		environment {
	    	_JAVA_AWT_WM_NONREPARENTING "1"
		}
	"#;
    doc_str.parse::<kdl::KdlDocument>()?;
    Ok(())
}