Repository navigation
feat(mcp): automate removal follow-ups through local agent tools - #167
Merged
Merged
Conversation
…rd-only - Batch previews are stored with a digest. Approval happens on the Automation page in the web UI or with eraser mcp approve, once and for 15 minutes. begin_batch spends that approval. - eraser mcp pending lists previews waiting for approval. - Brokers on shared privacy or form platforms may use only the exact tenant host in their catalog entry. Other brokers keep subdomain matching. - A response from a non-public address stops further actions on the page. - Automated pipeline updates only move a request forward. Web Confirm and Reject use an explicit override. - Document approval, platform hosts and the experimental status of eraser mcp.
|
Codecov Report❌ Patch coverage is 📢 Thoughts on this report? Let us know! |
… review fixes - retry_action needs a person's approval in Eraser, like begin_batch. - Page loads on shared platforms stay under the catalog URL's form path. - Observe refuses a page Chrome loaded from a non-public address. - eraser mcp approve needs a terminal; the web approval needs a same-origin browser request. - Approval times are stored and compared in UTC. - The CSP connect-src follows the exact-host rule. - A failed open closes its page and frees the slot. - Config and the broker list are loaded once per task call. - Automation screenshots keep quality 100 (PNG); existing flows stay at 90. - Instructions, tool descriptions, CLI help and docs describe the new approval flow.
…aser mcp experimental - Page and request hosts are the catalog's hosts and their www. form. Subdomains are not allowed, so ExactHosts and the wildcard CSP sources go away. The shared-platform list now only decides where page loads are pinned to the form path. - BatchApproval loses ApprovedAt and ApprovedBy, which nothing read. - eraser mcp help, its startup line and docs/mcp.md say it is experimental.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Picks up magnus188's #159 and applies the review asks from 2026-10-13 that gate merging. This is PR A of two. Actor attribution, the docs map and the receipt-detection note come in a follow-up PR.
Changes on top of his commit:
prepare_batchstores the preview and its digest inhistory.db. Approval happens on the new Automation page in the web UI, or witheraser mcp approve <plan_id>. Each approval works once and expires after 15 minutes.begin_batchspends it.eraser mcp pendinglists waiting previews.ActionItemscounters, with the MCP review predicate onPendingTask.eraser mcpis marked experimental indocs/stability.md. The approval flow and platform rule are indocs/mcp.md.Rebased onto
mainafter #165. Tests,-raceand golangci-lint pass locally.