Implement complete mcp server detection and quarantine read/writer daemon#2
Conversation
Adding the `armed` param pushed enroll to 8 args. It's a flat list of enrollment inputs (matching reconcile_once, which already allows this); grouping into a struct isn't worth it. Allow the lint. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Qodo reviews are paused for this user.Troubleshooting steps vary by plan Learn more → On a Teams plan? Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center? |
The workspace wasn't formatted with stock `cargo fmt`, so the CI `rustfmt` job (cargo fmt --all -- --check on @stable) was failing. Apply canonical formatting across all crates. No behavior change. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
Whoosh that's a lot of changes. Let me see if I can build and run this + the desktop app and I think that's good enough for me to approve |
It can wait no worries. I did not test E2E really yet after pushing some of the last fixes on desktop and I am also preparing the Windows part which has some work. |
|
Nice, all works and builds. A few findings:
Anyways, looks good. Both are good to merge. |
IliaMManolov
left a comment
IliaMManolov
left a comment
There was a problem hiding this comment.
See previous comment (dunno why I can't approve from comment instead of having to go to the review pane)
2 participants
No description provided.