misc: better build structure

See #52 (comment)

.gitignore

@@ -63,4 +63,4 @@ typings/
 
 # builds
 
-/build
+/lib

.npmignore

@@ -2,4 +2,6 @@
 .gitattributes
 .eslintrc.js
 test/
-.vscode/
+.vscode/
+src/
+!lib/

package.json

@@ -6,15 +6,16 @@
   "engines": {
     "node": ">=16"
   },
-  "main": "lib/index.ts",
   "scripts": {
     "test": "cross-env NODE_ENV=test ts-mocha test/**/*.ts",
+    "build": "tsc -p tsconfig.json",
+    "build:watch": "tsc -w -p tsconfig.json",
     "semantic-release": "semantic-release"
   },
   "bin": {
-    "openupm": "lib/index.ts",
-    "openupm-cli": "lib/index.ts",
-    "openupm-cn": "lib/index-cn.ts"
+    "openupm": "lib/index.js",
+    "openupm-cli": "lib/index.js",
+    "openupm-cn": "lib/index-cn.js"
   },
   "repository": {
     "type": "git",

lib/index-cn.ts → src/index-cn.ts

@@ -1,4 +1,4 @@
-#!/usr/bin/env -S node -r "ts-node/register"
+#!/usr/bin/env node
 
 if (!process.argv.includes("--cn")) process.argv.push("--cn");
 import "./cli";

src/index.ts

@@ -0,0 +1,3 @@
+#!/usr/bin/env node
+
+import "./cli";

src/types/allow-importing-package-json.d.ts

@@ -0,0 +1,2 @@
+// https://stackoverflow.com/a/61426303
+declare module "*.json";

test/test-cmd-add.ts

@@ -1,9 +1,9 @@
 import "assert";
 import nock from "nock";
 import "should";
-import { loadManifest } from "../lib/core";
+import { loadManifest } from "../src/core";
 
-import { add } from "../lib/cmd-add";
+import { add } from "../src/cmd-add";
 
 import {
   createWorkDir,

test/test-cmd-deps.ts

@@ -2,7 +2,7 @@ import "assert";
 import nock from "nock";
 import "should";
 
-import { deps } from "../lib/cmd-deps";
+import { deps } from "../src/cmd-deps";
 
 import {
   createWorkDir,

test/test-cmd-login.ts

@@ -5,7 +5,7 @@ import {
   generateNpmrcLines,
   getNpmrcPath,
   validateRegistry,
-} from "../lib/cmd-login";
+} from "../src/cmd-login";
 
 describe("cmd-login.ts", function () {
   describe("validateRegistry", function () {

test/test-cmd-remove.ts

@@ -1,8 +1,8 @@
 import "assert";
 import "should";
-import { loadManifest } from "../lib/core";
+import { loadManifest } from "../src/core";
 
-import { remove } from "../lib/cmd-remove";
+import { remove } from "../src/cmd-remove";
 
 import {
   createWorkDir,

test/test-cmd-search.ts

@@ -1,7 +1,7 @@
 import "assert";
 import nock from "nock";
 import "should";
-import { search } from "../lib/cmd-search";
+import { search } from "../src/cmd-search";
 
 import {
   createWorkDir,

test/test-cmd-view.ts

@@ -2,7 +2,7 @@ import "assert";
 import nock from "nock";
 import "should";
 
-import { view } from "../lib/cmd-view";
+import { view } from "../src/cmd-view";
 
 import {
   createWorkDir,

test/test-core.ts

@@ -13,7 +13,7 @@ import {
   parseEditorVersion,
   parseEnv,
   saveManifest,
-} from "../lib/core";
+} from "../src/core";
 
 import {
   createWorkDir,

test/test-pkg-name.ts

@@ -1,6 +1,6 @@
 import "assert";
 import "should";
-import { splitPkgName } from "../lib/utils/pkg-name";
+import { splitPkgName } from "../src/utils/pkg-name";
 
 describe("pkg-name.ts", function () {
   describe("splitPkgName", function () {

test/utils.ts

@@ -3,7 +3,7 @@ import nock from "nock";
 import path from "path";
 import os from "os";
 import testConsole from "test-console";
-import { PkgManifest } from "../lib/types/global";
+import { PkgManifest } from "../src/types/global";
 import _ from "lodash";
 
 export type ManifestCreationOptions = {

tsconfig.json

@@ -42,7 +42,7 @@
     // "resolvePackageJsonExports": true,                /* Use the package.json 'exports' field when resolving package imports. */
     // "resolvePackageJsonImports": true,                /* Use the package.json 'imports' field when resolving imports. */
     // "customConditions": [],                           /* Conditions to set in addition to the resolver-specific defaults when resolving imports. */
-    "resolveJsonModule": true,                           /* Enable importing .json files. */
+    "resolveJsonModule": false,                          /* Enable importing .json files. */
     // "allowArbitraryExtensions": true,                 /* Enable importing files with any extension, provided a declaration file is present. */
     // "noResolve": true,                                /* Disallow 'import's, 'require's or '<reference>'s from expanding the number of files TypeScript should add to a project. */
 
@@ -58,7 +58,7 @@
     "sourceMap": true,                                   /* Create source map files for emitted JavaScript files. */
     // "inlineSourceMap": true,                          /* Include sourcemap files inside the emitted JavaScript. */
     // "outFile": "./",                                  /* Specify a file that bundles all outputs into one JavaScript file. If 'declaration' is true, also designates a file that bundles all .d.ts output. */
-    "outDir": "./build",                                 /* Specify an output folder for all emitted files. */
+    "outDir": "./lib",                                   /* Specify an output folder for all emitted files. */
     // "removeComments": true,                           /* Disable emitting comments. */
     // "noEmit": true,                                   /* Disable emitting files from a compilation. */
     // "importHelpers": true,                            /* Allow importing helper functions from tslib once per project, instead of including them per-file. */
@@ -114,6 +114,6 @@
     /* Skip type checking all .d.ts files. */
   },
   "include": [
-    "./lib/**/*"
+    "./src/**/*"
   ]
 }