refactor

.github/workflows/draft-release.yml

@@ -32,11 +32,8 @@ jobs:
         target: ${{ matrix.target }}
         override: true
 
-    - name: Build with Cargo
-      run: cargo build --release --workspace --target=${{ matrix.target }}
-
     - name: Pack the client
-      run: cargo xtask package .
+      run: cargo xtask package --out-dir .
 
     - name: Create draft release
       uses: ncipollo/release-action@v1

xtask/src/cli.rs

@@ -17,7 +17,9 @@ pub enum Commands {
     CopyLspRelease,
     /// Build and package VSCode extension into a .vsix file
     Package {
-        output_dir: Option<String>
+        /// Output directory for the .vsix file; default is the current working directory
+        #[arg(long)]
+        out_dir: Option<String>
     },
     /// Build, package and install the VSCode extension
     Install

xtask/src/commands/package.rs

@@ -8,11 +8,11 @@ const LSP_DST: &str = "./editors/vscode/server/bin";
 const EXT_DIR: &str = "./editors/vscode";
 const VSIX_NAME: &str = "witcherscript-ide.vsix";
 
-pub fn package(output_dir: Option<String>) -> anyhow::Result<()> {
+pub fn package(out_dir: Option<String>) -> anyhow::Result<()> {
     let sh = Shell::new()?;
 
     // normalize the output path so it stays valid when we change cwd
-    let output_dir = if let Some(output_dir) = output_dir {
+    let output_dir = if let Some(output_dir) = out_dir {
         Some(PathBuf::from(&output_dir).canonicalize()?)
     } else {
         None

xtask/src/main.rs

@@ -11,7 +11,7 @@ fn main() -> anyhow::Result<()> {
     match cli.command {
         cli::Commands::CopyLsp => commands::copy_lsp(),
         cli::Commands::CopyLspRelease => commands::copy_lsp_release(),
-        cli::Commands::Package { output_dir } => commands::package(output_dir),
+        cli::Commands::Package { out_dir } => commands::package(out_dir),
         cli::Commands::Install => commands::install()
     }
 }