react-doctor-cli-dev 1.0.3 → 1.0.5

package/backend/.env

@@ -1,3 +1,2 @@
 PORT=3000
 API_KEY=react-doctor-secret-key-change-this
-DB_PATH=./data/reports.db

package/backend/dist/db.js

@@ -1,69 +1,26 @@
 "use strict";
-// ─────────────────────────────────────────────────────────────
-// backend/src/db.ts
-//
-// SQLite database via better-sqlite3 (synchronous, fast).
-//
-// SCHEMA:
-//
-//   reports — one row per analysis run
-//     id           — auto-increment PK
-//     project      — project name / hostname
-//     score        — 0-100 overall performance score
-//     grade        — A+, A, B, C, D, F
-//     analyzed_at  — ISO timestamp from the CLI
-//     created_at   — when the row was inserted
-//     static_json  — full StaticReport as JSON string
-//     runtime_json — full RuntimeReport map as JSON string
-//                    (screenshots stripped — stored as files)
-//     suggestions  — full Suggestion[] as JSON string
-//
-// WHY SPLIT COLUMNS INSTEAD OF ONE payload COLUMN?
-//   The old schema stored everything in a single `payload` TEXT
-//   column. This meant the dashboard couldn't query individual
-//   fields without parsing the whole blob. Splitting into three
-//   typed columns lets us:
-//     • SELECT only the part we need (e.g. just static_json for
-//       the issues page)
-//     • Keep the list endpoint fast (no giant blobs per row)
-//     • Add indices on score/grade later without changing schema
-//
-// SCREENSHOT STORAGE:
-//   Screenshots are base64 PNGs — up to 200KB each. Storing them
-//   in SQLite bloats the DB and makes every query slower. Instead:
-//     • The upload route strips dataUrls from runtime_json
-//     • Saves each screenshot as a .png file in data/screenshots/
-//     • Replaces dataUrl with a relative path /screenshots/<file>
-//   The dashboard fetches screenshots via the static file route.
-// ─────────────────────────────────────────────────────────────
 var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.screenshotsDir = void 0;
 const better_sqlite3_1 = __importDefault(require("better-sqlite3"));
 const path_1 = __importDefault(require("path"));
 const fs_1 = __importDefault(require("fs"));
-const dotenv_1 = __importDefault(require("dotenv"));
-dotenv_1.default.config({ path: path_1.default.join(__dirname, "..", ".env") });
-const PACKAGE_ROOT = path_1.default.resolve(__dirname, "..", "..");
-const DEFAULT_DB = path_1.default.join(PACKAGE_ROOT, "backend", "data", "reports.db");
-const dbPath = process.env.DB_PATH || DEFAULT_DB;
+const os_1 = __importDefault(require("os"));
+// Store DB in user's home directory — works regardless of where process spawns from
+const dbDir = path_1.default.join(os_1.default.homedir(), ".react-doctor");
+const dbPath = process.env.DB_PATH || path_1.default.join(dbDir, "reports.db");
 fs_1.default.mkdirSync(path_1.default.dirname(dbPath), { recursive: true });
-fs_1.default.mkdirSync(path_1.default.join(path_1.default.dirname(dbPath), "screenshots"), { recursive: true });
 const db = new better_sqlite3_1.default(dbPath);
 db.exec(`
-  CREATE TABLE IF NOT EXISTS reports (
-    id            INTEGER PRIMARY KEY AUTOINCREMENT,
-    project       TEXT    NOT NULL,
-    score         INTEGER NOT NULL,
-    grade         TEXT    NOT NULL,
-    analyzed_at   TEXT    NOT NULL,
-    created_at    TEXT    NOT NULL DEFAULT (datetime('now')),
-    static_json   TEXT    NOT NULL,
-    runtime_json  TEXT    NOT NULL,
-    suggestions   TEXT    NOT NULL
-  );
+CREATE TABLE IF NOT EXISTS reports (
+    id INTEGER PRIMARY KEY AUTOINCREMENT,
+    project TEXT NOT NULL,
+    score INTEGER NOT NULL,
+    grade TEXT NOT NULL,
+    analyzed_at TEXT NOT NULL,
+    created_at TEXT NOT NULL DEFAULT (datetime('now')),
+    payload TEXT NOT NULL
+);
 `);
-exports.screenshotsDir = path_1.default.join(path_1.default.dirname(dbPath), "screenshots");
 exports.default = db;

package/backend/dist/index.js

@@ -9,34 +9,27 @@ const helmet_1 = __importDefault(require("helmet"));
 const dotenv_1 = __importDefault(require("dotenv"));
 const path_1 = __importDefault(require("path"));
 const reports_1 = __importDefault(require("./routes/reports"));
-const db_1 = require("./db");
-dotenv_1.default.config({ path: path_1.default.join(__dirname, "..", ".env") });
+// Load .env from the backend folder, not from cwd
+dotenv_1.default.config({ path: path_1.default.join(__dirname, '..', '.env') });
 const app = (0, express_1.default)();
 const PORT = process.env.PORT || 3000;
-// ── Middleware ────────────────────────────────────────────────
+const API_KEY = process.env.API_KEY || "react-doctor-secret-key-change-this";
+// Make API_KEY available globally so auth middleware can use it
+process.env.API_KEY = API_KEY;
 app.use((0, helmet_1.default)());
 app.use((0, cors_1.default)());
-app.use(express_1.default.json({ limit: "50mb" }));
-// ── Static: screenshots saved by the upload route ────────────
-// Served at /screenshots/<filename>
-// The dashboard uses these as <img src="/screenshots/...">
-app.use("/screenshots", express_1.default.static(db_1.screenshotsDir));
-// ── Routes ────────────────────────────────────────────────────
-app.use("/api/reports", reports_1.default);
-// ── Health check ──────────────────────────────────────────────
-app.get("/health", (_req, res) => {
-    res.json({ status: "ok", timestamp: new Date().toISOString() });
+app.use(express_1.default.json({ limit: '50mb' }));
+app.use('/api/reports', reports_1.default);
+app.get('/health', (_req, res) => {
+    res.json({ status: 'ok', timestamp: new Date().toISOString() });
 });
-// ── 404 ───────────────────────────────────────────────────────
-app.use((_req, res) => {
-    res.status(404).json({ message: "Route not found" });
+app.use((req, res) => {
+    res.status(404).json({ message: 'Route not found' });
 });
-// ── Error handler ─────────────────────────────────────────────
 app.use((err, _req, res, _next) => {
     console.error(err.stack);
-    res.status(500).json({ message: "Internal Server Error" });
+    res.status(500).json({ message: 'Internal Server Error' });
 });
-// ── Start ─────────────────────────────────────────────────────
 app.listen(PORT, () => {
-    console.log(`React Doctor backend running on http://localhost:${PORT}`);
+    console.log(`🩺 React Doctor backend running on http://localhost:${PORT}`);
 });

package/backend/dist/routes/reports.js

@@ -31,39 +31,6 @@
 //     /screenshots/<filename>
 //   so the dashboard can load them as normal <img> tags.
 // ─────────────────────────────────────────────────────────────
-var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
-    if (k2 === undefined) k2 = k;
-    var desc = Object.getOwnPropertyDescriptor(m, k);
-    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
-      desc = { enumerable: true, get: function() { return m[k]; } };
-    }
-    Object.defineProperty(o, k2, desc);
-}) : (function(o, m, k, k2) {
-    if (k2 === undefined) k2 = k;
-    o[k2] = m[k];
-}));
-var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
-    Object.defineProperty(o, "default", { enumerable: true, value: v });
-}) : function(o, v) {
-    o["default"] = v;
-});
-var __importStar = (this && this.__importStar) || (function () {
-    var ownKeys = function(o) {
-        ownKeys = Object.getOwnPropertyNames || function (o) {
-            var ar = [];
-            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
-            return ar;
-        };
-        return ownKeys(o);
-    };
-    return function (mod) {
-        if (mod && mod.__esModule) return mod;
-        var result = {};
-        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
-        __setModuleDefault(result, mod);
-        return result;
-    };
-})();
 var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
@@ -71,8 +38,9 @@ Object.defineProperty(exports, "__esModule", { value: true });
 const express_1 = require("express");
 const path_1 = __importDefault(require("path"));
 const fs_1 = __importDefault(require("fs"));
-const db_1 = __importStar(require("../db"));
+const db_1 = __importDefault(require("../db"));
 const auth_1 = require("../middleware/auth");
+const screenshotsDir = "data/screenshots";
 const router = (0, express_1.Router)();
 // ── GET /api/reports ─────────────────────────────────────────
 // Summary list — no blobs, just the columns the history page needs.
@@ -241,7 +209,7 @@ function saveScreenshots(reportId, pending) {
         const safeRoute = shot.routeKey.replace(/[/:]/g, "-").replace(/^-+/, "");
         const safeLabel = shot.label.replace(/[^a-z0-9]/gi, "-");
         const filename = `${reportId}-${safeRoute}-${safeLabel}.png`;
-        const fullPath = path_1.default.join(db_1.screenshotsDir, filename);
+        const fullPath = path_1.default.join(screenshotsDir, filename);
         try {
             fs_1.default.writeFileSync(fullPath, shot.buffer);
             saved.push({

package/backend/src/db.ts

@@ -1,70 +1,25 @@
-// ─────────────────────────────────────────────────────────────
-// backend/src/db.ts
-//
-// SQLite database via better-sqlite3 (synchronous, fast).
-//
-// SCHEMA:
-//
-//   reports — one row per analysis run
-//     id           — auto-increment PK
-//     project      — project name / hostname
-//     score        — 0-100 overall performance score
-//     grade        — A+, A, B, C, D, F
-//     analyzed_at  — ISO timestamp from the CLI
-//     created_at   — when the row was inserted
-//     static_json  — full StaticReport as JSON string
-//     runtime_json — full RuntimeReport map as JSON string
-//                    (screenshots stripped — stored as files)
-//     suggestions  — full Suggestion[] as JSON string
-//
-// WHY SPLIT COLUMNS INSTEAD OF ONE payload COLUMN?
-//   The old schema stored everything in a single `payload` TEXT
-//   column. This meant the dashboard couldn't query individual
-//   fields without parsing the whole blob. Splitting into three
-//   typed columns lets us:
-//     • SELECT only the part we need (e.g. just static_json for
-//       the issues page)
-//     • Keep the list endpoint fast (no giant blobs per row)
-//     • Add indices on score/grade later without changing schema
-//
-// SCREENSHOT STORAGE:
-//   Screenshots are base64 PNGs — up to 200KB each. Storing them
-//   in SQLite bloats the DB and makes every query slower. Instead:
-//     • The upload route strips dataUrls from runtime_json
-//     • Saves each screenshot as a .png file in data/screenshots/
-//     • Replaces dataUrl with a relative path /screenshots/<file>
-//   The dashboard fetches screenshots via the static file route.
-// ─────────────────────────────────────────────────────────────
-
 import Database from "better-sqlite3";
-import path     from "path";
-import fs       from "fs";
-import dotenv   from "dotenv";
-
-dotenv.config({ path: path.join(__dirname, "..", ".env") });
+import path from "path";
+import fs from "fs";
+import os from "os";
 
-const PACKAGE_ROOT = path.resolve(__dirname, "..", "..");
-const DEFAULT_DB   = path.join(PACKAGE_ROOT, "backend", "data", "reports.db");
-const dbPath       = process.env.DB_PATH || DEFAULT_DB;
+// Store DB in user's home directory — works regardless of where process spawns from
+const dbDir = path.join(os.homedir(), ".react-doctor");
+const dbPath = process.env.DB_PATH || path.join(dbDir, "reports.db");
 
 fs.mkdirSync(path.dirname(dbPath), { recursive: true });
-fs.mkdirSync(path.join(path.dirname(dbPath), "screenshots"), { recursive: true });
 
 const db = new Database(dbPath);
-
 db.exec(`
-  CREATE TABLE IF NOT EXISTS reports (
-    id            INTEGER PRIMARY KEY AUTOINCREMENT,
-    project       TEXT    NOT NULL,
-    score         INTEGER NOT NULL,
-    grade         TEXT    NOT NULL,
-    analyzed_at   TEXT    NOT NULL,
-    created_at    TEXT    NOT NULL DEFAULT (datetime('now')),
-    static_json   TEXT    NOT NULL,
-    runtime_json  TEXT    NOT NULL,
-    suggestions   TEXT    NOT NULL
-  );
+CREATE TABLE IF NOT EXISTS reports (
+    id INTEGER PRIMARY KEY AUTOINCREMENT,
+    project TEXT NOT NULL,
+    score INTEGER NOT NULL,
+    grade TEXT NOT NULL,
+    analyzed_at TEXT NOT NULL,
+    created_at TEXT NOT NULL DEFAULT (datetime('now')),
+    payload TEXT NOT NULL
+);
 `);
 
-export const screenshotsDir = path.join(path.dirname(dbPath), "screenshots");
-export default db;
+export default db;

package/backend/src/index.ts

@@ -1,46 +1,39 @@
-import express, { Request, Response, NextFunction } from "express";
-import cors    from "cors";
-import helmet  from "helmet";
-import dotenv  from "dotenv";
-import path    from "path";
-import reportRoutes          from "./routes/reports";
-import db, { screenshotsDir } from "./db";
-
-dotenv.config({ path: path.join(__dirname, "..", ".env") });
-
-const app  = express();
+import express, { Request, Response, NextFunction } from 'express';
+import cors from 'cors';
+import helmet from 'helmet';
+import dotenv from 'dotenv';
+import path from 'path';
+import reportRoutes from './routes/reports';
+import db from './db';
+
+// Load .env from the backend folder, not from cwd
+dotenv.config({ path: path.join(__dirname, '..', '.env') });
+
+const app = express();
 const PORT = process.env.PORT || 3000;
+const API_KEY = process.env.API_KEY || "react-doctor-secret-key-change-this";
+
+// Make API_KEY available globally so auth middleware can use it
+process.env.API_KEY = API_KEY;
 
-// ── Middleware ────────────────────────────────────────────────
 app.use(helmet());
 app.use(cors());
-app.use(express.json({ limit: "50mb" }));
+app.use(express.json({ limit: '50mb' }));
+app.use('/api/reports', reportRoutes);
 
-// ── Static: screenshots saved by the upload route ────────────
-// Served at /screenshots/<filename>
-// The dashboard uses these as <img src="/screenshots/...">
-app.use("/screenshots", express.static(screenshotsDir));
-
-// ── Routes ────────────────────────────────────────────────────
-app.use("/api/reports", reportRoutes);
-
-// ── Health check ──────────────────────────────────────────────
-app.get("/health", (_req: Request, res: Response) => {
-  res.json({ status: "ok", timestamp: new Date().toISOString() });
+app.get('/health', (_req: Request, res: Response) => {
+  res.json({ status: 'ok', timestamp: new Date().toISOString() });
 });
 
-// ── 404 ───────────────────────────────────────────────────────
-app.use((_req: Request, res: Response) => {
-  res.status(404).json({ message: "Route not found" });
+app.use((req: Request, res: Response) => {
+  res.status(404).json({ message: 'Route not found' });
 });
 
-// ── Error handler ─────────────────────────────────────────────
 app.use((err: Error, _req: Request, res: Response, _next: NextFunction) => {
   console.error(err.stack);
-  res.status(500).json({ message: "Internal Server Error" });
+  res.status(500).json({ message: 'Internal Server Error' });
 });
 
-// ── Start ─────────────────────────────────────────────────────
 app.listen(PORT, () => {
-  console.log(`React Doctor backend running on http://localhost:${PORT}`);
-});
+  console.log(`🩺 React Doctor backend running on http://localhost:${PORT}`);
+});

package/backend/src/routes/reports.ts

@@ -34,9 +34,11 @@
 import { Router, Request, Response, RequestHandler } from "express";
 import path from "path";
 import fs   from "fs";
-import db, { screenshotsDir } from "../db";
+import db from "../db";
 import { requireApiKey }      from "../middleware/auth";
 
+const screenshotsDir = "data/screenshots";
+
 const router = Router();
 
 // ── GET /api/reports ─────────────────────────────────────────

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "react-doctor-cli-dev",
-  "version": "1.0.3",
+  "version": "1.0.5",
   "description": "React performance analyzer with static analysis, runtime profiling, rule engine, and dashboard upload",
   "main": "index.js",
   "type": "commonjs",