Merge pull request #5269 from gitbutlerapp/refactor-stack

Refactor - join stack and stack api crates

Cargo.toml

@@ -33,7 +33,6 @@ members = [
     "crates/gitbutler-edit-mode",
     "crates/gitbutler-cherry-pick",
     "crates/gitbutler-oxidize",
-    "crates/gitbutler-stack-api",
     "crates/gitbutler-stack",
     "crates/gitbutler-patch-reference",
 ]
@@ -91,7 +90,6 @@ gitbutler-operating-modes = { path = "crates/gitbutler-operating-modes" }
 gitbutler-edit-mode = { path = "crates/gitbutler-edit-mode" }
 gitbutler-cherry-pick = { path = "crates/gitbutler-cherry-pick" }
 gitbutler-oxidize = { path = "crates/gitbutler-oxidize" }
-gitbutler-stack-api = { path = "crates/gitbutler-stack-api" }
 gitbutler-stack = { path = "crates/gitbutler-stack" }
 gitbutler-patch-reference = { path = "crates/gitbutler-patch-reference" }
 gitbutler-forge = { path = "crates/gitbutler-forge" }

crates/gitbutler-branch-actions/Cargo.toml

@@ -29,7 +29,6 @@ gitbutler-operating-modes.workspace = true
 gitbutler-cherry-pick.workspace = true
 gitbutler-oxidize.workspace = true
 gitbutler-stack.workspace = true
-gitbutler-stack-api.workspace = true
 gitbutler-patch-reference.workspace = true
 serde = { workspace = true, features = ["std"] }
 bstr.workspace = true

crates/gitbutler-branch-actions/src/base.rs

@@ -8,8 +8,8 @@ use gitbutler_project::FetchResult;
 use gitbutler_reference::{Refname, RemoteRefname};
 use gitbutler_repo::{LogUntil, RepositoryExt};
 use gitbutler_repo_actions::RepoActionsExt;
+use gitbutler_stack::StackExt;
 use gitbutler_stack::{BranchOwnershipClaims, Stack, Target, VirtualBranchesHandle};
-use gitbutler_stack_api::StackExt;
 use serde::Serialize;
 
 use crate::{

crates/gitbutler-branch-actions/src/branch_manager/branch_creation.rs

@@ -13,8 +13,8 @@ use gitbutler_repo::{
     LogUntil, RepositoryExt,
 };
 use gitbutler_repo_actions::RepoActionsExt;
+use gitbutler_stack::StackExt;
 use gitbutler_stack::{BranchOwnershipClaims, Stack, StackId};
-use gitbutler_stack_api::StackExt;
 use gitbutler_time::time::now_since_unix_epoch_ms;
 use tracing::instrument;
 

crates/gitbutler-branch-actions/src/branch_upstream_integration.rs

@@ -6,7 +6,7 @@ use gitbutler_repo::{
     LogUntil, RepositoryExt as _,
 };
 use gitbutler_stack::StackId;
-use gitbutler_stack_api::{commit_by_oid_or_change_id, StackExt};
+use gitbutler_stack::{commit_by_oid_or_change_id, StackExt};
 
 use crate::{
     branch_trees::{

crates/gitbutler-branch-actions/src/integration.rs

@@ -12,8 +12,8 @@ use gitbutler_operating_modes::OPEN_WORKSPACE_REFS;
 use gitbutler_project::access::WorktreeWritePermission;
 use gitbutler_repo::SignaturePurpose;
 use gitbutler_repo::{LogUntil, RepositoryExt};
+use gitbutler_stack::StackExt;
 use gitbutler_stack::{Stack, VirtualBranchesHandle};
-use gitbutler_stack_api::StackExt;
 use tracing::instrument;
 
 use crate::{branch_manager::BranchManagerExt, conflicts, VirtualBranchesExt};

crates/gitbutler-branch-actions/src/move_commits.rs

@@ -7,8 +7,8 @@ use gitbutler_command_context::CommandContext;
 use gitbutler_commit::commit_ext::CommitExt;
 use gitbutler_project::access::WorktreeWritePermission;
 use gitbutler_repo::{rebase::cherry_rebase_group, LogUntil, RepositoryExt};
+use gitbutler_stack::StackExt;
 use gitbutler_stack::{OwnershipClaim, StackId};
-use gitbutler_stack_api::StackExt;
 use std::collections::HashMap;
 
 /// moves commit from the branch it's in to the top of the target branch

crates/gitbutler-branch-actions/src/reorder_commits.rs

@@ -2,8 +2,8 @@ use anyhow::{bail, Context as _, Result};
 use gitbutler_command_context::CommandContext;
 use gitbutler_project::access::WorktreeWritePermission;
 use gitbutler_repo::{rebase::cherry_rebase_group, LogUntil, RepositoryExt as _};
+use gitbutler_stack::StackExt;
 use gitbutler_stack::StackId;
-use gitbutler_stack_api::StackExt;
 
 use crate::{
     branch_trees::{

crates/gitbutler-branch-actions/src/stack.rs

@@ -7,10 +7,8 @@ use gitbutler_commit::commit_ext::CommitExt;
 use gitbutler_patch_reference::{CommitOrChangeId, PatchReference};
 use gitbutler_project::Project;
 use gitbutler_repo_actions::RepoActionsExt;
+use gitbutler_stack::{commit_by_oid_or_change_id, CommitsForId, PatchReferenceUpdate, StackExt};
 use gitbutler_stack::{Stack, StackId, Target};
-use gitbutler_stack_api::{
-    commit_by_oid_or_change_id, CommitsForId, PatchReferenceUpdate, StackExt,
-};
 use serde::{Deserialize, Serialize};
 
 use crate::{

crates/gitbutler-branch-actions/src/status.rs

@@ -15,8 +15,8 @@ use gitbutler_command_context::CommandContext;
 use gitbutler_diff::{diff_files_into_hunks, GitHunk, Hunk, HunkHash};
 use gitbutler_operating_modes::assure_open_workspace_mode;
 use gitbutler_project::access::WorktreeWritePermission;
+use gitbutler_stack::StackExt;
 use gitbutler_stack::{BranchOwnershipClaims, OwnershipClaim, Stack, StackId};
-use gitbutler_stack_api::StackExt;
 use tracing::instrument;
 
 /// Represents the uncommitted status of the applied virtual branches in the workspace.

crates/gitbutler-branch-actions/src/undo_commit.rs

@@ -2,8 +2,8 @@ use anyhow::{bail, Context as _, Result};
 use gitbutler_command_context::CommandContext;
 use gitbutler_commit::commit_ext::CommitExt as _;
 use gitbutler_repo::{rebase::cherry_rebase_group, LogUntil, RepositoryExt as _};
+use gitbutler_stack::StackExt;
 use gitbutler_stack::{Stack, StackId};
-use gitbutler_stack_api::StackExt;
 
 use crate::VirtualBranchesExt as _;
 

crates/gitbutler-branch-actions/src/upstream_integration.rs

@@ -7,8 +7,8 @@ use gitbutler_repo::{
     LogUntil, RepositoryExt as _,
 };
 use gitbutler_repo_actions::RepoActionsExt as _;
+use gitbutler_stack::StackExt;
 use gitbutler_stack::{Stack, StackId, Target, VirtualBranchesHandle};
-use gitbutler_stack_api::StackExt;
 use serde::{Deserialize, Serialize};
 
 use crate::{

crates/gitbutler-branch-actions/src/virtual.rs

@@ -28,10 +28,10 @@ use gitbutler_repo::{
     LogUntil, RepositoryExt,
 };
 use gitbutler_repo_actions::RepoActionsExt;
+use gitbutler_stack::StackExt;
 use gitbutler_stack::{
     reconcile_claims, BranchOwnershipClaims, Stack, StackId, Target, VirtualBranchesHandle,
 };
-use gitbutler_stack_api::StackExt;
 use gitbutler_time::time::now_since_unix_epoch_ms;
 use serde::Serialize;
 use std::collections::HashSet;

crates/gitbutler-branch-actions/tests/extra/mod.rs

@@ -20,8 +20,8 @@ use gitbutler_branch_actions::{
 use gitbutler_commit::{commit_ext::CommitExt, commit_headers::CommitHeadersV2};
 use gitbutler_reference::{Refname, RemoteRefname};
 use gitbutler_repo::RepositoryExt;
+use gitbutler_stack::StackExt;
 use gitbutler_stack::{BranchOwnershipClaims, Target, VirtualBranchesHandle};
-use gitbutler_stack_api::StackExt;
 use gitbutler_testsupport::{commit_all, virtual_branches::set_test_target, Case, Suite};
 use pretty_assertions::assert_eq;
 

crates/gitbutler-edit-mode/Cargo.toml

@@ -22,5 +22,4 @@ gitbutler-oplog.workspace = true
 gitbutler-diff.workspace = true
 gitbutler-stack.workspace = true
 gitbutler-cherry-pick.workspace = true
-gitbutler-stack-api.workspace = true
 serde.workspace = true

crates/gitbutler-edit-mode/src/lib.rs

@@ -25,8 +25,8 @@ use gitbutler_project::access::{WorktreeReadPermission, WorktreeWritePermission}
 use gitbutler_reference::{ReferenceName, Refname};
 use gitbutler_repo::{rebase::cherry_rebase, RepositoryExt};
 use gitbutler_repo::{signature, SignaturePurpose};
+use gitbutler_stack::StackExt;
 use gitbutler_stack::{Stack, VirtualBranchesHandle};
-use gitbutler_stack_api::StackExt;
 use serde::Serialize;
 
 pub mod commands;