Skip to content

chore: keep only web and desktop apps#5135

Closed
ComputelessComputer wants to merge 1 commit into
mainfrom
chore/keep-web-desktop-apps
Closed

chore: keep only web and desktop apps#5135
ComputelessComputer wants to merge 1 commit into
mainfrom
chore/keep-web-desktop-apps

Conversation

@ComputelessComputer

@ComputelessComputer ComputelessComputer commented Apr 26, 2026

Copy link
Copy Markdown
Collaborator

Remove extra app workspaces and clean related workspace metadata.


Note

Medium Risk
Medium risk because it reshapes the Cargo workspace and removes entire app crates, which can break builds/CI or downstream tooling that still expects those packages to exist.

Overview
Removes the apps/api, apps/claw, and apps/cli workspaces (and deletes the apps/api crate plus its deployment/config files), leaving the workspace focused on desktop + shared crates/plugins.

Cleans up developer tooling by dropping related Taskfile tasks (e.g., Stripe/CLI dev helpers) and heavily pruning Cargo.lock, removing now-unreferenced dependencies and normalizing versions after the workspace reduction.

Reviewed by Cursor Bugbot for commit 37bfd72. Bugbot is set up for automated code reviews on this repo. Configure here.

Remove extra app workspaces and clean related workspace metadata.
@netlify

netlify Bot commented Apr 26, 2026

Copy link
Copy Markdown

Deploy Preview for hyprnote failed.

Name Link
🔨 Latest commit 37bfd72
🔍 Latest deploy log https://app.netlify.com/projects/hyprnote/deploys/69edd19e9b9eea0008fe9605

@netlify

netlify Bot commented Apr 26, 2026

Copy link
Copy Markdown

Deploy Preview for unsigned-char failed.

Name Link
🔨 Latest commit 37bfd72
🔍 Latest deploy log https://app.netlify.com/projects/unsigned-char/deploys/69edd19ec4da6f0008879ca6

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 37bfd72. Configure here.

Comment thread Cargo.toml
members = [
"apps/api",
"apps/claw",
"apps/cli",

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removing workspace members breaks active CI workflows

High Severity

Removing apps/api, apps/claw, and apps/cli from the workspace breaks several still-present GitHub Actions workflows. Critically, .github/workflows/cli_desktop_ci.yml and .github/workflows/cli_standalone_ci.yml trigger on changes to crates/**, Cargo.toml, and Cargo.lock — meaning virtually every Rust PR will trigger these workflows, which will fail on cargo check -p cli. Similarly, .github/workflows/api_ci.yaml triggers on changes to crates/llm-proxy/** and crates/transcribe-proxy/** and runs cargo check -p api. These workflows need to be deleted or updated.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 37bfd72. Configure here.

@ComputelessComputer ComputelessComputer deleted the chore/keep-web-desktop-apps branch April 26, 2026 12:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant