Skip to content

Add ToolActivatedEvent and ToolCancelEvent interfaces - #146

Closed
beaufortfrancois wants to merge 4 commits into
webmachinelearning:executeToolsfrom
beaufortfrancois:events
Closed

beaufortfrancois wants to merge 4 commits into
webmachinelearning:executeToolsfrom
beaufortfrancois:events

Conversation

@beaufortfrancois

@beaufortfrancois beaufortfrancois commented Mar 19, 2026 •

Copy link
Copy Markdown
Collaborator

Following #126 (comment), this PR adds new ToolActivatedEvent and ToolCancelEvent interfaces.


Preview | Diff

Comment thread index.bs
};

dictionary ToolCancelEventInit : EventInit {
DOMString toolName = "";

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking at this interface, I'm seeing a potential problem with multiple agents and concurrent activations of the same tool. Which one should be cancelled if all we know is the tool name? I'll comment on the issue.

@beaufortfrancois

Copy link
Copy Markdown
Collaborator Author

@domfarolino Now that #164 has been merged, I guess another one will follow so that I can plug this PR to tool execution. Is that right?

@beaufortfrancois

Copy link
Copy Markdown
Collaborator Author

FYI I've rebased this PR because of conflicts

@beaufortfrancois
beaufortfrancois changed the base branch from main to executeTools August 6, 2026 09:35
@beaufortfrancois
beaufortfrancois marked this pull request as ready for review August 6, 2026 09:51
@domfarolino
domfarolino deleted the branch webmachinelearning:executeTools August 14, 2026 15:06
@domfarolino

Copy link
Copy Markdown
Collaborator

It looks like this closed mistakenly, since it was based off of the executeTools branch which got deleted when #226 landed. Either myself of @beaufortfrancois will get this back up again, based on main.

@beaufortfrancois

Copy link
Copy Markdown
Collaborator Author

Gotcha. Here's #245

brave-builds pushed a commit to brave/chromium that referenced this pull request Sep 27, 2026
This is part of a transition to standardized event attributes;
ontoolcancel will follow shortly, eventually replacing WebMCPEvents.

Spec PR: webmachinelearning/webmcp#146

Bug: 489045948
Change-Id: Iffce26011963e6e501a7843e72a6ea4f6bea0f3a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7642055
Reviewed-by: Dominic Farolino <[email protected]>
Commit-Queue: Fr <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1705649}
brave-builds pushed a commit to brave/chromium that referenced this pull request Sep 27, 2026
ontoolactivated and ontoolcancel attributes now supersede WebMCPEvents,
which are slated for removal in a future CL.

Spec PR: webmachinelearning/webmcp#146


Bug: 489045948
Change-Id: I2fc110b7979baecc752663e7d9b2fd4ba51d4cbf
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7653555
Reviewed-by: Dominic Farolino <[email protected]>
Commit-Queue: Fr <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1705650}
chromium-wpt-export-bot pushed a commit to web-platform-tests/wpt that referenced this pull request Sep 27, 2026
This is part of a transition to standardized event attributes;
ontoolcancel will follow shortly, eventually replacing WebMCPEvents.

Spec PR: webmachinelearning/webmcp#146

Bug: 489045948
Change-Id: Iffce26011963e6e501a7843e72a6ea4f6bea0f3a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7642055
Reviewed-by: Dominic Farolino <[email protected]>
Commit-Queue: Fr <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1705649}
chromium-wpt-export-bot pushed a commit to web-platform-tests/wpt that referenced this pull request Sep 27, 2026
This is part of a transition to standardized event attributes;
ontoolcancel will follow shortly, eventually replacing WebMCPEvents.

Spec PR: webmachinelearning/webmcp#146

Bug: 489045948
Change-Id: Iffce26011963e6e501a7843e72a6ea4f6bea0f3a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7642055
Reviewed-by: Dominic Farolino <[email protected]>
Commit-Queue: Fr <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1705649}
chromium-wpt-export-bot pushed a commit to web-platform-tests/wpt that referenced this pull request Sep 27, 2026
ontoolactivated and ontoolcancel attributes now supersede WebMCPEvents,
which are slated for removal in a future CL.

Spec PR: webmachinelearning/webmcp#146

Bug: 489045948
Change-Id: I2fc110b7979baecc752663e7d9b2fd4ba51d4cbf
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7653555
Reviewed-by: Dominic Farolino <[email protected]>
Commit-Queue: Fr <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1705650}
lando-worker Bot pushed a commit to mozilla-firefox/firefox that referenced this pull request Oct 6, 2026
…lactivated attribute, a=testonly

Automatic update from web-platform-tests
[WebMCP] Add document.modelContext.ontoolactivated attribute

This is part of a transition to standardized event attributes;
ontoolcancel will follow shortly, eventually replacing WebMCPEvents.

Spec PR: webmachinelearning/webmcp#146

Bug: 489045948
Change-Id: Iffce26011963e6e501a7843e72a6ea4f6bea0f3a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7642055
Reviewed-by: Dominic Farolino <[email protected]>
Commit-Queue: Fr <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1705649}

--

wpt-commits: a9871a23b1f1eccc152d592db479b08690f3fed0
wpt-pr: 62965
lando-worker Bot pushed a commit to mozilla-firefox/firefox that referenced this pull request Oct 6, 2026
…lcancel attribute, a=testonly

Automatic update from web-platform-tests
[WebMCP] Add document.modelContext.ontoolcancel attribute

ontoolactivated and ontoolcancel attributes now supersede WebMCPEvents,
which are slated for removal in a future CL.

Spec PR: webmachinelearning/webmcp#146

Bug: 489045948
Change-Id: I2fc110b7979baecc752663e7d9b2fd4ba51d4cbf
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7653555
Reviewed-by: Dominic Farolino <[email protected]>
Commit-Queue: Fr <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1705650}

--

wpt-commits: 2d66b9b7998bb58c336138c178323ddee857b586
wpt-pr: 62995
zerox80 pushed a commit to zerox80/firefox that referenced this pull request Oct 16, 2026
…lactivated attribute, a=testonly

Automatic update from web-platform-tests
[WebMCP] Add document.modelContext.ontoolactivated attribute

This is part of a transition to standardized event attributes;
ontoolcancel will follow shortly, eventually replacing WebMCPEvents.

Spec PR: webmachinelearning/webmcp#146

Bug: 489045948
Change-Id: Iffce26011963e6e501a7843e72a6ea4f6bea0f3a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7642055
Reviewed-by: Dominic Farolino <[email protected]>
Commit-Queue: Fr <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1705649}

--

wpt-commits: a9871a23b1f1eccc152d592db479b08690f3fed0
wpt-pr: 62965
zerox80 pushed a commit to zerox80/firefox that referenced this pull request Oct 16, 2026
…lcancel attribute, a=testonly

Automatic update from web-platform-tests
[WebMCP] Add document.modelContext.ontoolcancel attribute

ontoolactivated and ontoolcancel attributes now supersede WebMCPEvents,
which are slated for removal in a future CL.

Spec PR: webmachinelearning/webmcp#146

Bug: 489045948
Change-Id: I2fc110b7979baecc752663e7d9b2fd4ba51d4cbf
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7653555
Reviewed-by: Dominic Farolino <[email protected]>
Commit-Queue: Fr <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1705650}

--

wpt-commits: 2d66b9b7998bb58c336138c178323ddee857b586
wpt-pr: 62995
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.

3 participants