Throw an error in scarb package with non-[lib] packages

scarb/src/ops/package.rs

@@ -4,7 +4,7 @@ use std::io::{Seek, SeekFrom, Write};
 
 use anyhow::{bail, ensure, Context, Result};
 use camino::Utf8PathBuf;
-use indoc::{indoc, writedoc};
+use indoc::{formatdoc, indoc, writedoc};
 
 use crate::sources::client::PackageRepository;
 
@@ -192,6 +192,19 @@ fn list_one_impl(
 }
 
 fn prepare_archive_recipe(pkg: &Package, opts: &PackageOpts) -> Result<ArchiveRecipe> {
+    ensure!(
+        pkg.manifest.targets.iter().any(|x| x.is_lib()),
+        formatdoc!(
+            r#"
+            cannot archive package `{package_name}` without a `lib` target
+            help: add `[lib]` section to package manifest
+             --> Scarb.toml
+            +   [lib]
+            "#,
+            package_name = pkg.id.name,
+        )
+    );
+
     let mut recipe = source_files(pkg)?;
 
     // Sort the recipe before any checks, to ensure generated errors are reproducible.

scarb/tests/package.rs

@@ -1039,3 +1039,28 @@ fn ignore_whitelist_pattern() {
             src/lib.cairo
         "#}));
 }
+
+#[test]
+fn no_lib_target() {
+    let t = TempDir::new().unwrap();
+    ProjectBuilder::start()
+        .name("foo")
+        .version("1.0.0")
+        .manifest_extra(indoc! {r#"
+        [[target.starknet-contract]]
+        "#})
+        .build(&t);
+
+    Scarb::quick_snapbox()
+        .arg("package")
+        .current_dir(&t)
+        .assert()
+        .failure()
+        .stdout_matches(indoc! {r#"
+        [..] Packaging foo v1.0.0 [..]
+        error: cannot archive package `foo` without a `lib` target
+        help: add `[lib]` section to package manifest
+         --> Scarb.toml
+        +   [lib]
+        "#});
+}