Skip to content

Commit c8e6599

Browse files
Cover line-separator baseURL whitespace validation
Co-authored-by: Eric Allam <eric@trigger.dev>
1 parent e1ada23 commit c8e6599

File tree

3 files changed

+24
-0
lines changed

3 files changed

+24
-0
lines changed

docs/tasks/streams.mdx

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -682,6 +682,7 @@ Examples:
682682
-`https://api.trigger.dev/\rinternal`
683683
-`https://api.trigger.dev/\u200Binternal`
684684
-`https://api.trigger.dev/\u202Finternal`
685+
-`https://api.trigger.dev/\u2028internal`
685686
-`https://api.trigger.dev/\uFEFFinternal`
686687

687688
Validation errors use these exact messages:

packages/ai/README.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -189,6 +189,7 @@ Examples:
189189
-`https://api.trigger.dev/\rinternal` (internal carriage-return characters)
190190
-`https://api.trigger.dev/\u200Binternal` (internal zero-width-space characters)
191191
-`https://api.trigger.dev/\u202Finternal` (internal narrow no-break space characters)
192+
-`https://api.trigger.dev/\u2028internal` (internal line-separator characters)
192193
-`https://api.trigger.dev/\uFEFFinternal` (internal BOM characters)
193194

194195
Validation errors use these exact messages:

packages/ai/src/chatTransport.test.ts

Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -786,6 +786,17 @@ describe("TriggerChatTransport", function () {
786786
}).toThrowError("baseURL must not contain internal whitespace characters");
787787
});
788788

789+
it("throws when baseURL contains internal line-separator characters", function () {
790+
expect(function () {
791+
new TriggerChatTransport({
792+
task: "chat-task",
793+
accessToken: "pk_trigger",
794+
baseURL: "https://api.trigger.dev/\u2028internal",
795+
stream: "chat-stream",
796+
});
797+
}).toThrowError("baseURL must not contain internal whitespace characters");
798+
});
799+
789800
it("throws when baseURL contains internal carriage-return characters", function () {
790801
expect(function () {
791802
new TriggerChatTransport({
@@ -3481,6 +3492,17 @@ describe("TriggerChatTransport", function () {
34813492
}).toThrowError("baseURL must not contain internal whitespace characters");
34823493
});
34833494

3495+
it("throws from factory when baseURL contains internal line-separator characters", function () {
3496+
expect(function () {
3497+
createTriggerChatTransport({
3498+
task: "chat-task",
3499+
accessToken: "pk_trigger",
3500+
baseURL: "https://api.trigger.dev/\u2028internal",
3501+
stream: "chat-stream",
3502+
});
3503+
}).toThrowError("baseURL must not contain internal whitespace characters");
3504+
});
3505+
34843506
it("throws from factory when baseURL contains internal carriage-return characters", function () {
34853507
expect(function () {
34863508
createTriggerChatTransport({

0 commit comments

Comments
 (0)