fix(deps): update rust crate kdl to v6 (#59)

* fix(deps): update rust crate kdl to v6

* fix: update code for kdl v6 API compatibility

- Remove .value() calls as node.get() now returns Option<&KdlValue> directly
- Update KDL boolean syntax from use-outputs=true to use-outputs=#true
- Fix all tests, fixtures, and documentation to use proper KDL v6 syntax

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Pedro Piñera <pedro@pepicrft.me>

Author: renovate[bot]

CLAUDE.md

@@ -992,7 +992,7 @@ Fabrik provides **content-addressed caching for arbitrary scripts** (bash, node,
 #FABRIK input "package.json"
 #FABRIK output "dist/"
 #FABRIK env "NODE_ENV"
-#FABRIK depends "./build-deps.sh" use-outputs=true
+#FABRIK depends "./build-deps.sh" use-outputs=#true
 
 # Build TypeScript project
 npm run build
@@ -1004,7 +1004,7 @@ npm run build
 - `#FABRIK output "path"` - Declare output paths to archive/restore (files or directories)
 - `#FABRIK env "VAR_NAME"` - Track environment variable (invalidates cache when value changes)
 - `#FABRIK depends "script.sh"` - Declare dependency on another script
-- `#FABRIK depends "script.sh" use-outputs=true` - Add dependency outputs as inputs
+- `#FABRIK depends "script.sh" use-outputs=#true` - Add dependency outputs as inputs
 - `#FABRIK cache disable` - Disable caching for this script
 - `#FABRIK cache ttl="7d"` - Set cache expiration (e.g., "2h", "7d", "30d")
 - `#FABRIK cache key="custom"` - Override cache key computation

Cargo.lock

@@ -26,7 +26,7 @@ num_cpus = "1.16"
 crossbeam-channel = "0.5"
 sha2 = "0.10"
 hex = "0.4"
-kdl = "4.6"
+kdl = "6.0"
 glob = "0.3"
 tar = "0.4"
 zstd = "0.13"

Cargo.toml

@@ -160,7 +160,7 @@ npm ci
 # build.sh
 #FABRIK input "src/**/*.ts"
 #FABRIK input "tsconfig.json"
-#FABRIK depends "./install.sh" use-outputs=true
+#FABRIK depends "./install.sh" use-outputs=#true
 #FABRIK output "dist/"
 
 npm run build
@@ -171,7 +171,7 @@ npm run build
 #!/usr/bin/env bash
 # test.sh
 #FABRIK input "tests/**/*.test.ts"
-#FABRIK depends "./build.sh" use-outputs=true
+#FABRIK depends "./build.sh" use-outputs=#true
 #FABRIK output "coverage/"
 
 npm test -- --coverage

docs/cache/scripts/examples.md

@@ -126,7 +126,7 @@ Declare dependencies on other scripts. Fabrik will execute dependencies before t
 
 **With output reuse:**
 ```bash
-#FABRIK depends "build-deps.sh" use-outputs=true
+#FABRIK depends "build-deps.sh" use-outputs=#true
 ```
 
 **Examples:**
@@ -135,14 +135,14 @@ Declare dependencies on other scripts. Fabrik will execute dependencies before t
 #FABRIK depends "./prepare.sh"
 
 # Dependency with output reuse (adds dependency outputs as inputs)
-#FABRIK depends "./build-libs.sh" use-outputs=true
+#FABRIK depends "./build-libs.sh" use-outputs=#true
 
 # Multiple dependencies
 #FABRIK depends "./step1.sh"
 #FABRIK depends "./step2.sh"
 ```
 
-**With `use-outputs=true`:**
+**With `use-outputs=#true`:**
 - Dependency's outputs are automatically added as inputs to current script
 - Ensures cache invalidation when dependency outputs change
 - Useful for build pipelines (build → test → deploy)
@@ -395,7 +395,7 @@ Here's a comprehensive example using multiple directives:
 #FABRIK output "build-stats.json"
 #FABRIK env "NODE_ENV"
 #FABRIK env "BUILD_TARGET"
-#FABRIK depends "./install-deps.sh" use-outputs=true
+#FABRIK depends "./install-deps.sh" use-outputs=#true
 #FABRIK cache ttl="24h"
 #FABRIK runtime-version
 #FABRIK exec timeout="10m"

docs/cache/scripts/reference.md

@@ -1,5 +1,5 @@
 #!/usr/bin/env -S fabrik run bash
-#FABRIK depends "./build.sh" use-outputs=true
+#FABRIK depends "./build.sh" use-outputs=#true
 #FABRIK input "build/"
 #FABRIK output "deploy.log"
 #FABRIK env "DEPLOY_ENV"

fixtures/cache/scripts/bash/deploy.sh

@@ -175,7 +175,7 @@ fn parse_kdl_node(annotations: &mut ScriptAnnotations, node: &KdlNode) -> Result
 
             let hash = node
                 .get("hash")
-                .and_then(|e| e.value().as_string())
+                .and_then(|e| e.as_string())
                 .unwrap_or("content");
 
             let hash_method = match hash {
@@ -202,7 +202,7 @@ fn parse_kdl_node(annotations: &mut ScriptAnnotations, node: &KdlNode) -> Result
 
             let required = node
                 .get("required")
-                .and_then(|e| e.value().as_bool())
+                .and_then(|e| e.as_bool())
                 .unwrap_or(true);
 
             annotations.outputs.push(OutputSpec { path, required });
@@ -218,13 +218,13 @@ fn parse_kdl_node(annotations: &mut ScriptAnnotations, node: &KdlNode) -> Result
         }
 
         "cache" => {
-            if let Some(ttl) = node.get("ttl").and_then(|e| e.value().as_string()) {
+            if let Some(ttl) = node.get("ttl").and_then(|e| e.as_string()) {
                 annotations.cache_ttl = Some(parse_duration(ttl)?);
             }
-            if let Some(key) = node.get("key").and_then(|e| e.value().as_string()) {
+            if let Some(key) = node.get("key").and_then(|e| e.as_string()) {
                 annotations.cache_key = Some(key.to_string());
             }
-            if let Some(disabled) = node.get("disabled").and_then(|e| e.value().as_bool()) {
+            if let Some(disabled) = node.get("disabled").and_then(|e| e.as_bool()) {
                 annotations.cache_disabled = disabled;
             }
         }
@@ -235,10 +235,7 @@ fn parse_kdl_node(annotations: &mut ScriptAnnotations, node: &KdlNode) -> Result
                 annotations.runtime = runtime_name;
             }
 
-            if let Some(include) = node
-                .get("include-version")
-                .and_then(|e| e.value().as_bool())
-            {
+            if let Some(include) = node.get("include-version").and_then(|e| e.as_bool()) {
                 annotations.runtime_version = include;
             }
         }
@@ -256,13 +253,13 @@ fn parse_kdl_node(annotations: &mut ScriptAnnotations, node: &KdlNode) -> Result
         }
 
         "exec" => {
-            if let Some(cwd) = node.get("cwd").and_then(|e| e.value().as_string()) {
+            if let Some(cwd) = node.get("cwd").and_then(|e| e.as_string()) {
                 annotations.exec_cwd = Some(PathBuf::from(cwd));
             }
-            if let Some(timeout) = node.get("timeout").and_then(|e| e.value().as_string()) {
+            if let Some(timeout) = node.get("timeout").and_then(|e| e.as_string()) {
                 annotations.exec_timeout = Some(parse_duration(timeout)?);
             }
-            if let Some(shell) = node.get("shell").and_then(|e| e.value().as_bool()) {
+            if let Some(shell) = node.get("shell").and_then(|e| e.as_bool()) {
                 annotations.exec_shell = shell;
             }
         }
@@ -273,7 +270,7 @@ fn parse_kdl_node(annotations: &mut ScriptAnnotations, node: &KdlNode) -> Result
 
             let use_outputs = node
                 .get("use-outputs")
-                .and_then(|e| e.value().as_bool())
+                .and_then(|e| e.as_bool())
                 .unwrap_or(false);
 
             annotations.depends_on.push(DependencySpec {

src/script/annotations.rs

@@ -224,7 +224,7 @@ echo "build"
         fs::write(
             &main_script,
             r#"#!/usr/bin/env -S fabrik run bash
-#FABRIK depends "./dep.sh" use-outputs=true
+#FABRIK depends "./dep.sh" use-outputs=#true
 echo "deploy"
 "#,
         )