difftreelog
Merge pull request #138 from CertainLach/stable-async-traits
in: master
7 files changed
.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
Cargo.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]]
Cargo.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"
crates/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
crates/jrsonnet-evaluator/src/async_import.rsdiffbeforeafterboth1use 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}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}flake.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
rust-toolchain.tomldiffbeforeafterboth--- /dev/null
+++ b/rust-toolchain.toml
@@ -0,0 +1,3 @@
+[toolchain]
+channel = "nightly-2023-10-28"
+components = ["rustfmt", "clippy"]