acmer-coach 2.6.0 → 2.6.2

package/cordis.patch.yml

@@ -1,10 +1,155 @@
 # acmer-coach 组合层（bundle 形态）。
 #
-# ⚠️ 两条路**不能并存**：这份 insert 和 profile 的 cordis.patch.yml 里再来
-# 一条 insert，插件就会被挂两次。profile 那边现在是空的（留了说明）。
-#
-# ⚠️ id 必须和 index.js 导出的 `name` 逐字相同，name 必须和**包名**相同 ——
-# 对不上不会报错，工具会**静默不注册**，界面上完全看不出区别。
+# 教练实例与小鲸预设由此统一声明，不在 profile 重复 insert 相同条目。
+# 已有个人 xiaojing 预设使用不同 id，可继续并存。
+# 教练实例保留 acmer-coach 的 id/name；原生预设使用独立稳定 id。
 - insert:
     - id: acmer-coach
       name: 'acmer-coach'
+
+# 原生 standard 预设配置快照（dsh 0.2.0-rc.2）；升级宿主时同步清单。
+- insert:
+    - id: preset-acmer-coach-xiaojing
+      name: '@deepseek-ai/dsh-agent-preset'
+      config:
+        id: acmer-coach-xiaojing
+        name: 小鲸教练
+        description: 鲸鱼娘算法教练，保留原生标准预设的工具、计划、压缩与委派能力。
+        order: 20
+        plugins:
+          - id: persona
+            name: '@deepseek-ai/dsh-persona'
+            config:
+              suffix: Your working directory is {{cwd}}.
+              prefix: 你是小鲸教练。默认角色风格遵循 coach:persona，教学、来源与证据遵循 coach:rules；优先尊重用户当前偏好和宿主自定义风格。
+          - id: agent-instructions
+            name: '@deepseek-ai/dsh-agent-instructions'
+            config:
+              maxBytes: 65536
+          - id: tool-bash
+            name: '@deepseek-ai/dsh-tool-bash'
+            disabled: !!js process.platform === 'win32'
+          - id: tool-pwsh
+            name: '@deepseek-ai/dsh-tool-pwsh'
+            disabled: !!js process.platform !== 'win32'
+          - id: tool-fs
+            name: '@deepseek-ai/dsh-tool-fs'
+          - id: tool-fs-search
+            name: '@deepseek-ai/dsh-tool-fs-search'
+            config:
+              sampleOverCapGlobResults: false
+          - id: tool-jobs
+            name: '@deepseek-ai/dsh-tool-jobs'
+          - id: skill-filesystem
+            name: '@deepseek-ai/dsh-skill-filesystem'
+          - id: tool-skill
+            name: '@deepseek-ai/dsh-tool-skill'
+          - id: command-goal
+            name: '@deepseek-ai/dsh-command-goal'
+          - id: tool-goal
+            name: '@deepseek-ai/dsh-tool-goal'
+          - id: planning
+            name: cordis:group
+            group: true
+            isolate:
+              planMode: true
+            config:
+              - id: plan-mode
+                name: '@deepseek-ai/dsh-plan-mode'
+                config:
+                  section: |
+                    You are in plan mode. Stay in plan mode until exit_plan_mode succeeds or the user switches the session mode. Imperative language to implement changes means plan the implementation, not execute it. A user's conversational agreement — including an answer confirming something you asked — approves nothing and does not end plan mode; fold the confirmed decision into the plan and submit it through exit_plan_mode.
+
+                    Explore first. Use non-mutating reads, searches, static analysis, and checks to ground the plan in the actual repository. Do not edit or write files, change configuration, run formatters or code generation that rewrites tracked files, commit, or otherwise carry out the plan. Prefer existing functions and patterns over new machinery.
+
+                    The tool catalog stays the same across modes for request-cache stability. These plan-mode rules override any later tool description or guidance that suggests using mutation tools; those tools remain listed to keep the tool catalog unchanged. Do not use todo_write to track this planning phase: it tracks implementation after an approved plan, while the plan itself belongs in exit_plan_mode.
+
+                    Resolve discoverable facts by inspection. Use ask_user_question only for user-owned choices or material ambiguity that inspection cannot answer. Do not ask the user where code lives or how current behavior works when you can find out.
+
+                    Make the plan decision-complete: state the goal and success criteria; group implementation changes by subsystem; identify public API, schema, and data-flow changes; cover edge cases, failure modes, tests, acceptance criteria, and explicit assumptions. Keep it concise enough to review but detailed enough that another engineer can implement it without making design decisions.
+
+                    When ready, call exit_plan_mode with the complete plan markdown, starting with a # title. Make exit_plan_mode the only and final tool call in that assistant response: it presents the plan for approval, and implementation begins only in a later step after approval. Do not paste the final plan as a plain reply or ask "should I proceed?" through prose or ask_user_question. If review rejects it, incorporate the feedback and present again. If the review channel is unavailable or aborted, stay in plan mode and ask the user to switch modes manually; do not proceed with implementation.
+          - id: compaction
+            name: cordis:group
+            group: true
+            isolate:
+              compaction: true
+              toolResultPruner: true
+            config:
+              - id: compaction-basic
+                name: '@deepseek-ai/dsh-compaction-basic'
+              - id: command-compact
+                name: '@deepseek-ai/dsh-command-compact'
+              - id: tool-result-pruner
+                name: '@deepseek-ai/dsh-compaction-tool-result-pruner'
+                config:
+                  thresholdChars: 8192
+                  headChars: 4096
+                  tailChars: 1024
+          - id: delegation
+            name: cordis:group
+            group: true
+            isolate:
+              workflowEngine: true
+            config:
+              - id: tool-subagent-control
+                name: '@deepseek-ai/dsh-tool-subagent-control'
+              - id: tool-subagent-list-agents
+                name: '@deepseek-ai/dsh-tool-subagent-control/list-agents'
+              - id: tool-subagent
+                name: '@deepseek-ai/dsh-tool-subagent'
+                config:
+                  provider: spawn
+                  toolName: subagent
+                  modelSelectionSettings: true
+                  backgroundMode: continuable
+              - id: tool-subagent-fork
+                name: '@deepseek-ai/dsh-tool-subagent'
+                config:
+                  provider: fork
+                  toolName: subagent_fork
+                  backgroundMode: continuable
+              - id: tool-subagent-codex
+                name: '@deepseek-ai/dsh-tool-subagent'
+                disabled: true
+                config:
+                  provider: codex
+                  toolName: subagent_codex
+                  backgroundMode: one-shot
+                  maxDepth: provider-managed
+              - id: tool-subagent-claude-code
+                name: '@deepseek-ai/dsh-tool-subagent'
+                disabled: true
+                config:
+                  provider: claude-code
+                  toolName: subagent_claude_code
+                  backgroundMode: one-shot
+                  maxDepth: provider-managed
+              - id: workflow-ptc
+                name: '@deepseek-ai/dsh-workflow-ptc'
+                config:
+                  provider: spawn
+              - id: tool-workflow
+                name: '@deepseek-ai/dsh-tool-workflow'
+              - id: tool-ralph
+                name: '@deepseek-ai/dsh-tool-ralph'
+                disabled: true
+                config:
+                  subagentProvider: spawn
+                  maxRounds: 64
+          - id: tool-ask-user
+            name: '@deepseek-ai/dsh-tool-ask-user'
+          - id: tool-todo
+            name: '@deepseek-ai/dsh-tool-todo'
+            config:
+              allowParallelInProgress: true
+          - id: tool-web
+            name: '@deepseek-ai/dsh-tool-web'
+            config:
+              fetch: true
+              searchTimeoutMs: 60000
+          - id: present
+            name: '@deepseek-ai/dsh-tool-present'
+          - id: tool-plugin-manager
+            name: '@deepseek-ai/dsh-plugin-manager/tools'
+            disabled: true

package/dev/catalog-check.mjs

@@ -19,7 +19,15 @@ function sources(value, label) {
     assert.ok(typeof source.title === 'string' && source.title.trim(), `${label}: source title required`)
     const url = new URL(source.url)
     assert.equal(url.protocol, 'https:', `${label}: public HTTPS source required`)
-    assert.ok(!url.username && !url.password && !url.search, `${label}: no account credentials or query data`)
+    assert.ok(!url.username && !url.password, `${label}: no account credentials`)
+    if (url.search) {
+      assert.equal(url.hostname, 'contest.ucup.ac', `${label}: only public official attachments may use query parameters`)
+      assert.equal(url.pathname, '/download.php')
+      assert.deepEqual([...url.searchParams.keys()].sort(), ['id', 'r', 'type'])
+      assert.equal(url.searchParams.get('type'), 'attachments')
+      assert.match(url.searchParams.get('id'), /^\d+$/)
+      assert.match(url.searchParams.get('r'), /^\d+$/)
+    }
     assert.ok(url.pathname !== '/', `${label}: link to evidence, not a homepage`)
   }
 }
@@ -35,27 +43,54 @@ test('contest evidence exposes actual reviewed samples, never a complete syllabu
     const store = createStrategyStore({ dataDir: dir, competenciesPath: fileURLToPath(new URL('COMPETENCIES.yaml', knowledge)) })
     const profiles = store.target('profiles').availableProfiles
     const regional = profiles.find(profile => profile.id === 'regional-diagnostic')
-    assert.equal(regional.coverage.sampledProblems, 19)
-    assert.equal(regional.coverage.reviewedSolutions, 19)
-    assert.deepEqual(regional.coverage.years, [2022, 2023])
+    assert.equal(regional.coverage.sampledProblems, 70)
+    assert.equal(regional.coverage.reviewedSolutions, 70)
+    assert.deepEqual(regional.coverage.years, [2022, 2023, 2024, 2025])
     assert.equal(regional.coverage.isSyllabus, false)
     assert.deepEqual(regional.samples.filter(sample => sample.year === 2022).map(sample => sample.problem).sort(), [...'ABCDEFGHIJKLM'])
-    assert.equal(new Set(regional.samples.map(sample => `${sample.contest}:${sample.year}:${sample.problem}`)).size, 19)
+    assert.equal(new Set(regional.samples.map(sample => `${sample.contest}:${sample.year}:${sample.problem}`)).size, 70)
+    const additions = [
+      ["Xi'an", 2025, 'ABCDEFGHIJKLM', 'contest.ucup.ac', '/download.php'],
+      ['Hangzhou', 2024, 'ABCDEFGHIJKLM', 'contest.ucup.ac', '/download.php'],
+      ['Shanghai', 2025, 'ABCDEFGHIJKLM', 'ucup.ac', '/tutorials/tutorials-4-12-zh.pdf'],
+      ['Hong Kong', 2025, 'ABCDEFGHIJKL', 'ucup.ac', '/tutorials/tutorials-4-14-en.pdf'],
+    ]
+    for (const [city, year, letters, host, pathname] of additions) {
+      const samples = regional.samples.filter(sample => sample.contest === city && sample.year === year)
+      assert.deepEqual(samples.map(sample => sample.problem).sort(), [...letters], `${city}: reviewed full paper`)
+      for (const sample of samples) {
+        const url = new URL(sample.sourceUrl)
+        assert.equal(url.hostname, host)
+        assert.equal(url.pathname, pathname)
+        assert.match(url.hash, /^#page=[1-9]\d*$/)
+        if (city === "Xi'an" || city === 'Hangzhou') {
+          assert.equal(url.searchParams.get('type'), 'attachments')
+          assert.equal(url.searchParams.get('id'), city === "Xi'an" ? '2562' : '1893')
+          assert.equal(url.searchParams.get('r'), '1')
+        }
+      }
+    }
     assert.match(regional.title, /中国|亚洲/)
     for (const sample of regional.samples) {
       assert.ok(sample.model && sample.solutionReviewed)
-      assert.match(sample.sourceUrl, sample.year === 2022
+      if (sample.contest === 'Nanjing') assert.match(sample.sourceUrl, sample.year === 2022
         ? /^https:\/\/sua\.ac\/wiki\/2022-icpc-nanjing\/tutorial-sketch-zh\.pdf#page=\d+$/
         : /^https:\/\/universal-cup-website\.qoj\.ac\/files\/ucup\/tutorials\/tutorials-2-11\.pdf#page=\d+$/)
       assert.ok(sample.competencyIds.every(id => competencies.some(card => card.id === id)))
     }
     const xian = profiles.find(profile => profile.id === 'xian-pending')
-    assert.equal(xian.coverage.sampledProblems, 0)
-    assert.equal(xian.coverage.reviewedSolutions, 0)
+    assert.equal(xian.coverage.sampledProblems, 13)
+    assert.equal(xian.coverage.reviewedSolutions, 13)
+    assert.deepEqual(xian.coverage.years, [2025])
+    assert.deepEqual(xian.samples, regional.samples.filter(sample => sample.contest === "Xi'an"))
     assert.equal(xian.coverage.isSyllabus, false)
     assert.equal(store.target('set', { expectedRevision: 0, contest: '区域赛', date: '2026-10-17', result: '稳定', profileId: regional.id, weeklyHours: 25,
       priorities: [{ id: 'graph-modeling', weight: 10 }] }).ok, true)
-    assert.equal(store.focus().profile.coverage.sampledProblems, 19)
+    assert.equal(store.focus().profile.coverage.sampledProblems, 70)
+    assert.equal(store.target('set', { expectedRevision: 1, contest: '西安区域赛', date: '2026-10-17', result: '稳定', profileId: 'xian-pending', weeklyHours: 25,
+      priorities: [{ id: 'graph-modeling', weight: 10 }] }).ok, true)
+    assert.equal(store.focus().profile.coverage.reviewedSolutions, 13)
+    assert.equal(store.focus().profile.id, 'xian-pending')
     assert.equal(profiles.find(profile => profile.id === 'nwerc-reference').coverage.reviewedSolutions, 13)
   } finally { rmSync(dir, { recursive: true, force: true }) }
 })

package/dev/persona-check.mjs

@@ -0,0 +1,236 @@
+import test from 'node:test'
+import assert from 'node:assert/strict'
+import fs from 'node:fs'
+import { syncBuiltinESMExports } from 'node:module'
+import { tmpdir } from 'node:os'
+import { dirname, join } from 'node:path'
+import { fileURLToPath } from 'node:url'
+import { parse } from 'yaml'
+
+const personaPath = new URL('../assets/persona/xiaojing-coach.md', import.meta.url)
+const rulesPath = new URL('../assets/rules/coach-rules.md', import.meta.url)
+let imports = 0
+
+async function fixture(t) {
+  const dir = fs.mkdtempSync(join(tmpdir(), 'coach-persona-'))
+  const previous = Object.fromEntries(['COACH_DATA_DIR', 'COACH_DB', 'COACH_RULES_DIR'].map(key => [key, process.env[key]]))
+  process.env.COACH_DATA_DIR = dir
+  process.env.COACH_DB = join(dir, 'unused.db')
+  delete process.env.COACH_RULES_DIR
+  t.after(() => {
+    for (const [key, value] of Object.entries(previous)) {
+      if (value === undefined) delete process.env[key]
+      else process.env[key] = value
+    }
+    fs.rmSync(dir, { recursive: true, force: true })
+  })
+  const { apply } = await import(`../index.js?persona-check=${++imports}`)
+  const tools = new Map(), hooks = new Map(), sections = []
+  const ctx = {
+    tools: { register: tool => tools.set(tool.name, tool) },
+    get: name => name === 'systemPrompt' ? { section: section => sections.push(section) } : undefined,
+    on: (name, handler) => hooks.set(name, handler),
+    inject() {},
+  }
+  return { dir, apply, ctx, tools, hooks, sections }
+}
+
+function denyReads(t, denied) {
+  const read = fs.readFileSync
+  const mocked = t.mock.method(fs, 'readFileSync', (path, ...args) => {
+    if (denied(String(path).replaceAll('\\', '/'))) throw Object.assign(new Error('fixture missing file'), { code: 'ENOENT' })
+    return read(path, ...args)
+  })
+  syncBuiltinESMExports()
+  return () => { mocked.mock.restore(); syncBuiltinESMExports() }
+}
+
+function assertTools(f) {
+  assert.equal(f.tools.size, 31)
+  assert.ok(f.tools.has('coach_status'))
+  assert.ok(f.tools.has('coach_note'))
+}
+
+test('apply loads packaged static persona before custom style without local personas', async t => {
+  const f = await fixture(t)
+  const restore = denyReads(t, path => path.includes('/personas/'))
+  try { f.apply(f.ctx) } finally { restore() }
+  const persona = f.sections.find(section => section.name === 'coach:persona')
+  assert.ok(persona, 'packaged persona must load through the public apply entry')
+  assert.equal(persona.text, fs.readFileSync(personaPath, 'utf8'))
+  assert.equal(persona.order, -100)
+  const rules = f.sections.find(section => section.name === 'coach:rules')
+  assert.equal(rules.text, fs.readFileSync(rulesPath, 'utf8'))
+  assert.equal(rules.order, 400)
+  const custom = { name: 'deployment:persona-prefix', order: 0, text: '自定义风格' }
+  assert.deepEqual([...f.sections, custom].sort((a, b) => a.order - b.order).map(section => section.name),
+    ['coach:persona', 'deployment:persona-prefix', 'coach:rules'])
+  assertTools(f)
+})
+
+test('missing packaged persona leaves rules and tools available', async t => {
+  const f = await fixture(t)
+  const restore = denyReads(t, path => path.endsWith('/assets/persona/xiaojing-coach.md'))
+  try { f.apply(f.ctx) } finally { restore() }
+  assert.deepEqual(f.sections.map(section => section.name), ['coach:rules'])
+  assert.equal(f.sections[0].text, fs.readFileSync(rulesPath, 'utf8'))
+  assertTools(f)
+})
+
+test('rejected persona section leaves rules and tools available', async t => {
+  const f = await fixture(t)
+  f.ctx.get = () => ({ section(section) {
+    if (section.name === 'coach:persona') throw new Error('host rejects persona section')
+    f.sections.push(section)
+  } })
+  f.apply(f.ctx)
+  assert.deepEqual(f.sections.map(section => section.name), ['coach:rules'])
+  assert.equal(f.sections[0].text, fs.readFileSync(rulesPath, 'utf8'))
+  assertTools(f)
+})
+
+test('missing rules leaves packaged persona and tools available', async t => {
+  const f = await fixture(t)
+  const restore = denyReads(t, path => path.endsWith('/assets/rules/coach-rules.md'))
+  try { f.apply(f.ctx) } finally { restore() }
+  assert.deepEqual(f.sections.map(section => section.name), ['coach:persona'])
+  assert.equal(f.sections[0].text, fs.readFileSync(personaPath, 'utf8'))
+  assertTools(f)
+})
+
+test('rejected rules section leaves packaged persona and tools available', async t => {
+  const f = await fixture(t)
+  f.ctx.get = () => ({ section(section) {
+    if (section.name === 'coach:rules') throw new Error('host rejects rules section')
+    f.sections.push(section)
+  } })
+  f.apply(f.ctx)
+  assert.deepEqual(f.sections.map(section => section.name), ['coach:persona'])
+  assert.equal(f.sections[0].text, fs.readFileSync(personaPath, 'utf8'))
+  assertTools(f)
+})
+
+for (const unavailable of ['missing service', 'missing get', 'throwing get']) {
+  test(`all 31 tools load with ${unavailable}`, async t => {
+    const f = await fixture(t)
+    if (unavailable === 'missing get') delete f.ctx.get
+    else f.ctx.get = () => {
+      if (unavailable === 'throwing get') throw new Error('service unavailable')
+      return undefined
+    }
+    assert.doesNotThrow(() => f.apply(f.ctx))
+    assert.deepEqual(f.sections, [])
+    assertTools(f)
+    const result = await f.tools.get('coach_target').execute({ action: 'profiles' })
+    assert.equal(result.ok, true)
+    assert.ok(result.availableProfiles.length)
+  })
+}
+
+test('persona is read once into static sections, never messages or training ledgers', async t => {
+  const f = await fixture(t)
+  fs.writeFileSync(join(f.dir, 'MAP.yaml'), 'meta: {version: 1}\nnodes: [{id: fixture-node, name: 测试节点}]\n')
+  fs.writeFileSync(join(f.dir, 'PROGRESS.yaml'), 'version: 1\ncursor: fixture-node\nnodes: {}\npending: null\n')
+  const snapshot = () => fs.readdirSync(f.dir).sort().map(name => [name, fs.readFileSync(join(f.dir, name), 'utf8')])
+  const before = snapshot()
+  const reads = []
+  const restore = denyReads(t, path => {
+    if (path.endsWith('/assets/persona/xiaojing-coach.md') || path.endsWith('/assets/rules/coach-rules.md')) reads.push(path)
+    return false
+  })
+  try {
+    f.apply(f.ctx)
+    assert.equal(f.sections.length, 2)
+    assert.ok(f.sections.every(section => typeof section.text === 'string'))
+    const initial = structuredClone(f.sections)
+    const hook = f.hooks.get('agent/pre-step')
+    for (const step of [1, 2, 1]) {
+      const messages = [{ role: 'user', content: [{ type: 'text', text: '继续' }] }]
+      const result = await hook({ messages, step }, async () => ({ kind: 'enter', messages }))
+      assert.deepEqual(result.messages[0], messages[0])
+      for (const message of result.messages.slice(1)) {
+        assert.ok(message.content.every(block => block.type === 'text' && block.text.startsWith('<coach_state>\n')))
+      }
+      assert.ok(!JSON.stringify(result.messages).includes('鲸鱼娘'))
+    }
+    assert.deepEqual(f.sections, initial)
+    assert.equal(reads.filter(path => path.endsWith('/xiaojing-coach.md')).length, 1)
+    assert.equal(reads.filter(path => path.endsWith('/coach-rules.md')).length, 1)
+  } finally { restore() }
+  assert.deepEqual(snapshot(), before)
+  assertTools(f)
+})
+
+test('bundle declares a native 小鲸教练 preset with the stock standard capability composition', async () => {
+  const patches = parse(fs.readFileSync(new URL('../cordis.patch.yml', import.meta.url), 'utf8'), {
+    customTags: [{ tag: 'tag:yaml.org,2002:js', resolve: value => value }],
+  })
+  const entries = patches.flatMap(patch => patch.insert ?? [])
+  const preset = entries.find(entry => entry.config?.id === 'acmer-coach-xiaojing')
+  assert.ok(preset, 'installing the bundle must declare the selectable native 小鲸教练 preset')
+  assert.equal(preset.id, 'preset-acmer-coach-xiaojing')
+  assert.equal(preset.name, '@deepseek-ai/dsh-agent-preset')
+  const config = preset.config
+  assert.equal(config.name, '小鲸教练')
+  assert.equal(entries.filter(entry => entry.name === 'acmer-coach').length, 1)
+  assert.ok(!entries.some(entry => entry.id === 'agent-preset-registry'), 'bundle must not change the selected default')
+  const persona = config.plugins.find(entry => entry.id === 'persona')
+  assert.equal(persona.name, '@deepseek-ai/dsh-persona')
+  assert.ok(persona.config.prefix.includes('coach:persona'))
+  assert.ok(persona.config.prefix.includes('coach:rules'))
+  assert.ok(persona.config.prefix.length < 200, 'reuse packaged persona without duplicating its full text')
+  assert.equal(persona.config.suffix, 'Your working directory is {{cwd}}.')
+  const ids = config.plugins.map(entry => entry.id)
+  for (const id of ['tool-bash', 'tool-pwsh', 'tool-fs', 'tool-fs-search', 'tool-jobs', 'skill-filesystem', 'tool-skill',
+    'planning', 'compaction', 'delegation', 'tool-ask-user', 'tool-todo', 'tool-web', 'present']) assert.ok(ids.includes(id), id)
+  const delegation = config.plugins.find(entry => entry.id === 'delegation')
+  for (const id of ['tool-subagent-codex', 'tool-subagent-claude-code', 'tool-ralph']) {
+    assert.equal(delegation.config.find(entry => entry.id === id).disabled, true, `${id} stays disabled`)
+  }
+  assert.equal(config.plugins.find(entry => entry.id === 'tool-bash').disabled, "process.platform === 'win32'")
+  assert.equal(config.plugins.find(entry => entry.id === 'tool-pwsh').disabled, "process.platform !== 'win32'")
+  assert.equal(config.plugins.find(entry => entry.id === 'tool-plugin-manager').disabled, true)
+  assert.ok(!JSON.stringify(config).match(/[A-Z]:[\\/]|accountId|userId|handle|rating/), 'preset must contain no private profile paths or account data')
+})
+
+test('packaged preset passes the installed native loader and configuration schemas', async t => {
+  let boot, presets, registry, loader
+  try {
+    [boot, presets, registry, loader] = await Promise.all([
+      import('@deepseek-ai/dsh-app-boot'), import('@deepseek-ai/dsh-agent-preset'),
+      import('@deepseek-ai/dsh-agent-preset-registry'), import('@deepseek-ai/cordis-plugin-loader'),
+    ])
+  } catch (error) {
+    if (error.code !== 'ERR_MODULE_NOT_FOUND') throw error
+    t.skip('native preset packages are supplied by the dsh host; portable bundle checks still run')
+    return
+  }
+  const entries = boot.composeEntries([boot.loadOverlayPatches('dsh', fileURLToPath(new URL('../cordis.patch.yml', import.meta.url)))])
+  const row = entries.find(entry => entry.config?.id === 'acmer-coach-xiaojing')
+  assert.ok(row)
+  const config = presets.default.Config(row.config)
+  assert.equal(registry.entryListProblem(config.plugins), undefined)
+  const enabledShell = config.plugins.filter(entry => ['tool-bash', 'tool-pwsh'].includes(entry.id))
+    .filter(entry => !loader.interpolate({}, entry.disabled)).map(entry => entry.id)
+  assert.deepEqual(enabledShell, [process.platform === 'win32' ? 'tool-pwsh' : 'tool-bash'])
+  async function validate(rows) {
+    for (const entry of rows) {
+      if (entry.group) { await validate(entry.config); continue }
+      if (loader.interpolate({}, entry.disabled)) continue
+      const plugin = await import(entry.name)
+      const schema = plugin.Config ?? plugin.default?.Config
+      if (schema) assert.doesNotThrow(() => schema(entry.config ?? {}), entry.name)
+      if (plugin.resolveConfig) assert.doesNotThrow(() => plugin.resolveConfig(entry.config ?? {}), entry.name)
+    }
+  }
+  await validate(config.plugins)
+  let standardPath
+  try { standardPath = join(dirname(fileURLToPath(import.meta.resolve('@deepseek-ai/dsh-web-app/package.json'))), 'presets', 'standard.patch.yml') }
+  catch (error) { if (error.code !== 'ERR_MODULE_NOT_FOUND') throw error }
+  if (standardPath && fs.existsSync(standardPath)) {
+    const stock = boot.composeEntries([boot.loadOverlayPatches('dsh', standardPath)]).find(entry => entry.config?.id === 'standard')
+    const actual = structuredClone(config.plugins)
+    actual.find(entry => entry.id === 'persona').config.prefix = stock.config.plugins.find(entry => entry.id === 'persona').config.prefix
+    assert.deepEqual(actual, stock.config.plugins, 'stock capability snapshot differs only in the persona prefix')
+  }
+})

package/dev/run-all.sh

@@ -10,6 +10,7 @@
 #   curriculum-evidence     编排证据目录、引用与事件身份
 #   strategy-check          比赛目标、VP 证据、复盘与重点选择
 #   teaching                续课笔记、学生原话与进度保留
+#   persona                 随包人设与静态提示词注入
 #   parity-check            和线上那份的保真度对比
 #
 # 主验收跑的是**已安装副本**（不是源码）—— 那才是运行时真正加载的东西。
@@ -69,6 +70,7 @@ run "strategy"      node --test dev/strategy-check.mjs
 run "catalog"       node --test dev/catalog-check.mjs
 run "coach-contract" node --test dev/coaching-contract-check.mjs
 run "teaching"      node --test dev/teaching-check.mjs
+run "persona"       node --test dev/persona-check.mjs
 run "grading"       node --test dev/grading-check.mjs
 run "training"      node --test dev/training-check.mjs
 run "VP-client"     node dev/vp-client-check.mjs

package/dev.sh

@@ -35,7 +35,7 @@ done
 # 从副本目录跑 `bash dev.sh` 走的是另一条路（`dsh plugin remove/add`，而那条
 # 路不可靠），和这份的行为不一样。换句话说，**同一个命令，从哪个目录按回车
 # 决定跑哪份脚本** —— 两份 dev.sh 就是两份真相，正是这份文件在防的那种坑。
-SYNC_FILES='index.js package.json verify.mjs verify-shape.mjs progress-cli.mjs build-node-meta.mjs cordis.patch.yml README.md lib/client.js lib/vp-client.js lib/workspace-migration.js lib/setup.js lib/curriculum.js lib/curriculum-evidence.js lib/curriculum-planning.js lib/strategy.js lib/teaching.js assets/knowledge/COMPETENCIES.yaml assets/knowledge/NODE_SCOPES.yaml assets/knowledge/CONTEST_PROFILES.yaml assets/knowledge/vp.html assets/knowledge/skilltree.html assets/rules/coach-rules.md dev/curriculum-check.mjs dev/curriculum-evidence-check.mjs dev/strategy-check.mjs dev/workspace-migration-check.mjs dev/vp-client-check.mjs dev/vp-sync-check.mjs dev/catalog-check.mjs dev/coaching-contract-check.mjs dev/teaching-check.mjs dev/grading-check.mjs dev/training-check.mjs coach/skilltree_template.html dev/run-all.sh docs/长期课程机制.md docs/比赛与教学依据.md docs/设计方针.md dev.sh'
+SYNC_FILES='index.js package.json verify.mjs verify-shape.mjs progress-cli.mjs build-node-meta.mjs cordis.patch.yml README.md lib/client.js lib/vp-client.js lib/workspace-migration.js lib/setup.js lib/curriculum.js lib/curriculum-evidence.js lib/curriculum-planning.js lib/strategy.js lib/teaching.js assets/knowledge/COMPETENCIES.yaml assets/knowledge/NODE_SCOPES.yaml assets/knowledge/CONTEST_PROFILES.yaml assets/knowledge/vp.html assets/knowledge/skilltree.html assets/rules/coach-rules.md assets/persona/xiaojing-coach.md dev/persona-check.mjs dev/curriculum-check.mjs dev/curriculum-evidence-check.mjs dev/strategy-check.mjs dev/workspace-migration-check.mjs dev/vp-client-check.mjs dev/vp-sync-check.mjs dev/catalog-check.mjs dev/coaching-contract-check.mjs dev/teaching-check.mjs dev/grading-check.mjs dev/training-check.mjs coach/skilltree_template.html dev/run-all.sh docs/长期课程机制.md docs/比赛与教学依据.md docs/设计方针.md docs/china-contest-sources.md dev.sh'
 STALE=''
 for f in $SYNC_FILES; do
   for d in "${DSTS[@]}"; do
@@ -120,7 +120,7 @@ EVIDENCE_RC=${EVIDENCE_RC:-0}
 
 echo
 echo '── 策略、教学目录、真实工具与账号数据回归 ──'
-node --test "$SRC_BASH/dev/strategy-check.mjs" "$SRC_BASH/dev/catalog-check.mjs" "$SRC_BASH/dev/coaching-contract-check.mjs" "$SRC_BASH/dev/teaching-check.mjs" "$SRC_BASH/dev/grading-check.mjs" "$SRC_BASH/dev/training-check.mjs" "$SRC_BASH/dev/workspace-migration-check.mjs" "$SRC_BASH/dev/vp-client-check.mjs" "$SRC_BASH/dev/vp-sync-check.mjs" || CONTRACT_RC=$?
+node --test "$SRC_BASH/dev/strategy-check.mjs" "$SRC_BASH/dev/catalog-check.mjs" "$SRC_BASH/dev/coaching-contract-check.mjs" "$SRC_BASH/dev/teaching-check.mjs" "$SRC_BASH/dev/persona-check.mjs" "$SRC_BASH/dev/grading-check.mjs" "$SRC_BASH/dev/training-check.mjs" "$SRC_BASH/dev/workspace-migration-check.mjs" "$SRC_BASH/dev/vp-client-check.mjs" "$SRC_BASH/dev/vp-sync-check.mjs" || CONTRACT_RC=$?
 CONTRACT_RC=${CONTRACT_RC:-0}
 
 LEDGER_AFTER=$(ledger_fp)