npm - fixmind - Versions diffs - 1.0.2 → 1.0.3 - Mend

fixmind 1.0.2 → 1.0.3

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (16) hide show

package/README.md +87 -87
package/dist/dashboard/assets/index-Bl0HIoQR.js +14 -0
package/dist/dashboard/assets/{index-DJIWVHe0.css → index-WDmgeIrO.css} +14 -0
package/dist/dashboard/index.html +16 -16
package/dist/src/cli.js +0 -0
package/dist/src/launcher.d.ts +2 -0
package/dist/src/launcher.js +3 -0
package/dist/src/launcher.js.map +1 -0
package/dist/src/mcp.js +96 -96
package/dist/src/setup.js +15 -15
package/dist/src/storage.js +72 -72
package/dist/src/sync.js +61 -61
package/docs/mcp-integration.md +112 -104
package/package.json +45 -45
package/dist/dashboard/assets/index-BMTB62U_.js +0 -14
package/docs/sync-setup.md +0 -70

package/dist/dashboard/assets/{index-DJIWVHe0.css → index-WDmgeIrO.css} RENAMED Viewed

@@ -50,6 +50,7 @@
     --font-mono: "JetBrains Mono", monospace;
     --spacing: .25rem;
     --container-lg: 32rem;
+    --container-xl: 36rem;
     --container-2xl: 42rem;
     --text-xs: .75rem;
     --text-xs--line-height: calc(1 / .75);
@@ -730,6 +731,10 @@
     max-width: var(--container-lg);
   }
+  .max-w-xl {
+    max-width: var(--container-xl);
+  }
   .flex-1 {
     flex: 1;
   }
@@ -1057,6 +1062,10 @@
     padding: calc(var(--spacing) * 6);
   }
+  .p-8 {
+    padding: calc(var(--spacing) * 8);
+  }
   .px-2 {
     padding-inline: calc(var(--spacing) * 2);
   }
@@ -1361,6 +1370,11 @@
     box-shadow: var(--tw-inset-shadow), var(--tw-inset-ring-shadow), var(--tw-ring-offset-shadow), var(--tw-ring-shadow), var(--tw-shadow);
   }
+  .shadow-\[0_40px_100px_-30px_rgba\(0\,0\,0\,0\.35\)\] {
+    --tw-shadow: 0 40px 100px -30px var(--tw-shadow-color, #00000059);
+    box-shadow: var(--tw-inset-shadow), var(--tw-inset-ring-shadow), var(--tw-ring-offset-shadow), var(--tw-ring-shadow), var(--tw-shadow);
+  }
   .ring-2 {
     --tw-ring-shadow: var(--tw-ring-inset, ) 0 0 0 calc(2px + var(--tw-ring-offset-width)) var(--tw-ring-color, currentcolor);
     box-shadow: var(--tw-inset-shadow), var(--tw-inset-ring-shadow), var(--tw-ring-offset-shadow), var(--tw-ring-shadow), var(--tw-shadow);

package/dist/dashboard/index.html CHANGED Viewed

@@ -1,16 +1,16 @@
-<!doctype html>
-<html lang="en">
-  <head>
-    <meta charset="UTF-8" />
-    <meta name="viewport" content="width=device-width, initial-scale=1.0" />
-    <meta name="color-scheme" content="dark" />
-    <title>Fixmind</title>
-    <link rel="preconnect" href="https://fonts.googleapis.com">
-    <link rel="stylesheet" href="https://fonts.googleapis.com/css2?family=Space+Grotesk:wght@500;600;700&family=Inter:wght@400;500;600&family=JetBrains+Mono:wght@400;500;600&display=swap">
-    <script type="module" crossorigin src="/assets/index-BMTB62U_.js"></script>
-    <link rel="stylesheet" crossorigin href="/assets/index-DJIWVHe0.css">
-  </head>
-  <body>
-    <div id="root"></div>
-  </body>
-</html>
+<!doctype html>
+<html lang="en">
+  <head>
+    <meta charset="UTF-8" />
+    <meta name="viewport" content="width=device-width, initial-scale=1.0" />
+    <meta name="color-scheme" content="dark" />
+    <title>Fixmind</title>
+    <link rel="preconnect" href="https://fonts.googleapis.com">
+    <link rel="stylesheet" href="https://fonts.googleapis.com/css2?family=Space+Grotesk:wght@500;600;700&family=Inter:wght@400;500;600&family=JetBrains+Mono:wght@400;500;600&display=swap">
+    <script type="module" crossorigin src="/assets/index-Bl0HIoQR.js"></script>
+    <link rel="stylesheet" crossorigin href="/assets/index-WDmgeIrO.css">
+  </head>
+  <body>
+    <div id="root"></div>
+  </body>
+</html>

package/dist/src/cli.js CHANGED Viewed

File without changes

package/dist/src/launcher.d.ts ADDED Viewed

	@@ -0,0 +1,2 @@
1	+ #!/usr/bin/env node
2	+ import "./cli.js";

package/dist/src/launcher.js ADDED Viewed

@@ -0,0 +1,3 @@
+#!/usr/bin/env node
+import "./cli.js";
+//# sourceMappingURL=launcher.js.map

package/dist/src/launcher.js.map ADDED Viewed

	@@ -0,0 +1 @@
1	+ {"version":3,"file":"launcher.js","sourceRoot":"","sources":["../../src/launcher.ts"],"names":[],"mappings":";AACA,OAAO,UAAU,CAAC"}

package/dist/src/mcp.js CHANGED Viewed

@@ -8,102 +8,102 @@ import { readGitContext } from "./git.js";
 import { databasePath } from "./paths.js";
 import { initializeDataDirectory, createLessonStore } from "./storage.js";
 import { assessLessonQuality, validateLessonInput } from "./validation.js";
-export const MCP_INSTRUCTIONS = `
-You are the fixmind learning recorder. Your job is to capture lessons that help developers improve over time.
-BEFORE calling save_lesson, run this checklist:
-  1. Was real logic fixed? (a bug, an incorrect assumption, a missing guard, wrong API usage, bad state management, etc.)
-  2. Does the developer now understand something they did not understand before?
-  3. Can you write a concrete badCodeExample showing the wrong pattern?
-  4. Can you explain WHY the old code was wrong, not just WHAT changed?
-  5. Can you state when this fix would NOT apply - a different framework/version,
-     a context where the old code is actually correct, or a case that needs a
-     different fix entirely?
-If any answer is NO, do NOT call save_lesson.
-DO NOT save a lesson for:
-  - Moving code to a different file (pure relocation, no logic change)
-  - Renaming variables, functions, or files
-  - Formatting-only changes (whitespace, semicolons, quotes, line breaks)
-  - Generated files or build artifacts
-  - Adding a comment or doc-string without changing logic
-  - Splitting one file into multiple files without changing logic
-  - Purely mechanical refactors with no new understanding gained
-  - UI-only or styling-only changes (CSS, className, spacing, color, copy
-    tweaks) that don't change behavior or fix a bug
-  - Any change where you cannot point to a behavior that was wrong before
-    and correct after
-FIELD GUIDE - each field has a distinct job. Do not let them repeat each other:
-  - problem: The user-visible SYMPTOM. What broke, what error appeared, or
-    what the user/tester observed. This is the "what happened".
-  - mistake: The WRONG ASSUMPTION or approach in the code that caused the
-    symptom - describe the flawed thinking, not just the line that changed.
-  - rootCause: WHY the mistake produced the symptom. This must add new
-    information beyond problem and mistake - if you find yourself repeating
-    either of them, dig one level deeper (e.g. "the API resolves before the
-    body streams" rather than "the data was empty").
-  - fixSummary: WHY the new code avoids the root cause - not just what code
-    changed. A reader should understand why this fix actually works, so they
-    could apply the same reasoning elsewhere.
-REQUIRED fields — every lesson must answer these:
-  - mistake: What the developer actually did wrong in their thinking (not just what line changed)
-  - takeaway: One sentence to remember. Example: "fetch() resolves when headers arrive, not when the body is parsed."
-  - whenNotApplicable: When this advice does NOT apply. Example: "Doesn't apply inside Server Components, which can't use useEffect at all."
-STRONGLY RECOMMENDED fields — provide these when code is involved:
-  - badCodeExample: The minimal broken snippet. Example: "const data = await fetch(url)"
-  - goodCodeExample: The corrected version, paired with badCodeExample. Example: "const res = await fetch(url); const data = await res.json();"
-  - mistakePattern: A 2–4 word reusable category. Examples: "Missing await", "Stale closure", "Off-by-one", "Wrong event lifetime"
-  - tags: 1-3 entries naming the APIs/concepts involved, e.g. { "name": "MDN: URL.revokeObjectURL", "url": "https://developer.mozilla.org/en-US/docs/Web/API/URL/revokeObjectURL_static" }. Only set url when you are confident it is a real, official documentation page (MDN, the framework's own docs). If unsure, omit url and the tag is shown as a plain label.
-REVIEW QUESTIONS - write TRANSFER questions, not recall questions:
-  - Bad (recall): "What did you change?" / "Summarize the fix."
-  - Good (transfer): a question that asks the developer to apply the same
-    reasoning to a DIFFERENT situation, spot the same mistake in different
-    code, or predict what would happen under slightly different conditions.
-  - expectedAnswer should reference the underlying principle (rootCause /
-    takeaway), not just describe the diff.
-THE SAVE CAN BE REJECTED. If save_lesson returns an error, it means
-the lesson didn't clear the quality bar. Common reasons, with the fix for each:
-  - rootCause or fixSummary just repeated another field word-for-word - rewrite
-    it to add the missing WHY (the mechanism, not a restatement).
-  - mistake and fixSummary both read as a pure refactor with no code comparison
-    - add badCodeExample/goodCodeExample, or don't save this as a lesson.
-  - problem/mistake/fixSummary read as a UI-only or styling-only change (colors,
-    spacing, className, fonts, etc.) with no described behavior change - say
-    what BROKE (an element became unclickable, content overflowed and hid other
-    content, etc.), not just what looked different.
-  - mistake, rootCause, or fixSummary is a generic placeholder like "fixed the
-    bug" or "the code was wrong" - name the actual function, condition, or
-    value involved.
-  - every reviewQuestion is a recall question ("what did you change") - rewrite
-    or add one TRANSFER question per the REVIEW QUESTIONS section above.
-Re-read the FIELD GUIDE above, rewrite the offending field with real new
-information, and try again. If you genuinely cannot explain a root cause beyond
-the symptom, this was probably not a learning-worthy fix - do not save it.
-For codeExample, badCodeExample, and goodCodeExample: write multi-line snippets
-with real line breaks and normal indentation, the same way you'd write the code
-in a file. Do not flatten the snippet onto one line using the two characters
-"\" + "n" as a stand-in for a newline - the dashboard renders these fields
-verbatim, so literal "\n" text shows up as "\n" instead of a line break.
-SUPERSEDING A PREVIOUS LESSON:
-If you previously called save_lesson for a fix that turned out NOT to
-work, and you are now saving a lesson for the CORRECT fix, set:
-  - supersedesLessonId: the id of the earlier (wrong) lesson, from its
-    "Saved learning lesson <id>" response in this conversation.
-  - supersedeReason: one sentence on what was wrong with the earlier fix and
-    why this one replaces it.
-The old lesson stops appearing in search and spaced-repetition review, but
-stays in history with a link to this corrected lesson. Only use this for
-fixes that turned out to be incorrect or incomplete — not for pure rewording.
-Write as a teacher, not as an agent log. Keep lessons short and human-readable.
+export const MCP_INSTRUCTIONS = `
+You are the fixmind learning recorder. Your job is to capture lessons that help developers improve over time.
+BEFORE calling save_lesson, run this checklist:
+  1. Was real logic fixed? (a bug, an incorrect assumption, a missing guard, wrong API usage, bad state management, etc.)
+  2. Does the developer now understand something they did not understand before?
+  3. Can you write a concrete badCodeExample showing the wrong pattern?
+  4. Can you explain WHY the old code was wrong, not just WHAT changed?
+  5. Can you state when this fix would NOT apply - a different framework/version,
+     a context where the old code is actually correct, or a case that needs a
+     different fix entirely?
+If any answer is NO, do NOT call save_lesson.
+DO NOT save a lesson for:
+  - Moving code to a different file (pure relocation, no logic change)
+  - Renaming variables, functions, or files
+  - Formatting-only changes (whitespace, semicolons, quotes, line breaks)
+  - Generated files or build artifacts
+  - Adding a comment or doc-string without changing logic
+  - Splitting one file into multiple files without changing logic
+  - Purely mechanical refactors with no new understanding gained
+  - UI-only or styling-only changes (CSS, className, spacing, color, copy
+    tweaks) that don't change behavior or fix a bug
+  - Any change where you cannot point to a behavior that was wrong before
+    and correct after
+FIELD GUIDE - each field has a distinct job. Do not let them repeat each other:
+  - problem: The user-visible SYMPTOM. What broke, what error appeared, or
+    what the user/tester observed. This is the "what happened".
+  - mistake: The WRONG ASSUMPTION or approach in the code that caused the
+    symptom - describe the flawed thinking, not just the line that changed.
+  - rootCause: WHY the mistake produced the symptom. This must add new
+    information beyond problem and mistake - if you find yourself repeating
+    either of them, dig one level deeper (e.g. "the API resolves before the
+    body streams" rather than "the data was empty").
+  - fixSummary: WHY the new code avoids the root cause - not just what code
+    changed. A reader should understand why this fix actually works, so they
+    could apply the same reasoning elsewhere.
+REQUIRED fields — every lesson must answer these:
+  - mistake: What the developer actually did wrong in their thinking (not just what line changed)
+  - takeaway: One sentence to remember. Example: "fetch() resolves when headers arrive, not when the body is parsed."
+  - whenNotApplicable: When this advice does NOT apply. Example: "Doesn't apply inside Server Components, which can't use useEffect at all."
+STRONGLY RECOMMENDED fields — provide these when code is involved:
+  - badCodeExample: The minimal broken snippet. Example: "const data = await fetch(url)"
+  - goodCodeExample: The corrected version, paired with badCodeExample. Example: "const res = await fetch(url); const data = await res.json();"
+  - mistakePattern: A 2–4 word reusable category. Examples: "Missing await", "Stale closure", "Off-by-one", "Wrong event lifetime"
+  - tags: 1-3 entries naming the APIs/concepts involved, e.g. { "name": "MDN: URL.revokeObjectURL", "url": "https://developer.mozilla.org/en-US/docs/Web/API/URL/revokeObjectURL_static" }. Only set url when you are confident it is a real, official documentation page (MDN, the framework's own docs). If unsure, omit url and the tag is shown as a plain label.
+REVIEW QUESTIONS - write TRANSFER questions, not recall questions:
+  - Bad (recall): "What did you change?" / "Summarize the fix."
+  - Good (transfer): a question that asks the developer to apply the same
+    reasoning to a DIFFERENT situation, spot the same mistake in different
+    code, or predict what would happen under slightly different conditions.
+  - expectedAnswer should reference the underlying principle (rootCause /
+    takeaway), not just describe the diff.
+THE SAVE CAN BE REJECTED. If save_lesson returns an error, it means
+the lesson didn't clear the quality bar. Common reasons, with the fix for each:
+  - rootCause or fixSummary just repeated another field word-for-word - rewrite
+    it to add the missing WHY (the mechanism, not a restatement).
+  - mistake and fixSummary both read as a pure refactor with no code comparison
+    - add badCodeExample/goodCodeExample, or don't save this as a lesson.
+  - problem/mistake/fixSummary read as a UI-only or styling-only change (colors,
+    spacing, className, fonts, etc.) with no described behavior change - say
+    what BROKE (an element became unclickable, content overflowed and hid other
+    content, etc.), not just what looked different.
+  - mistake, rootCause, or fixSummary is a generic placeholder like "fixed the
+    bug" or "the code was wrong" - name the actual function, condition, or
+    value involved.
+  - every reviewQuestion is a recall question ("what did you change") - rewrite
+    or add one TRANSFER question per the REVIEW QUESTIONS section above.
+Re-read the FIELD GUIDE above, rewrite the offending field with real new
+information, and try again. If you genuinely cannot explain a root cause beyond
+the symptom, this was probably not a learning-worthy fix - do not save it.
+For codeExample, badCodeExample, and goodCodeExample: write multi-line snippets
+with real line breaks and normal indentation, the same way you'd write the code
+in a file. Do not flatten the snippet onto one line using the two characters
+"\" + "n" as a stand-in for a newline - the dashboard renders these fields
+verbatim, so literal "\n" text shows up as "\n" instead of a line break.
+SUPERSEDING A PREVIOUS LESSON:
+If you previously called save_lesson for a fix that turned out NOT to
+work, and you are now saving a lesson for the CORRECT fix, set:
+  - supersedesLessonId: the id of the earlier (wrong) lesson, from its
+    "Saved learning lesson <id>" response in this conversation.
+  - supersedeReason: one sentence on what was wrong with the earlier fix and
+    why this one replaces it.
+The old lesson stops appearing in search and spaced-repetition review, but
+stays in history with a link to this corrected lesson. Only use this for
+fixes that turned out to be incorrect or incomplete — not for pure rewording.
+Write as a teacher, not as an agent log. Keep lessons short and human-readable.
 `.trim();
 const reviewQuestionSchema = z.object({
     question: z.string().trim().min(1).describe("A TRANSFER question - apply the lesson to a different situation, spot the same mistake elsewhere, or predict an outcome. Do not ask 'what did you change' or 'summarize the fix'."),

package/dist/src/setup.js CHANGED Viewed

@@ -127,21 +127,21 @@ function configureCursor(homeDirectory, projectDirectory, scope, server, dryRun)
     };
 }
 const INSTRUCTION_MARKER = "fixmind:instructions";
-const INSTRUCTION_BODY = `\
-After fixing any meaningful bug or resolving a real coding mistake, always call \
-the fixmind save_lesson MCP tool before finishing your response.
-Call it for: bugs, incorrect API usage, wrong assumptions, bad state management, \
-logic errors, missing cleanup, security issues.
-Skip it for: renaming files or variables, formatting changes, moving code between \
-files, generated files, or adding comments without logic changes.
-When code is involved, always provide badCodeExample and goodCodeExample.
-When the mistake involves a documented API or concept, add tags: [{ name, url }] \
-linking to the official docs (MDN, the framework's docs, etc.). Only set url when \
-you're confident it's a real page — otherwise omit it and the tag still shows as a \
+const INSTRUCTION_BODY = `\
+After fixing any meaningful bug or resolving a real coding mistake, always call \
+the fixmind save_lesson MCP tool before finishing your response.
+Call it for: bugs, incorrect API usage, wrong assumptions, bad state management, \
+logic errors, missing cleanup, security issues.
+Skip it for: renaming files or variables, formatting changes, moving code between \
+files, generated files, or adding comments without logic changes.
+When code is involved, always provide badCodeExample and goodCodeExample.
+When the mistake involves a documented API or concept, add tags: [{ name, url }] \
+linking to the official docs (MDN, the framework's docs, etc.). Only set url when \
+you're confident it's a real page — otherwise omit it and the tag still shows as a \
 label.`;
 function instructionBlock(client) {
     if (client === "cursor") {

package/dist/src/storage.js CHANGED Viewed

@@ -16,42 +16,42 @@ export function initializeDataDirectory() {
 export function createLessonStore(filePath = databasePath()) {
     fs.mkdirSync(path.dirname(filePath), { recursive: true });
     const db = new DatabaseSync(filePath);
-    db.exec(`
-    CREATE TABLE IF NOT EXISTS lessons (
-      id TEXT PRIMARY KEY,
-      created_at TEXT NOT NULL,
-      updated_at TEXT NOT NULL,
-      tool TEXT NOT NULL,
-      project_path TEXT NOT NULL,
-      title TEXT NOT NULL,
-      original_prompt TEXT NOT NULL,
-      problem TEXT NOT NULL,
-      mistake TEXT NOT NULL,
-      root_cause TEXT NOT NULL,
-      fix_summary TEXT NOT NULL,
-      takeaway TEXT,
-      mistake_pattern TEXT,
-      when_not_applicable TEXT,
-      concepts TEXT NOT NULL,
-      files_changed TEXT NOT NULL,
-      code_example TEXT,
-      bad_code_example TEXT,
-      good_code_example TEXT,
-      code_explanation TEXT,
-      practice_task TEXT,
-      review_questions TEXT NOT NULL,
-      understanding TEXT NOT NULL,
-      next_review_at TEXT NOT NULL,
-      review_count INTEGER NOT NULL DEFAULT 0,
-      source_diff TEXT,
-      tags TEXT NOT NULL,
-      status TEXT NOT NULL DEFAULT 'active',
-      superseded_by TEXT,
-      supersedes TEXT,
-      supersede_reason TEXT
-    );
-    CREATE INDEX IF NOT EXISTS idx_lessons_created_at ON lessons(created_at DESC);
-    CREATE INDEX IF NOT EXISTS idx_lessons_next_review_at ON lessons(next_review_at);
+    db.exec(`
+    CREATE TABLE IF NOT EXISTS lessons (
+      id TEXT PRIMARY KEY,
+      created_at TEXT NOT NULL,
+      updated_at TEXT NOT NULL,
+      tool TEXT NOT NULL,
+      project_path TEXT NOT NULL,
+      title TEXT NOT NULL,
+      original_prompt TEXT NOT NULL,
+      problem TEXT NOT NULL,
+      mistake TEXT NOT NULL,
+      root_cause TEXT NOT NULL,
+      fix_summary TEXT NOT NULL,
+      takeaway TEXT,
+      mistake_pattern TEXT,
+      when_not_applicable TEXT,
+      concepts TEXT NOT NULL,
+      files_changed TEXT NOT NULL,
+      code_example TEXT,
+      bad_code_example TEXT,
+      good_code_example TEXT,
+      code_explanation TEXT,
+      practice_task TEXT,
+      review_questions TEXT NOT NULL,
+      understanding TEXT NOT NULL,
+      next_review_at TEXT NOT NULL,
+      review_count INTEGER NOT NULL DEFAULT 0,
+      source_diff TEXT,
+      tags TEXT NOT NULL,
+      status TEXT NOT NULL DEFAULT 'active',
+      superseded_by TEXT,
+      supersedes TEXT,
+      supersede_reason TEXT
+    );
+    CREATE INDEX IF NOT EXISTS idx_lessons_created_at ON lessons(created_at DESC);
+    CREATE INDEX IF NOT EXISTS idx_lessons_next_review_at ON lessons(next_review_at);
   `);
     return {
         save(input) {
@@ -90,17 +90,17 @@ export function createLessonStore(filePath = databasePath()) {
                 tags: input.tags ?? [],
                 status: "active",
             };
-            db.prepare(`
-        INSERT INTO lessons (
-          id, created_at, updated_at, tool, project_path, title, original_prompt,
-          problem, mistake, root_cause, fix_summary, takeaway, mistake_pattern,
-          when_not_applicable, concepts, files_changed, code_example, bad_code_example,
-          good_code_example, code_explanation, practice_task, review_questions,
-          understanding, next_review_at, review_count, source_diff, tags,
-          status, superseded_by, supersedes, supersede_reason
-        ) VALUES (
-          ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, NULL, NULL, NULL
-        )
+            db.prepare(`
+        INSERT INTO lessons (
+          id, created_at, updated_at, tool, project_path, title, original_prompt,
+          problem, mistake, root_cause, fix_summary, takeaway, mistake_pattern,
+          when_not_applicable, concepts, files_changed, code_example, bad_code_example,
+          good_code_example, code_explanation, practice_task, review_questions,
+          understanding, next_review_at, review_count, source_diff, tags,
+          status, superseded_by, supersedes, supersede_reason
+        ) VALUES (
+          ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, NULL, NULL, NULL
+        )
       `).run(lesson.id, lesson.createdAt, lesson.updatedAt, lesson.tool, lesson.projectPath, lesson.title, lesson.originalPrompt, lesson.problem, lesson.mistake, lesson.rootCause, lesson.fixSummary, lesson.takeaway ?? null, lesson.mistakePattern ?? null, lesson.whenNotApplicable ?? null, JSON.stringify(lesson.concepts), JSON.stringify(lesson.filesChanged), lesson.codeExample ?? null, lesson.badCodeExample ?? null, lesson.goodCodeExample ?? null, lesson.codeExplanation ?? null, lesson.practiceTask ?? null, JSON.stringify(lesson.reviewQuestions), lesson.understanding, lesson.nextReviewAt, lesson.reviewCount, lesson.sourceDiff ?? null, JSON.stringify(lesson.tags), lesson.status);
             if (input.supersedesLessonId && this.get(input.supersedesLessonId)) {
                 this.supersede(input.supersedesLessonId, lesson.id, input.supersedeReason);
@@ -120,31 +120,31 @@ export function createLessonStore(filePath = databasePath()) {
             return db.prepare("SELECT * FROM lessons WHERE updated_at > ? ORDER BY updated_at ASC").all(timestamp).map(fromDb);
         },
         upsertFromRemote(lesson) {
-            db.prepare(`
-        INSERT INTO lessons (
-          id, created_at, updated_at, tool, project_path, title, original_prompt,
-          problem, mistake, root_cause, fix_summary, takeaway, mistake_pattern,
-          when_not_applicable, concepts, files_changed, code_example, bad_code_example,
-          good_code_example, code_explanation, practice_task, review_questions,
-          understanding, next_review_at, review_count, source_diff, tags,
-          status, superseded_by, supersedes, supersede_reason
-        ) VALUES (
-          ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?
-        )
-        ON CONFLICT(id) DO UPDATE SET
-          created_at=excluded.created_at, updated_at=excluded.updated_at, tool=excluded.tool,
-          project_path=excluded.project_path, title=excluded.title, original_prompt=excluded.original_prompt,
-          problem=excluded.problem, mistake=excluded.mistake, root_cause=excluded.root_cause,
-          fix_summary=excluded.fix_summary, takeaway=excluded.takeaway, mistake_pattern=excluded.mistake_pattern,
-          when_not_applicable=excluded.when_not_applicable, concepts=excluded.concepts,
-          files_changed=excluded.files_changed, code_example=excluded.code_example,
-          bad_code_example=excluded.bad_code_example, good_code_example=excluded.good_code_example,
-          code_explanation=excluded.code_explanation, practice_task=excluded.practice_task,
-          review_questions=excluded.review_questions, understanding=excluded.understanding,
-          next_review_at=excluded.next_review_at, review_count=excluded.review_count,
-          source_diff=excluded.source_diff, tags=excluded.tags, status=excluded.status,
-          superseded_by=excluded.superseded_by, supersedes=excluded.supersedes,
-          supersede_reason=excluded.supersede_reason
+            db.prepare(`
+        INSERT INTO lessons (
+          id, created_at, updated_at, tool, project_path, title, original_prompt,
+          problem, mistake, root_cause, fix_summary, takeaway, mistake_pattern,
+          when_not_applicable, concepts, files_changed, code_example, bad_code_example,
+          good_code_example, code_explanation, practice_task, review_questions,
+          understanding, next_review_at, review_count, source_diff, tags,
+          status, superseded_by, supersedes, supersede_reason
+        ) VALUES (
+          ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?
+        )
+        ON CONFLICT(id) DO UPDATE SET
+          created_at=excluded.created_at, updated_at=excluded.updated_at, tool=excluded.tool,
+          project_path=excluded.project_path, title=excluded.title, original_prompt=excluded.original_prompt,
+          problem=excluded.problem, mistake=excluded.mistake, root_cause=excluded.root_cause,
+          fix_summary=excluded.fix_summary, takeaway=excluded.takeaway, mistake_pattern=excluded.mistake_pattern,
+          when_not_applicable=excluded.when_not_applicable, concepts=excluded.concepts,
+          files_changed=excluded.files_changed, code_example=excluded.code_example,
+          bad_code_example=excluded.bad_code_example, good_code_example=excluded.good_code_example,
+          code_explanation=excluded.code_explanation, practice_task=excluded.practice_task,
+          review_questions=excluded.review_questions, understanding=excluded.understanding,
+          next_review_at=excluded.next_review_at, review_count=excluded.review_count,
+          source_diff=excluded.source_diff, tags=excluded.tags, status=excluded.status,
+          superseded_by=excluded.superseded_by, supersedes=excluded.supersedes,
+          supersede_reason=excluded.supersede_reason
       `).run(lesson.id, lesson.createdAt, lesson.updatedAt, lesson.tool, lesson.projectPath, lesson.title, lesson.originalPrompt, lesson.problem, lesson.mistake, lesson.rootCause, lesson.fixSummary, lesson.takeaway ?? null, lesson.mistakePattern ?? null, lesson.whenNotApplicable ?? null, JSON.stringify(lesson.concepts), JSON.stringify(lesson.filesChanged), lesson.codeExample ?? null, lesson.badCodeExample ?? null, lesson.goodCodeExample ?? null, lesson.codeExplanation ?? null, lesson.practiceTask ?? null, JSON.stringify(lesson.reviewQuestions), lesson.understanding, lesson.nextReviewAt, lesson.reviewCount, lesson.sourceDiff ?? null, JSON.stringify(lesson.tags), lesson.status, lesson.supersededBy ?? null, lesson.supersedes ?? null, lesson.supersedeReason ?? null);
         },
         search(query, options = {}) {

package/dist/src/sync.js CHANGED Viewed

@@ -46,67 +46,67 @@ function escapeHtml(value) {
 }
 function oauthCallbackPage(options) {
     const tint = options.ok ? "124,92,255" : "255,92,114";
-    return `<!doctype html>
-<html lang="en">
-<head>
-<meta charset="utf-8">
-<title>Fixmind</title>
-<style>
-  :root { color-scheme: dark; }
-  body {
-    margin: 0; min-height: 100vh; display: flex; align-items: center; justify-content: center;
-    position: relative; overflow: hidden;
-    background-color: #08090d; color: #e9ecf4;
-    font-family: "Inter", -apple-system, BlinkMacSystemFont, sans-serif;
-  }
-  body::before {
-    content: ""; position: absolute; inset: 0; pointer-events: none;
-    background-image:
-      linear-gradient(to right, rgba(35,40,56,0.6) 1px, transparent 1px),
-      linear-gradient(to bottom, rgba(35,40,56,0.6) 1px, transparent 1px);
-    background-size: 48px 48px;
-    mask-image: radial-gradient(circle at 50% 35%, black, transparent 75%);
-  }
-  body::after {
-    content: ""; position: absolute; left: 50%; top: 0; width: 760px; height: 420px;
-    transform: translateX(-50%); pointer-events: none; border-radius: 9999px;
-    background: rgba(${tint},0.15); filter: blur(130px);
-  }
-  .card {
-    position: relative; z-index: 1; text-align: center; padding: 2.5rem 3rem; border-radius: 16px;
-    border: 1px solid #232838; background: rgba(17,20,27,0.92);
-  }
-  .icon {
-    width: 48px; height: 48px; margin: 0 auto 1.25rem; border-radius: 50%;
-    display: flex; align-items: center; justify-content: center; font-size: 22px;
-    background: rgba(${tint},0.15);
-    color: ${options.ok ? "#7c5cff" : "#ff5c72"};
-  }
-  h1 {
-    font-family: "Space Grotesk", "Inter", sans-serif; font-size: 1.25rem; font-weight: 600;
-    margin: 0 0 0.5rem;
-  }
-  p { margin: 0; color: #828a9c; font-size: 0.95rem; line-height: 1.5; }
-  .redirect {
-    display: inline-block; margin-top: 1.5rem; padding: 0.6rem 1.5rem; border-radius: 8px;
-    background: #7c5cff; color: #08090d; font-weight: 600; font-size: 0.9rem;
-    text-decoration: none;
-  }
-  .brand {
-    margin-top: 2rem; font-size: 0.75rem; letter-spacing: 0.1em; text-transform: uppercase;
-    color: #4b3aae;
-  }
-</style>
-</head>
-<body>
-  <div class="card">
-    <div class="icon">${options.ok ? "&#10003;" : "&#33;"}</div>
-    <h1>${options.ok ? "You're signed in" : "Sign in failed"}</h1>
-    <p>${escapeHtml(options.message)}</p>
-    ${options.redirectUrl ? `<a class="redirect" href="${escapeHtml(options.redirectUrl)}">Try again</a>` : ""}
-    <div class="brand">Fixmind</div>
-  </div>
-</body>
+    return `<!doctype html>
+<html lang="en">
+<head>
+<meta charset="utf-8">
+<title>Fixmind</title>
+<style>
+  :root { color-scheme: dark; }
+  body {
+    margin: 0; min-height: 100vh; display: flex; align-items: center; justify-content: center;
+    position: relative; overflow: hidden;
+    background-color: #08090d; color: #e9ecf4;
+    font-family: "Inter", -apple-system, BlinkMacSystemFont, sans-serif;
+  }
+  body::before {
+    content: ""; position: absolute; inset: 0; pointer-events: none;
+    background-image:
+      linear-gradient(to right, rgba(35,40,56,0.6) 1px, transparent 1px),
+      linear-gradient(to bottom, rgba(35,40,56,0.6) 1px, transparent 1px);
+    background-size: 48px 48px;
+    mask-image: radial-gradient(circle at 50% 35%, black, transparent 75%);
+  }
+  body::after {
+    content: ""; position: absolute; left: 50%; top: 0; width: 760px; height: 420px;
+    transform: translateX(-50%); pointer-events: none; border-radius: 9999px;
+    background: rgba(${tint},0.15); filter: blur(130px);
+  }
+  .card {
+    position: relative; z-index: 1; text-align: center; padding: 2.5rem 3rem; border-radius: 16px;
+    border: 1px solid #232838; background: rgba(17,20,27,0.92);
+  }
+  .icon {
+    width: 48px; height: 48px; margin: 0 auto 1.25rem; border-radius: 50%;
+    display: flex; align-items: center; justify-content: center; font-size: 22px;
+    background: rgba(${tint},0.15);
+    color: ${options.ok ? "#7c5cff" : "#ff5c72"};
+  }
+  h1 {
+    font-family: "Space Grotesk", "Inter", sans-serif; font-size: 1.25rem; font-weight: 600;
+    margin: 0 0 0.5rem;
+  }
+  p { margin: 0; color: #828a9c; font-size: 0.95rem; line-height: 1.5; }
+  .redirect {
+    display: inline-block; margin-top: 1.5rem; padding: 0.6rem 1.5rem; border-radius: 8px;
+    background: #7c5cff; color: #08090d; font-weight: 600; font-size: 0.9rem;
+    text-decoration: none;
+  }
+  .brand {
+    margin-top: 2rem; font-size: 0.75rem; letter-spacing: 0.1em; text-transform: uppercase;
+    color: #4b3aae;
+  }
+</style>
+</head>
+<body>
+  <div class="card">
+    <div class="icon">${options.ok ? "&#10003;" : "&#33;"}</div>
+    <h1>${options.ok ? "You're signed in" : "Sign in failed"}</h1>
+    <p>${escapeHtml(options.message)}</p>
+    ${options.redirectUrl ? `<a class="redirect" href="${escapeHtml(options.redirectUrl)}">Try again</a>` : ""}
+    <div class="brand">Fixmind</div>
+  </div>
+</body>
 </html>`;
 }
 function waitForOAuthCode(port, authUrl) {