github-starred/yaak
2
0
Fork 0
mirror of https://github.com/mountain-loop/yaak.git synced 2026-05-07 02:21:59 -05:00
Code Issues 71 Packages Projects Releases 101 Wiki Activity

[PR #398] [MERGED] CLI send enhancements and shared plugin event routing #1588

New Issue
Closed
opened 2026-05-06 19:37:57 -05:00 by GiteaMirror · 0 comments
GiteaMirror commented 2026-05-06 19:37:57 -05:00
Owner
Copy Link

📋 Pull Request Information

Original PR: https://github.com/mountain-loop/yaak/pull/398
Author: @gschier
Created: 2/17/2026
Status: ✅ Merged
Merged: 2/20/2026
Merged by: @gschier

Base: main ← Head: cli-improvements-2

📝 Commits (4)

  • a4f96fc Implement CLI send flows and refactor plugin event handling
  • 39fc9e8 Refine shared plugin event routing API
  • f71a3ea Remove completed CLI plan doc
  • 487e66f Use workspace dependency for schemars

📊 Changes

30 files changed (+1563 additions, -583 deletions)

View changed files

📝 Cargo.lock (+22 -0)
📝 Cargo.toml (+1 -0)
📝 crates-cli/yaak-cli/Cargo.toml (+2 -0)
➖ crates-cli/yaak-cli/PLAN.md (+0 -340)
📝 crates-cli/yaak-cli/src/cli.rs (+30 -5)
📝 crates-cli/yaak-cli/src/commands/environment.rs (+7 -7)
📝 crates-cli/yaak-cli/src/commands/folder.rs (+16 -14)
📝 crates-cli/yaak-cli/src/commands/request.rs (+263 -11)
📝 crates-cli/yaak-cli/src/commands/send.rs (+167 -1)
📝 crates-cli/yaak-cli/src/commands/workspace.rs (+5 -3)
📝 crates-cli/yaak-cli/src/context.rs (+20 -1)
📝 crates-cli/yaak-cli/src/main.rs (+4 -1)
➕ crates-cli/yaak-cli/src/plugin_events.rs (+212 -0)
📝 crates-cli/yaak-cli/src/utils/json.rs (+6 -9)
📝 crates-cli/yaak-cli/tests/common/mod.rs (+45 -1)
📝 crates-cli/yaak-cli/tests/request_commands.rs (+48 -3)
➕ crates-cli/yaak-cli/tests/send_commands.rs (+81 -0)
📝 crates-tauri/yaak-app/src/plugin_events.rs (+152 -166)
📝 crates/yaak-models/Cargo.toml (+1 -0)
📝 crates/yaak-models/src/models.rs (+6 -5)

...and 10 more files

📄 Description

Summary

  • add and wire a dedicated CLI plugin event bridge with key/value support and explicit unsupported-request errors
  • add top-level CLI send flow improvements and coverage for workspace/folder send behavior
  • refactor plugin event routing through shared grouping (Shared/Host/Ignore) and move non-request events out of app-side Ignore handling
  • rename workspace-list plugin event to list_open_workspaces_* and update runtime/bindings

Validation

  • cargo test -p yaak
  • cargo test -p yaak-cli
  • cargo check -p yaak-app
  • npm run -w packages/plugin-runtime-types build --if-present
  • npm run -w packages/plugin-runtime build --if-present

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/mountain-loop/yaak/pull/398 **Author:** [@gschier](https://github.com/gschier) **Created:** 2/17/2026 **Status:** ✅ Merged **Merged:** 2/20/2026 **Merged by:** [@gschier](https://github.com/gschier) **Base:** `main` ← **Head:** `cli-improvements-2` --- ### 📝 Commits (4) - [`a4f96fc`](https://github.com/mountain-loop/yaak/commit/a4f96fca1121b3cb4b21dc1a3b51f631c0f0e052) Implement CLI send flows and refactor plugin event handling - [`39fc9e8`](https://github.com/mountain-loop/yaak/commit/39fc9e81cd2d3323c84d01670a6e152ad39734df) Refine shared plugin event routing API - [`f71a3ea`](https://github.com/mountain-loop/yaak/commit/f71a3ea8fee585342b05d814c861ecc01921f783) Remove completed CLI plan doc - [`487e66f`](https://github.com/mountain-loop/yaak/commit/487e66faa4cb7f1ccf20cc0a9826d1ac0a0d15ba) Use workspace dependency for schemars ### 📊 Changes **30 files changed** (+1563 additions, -583 deletions) <details> <summary>View changed files</summary> 📝 `Cargo.lock` (+22 -0) 📝 `Cargo.toml` (+1 -0) 📝 `crates-cli/yaak-cli/Cargo.toml` (+2 -0) ➖ `crates-cli/yaak-cli/PLAN.md` (+0 -340) 📝 `crates-cli/yaak-cli/src/cli.rs` (+30 -5) 📝 `crates-cli/yaak-cli/src/commands/environment.rs` (+7 -7) 📝 `crates-cli/yaak-cli/src/commands/folder.rs` (+16 -14) 📝 `crates-cli/yaak-cli/src/commands/request.rs` (+263 -11) 📝 `crates-cli/yaak-cli/src/commands/send.rs` (+167 -1) 📝 `crates-cli/yaak-cli/src/commands/workspace.rs` (+5 -3) 📝 `crates-cli/yaak-cli/src/context.rs` (+20 -1) 📝 `crates-cli/yaak-cli/src/main.rs` (+4 -1) ➕ `crates-cli/yaak-cli/src/plugin_events.rs` (+212 -0) 📝 `crates-cli/yaak-cli/src/utils/json.rs` (+6 -9) 📝 `crates-cli/yaak-cli/tests/common/mod.rs` (+45 -1) 📝 `crates-cli/yaak-cli/tests/request_commands.rs` (+48 -3) ➕ `crates-cli/yaak-cli/tests/send_commands.rs` (+81 -0) 📝 `crates-tauri/yaak-app/src/plugin_events.rs` (+152 -166) 📝 `crates/yaak-models/Cargo.toml` (+1 -0) 📝 `crates/yaak-models/src/models.rs` (+6 -5) _...and 10 more files_ </details> ### 📄 Description ## Summary - add and wire a dedicated CLI plugin event bridge with key/value support and explicit unsupported-request errors - add top-level CLI send flow improvements and coverage for workspace/folder send behavior - refactor plugin event routing through shared grouping (`Shared`/`Host`/`Ignore`) and move non-request events out of app-side `Ignore` handling - rename workspace-list plugin event to `list_open_workspaces_*` and update runtime/bindings ## Validation - cargo test -p yaak - cargo test -p yaak-cli - cargo check -p yaak-app - npm run -w packages/plugin-runtime-types build --if-present - npm run -w packages/plugin-runtime build --if-present --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-05-06 19:37:57 -05:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
pull-request
Mirrored from GitHub Pull Request
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
GiteaMirror ninjasurge
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/yaak#1588
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?