@feardread/fear 2.0.0 → 2.0.1

package/FEAR.js

@@ -173,14 +173,11 @@ module.exports = FEAR = (() => {
 
       mailinfo.service = mailService;
       this.mailinfo = mailinfo;
-
       if ( !this.mailer ) {
         this.mailer = new smtp(this);
       }
     },
-    /**
-     * Parse allowed origins from environment
-     */
+
     getAllowedOrigins() {
       if (!this.env.ALLOWED_ORIGINS) return [];
 
@@ -190,9 +187,6 @@ module.exports = FEAR = (() => {
         .filter(origin => origin.length > 0);
     },
 
-    /**
-     * Get CORS configuration object
-     */
     getCorsConfig() {
       return {
         credentials: true,
@@ -209,9 +203,6 @@ module.exports = FEAR = (() => {
       };
     },
 
-    /**
-     * Auto-load routes from routes directory
-     */
     setupRoutes() {
       const routesDir = path.join(__dirname, "routes");
 
@@ -241,9 +232,6 @@ module.exports = FEAR = (() => {
       }
     },
 
-    /**
-     * Register a single router
-     */
     useRouter(router, routePath = DEFAULT_ROUTE_PATH, corsOptions = null) {
       if (!router || typeof router !== 'function') {
         throw new Error('Router must be a valid Express router instance');
@@ -264,9 +252,6 @@ module.exports = FEAR = (() => {
       return this;
     },
 
-    /**
-     * Register multiple routers
-     */
     useRouters(routers) {
       if (!Array.isArray(routers)) {
         throw new Error('Routers must be an array');
@@ -285,9 +270,6 @@ module.exports = FEAR = (() => {
       return this;
     },
 
-    /**
-     * Create a new router with FEAR utilities attached
-     */
     createRouter() {
       const router = express.Router();
 
@@ -305,9 +287,6 @@ module.exports = FEAR = (() => {
       return router;
     },
 
-    /**
-     * Get list of registered routers
-     */
     getRegisteredRouters() {
       return this.registeredRouters.map(info => ({
         path: info.path,
@@ -315,9 +294,6 @@ module.exports = FEAR = (() => {
       }));
     },
 
-    /**
-     * Start the HTTP server
-     */
     start(port = null) {
       const serverPort = port || this.app.get("PORT") || DEFAULT_PORT;
 
@@ -340,9 +316,6 @@ module.exports = FEAR = (() => {
       });
     },
 
-    /**
-     * Gracefully shutdown the server
-     */
     shutdown() {
       this.logger.info('Initiating graceful shutdown...');
 
@@ -382,9 +355,6 @@ module.exports = FEAR = (() => {
       });
     },
 
-    /**
-     * Close database connections
-     */
     closeDatabase() {
       return new Promise((resolve, reject) => {
         if (this.db && typeof this.db.disconnect === 'function') {
@@ -400,9 +370,6 @@ module.exports = FEAR = (() => {
       });
     },
 
-    /**
-     * Set FEAR utilities as global
-     */
     setAsGlobal() {
       global.FearRouter = {
         createRouter: () => this.createRouter(),
@@ -420,9 +387,6 @@ module.exports = FEAR = (() => {
   return FEAR;
 })();
 
-/**
- * Factory function to create new FEAR instance
- */
 exports.FearFactory = () => {
   return new FEAR();
 };

package/libs/emailer/smtp.js

@@ -24,7 +24,7 @@ module.exports = function (fear) {
                 throw new Error('Mailgun requires apiKey and domain in configuration.');
             }
 
-            const mailgun = new Mailgun(FormData);
+            const mailgun = new Mailgun(_this.mailConfig.mailgun);
             const clientOptions = {
                 username: 'api',
                 key: apiKey

package/models/category.js

@@ -125,15 +125,13 @@ categorySchema.virtual("postCount", {
 // Pre-save middleware
 categorySchema.pre("save", async function(next) {
   // Generate slug
-  if (this.isModified("name") || this.isModified("title")) {
-    const nameToSlugify = this.title || this.name;
+  if (this.isModified("title")) {
+    const nameToSlugify = this.title;
     this.slug = slugify(nameToSlugify, {
       lower: true,
       strict: true,
       remove: /[*+~.()'"!:@]/g
     });
-
-    // Ensure unique slug
     const slugRegEx = new RegExp(`^${this.slug}(-[0-9]*)?$`, "i");
     const categoriesWithSlug = await this.constructor.find({ 
       slug: slugRegEx,
@@ -145,22 +143,6 @@ categorySchema.pre("save", async function(next) {
     }
   }
 
-  // Sync name and title if one is missing
-  if (!this.name && this.title) {
-    this.name = this.title;
-  }
-  if (!this.title && this.name) {
-    this.title = this.name;
-  }
-
-  // Sync isActive with active
-  if (this.isModified("isActive")) {
-    this.active = this.isActive;
-  }
-  if (this.isModified("active")) {
-    this.isActive = this.active;
-  }
-
   // Calculate level and build ancestors
   if (this.isModified("parent")) {
     if (this.parent) {
@@ -185,43 +167,8 @@ categorySchema.pre("save", async function(next) {
       this.path = this.slug;
     }
   }
-
-  // Set SEO defaults
-  if (!this.seo.metaTitle) {
-    this.seo.metaTitle = (this.title || this.name).substring(0, 70);
-  }
-  
-  if (!this.seo.metaDescription && this.description) {
-    this.seo.metaDescription = this.description.substring(0, 160);
-  }
-
-  next();
-});
-
-// Pre-update middleware
-categorySchema.pre("findOneAndUpdate", async function(next) {
-  const update = this.getUpdate();
-  
-  if (update.name || update.title) {
-    const nameToSlugify = update.title || update.name;
-    update.slug = slugify(nameToSlugify, {
-      lower: true,
-      strict: true,
-      remove: /[*+~.()'"!:@]/g
-    });
-  }
-  
-  if (update.isActive !== undefined) {
-    update.active = update.isActive;
-  }
-  
-  if (update.active !== undefined) {
-    update.isActive = update.active;
-  }
-  
   next();
 });
-
 // Post-remove middleware to clean up references
 categorySchema.post("remove", async function(doc) {
   // Remove from parent's children

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@feardread/fear",
-  "version": "2.0.0",
+  "version": "2.0.1",
   "description": "",
   "main": "index.js",
   "scripts": {