@ossy/app 1.36.0 → 1.37.0

package/cli/build.task.js

@@ -20,10 +20,11 @@ import getPlatformFiles, {
   STARTUP_FILE_PATTERN,
   EMAIL_FILE_PATTERN,
   ACTION_FILE_PATTERN,
+  E2E_FILE_PATTERN,
 } from './get-platform-files.task.js'
 import { manifestPlugin } from './manifest-plugin.js'
 
-export { PAGE_FILE_PATTERN, API_FILE_PATTERN, TASK_FILE_PATTERN, RESOURCE_FILE_PATTERN, COMPONENT_FILE_PATTERN, AGGREGATE_FILE_PATTERN, INTEGRATION_FILE_PATTERN, STARTUP_FILE_PATTERN, EMAIL_FILE_PATTERN, ACTION_FILE_PATTERN }
+export { PAGE_FILE_PATTERN, API_FILE_PATTERN, TASK_FILE_PATTERN, RESOURCE_FILE_PATTERN, COMPONENT_FILE_PATTERN, AGGREGATE_FILE_PATTERN, INTEGRATION_FILE_PATTERN, STARTUP_FILE_PATTERN, EMAIL_FILE_PATTERN, ACTION_FILE_PATTERN, E2E_FILE_PATTERN }
 
 // Generated entry stubs live under `build/.ossy/entries/`. Putting them
 // inside `build/` means they're cleaned automatically with every build,
@@ -117,9 +118,7 @@ function generateTaskStub ({ stubAbs, sourceAbs }) {
   const importPath = relImport(stubAbs, sourceAbs)
   return [
     '// Generated by @ossy/app — do not edit',
-    `import * as _task from '${importPath}'`,
-    'export const metadata = _task.metadata',
-    'export const run = _task.run ?? _task.default',
+    `export * from '${importPath}'`,
     '',
   ].join('\n')
 }
@@ -207,6 +206,18 @@ function generateActionStub ({ stubAbs, sourceAbs }) {
   ].join('\n')
 }
 
+// E2e tests are Playwright test modules. The stub re-exports `id`, `feature`,
+// `requires`, and the default test function so the manifest plugin can validate
+// them and the e2e runner can import and execute them.
+function generateE2eStub ({ stubAbs, sourceAbs }) {
+  const importPath = relImport(stubAbs, sourceAbs)
+  return [
+    '// Generated by @ossy/app — do not edit',
+    `export { id, feature, requires, default } from '${importPath}'`,
+    '',
+  ].join('\n')
+}
+
 function stubFor (kind, args) {
   if (kind === 'page') return generatePageStub(args)
   if (kind === 'api') return generateApiStub(args)
@@ -217,6 +228,7 @@ function stubFor (kind, args) {
   if (kind === 'startup') return generateStartupStub(args)
   if (kind === 'email') return generateEmailStub(args)
   if (kind === 'action') return generateActionStub(args)
+  if (kind === 'e2e') return generateE2eStub(args)
   return generateTaskStub(args)
 }
 
@@ -263,6 +275,7 @@ export async function build (cliArgs = []) {
     ...platformFiles.startups,
     ...platformFiles.emails,
     ...platformFiles.actions,
+    ...platformFiles.e2es,
   ]
 
   const entriesByStub = new Map()
@@ -297,7 +310,20 @@ export async function build (cliArgs = []) {
     // bundled task files at build time — bundling sharp would trigger its binary
     // loader during the build, which fails on linux-arm64 builders that don't
     // have the darwin binaries installed.
-    external: (id) => id.startsWith('node:') || NODE_BUILTINS.has(id) || id === 'sharp' || id.startsWith('@img/sharp-'),
+    // Singleton packages must resolve to the already-loaded module in the
+    // Node.js module cache at runtime, not to a bundled copy with its own
+    // empty Map/state. Keeping them external means dynamic `import()`s of
+    // *.api.js / *.action.js bundles share the same ActionService instance
+    // that the server registered actions into at startup.
+    external: (id) => {
+      if (id.startsWith('node:') || NODE_BUILTINS.has(id)) return true
+      if (id === 'sharp' || id.startsWith('@img/sharp-')) return true
+      // Singleton packages — must not be bundled into entry chunks
+      if (id === '@ossy/platform' || id.startsWith('@ossy/platform/')) return true
+      if (id === '@ossy/event-store' || id.startsWith('@ossy/event-store/')) return true
+      if (id === '@ossy/observability' || id.startsWith('@ossy/observability/')) return true
+      return false
+    },
     plugins: [
       replace({
         preventAssignment: true,

package/cli/get-platform-files.task.js

@@ -11,9 +11,10 @@ export const INTEGRATION_FILE_PATTERN = /\.integration\.(mjs|cjs|js)$/
 export const STARTUP_FILE_PATTERN = /\.startup\.(mjs|cjs|js)$/
 export const EMAIL_FILE_PATTERN = /\.email\.(jsx?|tsx?)$/
 export const ACTION_FILE_PATTERN = /\.action\.(mjs|cjs|js)$/
+export const E2E_FILE_PATTERN = /\.e2e\.(mjs|cjs|js)$/
 
 /**
- * @typedef {'page' | 'api' | 'task' | 'resource' | 'component' | 'aggregate' | 'integration' | 'startup' | 'email' | 'action'} EntryKind
+ * @typedef {'page' | 'api' | 'task' | 'resource' | 'component' | 'aggregate' | 'integration' | 'startup' | 'email' | 'action' | 'e2e'} EntryKind
  *
  * @typedef {object} PlatformEntry
  * @property {EntryKind} kind          Which bucket this entry lives in.
@@ -32,6 +33,7 @@ export const ACTION_FILE_PATTERN = /\.action\.(mjs|cjs|js)$/
  * @property {PlatformEntry[]} startups      Discovered `*.startup.{js,mjs,cjs}` entries.
  * @property {PlatformEntry[]} emails        Discovered `*.email.{jsx,tsx,...}` entries.
  * @property {PlatformEntry[]} actions       Discovered `*.action.{js,mjs,cjs}` entries.
+ * @property {PlatformEntry[]} e2es          Discovered `*.e2e.{js,mjs,cjs}` entries (installed packages only).
  */
 export function discoverFilesByPattern (srcDir, filePattern) {
   const dir = path.resolve(srcDir)
@@ -60,6 +62,7 @@ function classifyFile (absPath) {
   if (STARTUP_FILE_PATTERN.test(base)) return 'startup'
   if (EMAIL_FILE_PATTERN.test(base)) return 'email'
   if (ACTION_FILE_PATTERN.test(base)) return 'action'
+  if (E2E_FILE_PATTERN.test(base)) return 'e2e'
   return null
 }
 
@@ -134,6 +137,7 @@ export function discoverInstalledPackageEntries (projectRoot) {
       ...discoverFilesByPattern(packageSrcDir, STARTUP_FILE_PATTERN),
       ...discoverFilesByPattern(packageSrcDir, EMAIL_FILE_PATTERN),
       ...discoverFilesByPattern(packageSrcDir, ACTION_FILE_PATTERN),
+      ...discoverFilesByPattern(packageSrcDir, E2E_FILE_PATTERN),
     ]
     for (const sourcePath of files) {
       const stat = fs.statSync(sourcePath)
@@ -177,12 +181,13 @@ export function discoverInstalledPackageEntries (projectRoot) {
  * @returns {Promise<PlatformFiles>}
  */
 export default async function getPlatformFiles (srcDir) {
-  const out = { pages: [], apis: [], tasks: [], resources: [], components: [], aggregates: [], integrations: [], startups: [], emails: [], actions: [] }
-  const bucketByKind = { page: 'pages', api: 'apis', task: 'tasks', resource: 'resources', component: 'components', aggregate: 'aggregates', integration: 'integrations', startup: 'startups', email: 'emails', action: 'actions' }
+  const out = { pages: [], apis: [], tasks: [], resources: [], components: [], aggregates: [], integrations: [], startups: [], emails: [], actions: [], e2es: [] }
+  const bucketByKind = { page: 'pages', api: 'apis', task: 'tasks', resource: 'resources', component: 'components', aggregate: 'aggregates', integration: 'integrations', startup: 'startups', email: 'emails', action: 'actions', e2e: 'e2es' }
 
-  // Aggregates are only discovered from installed packages — never from the
-  // local `src/` — to avoid pulling in per-project duplicates of the same
-  // aggregate class that is canonically owned by the feature package.
+  // Aggregates and e2e tests are only discovered from installed packages — never
+  // from the local `src/` — to avoid pulling in per-project duplicates of the
+  // same aggregate class or test suite that is canonically owned by the feature
+  // package.
   const localFiles = [
     ...discoverFilesByPattern(srcDir, PAGE_FILE_PATTERN),
     ...discoverFilesByPattern(srcDir, API_FILE_PATTERN),
@@ -198,7 +203,7 @@ export default async function getPlatformFiles (srcDir) {
     const stat = fs.statSync(sourcePath)
     if (!stat.isFile() || stat.size === 0) continue
     const kind = classifyFile(sourcePath)
-    if (!kind || kind === 'aggregate') continue
+    if (!kind || kind === 'aggregate' || kind === 'e2e') continue
     out[bucketByKind[kind]].push({ kind, sourcePath })
   }
 

package/cli/manifest-plugin.js

@@ -64,6 +64,12 @@ import { metadataIdFromFile, defaultPageRoute } from './get-platform-files.task.
  * @property {string} entry   URL the platform serves the action bundle from.
  * @property {string} access  Access level: `'public'` | `'authenticated'` | `'workspace'`.
  *
+ * @typedef {object} E2eManifestEntry
+ * @property {string} id       Unique test id (e.g. `'authentication/sign-in'`).
+ * @property {string} feature  Grouping label (e.g. `'authentication'`).
+ * @property {string[]} requires  Runtime dependencies needed (e.g. `['server', 'database']`).
+ * @property {string} entry    URL the platform serves the e2e bundle from.
+ *
  * @typedef {object} Manifest
  * @property {ManifestEntry[]} entries              Flat, ordered list of every routable thing.
  * @property {ComponentManifestEntry[]} components  Discovered `*.component.*` entries, keyed by id.
@@ -73,6 +79,7 @@ import { metadataIdFromFile, defaultPageRoute } from './get-platform-files.task.
  * @property {StartupManifestEntry[]} startups      Discovered `*.startup.js` entries.
  * @property {EmailManifestEntry[]} emails          Discovered `*.email.{jsx,tsx}` entries.
  * @property {ActionManifestEntry[]} actions        Discovered `*.action.js` entries.
+ * @property {E2eManifestEntry[]} e2es              Discovered `*.e2e.js` entries from installed packages.
  * @property {object} config                        Inlined `src/config.js` default export.
  */
 
@@ -137,6 +144,9 @@ export function manifestPlugin ({ entriesByStub, srcDir, staticOutDir, configVal
       const seenStartupIds = new Set()
       const seenEmailIds = new Set()
       const seenActionIds = new Set()
+      /** @type {E2eManifestEntry[]} */
+      const e2es = []
+      const seenE2eIds = new Set()
 
       for (const fileName of Object.keys(bundle)) {
         const chunk = bundle[fileName]
@@ -286,6 +296,39 @@ export function manifestPlugin ({ entriesByStub, srcDir, staticOutDir, configVal
           continue
         }
 
+        // E2e tests contain Playwright test logic. The bundle exports `id`
+        // (a unique namespaced slug), `feature` (grouping label), `requires`
+        // (dependency list), and a default export (the async test function).
+        // They don't produce routable entries; they accumulate into a separate
+        // `e2es` array so test runners can discover and execute them.
+        if (entryInfo.kind === 'e2e') {
+          const e2eId = mod && mod.id
+          const feature = mod && mod.feature
+          const requires = mod && mod.requires
+          const testFn = mod && mod.default
+          if (typeof e2eId !== 'string' || e2eId.trim() === '') {
+            this.error(
+              `[@ossy/app][build] e2e entry ${entryInfo.sourcePath} must export a non-empty string "id"`,
+            )
+          }
+          if (typeof testFn !== 'function') {
+            this.error(
+              `[@ossy/app][build] e2e entry ${entryInfo.sourcePath} must default-export a function`,
+            )
+          }
+          if (seenE2eIds.has(e2eId)) {
+            this.error(`[@ossy/app][build] Duplicate e2e id "${e2eId}"`)
+          }
+          seenE2eIds.add(e2eId)
+          e2es.push({
+            id: e2eId,
+            feature: typeof feature === 'string' ? feature : '',
+            requires: Array.isArray(requires) ? requires : [],
+            entry: url,
+          })
+          continue
+        }
+
         // Resources are pure data — the default export *is* the template.
         // They don't use `metadata`, don't need a derived id, and don't
         // produce a routable manifest entry; they accumulate into a separate
@@ -347,7 +390,7 @@ export function manifestPlugin ({ entriesByStub, srcDir, staticOutDir, configVal
       }
 
       /** @type {Manifest} */
-      const manifest = { entries, components, resourceTemplates, aggregates, integrations, startups, emails, actions, config: configValue || {} }
+      const manifest = { entries, components, resourceTemplates, aggregates, integrations, startups, emails, actions, e2es, config: configValue || {} }
       fs.mkdirSync(path.dirname(manifestPath), { recursive: true })
       fs.writeFileSync(manifestPath, JSON.stringify(manifest, null, 2), 'utf8')
     },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ossy/app",
-  "version": "1.36.0",
+  "version": "1.37.0",
   "description": "",
   "source": "./src/index.js",
   "main": "./src/index.js",
@@ -38,14 +38,14 @@
     "@babel/eslint-parser": "^7.15.8",
     "@babel/preset-react": "^7.26.3",
     "@babel/register": "^7.25.9",
-    "@ossy/design-system": "^1.36.0",
+    "@ossy/design-system": "^1.37.0",
     "@ossy/pages": "^1.23.0",
-    "@ossy/platform": "^1.35.0",
-    "@ossy/router": "^1.36.0",
-    "@ossy/router-react": "^1.36.0",
-    "@ossy/sdk": "^1.36.0",
-    "@ossy/sdk-react": "^1.36.0",
-    "@ossy/themes": "^1.36.0",
+    "@ossy/platform": "^1.36.0",
+    "@ossy/router": "^1.37.0",
+    "@ossy/router-react": "^1.37.0",
+    "@ossy/sdk": "^1.37.0",
+    "@ossy/sdk-react": "^1.37.0",
+    "@ossy/themes": "^1.37.0",
     "@rollup/plugin-alias": "^6.0.0",
     "@rollup/plugin-babel": "^7.0.0",
     "@rollup/plugin-commonjs": "^29.0.0",
@@ -79,5 +79,5 @@
     "README.md",
     "tsconfig.json"
   ],
-  "gitHead": "24df2bde0d5d8794c5a82b9e802a2594ca73a5d9"
+  "gitHead": "6c7724dea1b5eda89e83319dce69703d622aadb1"
 }