lightningcss 1.33.0.1-arm-linux-gnu → 1.33.0.2-arm-linux-gnu

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f996660960f0f35d06985936300415a8ce668ab5e8bc127662b0ea4d23196ea3
-  data.tar.gz: e0b2dc6dfa403e093bbb3da2308cdc422032e72e620a70c59cc625fa32e9afdc
+  metadata.gz: 5e39ac08bd6aa2cddde39814ee33d33e4f93aed2d42ebba1b26bfbbe762263ad
+  data.tar.gz: '090e13ec9176bfb0e3f33b38b329e508258357bffaac40bdd8010766b7f928aa'
 SHA512:
-  metadata.gz: c45fd06eff33a4ab77982392a9a1399f7103018a3039cea43306ba3876820b9bf7282d1d9988dbdf7ed809f43eef95d2b3f008e44b1ae33dfdb739a4a1a8a5bf
-  data.tar.gz: 6a5dbf61055622e5f5864905bfc36911a9db0f54a1e2e90cd94b3174a8ababc828965d409c1ff8b067067267b6d28eb8460a79238bc425bb3a2338ff36039e7b
+  metadata.gz: 1259d0850b3ba1263611a438ead3d90ac26ba35dc3a009c8187d16f44f9df3cc8865a2d053da01500148d821dce361d7e7395ddae8ae860c972338b6e2785290
+  data.tar.gz: 9fea6113eaab9550d06211fa958d0053a67b2510318944c4f515ca2c4efe84725183dbfe6f88ee7da51688b4e3ee12a4229f5ff3bee36734c60a5560c9d2ec4a

data/README.md

@@ -57,13 +57,25 @@ LightningCSS.minify(".a { color: #ff0000 }")
 
 #### Targeting browsers
 
-Give the browsers you support, each as its major version. Anything they cannot read is lowered, with the original kept after it so that a browser which understands it still wins.
+Give the browsers you support, each with the oldest version you support. Anything they cannot read is lowered, with the original kept after it so that a browser which understands it still wins.
 
 ```ruby
 LightningCSS.transform(".a { color: lab(50% 40 59) }", targets: { chrome: 80 }, minify: true).code
 #=> ".a{color:#bf5702;color:lab(50% 40 59)}"
 ```
 
+A version is an `Integer` for a major version, or a `Float` or `String` for a minor or patch release. Several features first shipped in a minor Safari release, so the minor version can change the output.
+
+```ruby
+LightningCSS.transform(".a { color: oklch(60% .2 30) }", targets: { safari: 15 }, minify: true).code
+#=> ".a{color:lab(51.6908% 61.9679 48.0969)}"
+
+LightningCSS.transform(".a { color: oklch(60% .2 30) }", targets: { safari: "15.4" }, minify: true).code
+#=> ".a{color:oklch(60% .2 30)}"
+```
+
+Prefer a `String` for a minor version above 9, since `16.10` as a `Float` is `16.1`.
+
 #### Bundling
 
 `bundle` resolves the `@import` statements a stylesheet was written with by reading the files they name, so it takes a path on disk to the entry stylesheet.
@@ -83,7 +95,7 @@ result.exports
 #=> {"application" => "_8Z4fiW_application"}
 
 result.warnings.first
-#=> "'deep' is not recognized as a valid pseudo-class. ... at app/assets/stylesheets/layout.css:0:9"
+#=> "'deep' is not recognized as a valid pseudo-class. ... at app/assets/stylesheets/layout.css:1:9"
 ```
 
 #### CSS modules
@@ -271,7 +283,7 @@ It answers `call` as well, so it can be handed to anything expecting something c
 | `filename`       | `String`       | The name to use in errors and warnings.                                                |
 | `minify`         | `bool`         | Whether to print the result as small as it goes.                                       |
 | `error_recovery` | `bool`         | Whether to carry on past a rule it cannot read. Off by default, so such a rule raises. |
-| `targets`        | `Hash`         | The browsers being compiled for, each as its major version.                            |
+| `targets`        | `Hash`         | The browsers being compiled for, each with the oldest version supported.               |
 | `css_modules`    | `bool`, `Hash` | Whether to compile as a CSS module, and how.                                           |
 | `scope`          | `String`       | A selector fragment to narrow every rule by.                                           |
 
@@ -325,13 +337,40 @@ result = LightningCSS.transform(".a:deep(.b) { color: red }", minify: true)
 result.warnings?
 #=> true
 
-result.warnings.first
-#=> "'deep' is not recognized as a valid pseudo-class. Did you mean '::deep' (pseudo-element) or is this a typo? at :0:9"
+result.warnings.first.to_s
+#=> "'deep' is not recognized as a valid pseudo-class. Did you mean '::deep' (pseudo-element) or is this a typo? at :1:9"
 
 result.code
 #=> ".a:deep(.b){color:red}"
 ```
 
+### Locations
+
+A warning is a `LightningCSS::Warning` with a `message`, a `type`, and a `location`, the same as a diagnostic in Prism or Herb. A `LightningCSS::Error` carries the same `type` and `location`, and keeps the location in its message.
+
+A `LightningCSS::Location` counts lines from 1 and columns from 0, the same as Prism, Herb, and Yerba, and prints the way an editor reads it:
+
+```ruby
+css = ".a { color: red; }\n.b, { color: blue; }\n"
+
+begin
+  LightningCSS.transform(css, filename: "main.css")
+rescue LightningCSS::ParseError => error
+  error.message  #=> "Invalid empty selector at main.css:2:5"
+  error.type     #=> "SelectorError"
+  error.location #=> #<data LightningCSS::Location filename="main.css", line=2, column=4>
+end
+
+warning = LightningCSS.transform(css, filename: "main.css", error_recovery: true).warnings.first
+
+warning.message       #=> "Invalid empty selector"
+warning.type          #=> "SelectorError"
+warning.location.line #=> 2
+warning.to_s          #=> "Invalid empty selector at main.css:2:5"
+```
+
+A warning converts to the string it always was wherever a `String` is expected and is equal to it, so `warnings.include?("...")` still works. An error without a location, such as an `OptionError`, has `nil` for `type` and `location`.
+
 ### Development
 
 The gem is a C extension over a Rust crate. `rust/` builds a static library and generates the C header with [cbindgen](https://github.com/mozilla/cbindgen), `ext/lightningcss/` wraps it, and `lib/` is the Ruby API over that.

data/ext/lightningcss/lightningcss.c

@@ -43,12 +43,12 @@ static VALUE error_class_for(enum LightningCssErrorCode code) {
 
 static VALUE unwrap(struct LightningCssResult result) {
   if (result.error) {
-    VALUE message = make_utf8_string(result.error);
+    VALUE payload = make_utf8_string(result.error);
     VALUE error_class = error_class_for(result.code);
 
     lightningcss_result_free(result);
 
-    rb_raise(error_class, "%s", StringValueCStr(message));
+    rb_exc_raise(rb_funcall(error_class, rb_intern("from_json"), 1, payload));
   }
 
   if (!result.value) {

data/lib/lightningcss/errors.rb

@@ -1,7 +1,28 @@
 # frozen_string_literal: true
 
 module LightningCSS
-  class Error < StandardError; end
+  class Error < StandardError
+    attr_reader :type #: String?
+    attr_reader :location #: LightningCSS::Location?
+
+    #: (String) -> instance
+    def self.from_json(payload)
+      parsed = JSON.parse(payload)
+      location = parsed["location"] && Location.from_hash(parsed["location"])
+      message = parsed.fetch("message")
+
+      new(location ? "#{message} at #{location}" : message, type: parsed["type"], location: location)
+    end
+
+    #: (?String?, ?type: String?, ?location: LightningCSS::Location?) -> void
+    def initialize(message = nil, type: nil, location: nil)
+      super(message)
+
+      @type = type
+      @location = location
+    end
+  end
+
   class ParseError < Error; end
   class OptionError < Error; end
   class BundleError < Error; end

data/lib/lightningcss/location.rb

@@ -0,0 +1,21 @@
+# frozen_string_literal: true
+
+module LightningCSS
+  Location = Data.define(
+    :filename, #: String?
+    :line, #: Integer
+    :column #: Integer
+  )
+
+  class Location
+    #: (Hash[String, untyped]) -> LightningCSS::Location
+    def self.from_hash(hash)
+      new(filename: hash["filename"], line: hash.fetch("line"), column: hash.fetch("column"))
+    end
+
+    #: () -> String
+    def to_s
+      "#{filename}:#{line}:#{column + 1}"
+    end
+  end
+end

data/lib/lightningcss/parse_result.rb

@@ -9,7 +9,7 @@ module LightningCSS
   class ParseResult
     attr_reader :source #: String?
     attr_reader :stylesheet #: Hash[String, untyped]
-    attr_reader :warnings #: Array[String]
+    attr_reader :warnings #: Array[LightningCSS::Warning]
 
     #: (String, ?String?) -> LightningCSS::ParseResult
     def self.from_json(payload, source = nil)
@@ -18,11 +18,11 @@ module LightningCSS
       new(
         source: source,
         stylesheet: parsed.fetch("stylesheet"),
-        warnings: parsed.fetch("warnings")
+        warnings: parsed.fetch("warnings").map { |warning| Warning.from_hash(warning) }
       )
     end
 
-    #: (stylesheet: Hash[String, untyped], warnings: Array[String], ?source: String?) -> void
+    #: (stylesheet: Hash[String, untyped], warnings: Array[LightningCSS::Warning], ?source: String?) -> void
     def initialize(stylesheet:, warnings:, source: nil)
       @source = source
       @stylesheet = stylesheet.freeze
@@ -44,7 +44,7 @@ module LightningCSS
     #: () -> String
     def inspect
       parts = ["rules=#{stylesheet.fetch("rules").length}"]
-      parts << "warnings=#{warnings.inspect}" if warnings?
+      parts << "warnings=#{warnings.map(&:to_s).inspect}" if warnings?
 
       "#<#{self.class.name} #{parts.join(" ")}>"
     end

data/lib/lightningcss/result.rb

@@ -13,7 +13,7 @@ module LightningCSS
   class Result
     attr_reader :code #: String
     attr_reader :exports #: Hash[String, String]?
-    attr_reader :warnings #: Array[String]
+    attr_reader :warnings #: Array[LightningCSS::Warning]
 
     #: (String) -> LightningCSS::Result
     def self.from_json(payload)
@@ -22,11 +22,11 @@ module LightningCSS
       new(
         code: parsed.fetch("code"),
         exports: parsed["exports"],
-        warnings: parsed.fetch("warnings")
+        warnings: parsed.fetch("warnings").map { |warning| Warning.from_hash(warning) }
       )
     end
 
-    #: (code: String, warnings: Array[String], ?exports: Hash[String, String]?) -> void
+    #: (code: String, warnings: Array[LightningCSS::Warning], ?exports: Hash[String, String]?) -> void
     def initialize(code:, warnings:, exports: nil)
       @code = code
       @exports = exports
@@ -49,7 +49,7 @@ module LightningCSS
     def inspect
       parts = ["code=#{code.inspect}"]
       parts << "exports=#{exports.inspect}" if exports
-      parts << "warnings=#{warnings.inspect}" if warnings?
+      parts << "warnings=#{warnings.map(&:to_s).inspect}" if warnings?
 
       "#<#{self.class.name} #{parts.join(" ")}>"
     end

data/lib/lightningcss/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module LightningCSS
-  VERSION = "1.33.0.1"
+  VERSION = "1.33.0.2"
 end

data/lib/lightningcss/warning.rb

@@ -0,0 +1,60 @@
+# frozen_string_literal: true
+
+module LightningCSS
+  class Warning
+    attr_reader :message #: String
+    attr_reader :type #: String?
+    attr_reader :location #: LightningCSS::Location?
+
+    #: (Hash[String, untyped]) -> LightningCSS::Warning
+    def self.from_hash(hash)
+      new(
+        message: hash.fetch("message"),
+        type: hash["type"],
+        location: hash["location"] && Location.from_hash(hash["location"])
+      )
+    end
+
+    #: (message: String, ?type: String?, ?location: LightningCSS::Location?) -> void
+    def initialize(message:, type: nil, location: nil)
+      @message = message
+      @type = type
+      @location = location
+
+      freeze
+    end
+
+    #: () -> Hash[Symbol, untyped]
+    def to_h
+      { message: message, type: type, location: location&.to_h }
+    end
+
+    #: () -> String
+    def to_s
+      location ? "#{message} at #{location}" : message
+    end
+
+    alias to_str to_s
+
+    #: (untyped) -> bool
+    def ==(other)
+      case other
+      when Warning then to_h == other.to_h
+      when String then to_s == other
+      else false
+      end
+    end
+
+    alias eql? ==
+
+    #: () -> Integer
+    def hash
+      to_h.hash
+    end
+
+    #: () -> String
+    def inspect
+      "#<#{self.class.name} #{to_s.inspect}>"
+    end
+  end
+end

data/lib/lightningcss.rb

@@ -3,6 +3,7 @@
 require "json"
 
 require_relative "lightningcss/version"
+require_relative "lightningcss/location"
 require_relative "lightningcss/errors"
 require_relative "lightningcss/backend"
 
@@ -14,6 +15,7 @@ rescue LoadError
 end
 
 require_relative "lightningcss/options"
+require_relative "lightningcss/warning"
 require_relative "lightningcss/result"
 require_relative "lightningcss/node"
 require_relative "lightningcss/parse_result"

data/rust/src/lib.rs

@@ -21,7 +21,7 @@ use lightningcss::bundler::{BundleErrorKind, Bundler, FileProvider};
 use lightningcss::stylesheet::{MinifyOptions, ParserOptions, PrinterOptions, StyleAttribute, StyleSheet};
 use lightningcss::visitor::Visit;
 
-use crate::options::{ParseResult, TransformOptions, TransformResult};
+use crate::options::{Diagnostic, ParseResult, TransformOptions, TransformResult};
 use crate::scope::Scoper;
 
 pub const VERSION: &str = env!("GEM_VERSION");
@@ -56,7 +56,7 @@ impl LightningCssResult {
   fn err(failure: Failure) -> Self {
     Self {
       value: ptr::null_mut(),
-      error: into_c_string(failure.message),
+      error: into_c_string(serde_json::to_string(&failure.diagnostic).unwrap_or(failure.diagnostic.message)),
       code: failure.code,
     }
   }
@@ -64,35 +64,35 @@ impl LightningCssResult {
 
 pub struct Failure {
   code: LightningCssErrorCode,
-  message: String,
+  diagnostic: Diagnostic,
 }
 
 impl Failure {
-  fn parse(message: impl Into<String>) -> Self {
+  fn parse(diagnostic: impl Into<Diagnostic>) -> Self {
     Self {
       code: LightningCssErrorCode::Parse,
-      message: message.into(),
+      diagnostic: diagnostic.into(),
     }
   }
 
-  fn option(message: impl Into<String>) -> Self {
+  fn option(diagnostic: impl Into<Diagnostic>) -> Self {
     Self {
       code: LightningCssErrorCode::Option,
-      message: message.into(),
+      diagnostic: diagnostic.into(),
     }
   }
 
-  fn bundle(message: impl Into<String>) -> Self {
+  fn bundle(diagnostic: impl Into<Diagnostic>) -> Self {
     Self {
       code: LightningCssErrorCode::Bundle,
-      message: message.into(),
+      diagnostic: diagnostic.into(),
     }
   }
 
-  fn internal(message: impl Into<String>) -> Self {
+  fn internal(diagnostic: impl Into<Diagnostic>) -> Self {
     Self {
       code: LightningCssErrorCode::Internal,
-      message: message.into(),
+      diagnostic: diagnostic.into(),
     }
   }
 }
@@ -135,11 +135,12 @@ fn parse_source<'i>(code: &'i str, options: &TransformOptions) -> Result<ParseRe
     ..ParserOptions::default()
   };
 
-  let stylesheet = StyleSheet::parse(code, parser_options).map_err(|error| Failure::parse(error.to_string()))?;
+  let stylesheet =
+    StyleSheet::parse(code, parser_options).map_err(|error| Failure::parse(Diagnostic::from_error(&error)))?;
 
   let warnings = collected
     .read()
-    .map(|warnings| warnings.iter().map(|warning| warning.to_string()).collect())
+    .map(|warnings| warnings.iter().map(Diagnostic::from_error).collect())
     .unwrap_or_default();
 
   Ok(ParseResult { stylesheet, warnings })
@@ -162,7 +163,8 @@ fn transform_source(code: &str, options: &TransformOptions) -> Result<TransformR
     ..ParserOptions::default()
   };
 
-  let mut stylesheet = StyleSheet::parse(code, parser_options).map_err(|error| Failure::parse(error.to_string()))?;
+  let mut stylesheet =
+    StyleSheet::parse(code, parser_options).map_err(|error| Failure::parse(Diagnostic::from_error(&error)))?;
 
   if let Some(fragment) = &options.scope {
     let mut scoper = Scoper::parse(fragment).map_err(Failure::option)?;
@@ -172,16 +174,14 @@ fn transform_source(code: &str, options: &TransformOptions) -> Result<TransformR
       .map_err(|error| Failure::internal(format!("Failed to scope stylesheet: {error:?}")))?;
   }
 
-  let targets = options.to_targets();
+  let targets = options.to_targets().map_err(Failure::option)?;
 
-  if options.minify {
-    stylesheet
-      .minify(MinifyOptions {
-        targets,
-        ..MinifyOptions::default()
-      })
-      .map_err(|error| Failure::internal(format!("Failed to minify: {error}")))?;
-  }
+  stylesheet
+    .minify(MinifyOptions {
+      targets,
+      ..MinifyOptions::default()
+    })
+    .map_err(|error| Failure::internal(format!("Failed to minify: {error}")))?;
 
   let printed = stylesheet
     .to_css(PrinterOptions {
@@ -203,7 +203,7 @@ fn transform_source(code: &str, options: &TransformOptions) -> Result<TransformR
 
   let warnings = collected
     .read()
-    .map(|warnings| warnings.iter().map(|warning| warning.to_string()).collect())
+    .map(|warnings| warnings.iter().map(Diagnostic::from_error).collect())
     .unwrap_or_default();
 
   Ok(TransformResult {
@@ -234,8 +234,8 @@ fn bundle_source(path: &str, options: &TransformOptions) -> Result<TransformResu
   let mut stylesheet = bundler
     .bundle(std::path::Path::new(path))
     .map_err(|error| match &error.kind {
-      BundleErrorKind::ParserError(_) => Failure::parse(error.to_string()),
-      _ => Failure::bundle(error.to_string()),
+      BundleErrorKind::ParserError(_) => Failure::parse(Diagnostic::from_error(&error)),
+      _ => Failure::bundle(Diagnostic::from_error(&error)),
     })?;
 
   if let Some(fragment) = &options.scope {
@@ -246,16 +246,14 @@ fn bundle_source(path: &str, options: &TransformOptions) -> Result<TransformResu
       .map_err(|error| Failure::internal(format!("Failed to scope stylesheet: {error:?}")))?;
   }
 
-  let targets = options.to_targets();
+  let targets = options.to_targets().map_err(Failure::option)?;
 
-  if options.minify {
-    stylesheet
-      .minify(MinifyOptions {
-        targets,
-        ..MinifyOptions::default()
-      })
-      .map_err(|error| Failure::internal(format!("Failed to minify: {error}")))?;
-  }
+  stylesheet
+    .minify(MinifyOptions {
+      targets,
+      ..MinifyOptions::default()
+    })
+    .map_err(|error| Failure::internal(format!("Failed to minify: {error}")))?;
 
   let printed = stylesheet
     .to_css(PrinterOptions {
@@ -274,7 +272,7 @@ fn bundle_source(path: &str, options: &TransformOptions) -> Result<TransformResu
 
   let warnings = collected
     .read()
-    .map(|warnings| warnings.iter().map(|warning| warning.to_string()).collect())
+    .map(|warnings| warnings.iter().map(Diagnostic::from_error).collect())
     .unwrap_or_default();
 
   Ok(TransformResult {
@@ -291,9 +289,10 @@ fn transform_attribute(code: &str, options: &TransformOptions) -> Result<Transfo
     ..ParserOptions::default()
   };
 
-  let mut attribute = StyleAttribute::parse(code, parser_options).map_err(|error| Failure::parse(error.to_string()))?;
+  let mut attribute =
+    StyleAttribute::parse(code, parser_options).map_err(|error| Failure::parse(Diagnostic::from_error(&error)))?;
 
-  let targets = options.to_targets();
+  let targets = options.to_targets().map_err(Failure::option)?;
 
   attribute.minify(MinifyOptions {
     targets,

data/rust/src/options.rs

@@ -1,6 +1,7 @@
 use std::collections::HashMap;
 
 use lightningcss::css_modules::{Config as CssModulesConfig, Pattern};
+use lightningcss::error::Error;
 use lightningcss::targets::{Browsers, Targets};
 use serde::{Deserialize, Serialize};
 
@@ -10,7 +11,7 @@ pub struct TransformOptions {
   pub filename: Option<String>,
   pub minify: bool,
   pub error_recovery: bool,
-  pub targets: Option<HashMap<String, u32>>,
+  pub targets: Option<HashMap<String, Version>>,
   pub css_modules: Option<CssModulesOptions>,
   pub scope: Option<String>,
 }
@@ -60,16 +61,61 @@ impl CssModulesOptions {
   }
 }
 
+#[derive(Debug, Deserialize)]
+#[serde(untagged)]
+pub enum Version {
+  Major(u32),
+  Float(f64),
+  String(String),
+}
+
+impl Version {
+  /// Encodes a version the way Lightning CSS reads it, as `major << 16 | minor << 8 | patch`.
+  fn encode(&self) -> Option<u32> {
+    match self {
+      Version::Major(major) => (*major <= 0xffff).then(|| major << 16),
+      Version::Float(float) => Self::parse(&float.to_string()),
+      Version::String(string) => Self::parse(string),
+    }
+  }
+
+  fn parse(version: &str) -> Option<u32> {
+    let mut parts = version.trim().split('.');
+
+    let major: u16 = parts.next()?.parse().ok()?;
+    let minor: u8 = parts.next().map_or(Some(0), |part| part.parse().ok())?;
+    let patch: u8 = parts.next().map_or(Some(0), |part| part.parse().ok())?;
+
+    if parts.next().is_some() {
+      return None;
+    }
+
+    Some(u32::from(major) << 16 | u32::from(minor) << 8 | u32::from(patch))
+  }
+
+  fn describe(&self) -> String {
+    match self {
+      Version::Major(major) => major.to_string(),
+      Version::Float(float) => float.to_string(),
+      Version::String(string) => format!("{string:?}"),
+    }
+  }
+}
+
 impl TransformOptions {
-  pub fn to_targets(&self) -> Targets {
+  pub fn to_targets(&self) -> Result<Targets, String> {
     let Some(versions) = &self.targets else {
-      return Targets::default();
+      return Ok(Targets::default());
     };
 
     let mut browsers = Browsers::default();
 
     for (name, version) in versions {
-      let encoded = Some(version << 16);
+      let encoded = Some(
+        version
+          .encode()
+          .ok_or_else(|| format!("Invalid version {} for {name} in targets", version.describe()))?,
+      );
 
       match name.as_str() {
         "android" => browsers.android = encoded,
@@ -85,14 +131,14 @@ impl TransformOptions {
       }
     }
 
-    Targets::from(browsers)
+    Ok(Targets::from(browsers))
   }
 }
 
 #[derive(Debug, Serialize)]
 pub struct ParseResult<T> {
   pub stylesheet: T,
-  pub warnings: Vec<String>,
+  pub warnings: Vec<Diagnostic>,
 }
 
 #[derive(Debug, Serialize)]
@@ -100,5 +146,66 @@ pub struct TransformResult {
   pub code: String,
   #[serde(skip_serializing_if = "Option::is_none")]
   pub exports: Option<HashMap<String, String>>,
-  pub warnings: Vec<String>,
+  pub warnings: Vec<Diagnostic>,
+}
+
+#[derive(Debug, Serialize)]
+pub struct Diagnostic {
+  pub message: String,
+  #[serde(rename = "type", skip_serializing_if = "Option::is_none")]
+  pub kind: Option<String>,
+  #[serde(skip_serializing_if = "Option::is_none")]
+  pub location: Option<Location>,
+}
+
+#[derive(Debug, Serialize)]
+pub struct Location {
+  pub filename: Option<String>,
+  pub line: u32,
+  pub column: u32,
+}
+
+impl Diagnostic {
+  pub fn from_error<T: std::fmt::Display + Serialize>(error: &Error<T>) -> Self {
+    Self {
+      message: error.kind.to_string(),
+      kind: serde_json::to_value(&error.kind).ok().as_ref().and_then(type_of),
+      location: error.loc.as_ref().map(|location| Location {
+        filename: (!location.filename.is_empty()).then(|| location.filename.clone()),
+        line: location.line + 1,
+        column: location.column.saturating_sub(1),
+      }),
+    }
+  }
+}
+
+impl From<String> for Diagnostic {
+  fn from(message: String) -> Self {
+    Self {
+      message,
+      kind: None,
+      location: None,
+    }
+  }
+}
+
+impl From<&str> for Diagnostic {
+  fn from(message: &str) -> Self {
+    Self::from(message.to_string())
+  }
+}
+
+fn type_of(value: &serde_json::Value) -> Option<String> {
+  match value {
+    serde_json::Value::String(name) => Some(name.clone()),
+    serde_json::Value::Object(fields) => match fields.get("type") {
+      Some(serde_json::Value::String(name)) => Some(name.clone()),
+      _ if fields.len() == 1 => fields.iter().next().map(|(name, inner)| match inner.get("type") {
+        Some(serde_json::Value::String(inner_name)) => inner_name.clone(),
+        _ => name.clone(),
+      }),
+      _ => None,
+    },
+    _ => None,
+  }
 }

data/rust/src/scope.rs

@@ -29,7 +29,8 @@ impl<'i> Scoper<'i> {
     let list = SelectorList::parse_string_with_options(fragment, ParserOptions::default()).map_err(|error| {
       format!(
         "Invalid scope selector {fragment:?} at :{}:{}",
-        error.location.line, error.location.column
+        error.location.line + 1,
+        error.location.column
       )
     })?;
 

data/sig/lightningcss/errors.rbs

@@ -2,6 +2,15 @@
 
 module LightningCSS
   class Error < StandardError
+    attr_reader type: String?
+
+    attr_reader location: LightningCSS::Location?
+
+    # : (String) -> instance
+    def self.from_json: (String) -> instance
+
+    # : (?String?, ?type: String?, ?location: LightningCSS::Location?) -> void
+    def initialize: (?String?, ?type: String?, ?location: LightningCSS::Location?) -> void
   end
 
   class ParseError < Error

data/sig/lightningcss/location.rbs

@@ -0,0 +1,26 @@
+# Generated from lib/lightningcss/location.rb with RBS::Inline
+
+module LightningCSS
+  class Location < Data
+    attr_reader filename(): String?
+
+    attr_reader line(): Integer
+
+    attr_reader column(): Integer
+
+    def self.new: (String? filename, Integer line, Integer column) -> instance
+                | (filename: String?, line: Integer, column: Integer) -> instance
+
+    def self.members: () -> [ :filename, :line, :column ]
+
+    def members: () -> [ :filename, :line, :column ]
+  end
+
+  class Location
+    # : (Hash[String, untyped]) -> LightningCSS::Location
+    def self.from_hash: (Hash[String, untyped]) -> LightningCSS::Location
+
+    # : () -> String
+    def to_s: () -> String
+  end
+end

data/sig/lightningcss/parse_result.rbs

@@ -10,13 +10,13 @@ module LightningCSS
 
     attr_reader stylesheet: Hash[String, untyped]
 
-    attr_reader warnings: Array[String]
+    attr_reader warnings: Array[LightningCSS::Warning]
 
     # : (String, ?String?) -> LightningCSS::ParseResult
     def self.from_json: (String, ?String?) -> LightningCSS::ParseResult
 
-    # : (stylesheet: Hash[String, untyped], warnings: Array[String], ?source: String?) -> void
-    def initialize: (stylesheet: Hash[String, untyped], warnings: Array[String], ?source: String?) -> void
+    # : (stylesheet: Hash[String, untyped], warnings: Array[LightningCSS::Warning], ?source: String?) -> void
+    def initialize: (stylesheet: Hash[String, untyped], warnings: Array[LightningCSS::Warning], ?source: String?) -> void
 
     # : () -> LightningCSS::Node
     def root: () -> LightningCSS::Node

data/sig/lightningcss/result.rbs

@@ -14,13 +14,13 @@ module LightningCSS
 
     attr_reader exports: Hash[String, String]?
 
-    attr_reader warnings: Array[String]
+    attr_reader warnings: Array[LightningCSS::Warning]
 
     # : (String) -> LightningCSS::Result
     def self.from_json: (String) -> LightningCSS::Result
 
-    # : (code: String, warnings: Array[String], ?exports: Hash[String, String]?) -> void
-    def initialize: (code: String, warnings: Array[String], ?exports: Hash[String, String]?) -> void
+    # : (code: String, warnings: Array[LightningCSS::Warning], ?exports: Hash[String, String]?) -> void
+    def initialize: (code: String, warnings: Array[LightningCSS::Warning], ?exports: Hash[String, String]?) -> void
 
     # : () -> bool
     def warnings?: () -> bool

data/sig/lightningcss/types.rbs

@@ -1,17 +1,19 @@
 # Note: This file is hand written, `rbs-inline` does not generate this file.
 
 module LightningCSS
+  type version = Integer | Float | String
+
   type browsers = {
-    ?android: Integer,
-    ?chrome: Integer,
-    ?edge: Integer,
-    ?firefox: Integer,
-    ?ie: Integer,
-    ?ios: Integer,
-    ?ios_saf: Integer,
-    ?opera: Integer,
-    ?safari: Integer,
-    ?samsung: Integer
+    ?android: version,
+    ?chrome: version,
+    ?edge: version,
+    ?firefox: version,
+    ?ie: version,
+    ?ios: version,
+    ?ios_saf: version,
+    ?opera: version,
+    ?safari: version,
+    ?samsung: version
   }
 
   type css_modules = bool | {

data/sig/lightningcss/warning.rbs

@@ -0,0 +1,36 @@
+# Generated from lib/lightningcss/warning.rb with RBS::Inline
+
+module LightningCSS
+  class Warning
+    attr_reader message: String
+
+    attr_reader type: String?
+
+    attr_reader location: LightningCSS::Location?
+
+    # : (Hash[String, untyped]) -> LightningCSS::Warning
+    def self.from_hash: (Hash[String, untyped]) -> LightningCSS::Warning
+
+    # : (message: String, ?type: String?, ?location: LightningCSS::Location?) -> void
+    def initialize: (message: String, ?type: String?, ?location: LightningCSS::Location?) -> void
+
+    # : () -> Hash[Symbol, untyped]
+    def to_h: () -> Hash[Symbol, untyped]
+
+    # : () -> String
+    def to_s: () -> String
+
+    alias to_str to_s
+
+    # : (untyped) -> bool
+    def ==: (untyped) -> bool
+
+    alias eql? ==
+
+    # : () -> Integer
+    def hash: () -> Integer
+
+    # : () -> String
+    def inspect: () -> String
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: lightningcss
 version: !ruby/object:Gem::Version
-  version: 1.33.0.1
+  version: 1.33.0.2
 platform: arm-linux-gnu
 authors:
 - Marco Roth
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2026-10-13 00:00:00.000000000 Z
+date: 2026-10-14 00:00:00.000000000 Z
 dependencies: []
 description: Ruby bindings for Lightning CSS, an extremely fast CSS parser, transformer,
   bundler, and minifier.
@@ -30,6 +30,7 @@ files:
 - lib/lightningcss/4.0/lightningcss.so
 - lib/lightningcss/backend.rb
 - lib/lightningcss/errors.rb
+- lib/lightningcss/location.rb
 - lib/lightningcss/node.rb
 - lib/lightningcss/options.rb
 - lib/lightningcss/parse_result.rb
@@ -37,6 +38,7 @@ files:
 - lib/lightningcss/transformer.rb
 - lib/lightningcss/version.rb
 - lib/lightningcss/visitor.rb
+- lib/lightningcss/warning.rb
 - licenses/README.md
 - licenses/lightningcss-MPL-2.0.txt
 - lightningcss.gemspec
@@ -51,6 +53,7 @@ files:
 - sig/lightningcss.rbs
 - sig/lightningcss/backend.rbs
 - sig/lightningcss/errors.rbs
+- sig/lightningcss/location.rbs
 - sig/lightningcss/node.rbs
 - sig/lightningcss/options.rbs
 - sig/lightningcss/parse_result.rbs
@@ -59,6 +62,7 @@ files:
 - sig/lightningcss/types.rbs
 - sig/lightningcss/version.rbs
 - sig/lightningcss/visitor.rbs
+- sig/lightningcss/warning.rbs
 homepage: https://github.com/marcoroth/lightningcss-ruby
 licenses:
 - MIT