dsh-nerd-mode 0.4.5 → 0.4.6

package/client.js

@@ -1556,6 +1556,24 @@ window.__ModuleLoader__.load({
       return model;
     }
 
+    // The page fills the dock body so the picker and course name stay in view
+    // while only the lecture rows scroll. The scrollport owns the side padding,
+    // so its bar sits at the page edge rather than inside the text.
+    const lecturesPanelStyle = { ...panelStyle, height: '100%', padding: '24px 0 0' };
+    const lecturesHeaderStyle = { ...panelHeaderStyle, padding: '0 16px' };
+    const lecturesMessageStyle = { ...panelMutedStyle, padding: '0 16px' };
+    const lecturesHeadingStyle = { ...panelHeadingStyle, padding: '0 16px' };
+    const lecturesSectionStyle = { ...panelSectionStyle, flex: 1, minHeight: 0 };
+    const lecturesScrollStyle = {
+      display: 'flex',
+      flex: 1,
+      flexDirection: 'column',
+      gap: 10,
+      minHeight: 0,
+      overflowY: 'auto',
+      padding: '0 16px 40px',
+    };
+
     // The rows and the fold are the only elements the host cannot style for us.
     const LECTURES_CSS = '.lectures-entry:focus-visible,.lectures-fold:focus-visible{outline:2px solid var(--dsw-alias-state-business-primary, currentColor);outline-offset:2px}';
     const lectureRowStyle = {
@@ -1633,14 +1651,14 @@ window.__ModuleLoader__.load({
     function lectureTopic(group, open, onToggle, onOpen) {
       const main = group.entries.filter((entry) => entry.mark !== 'archived');
       const archived = group.entries.filter((entry) => entry.mark === 'archived');
-      const nodes = [React.createElement('h2', { key: 'heading', style: panelHeadingStyle }, group.topic)];
+      const rows = [];
       if (group.entries.length === 0) {
-        nodes.push(React.createElement('p', { key: 'empty', style: panelMutedStyle }, 'No lectures yet.'));
+        rows.push(React.createElement('p', { key: 'empty', style: panelMutedStyle }, 'No lectures yet.'));
       } else if (main.length > 0) {
-        nodes.push(React.createElement('ul', { key: 'main', style: panelListStyle }, ...main.map((entry) => lectureRow(entry, onOpen))));
+        rows.push(React.createElement('ul', { key: 'main', style: panelListStyle }, ...main.map((entry) => lectureRow(entry, onOpen))));
       }
       if (archived.length > 0) {
-        nodes.push(React.createElement('button', {
+        rows.push(React.createElement('button', {
           key: 'fold',
           type: 'button',
           className: 'lectures-fold',
@@ -1649,10 +1667,13 @@ window.__ModuleLoader__.load({
           style: lectureFoldStyle,
         }, `Archived lectures (${archived.length})`));
         if (open) {
-          nodes.push(React.createElement('ul', { key: 'archive', style: panelListStyle }, ...archived.map((entry) => lectureRow(entry, onOpen))));
+          rows.push(React.createElement('ul', { key: 'archive', style: panelListStyle }, ...archived.map((entry) => lectureRow(entry, onOpen))));
         }
       }
-      return React.createElement('section', { key: group.topic, style: panelSectionStyle }, ...nodes);
+      // The Course key remounts the section on a switch, so its scrollport starts at the top.
+      return React.createElement('section', { key: group.topic, style: lecturesSectionStyle },
+        React.createElement('h2', { key: 'heading', style: lecturesHeadingStyle }, group.topic),
+        React.createElement('div', { style: lecturesScrollStyle }, ...rows));
     }
 
     function LecturesPage({ useTabInfo, sessionId, workspaceFiles }) {
@@ -1680,22 +1701,21 @@ window.__ModuleLoader__.load({
         setSelected(course);
       };
 
-      const open = (entry) => tab.actions.openResource(
-        sessionFileAddress(sessionId, entry.path),
-        { preferNewPane: true },
-      );
+      // No placement option: the preview opens as a tab in this pane and the
+      // host focuses it when that lecture is already open.
+      const open = (entry) => tab.actions.openResource(sessionFileAddress(sessionId, entry.path));
 
       let nodes;
       if (content.status === 'loading') {
-        nodes = [React.createElement('p', { role: 'status', 'aria-busy': true, style: panelMutedStyle }, 'Loading the lectures…')];
+        nodes = [React.createElement('p', { role: 'status', 'aria-busy': true, style: lecturesMessageStyle }, 'Loading the lectures…')];
       } else if (content.error === null && !content.rootFound) {
-        nodes = [React.createElement('p', { style: panelMutedStyle }, `No Course in ${TRACKER_ROOT}/. Run /study-plus to create one.`)];
+        nodes = [React.createElement('p', { style: lecturesMessageStyle }, `No Course in ${TRACKER_ROOT}/. Run /study-plus to create one.`)];
       } else {
         nodes = content.error === null
           ? []
-          : [React.createElement('p', { role: 'alert', style: panelMutedStyle }, `The lecture library could not be read: ${content.error}.`)];
+          : [React.createElement('p', { role: 'alert', style: lecturesMessageStyle }, `The lecture library could not be read: ${content.error}.`)];
         const picker = topicPicker(content.courses, content.course ?? selected, choose);
-        if (picker !== null) nodes.push(React.createElement('div', { style: panelHeaderStyle }, picker));
+        if (picker !== null) nodes.push(React.createElement('div', { style: lecturesHeaderStyle }, picker));
         if (content.course !== null) {
           nodes.push(lectureTopic(
             { topic: content.course, entries: content.entries },
@@ -1708,7 +1728,7 @@ window.__ModuleLoader__.load({
       return React.createElement('div', {
         role: 'region',
         'aria-label': LECTURES_LABEL,
-        style: panelStyle,
+        style: lecturesPanelStyle,
       }, React.createElement('style', null, LECTURES_CSS), ...nodes);
     }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-nerd-mode",
-  "version": "0.4.5",
+  "version": "0.4.6",
   "description": "A study switch and study skills for DeepSeek Harness.",
   "license": "MIT",
   "type": "module",

package/skills/study/SKILL.md

@@ -1,6 +1,6 @@
 ---
 name: study
-description: Teach one concept or help complete one task, using the supplied material as scope and defaulting to chat output.
+description: "[v0.4.6] Teach one concept or help complete one task, using the supplied material as scope and defaulting to chat output."
 argument-hint: "<concept or task>"
 ---
 # Study

package/skills/study-plus/SKILL.md

@@ -1,6 +1,6 @@
 ---
 name: study-plus
-description: Run one tracked session against a dated learning record. Load for an explicit command or after the user chooses tracked study in the upper router.
+description: "[v0.4.6] Run one tracked session against a dated learning record. Load for an explicit command or after the user chooses tracked study in the upper router."
 argument-hint: "<course-slug>, or empty to continue the current Course"
 ---
 # Study Plus
@@ -52,15 +52,15 @@ For a new graph:
 2. Make a concept table in prerequisite order. Each row has a one-line definition and prerequisite numbers. Keep prerequisite order separate from textbook reading order.
 3. Use the source's book and chapter hierarchy when present. Otherwise, propose logical chapter groupings.
 4. Show the full graph and chapter layout in the reply before the approval question, as a Markdown table. Keep the question text short, because it renders as one paragraph: ask whether to approve, edit, or rebuild the graph shown above, and state that approval starts draft generation for every concept, including any concept the learner then skips, and that drafts are not teaching. Do not put the table in the question text. Do not add another approval step.
-5. After approval, run calibration as one `question` call with up to three items: what the learner wants to be able to do after this Course, what they have already read, studied, or used in this area in their own words, and which concepts in the approved graph they know well enough to skip, as a multi-select list. Items 1 and 2 are free text. Ask the goal and background items only when nothing already answers them: not the current request, not the opening question, not the supplied material. Reuse an opening-question answer verbatim. Derive the level. Then write `graph.md`, create `state.md` with every concept `new` and undated except the concepts selected to skip, which are `skipped`, create an empty `log.md`, and generate one draft per concept (section 3). Do not create `plan.md`.
+5. After approval, run calibration as one question-tool call with up to three items: what the learner wants to be able to do after this Course, what they have already read, studied, or used in this area in their own words, and which concepts in the approved graph they know well enough to skip, as a multi-select list. Items 1 and 2 are free text. Ask the goal and background items only when nothing already answers them: not the current request, not the opening question, not the supplied material. Reuse an opening-question answer verbatim. Derive the level. Then write `graph.md`, create `state.md` with every concept `new` and undated except the concepts selected to skip, which are `skipped`, create an empty `log.md`, and generate one draft per concept (section 3). Do not create `plan.md`.
 
 Calibration writes `Goal`, `Background`, and `Teaching level` to `graph.md`. Derive the level from the two answers:
 
-| Answers | Level |
-|---|---|
-| Most prerequisites are unknown, or the goal is to learn the topic from zero | `beginner` |
+| Answers                                                                                    | Level        |
+| ------------------------------------------------------------------------------------------ | ------------ |
+| Most prerequisites are unknown, or the goal is to learn the topic from zero                | `beginner` |
 | Some prerequisites are known, and the goal is to understand the principles or pass an exam | `standard` |
-| Most prerequisites are known, and the goal is review, reference, or use in a project | `advanced` |
+| Most prerequisites are known, and the goal is review, reference, or use in a project       | `advanced` |
 
 When the two answers clearly conflict, you may choose an adjacent level and state the reason in one line. When the learner declines or gives no usable answer, write `Teaching level: standard` and omit the `Goal` and `Background` lines. Each selected concept becomes `skipped`, with no date and no passes.
 
@@ -136,7 +136,7 @@ If the row was already `in_process`, say that the last session stopped before th
 
 **Ask unseen questions.** Use the explicit count or type preference in the current `/study-plus` request when present. Otherwise, use the profile defaults in the top-level Nerd Mode instructions. Plan the configured number of unseen question slots, from 0 to 10. A hint, re-ask, or similar question stays in its current slot and does not add a slot. For each slot, write down the expected answer and criteria for a good answer. Keep both private. A draft never contains them. The criteria test the conclusion and whether each step taken is valid, not the route you had in mind. Ask one question at a time, then wait. One call asks one question. Outside `study-plus`, ignore these preferences.
 
-For `short-answer`, ask the question in the conversation. For `multiple-choice`, ask it as one `question` call with four options and exactly one correct answer; choosing it is sufficient. Give the labels alone, with no description, and shuffle the four positions for each slot. A host without a `question` tool gets the same four options as a numbered text list. A typed answer that names or repeats one option counts as choosing it. Any other typed answer is graded like a short answer. For `mixed`, choose short-answer or multiple-choice independently at random for each planned slot. Do not add question-type details to the session log. When the configured count is zero, skip unseen questions. A passed Teach-back alone is sufficient to pass. With one or more slots, require a passed Teach-back and `ceil(2 × slot count / 3)` correct slots.
+For `short-answer`, ask the question in the conversation. For `multiple-choice`, ask it as one question-tool call with four options and exactly one correct answer; choosing it is sufficient. Give the labels alone, with no description, and shuffle the four positions for each slot. A host without a question tool gets the same four options as a numbered text list. A typed answer that names or repeats one option counts as choosing it. Any other typed answer is graded like a short answer. For `mixed`, choose short-answer or multiple-choice independently at random for each planned slot. Do not add question-type details to the session log. When the configured count is zero, skip unseen questions. A passed Teach-back alone is sufficient to pass. With one or more slots, require a passed Teach-back and `ceil(2 × slot count / 3)` correct slots.
 
 A question exists to find the gap, not to stop the session. Nothing in this step blocks the learner.
 

package/skills/study-plus/scripts/book-client.js

@@ -13,8 +13,8 @@
     if (!panels.length) return;
 
     var L = book.lang === 'en'
-      ? { search: 'Search the book', toc: 'Contents', none: 'No match', hits: ' hits', chapter: 'Chapter', section: 'Section', sections: 'sections', menu: 'Chapters', draft: 'Draft' }
-      : { search: '搜索全书', toc: '目录', none: '没有匹配', hits: ' 处', chapter: '第', section: '第', sections: '节', menu: '章节', draft: '草稿' };
+      ? { search: 'Search the book', toc: 'Contents', none: 'No match', hits: ' hits', chapter: 'Chapter', sections: 'sections', menu: 'Chapters', draft: 'Draft' }
+      : { search: '搜索全书', toc: '目录', none: '没有匹配', hits: ' 处', chapter: '第', sections: '节', menu: '章节', draft: '草稿' };
 
     var layout = document.querySelector('.am-doc-layout');
     var body = document.querySelector('.am-doc-body');
@@ -33,10 +33,8 @@
         : L.chapter + chapter.n + '章 ' + chapter.name;
     }
 
-    function sectionLabel(unit) {
-      return book.lang === 'en'
-        ? L.section + ' ' + unit.section
-        : L.section + unit.section + '节';
+    function sectionNumber(unit) {
+      return unit.chapter + '.' + unit.section;
     }
 
     function chapterCount(chapter) {
@@ -46,9 +44,7 @@
     }
 
     function sectionTitle(unit) {
-      return book.lang === 'en'
-        ? L.section + ' ' + unit.section + '. ' + unit.title
-        : L.section + unit.section + '节 ' + unit.title;
+      return sectionNumber(unit) + ' ' + unit.title;
     }
 
     var style = document.createElement('style');
@@ -70,6 +66,8 @@
       '.am-doc-body .am-md h3 { font-size: 18px; font-weight: 650; margin-top: 28px; }',
       '.am-doc-body .am-md h3:first-child { margin-top: 0; }',
       '.am-doc-body .am-md h4 { font-size: 16px; font-weight: 600; margin-top: 20px; }',
+      // am's overflow-x:auto makes overflow-y auto too; a 1px overshoot draws scrollbar arrows on Windows.
+      '.am-math-block { overflow-y: hidden; }',
       '.bk-context { display: flex; align-items: baseline; justify-content: space-between; gap: 16px; margin: 0 0 8px; font-size: 13px; }',
       '.bk-context-chapter { color: var(--ink-2); font-weight: 600; }',
       '.bk-context-section { color: var(--ink-3); white-space: nowrap; }',
@@ -151,7 +149,7 @@
         chapter.textContent = chapterTitle(g.chapter);
         var section = document.createElement('span');
         section.className = 'bk-context-section';
-        section.innerHTML = esc(sectionLabel(u)) + draftTag(u);
+        section.innerHTML = esc(sectionNumber(u)) + draftTag(u);
         context.appendChild(chapter);
         context.appendChild(section);
         var heading = panel.querySelector('.am-panel-head');

package/skills/study-plus/scripts/build-book.mjs

@@ -82,9 +82,13 @@ function build() {
     ...(book.goal ? [`subtitle: ${JSON.stringify(book.goal)}`] : []),
     '---',
     '',
-    book.units.map((unit) => `## ${unit.section}. ${unit.title}\n\n${unit.body}`).join('\n\n'),
+    book.units.map((unit) => `## ${unit.chapter}.${unit.section} ${unit.title}\n\n${unit.body}`).join('\n\n'),
     '',
-  ].join('\n');
+  ].join('\n')
+    // am reads $ and $$ math only: a LaTeX display delimiter on its own line
+    // would reach the page as a literal bracket, so turn it into a $$ block.
+    .replace(/^ {0,3}\\\[\s*$/gm, () => '$$')
+    .replace(/^ {0,3}\\\]\s*$/gm, () => '$$');
   const outPath = join(dir, 'book.html');
   const backupPath = join(dir, 'book.html.bak');
   const guardPath = (path, what) => {
@@ -123,7 +127,9 @@ function build() {
       units: book.units.map(({ concept, title, chapter, section, index, status }) => ({ concept, title, chapter, section, index, status })),
     };
     const json = JSON.stringify(payload).replace(/</g, '\\u003c');
-    const client = readFileSync(CLIENT, 'utf8');
+    // HTML parsing turns CRLF into LF before the browser checks the CSP hash,
+    // so a Windows checkout would otherwise be blocked as the wrong hash.
+    const client = readFileSync(CLIENT, 'utf8').replace(/\r\n?/g, '\n');
     const html = readFileSync(tmpPath, 'utf8');
     const scriptText = `\n${client}\n`;
     const hash = createHash('sha256').update(scriptText, 'utf8').digest('base64');