git.delta.rocks / jrsonnet / refs/commits / e7e620bf3687

difftreelog

Merge pull request #138 from CertainLach/stable-async-traits

Yaroslav Bolyukin2023-10-29parents: #fd582d4 #69bb25d.patch.diff
in: master

7 files changed

modified.github/workflows/clippy_check.ymldiffbeforeafterboth
--- a/.github/workflows/clippy_check.yml
+++ b/.github/workflows/clippy_check.yml
@@ -7,7 +7,7 @@
       - uses: actions/checkout@v2
       - uses: actions-rs/toolchain@v1
         with:
-            toolchain: nightly
+            toolchain: nightly-2023-10-28
             components: clippy
             override: true
       - uses: actions-rs/clippy-check@v1
modifiedCargo.lockdiffbeforeafterboth
--- a/Cargo.lock
+++ b/Cargo.lock
@@ -112,17 +112,6 @@
 ]
 
 [[package]]
-name = "async-trait"
-version = "0.1.74"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "a66537f1bb974b254c98ed142ff995236e81b9d0fe4db0575f46612cb15eb0f9"
-dependencies = [
- "proc-macro2",
- "quote",
- "syn 2.0.38",
-]
-
-[[package]]
 name = "autocfg"
 version = "1.1.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
@@ -598,7 +587,6 @@
 dependencies = [
  "annotate-snippets",
  "anyhow",
- "async-trait",
  "bincode",
  "derivative",
  "hashbrown 0.14.2",
@@ -801,34 +789,25 @@
 
 [[package]]
 name = "logos"
-version = "0.13.0"
+version = "0.12.1"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "c000ca4d908ff18ac99b93a062cb8958d331c3220719c52e77cb19cc6ac5d2c1"
+checksum = "bf8b031682c67a8e3d5446840f9573eb7fe26efe7ec8d195c9ac4c0647c502f1"
 dependencies = [
  "logos-derive",
 ]
 
 [[package]]
-name = "logos-codegen"
-version = "0.13.0"
+name = "logos-derive"
+version = "0.12.1"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "dc487311295e0002e452025d6b580b77bb17286de87b57138f3b5db711cded68"
+checksum = "a1d849148dbaf9661a6151d1ca82b13bb4c4c128146a88d05253b38d4e2f496c"
 dependencies = [
  "beef",
  "fnv",
  "proc-macro2",
  "quote",
  "regex-syntax",
- "syn 2.0.38",
-]
-
-[[package]]
-name = "logos-derive"
-version = "0.13.0"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "dbfc0d229f1f42d790440136d941afd806bc9e949e2bcb8faa813b0f00d1267e"
-dependencies = [
- "logos-codegen",
+ "syn 1.0.109",
 ]
 
 [[package]]
modifiedCargo.tomldiffbeforeafterboth
--- a/Cargo.toml
+++ b/Cargo.toml
@@ -35,7 +35,6 @@
 rustc-hash = "1.1"
 bincode = "1.3"
 annotate-snippets = "0.9.1"
-async-trait = "0.1.60"
 num-bigint = "0.4.3"
 derivative = "2.2.0"
 strsim = "0.10.0"
@@ -45,7 +44,7 @@
 syn = "2.0"
 peg = "0.8.2"
 drop_bomb = "0.1.5"
-logos = "0.13.0"
+logos = "0.12.0"
 miette = "5.5.0"
 rowan = "0.15"
 text-size = "1.1"
modifiedcrates/jrsonnet-evaluator/Cargo.tomldiffbeforeafterboth
--- a/crates/jrsonnet-evaluator/Cargo.toml
+++ b/crates/jrsonnet-evaluator/Cargo.toml
@@ -14,7 +14,7 @@
 # Allows library authors to throw custom errors
 anyhow-error = ["anyhow"]
 # Adds ability to build import closure in async
-async-import = ["async-trait"]
+async-import = []
 
 # Allows to preserve field order in objects
 exp-preserve-order = []
@@ -54,8 +54,6 @@
 bincode = { workspace = true, optional = true }
 # Explaining traces
 annotate-snippets = { workspace = true, features = ["color"], optional = true }
-# Async imports
-async-trait = { workspace = true, optional = true }
 # Bigint
 num-bigint = { workspace = true, features = ["serde"], optional = true }
 derivative.workspace = true
modifiedcrates/jrsonnet-evaluator/src/async_import.rsdiffbeforeafterboth
before · crates/jrsonnet-evaluator/src/async_import.rs
1use std::{cell::RefCell, path::Path};23use async_trait::async_trait;4use jrsonnet_gcmodule::Trace;5use jrsonnet_interner::IStr;6use jrsonnet_parser::{7	ArgsDesc, AssertStmt, BindSpec, CompSpec, Destruct, Expr, FieldMember, FieldName, ForSpecData,8	IfSpecData, LocExpr, Member, ObjBody, Param, ParamsDesc, ParserSettings, SliceDesc, Source,9	SourcePath,10};1112use crate::{bail, gc::GcHashMap, FileData, ImportResolver, State};1314pub struct Import {15	path: IStr,16	expression: bool,17}1819pub struct FoundImports(Vec<Import>);2021// Visits all nodes, trying to find import statements22#[allow(clippy::too_many_lines)]23pub fn find_imports(expr: &LocExpr, out: &mut FoundImports) {24	fn in_destruct(dest: &Destruct, #[allow(unused_variables)] out: &mut FoundImports) {25		match dest {26			#[cfg(feature = "exp-destruct")]27			Destruct::Array {28				start,29				rest: _,30				end,31			} => {32				for dest in start {33					in_destruct(dest, out);34				}35				for dest in end {36					in_destruct(dest, out);37				}38			}39			#[cfg(feature = "exp-destruct")]40			Destruct::Object { fields, rest: _ } => {41				for (_, dest, default) in fields {42					if let Some(dest) = dest {43						in_destruct(dest, out);44					}45					if let Some(expr) = default {46						find_imports(expr, out);47					}48				}49			}50			#[cfg(feature = "exp-destruct")]51			Destruct::Skip => {}52			Destruct::Full(_) => {}53		}54	}55	fn in_compspec(specs: &[CompSpec], out: &mut FoundImports) {56		for spec in specs {57			match spec {58				CompSpec::IfSpec(IfSpecData(expr)) => find_imports(expr, out),59				CompSpec::ForSpec(ForSpecData(destruct, expr)) => {60					in_destruct(destruct, out);61					find_imports(expr, out);62				}63			}64		}65	}66	fn in_params(params: &ParamsDesc, out: &mut FoundImports) {67		for Param(dest, default) in &*params.0 {68			in_destruct(dest, out);69			if let Some(expr) = default {70				find_imports(expr, out);71			}72		}73	}74	fn in_bind(specs: &[BindSpec], out: &mut FoundImports) {75		for spec in specs {76			match spec {77				BindSpec::Field {78					into: dest,79					value: expr,80				} => {81					in_destruct(dest, out);82					find_imports(expr, out);83				}84				BindSpec::Function {85					name: _,86					params,87					value: expr,88				} => {89					in_params(params, out);90					find_imports(expr, out);91				}92			}93		}94	}95	fn in_args(ArgsDesc { unnamed, named }: &ArgsDesc, out: &mut FoundImports) {96		for expr in unnamed {97			find_imports(expr, out);98		}99		for (_, expr) in named {100			find_imports(expr, out);101		}102	}103	fn in_obj(obj: &ObjBody, out: &mut FoundImports) {104		match obj {105			ObjBody::MemberList(v) => {106				for member in v {107					match member {108						Member::Field(FieldMember {109							name,110							params,111							value,112							..113						}) => {114							match name {115								FieldName::Fixed(_) => {}116								FieldName::Dyn(expr) => find_imports(expr, out),117							}118							if let Some(params) = params {119								in_params(params, out);120							}121							find_imports(value, out);122						}123						Member::BindStmt(_) => todo!(),124						Member::AssertStmt(AssertStmt(expr, expr2)) => {125							find_imports(expr, out);126							if let Some(expr) = expr2 {127								find_imports(expr, out);128							}129						}130					}131				}132			}133			ObjBody::ObjComp(_) => todo!(),134		}135	}136	match &*expr.0 {137		Expr::Import(v) | Expr::ImportStr(v) | Expr::ImportBin(v) => {138			if let Expr::Str(s) = &*v.0 {139				out.0.push(Import {140					path: s.clone(),141					expression: matches!(&*expr.0, Expr::Import(_)),142				});143			}144			// Non-string import will fail in runtime145		}146147		Expr::Literal(_) | Expr::Str(_) | Expr::Num(_) | Expr::Var(_) => {}148149		Expr::Arr(arr) => {150			for expr in arr {151				find_imports(expr, out);152			}153		}154		Expr::ArrComp(expr, specs) => {155			find_imports(expr, out);156			in_compspec(specs, out);157		}158		Expr::Obj(obj) => in_obj(obj, out),159		Expr::ObjExtend(expr, obj) => {160			find_imports(expr, out);161			in_obj(obj, out);162		}163		Expr::BinaryOp(a, _, b) => {164			find_imports(a, out);165			find_imports(b, out);166		}167		Expr::AssertExpr(AssertStmt(expr, expr2), then) => {168			find_imports(expr, out);169			if let Some(expr) = expr2 {170				find_imports(expr, out);171			}172			find_imports(then, out);173		}174		Expr::LocalExpr(specs, expr) => {175			in_bind(specs, out);176			find_imports(expr, out);177		}178		Expr::Apply(expr, args, _) => {179			find_imports(expr, out);180			in_args(args, out);181		}182		Expr::Index { indexable, parts } => {183			find_imports(indexable, out);184			for part in parts {185				find_imports(&part.value, out);186			}187		}188		Expr::Function(params, expr) => {189			in_params(params, out);190			find_imports(expr, out);191		}192		Expr::IfElse {193			cond: IfSpecData(expr),194			cond_then,195			cond_else,196		} => {197			find_imports(expr, out);198			find_imports(cond_then, out);199			if let Some(expr) = cond_else {200				find_imports(expr, out);201			}202		}203		Expr::Slice(expr, SliceDesc { start, end, step }) => {204			find_imports(expr, out);205			if let Some(expr) = start {206				find_imports(expr, out);207			}208			if let Some(expr) = end {209				find_imports(expr, out);210			}211			if let Some(expr) = step {212				find_imports(expr, out);213			}214		}215		Expr::Parened(expr) | Expr::UnaryOp(_, expr) | Expr::ErrorStmt(expr) => {216			find_imports(expr, out);217		}218	}219}220221#[async_trait(?Send)]222pub trait AsyncImportResolver {223	type Error;224	/// Resolves file path, e.g. `(/home/user/manifests, b.libjsonnet)` can correspond225	/// both to `/home/user/manifests/b.libjsonnet` and to `/home/user/${vendor}/b.libjsonnet`226	/// where `${vendor}` is a library path.227	///228	/// `from` should only be returned from [`ImportResolver::resolve`], or from other defined file, any other value229	/// may result in panic230	async fn resolve_from(&self, from: &SourcePath, path: &str) -> Result<SourcePath, Self::Error>;231	async fn resolve_from_default(&self, path: &str) -> Result<SourcePath, Self::Error> {232		self.resolve_from(&SourcePath::default(), path).await233	}234	/// Resolves absolute path, doesn't supports jpath and other fancy things235	async fn resolve(&self, path: &Path) -> Result<SourcePath, Self::Error>;236237	/// Load resolved file238	/// This should only be called with value returned from [`ImportResolver::resolve_file`]/[`ImportResolver::resolve`],239	/// this cannot be resolved using associated type, as evaluator uses object instead of generic for [`ImportResolver`]240	async fn load_file_contents(&self, resolved: &SourcePath) -> Result<Vec<u8>, Self::Error>;241}242243#[derive(Trace)]244struct ResolvedImportResolver {245	resolved: RefCell<GcHashMap<(SourcePath, IStr), (SourcePath, bool)>>,246}247impl ImportResolver for ResolvedImportResolver {248	fn load_file_contents(&self, _resolved: &SourcePath) -> crate::Result<Vec<u8>> {249		unreachable!("all files should be loaded at this point");250	}251252	fn resolve_from(&self, from: &SourcePath, path: &str) -> crate::Result<SourcePath> {253		Ok(self254			.resolved255			.borrow()256			.get(&(from.clone(), path.into()))257			.expect("all imports should be resolved at this point")258			.0259			.clone())260	}261262	fn resolve_from_default(&self, path: &str) -> crate::Result<SourcePath> {263		self.resolve_from(&SourcePath::default(), path)264	}265266	fn resolve(&self, path: &Path) -> crate::Result<SourcePath> {267		bail!(crate::error::ErrorKind::AbsoluteImportNotSupported(268			path.to_owned()269		))270	}271272	fn as_any(&self) -> &dyn std::any::Any {273		self274	}275}276277enum Job {278	LoadFile { path: SourcePath, parse: bool },279	ParseFile(SourcePath),280	ResolveImport { from: SourcePath, import: Import },281}282283#[allow(clippy::future_not_send)]284pub async fn async_import<H>(s: State, handler: H, path: impl AsRef<Path>) -> Result<(), H::Error>285where286	H: AsyncImportResolver,287{288	let mut resolved = s289		.import_resolver()290		.as_any()291		.downcast_ref::<ResolvedImportResolver>()292		.map_or_else(GcHashMap::new, |resolver| {293			std::mem::take(&mut *resolver.resolved.borrow_mut())294		});295	let mut queue = vec![Job::LoadFile {296		path: handler.resolve(path.as_ref()).await?,297		parse: true,298	}];299	while let Some(job) = queue.pop() {300		match job {301			Job::LoadFile { path, parse } => {302				if !s.0.file_cache.borrow().contains_key(&path) {303					let data = handler.load_file_contents(&path).await?;304					s.0.file_cache305						.borrow_mut()306						.insert(path.clone(), FileData::new_bytes(data.as_slice().into()));307				}308				if parse {309					queue.push(Job::ParseFile(path));310				}311			}312			Job::ParseFile(path) => {313				if let Some(file) = s.0.file_cache.borrow_mut().get_mut(&path) {314					if file.parsed.is_none() {315						let Some(code) = file.get_string() else {316							continue;317						};318						let source = Source::new(path.clone(), code.clone());319						// If failed - then skip import320						file.parsed =321							jrsonnet_parser::parse(&code, &ParserSettings { source }).ok();322						if let Some(parsed) = &file.parsed {323							let mut imports = FoundImports(vec![]);324							find_imports(parsed, &mut imports);325							for import in imports.0 {326								queue.push(Job::ResolveImport {327									from: path.clone(),328									import,329								});330							}331						}332					}333				}334			}335			Job::ResolveImport { from, import } => {336				if let Some((resolved, expression)) =337					resolved.get_mut(&(from.clone(), import.path.clone()))338				{339					if import.expression && !*expression {340						*expression = true;341						queue.push(Job::ParseFile(resolved.clone()));342					}343					continue;344				}345				let resolved = handler.resolve_from(&from, &import.path).await?;346				queue.push(Job::LoadFile {347					path: resolved,348					parse: import.expression,349				});350			}351		}352	}353	s.set_import_resolver(ResolvedImportResolver {354		resolved: RefCell::new(resolved),355	});356	Ok(())357}
after · crates/jrsonnet-evaluator/src/async_import.rs
1use std::{cell::RefCell, future::Future, path::Path};23use jrsonnet_gcmodule::Trace;4use jrsonnet_interner::IStr;5use jrsonnet_parser::{6	ArgsDesc, AssertStmt, BindSpec, CompSpec, Destruct, Expr, FieldMember, FieldName, ForSpecData,7	IfSpecData, LocExpr, Member, ObjBody, Param, ParamsDesc, ParserSettings, SliceDesc, Source,8	SourcePath,9};1011use crate::{bail, gc::GcHashMap, FileData, ImportResolver, State};1213pub struct Import {14	path: IStr,15	expression: bool,16}1718pub struct FoundImports(Vec<Import>);1920// Visits all nodes, trying to find import statements21#[allow(clippy::too_many_lines)]22pub fn find_imports(expr: &LocExpr, out: &mut FoundImports) {23	fn in_destruct(dest: &Destruct, #[allow(unused_variables)] out: &mut FoundImports) {24		match dest {25			#[cfg(feature = "exp-destruct")]26			Destruct::Array {27				start,28				rest: _,29				end,30			} => {31				for dest in start {32					in_destruct(dest, out);33				}34				for dest in end {35					in_destruct(dest, out);36				}37			}38			#[cfg(feature = "exp-destruct")]39			Destruct::Object { fields, rest: _ } => {40				for (_, dest, default) in fields {41					if let Some(dest) = dest {42						in_destruct(dest, out);43					}44					if let Some(expr) = default {45						find_imports(expr, out);46					}47				}48			}49			#[cfg(feature = "exp-destruct")]50			Destruct::Skip => {}51			Destruct::Full(_) => {}52		}53	}54	fn in_compspec(specs: &[CompSpec], out: &mut FoundImports) {55		for spec in specs {56			match spec {57				CompSpec::IfSpec(IfSpecData(expr)) => find_imports(expr, out),58				CompSpec::ForSpec(ForSpecData(destruct, expr)) => {59					in_destruct(destruct, out);60					find_imports(expr, out);61				}62			}63		}64	}65	fn in_params(params: &ParamsDesc, out: &mut FoundImports) {66		for Param(dest, default) in &*params.0 {67			in_destruct(dest, out);68			if let Some(expr) = default {69				find_imports(expr, out);70			}71		}72	}73	fn in_bind(specs: &[BindSpec], out: &mut FoundImports) {74		for spec in specs {75			match spec {76				BindSpec::Field {77					into: dest,78					value: expr,79				} => {80					in_destruct(dest, out);81					find_imports(expr, out);82				}83				BindSpec::Function {84					name: _,85					params,86					value: expr,87				} => {88					in_params(params, out);89					find_imports(expr, out);90				}91			}92		}93	}94	fn in_args(ArgsDesc { unnamed, named }: &ArgsDesc, out: &mut FoundImports) {95		for expr in unnamed {96			find_imports(expr, out);97		}98		for (_, expr) in named {99			find_imports(expr, out);100		}101	}102	fn in_obj(obj: &ObjBody, out: &mut FoundImports) {103		match obj {104			ObjBody::MemberList(v) => {105				for member in v {106					match member {107						Member::Field(FieldMember {108							name,109							params,110							value,111							..112						}) => {113							match name {114								FieldName::Fixed(_) => {}115								FieldName::Dyn(expr) => find_imports(expr, out),116							}117							if let Some(params) = params {118								in_params(params, out);119							}120							find_imports(value, out);121						}122						Member::BindStmt(_) => todo!(),123						Member::AssertStmt(AssertStmt(expr, expr2)) => {124							find_imports(expr, out);125							if let Some(expr) = expr2 {126								find_imports(expr, out);127							}128						}129					}130				}131			}132			ObjBody::ObjComp(_) => todo!(),133		}134	}135	match &*expr.0 {136		Expr::Import(v) | Expr::ImportStr(v) | Expr::ImportBin(v) => {137			if let Expr::Str(s) = &*v.0 {138				out.0.push(Import {139					path: s.clone(),140					expression: matches!(&*expr.0, Expr::Import(_)),141				});142			}143			// Non-string import will fail in runtime144		}145146		Expr::Literal(_) | Expr::Str(_) | Expr::Num(_) | Expr::Var(_) => {}147148		Expr::Arr(arr) => {149			for expr in arr {150				find_imports(expr, out);151			}152		}153		Expr::ArrComp(expr, specs) => {154			find_imports(expr, out);155			in_compspec(specs, out);156		}157		Expr::Obj(obj) => in_obj(obj, out),158		Expr::ObjExtend(expr, obj) => {159			find_imports(expr, out);160			in_obj(obj, out);161		}162		Expr::BinaryOp(a, _, b) => {163			find_imports(a, out);164			find_imports(b, out);165		}166		Expr::AssertExpr(AssertStmt(expr, expr2), then) => {167			find_imports(expr, out);168			if let Some(expr) = expr2 {169				find_imports(expr, out);170			}171			find_imports(then, out);172		}173		Expr::LocalExpr(specs, expr) => {174			in_bind(specs, out);175			find_imports(expr, out);176		}177		Expr::Apply(expr, args, _) => {178			find_imports(expr, out);179			in_args(args, out);180		}181		Expr::Index { indexable, parts } => {182			find_imports(indexable, out);183			for part in parts {184				find_imports(&part.value, out);185			}186		}187		Expr::Function(params, expr) => {188			in_params(params, out);189			find_imports(expr, out);190		}191		Expr::IfElse {192			cond: IfSpecData(expr),193			cond_then,194			cond_else,195		} => {196			find_imports(expr, out);197			find_imports(cond_then, out);198			if let Some(expr) = cond_else {199				find_imports(expr, out);200			}201		}202		Expr::Slice(expr, SliceDesc { start, end, step }) => {203			find_imports(expr, out);204			if let Some(expr) = start {205				find_imports(expr, out);206			}207			if let Some(expr) = end {208				find_imports(expr, out);209			}210			if let Some(expr) = step {211				find_imports(expr, out);212			}213		}214		Expr::Parened(expr) | Expr::UnaryOp(_, expr) | Expr::ErrorStmt(expr) => {215			find_imports(expr, out);216		}217	}218}219220pub trait AsyncImportResolver {221	type Error;222	/// Resolves file path, e.g. `(/home/user/manifests, b.libjsonnet)` can correspond223	/// both to `/home/user/manifests/b.libjsonnet` and to `/home/user/${vendor}/b.libjsonnet`224	/// where `${vendor}` is a library path.225	///226	/// `from` should only be returned from [`ImportResolver::resolve`],227	/// or from other defined file, any other value may result in panic228	fn resolve_from(229		&self,230		from: &SourcePath,231		path: &str,232	) -> impl Future<Output = Result<SourcePath, Self::Error>>;233	fn resolve_from_default(234		&self,235		path: &str,236	) -> impl Future<Output = Result<SourcePath, Self::Error>> {237		async { self.resolve_from(&SourcePath::default(), path).await }238	}239	/// Resolves absolute path, doesn't supports jpath and other fancy things240	fn resolve(&self, path: &Path) -> impl Future<Output = Result<SourcePath, Self::Error>>;241242	/// Load resolved file243	/// This should only be called with value returned244	/// from [`ImportResolver::resolve_file`]/[`ImportResolver::resolve`],245	/// this cannot be resolved using associated type,246	/// as the evaluator uses object instead of generic for [`ImportResolver`]247	fn load_file_contents(248		&self,249		resolved: &SourcePath,250	) -> impl Future<Output = Result<Vec<u8>, Self::Error>>;251}252253#[derive(Trace)]254struct ResolvedImportResolver {255	resolved: RefCell<GcHashMap<(SourcePath, IStr), (SourcePath, bool)>>,256}257impl ImportResolver for ResolvedImportResolver {258	fn load_file_contents(&self, _resolved: &SourcePath) -> crate::Result<Vec<u8>> {259		unreachable!("all files should be loaded at this point");260	}261262	fn resolve_from(&self, from: &SourcePath, path: &str) -> crate::Result<SourcePath> {263		Ok(self264			.resolved265			.borrow()266			.get(&(from.clone(), path.into()))267			.expect("all imports should be resolved at this point")268			.0269			.clone())270	}271272	fn resolve_from_default(&self, path: &str) -> crate::Result<SourcePath> {273		self.resolve_from(&SourcePath::default(), path)274	}275276	fn resolve(&self, path: &Path) -> crate::Result<SourcePath> {277		bail!(crate::error::ErrorKind::AbsoluteImportNotSupported(278			path.to_owned()279		))280	}281282	fn as_any(&self) -> &dyn std::any::Any {283		self284	}285}286287enum Job {288	LoadFile { path: SourcePath, parse: bool },289	ParseFile(SourcePath),290	ResolveImport { from: SourcePath, import: Import },291}292293#[allow(clippy::future_not_send)]294pub async fn async_import<H>(s: State, handler: H, path: impl AsRef<Path>) -> Result<(), H::Error>295where296	H: AsyncImportResolver,297{298	let mut resolved = s299		.import_resolver()300		.as_any()301		.downcast_ref::<ResolvedImportResolver>()302		.map_or_else(GcHashMap::new, |resolver| {303			std::mem::take(&mut *resolver.resolved.borrow_mut())304		});305	let mut queue = vec![Job::LoadFile {306		path: handler.resolve(path.as_ref()).await?,307		parse: true,308	}];309	while let Some(job) = queue.pop() {310		match job {311			Job::LoadFile { path, parse } => {312				if !s.0.file_cache.borrow().contains_key(&path) {313					let data = handler.load_file_contents(&path).await?;314					s.0.file_cache315						.borrow_mut()316						.insert(path.clone(), FileData::new_bytes(data.as_slice().into()));317				}318				if parse {319					queue.push(Job::ParseFile(path));320				}321			}322			Job::ParseFile(path) => {323				if let Some(file) = s.0.file_cache.borrow_mut().get_mut(&path) {324					if file.parsed.is_none() {325						let Some(code) = file.get_string() else {326							continue;327						};328						let source = Source::new(path.clone(), code.clone());329						// If failed - then skip import330						file.parsed =331							jrsonnet_parser::parse(&code, &ParserSettings { source }).ok();332						if let Some(parsed) = &file.parsed {333							let mut imports = FoundImports(vec![]);334							find_imports(parsed, &mut imports);335							for import in imports.0 {336								queue.push(Job::ResolveImport {337									from: path.clone(),338									import,339								});340							}341						}342					}343				}344			}345			Job::ResolveImport { from, import } => {346				if let Some((resolved, expression)) =347					resolved.get_mut(&(from.clone(), import.path.clone()))348				{349					if import.expression && !*expression {350						*expression = true;351						queue.push(Job::ParseFile(resolved.clone()));352					}353					continue;354				}355				let resolved = handler.resolve_from(&from, &import.path).await?;356				queue.push(Job::LoadFile {357					path: resolved,358					parse: import.expression,359				});360			}361		}362	}363	s.set_import_resolver(ResolvedImportResolver {364		resolved: RefCell::new(resolved),365	});366	Ok(())367}
modifiedflake.nixdiffbeforeafterboth
--- a/flake.nix
+++ b/flake.nix
@@ -25,7 +25,7 @@
         lib = pkgs.lib;
         rust =
           (pkgs.rustChannelOf {
-            date = "2023-08-02";
+            date = "2023-10-28";
             channel = "nightly";
           })
           .default
addedrust-toolchain.tomldiffbeforeafterboth
--- /dev/null
+++ b/rust-toolchain.toml
@@ -0,0 +1,3 @@
+[toolchain]
+channel = "nightly-2023-10-28"
+components = ["rustfmt", "clippy"]