fix(SecureTemplater): implement PR suggestions

Signed-off-by: Justin Bryant <justintbry@gmail.com>

plugins/scaffolder-backend/src/lib/templating/SecureTemplater.ts

@@ -222,8 +222,12 @@ export class SecureTemplater {
       render,
       dispose: () => {
         if (context && !isolate.isDisposed) {
-          context.release();
-          isolate.dispose();
+          try {
+            context.release();
+            isolate.dispose();
+          } catch (error) {
+            // Ignore errors during dispose, as there's not much we can do about it
+          }
         }
       },
     };

plugins/scaffolder-backend/src/scaffolder/actions/builtin/fetch/templateFileActionHandler.ts

@@ -21,10 +21,8 @@ import {
   TemplateGlobal,
 } from '@backstage/plugin-scaffolder-node';
 import fs from 'fs-extra';
-import { createDefaultFilters } from '../../../../lib/templating/filters/createDefaultFilters';
-import { convertFiltersToRecord } from '../../../../util/templating';
-import { resolveSafeChildPath } from '@backstage/backend-plugin-api';
 import path from 'node:path';
+import { createDefaultFilters } from '../../../../lib/templating/filters/createDefaultFilters';
 import { SecureTemplater } from '../../../../lib/templating/SecureTemplater';
 import { convertFiltersToRecord } from '../../../../util/templating';