Replace Symbol impl with stringleton

Cargo.toml

@@ -34,6 +34,7 @@ toml_edit = "0.23.9"
 futures = "0.3.31"
 annotate-snippets = "0.12.10"
 anstream = "0.6.18"
+stringleton = { version = "0.2.1", features = ["serde"] }
 
 [workspace.lints.clippy]
 pedantic = { level = "warn", priority = -1 }

tests/Cargo.toml

@@ -25,6 +25,7 @@ werk-util.workspace = true
 anstream.workspace = true
 # Hijacking winnow for the Offset trait
 winnow.workspace = true
+stringleton.workspace = true
 
 [dev-dependencies]
 criterion = "0.8.1"

tests/lib.rs

@@ -1,2 +1,4 @@
 // This file intentionally left blank.
 pub mod mock_io;
+
+stringleton::enable!(werk_runner);

tests/test_expressions.rs

@@ -1,7 +1,7 @@
 use werk_runner::Value;
 
+use stringleton::Symbol;
 use tests::mock_io::*;
-use werk_util::Symbol;
 
 fn evaluate_global(source: &str, global_variable_name_to_check: &str) -> Value {
     let mut test = Test::new(source).unwrap();

tests/test_outdatedness.rs

@@ -2,9 +2,9 @@ use macro_rules_attribute::apply;
 use tests::mock_io;
 
 use mock_io::*;
+use stringleton::Symbol;
 use werk_fs::{Absolute, Path};
 use werk_runner::{BuildStatus, Outdatedness, Reason, ShellCommandLine, TaskId};
-use werk_util::Symbol;
 
 static WERK: &str = r#"
 config profile = env "PROFILE"

tests/test_path_resolution.rs

@@ -1,8 +1,11 @@
 use macro_rules_attribute::apply;
+use stringleton::sym;
 use tests::mock_io::*;
 use werk_fs::Absolute;
 use werk_runner::{Runner, TaskId, Value};
-use werk_util::{Annotated, Symbol};
+use werk_util::Annotated;
+
+stringleton::enable!(tests);
 
 #[test]
 fn test_path_resolution() {
@@ -44,41 +47,35 @@ let exists-not-explicit-workspace = "<exists-not:workspace>"
     };
     let globals = &workspace.manifest.global_variables;
     assert_eq!(
-        globals.get(&Symbol::new("exists-resolved")).unwrap().value,
+        globals.get(&sym!("exists-resolved")).unwrap().value,
         Value::from(foo_workspace.clone())
     );
     assert_eq!(
-        globals
-            .get(&Symbol::new("exists-explicit-out-dir"))
-            .unwrap()
-            .value,
+        globals.get(&sym!("exists-explicit-out-dir")).unwrap().value,
         Value::from(foo_output)
     );
     assert_eq!(
         globals
-            .get(&Symbol::new("exists-explicit-workspace"))
+            .get(&sym!("exists-explicit-workspace"))
             .unwrap()
             .value,
         Value::from(foo_workspace.clone())
     );
 
     assert_eq!(
-        globals
-            .get(&Symbol::new("exists-not-resolved"))
-            .unwrap()
-            .value,
+        globals.get(&sym!("exists-not-resolved")).unwrap().value,
         Value::from(bar_output.clone())
     );
     assert_eq!(
         globals
-            .get(&Symbol::new("exists-not-explicit-out-dir"))
+            .get(&sym!("exists-not-explicit-out-dir"))
             .unwrap()
             .value,
         Value::from(bar_output.clone())
     );
     assert_eq!(
         globals
-            .get(&Symbol::new("exists-not-explicit-workspace"))
+            .get(&sym!("exists-not-explicit-workspace"))
             .unwrap()
             .value,
         Value::from(bar_workspace)

werk-fs/Cargo.toml

@@ -13,6 +13,7 @@ winnow.workspace = true
 thiserror.workspace = true
 serde.workspace = true
 werk-util.workspace = true
+stringleton.workspace = true
 
 [lints]
 workspace = true

werk-fs/sym.rs

@@ -1,4 +1,4 @@
-use werk_util::Symbol;
+use stringleton::Symbol;
 
 use crate::{Absolute, Path, PathBuf};
 

werk-parser/Cargo.toml

@@ -20,6 +20,7 @@ regex.workspace = true
 annotate-snippets.workspace = true
 serde.workspace = true
 werk-util.workspace = true
+stringleton.workspace = true
 
 [dev-dependencies]
 anstream.workspace = true

werk-parser/ast.rs

@@ -9,7 +9,7 @@ pub mod token;
 
 pub use expr::*;
 pub use string::*;
-use werk_util::Symbol;
+use stringleton::Symbol;
 
 /// Whitespace and comments within statements and expressions (not doc
 /// comments).

werk-parser/ast/string.rs

@@ -1,6 +1,7 @@
 use std::{fmt::Write, hash::Hash as _};
 
-use werk_util::{SemanticHash, Span, Symbol};
+use stringleton::Symbol;
+use werk_util::{SemanticHash, Span};
 
 use crate::parser::{
     escape_pattern_literal, escape_string_literal, parse_pattern_expr_unquoted,

werk-parser/parser/string.rs

@@ -6,7 +6,8 @@ use crate::{
     fatal,
     parser::{Input, Parser as _},
 };
-use werk_util::{Offset, Symbol};
+use stringleton::Symbol;
+use werk_util::Offset;
 use winnow::{
     Parser,
     ascii::{dec_int, digit1, multispace1},

werk-runner/Cargo.toml

@@ -34,6 +34,7 @@ memchr = "2.7.4"
 annotate-snippets.workspace = true
 werk-util.workspace = true
 bitflags = "2.8.0"
+stringleton.workspace = true
 
 [lints]
 workspace = true

werk-runner/cache.rs

@@ -1,7 +1,7 @@
 use std::collections::BTreeMap;
 
+use stringleton::Symbol;
 use werk_fs::Absolute;
-use werk_util::Symbol;
 
 /// The contents of `.werk-cache`.
 #[derive(Debug, Default, serde::Serialize, serde::Deserialize)]

werk-runner/eval.rs

@@ -1,10 +1,11 @@
 mod string;
 mod used;
 pub(crate) use string::*;
+use stringleton::Symbol;
 pub use used::*;
 
 use werk_fs::Absolute;
-use werk_util::{DiagnosticFileId, Spanned as _, Symbol};
+use werk_util::{DiagnosticFileId, Spanned as _};
 
 use std::sync::Arc;
 

werk-runner/eval/string.rs

@@ -1,9 +1,10 @@
 use std::{borrow::Cow, fmt::Write as _, sync::Arc};
 
 use indexmap::IndexMap;
+use stringleton::Symbol;
 use werk_fs::Absolute;
 use werk_parser::ast;
-use werk_util::{DiagnosticFileId, DiagnosticSpan, Symbol};
+use werk_util::{DiagnosticFileId, DiagnosticSpan};
 
 use crate::{
     AmbiguousPatternError, EvalError, Lookup, LookupValue, Pattern, PatternMatcher, PatternRegex,

werk-runner/eval/used.rs

@@ -3,8 +3,8 @@ use std::{
     ops::{BitOr, BitOrAssign},
 };
 
+use stringleton::Symbol;
 use werk_fs::{Absolute, SymPath};
-use werk_util::Symbol;
 
 use crate::cache::Hash128;
 

werk-runner/ir.rs

@@ -1,7 +1,8 @@
 use indexmap::IndexMap;
+use stringleton::Symbol;
 use werk_fs::Absolute;
 use werk_parser::ast;
-use werk_util::{DiagnosticFileId, DiagnosticMainSourceMap, DiagnosticSpan, Symbol};
+use werk_util::{DiagnosticFileId, DiagnosticMainSourceMap, DiagnosticSpan};
 
 use crate::{
     AmbiguousPatternError, ConfigVar, EvalError, LocalVariables, Pattern, PatternMatchData, Value,

werk-runner/lib.rs

@@ -30,3 +30,5 @@ pub use which::Error as WhichError;
 
 #[doc(no_inline)]
 pub use globset;
+
+stringleton::enable!();

werk-runner/outdatedness.rs

@@ -3,8 +3,8 @@ use std::{
     ops::{BitOr, BitOrAssign},
 };
 
+use stringleton::Symbol;
 use werk_fs::{Absolute, SymPath};
-use werk_util::Symbol;
 
 use crate::{
     TaskId, Workspace,

werk-runner/runner.rs

@@ -3,8 +3,9 @@ use std::{future::Future, sync::Arc, time::SystemTime};
 use futures::{StreamExt, channel::oneshot, future};
 use indexmap::{IndexMap, map::Entry};
 use parking_lot::Mutex;
+use stringleton::sym;
 use werk_fs::{Absolute, Normalize as _, Path, SymPath};
-use werk_util::{Annotated, AsDiagnostic, DiagnosticSpan, Symbol, cancel};
+use werk_util::{Annotated, AsDiagnostic, DiagnosticSpan, cancel};
 
 use crate::{
     AmbiguousPatternError, BuildRecipeScope, ChildCaptureOutput, ChildLinesStream, Env, Error,
@@ -378,7 +379,7 @@ impl<'a> Inner<'a> {
     ) -> Result<BuildStatus, Error> {
         let mut scope = BuildRecipeScope::new(self.workspace, task_id, &recipe_match);
         scope.set(
-            Symbol::new("out"),
+            sym!("out"),
             Eval::inherent(Value::from(recipe_match.target_file.to_string())),
         );
 
@@ -514,7 +515,7 @@ impl<'a> Inner<'a> {
 
         // Make the `depfile` variable available to the recipe body.
         scope.set(
-            Symbol::from("depfile"),
+            sym!("depfile"),
             Eval::inherent(Value::String(depfile.clone())),
         );
 

werk-runner/runner/task.rs

@@ -1,6 +1,6 @@
 use futures::channel::oneshot;
+use stringleton::Symbol;
 use werk_fs::{Absolute, SymPath};
-use werk_util::Symbol;
 
 use crate::{Error, ir};