fix: restore CI after main rebase

Author: ubugeeei

Cargo.lock

@@ -19,12 +19,16 @@ Vite+ is the unified entry point for local web development. It combines [Vite](h
 - **`vp build`:** Build applications for production with Vite + Rolldown
 - **`vp run`:** Execute monorepo tasks with caching and dependency-aware scheduling
 - **`vp pack`:** Build libraries for npm publishing or standalone app binaries
-- **`vp release`:** Publish workspace releases
+- **`vp release`:** Version and publish workspace packages with native publish preflight during `--dry-run`, release checks before real publishes by default, retry-friendly exact version overrides via `--version`, optional changelog generation via `--changelog`, prerelease channels like `--preid alpha` / `beta` / `rc`, and `--projects` order respected between independent packages
 - **`vp create` / `vp migrate`:** Scaffold new projects and migrate existing ones
 
 All of this is configured from your project root and works across Vite's framework ecosystem.
 Vite+ is fully open-source under the MIT license.
 
+`vp release` detects likely checks from `build`, `pack`, `prepack`, `prepublishOnly`, `prepare`, and `vitePlus.release.checkScripts`. Real releases run those checks before publishing unless you pass `--no-run-checks`; dry-runs stay lightweight by default and can opt in with `--run-checks`. `--dry-run` also runs the native publisher in dry-run mode from a temporary release manifest state when the git worktree is clean. Use `--yes` for CI or other non-interactive runs, and `--version <x.y.z>` when retrying a partial publish at an exact version.
+
+Real releases always create git tags after a successful publish. When every released package shares the same target version, `vp release` also creates a repository-level `v<version>` tag so GitHub Releases and repo-wide release notes can follow the same watermark. Preview-only flags such as `--skip-publish` and `--no-git-tag` are therefore limited to `--dry-run`.
+
 ## Getting Started
 
 Install Vite+ globally as `vp`:
@@ -121,7 +125,7 @@ Use `vp migrate` to migrate to Vite+. It merges tool-specific config files such
 
 - **build** - Build for production
 - **pack** - Build libraries
-- **release** - Publish workspace releases
+- **release** - Version and publish workspace packages, with optional changelog generation
 - **preview** - Preview production build
 
 #### Manage Dependencies

README.md

@@ -18,7 +18,7 @@
 
 use std::{collections::HashMap, io::BufReader};
 
-use vite_install::package_manager::{PackageManager, PackageManagerType};
+use vite_install::package_manager::PackageManager;
 use vite_path::AbsolutePath;
 use vite_shared::{PrependOptions, prepend_to_path_env};
 
@@ -97,37 +97,6 @@ pub async fn build_package_manager(cwd: &AbsolutePath) -> Result<PackageManager,
     }
 }
 
-/// Build a PackageManager, falling back to a default npm instance when no
-/// package.json is found. Uses `build()` instead of `build_with_default()`
-/// to skip the interactive package manager selection prompt on the fallback path.
-///
-/// Requires `prepend_js_runtime_to_path_env` to be called first so npm is on PATH.
-pub async fn build_package_manager_or_npm_default(
-    cwd: &AbsolutePath,
-) -> Result<PackageManager, Error> {
-    match PackageManager::builder(cwd).build().await {
-        Ok(pm) => Ok(pm),
-        Err(vite_error::Error::WorkspaceError(vite_workspace::Error::PackageJsonNotFound(_)))
-        | Err(vite_error::Error::UnrecognizedPackageManager) => {
-            Ok(default_npm_package_manager(cwd))
-        }
-        Err(e) => Err(e.into()),
-    }
-}
-
-fn default_npm_package_manager(cwd: &AbsolutePath) -> PackageManager {
-    PackageManager {
-        client: PackageManagerType::Npm,
-        package_name: "npm".into(),
-        version: "latest".into(),
-        hash: None,
-        bin_name: "npm".into(),
-        workspace_root: cwd.to_absolute_path_buf(),
-        is_monorepo: false,
-        install_dir: cwd.to_absolute_path_buf(),
-    }
-}
-
 pub mod release;
 
 // Category B: JS Script Commands

crates/vite_global_cli/src/commands/mod.rs

@@ -297,10 +297,8 @@ impl<'a> ChecklistStep<'a> {
         let iter = lines.into_iter();
         let (lower, _) = iter.size_hint();
         let mut collected = Vec::with_capacity(lower);
-        for line in iter {
-            if let Some(line) = line {
-                collected.push(line);
-            }
+        for line in iter.flatten() {
+            collected.push(line);
         }
         Self { title, lines: collected }
     }
@@ -587,7 +585,7 @@ fn find_existing_release_workflow_path(cwd: &AbsolutePath) -> Option<String> {
             let mut path = String::with_capacity(file_name.len() + 18);
             path.push_str(".github/workflows/");
             path.push_str(&file_name);
-            let should_replace = best_path.as_ref().map_or(true, |best| {
+            let should_replace = best_path.as_ref().is_none_or(|best| {
                 let best_lowercase = best.to_ascii_lowercase();
                 let best_mentions_publish = best_lowercase.contains("publish");
                 (mentions_publish && !best_mentions_publish)

crates/vite_global_cli/src/commands/release/first_publish.rs

@@ -388,7 +388,7 @@ impl ReleaseManager {
                     plan.repository_url
                         .as_deref()
                         .and_then(parse_github_repo_slug)
-                        .map_or(true, |slug| slug != expected_repository)
+                        .is_none_or(|slug| slug != expected_repository)
                 })
                 .map(|plan| plan.name.as_str())
                 .collect();

crates/vite_global_cli/src/commands/release/manager.rs

@@ -6,6 +6,8 @@
 //! - Conventional Commits 1.0.0: https://www.conventionalcommits.org/en/v1.0.0/#specification
 //! - Conventional Commits FAQ: https://www.conventionalcommits.org/en/v1.0.0/#faq
 
+#![allow(clippy::wildcard_imports)]
+
 use std::{
     collections::{BTreeMap, HashMap, HashSet},
     fmt::{self, Write as _},

crates/vite_global_cli/src/commands/release/mod.rs

@@ -343,8 +343,7 @@ pub(super) fn find_latest_package_version(
     Ok(stdout
         .lines()
         .map(str::trim)
-        .filter_map(|tag_name| RELEASE_TAG_FORMAT.parse_version(tag_name))
-        .next())
+        .find_map(|tag_name| RELEASE_TAG_FORMAT.parse_version(tag_name)))
 }
 
 pub(super) fn find_latest_repository_release_tag(
@@ -362,7 +361,7 @@ pub(super) fn find_latest_repository_release_version(
     cwd: &AbsolutePath,
 ) -> Result<Option<Version>, Error> {
     let stdout = capture_git(cwd, ["tag", "--list", "--sort=-creatordate", "v*"])?;
-    Ok(stdout.lines().map(str::trim).filter_map(parse_repository_release_version).next())
+    Ok(stdout.lines().map(str::trim).find_map(parse_repository_release_version))
 }
 
 pub(super) fn find_latest_repository_stable_release_version(

crates/vite_global_cli/src/commands/release/planning.rs

@@ -568,7 +568,7 @@ pub(super) fn confirm_release(
         let joined_channels = custom_prerelease_channels.join(", ");
         let mut prompt = String::from("Continue with non-standard prerelease channel");
         prompt.push_str(suffix);
-        prompt.push_str(" ");
+        prompt.push(' ');
         prompt.push_str(&joined_channels);
         prompt.push_str("? [y/N] ");
         if !prompt_for_confirmation(&prompt, false)? {

crates/vite_global_cli/src/commands/release/reporting.rs

@@ -366,13 +366,11 @@ pub(super) async fn publish_packages(
 
 /// Applies final release artifacts transactionally, rolling back already-written files on error.
 pub(super) fn apply_release_artifact_edits(edits: &[ReleaseArtifactEdit]) -> Result<(), Error> {
-    let mut applied_count = 0usize;
-    for edit in edits {
+    for (applied_count, edit) in edits.iter().enumerate() {
         if let Err(error) = write_release_artifact_edit(edit) {
             let rollback_result = rollback_applied_release_artifact_edits(&edits[..applied_count]);
             return Err(release_artifact_error("write release artifacts", error, rollback_result));
         }
-        applied_count += 1;
     }
 
     Ok(())
@@ -567,8 +565,7 @@ pub(super) fn apply_manifest_edits(
     manifest_edits: &[ManifestEdit],
     restore_original: bool,
 ) -> Result<(), Error> {
-    let mut applied_edits = 0usize;
-    for edit in manifest_edits {
+    for (applied_edits, edit) in manifest_edits.iter().enumerate() {
         let contents =
             if restore_original { &edit.original_contents } else { &edit.updated_contents };
         if let Err(error) = fs::write(&edit.path, contents) {
@@ -589,7 +586,6 @@ pub(super) fn apply_manifest_edits(
             }
             return Err(Error::UserMessage(message.into()));
         }
-        applied_edits += 1;
     }
 
     Ok(())

crates/vite_global_cli/src/commands/release/storage.rs

@@ -211,12 +211,12 @@ impl PackageManager {
                 let can_use_native_yarn = !self.version.starts_with("1.")
                     && options.target.is_none()
                     && !options.recursive
-                    && options.filters.map_or(true, |filters| filters.is_empty())
+                    && options.filters.is_none_or(|filters| filters.is_empty())
                     && options.publish_branch.is_none()
                     && !options.report_summary
                     && options
                         .pass_through_args
-                        .map_or(true, |pass_through_args| pass_through_args.is_empty())
+                        .is_none_or(|pass_through_args| pass_through_args.is_empty())
                     && !options.force;
 
                 if can_use_native_yarn {