Fix warnings due to removed object types

Resolves #6

Author: Gerrit0

.vscode/launch.json

@@ -0,0 +1,17 @@
+{
+    // Use IntelliSense to learn about possible Node.js debug attributes.
+    // Hover to view descriptions of existing attributes.
+    // For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
+    "version": "0.2.0",
+    "configurations": [
+        {
+            "name": "Attach",
+            "port": 9229,
+            "request": "attach",
+            "internalConsoleOptions": "openOnSessionStart",
+            "skipFiles": ["<node_internals>/**"],
+            "type": "node",
+            "sourceMaps": true
+        }
+    ]
+}

package-lock.json

@@ -18,7 +18,7 @@
     "@types/node": "^20.14.8",
     "outdent": "^0.8.0",
     "prettier": "^3.3.2",
-    "typedoc": "^0.26.0",
+    "typedoc": "^0.26.6",
     "typescript": "^5.5.2",
     "vitest": "^1.6.0",
     "zod": "^3.23.8"

package.json

@@ -6,6 +6,7 @@ import {
     DeclarationReflection,
     ReferenceType,
     ReflectionKind,
+    makeRecursiveVisitor,
 } from "typedoc";
 
 export function load(app: Application) {
@@ -14,11 +15,20 @@ export function load(app: Application) {
 
     app.converter.on(Converter.EVENT_CREATE_DECLARATION, onCreateDeclaration);
     app.converter.on(Converter.EVENT_END, (context: Context) => {
+        const typeCleanup = makeRecursiveVisitor({
+            reflection: (type) => {
+                context.project.removeReflection(type.declaration);
+            },
+        });
+
         for (const [inferredType, refOrig] of schemaTypes) {
             if (
                 refOrig.reflection instanceof DeclarationReflection &&
                 refOrig.reflection.type instanceof ReferenceType
             ) {
+                refOrig.reflection.type.typeArguments?.forEach((t) =>
+                    t.visit(typeCleanup),
+                );
                 refOrig.reflection.type.typeArguments = [
                     ReferenceType.createResolvedReference(
                         inferredType.name,
@@ -29,6 +39,9 @@ export function load(app: Application) {
             }
         }
 
+        console.log(context.project.getReflectionById(35)?.getFullName());
+        console.log(context.project.getReflectionById(70)?.getFullName());
+
         schemaTypes.clear();
     });
 
@@ -57,6 +70,13 @@ export function load(app: Application) {
         if (!declaration) return;
 
         const type = context.getTypeAtLocation(declaration);
+        refl.type.visit(
+            makeRecursiveVisitor({
+                reflection: (type) => {
+                    context.project.removeReflection(type.declaration);
+                },
+            }),
+        );
         refl.type = context.converter.convertType(context, type);
 
         if (originalRef) {

src/plugin.ts

@@ -113,3 +113,21 @@ test("Schemas which have multiple declarations, #2", () => {
           Variable Foo: ZodObject<Foo>
     `);
 });
+
+test("Serialized/deserialized projects do not create warnings, #6", () => {
+    const project = convert("gh6.ts");
+    const ser = app.serializer.projectToObject(project, process.cwd());
+    app.deserializer.reviveProject(ser, "gh6", process.cwd(), project.files);
+
+    expect(project.toStringHierarchy()).toBe(outdent`
+        Project typedoc-plugin-zod
+          TypeAlias Foo: Object
+            TypeLiteral __type
+              Property a: string
+              Property b: number
+              Property c: unknown
+          Variable Foo: ZodObject<Foo>
+    `);
+
+    expect(app.logger.hasWarnings()).toBe(false);
+});

src/test/plugin.test.mts

@@ -0,0 +1,9 @@
+import { number, object, string, TypeOf, unknown } from "zod";
+
+export const Foo = object({
+    a: string(),
+    b: number(),
+    c: unknown(),
+});
+
+export type Foo = TypeOf<typeof Foo>;