Add provider-neutral agent feedback workflow - #22
Draft
angusbezzina wants to merge 6 commits into
Draft
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
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
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.
Margin comments can now drive coding-agent edits without a generated inbox or provider integration. A user marks an unresolved thread For agent, then copies one/selected/all comments or lets any shell-capable host read the same local source context through
attn feedbackandattn feedback --watch.The implementation adds:
attn feedback [PATH];attn feedback --watch [PATH]with complete snapshots, upserts, removals, resets, disconnect errors, and closed-pipe cleanup;Agents edit the original source with their existing permissions. Reading, copying, and watching leave the review thread open and never run a model or post a reply. Native marks are available to the local CLI; browser-only marks remain private to that browser. Cross-device mark sync, browser-to-local pairing, and private comments on unshared documents remain outside this change.
Validation:
cargo fmt --all -- --checkcargo test --workspace --locked --quiet— all Rust suites passed, including feedback CLI subprocess and review convergence testsnpm run checkinweb/— 0 errors and 0 warningsnpm testinweb/— all 144 test files passednpm run buildandnpm run build:browserinweb/— passed; existing mixed-import and large-chunk warnings remainnode --check examples/feedback-watch.mjssource_changed; provider-free watch delivery measured 209.4 msImplementation evidence:
planning/collab/agent-feedback-implementation-evidence.md. Beads epicattn-aysuand all 24 leaf tasks are closed and pushed.