Label readonly properties in the docs (mozilla#158)

sphinx_js/ir.py

@@ -274,6 +274,7 @@ class Attribute(TopLevel, _Member):
 
     #: The type this property's value can have
     type: Type
+    readonly: bool = False
     kind: Literal["attribute"] = "attribute"
 
 

sphinx_js/js/convertTopLevel.ts

@@ -471,6 +471,7 @@ export class Converter {
     const result: Attribute = {
       ...this.memberProps(v),
       ...this.topLevelProperties(v),
+      readonly: false,
       kind: "attribute",
       type,
     };
@@ -542,12 +543,12 @@ export class Converter {
     } else {
       type = this.convertType(prop.type!);
     }
-    // TODO: add a readonly indicator if it's readonly
     const result: Attribute = {
       type,
       ...this.memberProps(prop),
       ...this.topLevelProperties(prop),
       description: renderCommentSummary(prop.comment),
+      readonly: prop.flags.isReadonly,
       kind: "attribute",
     };
     return [result, prop.children];
@@ -581,9 +582,11 @@ export class Converter {
       sig = prop.setSignature;
       type = sig.parameters![0].type!;
     }
-    // TODO: add a readonly indicator if there's no setter.
+    // If there's no setter say it's readonly
+    const readonly = !prop.setSignature;
     const result: Attribute = {
       type: this.convertType(type),
+      readonly,
       ...this.memberProps(prop),
       ...this.topLevelProperties(prop),
       kind: "attribute",

sphinx_js/js/ir.ts

@@ -113,6 +113,7 @@ export type TopLevel = {
 export type Attribute = TopLevel &
   Member & {
     type: Type;
+    readonly: boolean;
     kind: "attribute";
   };
 

sphinx_js/renderers.py

@@ -657,8 +657,11 @@ class AutoAttributeRenderer(JsRenderer):
 
     def _template_vars(self, name: str, obj: Attribute | TypeAlias) -> dict[str, Any]:  # type: ignore[override]
         is_optional = False
+        ty = self.render_type(obj.type)
         if isinstance(obj, Attribute):
             is_optional = obj.is_optional
+            if obj.readonly:
+                ty = "readonly " + ty
         type_params = ""
         is_type_alias = isinstance(obj, TypeAlias)
         fields: Iterator[tuple[list[str], str]] = iter([])
@@ -675,7 +678,7 @@ def _template_vars(self, name: str, obj: Attribute | TypeAlias) -> dict[str, Any
             is_optional=is_optional,
             see_also=obj.see_alsos,
             examples=[render_description(ex) for ex in obj.examples],
-            type=self.render_type(obj.type),
+            type=ty,
             content="\n".join(self._content),
         )
 

tests/test_build_ts/source/class.ts

@@ -109,6 +109,9 @@ export function weirdCodeInDescription() {}
 export function spinxLinkInDescription() {}
 
 export class GetSetDocs {
+  readonly x: number;
+  y: number;
+
   /**
    * Getter with comment
    */
@@ -119,7 +122,7 @@ export class GetSetDocs {
   /**
    * Setter with comment
    */
-  set b(x) {}
+  set b(x: number) {}
 }
 
 export class Base {

tests/test_build_ts/test_build_ts.py

@@ -193,21 +193,29 @@ def test_get_set(self):
             "getset",
             dedent(
                 """\
-                class GetSetDocs()
+class GetSetDocs()
 
-                   *exported from* "class"
+   *exported from* "class"
+
+   GetSetDocs.a
+
+      type: readonly number
+
+      Getter with comment
+
+   GetSetDocs.b
 
-                   GetSetDocs.a
+      type: number
 
-                      type: number
+      Setter with comment
 
-                      Getter with comment
+   GetSetDocs.x
 
-                   GetSetDocs.b
+      type: readonly number
 
-                      type: any
+   GetSetDocs.y
 
-                      Setter with comment
+      type: number
                 """
             ),
         )
@@ -224,7 +232,7 @@ class Base()
 
                    Base.a
 
-                      type: number
+                      type: readonly number
 
                    Base.f()
 

tests/test_ir.py

@@ -59,6 +59,7 @@ def test_missing_default():
     is_optional=False,
     is_static=False,
     is_private=False,
+    readonly=False,
     type=[]
 )
 attr_dict = {k: v for k, v in getmembers(attribute_base) if not k.startswith("_")}