@tishlang/tish-lint 3.11.0 → 3.12.0

package/crates/tish_lint/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "tishlang_lint"
-version = "3.11.0"
+version = "3.12.0"
 edition = "2021"
 description = "AST-based linter for Tish"
 license-file = { workspace = true }

package/crates/tish_native/src/build.rs

@@ -45,6 +45,13 @@ fn runtime_features_for_cargo(features: &[String]) -> Vec<String> {
 }
 
 /// `[profile.release]` for nested `cargo build` of generated crates.
+///
+/// - default: fat LTO, one codegen unit — smallest and fastest binary, slowest to produce (the
+///   final link is single-threaded and dominates a warm build of a large app).
+/// - `TISH_FAST_NATIVE_BUILD=1`: iteration profile, no LTO, opt-level 1.
+/// - `TISH_NATIVE_THIN_LTO=1`: thin LTO over 8 codegen units — keeps most of fat's size/speed win
+///   while letting rustc parallelise the link. Same trade the GBA path offers via
+///   `TISH_GBA_THIN_LTO`; for desktop apps this is the CI-friendly shipping profile.
 fn nested_release_profile_toml() -> &'static str {
     if std::env::var("TISH_FAST_NATIVE_BUILD").as_deref() == Ok("1") {
         r#"[profile.release]
@@ -55,6 +62,13 @@ incremental = true
 strip = false
 debug = 0
 panic = "abort"
+"#
+    } else if std::env::var("TISH_NATIVE_THIN_LTO").as_deref() == Ok("1") {
+        r#"[profile.release]
+strip = true
+panic = "abort"
+codegen-units = 8
+lto = "thin"
 "#
     } else {
         r#"[profile.release]
@@ -1179,6 +1193,7 @@ edition = "2021"
 
 #[cfg(test)]
 mod tests {
+    use super::nested_release_profile_toml;
     use super::runtime_features_for_cargo;
 
     /// The generated ROM crate must inherit the project's pin, or GBA builds float against whatever
@@ -1219,6 +1234,27 @@ mod tests {
         let _ = fs::remove_dir_all(&base);
     }
 
+    /// `TISH_NATIVE_THIN_LTO=1` selects thin LTO for native release builds; fast wins over it.
+    #[test]
+    fn native_release_profile_follows_env() {
+        use std::sync::Mutex;
+        static ENV_LOCK: Mutex<()> = Mutex::new(());
+        let _g = ENV_LOCK.lock().unwrap_or_else(|e| e.into_inner());
+        std::env::remove_var("TISH_FAST_NATIVE_BUILD");
+        std::env::remove_var("TISH_NATIVE_THIN_LTO");
+        assert!(nested_release_profile_toml().contains("lto = \"fat\""));
+        std::env::set_var("TISH_NATIVE_THIN_LTO", "1");
+        let thin = nested_release_profile_toml();
+        assert!(thin.contains("lto = \"thin\"") && thin.contains("codegen-units = 8"));
+        std::env::set_var("TISH_FAST_NATIVE_BUILD", "1");
+        assert!(
+            nested_release_profile_toml().contains("lto = false"),
+            "fast wins over thin"
+        );
+        std::env::remove_var("TISH_FAST_NATIVE_BUILD");
+        std::env::remove_var("TISH_NATIVE_THIN_LTO");
+    }
+
     /// `TISH_NATIVE_TARGET_DIR`: unset/empty → None (temp-dir build); set → that dir, created.
     #[test]
     fn shared_native_target_dir_follows_env() {

package/crates/tish_runtime/src/fs_ext.rs

@@ -218,9 +218,25 @@ pub fn copy_file_core(args: &[Value]) -> Result<Value, Value> {
 }
 pub fn realpath_core(args: &[Value]) -> Result<Value, Value> {
     std::fs::canonicalize(path_arg(args, 0))
-        .map(|p| Value::String(p.to_string_lossy().into()))
+        .map(|p| Value::String(strip_verbatim_prefix(&p.to_string_lossy()).into()))
         .map_err(io_err)
 }
+
+/// `std::fs::canonicalize` on Windows returns the verbatim form — `\\?\C:\Users\…` for a drive
+/// path, `\\?\UNC\server\share\…` for a network one. Node's `fs.realpath` never does: it hands
+/// back `C:\Users\…` / `\\server\share\…`, and programs treat the result as a display / URL /
+/// comparison string, not just an OS handle (Dune showed `\\?\C:\…` as its workspace title, and a
+/// `?` inside a URL path starts the query string). Match Node. No-op on POSIX and for paths
+/// that carry no prefix.
+pub(crate) fn strip_verbatim_prefix(p: &str) -> String {
+    if let Some(rest) = p.strip_prefix("\\\\?\\UNC\\") {
+        format!("\\\\{rest}")
+    } else if let Some(rest) = p.strip_prefix("\\\\?\\") {
+        rest.to_string()
+    } else {
+        p.to_string()
+    }
+}
 pub fn readlink_core(args: &[Value]) -> Result<Value, Value> {
     std::fs::read_link(path_arg(args, 0))
         .map(|p| Value::String(p.to_string_lossy().into()))
@@ -393,3 +409,16 @@ pub fn constants() -> Value {
     m.insert("X_OK".into(), Value::Number(1.0));
     Value::object(m)
 }
+
+#[cfg(test)]
+mod verbatim_prefix_tests {
+    use super::strip_verbatim_prefix;
+
+    #[test]
+    fn strips_windows_verbatim_prefixes_like_node() {
+        assert_eq!(strip_verbatim_prefix("\\\\?\\C:\\Users\\x\\ui"), "C:\\Users\\x\\ui");
+        assert_eq!(strip_verbatim_prefix("\\\\?\\UNC\\srv\\share\\d"), "\\\\srv\\share\\d");
+        assert_eq!(strip_verbatim_prefix("C:\\Users\\x"), "C:\\Users\\x");
+        assert_eq!(strip_verbatim_prefix("/Users/x/ui"), "/Users/x/ui");
+    }
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tishlang/tish-lint",
-  "version": "3.11.0",
+  "version": "3.12.0",
   "description": "Tish code linter",
   "license": "PIF",
   "repository": {