@travetto/manifest 7.0.0-rc.2 → 7.0.0-rc.3

package/README.md

@@ -43,7 +43,7 @@ Once the manifest is created, the application runtime can now read this manifest
    *  Providing contextual information when provided a filename, import name, etc (e.g. logging, testing output)
 
 ## Path Normalization
-By default, all paths within the framework are assumed to be in a POSIX style, and all input paths are converted to the POSIX style.  This works appropriately within a Unix and a Windows environment.  This module offers up [path](https://github.com/travetto/travetto/tree/main/module/manifest/src/path.ts#L9) as an equivalent to [Node](https://nodejs.org)'s [path](https://nodejs.org/api/path.html) library.  This allows for consistent behavior across all file-interactions.
+By default, all paths within the framework are assumed to be in a POSIX style, and all input paths are converted to the POSIX style.  This works appropriately within a Unix and a Windows environment.  This module offers up [path](https://github.com/travetto/travetto/tree/main/module/manifest/src/path.ts#L46) as an equivalent to [Node](https://nodejs.org)'s [path](https://nodejs.org/api/path.html) library.  This allows for consistent behavior across all file-interactions.
 
 ## Anatomy of a Manifest
 
@@ -56,7 +56,6 @@ By default, all paths within the framework are assumed to be in a POSIX style, a
     "path": "<generated>",
     "mono": true,
     "manager": "npm",
-    "type": "commonjs",
     "defaultEnv": "local"
   },
   "build": {
@@ -132,7 +131,6 @@ The general context describes the project-space and any important information fo
 
 The context contains:
    *  A generated timestamp
-   *  Module Type: `commonjs`([CommonJS](https://nodejs.org/api/modules.html)) or `module`([Ecmascript Module](https://nodejs.org/api/esm.html))
    *  The main module to execute. (*This primarily pertains to mono-repo support when there are multiple modules in the project*)
    *  The root path of the project/workspace
    *  Whether or not the project is a mono-repo. (*This is determined by using the 'workspaces' field in your [Package JSON](https://docs.npmjs.com/cli/v9/configuring-npm/package-json)*)

package/__index__.ts

@@ -9,4 +9,4 @@ export * from './src/types/context.ts';
 export * from './src/types/package.ts';
 export * from './src/types/manifest.ts';
 export * from './src/types/common.ts';
-export { path } from './src/path.ts';
+export { default as path } from './src/path.ts';

package/package.json

@@ -1,6 +1,7 @@
 {
   "name": "@travetto/manifest",
-  "version": "7.0.0-rc.2",
+  "version": "7.0.0-rc.3",
+  "type": "module",
   "description": "Support for project indexing, manifesting, along with file watching",
   "keywords": [
     "path",

package/src/context.ts

@@ -53,13 +53,17 @@ export function getManifestContext(root: string = process.cwd()): ManifestContex
     readPackage(resolve(`${process.env.TRV_MODULE}/package.json`)) :
     findPackage(root, pkg => !!pkg) ?? workspace;
 
+  if (workspace.type !== 'module') {
+    console.error('ERROR: Only ESM modules are supported, package.json must be of type module');
+    process.exit(1);
+  }
+
   return {
     workspace: {
       name: workspace.name ?? 'untitled',
       path: workspace.path,
       mono: !!workspace.workspaces,
       manager: existsSync(path.resolve(workspace.path, 'yarn.lock')) ? 'yarn' : 'npm',
-      type: workspace.type ?? 'commonjs',
       defaultEnv: workspace.travetto?.defaultEnv ?? 'local'
     },
     build: {

package/src/delta.ts

@@ -1,7 +1,7 @@
 import fs from 'node:fs/promises';
 
 import { ManifestModuleUtil } from './module.ts';
-import { path } from './path.ts';
+import path from './path.ts';
 
 import type { ManifestModule, ManifestModuleCore, ManifestModuleFile, ManifestRoot } from './types/manifest.ts';
 import type { ChangeEventType, ManifestModuleFileType, ManifestModuleFolderType } from './types/common.ts';

package/src/dependencies.ts

@@ -1,5 +1,5 @@
 import { PackageUtil } from './package.ts';
-import { path } from './path.ts';
+import path from './path.ts';
 
 import type { Package, PackageDependencyType } from './types/package.ts';
 import type { ManifestContext } from './types/context.ts';

package/src/file.ts

@@ -2,7 +2,7 @@ import fs from 'node:fs/promises';
 import { readFileSync } from 'node:fs';
 import os from 'node:os';
 
-import { path } from './path.ts';
+import path from './path.ts';
 
 export class ManifestFileUtil {
   /**

package/src/manifest-index.ts

@@ -1,7 +1,7 @@
 import { existsSync } from 'node:fs';
 
 import { ManifestModuleUtil } from './module.ts';
-import { path } from './path.ts';
+import path from './path.ts';
 import { ManifestUtil } from './util.ts';
 
 import type { ManifestModule, ManifestRoot, ManifestModuleFile, IndexedModule, IndexedFile, FindConfig } from './types/manifest.ts';

package/src/module.ts

@@ -1,6 +1,6 @@
 import fs from 'node:fs/promises';
 
-import { path } from './path.ts';
+import path from './path.ts';
 import { PackageModuleVisitor } from './dependencies.ts';
 
 import type { ManifestModuleFileType, ManifestModuleRole, ManifestModuleFolderType } from './types/common.ts';
@@ -18,7 +18,7 @@ const EXT_MAPPING: Record<string, ManifestModuleFileType> = {
 };
 
 const INDEX_FILES = new Set(
-  ['__index__', '__index', 'index', 'jsx-runtime'].flatMap(file =>
+  ['__index__', '__index', 'index'].flatMap(file =>
     ['ts', 'tsx', 'js'].map(ext => `${file}.${ext}`)
   )
 );

package/src/package.ts

@@ -2,7 +2,7 @@ import { createRequire } from 'node:module';
 import { execSync } from 'node:child_process';
 import { existsSync } from 'node:fs';
 
-import { path } from './path.ts';
+import path from './path.ts';
 import { ManifestFileUtil } from './file.ts';
 
 import { PackagePathSymbol, type Package, type PackageWorkspaceEntry } from './types/package.ts';

package/src/path.ts

@@ -6,8 +6,8 @@ const toPosix = (file: string): string => file.replaceAll('\\', '/');
 const toNative = (file: string): string => file.replace(/[\/]/g, native.sep);
 const cwd = (): string => toPosix(process.cwd());
 
-export const path: posix.PlatformPath & {
-  native: posix.PlatformPath;
+const path: typeof posix & {
+  native: typeof posix;
   toPosix: typeof toPosix;
   toNative: typeof toNative;
 } = {

package/src/types/common.ts

@@ -1,4 +1,3 @@
-export type NodeModuleType = 'module' | 'commonjs';
 export type NodePackageManager = 'yarn' | 'npm';
 
 export type ManifestModuleFileType = 'typings' | 'ts' | 'js' | 'json' | 'package-json' | 'unknown' | 'fixture' | 'md';

package/src/types/context.ts

@@ -1,4 +1,4 @@
-import type { NodeModuleType, NodePackageManager } from './common.ts';
+import type { NodePackageManager } from './common.ts';
 
 export type ManifestContext = {
   workspace: {
@@ -8,8 +8,6 @@ export type ManifestContext = {
     name: string;
     /** Is the workspace a monorepo? */
     mono?: boolean;
-    /** The module type of the workspace */
-    type: NodeModuleType;
     /** The package manager of the workspace */
     manager: NodePackageManager;
     /** The default env name */

package/src/types/package.ts

@@ -1,4 +1,4 @@
-import type { ManifestModuleRole, NodeModuleType } from './common.ts';
+import type { ManifestModuleRole } from './common.ts';
 import type { ManifestContext } from './context.ts';
 
 export const PackagePathSymbol = Symbol.for('@travetto/manifest:package-path');
@@ -6,7 +6,7 @@ export const PackagePathSymbol = Symbol.for('@travetto/manifest:package-path');
 export type Package = {
   [PackagePathSymbol]?: string;
   name: string;
-  type?: NodeModuleType;
+  type?: string;
   version: string;
   description?: string;
   license?: string;

package/src/util.ts

@@ -1,4 +1,4 @@
-import { path } from './path.ts';
+import path from './path.ts';
 import { ManifestModuleUtil } from './module.ts';
 import { ManifestFileUtil } from './file.ts';
 import { PackageUtil } from './package.ts';