Skip to content

Reject registerTool() promise when signal is aborted #202

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
beaufortfrancois wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Reject registerTool() promise when signal is aborted #202

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
7f4c440
Reject registerTool promise if signal is aborted
beaufortfrancois Jun 9, 2026
8b46545
Update abort algorithm
beaufortfrancois Jun 9, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 9 additions & 7 deletions index.bs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -395,16 +395,20 @@ The <dfn method for=ModelContext>registerTool(<var>tool</var>, <var>options</var
1. Let |untrusted content hint| be true if |tool|'s {{ModelContextTool/annotations}} [=map/exists=] and
its {{ToolAnnotations/untrustedContentHint}} is true. Otherwise, let it be false.

1. Let |promise| be [=a new promise=] created in [=this=]'s [=relevant realm=].

1. Let |signal| be |options|'s {{ModelContextRegisterToolOptions/signal}}.

1. If |signal| [=map/exists=], then:

1. If |signal| is [=AbortSignal/aborted=], then optionally [=report a warning to the console=]
indicating that the tool was not registered because the {{AbortSignal}} was already
[=AbortSignal/aborted=], and return.
1. If |signal| is [=AbortSignal/aborted=], then return [=a promise rejected with=]
|signal|'s [=AbortSignal/abort reason=].

1. [=AbortSignal/add|Add the following abort steps=] to |signal|:

1. [=AbortSignal/add|Add an abort algorithm=] to |signal| that [=model context/unregisters a
tool=] given [=this=] and |tool name|.
1. [=model context/Unregister a tool=] given [=this=] and |tool name|.

1. [=Reject=] |promise| with |signal|'s [=AbortSignal/abort reason=].

1. Let |exposed origins| be an empty [=list=] of [=origins=].

Expand Down Expand Up @@ -448,8 +452,6 @@ The <dfn method for=ModelContext>registerTool(<var>tool</var>, <var>options</var

1. Set [=this=]'s [=ModelContext/internal context=][|tool name|] to |tool definition|.

1. Let |promise| be [=a new promise=] created in [=this=]'s [=relevant realm=].

1. Run the following steps [=in parallel=]:

1. [=Notify documents of a tool change=] given |tool owner| and |exposed origins|.
Expand Down
Loading