Skip to content

test(mv3): require bounded bookmark mutation diagnostics - #178

Draft
cursor[bot] wants to merge 18 commits into
test/mv3-downloadsfrom
cursor/bc-62f8754b-f579-4115-9089-cace87f78155-206a
Draft

test(mv3): require bounded bookmark mutation diagnostics#178
cursor[bot] wants to merge 18 commits into
test/mv3-downloadsfrom
cursor/bc-62f8754b-f579-4115-9089-cace87f78155-206a

Conversation

@cursor

@cursor cursor Bot commented Aug 16, 2026

Copy link
Copy Markdown

Close the buyer-visible gap on the Manifest V3 bookmark-mutation lane: a bare bookmarks: missing result did not identify which controlled create/get/remove stage failed.

Implemented boundary

Exact current head is 2ac31311ac4cd6611c0d4ded36f6b619d02a4dba.

  • The controlled fixture returns only allow-listed bookmark stage tokens (bookmark-source-rejected, bookmark-create-rejected, bookmark-get-missing, exact mismatch tokens, bookmark-remove-rejected, and bookmark-complete-ready) and still removes the synthetic bookmark in finally cleanup.
  • The compatibility runner retains only reviewed tokens, normalizes all other values to unexpected, and does not retain raw Chrome exception text, fixture titles, bookmark IDs, or bookmark URLs as diagnostic evidence.
  • Doctoring records the current Chrome Extensions chrome.bookmarks primary reference in APA 7th form. This remains compatibility evidence only and is not an OriginWeave Agent bookmark grant.

Repaired stack

PR #56 exact head 71420d03483ca8d4377f6712fe54cd9a9f6ab635 is the lifecycle predecessor. This branch now has a non-destructive second-parent ancestry edge to that exact head while preserving the diagnostic tree. Comparing #56 → this head reports ahead / behind 0, with merge base exactly #56.

The PR base metadata still names the downloads branch (#43) because GitHub has not accepted a safe metadata retarget. Source ancestry is therefore the authoritative dependency relation. Integrate in semantic order: #43#56 bookmark mutation → this diagnostic close → #59 history mutation. PR #59 exact head 7992e13aea34c4784ba96c806e0be1d2c18a33e4 already descends from this head and independently revalidated its combined tree.

Keep this PR Draft while the prerequisite stack remains active. No predecessor check, review, status, or approval transfers across heads.

Exact-current verification

On unchanged exact head 2ac31311ac4cd6611c0d4ded36f6b619d02a4dba:

  • CI run 31964838767: success;
  • Manifest V3 Compatibility run 31964838750: success on pinned Chromium; and
  • GitHub freshly reports the Draft mergeable.

Feature-base organization security/review workflows that did not execute on this exact head are not represented as passing. The requested non-author reviewer remains seonghobae; no approval is inferred from that request.

Scope boundary

Do not interpret passing bookmark compatibility as Agent bookmark/history/filesystem authority, access to an ambient human profile, or a general bookmark tool. This scheduled actor does not merge, self-approve, alter workflows, add secrets, weaken checks, tag, or publish.

Open in Web View Automation 

seonghobae and others added 14 commits August 11, 2026 04:04
Bookmark create/get/remove failures previously collapsed to ready/missing,
so operators could not tell source rejection from leftover residue. Require
allow-listed stage tokens, sanitize raw Chrome errors, and record the
current chrome.bookmarks APA citation as compatibility evidence only.

Co-authored-by: Seongho Bae <[email protected]>
…lures

Keep the bookmark stage-token close on the current downloads prerequisite
after #43 classified WebDriver HTTP/1.1 parser failures. Preserve both
allow-listed bookmark diagnostics and the RFC 9112 transport token.

Co-authored-by: Seongho Bae <[email protected]>
@cursor
cursor Bot requested a review from seonghobae August 16, 2026 16:50
Preserve the bookmark diagnostic stage-token work while importing the exact current #43 browser-version diagnostic hardening, regression contract, and changelog evidence. This is a non-destructive stack alignment; prerequisite checks and reviews do not transfer.
Preserve the current bookmark diagnostic tree while adding exact ancestry to PR #56 head 71420d0. The diagnostic branch already contains the bookmark mutation lifecycle plus the same hardened downloads-root changes; this non-destructive merge repairs stack topology without transferring predecessor checks or reviews.
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.

2 participants