Multiple Extra Bpl

[andrii-a8c]

Related: #548

---

Note

Medium Risk
Changes how the prover ingests extra_bpl by allowing multiple files and altering error handling (now continues on invalid paths), plus a broad dependency lockfile update that could affect build/runtime behavior.

Overview
Adds support for specifying multiple extra_bpl attributes on specs/modules by changing validate_and_read_extra_bpl to accept a list of paths, read all valid .bpl files, and concatenate their contents (instead of a single optional file).

Adds integration test cases and snapshots covering both module-level and function-level multi-extra_bpl usage, and updates Cargo.lock with a broad set of dependency bumps (including tokio, rustls, clap, etc.) and new transitive crates.

^{Written by Cursor Bugbot for commit 2132e5c. This will update automatically on new commits. Configure here.}

[andreistefanescu]

👍

[cursor]

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Free Tier Details

Your team is on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle for each member of your team.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

Bugbot Autofix prepared a fix for the issue found in the latest run.

• ✅ Fixed: Partial success on error returns incomplete BPL content
  • Updated extra_bpl loading to collect diagnostics but return None if any file validation/read error occurs, preventing partial BPL content from being used.

Or push these changes by commenting:

@cursor push f3cb350510

Preview (f3cb350510)

diff --git a/crates/move-stackless-bytecode/src/package_targets.rs b/crates/move-stackless-bytecode/src/package_targets.rs
--- a/crates/move-stackless-bytecode/src/package_targets.rs
+++ b/crates/move-stackless-bytecode/src/package_targets.rs
@@ -764,6 +764,7 @@
         extra_bpl: &Vec<String>,
     ) -> Option<String> {
         let mut contents = Vec::new();
+        let mut has_errors = false;
         for path_str in extra_bpl {
             let extra_path = Path::new(path_str);
 
@@ -773,6 +774,7 @@
                     loc,
                     &format!("extra_bpl path must have .bpl extension: '{}'", path_str),
                 );
+                has_errors = true;
                 continue;
             }
 
@@ -795,6 +797,7 @@
                         resolved_path.display()
                     ),
                 );
+                has_errors = true;
                 continue;
             }
 
@@ -810,10 +813,11 @@
                             err
                         ),
                     );
+                    has_errors = true;
                 }
             }
         }
-        if contents.is_empty() {
+        if has_errors || contents.is_empty() {
             None
         } else {
             Some(contents.join("\n"))

_{This Bugbot Autofix run was free. To enable autofix for future PRs, go to the Cursor dashboard.}