feat: json, json-all generators #287

flakey5 · 2025-05-31T18:21:03Z

Closes #214

TODO:

Dedupe code for type list parsing (re feat: json, json-all generators #287 (comment))

codecov-commenter · 2025-05-31T18:21:34Z

Codecov Report

❌ Patch coverage is 93.86531% with 266 lines in your changes missing coverage. Please review.
✅ Project coverage is 82.42%. Comparing base (9285e74) to head (1cf5261).
✅ All tests successful. No failed tests found.

Files with missing lines	Patch %	Lines
src/generators/json/utils/sections/method.mjs	80.96%	67 Missing ⚠️
src/generators/json/utils/sections/event.mjs	76.11%	42 Missing and 1 partial ⚠️
src/utils/generator-error.mjs	21.05%	30 Missing ⚠️
src/generators/json/utils/sections/property.mjs	81.16%	29 Missing ⚠️
src/generators/json/utils/sections/index.mjs	78.94%	23 Missing and 1 partial ⚠️
src/generators/json/index.mjs	80.20%	19 Missing ⚠️
src/utils/assertAstType.mjs	80.00%	11 Missing ⚠️
...generators/json/utils/createParameterGroupings.mjs	91.17%	9 Missing ⚠️
src/generators/json-all/index.mjs	89.74%	8 Missing ⚠️
src/generators/json/utils/sections/base.mjs	95.07%	7 Missing ⚠️
... and 7 more

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #287      +/-   ##
==========================================
+ Coverage   74.60%   82.42%   +7.81%     
==========================================
  Files         112      140      +28     
  Lines       10700    15034    +4334     
  Branches      722     1027     +305     
==========================================
+ Hits         7983    12392    +4409     
+ Misses       2714     2637      -77     
- Partials        3        5       +2

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

avivkeller · 2025-09-30T01:14:13Z

@flakey5 is this still in progress? Need any help?

flakey5 · 2025-09-30T01:54:04Z

Still in progress just haven't been committing, I do wanna get this ready for review within the next coming weeks though

vercel · 2025-10-06T20:16:35Z

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Preview	Updated (UTC)
api-docs-tooling	Ready	Preview	Dec 1, 2025 1:38am

flakey5 · 2025-11-10T16:47:17Z

Bulk of this is done, with only two main things left:

Couple of failing tests for createParameterGroupings
Adding the @constructor property to class sections

I'm leaving this as a draft until those are done, but the rest is ready for review

avivkeller

the rest is ready for review

I've left a first round of reviews. Thank you so much for this effort!

avivkeller · 2025-11-10T20:59:22Z

src/utils/generator-error.mjs

Do we need a seperate generator error?

It exists just to give the general spot in the ast that an error happens to assist with debugging, can definitely be removed if unwanted

@nodejs/web-infra im +1 on removing, WDYT?

src/generators/json/utils/stringifyNode.mjs

src/generators/json/utils/createSectionBase.mjs

src/generators/json/utils/createMethodSection.mjs

src/generators/json/utils/createModuleSection.mjs

avivkeller · 2025-11-12T00:25:44Z

src/generators/json/constants.mjs

+'use strict';
+
+// Grabs the default value if present
+export const DEFAULT_EXPRESSION = /^(D|d)efault(s|):$/;


Let's re-use the DEFAULT_EXPRESSION constant from the legacy generator

We can't since this is used for matching against the text in the ast which doesn't have the asterisks, while the one in the legacy generator is going off of the stringified version in textRaw which does

src/generators/json-all/util/generateJsonSchema.mjs

src/generators/json-all/index.mjs

src/generators/index.mjs

tools/generate-json-types.mjs

Closes #214 Signed-off-by: flakey5 <73616808+flakey5@users.noreply.github.com>

flakey5 · 2025-11-29T02:49:36Z

Marking ready for review, should be all there minus #287 (comment)

src/generators/json-all/index.mjs

avivkeller · 2025-11-29T18:27:15Z

src/generators/json-all/index.mjs

+    input.forEach(section => {
+      const copiedSection = {};
+
+      Object.keys(section).forEach(key => {
+        if (!propertiesToIgnore.includes(key)) {
+          copiedSection[key] = section[key];
+        }
+      });


Suggested change

input.forEach(section => {

const copiedSection = {};

Object.keys(section).forEach(key => {

if (!propertiesToIgnore.includes(key)) {

copiedSection[key] = section[key];

}

});

input.forEach({ ignoreThisKey, ...section } => {

WDYT?

avivkeller · 2025-11-29T18:28:59Z

src/generators/json-all/index.mjs

+      switch (section.type) {
+        case 'module':
+          generatedValue.modules.push(copiedSection);
+          break;
+        case 'text':
+          generatedValue.text.push(copiedSection);
+          break;
+        default:
+          throw new TypeError(`unsupported root section type ${section.type}`);
+      }


Suggested change

switch (section.type) {

case 'module':

generatedValue.modules.push(copiedSection);

break;

case 'text':

generatedValue.text.push(copiedSection);

break;

default:

throw new TypeError(`unsupported root section type ${section.type}`);

}

generatedValue[type === 'module' ? 'modules' : type].push(copiedSection)

nit

avivkeller · 2025-11-29T18:29:50Z

src/generators/json-all/index.mjs

+   */
+  async generate(input, { version, output }) {
+    const generatedValue = {
+      $schema: `${BASE_URL}/docs/${DOC_NODE_VERSION}/api/node-doc-all-schema.jsonc`,


Can we make a SCHEMA_URL constant?

Not really since it intakes the version given by the generator options, same for the rest of the schema url instances

avivkeller · 2025-11-29T18:30:08Z

src/generators/json-all/util/generateJsonSchema.mjs

+ * @param {string} version
+ */
+export function generateJsonSchema(version) {
+  const jsonSchemaUrl = `${BASE_URL}/docs/${version}/api/node-doc-schema.json`;


Ditto on the constant

avivkeller · 2025-11-29T18:31:34Z

src/generators/json/constants.mjs

+// Returns: {string}
+// Returns {string}
+// Returns: {string} bla bla bla
+export const METHOD_RETURN_TYPE_EXTRACTOR = /^(?:R|r)eturns(:?) {(.*)}( .*)?$/;


I'd rather do a case insensitive search

This isn't just checking if it starts with Returns: though, it's extracting the type information and possible description from it as well

Can we change it to /^Returns(:?) {(.*)}( .*)?$/i?

src/generators/json/index.mjs

avivkeller · 2025-11-29T18:34:28Z

src/generators/json/utils/sections/index.mjs

+  }
+
+  return {
+    $schema: `${BASE_URL}docs/${version}/api/node-doc-schema.json`,


Ditto on the constant

avivkeller · 2025-11-29T18:35:14Z

src/utils/generator-error.mjs

@nodejs/web-infra im +1 on removing, WDYT?

Co-authored-by: Aviv Keller <me@aviv.sh>

Signed-off-by: flakey5 <73616808+flakey5@users.noreply.github.com>

ovflowd · 2025-11-30T16:17:03Z

src/generators/json/utils/sections/event.mjs

+          );
+        }
+
+        if (rest[0]?.type === 'link') {


Can we have these specific extractors on dedicated function for ... extracting these? Also are the types here only link and "else"?

ovflowd · 2025-11-30T16:37:49Z

src/generators/json/utils/sections/event.mjs

+
+        parameter['@name'] = 'value';
+
+        const { types, endingIndex } = parseTypeList(parameterAst.children);


Feels like we could dedup this

src/generators/json/utils/sections/event.mjs

ovflowd · 2025-11-30T16:41:05Z

src/generators/json/utils/sections/method.mjs

+
+      break;
+    }
+    case 'text': {


Looks like more duped code

ovflowd · 2025-11-30T16:41:08Z

src/generators/json/utils/sections/method.mjs

+      // it
+      const { types, endingIndex } = parseTypeList(paragraph.children, 2);
+
+      if (types.length === 0) {


Looks like more duped code

ovflowd · 2025-11-30T16:41:14Z

src/generators/json/utils/sections/method.mjs

+        }
+        case 'link': {
+          const { types, endingIndex } = parseTypeList(paragraph.children, 1);
+          if (types.length === 0) {


Looks like more duped code

src/generators/json/utils/sections/method.mjs

src/generators/json/utils/parameterTree.mjs

ovflowd · 2025-11-30T17:16:40Z

src/generators/json/generated.d.ts

Can we mhave this file output be in a different folder? Like a generated folder?

src/generators/json/index.mjs

ovflowd · 2025-11-30T17:17:32Z

src/generators/json/schema.jsonc

Shouldn't this be automatically generated?

The schema should be maintained by hand. It doesn't need to be in a standalone json file though, we could have it be a function like with the json-all's schema. Only reason it is standalone currently is to make it so we can have better intellisense/autocomplete in editors when modifying the schema

scripts/generate-json-types.mjs

ovflowd

Mostly LGTM, I think that we can dedup lots of the code, extract many of the utility functions within parsing things (such as event, property) into dedicated tiny functions that do one thing and then unit test them.

Makes the parent functions simpler, easier to read the code and understand what pieces does what, reusability and unit testing.

Signed-off-by: flakey5 <73616808+flakey5@users.noreply.github.com>

avivkeller · 2025-12-03T22:51:31Z

@flakey5 Is there a reason why many properties start with @? I feel like that makes it harder for consumers, since they now need to <>.["@prop"] instead of just <>.prop

Copilot

Pull request overview

This PR introduces new JSON generators (json and json-all) for the Node.js documentation toolkit, providing machine-readable JSON representations of API documentation.

Key Changes:

New json generator that outputs individual JSON files for each API module with comprehensive type information, method signatures, properties, and events
New json-all generator that combines all JSON outputs into a single file
Comprehensive JSON schema definition with TypeScript type generation support

Reviewed changes

Copilot reviewed 39 out of 44 changed files in this pull request and generated 9 comments.

Show a summary per file

File	Description
`src/generators/json/index.mjs`	Main JSON generator implementation
`src/generators/json-all/index.mjs`	Aggregator generator for combined JSON output
`src/generators/json/schema.jsonc`	JSON schema definition for documentation structure
`src/generators/json/utils/sections/*.mjs`	Section parsers for modules, classes, methods, properties, and events
`src/generators/json/utils/parseTypeList.mjs`	Type list parsing utility
`src/generators/json/utils/createParameterGroupings.mjs`	Method signature parameter grouping logic
`src/utils/buildHierarchy.mjs`	Shared utility for building hierarchical entry structures
`src/utils/assertAstType.mjs`	AST node type assertion utilities
`src/utils/generator-error.mjs`	Custom error class for generator errors
`src/utils/unist.mjs`	Enhanced node transformation with support for break, delete, and link nodes
`scripts/generate-json-types.mjs`	Script to generate TypeScript definitions from JSON schema
`package.json`	New dependencies for JSON parsing and schema validation
`.github/workflows/generate.yml`	CI workflow integration for JSON generator

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2025-12-03T22:56:12Z

scripts/generate-json-types.mjs

+// Parse the JSON schema into an object
+const schema = await parse(schemaString);
+
+// Compile the the JSON schema into TypeScript typedefs


The comment "the the" contains a duplicate word. It should be "the JSON schema".

Suggested change

// Compile the the JSON schema into TypeScript typedefs

// Compile the JSON schema into TypeScript typedefs

Copilot · 2025-12-03T22:56:13Z

src/generators/json/utils/sections/method.mjs

+  // TODO: if the type of the parameter is `object`, sometimes it's followed
+  // by a `list` node that contains the properties expected on that object.
+  // I'd really like those to be included in the json output, but for now
+  // they're not going to be for simplicitly sakes (they still should be in


The comment "for simplicitly sakes" contains a spelling error. It should be "for simplicity's sake" or "for simplicity sake".

Copilot · 2025-12-03T22:56:13Z

src/generators/json/utils/sections/method.mjs

+
+/**
+ * Given a list of parameter names in the order that they should appear and
+ * a map of paramter names to their type info, let's create the signature


The comment contains a grammatical error. "a map of paramter names" should be "a map of parameter names".

Copilot · 2025-12-03T22:56:14Z

src/generators/json/utils/sections/__tests__/base.test.mjs

+    assert.equal(base.changes, undefined);
+    assert.equal(base['@since'], undefined);
+    assert.equal(base.napiVersion, undefined);
+    assert.equal(base.removedIn, undefined);
+    assert.equal(base['@deprecated'], undefined);


[nitpick] The assertion uses assert.equal() which performs loose equality. For more explicit comparison, consider using assert.strictEqual() instead.

Suggested change

assert.equal(base.changes, undefined);

assert.equal(base['@since'], undefined);

assert.equal(base.napiVersion, undefined);

assert.equal(base.removedIn, undefined);

assert.equal(base['@deprecated'], undefined);

assert.strictEqual(base.changes, undefined);

assert.strictEqual(base['@since'], undefined);

assert.strictEqual(base.napiVersion, undefined);

assert.strictEqual(base.removedIn, undefined);

assert.strictEqual(base['@deprecated'], undefined);

Copilot · 2025-12-03T22:56:14Z

src/generators/json/schema.jsonc

+          "additionalProperties": false
+        },
+      ],
+    },
+
+    "Class": {
+      "allOf": [
+        { "$ref": "#/definitions/SectionBase" },
+        {
+          "type": "object",
+          "properties": {
+            "type": {
+              "type": "string",
+              "enum": ["class"],
+            },
+            "@constructor": {
+              "type": "array",
+              "items": { "$ref": "#/definitions/MethodSignature" },
+            },
+            "methods": {
+              "type": "array",
+              "items": { "$ref": "#/definitions/Method" },
+            },
+            "staticMethods": {
+              "type": "array",
+              "items": { "$ref": "#/definitions/Method" },
+            },
+            "properties": {
+              "type": "array",
+              "items": { "$ref": "#/definitions/Property" },
+            },
+            "events": {
+              "type": "array",
+              "items": { "$ref": "#/definitions/Event" },
+            },
+          },
+          "required": [
+            "type",
+          ],
+      "additionalProperties": false
+        },
+      ],
+    },
+
+    "Method": {
+      "description": "A JavaScript function.",
+      "allOf": [
+        { "$ref": "#/definitions/SectionBase" },
+        {
+          "type": "object",
+          "properties": {
+            "type": {
+              "type": "string",
+              "enum": ["method"],
+            },
+            "signatures": {
+              "type": "array",
+              "items": { "$ref": "#/definitions/MethodSignature" },
+            },
+          },
+          "required": ["type", "signatures"],
+      "additionalProperties": false
+        },
+      ],
+    },
+
+    "MethodSignature": {
+      "type": "object",
+      "properties": {
+        "parameters": {
+          "type": "array",
+          "items": { "$ref": "#/definitions/MethodParameter" },
+        },
+        "@returns": {
+          "$ref": "#/definitions/MethodReturnType",
+        },
+      },
+      "required": ["parameters", "@returns"],
+      "additionalProperties": false
+    },
+
+    "MethodParameter": {
+      "type": "object",
+      "properties": {
+        "@name": {
+          "type": "string",
+          "description": "Name of the parameter.",
+        },
+        "@type": {
+          "description": "Type of the parameter",
+          "oneOf": [
+            {
+              "type": "string",
+              "examples": ["string"],
+            },
+            {
+              "type": "array",
+              "items": {
+                "type": "string",
+              },
+              "minItems": 1,
+            },
+          ],
+        },
+        "description": {
+          "type": "string",
+        },
+        "@default": {
+          "type": "string",
+          "description": "The parameter's default value",
+          "examples": ["foo"],
+        },
+      },
+      "required": ["@name", "@type"],
+      "additionalProperties": false
+    },
+
+    "MethodReturnType": {
+      "type": "object",
+      "description": "A method signature's return type.",
+      "properties": {
+        "description": {
+          "type": "string",
+        },
+        "@type": {
+          "description": "The method signature's return type.",
+          "oneOf": [
+            {
+              "type": "string",
+              "examples": ["string"],
+            },
+            {
+              "type": "array",
+              "items": { "type": "string" },
+              "examples": [["string", "Promise<string>"]],
+              "minItems": 1,
+            },
+          ],
+        },
+      },
+      "required": ["@type"],
+      "additionalProperties": false
+    },
+
+    "Global": {
+      "type": "object",
+      "properties": {},
+      "required": [],
+    },
+
+    "Property": {
+      "description": "A property on a JavaScript object or class.",
+      "allOf": [
+        { "$ref": "#/definitions/SectionBase" },
+        {
+          "type": "object",
+          "properties": {
+            "type": {
+              "type": "string",
+              "enum": ["property"],
+            },
+            "@type": {
+              "description": "JavaScript type of the property.",
+              "oneOf": [
+                {
+                  "type": "string",
+                  "examples": ["string"],
+                },
+                {
+                  "type": "array",
+                  "items": {
+                    "type": "string",
+                  },
+                  "minItems": 1,
+                },
+              ],
+            },
+            "mutable": {
+              "type": "boolean",
+              "description": "Is this property modifiable by user code?",
+              "default": false,
+            },
+          },
+          "required": ["type"],
+      "additionalProperties": false
+        },
+      ],
+    },
+
+    "Event": {
+      "description": "An event that can be emitted by the parent object or class.",
+      "allOf": [
+        { "$ref": "#/definitions/SectionBase" },
+        {
+          "type": "object",
+          "properties": {
+            "type": {
+              "type": "string",
+              "enum": ["event"],
+            },
+            "parameters": {
+              "type": "array",
+              "items": { "$ref": "#/definitions/MethodParameter" },
+            },
+          },
+          "required": ["type", "parameters"],
+      "additionalProperties": false


[nitpick] Inconsistent indentation: the additionalProperties: false statements on lines 156, 195, 217, 234, 270, 297, 340, 362, and 399 are not indented consistently with the surrounding JSON structure. They should be indented at the same level as the other properties in their respective objects.

Copilot · 2025-12-03T22:56:14Z

src/generators/json/schema.jsonc

+          "$ref": "#/definitions/MethodReturnType",
+        },
+      },
+      "required": ["parameters", "@returns"],


[nitpick] The MethodSignature.parameters property is defined as required (line 233), but in several places in the code, signatures are created with an empty parameters array instead of omitting it when there are no parameters. Consider making this property optional or consistently handling empty arrays.

Suggested change

"required": ["parameters", "@returns"],

"required": ["@returns"],

Copilot · 2025-12-03T22:56:15Z

src/generators/json/utils/sections/index.mjs

+export function createSection(head, entries, version) {
+  const entryHierarchy = buildHierarchy(entries);
+
+  if (entryHierarchy.length != 1) {


[nitpick] Inconsistent comparison operator: The code uses != (loose inequality) instead of !== (strict inequality). For consistency with modern JavaScript best practices, use strict equality operators.

Copilot · 2025-12-03T22:56:15Z

src/generators/json/utils/sections/index.mjs

+   */
+  let section;
+  try {
+    section = createSectionBase(entry, parent?.type);


Superfluous argument passed to function createSectionBase.

Copilot · 2025-12-03T22:56:15Z

src/generators/json/schema.jsonc

+   *  - Run `scripts/generate-json-types.mjs` and ensure there aren't type errors
+   */
+
+  "$schema": "http://json-schema.org/draft-07/schema#",


The $schema reference uses an http URL, which enables man-in-the-middle tampering when tools fetch the JSON Schema for validation. An attacker intercepting the connection could serve a modified schema to alter validation behavior or inject malicious constraints. Fix by switching to https:

"$schema": "https://json-schema.org/draft-07/schema#"

Suggested change

"$schema": "http://json-schema.org/draft-07/schema#",

"$schema": "https://json-schema.org/draft-07/schema#",

avivkeller

More comments :-)

avivkeller · 2025-12-03T22:47:41Z

src/generators/json/utils/sections/base.mjs

+
+    // Not code, let's stringify it and add it to the description.
+    section.description ??= '';
+    section.description += `${transformNodeToString(node)}${node.type === 'paragraph' ? '\n' : ' '}`;


Is this added newline needed?

avivkeller · 2025-12-03T22:48:42Z

src/generators/json/utils/sections/base.mjs

+
+  if (section.description) {
+    section.description = section.description.trim();
+  }


Suggested change

}

section.description &&= section.description.trim();

avivkeller · 2025-12-03T22:50:17Z

src/generators/json/utils/sections/base.mjs

+      } else {
+        section['@example'] = node.value;
+      }
+


Suggested change

const examples = [...enforceArray(section['@example']), node.value];

section['@example'] = examples.length === 1 ? examples[0] : examples;

avivkeller · 2025-12-03T22:53:23Z

src/generators/json/utils/sections/base.mjs

+    value = Number(value);
+
+    if (isNaN(value)) {
+      throw new GeneratorError(`Stability index ${stability.index} NaN`);
+    }
+  }
+


Suggested change

value = Number(value);

if (isNaN(value)) {

throw new GeneratorError(`Stability index ${stability.index} NaN`);

}

}

const value = parseFloat(stability.index);

Do we really need to throw an error if it's NaN? I'd rather produce JSON with the NaN (even though this should never happen) than error the entire generator?

avivkeller · 2025-12-03T22:53:54Z

src/generators/json/utils/sections/base.mjs

+  section.stability = {
+    value,
+    text: stability.description,
+  };
+}


Can we just do section.stability = stability? Like keep the { index, description } format?

avivkeller · 2025-12-04T21:17:09Z

src/generators/json/utils/sections/index.mjs

+  if (entryHierarchy.length != 1) {
+    throw new TypeError(`${head.api_doc_source} has multiple root elements`);
+  }
+


I don't think there will ever be more than one root, right?

Let's just ignore the extra data, not error on it

avivkeller · 2025-12-04T21:17:52Z

src/generators/json/utils/sections/index.mjs

+  if (section.type !== 'module' && section.type !== 'text') {
+    throw new GeneratorError(
+      `expected root section to be a module or text, got ${section.type}`,
+      { entry: head }
+    );
+  }
+


I think these if checks are too strict, IMO. I feel like there's nothing wrong with it having the wrong type (like, the generator should still work)

avivkeller · 2025-12-04T21:20:04Z

src/generators/json/constants.mjs

+// Returns: {string}
+// Returns {string}
+// Returns: {string} bla bla bla
+export const METHOD_RETURN_TYPE_EXTRACTOR = /^(?:R|r)eturns(:?) {(.*)}( .*)?$/;


Can we change it to /^Returns(:?) {(.*)}( .*)?$/i?

avivkeller · 2025-12-04T21:22:16Z

src/generators/json-all/index.mjs

+   * @returns {Promise<object>}
+   */
+  async generate(input, { version, output }) {
+    const versionString = `v${version.toString()}`;


nit

Suggested change

const versionString = `v${version.toString()}`;

const versionString = `v${version.raw}`;

avivkeller · 2025-12-04T21:24:11Z

src/generators/json/utils/findParentSection.mjs

+export function findParentSection(section, type) {
+  type = enforceArray(type);
+
+  let parent = section.parent;
+
+  while (parent) {
+    if (type.includes(parent.type)) {
+      return parent;
+    }
+
+    parent = parent.parent;
+  }
+
+  return undefined;
+}


Suggested change

export function findParentSection(section, type) {

type = enforceArray(type);

let parent = section.parent;

while (parent) {

if (type.includes(parent.type)) {

return parent;

}

parent = parent.parent;

}

return undefined;

}

export function findParentSection({ parent }, type) {

type = enforceArray(type);

while (parent) {

if (type.includes(parent.type)) {

return parent;

}

parent = parent.parent;

}

return undefined;

}

avivkeller mentioned this pull request Jun 24, 2025

feat(web): add web generator #285

Merged

6 tasks

flakey5 force-pushed the flakey5/20240909/new-json-gen branch from a4cd425 to d711fee Compare October 6, 2025 20:16

vercel bot deployed to Preview October 6, 2025 20:19 View deployment

vercel bot deployed to Preview October 6, 2025 23:19 View deployment

flakey5 mentioned this pull request Oct 11, 2025

Making it easier for DTS consumers #441

Open

flakey5 force-pushed the flakey5/20240909/new-json-gen branch from 2d10df1 to e4dbbe4 Compare October 23, 2025 04:17

vercel bot deployed to Preview October 23, 2025 04:20 View deployment

vercel bot deployed to Preview October 23, 2025 18:16 View deployment

vercel bot deployed to Preview October 27, 2025 00:44 View deployment

flakey5 force-pushed the flakey5/20240909/new-json-gen branch from 6bfc555 to 1bcf93d Compare November 10, 2025 16:42

vercel bot deployed to Preview November 10, 2025 16:45 View deployment

avivkeller reviewed Nov 12, 2025

View reviewed changes

vercel bot deployed to Preview November 16, 2025 08:02 View deployment

vercel bot deployed to Preview November 29, 2025 01:15 View deployment

flakey5 force-pushed the flakey5/20240909/new-json-gen branch from ee1684d to b37daee Compare November 29, 2025 02:34

vercel bot deployed to Preview November 29, 2025 02:36 View deployment

feat: json, json-all generators

8ebb3d6

Closes #214 Signed-off-by: flakey5 <73616808+flakey5@users.noreply.github.com>

flakey5 force-pushed the flakey5/20240909/new-json-gen branch from b37daee to 8ebb3d6 Compare November 29, 2025 02:44

vercel bot deployed to Preview November 29, 2025 02:46 View deployment

flakey5 marked this pull request as ready for review November 29, 2025 02:49

flakey5 requested a review from a team as a code owner November 29, 2025 02:49

avivkeller reviewed Nov 29, 2025

View reviewed changes

Apply suggestions from code review

bcde0e4

Co-authored-by: Aviv Keller <me@aviv.sh>

vercel bot deployed to Preview November 30, 2025 05:19 View deployment

flakey5 added 2 commits November 29, 2025 22:28

review

ee9b725

Signed-off-by: flakey5 <73616808+flakey5@users.noreply.github.com>

remove walk function

826cdfc

Signed-off-by: flakey5 <73616808+flakey5@users.noreply.github.com>

vercel bot deployed to Preview November 30, 2025 06:33 View deployment