fix: useTask's types not available in some consuming apps

.github/workflows/ci.yml

@@ -13,11 +13,23 @@ env:
   CI: true
 
 jobs:
+  build_test:
+    name: Build Tests
+    timeout-minutes: 5
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v2
+      - uses: volta-cli/action@v1
+      - run: yarn install
+      - run: yarn test
+        working-directory: ./testing/build
+
+
   tests:
-    if: "! contains(toJSON(github.event.commits.*.message), '[skip ci]')"
     name: Base Tests
     timeout-minutes: 5
     runs-on: ubuntu-latest
+    needs: build_test
     strategy:
       matrix:
         node:
@@ -59,7 +71,6 @@ jobs:
   #       run: yarn ember test
 
   try-scenarios:
-    if: "! contains(toJSON(github.event.commits.*.message), '[skip ci]')"
     name: "Compatibility"
     timeout-minutes: 7
     runs-on: ubuntu-latest

ember-resources/package.json

@@ -50,8 +50,6 @@
     "@babel/preset-typescript": "7.16.7",
     "@embroider/addon-dev": "1.2.0",
     "@nullvoxpopuli/eslint-configs": "^2.1.1",
-    "@rollup/plugin-babel": "5.3.1",
-    "@rollup/plugin-node-resolve": "^13.0.6",
     "@semantic-release/changelog": "^5.0.0",
     "@semantic-release/git": "^9.0.0",
     "@types/ember__destroyable": "^4.0.0",

ember-resources/rollup.config.js

@@ -19,7 +19,8 @@ export default defineConfig({
     // These are the modules that should get reexported into the traditional
     // "app" tree. Things in here should also be in publicEntrypoints above, but
     // not everything in publicEntrypoints necessarily needs to go here.
-    addon.appReexports([]),
+    // addon.appReexports([]),
+
     // This babel config should *not* apply presets or compile away ES modules.
     // It exists only to provide development niceties for you, like automatic
     // template colocation.
@@ -42,7 +43,7 @@ export default defineConfig({
     addon.dependencies(),
 
     // Ensure that standalone .hbs files are properly integrated as Javascript.
-    addon.hbs(),
+    // addon.hbs(),
 
     // addons are allowed to contain imports of .css files, which we want rollup
     // to leave alone and keep in the published output.

ember-resources/src/index.ts

@@ -15,4 +15,7 @@ export type { ArgsWrapper, Named, Positional } from './-private/types';
 
 // Protected Type Utilities that need to be documented,
 // but hopefully shouldn't be used in consuming apps.
+// These types may *need* to be exported for folks relying on
+// inference
+export type { TaskInstance, TaskIsh } from './-private/resources/ember-concurrency-task';
 export type { Thunk } from './-private/types';

testing/build/expected-output-files.txt

@@ -0,0 +1,3 @@
+index.js
+index.d.ts
+index.js.map

testing/build/package.json

@@ -0,0 +1,16 @@
+{
+  "name": "ember-resources-build-test",
+  "private": true,
+  "version": "0.0.0",
+  "type": "module",
+  "scripts": {
+    "test": "node ./test.js"
+  },
+  "devDependencies": {
+    "chai": "^4.3.6",
+    "execa": "^6.1.0"
+  },
+  "volta": {
+    "extends": "../../package.json"
+  }
+}

testing/build/test.js

@@ -0,0 +1,43 @@
+'use strict';
+
+import fs from 'fs/promises';
+import path from 'path';
+import { fileURLToPath } from 'url';
+import { execa } from 'execa';
+import { assert } from 'chai';
+
+const __dirname = path.dirname(fileURLToPath(import.meta.url));
+const addonPath = path.join(__dirname, '..', '..', 'ember-resources');
+
+async function main() {
+  await build();
+
+  let expected = await getExpected();
+  let actual = await listFiles();
+
+  expected.sort();
+  actual.sort();
+
+  console.debug({ expected, actual });
+
+  assert.deepEqual(actual, expected);
+}
+
+async function build() {
+  await execa('yarn', ['build:js'], { cwd: addonPath, preferLocal: true });
+}
+
+async function getExpected() {
+  let expectedFile = await fs.readFile(path.join(__dirname, 'expected-output-files.txt'));
+  let expected = expectedFile.toString().split('\n');
+
+  return expected.filter(Boolean);
+}
+
+async function listFiles(ofDir = path.join(addonPath, 'dist')) {
+  let read = await fs.readdir(ofDir, { withFileTypes: true });
+
+  return read.filter(Boolean).map((item) => item.name);
+}
+
+main();