Cleaned up the way you make controllers and started working on the API vs. pages organization.

api/todo_alpine.js

@@ -0,0 +1,17 @@
+import { ToDo } from "../lib/models.js";
+import nunjucks from "nunjucks";
+
+export const get = async (req, rep) => {
+  const todo_list = await ToDo.all({});
+  rep.code(200).send(todo_list);
+}
+
+export const post = async (req, rep) => {
+  if(req.query.todo_id !== undefined) {
+    await ToDo.delete({id: req.query.todo_id});
+  } else {
+    await ToDo.insert({task: req.body.task});
+  }
+
+  await get(req, rep);
+}

commands/app.js

@@ -23,17 +23,21 @@ const configure = async (fastify, opts) => {
   const app = fastify(opts);
 
   // forces reload of the modules
-  for(let mod of fg.sync("controllers/*.js")) {
+  for(let mod of fg.sync("{api,pages}/*.js")) {
-    const control_mod = await import(`../${mod}?update=${new Date()}`);
+    const control_mod = await import(`../${mod}?update=${Date.now()}`);
-    // TODO: mappings are very simple for now
+    const pf = path.parse(mod);
-    for(let [url, controller] of Object.entries(control_mod.default)) {
+    const url = `/${pf.name}`;
-      const handler = new controller();
+
-      // this goes through all the allowed verbs
+    console.log("URL IS", url);
+
     for(let verb of ["get", "post", "put", "delete"]) {
-        if(verb in handler) {
+      const handler = control_mod[verb];
-          app[verb](`/${url}`, async (req, rep) => {
+
+      if(handler !== undefined) {
+        console.log("ADDING VERB", verb);
+        app[verb](url, async (req, rep) => {
           try {
-              await handler[verb](req, rep);
+            await handler(req, rep);
           } catch(error) {
             console.error(error);
             console.error(error.stack);
@@ -43,10 +47,14 @@ const configure = async (fastify, opts) => {
       }
     }
   }
-  }
 
   app.register(Formbody);
 
+  app.get("/pages/todo_alpine.html", (req, rep) => {
+    const result = nunjucks.render("todo_alpine.html");
+    rep.code(200).type("text/html").send(result);
+  });
+
   app.register(FastifyStatic, {
     root: path.join(path.resolve("."), 'static'),
     prefix: '/', // optional: default '/'

controllers/todo.js

@@ -1,29 +0,0 @@
-import { ToDo } from "../lib/models.js";
-import nunjucks from "nunjucks";
-
-export class Todo {
-  async get(req, rep) {
-    const todo_list = await ToDo.all({});
-
-    const result = nunjucks.render("todo.html",
-      {todo_list, name: "Zed"});
-
-    rep.code(200).type("text/html").send(result);
-  }
-
-  async post(req, rep) {
-    if(req.query.todo_id !== undefined) {
-      await ToDo.delete({id: req.query.todo_id});
-    } else {
-      console.log("ADDING TASK", req.body.task);
-      await ToDo.insert({task: req.body.task});
-    }
-
-    // this is how you can prevent the annoying double submit
-    rep.redirect("/todo");
-  }
-}
-
-// you use the export default to set the route URL?
-// I mean, it works not sure if I like it though
-export default { todo: Todo };

controllers/todo_alpine.js

@@ -1,30 +0,0 @@
-import { ToDo } from "../lib/models.js";
-import nunjucks from "nunjucks";
-
-export class Todo {
-  async get(req, rep) {
-    if(req.headers.accept === "application/json") {
-      const todo_list = await ToDo.all({});
-      rep.code(200).send(todo_list);
-    } else {
-      const result = nunjucks.render("todo_alpine.html", {name: "Zed"});
-      rep.code(200).type("text/html").send(result);
-    }
-  }
-
-  async post(req, rep) {
-    if(req.query.todo_id !== undefined) {
-      await ToDo.delete({id: req.query.todo_id});
-    } else {
-      await ToDo.insert({task: req.body.task});
-    }
-
-    await this.get(req, rep);
-  }
-}
-
-// you use the export default to set the route URL?
-// I mean, it works not sure if I like it though
-export default {
-  todo_alpine: Todo,
-};

pages/todo.js

@@ -0,0 +1,23 @@
+import { ToDo } from "../lib/models.js";
+import nunjucks from "nunjucks";
+
+export const get = async (req, rep) => {
+  const todo_list = await ToDo.all({});
+
+  const result = nunjucks.render("todo.html",
+    {todo_list, name: "Zed"});
+
+  rep.code(200).type("text/html").send(result);
+}
+
+export const post = async (req, rep) => {
+  if(req.query.todo_id !== undefined) {
+    await ToDo.delete({id: req.query.todo_id});
+  } else {
+    console.log("ADDING TASK", req.body.task);
+    await ToDo.insert({task: req.body.task});
+  }
+
+  // this is how you can prevent the annoying double submit
+  rep.redirect("/todo");
+}