PR Action Compiling Demo Apps

[kraftp]

To more easily detect breaking interface changes, we should add a PR hook attempting to compile the demo apps (https://github.com/dbos-inc/dbos-demo-apps) with any new PR. Failures to compile indicate breaking changes that should be fixed.

[kraftp]

@demetris-manikas We'll work on this to help automatically catch future breaking changes

[demetris-manikas]

I saw a video about implicit tests that maybe a simpler and safer solution than relying in what the test apps use.

I did a small POC that seems to work.
I created a public_interfaces_v1.test containing

import { Knex } from "knex";
import { TransactionContext, WorkflowContext, CommunicatorContext } from "../src";
import { Communicator, Transaction, Workflow } from '../src/decorators'

class _DBOSClass {
  @Transaction()
  transaction(_ctxt: TransactionContext<Knex>, _a: string, _b: number) {
    return Promise.resolve(false)
  }
  @Workflow()
  workflow(_ctxt: WorkflowContext, _a: string, _b: number) {
    return Promise.resolve(false)
  }

  @Communicator()
  Communicator(_ctxt: CommunicatorContext, _a: string, _b: number) {
    return Promise.resolve(false)
  }
}

describe("vi-interfaces", () => {
  test("all", () => { expect(true).toBe(true)});
});

Which will choke when I say change the TransactionContext's signature in all the code.

Changes of signatures in this file should be a red flag.

Just an idea