test: linting flows

packages/eslint-config/eslint-config.cjs

@@ -1,4 +1,16 @@
 /* eslint-env node */
+
+const orchestrationFlowRestrictions = [
+  {
+    selector: "Identifier[name='heapVowE']",
+    message: 'Eventual send is not yet supported within an orchestration flow',
+  },
+  {
+    selector: "Identifier[name='E']",
+    message: 'Eventual send is not yet supported within an orchestration flow',
+  },
+];
+
 module.exports = {
   extends: [
     'airbnb-base',
@@ -95,5 +107,12 @@ module.exports = {
         'no-unused-vars': 'off',
       },
     },
+    {
+      // Orchestration flows
+      files: ['**/*.flows.js'],
+      rules: {
+        'no-restricted-syntax': ['error', ...orchestrationFlowRestrictions],
+      },
+    },
   ],
 };

packages/orchestration/test/examples/bad.flows.js

@@ -0,0 +1,24 @@
+/**
+ * @file Module with linting errors, to verify the linting config detects them.
+ *   This assumes `reportUnusedDisableDirectives: "error" in the local config.
+ */
+
+// TODO error on exports that:
+// - aren't hardened (probably a new rule in @endo/eslint-plugin )
+// - don't satisfy orchestration flow type
+
+// eslint-disable-next-line no-restricted-syntax -- intentional for test
+import { E } from '@endo/far';
+
+export function notFlow() {
+  console.log('This function is not a flow');
+}
+
+export async function notHardened() {
+  console.log('This function is the most minimal flow, but it’s not hardened');
+}
+
+export async function usesE(orch, { someEref }) {
+  // eslint-disable-next-line no-restricted-syntax -- intentional for test
+  await E(someEref).foo();
+}