fix: resolve import collisions for interfaces (#300)

Fixes #298

integration/avoid-import-conflicts-types-only/parameters.txt

@@ -0,0 +1 @@
+outputJsonMethods=false,outputEncodeMethods=false,outputClientImpl=false,nestJs=false

integration/avoid-import-conflicts-types-only/simple-test.ts

@@ -0,0 +1,13 @@
+import { Simple } from './simple';
+
+describe('Simple', () => {
+  it('type checking works correctly', () => {
+    const simple: Simple = {
+      name: 'foo',
+      otherSimple: {
+        simple2Name: 'bar',
+        simple2Age: 1,
+      },
+    }
+  });
+})

integration/avoid-import-conflicts-types-only/simple.proto

@@ -0,0 +1,8 @@
+syntax = "proto3";
+package simple;
+import "simple2.proto";
+
+message Simple {
+  string name = 1;
+  simple2.Simple otherSimple = 2;
+}

integration/avoid-import-conflicts-types-only/simple.ts

@@ -0,0 +1,9 @@
+/* eslint-disable */
+import { Simple as Simple1 } from './simple2';
+
+export const protobufPackage = 'simple';
+
+export interface Simple {
+  name: string;
+  otherSimple: Simple1 | undefined;
+}

integration/avoid-import-conflicts-types-only/simple2.proto

@@ -0,0 +1,8 @@
+syntax = "proto3";
+package simple2;
+
+message Simple {
+  string simple2Name = 1;
+  int32 simple2Age = 2;
+}
+

integration/avoid-import-conflicts-types-only/simple2.ts

@@ -0,0 +1,7 @@
+/* eslint-disable */
+export const protobufPackage = 'simple2';
+
+export interface Simple {
+  simple2Name: string;
+  simple2Age: number;
+}

src/main.ts

@@ -498,7 +498,8 @@ function generateInterfaceDeclaration(
   const chunks: Code[] = [];
 
   maybeAddComment(sourceInfo, chunks, messageDesc.options?.deprecated);
-  chunks.push(code`export interface ${fullName} {`);
+  // interface name should be defined to avoid import collisions
+  chunks.push(code`export interface ${def(fullName)} {`);
 
   if (ctx.options.outputTypeRegistry) {
     chunks.push(code`$type: '${fullTypeName}',`);