Refactor notes functionality for shared family access

- Updated the notes API to remove member-specific access, allowing for a single shared family notebook.
- Modified frontend components to reflect the shared nature of notes, enhancing the user experience.
- Adjusted serialization and storage methods to support the new shared notes structure.
- Incremented asset versioning in index.html to ensure proper caching.

This commit improves the notes feature by centralizing access for all family members, fostering collaboration and ease of use.

app.py

@@ -126,9 +126,9 @@ def recipes() -> list:
 
 
 @app.api(name="notes")
-def notes(member: str = "elena") -> list:
-    """A member's notes, projected as note cards."""
-    return serialize.notes(member)
+def notes() -> list:
+    """The shared family notes, projected as note cards."""
+    return serialize.notes()
 
 
 @app.api(name="meal_plan")

frontend/app/app.jsx

@@ -18,7 +18,7 @@ const TITLES = {
   calendar: ["Calendar", "The shared family week"],
   recipes: ["Recipes", "Your recipe box, stored as plain Markdown"],
   meals: ["Meal plan", "A week of lunches and dinners, planned and shopped for"],
-  notes: ["Notes", "A private space for each person"],
+  notes: ["Notes", "One shared notebook for the whole family"],
   settings: ["Settings", "Your family, stored locally in data/members.json"],
 };
 
@@ -218,9 +218,7 @@ function App() {
           {route === "meals" && (
             <Meals plan={plan} onOpenRecipe={(r) => setRecipeOpen(r)} />
           )}
-          {route === "notes" && (
-            <Notes activeMember={active} members={members} />
-          )}
+          {route === "notes" && <Notes />}
           {route === "settings" && (
             <Settings
               members={members}

frontend/app/live.jsx

@@ -20,7 +20,7 @@ const MFLive = {
     return res.data[0];
   },
   recipes()       { return this._call("/recipes", {}); },
-  notes(member)   { return this._call("/notes", { member }); },
+  notes()         { return this._call("/notes", {}); },
   mealPlan()      { return this._call("/meal_plan", {}); },
   members()       { return this._call("/members", {}); },
   addMember(name, foodPreferences) {

frontend/app/screens3.jsx

@@ -122,71 +122,47 @@ function NoteCard({ note }) {
   );
 }
 
-function Notes({ activeMember, members }) {
-  const roster = members || [];
-  const [who, setWho] = useState(activeMember);
-  useEffect(() => { if (activeMember) setWho(activeMember); }, [activeMember]);
-
-  // Pull every member's notes live. Re-runs when the roster changes so a newly
-  // added member's (empty) notes show up. 'shared' has no backend file yet.
-  const [liveMap, setLiveMap] = useState(null);
-  const rosterKey = roster.map(m => m.id).join(",");
+function Notes() {
+  // One shared notebook for the whole family (data/shared/notes.md).
+  const [notes, setNotes] = useState(null);
   useEffect(() => {
     let alive = true;
-    let busy = false;
-    const run = () => {
-      if (busy) return;
-      busy = true;
-      Promise.all(roster.map(m =>
-        MFLive.notes(m.id).then(n => [m.id, n]).catch(() => [m.id, null])
-      )).then(pairs => {
-        if (!alive) return;
-        const map = {};
-        pairs.forEach(([k, v]) => { if (v != null) map[k] = v; });
-        setLiveMap(map);
-      }).finally(() => { busy = false; });
-    };
+    const run = () => MFLive.notes().then((n) => { if (alive) setNotes(n); }).catch(() => {});
     run();
-    const off = MFRefresh.subscribe(run); // pick up hand-edited notes files
+    const off = MFRefresh.subscribe(run); // pick up agent / hand edits
     return () => { alive = false; off(); };
-  }, [rosterKey]);
-  const notesFor = (id) => (liveMap && liveMap[id]) || [];
-
-  const tabs = [...roster, { id: "shared", name: "Shared", color: "var(--ash)", init: "•", full: "Whole family" }];
-  const notes = notesFor(who);
+  }, []);
+  const list = notes || [];
 
   return (
     <div className="view-wide fade-in">
       <div className="notes-layout">
         <div>
-          <div className="note-members">
-            {tabs.map(t => (
-              <button key={t.id} className={`note-mtab ${who === t.id ? "on" : ""}`} onClick={() => setWho(t.id)}>
-                {t.id === "shared" ? <Icon name="home" size={17} style={{ color: "var(--ash)" }} /> : <Avatar member={t} size="xs" />}
-                {t.id === "shared" ? "Shared" : t.name}
-                <span className="ct">{notesFor(t.id).length}</span>
-              </button>
-            ))}
-          </div>
-          <button className="btn btn-ink btn-sm" style={{ width: "100%", marginTop: 14 }}><Icon name="plus" size={15} /> New note</button>
+          <SectionH icon="notes" title="Family notes" />
+          <button className="btn btn-ink btn-sm" style={{ width: "100%", marginTop: 14 }}>
+            <Icon name="plus" size={15} /> New note
+          </button>
           <div className="muted" style={{ fontSize: 11, marginTop: 14, lineHeight: 1.6, padding: "0 4px" }}>
-            Each person's notes live in their own Markdown file. Ask Mini Fam to read or add to any of them.
+            One shared notebook for the whole family, stored as Markdown at
+            <span className="mono"> shared/notes.md</span>. Ask Mini Fam to add to it.
           </div>
         </div>
 
         <div>
           <div style={{ display: "flex", alignItems: "center", gap: 10, marginBottom: 16 }}>
-            {who !== "shared" ? <Avatar member={who} size="md" /> : <div className="av av-md" style={{ "--c": "var(--ash)" }}>•</div>}
+            <div className="av av-md" style={{ "--c": "var(--ash)" }}>
+              <Icon name="home" size={18} style={{ color: "#fff" }} />
+            </div>
             <div>
-              <div style={{ fontSize: 18, fontWeight: 600 }}>{who === "shared" ? "Shared notes" : memberById(who).name + "'s notes"}</div>
-              <div className="muted mono" style={{ fontSize: 11.5, marginTop: 2 }}>{who === "shared" ? "shared/notes.md" : `members/${who}/notes.md`}</div>
+              <div style={{ fontSize: 18, fontWeight: 600 }}>Shared notes</div>
+              <div className="muted mono" style={{ fontSize: 11.5, marginTop: 2 }}>shared/notes.md</div>
             </div>
           </div>
-          {notes.length ? (
+          {list.length ? (
             <div className="note-cols">
-              {notes.map((n, i) => <NoteCard note={n} key={who + i} />)}
+              {list.map((n, i) => <NoteCard note={n} key={i} />)}
             </div>
-          ) : <div className="empty">No notes yet.</div>}
+          ) : <div className="empty">No notes yet — ask Mini Fam to jot one down.</div>}
         </div>
       </div>
     </div>

frontend/index.html

@@ -4,11 +4,11 @@
   <meta charset="UTF-8" />
   <meta name="viewport" content="width=device-width, initial-scale=1.0" />
   <title>Mini Fam — the family assistant that stays home</title>
-  <link rel="icon" type="image/png" href="app/logo.png?v=17" />
+  <link rel="icon" type="image/png" href="app/logo.png?v=19" />
   <link rel="preconnect" href="https://fonts.googleapis.com" />
   <link rel="preconnect" href="https://fonts.gstatic.com" crossorigin />
   <link href="https://fonts.googleapis.com/css2?family=Hanken+Grotesk:wght@300;400;500;600;700&family=JetBrains+Mono:wght@400;500&display=swap" rel="stylesheet" />
-  <link rel="stylesheet" href="app/minifam.css?v=17" />
+  <link rel="stylesheet" href="app/minifam.css?v=19" />
 </head>
 <body>
   <div id="root"></div>
@@ -26,12 +26,12 @@
   <script src="https://unpkg.com/react-dom@18.3.1/umd/react-dom.development.js" integrity="sha384-u6aeetuaXnQ38mYT8rp6sbXaQe3NL9t+IBXmnYxwkUI2Hw4bsp2Wvmx4yRQF1uAm" crossorigin="anonymous"></script>
   <script src="https://unpkg.com/@babel/standalone@7.29.0/babel.min.js" integrity="sha384-m08KidiNqLdpJqLq95G/LEi8Qvjl/xUYll3QILypMoQ65QorJ9Lvtp2RXYGBFj1y" crossorigin="anonymous"></script>
 
-  <script type="text/babel" src="app/seed.jsx?v=17"></script>
-  <script type="text/babel" src="app/ui.jsx?v=17"></script>
-  <script type="text/babel" src="app/live.jsx?v=17"></script>
-  <script type="text/babel" src="app/screens1.jsx?v=17"></script>
-  <script type="text/babel" src="app/screens2.jsx?v=17"></script>
-  <script type="text/babel" src="app/screens3.jsx?v=17"></script>
-  <script type="text/babel" src="app/app.jsx?v=17"></script>
+  <script type="text/babel" src="app/seed.jsx?v=19"></script>
+  <script type="text/babel" src="app/ui.jsx?v=19"></script>
+  <script type="text/babel" src="app/live.jsx?v=19"></script>
+  <script type="text/babel" src="app/screens1.jsx?v=19"></script>
+  <script type="text/babel" src="app/screens2.jsx?v=19"></script>
+  <script type="text/babel" src="app/screens3.jsx?v=19"></script>
+  <script type="text/babel" src="app/app.jsx?v=19"></script>
 </body>
 </html>

minifam/agent.py

@@ -20,7 +20,7 @@ from . import config, tools
 client = OpenAI(base_url=config.LLM_BASE_URL, api_key=config.LLM_API_KEY)
 
 SYSTEM_PROMPT = """You are MiniFam, a warm and concise assistant that runs \
-entirely on a family's own computer. You help family members keep personal \
+entirely on a family's own computer. You help the family keep shared \
 notes, manage a shared recipe cookbook, plan weekly meals — lunch and dinner \
 for each day, always respecting each person's dietary preferences — build \
 shopping lists, and keep a shared family calendar of events and tasks.

minifam/serialize.py

@@ -145,16 +145,13 @@ def _recipe_by_title(query: str) -> dict | None:
     return _recipe_from_post(f.stem, frontmatter.load(str(f)))
 
 
-def notes(member: str) -> list[dict]:
-    """A member's notes file projected as a single checklist card.
+def notes() -> list[dict]:
+    """The shared family notebook projected as a single checklist card.
 
-    Real notes are an append-only log of "- [stamp] text" lines; we surface them
-    as one tickable card so the screen reads well and stays honest to the file.
+    Notes are an append-only log of "- [stamp] text" lines; we surface them as
+    one tickable card so the screen reads well and stays honest to the file.
     """
-    m = config.member(member)
-    if not m:
-        return []
-    raw = storage.list_notes(m["id"])
+    raw = storage.list_notes()
     items, last = [], ""
     for line in raw.splitlines():
         match = re.match(r"-\s*\[([^\]]*)\]\s*(.*)", line.strip())
@@ -167,7 +164,7 @@ def notes(member: str) -> list[dict]:
         {
             "type": "check",
             "title": "Reminders",
-            "color": m["color"],
+            "color": "#7a9c6e",
             "updated": f"Updated {last}" if last else "",
             "items": items,
         }

minifam/storage.py

@@ -26,26 +26,27 @@ def _write_frontmatter(path: Path, metadata: dict, body: str) -> None:
     path.write_text(f"---\n{fm}\n---\n\n{body}\n", encoding="utf-8")
 
 
-def _member_dir(member: str) -> Path:
-    d = config.DATA_DIR / "members" / member.lower()
+def _notes_file() -> Path:
+    """One shared notebook for the whole family: data/shared/notes.md."""
+    d = config.DATA_DIR / "shared"
     d.mkdir(parents=True, exist_ok=True)
-    return d
+    return d / "notes.md"
 
 
-def add_note(member: str, text: str) -> str:
-    path = _member_dir(member) / "notes.md"
+def add_note(text: str) -> str:
+    path = _notes_file()
     stamp = datetime.now().strftime("%Y-%m-%d %H:%M")
     with path.open("a", encoding="utf-8") as f:
         f.write(f"- [{stamp}] {text}\n")
-    return f"Saved a note for {member}."
+    return "Saved a note."
 
 
-def list_notes(member: str) -> str:
-    path = _member_dir(member) / "notes.md"
+def list_notes() -> str:
+    path = _notes_file()
     if not path.exists():
-        return f"{member} has no notes yet."
+        return "No notes yet."
     content = path.read_text(encoding="utf-8").strip()
-    return content or f"{member} has no notes yet."
+    return content or "No notes yet."
 
 
 # --- Recipes -----------------------------------------------------------------

minifam/tools/notes.py

@@ -1,4 +1,7 @@
-"""Per-member note-taking tools."""
+"""Shared family note-taking tools.
+
+Notes are a single shared notebook for the whole household (data/shared/
+notes.md) — not per person."""
 
 from . import tool
 from .. import storage
@@ -10,8 +13,8 @@ from .. import storage
         "function": {
             "name": "add_note",
             "description": (
-                "Save a personal note or reminder for a family member. "
-                "Use whenever someone wants to remember or jot down something."
+                "Save a note or reminder to the family's shared notebook. Use "
+                "whenever someone wants to jot down or remember something."
             ),
             "parameters": {
                 "type": "object",
@@ -20,10 +23,6 @@ from .. import storage
                         "type": "string",
                         "description": "The note content to save.",
                     },
-                    "member": {
-                        "type": "string",
-                        "description": "Who the note is for. Omit to use the current user.",
-                    },
                 },
                 "required": ["text"],
             },
@@ -31,8 +30,7 @@ from .. import storage
     }
 )
 def add_note(args: dict, context: dict) -> str:
-    member = args.get("member") or context["member"]
-    return storage.add_note(member, args["text"])
+    return storage.add_note(args["text"])
 
 
 @tool(
@@ -40,19 +38,10 @@ def add_note(args: dict, context: dict) -> str:
         "type": "function",
         "function": {
             "name": "list_notes",
-            "description": "List the saved notes for a family member.",
-            "parameters": {
-                "type": "object",
-                "properties": {
-                    "member": {
-                        "type": "string",
-                        "description": "Whose notes to list. Omit to use the current user.",
-                    },
-                },
-            },
+            "description": "List the family's shared notes.",
+            "parameters": {"type": "object", "properties": {}},
         },
     }
 )
 def list_notes(args: dict, context: dict) -> str:
-    member = args.get("member") or context["member"]
-    return storage.list_notes(member)
+    return storage.list_notes()