fix parse() throws error on valid ReflectOnly Blink extended attributes

lib/productions/extended-attributes.js

@@ -1,8 +1,41 @@
 import { Base } from "./base.js";
 import { ArrayBase } from "./array-base.js";
-import { list, identifiersOrStrings, argument_list, autoParenter } from "./helpers.js";
+import { Token } from "./token.js";
+import { list, argument_list, autoParenter } from "./helpers.js";
 import { validationError } from "../error.js";
 
+/**
+ * @param {import("../tokeniser").Tokeniser} tokeniser
+ */
+function identifiers(tokeniser) {
+  const ids = list(tokeniser, { parser: Token.parser(tokeniser, "identifier"), listName: "identifier list" });
+  if (!ids.length) {
+    tokeniser.error("Expected identifiers but none found");
+  }
+  return ids;
+}
+/**
+ * 
+ * @param {import("../tokeniser").Tokeniser} tokeniser 
+ * @param {string} tokenName 
+ */
+function tokens(tokeniser, tokenName) {
+  const toks = list(tokeniser, {
+    parser: Token.parser(tokeniser, tokenName),
+    listName: tokenName + " list"
+  });
+  if (!toks.length) {
+    tokeniser.error(`Expected ${tokenName}s but none found`);
+  }
+  return toks;
+}
+
+// This will allow a set of strings to be parsed
+ function identifiersOrStrings(tokeniser) {
+  return tokens(tokeniser, "identifier") || tokens(tokeniser, "string");
+}
+
+
 class ExtendedAttributeParameters extends Base {
   /**
    * @param {import("../tokeniser").Tokeniser} tokeniser

lib/productions/helpers.js

@@ -1,6 +1,5 @@
 import { Type } from "./type.js";
 import { Argument } from "./argument.js";
-import { Token } from "./token.js";
 import { ExtendedAttributes, SimpleExtendedAttribute } from "./extended-attributes.js";
 import { Operation } from "./operation.js";
 import { Attribute } from "./attribute.js";
@@ -43,9 +42,7 @@ export function list(tokeniser, { parser, allowDangler, listName = "list" }) {
   return items;
 }
 
-/**
- * @param {import("../tokeniser").Tokeniser} tokeniser
- */
+
 export function const_value(tokeniser) {
   return tokeniser.consume("true", "false", "Infinity", "-Infinity", "NaN", "decimal", "integer");
 }
@@ -109,17 +106,6 @@ export function primitive_type(tokeniser) {
   }
 }
 
-/**
- * @param {import("../tokeniser").Tokeniser} tokeniser
- */
-export function identifiers(tokeniser) {
-  const ids = list(tokeniser, { parser: Token.parser(tokeniser, "identifier"), listName: "identifier list" });
-  if (!ids.length) {
-    tokeniser.error("Expected identifiers but none found");
-  }
-  return ids;
-}
-
 /**
  * @param {import("../tokeniser").Tokeniser} tokeniser
  */
@@ -289,25 +275,3 @@ export function autoParenter(data, parent) {
     }
   });
 }
-
-/**
- * Returns a proxy that auto-assign `parent` field.
- * @template T
- * @param {T} tokeniser
- * @param {*} [tokenName] either can be a string or identifier
- * @return {T}
- */
-function tokens(tokeniser, tokenName) {
-  const toks = list(tokeniser, {
-    parser: Token.parser(tokeniser, tokenName),
-    listName: tokenName + " list"
-  });
-  if (!toks.length) {
-    tokeniser.error(`Expected ${tokenName}s but none found`);
-  }
-  return toks;
-}
-// This will allow a set of strings to be parsed
-export function identifiersOrStrings(tokeniser) {
-  return tokens(tokeniser, "identifier") || tokens(tokeniser, "string");
-}