build: add more profiles

Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 4 to 7.
- [Release notes](https://github.com/actions/upload-artifact/releases)
- [Commits](https://github.com/actions/upload-artifact/compare/v4...v7)

---
updated-dependencies:
- dependency-name: actions/upload-artifact
  dependency-version: '7'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

.github/workflows/build.yaml

@@ -24,13 +24,13 @@ jobs:
 
         steps:
             - name: Clone repository
-              uses: actions/checkout@v4
+              uses: actions/checkout@v6
 
             - name: Install build dependencies
               run: sudo apt install -y libluajit-5.1-dev mold
 
             - name: Set up build cache
-              uses: actions/cache@v4
+              uses: actions/cache@v5
               with:
                   path: |
                       ~/.cargo/bin/
@@ -46,7 +46,7 @@ jobs:
             - run: cargo build --release ${{ matrix.feature.flags }}
 
             - name: Upload build artifacts
-              uses: actions/upload-artifact@v4
+              uses: actions/upload-artifact@v7
               with:
                   name: errornowatcher_${{ matrix.feature.name }}_${{ matrix.os }}
                   path: target/release/errornowatcher

.github/workflows/lint.yaml

@@ -15,7 +15,7 @@ jobs:
 
         steps:
             - name: Clone repository
-              uses: actions/checkout@v4
+              uses: actions/checkout@v6
 
             - name: Install taplo
               uses: uncenter/setup-taplo@v1
@@ -47,13 +47,13 @@ jobs:
 
         steps:
             - name: Clone repository
-              uses: actions/checkout@v4
+              uses: actions/checkout@v6
 
             - name: Install build dependencies
               run: sudo apt install -y libluajit-5.1-dev mold
 
             - name: Set up build cache
-              uses: actions/cache@v4
+              uses: actions/cache@v5
               with:
                   path: |
                       ~/.cargo/bin/
@@ -69,7 +69,7 @@ jobs:
             - name: Install components
               run: rustup component add clippy rustfmt
 
-            - run: cargo clippy ${{ matrix.feature.flags }} -D clippy::pedantic
+            - run: cargo clippy ${{ matrix.feature.flags }} -- -D clippy::pedantic
 
             - if: always()
               run: cargo fmt --check

Cargo.toml

@@ -15,6 +15,14 @@ codegen-units = 1
 lto = true
 strip = true
 
+[profile.release-no-lto]
+inherits = "release"
+lto = false
+
+[profile.small]
+inherits = "release"
+opt-level = "z"
+
 [build-dependencies]
 built = { version = "0", features = ["git2"] }
 

src/lua/client/movement.rs

@@ -92,7 +92,8 @@ pub async fn go_to(
     client
         .goto_with_opts(
             goal,
-            PathfinderOpts::new().allow_mining(options.get("without_mining").unwrap_or_default()),
+            PathfinderOpts::new()
+                .allow_mining(!options.get::<bool>("without_mining").unwrap_or_default()),
         )
         .await;
 
@@ -116,7 +117,8 @@ pub async fn start_go_to(
     )?;
     client.start_goto_with_opts(
         goal,
-        PathfinderOpts::new().allow_mining(options.get("without_mining").unwrap_or_default()),
+        PathfinderOpts::new()
+            .allow_mining(!options.get::<bool>("without_mining").unwrap_or_default()),
     );
     let _ = client.get_tick_broadcaster().recv().await;
 

src/lua/client/world/queries.rs

@@ -2,7 +2,7 @@
 macro_rules! get_entities {
     ($client:ident) => {{
         let ecs = $client.ecs.read();
-        if let Some(mut query) = ecs.try_query::<(
+        ecs.try_query::<(
             &AzaleaPosition,
             &CustomName,
             &EntityKindComponent,
@@ -11,7 +11,8 @@ macro_rules! get_entities {
             &MinecraftEntityId,
             Option<&Owneruuid>,
             &Pose,
-        )>() {
+        )>()
+        .map(|mut query| {
             query
                 .iter(&ecs)
                 .map(
@@ -29,9 +30,8 @@ macro_rules! get_entities {
                     },
                 )
                 .collect::<Vec<_>>()
-        } else {
-            Vec::new()
-        }
+        })
+        .unwrap_or_default()
     }};
 }
 
@@ -39,7 +39,7 @@ macro_rules! get_entities {
 macro_rules! get_players {
     ($client:ident) => {{
         let ecs = $client.ecs.read();
-        if let Some(mut query) = ecs.try_query_filtered::<(
+        ecs.try_query_filtered::<(
             &MinecraftEntityId,
             &EntityUuid,
             &EntityKindComponent,
@@ -47,7 +47,7 @@ macro_rules! get_players {
             &LookDirection,
             &Pose,
         ), (With<Player>, Without<Dead>)>()
-        {
+            .map(|mut query| {
                 query
                     .iter(&ecs)
                     .map(|(id, uuid, kind, position, direction, pose)| {
@@ -61,8 +61,7 @@ macro_rules! get_players {
                         )
                     })
                     .collect::<Vec<_>>()
-        } else {
-            Vec::new()
-        }
+            })
+            .unwrap_or_default()
     }};
 }