Skip to content

Commit 7924efb

Browse files
Cover additional unicode spacing baseURL rejections
Co-authored-by: Eric Allam <eric@trigger.dev>
1 parent 25f4cde commit 7924efb

File tree

3 files changed

+48
-0
lines changed

3 files changed

+48
-0
lines changed

docs/tasks/streams.mdx

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -694,7 +694,9 @@ Examples:
694694
-`https://api.trigger.dev/\u200Cinternal`
695695
-`https://api.trigger.dev/\u200Dinternal`
696696
-`https://api.trigger.dev/\u1680internal`
697+
-`https://api.trigger.dev/\u2007internal`
697698
-`https://api.trigger.dev/\u202Finternal`
699+
-`https://api.trigger.dev/\u205Finternal`
698700
-`https://api.trigger.dev/\u3000internal`
699701
-`https://api.trigger.dev/\u2028internal`
700702
-`https://api.trigger.dev/\u2029internal`

packages/ai/README.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -201,7 +201,9 @@ Examples:
201201
-`https://api.trigger.dev/\u200Cinternal` (internal zero-width-non-joiner characters)
202202
-`https://api.trigger.dev/\u200Dinternal` (internal zero-width-joiner characters)
203203
-`https://api.trigger.dev/\u1680internal` (internal ogham-space-mark characters)
204+
-`https://api.trigger.dev/\u2007internal` (internal figure-space characters)
204205
-`https://api.trigger.dev/\u202Finternal` (internal narrow no-break space characters)
206+
-`https://api.trigger.dev/\u205Finternal` (internal medium-mathematical-space characters)
205207
-`https://api.trigger.dev/\u3000internal` (internal ideographic-space characters)
206208
-`https://api.trigger.dev/\u2028internal` (internal line-separator characters)
207209
-`https://api.trigger.dev/\u2029internal` (internal paragraph-separator characters)

packages/ai/src/chatTransport.test.ts

Lines changed: 44 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -841,6 +841,28 @@ describe("TriggerChatTransport", function () {
841841
}).toThrowError("baseURL must not contain internal whitespace characters");
842842
});
843843

844+
it("throws when baseURL contains internal figure-space characters", function () {
845+
expect(function () {
846+
new TriggerChatTransport({
847+
task: "chat-task",
848+
accessToken: "pk_trigger",
849+
baseURL: "https://api.trigger.dev/\u2007internal",
850+
stream: "chat-stream",
851+
});
852+
}).toThrowError("baseURL must not contain internal whitespace characters");
853+
});
854+
855+
it("throws when baseURL contains internal medium-mathematical-space characters", function () {
856+
expect(function () {
857+
new TriggerChatTransport({
858+
task: "chat-task",
859+
accessToken: "pk_trigger",
860+
baseURL: "https://api.trigger.dev/\u205Finternal",
861+
stream: "chat-stream",
862+
});
863+
}).toThrowError("baseURL must not contain internal whitespace characters");
864+
});
865+
844866
it("throws when baseURL contains internal line-separator characters", function () {
845867
expect(function () {
846868
new TriggerChatTransport({
@@ -3745,6 +3767,28 @@ describe("TriggerChatTransport", function () {
37453767
}).toThrowError("baseURL must not contain internal whitespace characters");
37463768
});
37473769

3770+
it("throws from factory when baseURL contains internal figure-space characters", function () {
3771+
expect(function () {
3772+
createTriggerChatTransport({
3773+
task: "chat-task",
3774+
accessToken: "pk_trigger",
3775+
baseURL: "https://api.trigger.dev/\u2007internal",
3776+
stream: "chat-stream",
3777+
});
3778+
}).toThrowError("baseURL must not contain internal whitespace characters");
3779+
});
3780+
3781+
it("throws from factory when baseURL contains internal medium-mathematical-space characters", function () {
3782+
expect(function () {
3783+
createTriggerChatTransport({
3784+
task: "chat-task",
3785+
accessToken: "pk_trigger",
3786+
baseURL: "https://api.trigger.dev/\u205Finternal",
3787+
stream: "chat-stream",
3788+
});
3789+
}).toThrowError("baseURL must not contain internal whitespace characters");
3790+
});
3791+
37483792
it("throws from factory when baseURL contains internal line-separator characters", function () {
37493793
expect(function () {
37503794
createTriggerChatTransport({

0 commit comments

Comments
 (0)