[INFO] cloning repository https://github.com/sorunokoe/brief
[INFO] running `Command { std: "git" "-c" "credential.helper=" "-c" "credential.helper=/workspace/cargo-home/bin/git-credential-null" "clone" "--bare" "https://github.com/sorunokoe/brief" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fsorunokoe%2Fbrief", kill_on_drop: false }`
[INFO] [stderr] Cloning into bare repository '/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fsorunokoe%2Fbrief'...
[INFO] running `Command { std: "git" "rev-parse" "HEAD", kill_on_drop: false }`
[INFO] [stdout] ea5167f9e24aaf010424462811e4d368d990ffd7
[INFO] checking sorunokoe/brief against 011c518242557d21690f7936b5812318dc2fec47 for pr-159877
[INFO] running `Command { std: "git" "clone" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fsorunokoe%2Fbrief" "/workspace/builds/worker-0-tc2/source", kill_on_drop: false }`
[INFO] [stderr] Cloning into '/workspace/builds/worker-0-tc2/source'...
[INFO] [stderr] done.
[INFO] started tweaking git repo https://github.com/sorunokoe/brief
[INFO] finished tweaking git repo https://github.com/sorunokoe/brief
[INFO] tweaked toml for git repo https://github.com/sorunokoe/brief written to /workspace/builds/worker-0-tc2/source/Cargo.toml
[INFO] validating manifest of git repo https://github.com/sorunokoe/brief on toolchain 011c518242557d21690f7936b5812318dc2fec47
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+011c518242557d21690f7936b5812318dc2fec47" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }`
[INFO] crate git repo https://github.com/sorunokoe/brief already has a lockfile, it will not be regenerated
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+011c518242557d21690f7936b5812318dc2fec47" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }`
[INFO] [stderr]     Blocking waiting for file lock on package cache
[INFO] [stderr]     Updating crates.io index
[INFO] [stderr]     Blocking waiting for file lock on package cache
[INFO] [stderr]  Downloading crates ...
[INFO] [stderr]   Downloaded pin-project-internal v1.1.12
[INFO] [stderr]   Downloaded pin-project v1.1.12
[INFO] [stderr]   Downloaded tower-lsp-macros v0.9.0
[INFO] [stderr]   Downloaded logos-derive v0.14.4
[INFO] [stderr]   Downloaded lsp-types v0.94.1
[INFO] [stderr]   Downloaded kqueue-sys v1.1.1
[INFO] [stderr]   Downloaded clap_complete v4.6.5
[INFO] [stderr]   Downloaded tower-lsp v0.20.0
[INFO] [stderr]   Downloaded logos-codegen v0.14.4
[INFO] [stderr]   Downloaded logos v0.14.4
[INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-0-tc2/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-0-tc2/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/cargo-home:/opt/rustwide/cargo-home:ro,Z" "-v" "/var/lib/crater-agent-workspace/rustup-home:/opt/rustwide/rustup-home:ro,Z" "-m" "1610612736" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:1fa6594d87c8fa7b311925a7a77a16f539833322d9dfba4ce891f211db2f90b1" "sleep" "infinity", kill_on_drop: false }`
[INFO] [stdout] 749ead86e1ad49f46c669a3f3e8e6d1431f802b906ba5b28cc9226da5fefa1ed
[INFO] running `Command { std: "docker" "start" "749ead86e1ad49f46c669a3f3e8e6d1431f802b906ba5b28cc9226da5fefa1ed", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "749ead86e1ad49f46c669a3f3e8e6d1431f802b906ba5b28cc9226da5fefa1ed", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "exec" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-w" "/opt/rustwide/workdir" "--user" "0:0" "749ead86e1ad49f46c669a3f3e8e6d1431f802b906ba5b28cc9226da5fefa1ed" "/opt/rustwide/cargo-home/bin/cargo" "+011c518242557d21690f7936b5812318dc2fec47" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "749ead86e1ad49f46c669a3f3e8e6d1431f802b906ba5b28cc9226da5fefa1ed", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "exec" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_INCREMENTAL=0" "-e" "RUST_BACKTRACE=full" "-e" "RUSTFLAGS=--cap-lints=forbid" "-e" "RUSTDOCFLAGS=--cap-lints=forbid" "-w" "/opt/rustwide/workdir" "--user" "0:0" "749ead86e1ad49f46c669a3f3e8e6d1431f802b906ba5b28cc9226da5fefa1ed" "/opt/rustwide/cargo-home/bin/cargo" "+011c518242557d21690f7936b5812318dc2fec47" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }`
[INFO] [stderr]     Checking futures-channel v0.3.32
[INFO] [stderr]     Checking generic-array v0.14.7
[INFO] [stderr]     Checking serde v1.0.228
[INFO] [stderr]     Checking ring v0.17.14
[INFO] [stderr]     Checking serde_json v1.0.149
[INFO] [stderr]    Compiling pin-project-internal v1.1.12
[INFO] [stderr]    Compiling rustls v0.23.40
[INFO] [stderr]    Compiling logos-codegen v0.14.4
[INFO] [stderr]     Checking tokio v1.52.3
[INFO] [stderr]     Checking inotify-sys v0.1.5
[INFO] [stderr]     Checking hashbrown v0.14.5
[INFO] [stderr]     Checking tracing v0.1.44
[INFO] [stderr]     Checking webpki-roots v0.26.11
[INFO] [stderr]     Checking clap v4.6.1
[INFO] [stderr]     Checking mio v0.8.11
[INFO] [stderr]    Compiling tower-lsp-macros v0.9.0
[INFO] [stderr]    Compiling auto_impl v1.3.0
[INFO] [stderr]     Checking clap_complete v4.6.5
[INFO] [stderr]     Checking inotify v0.9.6
[INFO] [stderr]     Checking tempfile v3.27.0
[INFO] [stderr]     Checking futures-util v0.3.32
[INFO] [stderr]     Checking dashmap v5.5.3
[INFO] [stderr]     Checking notify v6.1.1
[INFO] [stderr]     Checking url v2.5.8
[INFO] [stderr]     Checking serde_spanned v0.6.9
[INFO] [stderr]     Checking toml_datetime v0.6.11
[INFO] [stderr]     Checking block-buffer v0.10.4
[INFO] [stderr]     Checking crypto-common v0.1.7
[INFO] [stderr]     Checking toml_edit v0.22.27
[INFO] [stderr]     Checking digest v0.10.7
[INFO] [stderr]     Checking pin-project v1.1.12
[INFO] [stderr]     Checking lsp-types v0.94.1
[INFO] [stderr]     Checking sha2 v0.10.9
[INFO] [stderr]    Compiling logos-derive v0.14.4
[INFO] [stderr]     Checking logos v0.14.4
[INFO] [stderr]     Checking rustls-webpki v0.103.13
[INFO] [stderr]     Checking toml v0.8.23
[INFO] [stderr]     Checking futures v0.3.32
[INFO] [stderr]     Checking tower v0.4.13
[INFO] [stderr]     Checking ureq v2.12.1
[INFO] [stderr]     Checking tokio-util v0.7.18
[INFO] [stderr]     Checking tower-lsp v0.20.0
[INFO] [stderr]     Checking briefc v1.0.0 (/opt/rustwide/workdir/briefc)
[INFO] [stdout] warning: unused import: `PathBuf`
[INFO] [stdout]  --> briefc/src/skillsync.rs:9:23
[INFO] [stdout]   |
[INFO] [stdout] 9 | use std::path::{Path, PathBuf};
[INFO] [stdout]   |                       ^^^^^^^
[INFO] [stdout]   |
[INFO] [stdout]   = note: `#[warn(unused_imports)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `std::io::Write`
[INFO] [stdout]    --> briefc/src/audit.rs:149:9
[INFO] [stdout]     |
[INFO] [stdout] 149 |     use std::io::Write;
[INFO] [stdout]     |         ^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(unused_imports)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `std::io::Write`
[INFO] [stdout]    --> briefc/src/policy_check.rs:158:9
[INFO] [stdout]     |
[INFO] [stdout] 158 |     use std::io::Write;
[INFO] [stdout]     |         ^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `PathBuf`
[INFO] [stdout]  --> briefc/src/skillsync.rs:9:23
[INFO] [stdout]   |
[INFO] [stdout] 9 | use std::path::{Path, PathBuf};
[INFO] [stdout]   |                       ^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `std::io::Write`
[INFO] [stdout]    --> briefc/src/suggest.rs:329:9
[INFO] [stdout]     |
[INFO] [stdout] 329 |     use std::io::Write;
[INFO] [stdout]     |         ^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `env_vals`
[INFO] [stdout]   --> briefc/src/suggest.rs:91:9
[INFO] [stdout]    |
[INFO] [stdout] 91 |     let env_vals: Vec<(String, String)> = declared_needs.iter()
[INFO] [stdout]    |         ^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_env_vals`
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variable does not need to be mutable
[INFO] [stdout]   --> briefc/src/verify.rs:98:13
[INFO] [stdout]    |
[INFO] [stdout] 98 |         let mut all_verified = needs_results;
[INFO] [stdout]    |             ----^^^^^^^^^^^^
[INFO] [stdout]    |             |
[INFO] [stdout]    |             help: remove this `mut`
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(unused_mut)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `env_vals`
[INFO] [stdout]   --> briefc/src/suggest.rs:91:9
[INFO] [stdout]    |
[INFO] [stdout] 91 |     let env_vals: Vec<(String, String)> = declared_needs.iter()
[INFO] [stdout]    |         ^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_env_vals`
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variable does not need to be mutable
[INFO] [stdout]   --> briefc/src/verify.rs:98:13
[INFO] [stdout]    |
[INFO] [stdout] 98 |         let mut all_verified = needs_results;
[INFO] [stdout]    |             ----^^^^^^^^^^^^
[INFO] [stdout]    |             |
[INFO] [stdout]    |             help: remove this `mut`
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(unused_mut)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: field `ts` is never read
[INFO] [stdout]    --> briefc/src/audit.rs:140:5
[INFO] [stdout]     |
[INFO] [stdout] 135 | struct CallEntry {
[INFO] [stdout]     |        --------- field in this struct
[INFO] [stdout] ...
[INFO] [stdout] 140 |     ts: String,
[INFO] [stdout]     |     ^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: function `check_source_no_ctx` is never used
[INFO] [stdout]     --> briefc/src/checker.rs:3510:8
[INFO] [stdout]      |
[INFO] [stdout] 3510 |     fn check_source_no_ctx(source: &str) -> Vec<BriefError> {
[INFO] [stdout]      |        ^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: method `is_permitted` is never used
[INFO] [stdout]   --> briefc/src/enforcer.rs:22:12
[INFO] [stdout]    |
[INFO] [stdout] 21 | impl CallDecision {
[INFO] [stdout]    | ----------------- method in this implementation
[INFO] [stdout] 22 |     pub fn is_permitted(&self) -> bool {
[INFO] [stdout]    |            ^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variants `UnknownEffectGroup`, `NeedNotMet`, and `EmptyAllowBlock` are never constructed
[INFO] [stdout]   --> briefc/src/errors.rs:26:5
[INFO] [stdout]    |
[INFO] [stdout] 12 | pub enum ErrorCode {
[INFO] [stdout]    |          --------- variants in this enum
[INFO] [stdout] ...
[INFO] [stdout] 26 |     UnknownEffectGroup,
[INFO] [stdout]    |     ^^^^^^^^^^^^^^^^^^
[INFO] [stdout] ...
[INFO] [stdout] 49 |     NeedNotMet,
[INFO] [stdout]    |     ^^^^^^^^^^
[INFO] [stdout] ...
[INFO] [stdout] 57 |     EmptyAllowBlock,
[INFO] [stdout]    |     ^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `ErrorCode` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: field `project` is never read
[INFO] [stdout]   --> briefc/src/manifest.rs:41:9
[INFO] [stdout]    |
[INFO] [stdout] 40 | pub struct BriefManifest {
[INFO] [stdout]    |            ------------- field in this struct
[INFO] [stdout] 41 |     pub project:   ProjectSection,
[INFO] [stdout]    |         ^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `BriefManifest` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: fields `name`, `version`, and `authors` are never read
[INFO] [stdout]    --> briefc/src/manifest.rs:135:9
[INFO] [stdout]     |
[INFO] [stdout] 134 | pub struct ProjectSection {
[INFO] [stdout]     |            -------------- fields in this struct
[INFO] [stdout] 135 |     pub name:    String,
[INFO] [stdout]     |         ^^^^
[INFO] [stdout] 136 |     #[serde(default = "default_version")]
[INFO] [stdout] 137 |     pub version: String,
[INFO] [stdout]     |         ^^^^^^^
[INFO] [stdout] 138 |     #[serde(default)]
[INFO] [stdout] 139 |     pub authors: Vec<String>,
[INFO] [stdout]     |         ^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: `ProjectSection` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: method `skill_config` is never used
[INFO] [stdout]    --> briefc/src/manifest.rs:167:12
[INFO] [stdout]     |
[INFO] [stdout] 152 | impl BriefManifest {
[INFO] [stdout]     | ------------------ method in this implementation
[INFO] [stdout] ...
[INFO] [stdout] 167 |     pub fn skill_config(&self, name: &str) -> Option<SkillConfig> {
[INFO] [stdout]     |            ^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: function `collect_perform_calls` is never used
[INFO] [stdout]    --> briefc/src/runner.rs:375:4
[INFO] [stdout]     |
[INFO] [stdout] 375 | fn collect_perform_calls(expr: &crate::ast::Expr) -> Vec<String> {
[INFO] [stdout]     |    ^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: field `ts` is never read
[INFO] [stdout]    --> briefc/src/audit.rs:140:5
[INFO] [stdout]     |
[INFO] [stdout] 135 | struct CallEntry {
[INFO] [stdout]     |        --------- field in this struct
[INFO] [stdout] ...
[INFO] [stdout] 140 |     ts: String,
[INFO] [stdout]     |     ^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: method `is_permitted` is never used
[INFO] [stdout]   --> briefc/src/enforcer.rs:22:12
[INFO] [stdout]    |
[INFO] [stdout] 21 | impl CallDecision {
[INFO] [stdout]    | ----------------- method in this implementation
[INFO] [stdout] 22 |     pub fn is_permitted(&self) -> bool {
[INFO] [stdout]    |            ^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variants `UnknownEffectGroup`, `NeedNotMet`, and `EmptyAllowBlock` are never constructed
[INFO] [stdout]   --> briefc/src/errors.rs:26:5
[INFO] [stdout]    |
[INFO] [stdout] 12 | pub enum ErrorCode {
[INFO] [stdout]    |          --------- variants in this enum
[INFO] [stdout] ...
[INFO] [stdout] 26 |     UnknownEffectGroup,
[INFO] [stdout]    |     ^^^^^^^^^^^^^^^^^^
[INFO] [stdout] ...
[INFO] [stdout] 49 |     NeedNotMet,
[INFO] [stdout]    |     ^^^^^^^^^^
[INFO] [stdout] ...
[INFO] [stdout] 57 |     EmptyAllowBlock,
[INFO] [stdout]    |     ^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `ErrorCode` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: field `authors` is never read
[INFO] [stdout]    --> briefc/src/manifest.rs:139:9
[INFO] [stdout]     |
[INFO] [stdout] 134 | pub struct ProjectSection {
[INFO] [stdout]     |            -------------- field in this struct
[INFO] [stdout] ...
[INFO] [stdout] 139 |     pub authors: Vec<String>,
[INFO] [stdout]     |         ^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: `ProjectSection` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: method `skill_config` is never used
[INFO] [stdout]    --> briefc/src/manifest.rs:167:12
[INFO] [stdout]     |
[INFO] [stdout] 152 | impl BriefManifest {
[INFO] [stdout]     | ------------------ method in this implementation
[INFO] [stdout] ...
[INFO] [stdout] 167 |     pub fn skill_config(&self, name: &str) -> Option<SkillConfig> {
[INFO] [stdout]     |            ^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: function `collect_perform_calls` is never used
[INFO] [stdout]    --> briefc/src/runner.rs:375:4
[INFO] [stdout]     |
[INFO] [stdout] 375 | fn collect_perform_calls(expr: &crate::ast::Expr) -> Vec<String> {
[INFO] [stdout]     |    ^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr]     Finished `dev` profile [unoptimized + debuginfo] target(s) in 27.13s
[INFO] running `Command { std: "docker" "inspect" "749ead86e1ad49f46c669a3f3e8e6d1431f802b906ba5b28cc9226da5fefa1ed", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "749ead86e1ad49f46c669a3f3e8e6d1431f802b906ba5b28cc9226da5fefa1ed", kill_on_drop: false }`
[INFO] [stdout] 749ead86e1ad49f46c669a3f3e8e6d1431f802b906ba5b28cc9226da5fefa1ed
