fix: move external imports to top of file

Author: constewart9

.github/workflows/types-codegen-pr.yaml

@@ -6,7 +6,6 @@ on:
       - main
     paths:
       - 'types/codegen/schema/**'
-  workflow_dispatch:
 
 jobs:
   validate-pr:
@@ -21,7 +20,6 @@ jobs:
         uses: actions/setup-node@v4
         with:
           node-version: '20.x'
-          #should we cache: 'npm'
 
       - name: Install root dependencies
         working-directory: .

types/codegen/custom-templates/typescript/index.mustache

@@ -1,5 +1,3 @@
-/* tslint:disable */
-/* eslint-disable */
 {{!This is a comment. Removed the export of runtimes because we do not need 
 those functions. Was causing error because exported a def called ResponseError
 which is also used by vscode LSP}}

types/codegen/scripts/post-typescript.js

@@ -78,12 +78,19 @@ const importResult = processImportMappings()
 // Read the generated index.ts file
 let indexContent = fs.readFileSync(indexPath, 'utf8')
 
-// Find the position after existing imports
-const importEndPos = indexContent.lastIndexOf('import')
+// Find the insertion position - either at top of file or after lint disable comments
 let insertPos = 0
 
-if (importEndPos !== -1) {
-    insertPos = indexContent.indexOf('\n', importEndPos) + 1
+// Check if file starts with lint disable comments
+const lintDisablePattern = /^(\s*\/\*\s*tslint:disable\s*\*\/\s*\/\*\s*eslint-disable\s*\*\/\s*)/
+const lintDisableMatch = indexContent.match(lintDisablePattern)
+
+if (lintDisableMatch) {
+    // Insert after the lint disable comments
+    insertPos = lintDisableMatch[0].length
+} else {
+    // Insert at the very top of the file
+    insertPos = 0
 }
 
 // Insert import statements and constants after existing imports