From efaf2fc1ca3dcd93e4f392e5d9c72840813ad5ea Mon Sep 17 00:00:00 2001
From: Justin Fagnani <justinfagnani@google.com>
Date: Sat, 2 Sep 2023 10:57:13 -0700
Subject: [PATCH] Clarify reflected attributes (#126)

---
 CHANGELOG.md |  6 +++++-
 schema.d.ts  | 11 +++++++----
 schema.json  |  8 ++++----
 3 files changed, 16 insertions(+), 9 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 50f6c05..2673543 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -17,7 +17,11 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
 <!-- ### Removed -->
 <!-- ### Fixed -->
 
-<!-- ## [x.y.z] - YYYY-MM-DD -->
+## Unreleased
+
+### Fixed
+
+- Clarified that all attributes of a `CustomElement` must be listed in the the `attributes` array, even those reflected from a `CustomElementField`.
 
 ## [2.0.0] - 2022-09-12
 
diff --git a/schema.d.ts b/schema.d.ts
index 3ad53ae..946504d 100644
--- a/schema.d.ts
+++ b/schema.d.ts
@@ -466,9 +466,9 @@ export interface ClassLike {
   /**
    * The superclass of this class.
    *
-   * If this class is defined with mixin
-   * applications, the prototype chain includes the mixin applications
-   * and the true superclass is computed from them.
+   * If this class is defined with mixin applications, the prototype chain
+   * includes the mixin applications and the true superclass is computed
+   * from them.
    */
   superclass?: Reference;
 
@@ -573,13 +573,16 @@ export interface ClassField extends PropertyLike {
 export interface CustomElementField extends ClassField {
   /**
    * The corresponding attribute name if there is one.
+   * 
+   * If this property is defined, the attribute must be listed in the classes'
+   * `attributes` array.
    */
   attribute?: string;
 
   /**
    * If the property reflects to an attribute.
    *
-   * If this is true, `attribute` must be defined.
+   * If this is true, the `attribute` property must be defined.
    */
   reflects?: boolean;
 }
diff --git a/schema.json b/schema.json
index c533341..816dd02 100644
--- a/schema.json
+++ b/schema.json
@@ -93,7 +93,7 @@
                 },
                 "superclass": {
                     "$ref": "#/definitions/Reference",
-                    "description": "The superclass of this class.\n\nIf this class is defined with mixin\napplications, the prototype chain includes the mixin applications\nand the true superclass is computed from them."
+                    "description": "The superclass of this class.\n\nIf this class is defined with mixin applications, the prototype chain\nincludes the mixin applications and the true superclass is computed\nfrom them."
                 }
             },
             "required": [
@@ -383,7 +383,7 @@
                 },
                 "superclass": {
                     "$ref": "#/definitions/Reference",
-                    "description": "The superclass of this class.\n\nIf this class is defined with mixin\napplications, the prototype chain includes the mixin applications\nand the true superclass is computed from them."
+                    "description": "The superclass of this class.\n\nIf this class is defined with mixin applications, the prototype chain\nincludes the mixin applications and the true superclass is computed\nfrom them."
                 },
                 "tagName": {
                     "description": "An optional tag name that should be specified if this is a\nself-registering element.\n\nSelf-registering elements must also include a CustomElementExport\nin the module's exports.",
@@ -549,7 +549,7 @@
                 },
                 "superclass": {
                     "$ref": "#/definitions/Reference",
-                    "description": "The superclass of this class.\n\nIf this class is defined with mixin\napplications, the prototype chain includes the mixin applications\nand the true superclass is computed from them."
+                    "description": "The superclass of this class.\n\nIf this class is defined with mixin applications, the prototype chain\nincludes the mixin applications and the true superclass is computed\nfrom them."
                 },
                 "tagName": {
                     "description": "An optional tag name that should be specified if this is a\nself-registering element.\n\nSelf-registering elements must also include a CustomElementExport\nin the module's exports.",
@@ -853,7 +853,7 @@
                 },
                 "superclass": {
                     "$ref": "#/definitions/Reference",
-                    "description": "The superclass of this class.\n\nIf this class is defined with mixin\napplications, the prototype chain includes the mixin applications\nand the true superclass is computed from them."
+                    "description": "The superclass of this class.\n\nIf this class is defined with mixin applications, the prototype chain\nincludes the mixin applications and the true superclass is computed\nfrom them."
                 }
             },
             "required": [