Merge pull request #2 from httpland/beta

Beta

Author: TomokiMiyauci

CHANGELOG.md

@@ -1,3 +1,10 @@
+## [1.0.1-beta.1](https://github.com/httpland/accept-ranges-parser/compare/1.0.0...1.0.1-beta.1) (2023-04-29)
+
+
+### Bug Fixes
+
+* **parse:** accept list what includes empty element ([8dea1c5](https://github.com/httpland/accept-ranges-parser/commit/8dea1c5defe575b866ffb22b2a51cd468dbc00a3))
+
 # 1.0.0 (2023-03-27)
 
 

_tools/meta.ts

@@ -1,4 +1,4 @@
-import { BuildOptions } from "https://deno.land/x/dnt@0.33.1/mod.ts";
+import { BuildOptions } from "https://deno.land/x/dnt@0.34.0/mod.ts";
 
 export const makeOptions = (version: string): BuildOptions => ({
   test: false,
@@ -37,10 +37,25 @@ export const makeOptions = (version: string): BuildOptions => ({
   },
   packageManager: "pnpm",
   mappings: {
-    "https://deno.land/x/isx@1.1.0/is_string.ts": {
+    "https://deno.land/x/isx@1.3.1/is_string.ts": {
       name: "@miyauci/isx",
-      version: "1.1.0",
-      subPath: "is_string",
+      version: "1.3.1",
+      subPath: "is_string.js",
+    },
+    "https://deno.land/x/[email protected]/iterable/is_not_empty.ts": {
+      name: "@miyauci/isx",
+      version: "1.3.1",
+      subPath: "iterable/is_not_empty.js",
+    },
+    "https://deno.land/x/[email protected]/token.ts": {
+      name: "@httpland/http-utils",
+      version: "1.2.0",
+      subPath: "token.js",
+    },
+    "https://deno.land/x/[email protected]/list.ts": {
+      name: "@httpland/http-utils",
+      version: "1.2.0",
+      subPath: "list.js",
     },
   },
 });

_tools/publish_npm.ts

@@ -16,11 +16,12 @@ if (import.meta.main) {
   const tag = isPrerelease?.[0] ?? "latest";
 
   const pkg = makeOptions(version);
-  const result = await Deno.run({
-    cmd: ["npm", "publish", pkg.outDir, "--tag", String(tag)],
-    stdout: "piped",
-  })
-    .output();
+  const command = new Deno.Command("npm", {
+    args: ["publish", pkg.outDir, "--tag", String(tag)],
+  });
+  const result = await command.output();
 
-  console.log(new TextDecoder().decode(result));
+  if (!result.success) {
+    console.error(new TextDecoder().decode(result.stderr));
+  }
 }

deno.lock

@@ -1,78 +1,10 @@
 // Copyright 2023-latest the httpland authors. All rights reserved. MIT license.
 // This module is browser compatible.
 
-export { isString } from "https://deno.land/x/[email protected]/is_string.ts";
-
-// TODO:(miyauci) Packaging and externalization this module.
-
-/** Types for uppercase letter. */
-export type UppercaseLetter =
-  | "A"
-  | "B"
-  | "C"
-  | "D"
-  | "E"
-  | "F"
-  | "G"
-  | "H"
-  | "I"
-  | "J"
-  | "K"
-  | "L"
-  | "M"
-  | "N"
-  | "O"
-  | "P"
-  | "Q"
-  | "R"
-  | "S"
-  | "T"
-  | "U"
-  | "V"
-  | "W"
-  | "X"
-  | "Y"
-  | "Z";
-
-/** Types for lowercase letter. */
-export type LowercaseLetter = Lowercase<UppercaseLetter>;
-
-/** Types for letter. */
-export type Letter = UppercaseLetter | LowercaseLetter;
-
-/** Types for digit. */
-export type Digit = 0 | 1 | 2 | 3 | 4 | 5 | 6 | 7 | 8 | 9;
-
-/** Deserialize types. */
-export type Stringify<
-  T extends string | number | bigint | boolean | null | undefined,
-> = `${T}`;
-
-/** Representation of [`<DIGIT>`](https://www.rfc-editor.org/rfc/rfc5234#appendix-B.1). */
-export type DIGIT = Stringify<Digit>;
-
-/** Representation of [`<ALPHA>`](https://www.rfc-editor.org/rfc/rfc5234#appendix-B.1). */
-export type ALPHA = Letter;
-
-/** Representation of [`<tchar>`](https://www.rfc-editor.org/rfc/rfc9110#section-5.6.2-2). */
-export type TChar =
-  | "!"
-  | "#"
-  | "$"
-  | "%"
-  | "&"
-  | "'"
-  | "*"
-  | "+"
-  | "-"
-  | "."
-  | "^"
-  | "_"
-  | "`"
-  | "|"
-  | "~"
-  | ALPHA
-  | DIGIT;
-
-/** Representation of [`<token>`](https://www.rfc-editor.org/rfc/rfc9110#section-5.6.2-2). */
-export type Token = `${TChar}${string}`;
+export { isString } from "https://deno.land/x/[email protected]/is_string.ts";
+export { isNotEmpty } from "https://deno.land/x/[email protected]/iterable/is_not_empty.ts";
+export {
+  isToken,
+  type Token,
+} from "https://deno.land/x/[email protected]/token.ts";
+export { parseListFields } from "https://deno.land/x/[email protected]/list.ts";

deps.ts

@@ -1,7 +1,7 @@
 // Copyright 2023-latest the httpland authors. All rights reserved. MIT license.
 // This module is browser compatible.
 
-import { isTokenFormat } from "./utils.ts";
+import { isNotEmpty, isToken, parseListFields } from "./deps.ts";
 import { Msg } from "./constants.ts";
 import type { AcceptRanges } from "./types.ts";
 
@@ -18,15 +18,13 @@ import type { AcceptRanges } from "./types.ts";
  * @throws {SyntaxError} If the input is invalid [`<Accept-Ranges>`](https://www.rfc-editor.org/rfc/rfc9110.html#section-14.3-2) syntax.
  */
 export function parseAcceptRanges(input: string): AcceptRanges {
-  const acceptableRanges = input
-    .trim()
-    .split(",")
-    .map((v) => v.trim());
+  const acceptableRanges = parseListFields(input);
+  const msg = `${Msg.InvalidSyntax} "${input}"`;
+
+  if (!isNotEmpty(acceptableRanges)) throw new SyntaxError(msg);
 
   acceptableRanges.forEach((token) => {
-    if (!isTokenFormat(token)) {
-      throw SyntaxError(`${Msg.InvalidSyntax} "${input}"`);
-    }
+    if (!isToken(token)) throw new SyntaxError(msg);
   });
 
   return acceptableRanges as AcceptRanges;

parse.ts

@@ -18,8 +18,6 @@ describe("parseAcceptRanges", () => {
       ",",
       ",,,",
       ", , ,",
-      "a,",
-      "a, , a",
       "あ",
       "a, あ",
       `""`,
@@ -38,6 +36,7 @@ describe("parseAcceptRanges", () => {
       [" abc ", ["abc"]],
       ["abc, def", ["abc", "def"]],
       ["bytes,none", ["bytes", "none"]],
+      ["bytes, ,,none", ["bytes", "none"]],
     ];
 
     table.forEach(([input, expected]) => {

parse_test.ts

@@ -1,5 +1,4 @@
-import { isString } from "./deps.ts";
-import { isTokenFormat } from "./utils.ts";
+import { isString, isToken } from "./deps.ts";
 import { Msg } from "./constants.ts";
 
 /** Serialize string of array into string.
@@ -20,7 +19,7 @@ export function stringifyAcceptRanges(
   const targets = isString(acceptRanges) ? [acceptRanges] : acceptRanges;
 
   targets.forEach((rangeUnit) => {
-    if (!isTokenFormat(rangeUnit)) {
+    if (!isToken(rangeUnit)) {
       throw TypeError(`${Msg.InvalidRangeUnit} "${rangeUnit}"`);
     }
   });