tina4-nodejs 3.10.42 → 3.10.44

package/packages/core/src/index.ts

@@ -68,7 +68,7 @@ export type { ResponseCacheConfig } from "./cache.js";
 export { Api } from "./api.js";
 export type { ApiResult } from "./api.js";
 export { Events } from "./events.js";
-export { DevAdmin, MessageLog, RequestInspector, ErrorTracker, DevMailboxStore, DevQueue, WsTracker } from "./devAdmin.js";
+export { DevAdmin, MessageLog, RequestInspector, ErrorTracker, DevMailboxStore, DevQueue, WsTracker, renderDashboard } from "./devAdmin.js";
 export { Messenger } from "./messenger.js";
 export type { SendResult, EmailMessage } from "./messenger.js";
 export { DevMailbox, createMessenger } from "./devMailbox.js";

package/packages/core/src/metrics.ts

@@ -54,6 +54,18 @@ function relativePath(filePath: string): string {
   return path.relative(".", filePath);
 }
 
+// ── Test file detection ─────────────────────────────────────
+
+function hasMatchingTest(relPath: string): boolean {
+  const name = relPath.split('/').pop()?.replace('.ts', '').replace('.js', '') || '';
+  const patterns = [
+    `test/${name}.test.ts`,
+    `${relPath.replace('.ts', '.test.ts').replace('.js', '.test.js')}`,
+    `tests/${name}.test.ts`,
+  ];
+  return patterns.some(p => fs.existsSync(p));
+}
+
 // ── Line counting ────────────────────────────────────────────
 
 interface LineCounts {
@@ -517,9 +529,27 @@ function detectViolations(
   return violations;
 }
 
+// ── Root Resolution ──────────────────────────────────────────
+
+/**
+ * Pick the right directory to scan.
+ *
+ * If the root dir has .ts files, scan the user's project code.
+ * Otherwise, scan the framework itself — so the bubble chart is never empty.
+ */
+function resolveRoot(root: string = "src"): string {
+  const rootPath = path.resolve(root);
+  if (fs.existsSync(rootPath) && walkFiles(rootPath, [".ts", ".js"]).length > 0) {
+    return root;
+  }
+  // Fallback: scan the framework package itself
+  return path.resolve(path.dirname(new URL(import.meta.url).pathname));
+}
+
 // ── Quick Metrics ────────────────────────────────────────────
 
 export function quickMetrics(root: string = "src"): Record<string, any> {
+  root = resolveRoot(root);
   const rootPath = path.resolve(root);
   if (!fs.existsSync(rootPath)) {
     return { error: `Directory not found: ${root}` };
@@ -644,6 +674,7 @@ function filesHash(root: string = "src"): string {
 }
 
 export function fullAnalysis(root: string = "src"): Record<string, any> {
+  root = resolveRoot(root);
   const currentHash = filesHash(root);
   const now = Date.now() / 1000;
 
@@ -730,6 +761,8 @@ export function fullAnalysis(root: string = "src"): Record<string, any> {
       coupling_afferent: ca,
       coupling_efferent: ce,
       instability: Math.round(instability * 1000) / 1000,
+      has_tests: hasMatchingTest(relPath),
+      dep_count: ce,
     });
   }
 
@@ -746,6 +779,10 @@ export function fullAnalysis(root: string = "src"): Record<string, any> {
   const totalMI = fileMetrics.reduce((sum, f) => sum + f.maintainability, 0);
   const avgMI = fileMetrics.length > 0 ? totalMI / fileMetrics.length : 0;
 
+  // Detect if we're scanning framework or project
+  const frameworkDir = path.resolve(path.dirname(new URL(import.meta.url).pathname));
+  const scanningFramework = rootPath === frameworkDir || rootPath.startsWith(frameworkDir + path.sep);
+
   const result: Record<string, any> = {
     files_analyzed: fileMetrics.length,
     total_functions: allFunctions.length,
@@ -755,6 +792,8 @@ export function fullAnalysis(root: string = "src"): Record<string, any> {
     file_metrics: fileMetrics,
     violations,
     dependency_graph: importGraph,
+    scan_mode: scanningFramework ? "framework" : "project",
+    scan_root: rootPath,
   };
 
   _fullCache = { hash: currentHash, data: result, time: now };

package/packages/core/src/server.ts

@@ -302,6 +302,8 @@ h1{font-size:3rem;font-weight:700;margin-bottom:0.25rem;letter-spacing:-1px}
 .gbtn-deploy{background:#3b82f6;color:#fff}
 .gbtn-deploy:hover{background:#2563eb}
 .gbtn-deployed{background:transparent;border:1px solid #22c55e;color:#22c55e;cursor:default;font-size:0.7rem}
+@keyframes wiggle{0%{transform:rotate(0deg)}15%{transform:rotate(14deg)}30%{transform:rotate(-10deg)}45%{transform:rotate(8deg)}60%{transform:rotate(-4deg)}75%{transform:rotate(2deg)}100%{transform:rotate(0deg)}}
+.star-wiggle{display:inline-block;transform-origin:center}
 </style>
 </head>
 <body>
@@ -315,7 +317,7 @@ h1{font-size:3rem;font-weight:700;margin-bottom:0.25rem;letter-spacing:-1px}
         <a href="/__dev" class="btn">Dev Admin</a>
         <a href="#gallery" class="btn">Gallery</a>
         <a href="https://github.com/tina4stack/tina4-nodejs" class="btn" target="_blank">GitHub</a>
-        <a href="https://github.com/tina4stack/tina4-nodejs/stargazers" class="btn" target="_blank">&#11088; Star</a>
+        <a href="https://github.com/tina4stack/tina4-nodejs/stargazers" class="btn" target="_blank"><span class="star-wiggle">&#9734;</span> Star</a>
     </div>
     <div class="status">
         <span><span class="dot"></span>Server running</span>
@@ -364,6 +366,20 @@ function deployGallery(name) {
     })
     .catch(function(e) { alert('Deploy error: ' + e.message); });
 }
+(function(){
+    var star=document.querySelector('.star-wiggle');
+    if(!star)return;
+    function doWiggle(){
+        star.style.animation='wiggle 1.2s ease-in-out';
+        star.addEventListener('animationend',function onEnd(){
+            star.removeEventListener('animationend',onEnd);
+            star.style.animation='none';
+            var delay=3000+Math.random()*15000;
+            setTimeout(doWiggle,delay);
+        });
+    }
+    setTimeout(doWiggle,3000);
+})();
 </script>
 </body>
 </html>`;
@@ -731,7 +747,7 @@ ${reset}
         let result: unknown;
         const routeParams = req.params || {};
         const fnStr = match.handler.toString();
-        const argMatch = fnStr.match(/^(?:async\s+)?(?:function\s*\w*)?\s*\(([^)]*)\)/);
+        const argMatch = fnStr.match(/^(?:async\s*)?(?:function\s*\w*)?\s*\(([^)]*)\)/);
         const argNames = argMatch?.[1]?.split(",").map((s: string) => s.trim().replace(/[:=].*/,"")) ?? [];
         const filteredArgs = argNames.filter((n: string) => n.length > 0);
 

package/packages/orm/src/adapters/sqlite.ts

@@ -57,9 +57,13 @@ export class SQLiteAdapter implements DatabaseAdapter {
   fetch<T = Record<string, unknown>>(sql: string, params?: unknown[], limit?: number, skip?: number): T[] {
     let effectiveSql = sql;
     if (limit !== undefined) {
-      effectiveSql += ` LIMIT ${limit}`;
-      if (skip !== undefined && skip > 0) {
-        effectiveSql += ` OFFSET ${skip}`;
+      // Skip appending LIMIT when the SQL already contains one (dedup)
+      const sqlBeforeComment = sql.toUpperCase().split("--")[0];
+      if (!sqlBeforeComment.includes("LIMIT")) {
+        effectiveSql += ` LIMIT ${limit}`;
+        if (skip !== undefined && skip > 0) {
+          effectiveSql += ` OFFSET ${skip}`;
+        }
       }
     }
     return this.query<T>(effectiveSql, params);

package/packages/orm/src/baseModel.ts

@@ -17,6 +17,15 @@ export function camelToSnake(name: string): string {
   return name.replace(/[A-Z]/g, (c) => `_${c.toLowerCase()}`);
 }
 
+/**
+ * Check whether ORM_PLURAL_TABLE_NAMES is enabled in .env.
+ * When true, hasMany relationship keys get an "s" suffix (e.g. "posts" instead of "post").
+ */
+function _pluralRelKeys(): boolean {
+  const v = process.env.ORM_PLURAL_TABLE_NAMES ?? "";
+  return /^(true|1|yes)$/i.test(v);
+}
+
 /**
  * BaseModel provides instance methods for ORM models.
  * Models extend this class and define static properties.
@@ -395,7 +404,8 @@ export class BaseModel {
       }
       if (ModelClass.hasMany) {
         for (const rel of ModelClass.hasMany) {
-          const relKey = rel.model.toLowerCase() + "s";
+          const base = rel.model.toLowerCase();
+          const relKey = _pluralRelKeys() ? base + "s" : base;
           if (this[relKey] !== undefined) {
             result[relKey] = this[relKey];
           }
@@ -818,8 +828,9 @@ export class BaseModel {
     // Check hasMany
     if (ModelClass.hasMany) {
       const rel = ModelClass.hasMany.find((r) => {
-        const key = r.model.toLowerCase() + "s";
-        return key === relName || r.model.toLowerCase() === relName || r.model === relName;
+        const base = r.model.toLowerCase();
+        const key = _pluralRelKeys() ? base + "s" : base;
+        return key === relName || base === relName || r.model === relName;
       });
       if (rel) {
         const relatedClass = BaseModel._modelRegistry[rel.model];
@@ -877,8 +888,9 @@ export class BaseModel {
       }
       if (!relDef && ModelClass.hasMany) {
         relDef = ModelClass.hasMany.find((r) => {
-          const key = r.model.toLowerCase() + "s";
-          return key === relName || r.model.toLowerCase() === relName || r.model === relName;
+          const base = r.model.toLowerCase();
+          const key = _pluralRelKeys() ? base + "s" : base;
+          return key === relName || base === relName || r.model === relName;
         });
         if (relDef) relType = "hasMany";
       }