Trim down package contents for publish. Ensure right exports. Fixing version specifications. (#3)

Author: rlandav

.gitignore

@@ -2,4 +2,5 @@
 lib
 dist
 temp
-node_modules
+node_modules
+*.tgz

change/@minecraft-core-build-tasks-1b4eb3a5-4a66-411f-a888-a12cfe2d1de9.json

@@ -0,0 +1,7 @@
+{
+  "type": "patch",
+  "comment": "Trim down package contents for publish. Ensure right exports. Fixing version specifications.",
+  "packageName": "@minecraft/core-build-tasks",
+  "email": "[email protected]",
+  "dependentChangeType": "patch"
+}

change/@minecraft-math-01e5a3f9-5d48-403c-8a89-2c249664b58c.json

@@ -0,0 +1,7 @@
+{
+  "type": "patch",
+  "comment": "Trim down package contents for publish. Ensure right exports. Fixing version specifications.",
+  "packageName": "@minecraft/math",
+  "email": "[email protected]",
+  "dependentChangeType": "patch"
+}

change/eslint-plugin-minecraft-linting-6e1e6537-ed89-434c-a0e8-86c8e1c7707e.json

@@ -0,0 +1,7 @@
+{
+  "type": "patch",
+  "comment": "Trim down package contents for publish. Ensure right exports. Fixing version specifications.",
+  "packageName": "eslint-plugin-minecraft-linting",
+  "email": "[email protected]",
+  "dependentChangeType": "patch"
+}

libraries/math/api-extractor.json

@@ -2,6 +2,6 @@
  * Config file for API Extractor.  For more info, please visit: https://api-extractor.com
  */
 {
-  "$schema": "https://developer.microsoft.com/json-schemas/api-extractor/v7/api-extractor.schema.json",
-  "extends": "@minecraft/api-extractor-base/api-extractor-base.json"
+    "$schema": "https://developer.microsoft.com/json-schemas/api-extractor/v7/api-extractor.schema.json",
+    "extends": "@minecraft/api-extractor-base/api-extractor-base.json"
 }

libraries/math/api-report/math.api.md

@@ -19,6 +19,9 @@ export function clamp(v: Vector3, limits?: {
     max?: Partial<Vector3>;
 }): Vector3;
 
+// @public
+export function clamp_number(val: number, min: number, max: number): number;
+
 // @public
 export function cross(a: Vector3, b: Vector3): Vector3;
 

libraries/math/just.config.ts

@@ -1,6 +1,12 @@
 import { execSync } from 'child_process';
 import { argv, series, task, tscTask } from 'just-scripts';
-import { apiExtractorTask, cleanTask, coreLint, vitestTask } from '@minecraft/core-build-tasks';
+import {
+    DEFAULT_CLEAN_DIRECTORIES,
+    apiExtractorTask,
+    cleanTask,
+    coreLint,
+    vitestTask,
+} from '@minecraft/core-build-tasks';
 
 const isOnlyBuild = argv()._.findIndex(arg => arg === 'test') === -1;
 
@@ -23,4 +29,4 @@ task('vitest', vitestTask());
 task('test', series('api-extractor-validate', 'vitest'));
 
 // Clean
-task('clean', cleanTask([]));
+task('clean', cleanTask(DEFAULT_CLEAN_DIRECTORIES));

libraries/math/package.json

@@ -3,14 +3,20 @@
     "version": "0.0.1",
     "author": "Raphael Landaverde ([email protected])",
     "description": "Math utilities for use with minecraft scripting modules",
-    "main": "index.js",
-    "types": "types/minecraft-math-public.d.ts",
+    "main": "lib/index.js",
+    "types": "lib/types/math-public.d.ts",
     "scripts": {
         "build": "just build",
         "lint": "just lint",
-        "test": "just test"
+        "test": "just test",
+        "clean": "just clean"
     },
     "license": "MIT",
+    "files": [
+        "dist",
+        "lib",
+        "api-report"
+    ],
     "dependencies": {
         "@minecraft/server": "^1.6.0"
     },

libraries/math/src/general/index.ts

@@ -0,0 +1,3 @@
+// Copyright (c) Mojang AB.  All rights reserved.
+
+export * from './clamp';

libraries/math/src/index.ts

@@ -1,3 +1,4 @@
 // Copyright (c) Mojang AB.  All rights reserved.
 
 export * from './vector3';
+export * from './general';

libraries/math/src/vector3/coreHelpers.ts

@@ -1,6 +1,6 @@
 // Copyright (c) Mojang AB.  All rights reserved.
 
-import { Vector2, Vector3 } from '@minecraft/server';
+import type { Vector2, Vector3 } from '@minecraft/server';
 import { clamp_number } from '../general/clamp';
 
 /**

libraries/math/src/vector3/vectorWrapper.test.ts

@@ -1,6 +1,6 @@
 // Copyright (c) Mojang AB.  All rights reserved.
 
-import { Vector3 } from '@minecraft/server';
+import type { Vector3 } from '@minecraft/server';
 import { describe, expect, it } from 'vitest';
 import {
     add,

libraries/math/src/vector3/vectorWrapper.ts

@@ -1,6 +1,6 @@
 // Copyright (c) Mojang AB.  All rights reserved.
 
-import { Vector2, Vector3 } from '@minecraft/server';
+import type { Vector2, Vector3 } from '@minecraft/server';
 import {
     add,
     clamp,

libraries/math/tsconfig.json

@@ -3,6 +3,7 @@
     "include": ["src/**/*"],
     "exclude": ["dist", "build", "node_modules"],
     "compilerOptions": {
-        "outDir": "lib"
+        "outDir": "lib",
+        "declarationDir": "temp/types"
     }
 }

package-lock.json

@@ -15,6 +15,7 @@
         "build": "turbo run build",
         "test": "turbo run test",
         "lint": "turbo run lint",
+        "clean": "turbo run clean",
         "change": "beachball change",
         "release": "beachball publish --access public"
     },

package.json

@@ -14,7 +14,7 @@
    *
    * SUPPORTED TOKENS: <projectFolder>, <packageName>, <unscopedPackageName>
    */
-  "mainEntryPointFilePath": "<projectFolder>/lib/index.d.ts",
+  "mainEntryPointFilePath": "<projectFolder>/temp/types/index.d.ts",
   /**
    * A list of NPM package names whose exports should be treated as part of this package.
    *

tools/api-extractor-base/api-extractor-base.json

@@ -10,6 +10,9 @@
         "clean-tools": "just-scripts clean-tools",
         "test": "just-scripts test"
     },
+    "files": [
+        "lib"
+    ],
     "dependencies": {
         "@rushstack/node-core-library": "^3.59.6",
         "@microsoft/api-extractor": "^7.38.3",

tools/core-build-tasks/package.json

@@ -11,6 +11,9 @@
         "test": "just-scripts test",
         "just": "just-scripts"
     },
+    "files": [
+        "lib"
+    ],
     "dependencies": {
         "@minecraft/core-build-tasks": "*",
         "@types/node": "^20.0.0",

tools/eslint-plugin-minecraft-linting/package.json

@@ -4,6 +4,7 @@
     "exclude": ["dist", "build", "node_modules"],
     "compilerOptions": {
         "outDir": "lib",
+        "declarationDir": "lib",
         "moduleResolution": "Node10",
         "module": "CommonJS",
         "isolatedModules": false

tools/eslint-plugin-minecraft-linting/tsconfig.json

@@ -4,7 +4,6 @@
     "compilerOptions": {
         "composite": false,
         "declaration": true,
-        "declarationMap": true,
         "esModuleInterop": true,
         "forceConsistentCasingInFileNames": true,
         "inlineSources": false,