refactor: cleanup of some console statements, fixes for the test setup

MihalyToth20 · MihalyToth20 · commit c94a56219bf9 · 2025-10-12T15:29:35.000+02:00
diff --git a/.gitignore b/.gitignore
@@ -35,3 +35,6 @@ calyInstance.test.setup.json
 
 # Turborepo
 .turbo
+
+# Jest
+coverage/
diff --git a/README.md b/README.md
@@ -25,7 +25,7 @@
 
 ---
 
-## 🚀 Quick Start
+## 🚀 [Dev] Quick Start
 
 ```
 # 1. Clone the repo
@@ -34,7 +34,7 @@
 pnpm install
 
 # 3. Build the CLI
-pnpm build:packages
+pnpm build
 
 # 4. Run any command
 xano
@@ -89,7 +89,7 @@ jobs:
 
 ### Xano Registry **(WIP)**
 
-I have been astonished by the shadcn/ui CLI and the core principles of code distribution implemented in that tool. While that is primarily for frontend developers, their attempt to generalise their registry pointed me into the direction where I have started rebuilding a registry system for Xano powered by `xanoscript`. The reason for this is to overcome the bugs of Xano Snippets, avoid dependency of external providers of Xano Actions and in general to allow any team to build their own registries. These registries should also server as guidance for LLMs to eventually start generating more and more reliable `xanoscript`.
+I have been astonished by the shadcn/ui CLI and the core principles of code distribution implemented in that tool. While that is primarily for frontend developers, their attempt to generalise their registry pointed me into the direction where I have started rebuilding a registry system for Xano powered by `xanoscript`. The reason for this is to overcome the bugs of Xano Snippets, avoid dependency of external providers of Xano Actions and in general to allow any team to build their own registries with ease. These registries should also serve as guidance for LLMs to eventually start generating more and more reliable `xanoscript`.
 
 <details>
 <summary>How to use the registry feature?</summary>
@@ -110,7 +110,7 @@ I have been astonished by the shadcn/ui CLI and the core principles of code dist
    ```
 
 > **Notes:**
-> Currently there is no way of automatically build out the registry from a collection of `xanoscript` files, so this is why
+> Currently there is no way of automatically building the registry from a collection of `xanoscript` files, so this is why
 > it is important to always keep the registry/definitions/index.json and the individual definition files in sync.
 > Currently there is theoretic support for registry:function and registry:table components, registry:query but registry:snippet is planned.
 > With the registry:snippet I aim to have a shot at fixing Xano's Snippets and make it searchable and reusable by also LLMs.
diff --git a/jest.config.js b/jest.config.js
@@ -1,20 +1,11 @@
-/** @type {import('jest').Config} */
+import { createDefaultPreset } from 'ts-jest';
+
+const tsJestTransformCfg = createDefaultPreset().transform;
+
+/** @type {import("jest").Config} **/
 export default {
-   preset: 'ts-jest/presets/default-esm',
    testEnvironment: 'node',
-   extensionsToTreatAsEsm: ['.ts'],
    transform: {
-      '^.+\\.ts$': ['ts-jest', { useESM: true }],
+      ...tsJestTransformCfg,
    },
-   globals: {
-      'ts-jest': {
-         useESM: true,
-      },
-   },
-   testMatch: ['**/__tests__/**/*.test.ts', '**/?(*.)+(spec|test).ts'],
-   passWithNoTests: true,
-   moduleFileExtensions: ['ts', 'js', 'json'],
-   roots: ['<rootDir>/packages/'],
-   coverageDirectory: 'coverage',
-   collectCoverageFrom: ['packages/*/src/**/*.{ts,js}'],
 };
diff --git a/package.json b/package.json
@@ -36,6 +36,7 @@
       "eslint": "^9.35.0",
       "eslint-plugin-import": "^2.32.0",
       "jest": "^30.2.0",
+      "jest-html-reporter": "^4.3.0",
       "knip": "^5.63.1",
       "rollup": "^4.50.0",
       "rollup-plugin-dts": "^6.2.3",
@@ -46,5 +47,5 @@
       "turbo": "^2.5.8",
       "typescript": "^5.9.2"
    },
-   "packageManager": "pnpm@10.15.1"
+   "packageManager": "pnpm@10.18.2"
 }
diff --git a/packages/cli/jest.config.js b/packages/cli/jest.config.js
@@ -0,0 +1,2 @@
+import config from '../../jest.config.js';
+export default config;
diff --git a/packages/cli/package.json b/packages/cli/package.json
@@ -59,7 +59,7 @@
         "clean": "trash dist tsconfig.tsbuildinfo",
         "build:js": "tsx esbuild.config.ts",
         "build:chmod": "shx chmod +x dist/index.cjs",
-        "build": "pnpm build:js && pnpm build:chmod && pnpm link -g",
+        "build": "pnpm clean && pnpm build:js && pnpm build:chmod && pnpm link -g",
         "xano": "node dist/index.cjs"
     },
     "dependencies": {
diff --git a/packages/cli/src/features/code-gen/open-api-generator.ts b/packages/cli/src/features/code-gen/open-api-generator.ts
@@ -10,32 +10,35 @@ export async function runOpenApiGenerator({
    additionalArgs = [],
    logger = false,
 }) {
-      // Always use npx and the official package
-      const cliBin = process.platform === 'win32' ? 'npx.cmd' : 'npx';
-      const inputPath = resolve(input).replace(/\\/g, '/');
-      const outputPath = resolve(output);
+   // [ ] Add list of orval supported generators
+   // [ ] Add orval process runner on the analogy of the openapitools/openapi-generator-cli
 
-      const cliArgs = [
-         '@openapitools/openapi-generator-cli',
-         'generate',
-         '-i',
-         inputPath,
-         '-g',
-         generator,
-         '-o',
-         outputPath,
-         ...additionalArgs,
-      ].filter(Boolean);
+   // Always use npx and the official package
+   const cliBin = process.platform === 'win32' ? 'npx.cmd' : 'npx';
+   const inputPath = resolve(input).replace(/\\/g, '/');
+   const outputPath = resolve(output);
 
-      let logStream = null;
-      let logPath = null;
+   const cliArgs = [
+      '@openapitools/openapi-generator-cli',
+      'generate',
+      '-i',
+      inputPath,
+      '-g',
+      generator,
+      '-o',
+      outputPath,
+      ...additionalArgs,
+   ].filter(Boolean);
 
-      if (logger) {
-         const logsDir = join(process.cwd(), 'output', '_logs');
-         await mkdir(logsDir, { recursive: true });
-         logPath = join(logsDir, `openapi-generator-${Date.now()}.log`);
-         logStream = createWriteStream(logPath);
-      }
+   let logStream = null;
+   let logPath = null;
+
+   if (logger) {
+      const logsDir = join(process.cwd(), 'output', '_logs');
+      await mkdir(logsDir, { recursive: true });
+      logPath = join(logsDir, `openapi-generator-${Date.now()}.log`);
+      logStream = createWriteStream(logPath);
+   }
 
    return new Promise((resolvePromise, reject) => {
       const proc = spawn(cliBin, cliArgs, { shell: true, stdio: ['ignore', 'pipe', 'pipe'] });
@@ -73,4 +76,3 @@ export async function runOpenApiGenerator({
       });
    });
 }
-
diff --git a/packages/cli/src/program.ts b/packages/cli/src/program.ts
@@ -26,22 +26,19 @@ const core = new Caly(nodeConfigStorage);
 // Store start time on the command object
 program.hook('preAction', (thisCommand, actionCommand) => {
   commandStartTimes.set(thisCommand, Date.now());
-  InitializedPostHog.capture({
+  InitializedPostHog.captureImmediate({
     distinctId: 'anonymous',
     event: 'command_started',
     properties: {
       $process_person_profile: false,
       "command": actionCommand.name(),
     }
   });
-  InitializedPostHog.shutdown();
 });
 
 program.hook('postAction', (thisCommand, actionCommand) => {
   const start = commandStartTimes.get(thisCommand);
   if (!start) {
-    // Could happen if preAction failed, or if there's a bug
-    console.warn('⏱️  Command timer missing start time.');
     return;
   }
   const duration = ((Date.now() - start) / 1000).toFixed(2);
@@ -52,7 +49,7 @@ program.hook('postAction', (thisCommand, actionCommand) => {
     : actionCommand.name();
 
   console.log(`\n⏱️  Command "${commandPath}" completed in ${duration}s`);
-  InitializedPostHog.capture({
+  InitializedPostHog.captureImmediate({
     distinctId: 'anonymous',
     event: 'command_finished',
     properties: {
diff --git a/packages/cli/src/utils/feature-focused/registry/scaffold.ts b/packages/cli/src/utils/feature-focused/registry/scaffold.ts
@@ -70,8 +70,6 @@ async function scaffoldRegistry(
    await writeFile(functionFilePath, sampleFunctionContent, 'utf8');
    await writeFile(functionDefPath, JSON.stringify(sampleRegistryItem, null, 2), 'utf8');
    await writeFile(indexPath, JSON.stringify(sampleIndex, null, 2), 'utf8');
-
-   console.log(`✅ Registry scaffolded at "${registryRoot}" with a sample component!`);
 }
 
 export { scaffoldRegistry };
diff --git a/packages/core/jest.config.js b/packages/core/jest.config.js
@@ -0,0 +1,2 @@
+import config from '../../jest.config.js';
+export default config;
diff --git a/packages/core/package.json b/packages/core/package.json
@@ -58,6 +58,6 @@
         "build:types": "tsc",
         "bundle:types": "rollup -c rollup.config.js",
         "postbuild": "tsx ../../scripts/clean-types.ts . index.bundled.d.ts",
-        "build": "pnpm build:js && pnpm build:types && pnpm bundle:types && pnpm postbuild"
+        "build": "pnpm clean && pnpm build:js && pnpm build:types && pnpm bundle:types && pnpm postbuild"
     }
 }
diff --git a/packages/core/src/index.ts b/packages/core/src/index.ts
@@ -334,9 +334,6 @@ export class Caly extends TypedEmitter<EventMap> {
     *   instanceConfig: instanceConfig,
     *   workspaceConfig: workspaceConfig
     * });
-    *
-    * console.log('Enhanced OAS:', result.oas);
-    * console.log('Generated files:', result.generatedItems);
     * ```
     */
    async doOasUpdate({ inputOas, instanceConfig, workspaceConfig }): Promise<{
@@ -373,9 +370,6 @@ export class Caly extends TypedEmitter<EventMap> {
     *   workspace: 'main',
     *   branch: 'master'
     * });
-    *
-    * console.log('Instance URL:', context.instanceConfig.url);
-    * console.log('Workspace ID:', context.workspaceConfig.id);
     * ```
     */
    async loadAndValidateContext({ instance, workspace, branch, startDir }): Promise<{
@@ -402,7 +396,6 @@ export class Caly extends TypedEmitter<EventMap> {
     * ```typescript
     * // Get current context
     * const currentContext = await calyInstance.getCurrentContextConfig();
-    * console.log('Current instance:', currentContext.instance);
     *
     * // Override specific context values
     * const overriddenContext = await calyInstance.getCurrentContextConfig(null, {
@@ -431,8 +424,6 @@ export class Caly extends TypedEmitter<EventMap> {
     * @example
     * ```typescript
     * const config = await calyInstance.loadGlobalConfig();
-    * console.log('Current instance:', config.currentContext.instance);
-    * console.log('Available instances:', config.instances);
     * ```
     */
    async loadGlobalConfig(): Promise<any> {
diff --git a/packages/types/jest.config.js b/packages/types/jest.config.js
@@ -0,0 +1,2 @@
+import config from '../../jest.config.js';
+export default config;
diff --git a/packages/types/package.json b/packages/types/package.json
@@ -2,7 +2,7 @@
     "name": "@repo/types",
     "version": "0.4.0",
     "description": "TypeScript type definitions for the Caly Xano tooling ecosystem",
-    "private": "true",
+    "private": true,
     "publishConfig": {
         "access": "restricted"
     },
@@ -48,6 +48,6 @@
         "build:types": "tsc",
         "bundle:types": "rollup -c rollup.config.js",
         "postbuild": "tsx ../../scripts/clean-types.ts . index.bundled.d.ts",
-        "build": "pnpm build:types && pnpm bundle:types && pnpm postbuild"
+        "build": "pnpm clean && pnpm build:types && pnpm bundle:types && pnpm postbuild"
     }
 }
diff --git a/packages/utils/__tests__/is-empty.spec.ts b/packages/utils/__tests__/is-empty.spec.ts
@@ -0,0 +1,67 @@
+import { isEmptySchema, isNotEmpty } from '../src';
+
+describe('isEmptySchema', () => {
+   it('returns true for null or undefined', () => {
+      expect(isEmptySchema(null)).toBe(true);
+      expect(isEmptySchema(undefined)).toBe(true);
+   });
+
+   it('returns true for empty array', () => {
+      expect(isEmptySchema([])).toBe(true);
+   });
+
+   it('returns false for non-empty array', () => {
+      expect(isEmptySchema([1])).toBe(false);
+   });
+
+   it('returns true for empty object {}', () => {
+      expect(isEmptySchema({})).toBe(true);
+   });
+
+   it('returns false for object with type and properties', () => {
+      expect(isEmptySchema({ type: 'object', properties: { foo: { type: 'string' } } })).toBe(
+         false
+      );
+   });
+
+   it('returns true for object with type:object and empty properties', () => {
+      expect(isEmptySchema({ type: 'object', properties: {} })).toBe(true);
+   });
+
+   it('returns true for object with only description', () => {
+      expect(isEmptySchema({ description: 'foo' })).toBe(true);
+   });
+
+   it('returns true for object with only ignored keys', () => {
+      expect(isEmptySchema({ type: undefined, description: undefined })).toBe(true);
+   });
+
+   it('returns false for object with extra meaningful keys', () => {
+      expect(isEmptySchema({ type: 'string', format: 'email' })).toBe(false);
+   });
+});
+
+describe('isNotEmpty', () => {
+   it('returns false for null, undefined, or empty string', () => {
+      expect(isNotEmpty(null)).toBe(false);
+      expect(isNotEmpty(undefined)).toBe(false);
+      expect(isNotEmpty('')).toBe(false);
+   });
+
+   it('returns true for non-empty string', () => {
+      expect(isNotEmpty('abc')).toBe(true);
+   });
+
+   it('returns true for number 0', () => {
+      expect(isNotEmpty(0)).toBe(true);
+   });
+
+   it('returns true for false boolean', () => {
+      expect(isNotEmpty(false)).toBe(true);
+   });
+
+   it('returns true for objects and arrays', () => {
+      expect(isNotEmpty([])).toBe(true);
+      expect(isNotEmpty({})).toBe(true);
+   });
+});
diff --git a/packages/utils/__tests__/path.spec.ts b/packages/utils/__tests__/path.spec.ts
@@ -0,0 +1,49 @@
+import { joinPath, dirname } from '../src';
+
+describe('joinPath', () => {
+   it('joins simple segments', () => {
+      expect(joinPath('a', 'b', 'c')).toBe('a/b/c');
+   });
+
+   it('removes leading/trailing slashes appropriately', () => {
+      expect(joinPath('/a/', '/b/', '/c/')).toBe('a/b/c');
+      expect(joinPath('/a', 'b/', '/c/')).toBe('a/b/c');
+      expect(joinPath('///a///', '///b///', '///c///')).toBe('a/b/c');
+   });
+
+   it('filters out empty segments', () => {
+      expect(joinPath('a', '', 'b')).toBe('a/b');
+      expect(joinPath('', '', 'a')).toBe('a');
+      expect(joinPath('', '', '')).toBe('');
+   });
+
+   it('handles single and no segments', () => {
+      expect(joinPath('a')).toBe('a');
+      expect(joinPath()).toBe('');
+   });
+});
+
+describe('dirname', () => {
+   it('returns parent directory for a typical path', () => {
+      expect(dirname('a/b/c')).toBe('a/b');
+      expect(dirname('a/b/c/')).toBe('a/b');
+   });
+
+   it('returns . for no slash', () => {
+      expect(dirname('abc')).toBe('.');
+   });
+
+   it('returns / for root-level file', () => {
+      expect(dirname('/foo')).toBe('/');
+      expect(dirname('/foo/')).toBe('/');
+   });
+
+   it('returns / for root', () => {
+      expect(dirname('/')).toBe('.');
+      expect(dirname('///')).toBe('.');
+   });
+
+   it('handles empty string', () => {
+      expect(dirname('')).toBe('.');
+   });
+});
diff --git a/packages/utils/__tests__/replace-placeholders.spec.ts b/packages/utils/__tests__/replace-placeholders.spec.ts
diff --git a/packages/utils/jest.config.js b/packages/utils/jest.config.js
diff --git a/packages/utils/package.json b/packages/utils/package.json
diff --git a/packages/utils/src/methods/path.ts b/packages/utils/src/methods/path.ts
diff --git a/packages/utils/src/methods/sanitize.ts b/packages/utils/src/methods/sanitize.ts

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,2 @@`
	`1`	`+import config from '../../jest.config.js';`
	`2`	`+export default config;`
Original file line number	Diff line number	Diff line change
`@@ -70,8 +70,6 @@ async function scaffoldRegistry(`
`70`	`70`	`await writeFile(functionFilePath, sampleFunctionContent, 'utf8');`
`71`	`71`	`await writeFile(functionDefPath, JSON.stringify(sampleRegistryItem, null, 2), 'utf8');`
`72`	`72`	`await writeFile(indexPath, JSON.stringify(sampleIndex, null, 2), 'utf8');`
`73`		`-`
`74`		- console.log(`✅ Registry scaffolded at "${registryRoot}" with a sample component!`);
`75`	`73`	`}`
`76`	`74`
`77`	`75`	`export { scaffoldRegistry };`