@atproto/syntax 0.5.2 → 0.5.4

package/tests/aturi-string.test.ts

@@ -0,0 +1,211 @@
+import { readFileSync } from 'node:fs'
+import { describe, expect, test } from 'vitest'
+import { InvalidAtUriError, assertAtUriString, isAtUriString } from '../src'
+
+describe('valid interop', () => {
+  for (const value of readLines(
+    `${__dirname}/../../../interop-test-files/syntax/aturi_syntax_valid.txt`,
+  )) {
+    testValid(value)
+  }
+})
+
+describe('invalid interop', () => {
+  for (const value of readLines(
+    `${__dirname}/../../../interop-test-files/syntax/aturi_syntax_invalid.txt`,
+  )) {
+    testInvalid(value)
+  }
+})
+
+describe('custom cases', () => {
+  describe('valid spec basics', () => {
+    testValid('at://did:plc:asdf123')
+    testValid('at://user.bsky.social')
+    testValid('at://did:plc:asdf123/com.atproto.feed.post')
+    testValid('at://did:plc:asdf123/com.atproto.feed.post/record')
+
+    testValid('at://did:plc:asdf123#/frag')
+    testValid('at://user.bsky.social#/frag')
+    testValid('at://did:plc:asdf123/com.atproto.feed.post#/frag')
+    testValid('at://did:plc:asdf123/com.atproto.feed.post/record#/frag')
+  })
+
+  describe('invalid spec basics', () => {
+    testInvalid('a://did:plc:asdf123')
+    testInvalid('at//did:plc:asdf123')
+    testInvalid('at:/a/did:plc:asdf123')
+    testInvalid('at:/did:plc:asdf123')
+    testInvalid('AT://did:plc:asdf123')
+    testInvalid('http://did:plc:asdf123')
+    testInvalid('://did:plc:asdf123')
+    testInvalid('at:did:plc:asdf123')
+    testInvalid('at:/did:plc:asdf123')
+    testInvalid('at:///did:plc:asdf123')
+    testInvalid('at://:/did:plc:asdf123')
+    testInvalid('at:/ /did:plc:asdf123')
+    testInvalid('at://did:plc:asdf123 ')
+    testInvalid('at://did:plc:asdf123/ ')
+    testInvalid(' at://did:plc:asdf123')
+    testInvalid('at://did:plc:asdf123/com.atproto.feed.post ')
+    testInvalid('at://did:plc:asdf123/com.atproto.feed.post# ')
+    testInvalid('at://did:plc:asdf123/com.atproto.feed.post#/ ')
+    testInvalid('at://did:plc:asdf123/com.atproto.feed.post#/frag ')
+    testInvalid('at://did:plc:asdf123/com.atproto.feed.post#fr ag')
+    testInvalid('//did:plc:asdf123')
+    testInvalid('at://name')
+    testInvalid('at://name.0')
+    testInvalid('at://diD:plc:asdf123')
+    testInvalid('at://did:plc:asdf123/com.atproto.feed.p@st')
+    testInvalid('at://did:plc:asdf123/com.atproto.feed.p$st')
+    testInvalid('at://did:plc:asdf123/com.atproto.feed.p%st')
+    testInvalid('at://did:plc:asdf123/com.atproto.feed.p&st')
+    testInvalid('at://did:plc:asdf123/com.atproto.feed.p()t')
+    testInvalid('at://did:plc:asdf123/com.atproto.feed_post')
+    testInvalid('at://did:plc:asdf123/-com.atproto.feed.post')
+    testInvalid('at://did:plc:asdf@123/com.atproto.feed.post')
+
+    testInvalid('at://did:plc:asdf123?a')
+    testInvalid('at://user.bsky.social?a=B')
+    testInvalid('at://did:plc:asdf123/com.atproto.feed.post?foo=bar')
+    testInvalid('at://did:plc:asdf123/com.atproto.feed.post/record?q=3')
+
+    testInvalid('at://did:plc:asdf123?a=b#/frag')
+    testInvalid('at://user.bsky.social?a=b#/frag')
+    testInvalid('at://did:plc:asdf123/com.atproto.feed.post?a=b#/frag')
+    testInvalid('at://did:plc:asdf123/com.atproto.feed.post/record?a=b#/frag')
+
+    testInvalid('at://DID:plc:asdf123')
+    testInvalid('at://user.bsky.123')
+    testInvalid('at://bsky')
+    testInvalid('at://did:plc:')
+    testInvalid('at://did:plc:')
+    testInvalid('at://frag')
+  })
+
+  describe('very long strings', () => {
+    testValid('at://did:plc:asdf123/com.atproto.feed.post/' + 'o'.repeat(512))
+    testValid(`at://did:web:x${'.y'.repeat(100)}/com.atproto.feed.post/record`)
+    testInvalid(`at://did:plc:${'o'.repeat(8200)}/com.atproto.feed.post/record`)
+    testInvalid('at://did:plc:asdf123/com.atproto.feed.post/' + 'o'.repeat(513))
+    testInvalid('at://did:plc:asdf123/com.atproto.feed.post/' + 'o'.repeat(800))
+  })
+
+  describe('invalid collection', () => {
+    testInvalid('at://did:plc:asdf123/short/stuff')
+    testInvalid('at://did:plc:asdf123/12345')
+  })
+
+  describe('invalid repeated slashes', () => {
+    testInvalid('at://user.bsky.social//')
+    testInvalid('at://user.bsky.social//com.atproto.feed.post')
+    testInvalid('at://user.bsky.social/com.atproto.feed.post//')
+  })
+
+  describe('invalid trailing slashes', () => {
+    testInvalid('at://did:plc:asdf123/')
+    testInvalid('at://user.bsky.social/')
+    testInvalid('at://did:plc:asdf123/com.atproto.feed.post/')
+    testInvalid('at://did:plc:asdf123/com.atproto.feed.post/record/')
+    testInvalid('at://did:plc:asdf123/com.atproto.feed.post/record/#/frag')
+  })
+
+  describe('invalid segment count', () => {
+    testInvalid('at://did:plc:asdf123/com.atproto.feed.post/asdf123/asdf')
+    testInvalid('at://did:plc:asdf123/com.atproto.feed.post/asdf123/more/more')
+  })
+
+  describe('valid record key', () => {
+    testValid('at://did:plc:asdf123/com.atproto.feed.post/a')
+    testValid('at://did:plc:asdf123/com.atproto.feed.post/asdf123')
+  })
+
+  describe('loosely valid trailing slash', () => {
+    testLoose('at://did:plc:asdf123/')
+    testLoose('at://user.bsky.social/')
+    testLoose('at://did:plc:asdf123/com.atproto.feed.post/')
+    testLoose('at://did:plc:asdf123/com.atproto.feed.post/record/')
+    testLoose('at://did:plc:asdf123/com.atproto.feed.post/record/#/frag')
+  })
+
+  describe('loosely valid record keys', () => {
+    testLoose('at://did:plc:asdf123/com.atproto.feed.post/%23')
+
+    testLoose('at://did:plc:asdf123/com.atproto.feed.post/$@!*)(:,;~.sdf123')
+    testLoose("at://did:plc:asdf123/com.atproto.feed.post/~'sdf123")
+
+    testLoose('at://did:plc:asdf123/com.atproto.feed.post/$')
+    testLoose('at://did:plc:asdf123/com.atproto.feed.post/@')
+    testLoose('at://did:plc:asdf123/com.atproto.feed.post/!')
+    testLoose('at://did:plc:asdf123/com.atproto.feed.post/*')
+    testLoose('at://did:plc:asdf123/com.atproto.feed.post/(')
+    testLoose('at://did:plc:asdf123/com.atproto.feed.post/,')
+    testLoose('at://did:plc:asdf123/com.atproto.feed.post/;')
+    testLoose('at://did:plc:asdf123/com.atproto.feed.post/abc%30123')
+
+    testLoose('at://did:plc:asdf123/com.atproto.feed.post/%30')
+    testLoose('at://did:plc:asdf123/com.atproto.feed.post/%3')
+    testLoose('at://did:plc:asdf123/com.atproto.feed.post/%')
+    testLoose('at://did:plc:asdf123/com.atproto.feed.post/%zz')
+    testLoose('at://did:plc:asdf123/com.atproto.feed.post/%%%')
+
+    testLoose('at://did:plc:asdf123/com.atproto.feed.post/[]')
+    testLoose('at://did:plc:asdf123/com.atproto.feed.post/foo[')
+    testLoose('at://did:plc:asdf123/com.atproto.feed.post/bar]')
+    testLoose('at://did:plc:asdf123/com.atproto.feed.post/[baz]')
+  })
+
+  describe('valid fragment', () => {
+    testValid('at://did:plc:asdf123#/frac')
+  })
+
+  describe('invalid fragment', () => {
+    testValid('at://did:plc:asdf123#/com.atproto.feed.post')
+    testValid('at://did:plc:asdf123#/com.atproto.feed.post/')
+    testValid('at://did:plc:asdf123#/asdf/')
+
+    testValid('at://did:plc:asdf123/com.atproto.feed.post#/$@!*():,;~.sdf123')
+    testValid('at://did:plc:asdf123#/[asfd]')
+
+    testValid('at://did:plc:asdf123#/$')
+    testValid('at://did:plc:asdf123#/*')
+    testValid('at://did:plc:asdf123#/;')
+    testValid('at://did:plc:asdf123#/,')
+
+    testInvalid('at://did:plc:asdf123#')
+    testInvalid('at://did:plc:asdf123##')
+    testInvalid('#at://did:plc:asdf123')
+    testInvalid('at://did:plc:asdf123#/asdf#/asdf')
+  })
+})
+
+function testValid(value: string) {
+  test(value, () => {
+    expect(isAtUriString(value)).toBe(true)
+    expect(isAtUriString(value, { strict: false })).toBe(true)
+    expect(() => assertAtUriString(value)).not.toThrow()
+    expect(() => assertAtUriString(value, { strict: false })).not.toThrow()
+  })
+}
+
+function testInvalid(value: string) {
+  test(value, () => {
+    expect(isAtUriString(value)).toBe(false)
+    expect(() => assertAtUriString(value)).toThrow(InvalidAtUriError)
+  })
+}
+
+function testLoose(value: string) {
+  test(value, () => {
+    expect(isAtUriString(value)).toBe(false)
+    expect(isAtUriString(value, { strict: false })).toBe(true)
+    expect(() => assertAtUriString(value)).toThrow()
+    expect(() => assertAtUriString(value, { strict: false })).not.toThrow()
+  })
+}
+
+function readLines(filePath: string): string[] {
+  return readFileSync(filePath, 'utf-8')
+    .split(/\r?\n/)
+    .filter((line) => !line.startsWith('#') && line.length > 0)
+}

package/tests/aturi.test.ts

@@ -1,12 +1,26 @@
-import * as fs from 'node:fs'
-import * as readline from 'node:readline'
-import { describe, expect, it } from 'vitest'
-import { AtUri, ensureValidAtUri, ensureValidAtUriRegex } from '../src'
+import { readFileSync } from 'node:fs'
+import { describe, expect, it, test } from 'vitest'
+import { AtUri } from '../src'
 
 describe(AtUri, () => {
-  it('parses valid at uris', () => {
-    //                 input   host    path    query   hash
-    type AtUriTest = [string, string, string, string, string]
+  describe('parses valid interop', () => {
+    for (const value of readLines(
+      `${__dirname}/../../../interop-test-files/syntax/aturi_syntax_valid.txt`,
+    )) {
+      test(value, () => {
+        expect(() => new AtUri(value)).not.toThrow()
+      })
+    }
+  })
+
+  describe('valid at uris', () => {
+    type AtUriTest = [
+      input: string,
+      host: string,
+      path: string,
+      query: string,
+      hash: string,
+    ]
     const TESTS: AtUriTest[] = [
       ['foo.com', 'foo.com', '', '', ''],
       ['at://foo.com', 'foo.com', '', '', ''],
@@ -246,15 +260,17 @@ describe(AtUri, () => {
         '',
       ],
     ]
-    for (const [uri, hostname, pathname, search, hash] of TESTS) {
-      const urip = new AtUri(uri)
-      expect(urip.protocol).toBe('at:')
-      expect(urip.host).toBe(hostname)
-      expect(urip.hostname).toBe(hostname)
-      expect(urip.origin).toBe(`at://${hostname}`)
-      expect(urip.pathname).toBe(pathname)
-      expect(urip.search).toBe(search)
-      expect(urip.hash).toBe(hash)
+    for (const [input, host, path, search, hash] of TESTS) {
+      test(input, () => {
+        const urip = new AtUri(input)
+        expect(urip.protocol).toBe('at:')
+        expect(urip.host).toBe(host)
+        expect(urip.hostname).toBe(host)
+        expect(urip.origin).toBe(`at://${host}`)
+        expect(urip.pathname).toBe(path)
+        expect(urip.search).toBe(search)
+        expect(urip.hash).toBe(hash)
+      })
     }
   })
 
@@ -278,16 +294,16 @@ describe(AtUri, () => {
 
   it('supports modifications', () => {
     const urip = new AtUri('at://foo.com')
-    expect(urip.toString()).toBe('at://foo.com/')
+    expect(urip.toString()).toBe('at://foo.com')
 
     urip.host = 'bar.com'
-    expect(urip.toString()).toBe('at://bar.com/')
+    expect(urip.toString()).toBe('at://bar.com')
     urip.host = 'did:web:localhost%3A1234'
-    expect(urip.toString()).toBe('at://did:web:localhost%3A1234/')
+    expect(urip.toString()).toBe('at://did:web:localhost%3A1234')
     urip.host = 'foo.com'
 
     urip.pathname = '/'
-    expect(urip.toString()).toBe('at://foo.com/')
+    expect(urip.toString()).toBe('at://foo.com')
     urip.pathname = '/foo'
     expect(urip.toString()).toBe('at://foo.com/foo')
     urip.pathname = 'foo'
@@ -316,11 +332,9 @@ describe(AtUri, () => {
     expect(urip.toString()).toBe('at://foo.com/foo?foo=bar&baz=buux#hash')
   })
 
-  it('supports relative URIs', () => {
-    //                 input   path    query   hash
-    type AtUriTest = [string, string, string, string]
+  describe('relative URIs', () => {
+    type AtUriTest = [input: string, path: string, search: string, hash: string]
     const TESTS: AtUriTest[] = [
-      // input hostname pathname query hash
       ['', '', '', ''],
       ['/', '/', '', ''],
       ['/foo', '/foo', '', ''],
@@ -347,180 +361,23 @@ describe(AtUri, () => {
     ]
 
     for (const base of BASES) {
-      const basep = new AtUri(base)
-      for (const [relative, pathname, search, hash] of TESTS) {
-        const urip = new AtUri(relative, base)
-        expect(urip.protocol).toBe('at:')
-        expect(urip.host).toBe(basep.host)
-        expect(urip.hostname).toBe(basep.hostname)
-        expect(urip.origin).toBe(basep.origin)
-        expect(urip.pathname).toBe(pathname)
-        expect(urip.search).toBe(search)
-        expect(urip.hash).toBe(hash)
-      }
+      describe(base, () => {
+        for (const [input, path, search, hash] of TESTS) {
+          test(input, () => {
+            const baseUri = new AtUri(base)
+            const uri = new AtUri(input, base)
+            expect(uri.protocol).toBe('at:')
+            expect(uri.host).toBe(baseUri.host)
+            expect(uri.hostname).toBe(baseUri.hostname)
+            expect(uri.origin).toBe(baseUri.origin)
+            expect(uri.pathname).toBe(path)
+            expect(uri.search).toBe(search)
+            expect(uri.hash).toBe(hash)
+          })
+        }
+      })
     }
   })
-})
-
-describe('AtUri validation', () => {
-  const expectValid = (h: string) => {
-    ensureValidAtUri(h)
-    ensureValidAtUriRegex(h)
-  }
-  const expectInvalid = (h: string) => {
-    expect(() => ensureValidAtUri(h)).toThrow()
-    expect(() => ensureValidAtUriRegex(h)).toThrow()
-  }
-
-  it('enforces spec basics', () => {
-    expectValid('at://did:plc:asdf123')
-    expectValid('at://user.bsky.social')
-    expectValid('at://did:plc:asdf123/com.atproto.feed.post')
-    expectValid('at://did:plc:asdf123/com.atproto.feed.post/record')
-
-    expectValid('at://did:plc:asdf123#/frag')
-    expectValid('at://user.bsky.social#/frag')
-    expectValid('at://did:plc:asdf123/com.atproto.feed.post#/frag')
-    expectValid('at://did:plc:asdf123/com.atproto.feed.post/record#/frag')
-
-    expectInvalid('a://did:plc:asdf123')
-    expectInvalid('at//did:plc:asdf123')
-    expectInvalid('at:/a/did:plc:asdf123')
-    expectInvalid('at:/did:plc:asdf123')
-    expectInvalid('AT://did:plc:asdf123')
-    expectInvalid('http://did:plc:asdf123')
-    expectInvalid('://did:plc:asdf123')
-    expectInvalid('at:did:plc:asdf123')
-    expectInvalid('at:/did:plc:asdf123')
-    expectInvalid('at:///did:plc:asdf123')
-    expectInvalid('at://:/did:plc:asdf123')
-    expectInvalid('at:/ /did:plc:asdf123')
-    expectInvalid('at://did:plc:asdf123 ')
-    expectInvalid('at://did:plc:asdf123/ ')
-    expectInvalid(' at://did:plc:asdf123')
-    expectInvalid('at://did:plc:asdf123/com.atproto.feed.post ')
-    expectInvalid('at://did:plc:asdf123/com.atproto.feed.post# ')
-    expectInvalid('at://did:plc:asdf123/com.atproto.feed.post#/ ')
-    expectInvalid('at://did:plc:asdf123/com.atproto.feed.post#/frag ')
-    expectInvalid('at://did:plc:asdf123/com.atproto.feed.post#fr ag')
-    expectInvalid('//did:plc:asdf123')
-    expectInvalid('at://name')
-    expectInvalid('at://name.0')
-    expectInvalid('at://diD:plc:asdf123')
-    expectInvalid('at://did:plc:asdf123/com.atproto.feed.p@st')
-    expectInvalid('at://did:plc:asdf123/com.atproto.feed.p$st')
-    expectInvalid('at://did:plc:asdf123/com.atproto.feed.p%st')
-    expectInvalid('at://did:plc:asdf123/com.atproto.feed.p&st')
-    expectInvalid('at://did:plc:asdf123/com.atproto.feed.p()t')
-    expectInvalid('at://did:plc:asdf123/com.atproto.feed_post')
-    expectInvalid('at://did:plc:asdf123/-com.atproto.feed.post')
-    expectInvalid('at://did:plc:asdf@123/com.atproto.feed.post')
-
-    expectInvalid('at://DID:plc:asdf123')
-    expectInvalid('at://user.bsky.123')
-    expectInvalid('at://bsky')
-    expectInvalid('at://did:plc:')
-    expectInvalid('at://did:plc:')
-    expectInvalid('at://frag')
-
-    expectValid('at://did:plc:asdf123/com.atproto.feed.post/' + 'o'.repeat(800))
-    expectInvalid(
-      'at://did:plc:asdf123/com.atproto.feed.post/' + 'o'.repeat(8200),
-    )
-  })
-
-  it('has specified behavior on edge cases', () => {
-    expectInvalid('at://user.bsky.social//')
-    expectInvalid('at://user.bsky.social//com.atproto.feed.post')
-    expectInvalid('at://user.bsky.social/com.atproto.feed.post//')
-    expectInvalid(
-      'at://did:plc:asdf123/com.atproto.feed.post/asdf123/more/more',
-    )
-    expectInvalid('at://did:plc:asdf123/short/stuff')
-    expectInvalid('at://did:plc:asdf123/12345')
-  })
-
-  it('enforces no trailing slashes', () => {
-    expectValid('at://did:plc:asdf123')
-    expectInvalid('at://did:plc:asdf123/')
-
-    expectValid('at://user.bsky.social')
-    expectInvalid('at://user.bsky.social/')
-
-    expectValid('at://did:plc:asdf123/com.atproto.feed.post')
-    expectInvalid('at://did:plc:asdf123/com.atproto.feed.post/')
-
-    expectValid('at://did:plc:asdf123/com.atproto.feed.post/record')
-    expectInvalid('at://did:plc:asdf123/com.atproto.feed.post/record/')
-    expectInvalid('at://did:plc:asdf123/com.atproto.feed.post/record/#/frag')
-  })
-
-  it('enforces strict paths', () => {
-    expectValid('at://did:plc:asdf123/com.atproto.feed.post/asdf123')
-    expectInvalid('at://did:plc:asdf123/com.atproto.feed.post/asdf123/asdf')
-  })
-
-  it('is very permissive about record keys', () => {
-    expectValid('at://did:plc:asdf123/com.atproto.feed.post/asdf123')
-    expectValid('at://did:plc:asdf123/com.atproto.feed.post/a')
-    expectValid('at://did:plc:asdf123/com.atproto.feed.post/%23')
-
-    expectValid('at://did:plc:asdf123/com.atproto.feed.post/$@!*)(:,;~.sdf123')
-    expectValid("at://did:plc:asdf123/com.atproto.feed.post/~'sdf123")
-
-    expectValid('at://did:plc:asdf123/com.atproto.feed.post/$')
-    expectValid('at://did:plc:asdf123/com.atproto.feed.post/@')
-    expectValid('at://did:plc:asdf123/com.atproto.feed.post/!')
-    expectValid('at://did:plc:asdf123/com.atproto.feed.post/*')
-    expectValid('at://did:plc:asdf123/com.atproto.feed.post/(')
-    expectValid('at://did:plc:asdf123/com.atproto.feed.post/,')
-    expectValid('at://did:plc:asdf123/com.atproto.feed.post/;')
-    expectValid('at://did:plc:asdf123/com.atproto.feed.post/abc%30123')
-  })
-
-  it('is probably too permissive about URL encoding', () => {
-    expectValid('at://did:plc:asdf123/com.atproto.feed.post/%30')
-    expectValid('at://did:plc:asdf123/com.atproto.feed.post/%3')
-    expectValid('at://did:plc:asdf123/com.atproto.feed.post/%')
-    expectValid('at://did:plc:asdf123/com.atproto.feed.post/%zz')
-    expectValid('at://did:plc:asdf123/com.atproto.feed.post/%%%')
-  })
-
-  it('is very permissive about fragments', () => {
-    expectValid('at://did:plc:asdf123#/frac')
-
-    expectInvalid('at://did:plc:asdf123#')
-    expectInvalid('at://did:plc:asdf123##')
-    expectInvalid('#at://did:plc:asdf123')
-    expectInvalid('at://did:plc:asdf123#/asdf#/asdf')
-
-    expectValid('at://did:plc:asdf123#/com.atproto.feed.post')
-    expectValid('at://did:plc:asdf123#/com.atproto.feed.post/')
-    expectValid('at://did:plc:asdf123#/asdf/')
-
-    expectValid('at://did:plc:asdf123/com.atproto.feed.post#/$@!*():,;~.sdf123')
-    expectValid('at://did:plc:asdf123#/[asfd]')
-
-    expectValid('at://did:plc:asdf123#/$')
-    expectValid('at://did:plc:asdf123#/*')
-    expectValid('at://did:plc:asdf123#/;')
-    expectValid('at://did:plc:asdf123#/,')
-  })
-
-  it('conforms to interop valid ATURIs', () => {
-    const lineReader = readline.createInterface({
-      input: fs.createReadStream(
-        `${__dirname}/interop-files/aturi_syntax_valid.txt`,
-      ),
-      terminal: false,
-    })
-    lineReader.on('line', (line) => {
-      if (line.startsWith('#') || line.length === 0) {
-        return
-      }
-      expectValid(line)
-    })
-  })
 
   it('properly checks that the did property is a valid did', () => {
     const urip = new AtUri('at://did:example:123')
@@ -568,6 +425,10 @@ describe('AtUri validation', () => {
     expect(urip.rkey).toBe('not a valid rkey')
     expect(() => urip.rkeySafe).toThrow()
   })
-
-  // NOTE: this package is currently more permissive than spec about AT URIs, so invalid cases are not errors
 })
+
+function readLines(filePath: string): string[] {
+  return readFileSync(filePath, 'utf-8')
+    .split(/\r?\n/)
+    .filter((line) => !line.startsWith('#') && line.length > 0)
+}

package/tests/datetime.test.ts

@@ -128,9 +128,28 @@ describe(normalizeDatetime, () => {
     expect(normalizeDatetime('1985-04-12T10:20:50.1+01:00')).toEqual(
       '1985-04-12T09:20:50.100Z',
     )
+    expect(normalizeDatetime('Fri, 02 Jan 1999 12:34:56+1212')).toEqual(
+      '1999-01-02T00:22:56.000Z',
+    )
+    expect(normalizeDatetime('Fri, 02 Jan 1999 12:34:56Z')).toEqual(
+      '1999-01-02T12:34:56.000Z',
+    )
     expect(normalizeDatetime('Fri, 02 Jan 1999 12:34:56 GMT')).toEqual(
       '1999-01-02T12:34:56.000Z',
     )
+    expect(normalizeDatetime('Fri, 02 Jan 1999 12:34:56 PST')).toEqual(
+      '1999-01-02T20:34:56.000Z',
+    )
+    expect(normalizeDatetime('Fri, 02 Jan 1999 12:34:56 EST')).toEqual(
+      '1999-01-02T17:34:56.000Z',
+    )
+    // @NOTE "(Central European Standard Time)" is not used by "Date" to infer
+    // the right timezone offset, so these will be parsed as UTC
+    expect(
+      normalizeDatetime(
+        'Fri, 02 Jan 1999 12:34:56 (Central European Standard Time)',
+      ),
+    ).toEqual('1999-01-02T12:34:56.000Z')
     expect(normalizeDatetime('0001-01-01T00:00:00+01:00')).toEqual(
       '0000-12-31T23:00:00.000Z',
     )
@@ -170,6 +189,9 @@ describe(normalizeDatetime, () => {
     expect(() => normalizeDatetime('1999-19-39T23:20:50.123Z')).toThrow(
       InvalidDatetimeError,
     )
+    expect(() => normalizeDatetime('Fri, 02 Jan 1999 12:34:56 AFT')).toThrow(
+      InvalidDatetimeError,
+    )
     expect(() => normalizeDatetime('-000001-12-31T23:00:00.000Z')).toThrow(
       InvalidDatetimeError,
     )

package/tsconfig.build.tsbuildinfo

@@ -1 +1 @@
-{"root":["./src/at-identifier.ts","./src/aturi.ts","./src/aturi_validation.ts","./src/datetime.ts","./src/did.ts","./src/handle.ts","./src/index.ts","./src/language.ts","./src/nsid.ts","./src/recordkey.ts","./src/tid.ts","./src/uri.ts"],"version":"5.8.2"}
+{"root":["./src/at-identifier.ts","./src/aturi.ts","./src/aturi_validation.ts","./src/datetime.ts","./src/did.ts","./src/handle.ts","./src/index.ts","./src/language.ts","./src/nsid.ts","./src/recordkey.ts","./src/tid.ts","./src/uri.ts","./src/lib/result.ts"],"version":"5.8.2"}